Repository navigation
feat(runner-policy): register the ci-workflows v0.65.3 pr-review-cli contract - #867
Conversation
…contract v0.65.3 changes only the hold step's cost-decision matching; the workflow file is unchanged from v0.65.2, so the contract copies its v0.65.2 entry forward. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
claude-review reviewed No findings. What I checked
Not verified
Hold for a person
|
|
claude-security-review reviewed Security review: PR #867 (head caade90)No security findings. The PR adds one
|
…contract (#870) Refs: melodic-software/claude-code-plugins#9589 ## Summary Registers `pr-review-cli.yml@31b44f3ffc7f208fb281dd2b64decce67c53d316`, which is ci-workflows [v0.65.4](https://github.com/melodic-software/ci-workflows/releases/tag/v0.65.4). claude-code-plugins re-pins its review callers to it in [claude-code-plugins#9589](melodic-software/claude-code-plugins#9589). ## Fix v0.65.4 ([ci-workflows#877](melodic-software/ci-workflows#877)) carries a recorded cost decision only while the pull request's net diff against its base is unchanged. In `pr-review-cli.yml` the only non-comment change is `BASE_REF` added to the hold step's `env`. The `workflow_call` inputs differ only in one input description, secrets and permissions are untouched, and the runner is unchanged. The contract copies the v0.65.3 entry from [#867](#867) forward verbatim, with a copy-forward test. ## Verification - `node components/runner-policy/runner-policy.mjs --root .`: Runner policy passed. - `node --test components/runner-policy/runner-policy.test.mjs`: 346 pass, 0 fail, including the new copy-forward test. ## Related - [#867](#867) (v0.65.3) and [#866](#866) (v0.65.2) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Sonnet 5.5 <noreply@anthropic.com>
No related issue: follow-up to ci-workflows#876. claude-code-plugins' review callers pin to this release in claude-code-plugins#9589.
Summary
Registers
pr-review-cli.yml@f133c6651f866845e18595d26cf59d4043b5b642, which is ci-workflows v0.65.3.Fix
v0.65.3 changes only
apply-review-hold/hold.cjsand its test.git diff --stat e7f935fc f133c665lists those two files and nothing else.pr-review-cli.ymlis byte-identical to v0.65.2, so the contract copies the v0.65.2 entry from #866 forward verbatim.Verification
node --test runner-policy.test.mjsincomponents/runner-policy: 345 pass, 0 fail. This includes the new copy-forward test.Related
🤖 Generated with Claude Code