build(deps): align CodeQL init and analyze on 4.37.6 - #106
build(deps): align CodeQL init and analyze on 4.37.6#106dependabot[bot] wants to merge 2 commits into
Conversation
Bumps [github/codeql-action/analyze](https://github.com/github/codeql-action) from 4.37.0 to 4.37.6. - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@99df26d...5595cca) --- updated-dependencies: - dependency-name: github/codeql-action/analyze dependency-version: 4.37.6 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Pull request overview
OpenCode cannot approve yet because required coverage evidence did not pass.
Review outcome
1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence
-
Problem: The required coverage-evidence job result was
failure, so OpenCode cannot establish approval sufficiency for this head. -
Root cause: Automated approval is only valid when the same-head coverage-evidence job proves supported repository test suites passed and configured docstring gates passed or were advisory, or reports not applicable because no supported source files or package manifests exist. Missing, failed, skipped, unavailable, or unsupported-tooling test evidence is a blocker.
-
Fix: Install or configure the repository test/docstring evidence tooling when source files or package manifests exist, rerun the current-head coverage-evidence job, and approve only after it reports
successwith required evidence or explicit no-source not-applicable evidence. -
Regression test: Keep the approval branch checking
needs.coverage-evidence.result == successbefore posting APPROVE, and publish REQUEST_CHANGES when coverage-evidence blocker states such as cancelled, skipped, failed, unsupported-tooling, or below-100 evidence are present. -
Result: REQUEST_CHANGES
-
Reason: coverage-evidence result was
failure, so required test/docstring evidence was not proven for current headae45f60bd694c9c7a8d6180ff64769ecbf271304. -
Head SHA:
ae45f60bd694c9c7a8d6180ff64769ecbf271304 -
Workflow run: 31332122757
-
Workflow attempt: 1
Coverage evidence
Coverage evidence job did not run or did not publish coverage evidence.
Changed-File Evidence Map
flowchart LR
PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
Evidence --> S1["Workflow: security.yml"]
S1 --> I1["GitHub Actions review job"]
I1 --> R1["Review risk: Workflow: security.yml"]
R1 --> V1["actionlint plus required checks"]
OpenCode Review Overview
Pull request overviewOpenCode cannot approve yet because required coverage evidence did not pass. Review outcome1. HIGH .github/workflows/opencode-review.yml:1 - Coverage evidence did not prove required test/docstring evidence
Coverage evidenceCoverage evidence job did not run or did not publish coverage evidence. Changed-File Evidence Mapflowchart LR
PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
Evidence --> S1["Workflow: security.yml"]
S1 --> I1["GitHub Actions review job"]
I1 --> R1["Review risk: Workflow: security.yml"]
R1 --> V1["actionlint plus required checks"]
|
|
OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting If you change your mind, just re-open this PR and I'll resolve any conflicts on it. |
This Dependabot PR only bumped codeql-action/init to v4.37.6, leaving
codeql-action/analyze pinned at v4.37.0 -- CodeQL requires init and
analyze to run the same version and fails fast otherwise ("Loaded a
configuration file for version '4.37.6', but running version '4.37.0'"),
confirmed by this PR's own failing CodeQL check. This exact split-bump
problem has recurred several times in this repo's Dependabot history
(#61/#62, #67/#70, #80/#81, and an unmerged #106 attempting the same
alignment) since Dependabot treats the init and analyze action references
as independent update targets even though they must move together.
Bumped analyze to the same v4.37.6 commit (5595ccaf, resolved from the
annotated v4.37.6 tag) so both steps match.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…together Root cause of the recurring split-bump problem this session hit (and that recurred at least 3 times before: #61/#62, #67/#70, #80/#81, plus an unmerged #106 manual-alignment attempt): Dependabot treats github/codeql-action/init and github/codeql-action/analyze as independent update targets, proposing them as separate PRs. Merging just one breaks CodeQL (it requires both steps on the same version) until the other lands. Group them so Dependabot proposes a single PR bumping both together. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Superseded
Closed unmerged in favor of #107.
PR #106 targeted protected
main@6841b71935e0b7cb98fb52bcb4709cc5100c8d87. Its repaired contributor headaef32151e22c8b4044c16ca91b41ee664188ddf6correctly alignedgithub/codeql-action/initandanalyzeon immutable v4.37.6, but SAST run31344272336checked out synthetic mergeaf0bfebc046b9b6a4832562c929224ad2c429ebaand failed on five inherited protected-main findings incost_ledger.pyandorchestrator.py. The one-file dependency diff did not touch those findings.#107 reconstructs the same atomic CodeQL 4.37.6 update directly on exact PR #96 security head
3703d0da9823b8258a0be94f1801aa5d61bfad9f. Exact comparison proves #107 is one commit ahead, zero behind that base and changes only.github/workflows/security.ymlby two additions/two deletions.All #106 checks, reviews, statuses, and synthetic-merge evidence are historical and do not transfer to #107. #107 remains Draft pending #96 protected integration, fresh exact-head checks/security evidence, zero valid unresolved findings, and qualifying independent non-author approval.