docs!: remove obsolete spec and test matrix#207
Conversation
overtrue
changed the title
overtrue
force-pushed
the
overtrue/remove-spec-test-matrix
branch
from
0f877cc to
250cb98
Compare
There was a problem hiding this comment.
Pull request overview
Documentation-only cleanup removing two obsolete docs and adding cross-links between the root README and the generated rc command reference.
Changes:
- Delete obsolete
docs/SPEC.mdanddocs/TEST_MATRIX.md. - Add a link from
README.mdto therccommand reference. - Add a reciprocal link from the
rcreference back to the root README.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| README.md | Adds pointer to full rc command reference. |
| docs/SPEC.md | Removed; superseded by generated reference. |
| docs/TEST_MATRIX.md | Removed; obsolete test matrix. |
| docs/reference/rc/README.md | Adds back-link to project README. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![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: 250cb98c26
ℹ️ 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".
| # rc CLI Specification v2 | ||
|
|
||
| > **PROTECTED FILE**: Changes to this specification require the Breaking Change process. | ||
| > See AGENTS.md for the required workflow. |
There was a problem hiding this comment.
In this repo, AGENTS.md still names docs/SPEC.md as the protected CLI behavior contract and directs future breaking changes to update relevant SPEC sections, but this commit deletes that file without updating the workflow or adding an equivalent protected replacement. When future CLI-contract changes follow the documented process, they now point to a non-existent authority, which makes compatibility review and migration tracking ambiguous despite the commit being marked breaking.
Useful? React with 👍 / 👎.
Summary
docs/SPEC.mdanddocs/TEST_MATRIX.md.rccommand reference.BREAKING
This removes the legacy CLI specification and test matrix documents. The maintained command documentation now lives under
docs/reference/rc/README.md.Validation
git diff --check