build(deps): align CodeQL init and analyze on 4.37.4 - #80
Conversation
Bumps [github/codeql-action/init](https://github.com/github/codeql-action) from 4.37.0 to 4.37.4. - [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...f205ea1) --- updated-dependencies: - dependency-name: github/codeql-action/init dependency-version: 4.37.4 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com>
|
Warning Review limit reached
Next review available in: 7 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Comment |
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 head141d41f4f54a643dc1de4934202c22649b317193. -
Head SHA:
141d41f4f54a643dc1de4934202c22649b317193 -
Workflow run: 30764813433
-
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"]
|
seonghobae
left a comment
There was a problem hiding this comment.
Reviewed the exact Dependabot head. CodeQL init and analyze now use the same immutable 4.37.4 commit, avoiding mixed-version state while preserving languages, build mode, category, permissions, and triggers. Tests, Fuzz, Security (including CodeQL), and Security Scan pass. The remaining Semgrep failure is the pre-existing base finding set repaired by #76; merge remains gated on the rebased current head.
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 headee9e08acb2f3b864c02048f9f7ebe046dab44a61. -
Head SHA:
ee9e08acb2f3b864c02048f9f7ebe046dab44a61 -
Workflow run: 30909471408
-
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"]
|
@dependabot rebase Rebase onto the integrated security/Atheris prerequisite base |
|
Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry! If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request |
|
A newer version of github/codeql-action/init exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged. |
|
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
This stale Draft is superseded by PR #106, which now keeps
github/codeql-action/initandgithub/codeql-action/analyzeatomic on the newer reviewed v4.37.6 commit. #80 targeted the same one-file security workflow boundary at v4.37.4 and no longer has independent integration value.No checks, reviews, approvals, or stale-base evidence from this closed PR transfer to #106. PR #106 must satisfy its own exact-head repository checks, security gates, required automated review evidence, and qualifying independent non-author approval before protected merge.