Skip to content

Investigate and harden PureLock coverage safe-output pathing for PR #55077 - #55094

Merged
pelikhan merged 1 commit into
mainfrom
copilot/investigate-update-code-coverage-failure
Aug 23, 2026
Merged

Investigate and harden PureLock coverage safe-output pathing for PR #55077#55094
pelikhan merged 1 commit into
mainfrom
copilot/investigate-update-code-coverage-failure

Conversation

Copilot AI commented Aug 23, 2026

Copy link
Copy Markdown
Contributor

The update-code-coverage (upload_code_coverage) safe output in the PureLock flow failed because coverage output was written to a runner temp location that was read-only in this execution context. This change updates PureLock’s coverage artifact handoff to use a writable path and an explicit absolute file reference.

  • Root cause identified

    • PureLock instructions directed Cobertura output to $RUNNER_TEMP/gh-aw/safeoutputs/upload-code-coverage/, which was not writable in the failing runtime.
  • Workflow instruction update

    • Updated .github/workflows/purelock.md to generate cobertura.xml under /tmp/gh-aw/purelock/.
    • Updated the safe-output call contract to pass file: "/tmp/gh-aw/purelock/cobertura.xml".
  • Compiled workflow alignment

    • Recompiled .github/workflows/purelock.lock.yml so generated workflow behavior matches the updated markdown source.
mkdir -p /tmp/gh-aw/purelock
go test ./<unique-succeeded-package-dir>/... -count=1 -covermode=atomic -coverprofile=/tmp/gh-aw/purelock/generated-tests.out
"$(go env GOPATH)/bin/gocover-cobertura" < /tmp/gh-aw/purelock/generated-tests.out > /tmp/gh-aw/purelock/cobertura.xml
# safe output payload: file="/tmp/gh-aw/purelock/cobertura.xml"

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title Investigate and fix PureLock coverage safe output failure from PR #55077 Investigate and harden PureLock coverage safe-output pathing for PR #55077 Aug 23, 2026
Copilot AI requested a review from pelikhan August 23, 2026 14:21
@pelikhan
pelikhan marked this pull request as ready for review August 23, 2026 14:27
Copilot AI balanced review requested due to automatic review settings August 23, 2026 14:27
@pelikhan
pelikhan merged commit 562d840 into main Aug 23, 2026
@pelikhan
pelikhan deleted the copilot/investigate-update-code-coverage-failure branch August 23, 2026 14:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Moves PureLock coverage output to a writable temporary path and updates its safe-output reference.

Changes:

  • Writes Cobertura output under /tmp/gh-aw/purelock.
  • Refreshes compiled workflow metadata.
Show a summary per file
File Description
.github/workflows/purelock.md Updates coverage generation and upload path.
.github/workflows/purelock.lock.yml Updates the generated body hash.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Only after verifying `cobertura.xml` exists and is non-empty, call `upload_code_coverage` with
`file: "cobertura.xml"`, `language: "Go"`, and `label: "purelock/generated-tests"`.
Only after verifying `/tmp/gh-aw/purelock/cobertura.xml` exists and is non-empty, call `upload_code_coverage` with
`file: "/tmp/gh-aw/purelock/cobertura.xml"`, `language: "Go"`, and `label: "purelock/generated-tests"`.
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.87.5

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants