Repository navigation
feat(runner-policy): register the ci-workflows v0.65.5 pr-review-cli contract - #871
Conversation
…contract v0.65.5 (ci-workflows#878) changes only the hold step's cost-decision carry-over: a decision carries only from an unedited hold marker. Inputs, secrets and runner are unchanged, so the v0.65.4 contract copies forward. Co-Authored-By: Claude Sonnet 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. The PR adds a
|
|
claude-security-review reviewed Security review: PR #871 (head 2117a55)No security findings. critical-findings: 0 What changed:
Trust-boundary checks:
Limitation: I could not confirm that the pinned SHA is the commit behind the ci-workflows The protected-class register was not consulted, because the diff removes no instructions. |
…contract (#874) Refs: melodic-software/ci-workflows#879 ## Summary Registers the `pr-review-cli.yml` reusable-workflow contract at ci-workflows [v0.65.6](https://github.com/melodic-software/ci-workflows/releases/tag/v0.65.6) (`914264587c35692c15e05c965103426eec18d1f0`), so callers pinned to that release pass runner-policy. ## Fix Copies the v0.65.5 entry (`@f1096608192c54cfb3dc252e36670e6f86700b9b`) forward in `components/runner-policy/policy.json` and adds the matching copy-forward test in `runner-policy.test.mjs`, as #870 and #871 did. Earlier pins stay as history. `gh api compare v0.65.5...v0.65.6` on ci-workflows lists three changed files; the `pr-review-cli.yml` patch is comment-only (ci-workflows#879: a cost decision carries no further once a pull request comment was deleted since its hold marker), so inputs, secrets and permissions are unchanged. ## Verification - `node components/runner-policy/runner-policy.mjs --root <worktree>`: Runner policy passed. - `node --test components/runner-policy/runner-policy.test.mjs`: 348 tests, 348 pass, 0 fail. ## Related - melodic-software/ci-workflows#879 (the fix released as v0.65.6) - #871 (the v0.65.5 contract) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Refs: melodic-software/claude-code-plugins#9589
Summary
Registers the
pr-review-cli.ymlreusable-workflow contract at ci-workflows v0.65.5 (f1096608192c54cfb3dc252e36670e6f86700b9b), so callers pinned to that release pass runner-policy.Fix
Copies the v0.65.4 entry (
@31b44f3ffc7f208fb281dd2b64decce67c53d316) forward incomponents/runner-policy/policy.jsonand adds the matching copy-forward test inrunner-policy.test.mjs, as #870 did.gh api compare v0.65.4...v0.65.5on ci-workflows lists three changed files; thepr-review-cli.ymlpatch is comment-only (ci-workflows#878: a cost decision carries only from an unedited hold marker), so inputs, secrets and permissions are unchanged.Verification
node components/runner-policy/runner-policy.mjs --root <worktree>: Runner policy passed.node --test components/runner-policy/runner-policy.test.mjs: 347 tests, 347 pass, 0 fail.Related
🤖 Generated with Claude Code