Skip to content

fix(tool-cache): persist runner diagnostic events - #170

Merged
rldyourmnd merged 1 commit into
mainfrom
fix/tool-cache-runner-diagnostic-event
Aug 25, 2026
Merged

fix(tool-cache): persist runner diagnostic events#170
rldyourmnd merged 1 commit into
mainfrom
fix/tool-cache-runner-diagnostic-event

Conversation

@rldyourmnd

Copy link
Copy Markdown
Contributor

Summary

  • keep the existing structured job-console event
  • mirror the same bounded non-secret record into the runner _diag directory
  • fail open when a nonstandard runner does not expose a safe writable diagnostic directory
  • test byte-for-byte parity between console and diagnostic evidence

Root cause

Natural run NDDev-it-com/captcha#32783048547 emitted four tool-cache markers in the GitHub job log, but its verified teardown bundle for runner nddev-w2rnoibatea3 contained zero markers across four runner diagnostic files. GitHub Runner does not persist action stdout in _diag.

Verification

  • bash -n actions/tool-cache/tool-cache.sh
  • go test -race ./internal/repositorycontract ./internal/diagnosticexport
  • make verify

@rldyourmnd
rldyourmnd merged commit 470c95c into main Aug 25, 2026
10 checks passed
@rldyourmnd
rldyourmnd deleted the fix/tool-cache-runner-diagnostic-event branch August 25, 2026 01:45
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.

1 participant