docs(archive): document capability retirement workflow - #1751
Conversation
Deploying openspec-docs with
|
| Latest commit: |
6260a94
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://bf1b4c81.openspec-docs.pages.dev |
| Branch Preview URL: | https://codex-fix-1750-retire-capabi.openspec-docs.pages.dev |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 8 remain after this review. 📝 WalkthroughWalkthroughThe CLI documentation now describes how to retire a capability during archive. It covers the ChangesCapability retirement
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This documentation-only change clarifies the capability retirement workflow without changing product behavior. No actionable merge-blocking risk remains after normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Linked Issues checkExplanation The documentation explains the first-class retirement workflow requested by issue Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
alfred-openspec
left a comment
There was a problem hiding this comment.
The exact head 6260a946 matches the shipped v1.8+ retirement contract. The example uses the required boolean metadata marker, preserves mixed-change behavior, and accurately states the pre-mutation abort when the marker is absent. The narrower spec.md wording is also correct because retirement prunes the capability directory only when it is empty. Every required hosted check is green. Approved.
Status
LGTM. This is ready for review.
What was missing
OpenSpec has supported first-class capability retirement since v1.8.0, but the
archivecommand examples did not show how to use it. That made the released solution to #1750 easy to miss, especially for users coming from v1.7.0.What it does
.openspec.yamlexample withretire_capabilities: true.Proof it works
test/core/archive.test.ts: 215/215 passing.applies a retirement and an ordinary update in the same archive).git diff --checkpasses.Notes / nits
Closes #1750
Summary by CodeRabbit