chore(openspec): archive introduce-executable-architecture-contracts - #480
Conversation
All 26 tasks across five phases delivered and merged (#459-era proposal through #479). The change moves to openspec/changes/archive/2026-08-10-introduce-executable-architecture-contracts, its ten requirements land in openspec/specs/executable-architecture-contracts, and the ledger row is archived with subject_commit rebound to the #479 squash (d0125cc, permanently reachable main history - the #477 convention). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015QTVFY89rS2xRwRB2TpFP6
|
Warning Review limit reached
Next review available in: 20 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 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 configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (6)
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. Comment |
monkey1sai-blip
left a comment
monkey1sai-blip
left a comment
There was a problem hiding this comment.
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 50a8b8cb974c8d92f406c7e482c6986d057b3baf. This is the mechanism the GitHub App cannot satisfy: an App's approving review does not count toward required_approving_review_count.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 50a8b8cb97
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
Pull request overview
This PR performs the OpenSpec archive closeout for the completed change introduce-executable-architecture-contracts (all 26 tasks across five phases delivered via #475/#478/#479). It moves the change artifacts into the dated archive directory, lands the canonical capability spec, and flips the lifecycle-ledger row to archived with subject_commit rebound to the #479 squash d0125cc. It fits the repo's machine-truth governance model where openspec/lifecycle-ledger.json is the single source of truth for lifecycle/task/evidence state.
Changes:
- Flip the lifecycle-ledger row to
archived(updatingcurrent_slice,last_verified,subject_commit). - Add the archived change artifacts (
proposal.md,design.md,tasks.md, spec delta) underopenspec/changes/archive/2026-08-10-…/. - Publish the canonical capability spec at
openspec/specs/executable-architecture-contracts/spec.md.
Reviewed changes
Copilot reviewed 2 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
openspec/lifecycle-ledger.json |
Flips the row to archived; but evidence_refs were left pointing at the pre-archive path (bug), and the NOW projection was not updated. |
openspec/specs/executable-architecture-contracts/spec.md |
New canonical capability spec (10 requirements); standard archive boilerplate Purpose placeholder. |
openspec/changes/archive/2026-08-10-…/tasks.md |
Archived task ledger; all 26 tasks checked, matching task_ledger 26/26. |
openspec/changes/archive/2026-08-10-…/specs/executable-architecture-contracts/spec.md |
Archived ## ADDED Requirements spec delta (faithful copy). |
openspec/changes/archive/2026-08-10-…/proposal.md |
Archived proposal (faithful copy). |
openspec/changes/archive/2026-08-10-…/design.md |
Archived design doc including phase-by-phase rollout notes (faithful copy). |
Key findings:
- Bug (evidence_refs): The archived row's
evidence_refsstill referenceopenspec/changes/introduce-executable-architecture-contracts/…, which no longer exists after archiving. Every other archived row (e.g. the adjacentintroduce-host-native-conversion-authority-service) uses the datedopenspec/changes/archive/…path, and the machine-truth verifier requires the archive-path evidence to exist and be Git-tracked. - Consistency (NOW projection):
docs/plans/NOW.mdstill lists this change asactive, contradicting the now-archivedledger row; the documented archive workflow (openspec/AGENTS.md) requires the NOW current-scope projection to be updated in the same change.
Suppressed comments (1)
openspec/lifecycle-ledger.json:1393
- This row is now
archived, but itsevidence_refsstill point to the pre-archive locationopenspec/changes/introduce-executable-architecture-contracts/…. Afteropenspec archivemoved the change intoopenspec/changes/archive/2026-08-10-introduce-executable-architecture-contracts/, those source paths no longer exist, so the references are dangling. Every other archived row references the archive path — see the adjacentintroduce-host-native-conversion-authority-serviceentry, whoseevidence_refsuseopenspec/changes/archive/2026-05-20-…/proposal.mdand…/tasks.md. The machine-truth verifier also requires archived rows to reference the dated archive-pathproposal.md/tasks.mdand for those files to exist and be Git-tracked (required_evidence_missing/evidence_target_missing, and anartifact_untrustedfailure for an untracked path). Update both refs to the archive location.
"openspec/changes/introduce-executable-architecture-contracts/proposal.md",
"openspec/changes/introduce-executable-architecture-contracts/tasks.md"
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
| { | ||
| "id": "introduce-executable-architecture-contracts", | ||
| "status": "active", | ||
| "status": "archived", |
…, projection) (#481) * chore(openspec): archive closeout follow-up for executable-architecture-contracts Adopts the #480 review findings: the canonical spec prose is now Traditional Chinese per openspec/AGENTS.md (parser headings and SHALL protocol identifiers preserved), the ledger row's evidence_refs point at the dated archive directory, subject_commit rebinds to the landed archive snapshot (#480 squash c41e29d - sources unchanged since), and the NOW projection drops the archived row per the ledger-first rule. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015QTVFY89rS2xRwRB2TpFP6 * chore(openspec): re-reconcile the isolated-branch-stack row after its NOW.md evidence moved docs/plans/NOW.md is that row's evidence file; dropping the archived projection row changed it, so the row rebinds to the commit carrying the change (post-merge it rebinds to the squash per the #477 convention). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015QTVFY89rS2xRwRB2TpFP6 --------- Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
What
Archives
introduce-executable-architecture-contracts: all 26 tasks across five phases are delivered and merged (#475 Phase 4, #478 extractor holes, #479 Phase 5, with #476/#477 alongside). The change moves toopenspec/changes/archive/2026-08-10-..., its ten requirements land inopenspec/specs/executable-architecture-contracts/(openspec archive output), and the ledger row is archived withsubject_commitrebound to the #479 squashd0125ccper the #477 convention.Validation
npx openspec validate --all --strict- 71 passed, 0 failednode scripts/tests/test-openspec-machine-truth.mjs- 24/24 (archived row, subject d0125cc reachable)python -m pytest tests -q -p no:cacheprovider- 483 passed, 9 skippedgit diff --check- clean🤖 Generated with Claude Code
https://claude.ai/code/session_015QTVFY89rS2xRwRB2TpFP6