Repository navigation
feat(runner-policy): register the ci-workflows v0.63.1 pr-review-cli contract - #849
Conversation
…contract v0.63.1 (83b739f6) changes only pr-review-cli's publish step (ci-workflows#865), so its contract copies the v0.63.0 entry forward verbatim. 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-security-review reviewed Security review: PR #849 at 9651290No security findings. critical-findings: 0 What changed: The PR touches three files in Checks:
What I could not check: The README says v0.63.1 changes only the publish step, with no input, secret, permission or routing change. I could not verify that against the workflow file at the two ci-workflows tags. Reading another repository's contents needed approval that was not available in this session. A reviewer with access should confirm that |
|
claude-review reviewed No findings. I read the full diff and the PR context, and compared the new policy entry with the existing one.
|
Refs: melodic-software/claude-code-plugins#9276 ## Summary Registers `pr-review-cli.yml@4edeccd87cf60ef539ce5109b78454217d5f2049` (ci-workflows [v0.64.0](https://github.com/melodic-software/ci-workflows/releases/tag/v0.64.0)) so callers can pin it and pass its two new optional inputs, `hold-label` and `hold-paths`. ## Fix v0.64.0 adds an optional hold step to the status job ([ci-workflows#864](melodic-software/ci-workflows#864)). The new contract copies the v0.63.1 entry forward and appends `hold-label` and `hold-paths` to `allowedInputs`. Routing (`hosted-only`), `fixedRunsOn` (`ubuntu-24.04`) and the two reviews-role secrets are unchanged. No caller permission is added: the hold step writes through the status job's existing `pull-requests: write` grant. A diff of the workflow between v0.63.1 and v0.64.0 shows no other input, secret or `runs-on` change. ## Verification - `node --test runner-policy.test.mjs` in `components/runner-policy`: 338 pass, 0 fail, including the new test that the v0.64.0 entry equals v0.63.1 plus the two hold inputs. - `npm run lint:runner-policy`: passed. - `node --test components/claude-lanes/repin-policy-lockstep.test.mjs`: 23 pass. - Pre-commit (typos, editorconfig, markdownlint, gitleaks, biome): clean. ## Related - Same shape as #849 (v0.63.1) and #848 (v0.63.0). - After merge, the push-triggered `maintenance-sync-standards` run opens the sync PR into claude-code-plugins' `.github/standards/runner-policy/policy.json`. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…contract (#851) Refs: melodic-software/claude-code-plugins#9326 ## Summary Registers `pr-review-cli.yml@9aca743c4ed9aaf7870288a36586c3994e9321e4` (ci-workflows [v0.64.1](https://github.com/melodic-software/ci-workflows/releases/tag/v0.64.1)), so claude-code-plugins callers can re-pin to it. ## Fix v0.64.1 changes only pr-review-cli's extract and publish steps and its `continue-on-error` wiring ([ci-workflows#868](melodic-software/ci-workflows#868)). A malformed review comment no longer drops the whole review, and the status job now goes red in comments mode when no review happened. The contract copies the v0.64.0 entry forward verbatim: same routing, inputs (including `hold-label` and `hold-paths`) and secrets. ## Verification - `node --test runner-policy.test.mjs` in `components/runner-policy`: 339 pass, including the new copy-forward test. ## Related - Same shape as [#849](#849) (v0.63.1). 🤖 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#9307
Summary
Registers
pr-review-cli.yml@83b739f625b501ad40c65dee7168d836883195e1(ci-workflows v0.63.1) so claude-code-plugins#9307 can re-pin its security lane to it.Fix
v0.63.1 changes only the publish step (ci-workflows#865: titled completion notice, demoted CRITICAL findings counted). The contract copies the v0.63.0 entry forward verbatim: same routing, inputs and secrets.
Verification
node --test runner-policy.test.mjsincomponents/runner-policy: 337 pass, including the new copy-forward test.Related
🤖 Generated with Claude Code