Repository navigation
feat(runner-policy): register ci-workflows v0.62.0 intake and review contracts - #846
Conversation
…contracts Copy the v0.61.0 intake-triage, pr-review and pr-review-security contracts forward verbatim to the v0.62.0 tag 2fe38c031867900824c164e33a140a335ed9ecf0. v0.62.0 changes only the bounded subprocess isolation install step (ci-workflows#858); interfaces and commit-relative actions are unchanged. 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. I reported nothing inline. What I checked (read in the diff and the checkout at f0f1ee7):
Not verified:
|
|
claude-security-review reviewed Security review: PR #846 (head f0f1ee7)No security findings. The PR changes three files:
I did not consult the instruction-exception register, because the diff removes no instruction. |
No related issue: registers the ci-workflows v0.62.0 contracts so claude-code-plugins can repin to it.
Summary
Registers the ci-workflows v0.62.0 tag (
2fe38c031867900824c164e33a140a335ed9ecf0) contracts forintake-triage.yml,pr-review.ymlandpr-review-security.yml, in the same shape as #843 (v0.61.0) and #842 (v0.60.0). v0.62.0 adds only ci-workflows#858, which bounds the subprocess isolation install that timed out in two hosted security-review runs on claude-code-plugins.Fix
components/runner-policy/policy.json: three entries keyed@2fe38c03…, verbatim copies of the@164c2809…(v0.61.0) entries, each placed right after its v0.61.0 entry.components/runner-policy/README.md: a v0.62.0 rollout paragraph after the v0.61.0 one.components/runner-policy/runner-policy.test.mjs:REPINE_LANE_SHA_V0_62_0and a test that the three v0.62.0 contracts equal their v0.61.0 entries and that exactly three v0.62.0 contracts exist.Review basis
git diff 164c2809 2fe38c03(ci-workflows compare API) touches four files: the three workflows and.github/scripts/claude-lane-install-isolation.test.cjs. No commit-relative action changes.yqcomparison of.on(workflow_callinputs and secrets),.permissions, each job'spermissions,runs-onandtimeout-minutes, the job list, and every step'suses: identical between the two tags.pr-review.yml,pr-review-security.yml) and the two "Isolate the runner" steps (intake-triage.yml): apt is skipped whenbwrapandsocatare present, everyapt-getcall runs undersudo timeoutagainst one shared deadline (50 s in the review lanes, 100 s in intake), and the final check requires both tools.Verification
npm run test:runner-policy: 335 tests, 335 pass, 0 fail, including✔ the intake-triage and review contracts at the v0.62.0 tag copy their v0.61.0 entries forward verbatim.npm run lint:runner-policy:Runner policy passed.@2fe38c03…): its vendored policy gives threerunner-target-contractfailures (auto-approval declined); this branch'srunner-policy.mjsandpolicy.jsongiveRunner policy passed.Related
🤖 Generated with Claude Code