Repository navigation
Check that every rev reference agrees with the policy block - #5
Conversation
A rev bump touched several places CI did not check. Add a stdlib-only step that reads the rev from inside the policy block and requires the frontmatter policy-rev, the step-4 text, evals cases 1 and 2, and each port's README rev to agree with their sources. Line 50 no longer hard-codes the rev, so a bump needs no workflow edit. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
![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: b1f399e9a9
ℹ️ 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".
The revision step matched any indented "policy-rev: N" line anywhere in the SKILL.md frontmatter, so moving it under another mapping still passed. Require exactly one whole-line "metadata:" key in the frontmatter and exactly one " policy-rev: N" among its directly indented lines, with clear errors when either is missing or duplicated. Still stdlib-only. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. Swish! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
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". |
1 participant
What
CI-only change to
.github/workflows/validate.yml. Bumpingpolicy-revand versions in 3.1.0 and 3.1.1 needed manual greps because CI did not catch a missed reference.policy-rev: 5. It requires exactly one whole-line<!-- policy-rev: N -->in SKILL.md, so a bump needs no workflow edit.policy-revpolicy-rev: Nis present inside the block")stackwich-<port>-rev Nmention vs. the rev comment inside that port's own blockEvery check is region-scoped or a whole-line match, since these files document their own marker syntax in prose and substring greps have matched the documentation before.
Not in this PR
No policy block change, so no policy-rev bump, version bump, CHANGELOG entry, tag, or
~/.claudesync. Same precedent as #2. Ships with the next release.Verification
run:text through bash): 9 of 9 PASS on the real repo,revisions OKamong them.Before merging
Read all reviews and comments, wait for the Codex review on this head commit, and confirm the validate run shows the new step passing.
🤖 Generated with Claude Code