Skip to content

chore(openspec): rebind runtime-command-authority row to the landed #506 squash - #514

Merged
monkey1sai merged 1 commit into
mainfrom
chore/rebind-runtime-auth-row-506-squash
Aug 12, 2026
Merged

chore(openspec): rebind runtime-command-authority row to the landed #506 squash#514
monkey1sai merged 1 commit into
mainfrom
chore/rebind-runtime-auth-row-506-squash

Conversation

@monkey1sai

Copy link
Copy Markdown
Owner

Summary

AI Coding Governance

Machine values: Change lane = F / B / G / S; Behavior contract changed = yes / no; Requirement source = issue / docs/plans / superpowers spec / existing contract / not applicable.

Item Result
Change lane F
Behavior contract changed no
Linked issue none(#506 merge 的後續 rebind)
Requirement source existing contract(openspec lifecycle machine-truth 的 subject ancestry 規則;#482/#501/#512 先例)
CODEOWNERS / owner review requested
GitNexus evidence not needed
Browser E2E evidence not user-facing
Agent workflow changed? no
Required checks expected CI / Agent Governance / PR Metadata Contract

Frontend Verification

User-facing changes must pass two independent producers: real frontend/runtime operability evidence and the pinned docs/plans/design-system-reference.manifest.json fidelity gate. Scope is derived from changed paths plus the base/head manifest union; the PR body cannot select an easier screen. mixed and partial_reference_missing permit honest partial work but require Full completion claimed = no. Semantic evidence is produced only by the design-semantic-visual CI Playwright job, never supplied as PR input; PR Metadata Contract validates the live PR metadata, while normal protected CI checks determine mergeability.

Item Result
Frontend route not applicable
Main button(s) tested not applicable
Fixture used not applicable
Backend API called not applicable
Runtime action not applicable
Visible success state not applicable
E2E command not applicable
Screenshot / trace not applicable
Design gate status not applicable
Design screen(s) not applicable
Reference-missing route(s) / surface(s) not applicable
Full completion claimed no
Design reference manifest docs/plans/design-system-reference.manifest.json
Visual fidelity result not applicable
Visual comparison not applicable
Visual artifacts not applicable
Manual test steps not applicable
Known gaps not applicable

Deploy Path Verification

Required for runtime / Docker / Kit / viewer / ports / env / conversion-service changes.

Item Result
Affects runtime / docker / Kit / viewer / ports / env? no
Canonical deploy path updated? not needed
New root script added? no
Deploy dry-run command not applicable
Full deploy tested not available
Verify command not applicable
Frontend URL verified not applicable
Evidence path openspec/lifecycle-ledger.json(單行 diff)

Self-Referential Bootstrap

Required when the PR changes the verification mechanism itself (deploy path / evidence harness / gate script). Rule: docs/agents/self-referential-bootstrap.md. Open ledger debt in scripts/self-referential-bootstrap-ledger.json blocks further mechanism PRs until fixpoint closure.

Item Result
Self-referential bootstrap no
Bootstrap ledger entry not applicable
Bootstrap reason not applicable

Validation

  • node --test scripts/tests/test-openspec-machine-truth.mjs → 24 pass/0 fail。
  • node --test scripts/tests/test-ai-coding-metrics.mjs → 13 pass/0 fail。
  • git diff --check → clean。

Known Risks

  • 無 runtime 影響(ledger 單行 rebind)。

🤖 Generated with Claude Code

 squash

Same pattern as #482/#501/#512: the row's subject_commit pointed at the pre-squash branch commit (1229171) which is not an ancestor of main after the #506 squash (cba6265) and whose branch was auto-deleted; any PR CI running the machine-truth source observation against this ledger fails subject_unavailable. Rebound to the landed squash. Local gates: machine-truth 24/0, metrics 13/0.

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings August 12, 2026 04:49
@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Warning

Review limit reached

@monkey1sai, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 17 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 556e5cb0-2f2d-44e8-af5d-9f9af2a7953f

📥 Commits

Reviewing files that changed from the base of the PR and between cba6265 and 744a0a2.

📒 Files selected for processing (1)
  • openspec/lifecycle-ledger.json

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR rebinds the subject_commit for the implement-runtime-command-authority-and-rejection lifecycle row in openspec/lifecycle-ledger.json from the pre-squash branch commit (1229171…, discarded by the #506 squash merge and no longer a main ancestor) to the landed squash commit (cba6265…). This restores the OpenSpec machine-truth source observation so that PR CI (agent-governance) no longer fails with subject_not_ancestor / subject_unavailable. It follows the identical remediation pattern established in #482, #501, and #512.

Changes:

  • Repoint one ledger row's subject_commit to the permanently reachable landed #506 squash SHA.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@monkey1sai
monkey1sai enabled auto-merge (squash) August 12, 2026 05:26

@monkey1sai-blip monkey1sai-blip left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved by monkey1sai-blip (the reviewer account pinned by the repo's merge governance).

Submitted through scripts/blip_review.py — a scripted approval carrying the operator's authority, pinned to head 744a0a28cba94fbe6d56cd5064305daa83d5c79a. This is the mechanism the GitHub App cannot satisfy: an App's approving review does not count toward required_approving_review_count.

@monkey1sai
monkey1sai merged commit ae547ab into main Aug 12, 2026
21 checks passed
@monkey1sai
monkey1sai deleted the chore/rebind-runtime-auth-row-506-squash branch August 12, 2026 05:28
monkey1sai added a commit that referenced this pull request Aug 12, 2026
monkey1sai added a commit that referenced this pull request Aug 12, 2026
…on carve-out (#519)

* feat(governance): admit @claude mention workflow via literal permission carve-out (#518)

Get-WorkflowPermissionViolations gains a path-scoped carve-out: only
.github/workflows/claude.yml may carry the literal read set plus
'id-token: write' (OIDC -> Claude GitHub App token exchange). GITHUB_TOKEN
stays read-only everywhere; positive and negative self-tests pin the
carve-out to that exact path and entry set.

Replaces installer PR #515: the auto-review workflow is intentionally
dropped (overlaps CodeRabbit/Copilot/local tri-adversarial bot).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01922NtPuNyhxkB8RQ3hJu76

* chore: refresh merge ref to pick up the #514 ledger rebind on main

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01922NtPuNyhxkB8RQ3hJu76

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
monkey1sai added a commit that referenced this pull request Aug 12, 2026
Same pattern as #482/#501/#512/#514: subject_commit pointed at the
pre-squash branch commit (af60c29) which is not an ancestor of main after
the #507 squash (4187102); PR CIs fail subject_not_ancestor until rebound.
Local gates: machine-truth 24/0, metrics 13/0.

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
monkey1sai added a commit that referenced this pull request Aug 12, 2026
…quash (#528)

#511 squash-merge 後 gpu-session-baseline-and-idle-reclaim row 仍綁 pre-squash
commit 405e2b6(已被 squash 丟棄)→ machine-truth test 25 對所有後續 PR 假紅。
依 #482/#501/#512/#514/#524 慣例 rebind 至 landed squash 797c5c8。

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants