Repository navigation
chore(ci): pin ci-workflows v0.62.0 - #9140
Conversation
Repin the intake-triage, pr-review and pr-review-security callers from v0.60.0 (82ca5ca9) to v0.62.0 (2fe38c031867900824c164e33a140a335ed9ecf0), which bounds the subprocess isolation install (ci-workflows#858). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…0.62.0 Keeps the v0.62.0 pins over main's v0.61.0 pins (#9074) on the three callers. 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. |
Review lanes: substitute review on 3bae723Neither lane reviewed this head. Both ended In their place, a fresh-context security reviewer read the code the repin actually changes: ci-workflows 164c2809...2fe38c03, one commit (ci-workflows#858). Verdict: no blocking findings.
The caller diff against main is the three |
No related issue: repin to ci-workflows v0.62.0, which fixes the hosted security-review install timeouts seen on this repository.
Summary
Repins the intake-triage caller and the two AI review lane callers from ci-workflows v0.60.0 (
82ca5ca9) to v0.62.0 (2fe38c031867900824c164e33a140a335ed9ecf0). v0.62.0 carries ci-workflows#858, which bounds the subprocess isolation install that hit its 1-minute step limit in hosted security-review runs here (run 39786002588, run 39788017796).Depends on standards#846 and its sync here: until the v0.62.0 contracts land in
.github/standards/runner-policy/policy.json, runner-policy fails this PR on all three callers.Fix
.github/workflows/intake-triage.yml:intake-triage.yml@82ca5ca…(v0.60.0) to@2fe38c0…(v0.62.0)..github/workflows/pr-review-hosted.yml:pr-review.yml@82ca5ca…(v0.60.0) to@2fe38c0…(v0.62.0)..github/workflows/pr-review-security-hosted.yml:pr-review-security.yml@82ca5ca…(v0.60.0) to@2fe38c0…(v0.62.0).What the reusables change between v0.60.0 and v0.62.0 (v0.61.0 plus #858):
intake-triage.yml: no change toworkflow_callinputs, secrets, permissions orruns-on. Each leg writes its own lane-outcome record, a key handoff included, and the status line names the credential (ci-workflows#856); both "Isolate the runner" steps get the bounded install (docs(adr): record Phase II step 3 applied — required check live, skip-actor exception #858).pr-review.ymlandpr-review-security.yml: two new optional inputs,credential-name(defaultCLAUDE_CODE_OAUTH_TOKEN, the secret these callers map) andfallback-credential-name(default empty); the callers pass neither, sowith:,secrets:and grants stay as they are. The report step names the credential in the lane-outcome record (knowledge/youtube-digest: centralize .work/<epic>/<slug> sub-path shape + binary-retention landing — deferred from #558 #856), and the install step skips apt whenbwrapandsocatexist and bounds everyapt-getcall by one 50 s deadline (docs(adr): record Phase II step 3 applied — required check live, skip-actor exception #858).Not changed here: the other pins (intake-label-needs-triage at v0.58.0, maintenance-check-links-online at v0.59.0), which do not use these reusables. No changelog fragment: #9066 repinned these callers without one.
Verification
actionlinton the three workflow files: no findings.runner-target-contractfailures, auto-approval declined for each caller; standards#846'srunner-policy.mjsandpolicy.jsongiveRunner policy passed.Related
🤖 Generated with Claude Code