Repository navigation
chore(ci): re-pin the review callers to ci-workflows v0.65.1 - #9556
Conversation
v0.65.1 passes the review between pr-review-cli's jobs as an artifact, so a review that quotes a registered secret's value no longer reaches the status job empty (ci-workflows#874). 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 read the diff and the PR context and ran one grep for the old and new SHAs. The PR changes the Checked with the grep:
Not run: I did not run the test suites, Hold: none. The change adds no model call, no new trigger and no new runner. It touches no trusted-actors, |
|
claude-security-review reviewed No security issues found in this pull request. The diff re-pins the two review caller workflows (
critical-findings: 0 |
No related issue: the review lanes on #9545 (runs 40054659344 and 40054659337) finished the model leg, but the runner dropped the review output ("Skip output 'review' since it may contain secret"), and the status job reported
incomplete.Summary
Re-pins both review callers to ci-workflows v0.65.1 (
b49ac366). That release passes the review betweenpr-review-cli.yml's jobs as an artifact checked by sha256, so a review that quotes a registered secret's value still publishes (ci-workflows#874).Fix
.github/workflows/pr-review-hosted.ymland.github/workflows/pr-review-security-hosted.yml: only theuses:pin line changes,d02b8697(v0.65.0) tob49ac366(v0.65.1).scripts/ci-review-lanes.test.sh: the pin it asserts..github/standards/runner-policy/policy.jsonalready registers the new SHA, from standards#864, synced here in #9555. Inputs and secrets are unchanged.Verification
bash scripts/ci-review-lanes.test.sh: PASS=55 FAIL=0.node --test .github/actions/resolve-config/callers.test.mjs: 161 pass, 0 fail. It names no pr-review-cli SHA.actionlinton both callers: clean.zizmor --offline: no findings.pr-review-cli-api-calls.test.cjs). The artifact upload and download use the Actions runtime token.Related
🤖 Generated with Claude Code