Skip to content

Harden trusted release source checkout - #56

Merged
GGULBAE merged 1 commit into
masterfrom
codex/harden-trusted-release-checkout
Aug 13, 2026
Merged

Harden trusted release source checkout#56
GGULBAE merged 1 commit into
masterfrom
codex/harden-trusted-release-checkout

Conversation

@GGULBAE

@GGULBAE GGULBAE commented Aug 13, 2026

Copy link
Copy Markdown
Owner

What changed

  • checkout only GitHub's immutable event SHA in Trusted Release and reusable Compatibility jobs
  • require publish confirmation plus protected master and exact event/input SHA equality before Compatibility starts
  • verify event ref, event SHA, checked-out HEAD, supplied SHA, and freshly fetched origin/master before repository code runs in preflight and publish jobs
  • disable dependency caching in privileged release jobs
  • add fail-closed workflow supply-chain contracts and mutation tests for tainted refs, forwarded inputs, reordered/missing identity checks, and cache restoration
  • document the resulting release trust boundary

Why

CodeQL identified seven high-severity cache-poisoning findings that all originated from one root cause: a workflow_dispatch input was used as the executable checkout ref before local actions, installs, builds, and package inspection. The previous post-checkout SHA validation was fail-closed at runtime but did not remove the untrusted-code execution boundary, and the parallel Compatibility gate could execute before that validation.

Impact

Trusted Release still accepts an exact reviewed source SHA, but the input is now an assertion only. Executable source always comes from the immutable workflow event commit on protected master. Accidental unconfirmed dispatches also avoid the eight-lane Compatibility cost.

Validation

  • VITEST_MAX_WORKERS=2 pnpm verify — 59 files / 547 tests; coverage 94.10% statements, 86.13% branches, 98.40% functions, 94.58% lines
  • pnpm example:typecheck
  • workflow supply-chain and shell syntax tests — 16/16
  • workflow YAML parse
  • git diff --check
  • independent adversarial review found no remaining P0/P1 issue

@GGULBAE
GGULBAE marked this pull request as ready for review August 13, 2026 15:31
@GGULBAE
GGULBAE merged commit 4e1f2b7 into master Aug 13, 2026
16 checks passed
@GGULBAE
GGULBAE deleted the codex/harden-trusted-release-checkout branch August 13, 2026 15:31
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