chore(ci): update CodeQL action coherently to 4.37.8 - #1690
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe CodeQL workflow now uses a new pinned action commit. The contract test centralizes the commit and verifies that both CodeQL actions appear twice with the expected pin. ChangesCodeQL action pin update
Estimated code review effort: 2 (Simple) | ~5 minutes Merge Risk: ⚪ Minimal · up to This PR coherently updates the pinned CodeQL actions and the test enforcing those pins while preserving existing workflow triggers, permissions, checkout pinning, and upload behavior. No actionable merge-blocking risk remains beyond normal checks and review. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 1 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Supersedes #1685 and #1686, which independently updated
analyzeandiniton the same workflow and therefore could not satisfy the repository's pinned-SHA regression contract on their own.github/codeql-action/initandgithub/codeql-action/analyzeuse to the reviewed v4.37.8 release SHAdb488ddef3bf6cb639b32c2e9a7c0a7ea8271d28;tests/test_codeql_workflow_contract.pyso both jobs must use exactly twoinitand twoanalyzereferences, all on that same SHA;upload: never, checkout pin, and least-privilege permissions.Test-first lineage
edaf86051aa8e062ef4706a4c5c975ae0e02b985: the regression contract requires the v4.37.8 SHA while the workflow still contains v4.37.7, deterministically making the pin contract unsatisfied.c7f08bb0b1ad00019f664b59f885e3991ba201ce: updates all four CodeQL action call sites coherently to v4.37.8 so the exact-count/same-release contract is satisfied.f811f4714112accc0972a3f0fdb99f8d74ba5055: non-force reconciliation with protectedmain@b5a3a0c1057d4b53d7a4bb18e0de69f630c2b45c;main...headleaves only the intended two-file CodeQL pin/contract delta. The protected binary-response Measurement files carried by the intervening ancestry are already present onmainand are not part of this PR's effective delta.No hosted RED is claimed because the RED commit existed only as isolated source lineage before the PR was opened.
Exact-current hosted evidence
On unchanged
f811f4714112accc0972a3f0fdb99f8d74ba5055, repository CI33633947456is now terminal success. The six substantive jobs all ran on GitHub-hosted runners with checkout/source execution and passed: Python 3.14100259967637, GPU smoke100259967802, Rust100259967902, fuzz100259967954, Python 3.12100259968046, and package100259968061; the downstream required aggregatepythonjob100469330106also acquired a runner and completed success at 2026-09-03T12:25:29Z. Repository CodeQL33633947418, SAST Semgrep33633947570, OSV-Scanner PR33633947983, Security Scan33633947183, and Scorecard PR33633947204are likewise terminal success. The repository-requiredAnalyze (actions)check is therefore GREEN on this exact head.Remaining current-head evidence is not all passing. OpenCode required-workflow run
33633945191hascoverage-source-tree100467837180terminal success, butcoverage-evidence100644558073remains queued before runner assignment/source execution (runner_id=0, empty runner/group identity,steps=[]). Strix run33633945187hasstrixjob100259961027terminal cancelled without runner or steps; its latercancel-superseded-pr-runsjob100259960620did execute successfully, so the cancelled Strix verdict is not source/security evidence and cannot be promoted to GREEN. Noema run33633945304did acquire a hosted runner, validated the exact head, provisioned the contextual-orchestrator sidecar, then failed at the first causal boundaryPrepare Noema model verdict(100259961436, step 13) after the earlier credential/head/sidecar steps had succeeded. The exposed job metadata does not establish which already-tracked model-output/provider sub-class caused that model-verdict failure, so it is classified only as central Noema review-control/runtime evidence rather than a fast-mlsirm source finding.The historical central
CodeQL PRrun33633948806still recordsstartup_failure, but CodeQL is no longer one of the live organization ruleset's nine required-workflow paths. It must not be misreported as the current organization-required blocker. Current central queue/admission evidence for runnerless coverage/scan lanes is tracked onContextualWisdomLab/.github#712; the open Noema model-output/retry owner issues remain in the central.githubrepository. No leaf no-op retrigger or source churn is warranted.Review state is also non-landing: the only submitted #1690 review is Devin
COMMENTED, there are zero inline review threads, and a fresh organization-wide search finds zero open fast-mlsirm PRs with a qualifying currentAPPROVEDstate.Safety / scope
This is CI dependency maintenance only. It changes no package/runtime dependencies, model formulas, statistical or psychometric arithmetic, Rust/Python backend ownership, security thresholds, or protected-branch policy. It does not weaken any required gate.
Landing remains subject to exact-current required workflows, resolved review conversations, and the live approval policy. No self-approval, administrator bypass, force update, gate weakening, or predecessor-evidence transfer is requested.
Summary by CodeRabbit
Security
Tests