Skip to content

CI: eliminate upload-sarif missing PR merge-object warning at exact-head scans #278

Description

@seonghobae

Exact-head AppGuardrail run 34637792706, job 103389923269, on #276 head 7973277a4c97db6eaa071ff2726187f6b33009f5 completed successfully, but github/codeql-action/upload-sarif emitted:

git call failed. Will calculate the base branch SHA on the server. Error: fatal: bad object b9f64dae92f9cbf16db5401dfdd0e3cede8641e7

The scanner checks out the exact contributor head with a depth-1 fetch, while GitHub supplies a PR merge-commit identity that is not present locally. upload-sarif falls back to server-side base resolution, so this does not currently block the gate, but the warning violates the no-warning CI contract and obscures whether local provenance is complete.

Acceptance:

  • identify whether the missing object is the PR merge commit/base provenance expected by upload-sarif on pull_request runs;
  • provide the minimum bounded checkout/fetch or explicit SARIF ref/base metadata needed so current-head upload does not attempt a missing Git object;
  • preserve exact contributor-head analysis and persist-credentials: false;
  • do not use stderr filtering, || true, shallow-warning suppression, or a broader fetch than necessary without evidence;
  • add a reproducible hosted regression and verify the warning is absent while SARIF still binds to the exact contributor head.

The separate invalid AppGuardrail SARIF helpUri warnings are producer-owned at ContextualWisdomLab/appguardrail#1216 and should not be conflated with this LifeOS checkout/provenance warning.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions