[codex] Add release artifact checksum manifest#24
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: bce65667a6
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| --notes "Release assets for ${RELEASE_TAG}. See docs/release-provenance.md for provenance verification guidance." | ||
| fi | ||
|
|
||
| gh release upload "${RELEASE_TAG}" "${assets[@]}" --repo "${GH_REPO}" |
There was a problem hiding this comment.
Keep release upload idempotent for draft reruns
The publish step now calls gh release upload without --clobber, which makes tag-job reruns fail when a draft release already contains assets with the same filenames (for example after a partial prior run). GitHub CLI documents --clobber as the way to overwrite same-name assets, so removing it turns a previously recoverable rerun path into a hard failure before gh release edit can publish the draft.
Useful? React with 👍 / 👎.
Brief Design Summary
This PR adds a deterministic checksum manifest for
sbom-diff-and-riskrelease artifacts.The tag-gated release path now generates
sbom-diff-and-risk-SHA256SUMS.txtafter building the wheel and sdist. The manifest is filename-sorted, included in the workflow artifact, and uploaded alongside the GitHub Release assets onv*tag pushes.This adds a simple reviewer/consumer verification layer next to the existing GitHub artifact attestation and release provenance story. It does not change CLI behavior, package metadata, TestPyPI behavior, production PyPI status, or release tags.
Files Changed
.github/workflows/sbom-diff-and-risk-ci.ymltools/sbom-diff-and-risk/README.mdtools/sbom-diff-and-risk/docs/release-provenance.mdtools/sbom-diff-and-risk/docs/reviewer-evidence-pack.mdtools/sbom-diff-and-risk/docs/verification.mdValidation
Local validation completed:
The local checksum manifest included both expected files, in deterministic filename order:
PowerShell
Get-FileHashverification passed locally.Additional checks:
refs/tags/vgh release upload --clobberOut of Scope