Repository navigation
feat(runner-policy): register the ci-workflows v0.65.0 intake-triage and pr-review-cli contracts - #856
Conversation
…and pr-review-cli contracts v0.65.0 adds the optional costUsd field to the lane-outcome record (ci-workflows#872). intake-triage.yml and pr-review-cli.yml are unchanged and reach the action through $/, so both contracts copy their prior entries 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 #856 at 652b43ecritical-findings: 0 I found no security issues in this pull request. What changed. The PR adds two entries to Checks.
|
|
claude-review reviewed No findings. The PR adds two
|
…e funding role (#863) No related issue: lane-funding roles plan (D)5, owner-approved 2026-10-24. ## Summary Registers `intake-triage.yml@b0ecc273abe492d12f2efa98ee4741a84c219ed2` on the `intake` funding role, so claude-code-plugins' intake caller can read `INTAKE_CLAUDE_API_KEY` and `INTAKE_CLAUDE_CODE_OAUTH_TOKEN` instead of the `AUTOMATIONS_*` pair. ## Fix A contract carries one `role`, and its caller must pass that literal (`runner-policy.mjs`, the `role` check in the reusable-call audit). Changing the v0.65.0 entry in place would fail runner-policy on claude-code-plugins' sync PR, whose caller still passes `role: automations`. So this adds a contract at a new SHA and leaves the v0.65.0 entry until the caller repins, as the v0.51.0 registration did. [b0ecc27](melodic-software/ci-workflows@b0ecc27) is ci-workflows main, one commit past [v0.65.0](https://github.com/melodic-software/ci-workflows/releases/tag/v0.65.0) that changes only `docs/conventions/source-control/commit-convention.yml` (`git diff v0.65.0 b0ecc27 -- .github` is empty). It is untagged, so the caller pin takes the pin-comment convention's fallback form. The entry is the v0.65.0 entry with `role: intake` and the two `INTAKE_*` secrets; inputs, routing and caller permissions are unchanged. ## Verification - `node --test runner-policy.test.mjs` in `components/runner-policy`: 342 pass. That count includes the new test, which checks the b0ecc27 entry deep-equals the v0.65.0 entry apart from role and secrets, and that one contract names the new SHA. ## Related - Same shape as [#856](#856) (v0.65.0). - Next: the standards sync into claude-code-plugins, then that repository's intake, pr-unblock and pr-verify wiring PR. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
…contract (#864) No related issue: follow-up to [ci-workflows#874](melodic-software/ci-workflows#874), which fixed a review-lane failure on [claude-code-plugins#9545](melodic-software/claude-code-plugins#9545). ## Summary Registers `pr-review-cli.yml@b49ac366bb3426039363a6927c41512e44533808` (ci-workflows [v0.65.1](https://github.com/melodic-software/ci-workflows/releases/tag/v0.65.1)), so claude-code-plugins' two review callers can re-pin to the release that passes the review between jobs as an artifact. ## Fix v0.65.1 changes only `pr-review-cli.yml`'s jobs and its tests: the review and the last marker's summary now travel as a checked artifact instead of a job output, which the runner dropped whenever the text contained a registered secret's value. The workflow's `on.workflow_call` inputs, secrets and runner did not change (`git diff d02b8697..b49ac366 -- .github/workflows/pr-review-cli.yml` touches the header comment and `jobs:` only), so the contract copies its v0.65.0 entry forward verbatim. ## Verification - `node --test runner-policy.test.mjs` in `components/runner-policy`: 343 pass, 0 fail. That includes the new copy-forward test, which checks the new entry deep-equals the v0.65.0 entry and that exactly one contract names the new SHA. - `npx --no-install @biomejs/biome check` (2.5.15) on both changed files: clean. ## Related - Same shape as [#856](#856) (v0.65.0) and [#853](#853) (v0.64.2). - Next: the standards sync into claude-code-plugins, then that repository's review-caller re-pin. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
No related issue: G6 part 2 of the lane-funding plan (owner-approved 2026-10-23).
Summary
Registers
intake-triage.yml@d02b8697e092705153f0996549b05f95a446bb52andpr-review-cli.yml@d02b8697e092705153f0996549b05f95a446bb52(ci-workflows v0.65.0), so claude-code-plugins' intake and review callers can re-pin to the release whose lane-outcome record carriescostUsd.Fix
v0.65.0 changes only report-lane-outcome's
classify.cjs(ci-workflows#872). Both reusables reach that action through$/. Neither workflow file changed since its last registered SHA (git log 2fe38c03..v0.65.0 -- .github/workflows/intake-triage.ymlandgit log v0.64.2..v0.65.0 -- .github/workflows/pr-review-cli.ymllist no commits), so both contracts copy their prior entries forward verbatim.Verification
node --test runner-policy.test.mjsincomponents/runner-policy: 341 pass. That count includes the new copy-forward test, which checks both entries deep-equal their prior SHAs' entries and that exactly two contracts name the new SHA.Related
🤖 Generated with Claude Code