Repository navigation
feat(runner-policy): register the ci-workflows v0.65.11 intake-triage and pr-review-cli contracts - #880
Conversation
… and pr-review-cli contracts Copies the v0.65.10 intake-triage and v0.65.9 pr-review-cli entries forward verbatim to e5167fc29d948ab7d0923cf62f5ea9eeeef8e9e8 (ci-workflows#889), keeping past pins. 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
|
|
claude-security-review reviewed No security findings. critical-findings: 0 The PR adds two approved-workflow contract entries to
|
… and pr-review-cli contracts (#881) Refs: melodic-software/ci-workflows#890 ## Summary Registers the `intake-triage.yml` and `pr-review-cli.yml` reusable-workflow contracts at ci-workflows [v0.65.12](https://github.com/melodic-software/ci-workflows/releases/tag/v0.65.12) (`78ee7a2609a84b0073c5e09dfe3bc59613e689dc`), so claude-code-plugins can repin its intake and review callers and pass runner-policy. ## Fix - Copies the v0.65.11 `intake-triage` and `pr-review-cli` entries (`@e5167fc29d948ab7d0923cf62f5ea9eeeef8e9e8`) forward in `components/runner-policy/policy.json` and adds one secret to each: `FALLBACK_CLAUDE_CODE_OAUTH_TOKEN` mapped to `${{ secrets.FALLBACK_CLAUDE_CODE_OAUTH_TOKEN }}`. Past entries stay. - Why not a verbatim copy, as in #880: v0.65.12 includes [ci-workflows#890](melodic-software/ci-workflows#890), which adds that optional `workflow_call` secret to both workflows for the third, fallback job. runner-policy rejects a caller secret the contract does not list (`exactCanonicalMap`, "has unapproved properties"), so without the mapping the repinned callers could not pass the secret and the fallback leg would never run. Inputs and job permissions are unchanged between the two tags. - Adds a test that each v0.65.12 entry equals its v0.65.11 entry plus exactly that secret, and a rollout line in the component README. ## Verification - `node --test components/runner-policy/runner-policy.test.mjs`: 354 pass, 0 fail. - `node components/runner-policy/runner-policy.mjs --root <worktree>`: Runner policy passed. - Pre-commit biome, markdownlint, typos and gitleaks: clean. ## Related - [melodic-software/ci-workflows#890](melodic-software/ci-workflows#890) and [#891](melodic-software/ci-workflows#891) (released as v0.65.12) - [#880](#880) (v0.65.11 contracts) - Next: the standards sync into claude-code-plugins, then the lane caller repin, which passes the fallback secret. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Refs: melodic-software/ci-workflows#889
Summary
Registers the
intake-triage.ymlandpr-review-cli.ymlreusable-workflowcontracts at ci-workflows
v0.65.11
(
e5167fc29d948ab7d0923cf62f5ea9eeeef8e9e8), so claude-code-plugins can repinits intake and review callers and pass runner-policy.
Fix
intake-triageentry (@215cec5c362f953f8b6196effb04451696c3f970)and the v0.65.9
pr-review-clientry (@4fbceb4b75ffd568dfa5db31de0d0e753c0c64e6)forward verbatim in
components/runner-policy/policy.json. Past entries stay.runner-policy.test.mjsand arollout line in the component README.
passes
log-filetoreport-lane-outcome, which records per-leg token usageand rate-limit windows. In both workflows the diff adds only that
with:line: no input, secret or job permission changes.
Verification
node --test components/runner-policy/runner-policy.test.mjs: 353 pass, 0 fail.node components/runner-policy/runner-policy.mjs --root <worktree>: Runner policy passed.Related
🤖 Generated with Claude Code