Skip to content

fix(ci): keep SBOM output out of dist/ so PyPI publish doesn't choke on it - #371

Merged
dwolfson merged 1 commit into
mainfrom
fix/release-sbom-dist-collision
Sep 16, 2026
Merged

dwolfson merged 1 commit into
mainfrom
fix/release-sbom-dist-collision

Conversation

@dwolfson

Copy link
Copy Markdown
Member

Summary

  • The "Generate SBOM" step in release.yml (added in Enhance FOSS Compliance and OSSF Scorecard Posture #368) writes bom.json/bom.xml into dist/ alongside the real wheel/sdist. pypa/gh-action-pypi-publish globs dist/* by default and tried to upload the SBOM files as distribution artifacts, failing the pre-upload check ("InvalidDistribution: Unknown distribution format: 'bom.json'") and aborting before anything was uploaded.
  • Confirmed live on the v6.1.16 tag push — the first release run since this step existed. Nothing reached PyPI and no GitHub Release was created, so there's no partial state to clean up.
  • Fix: write SBOM output to its own sbom/ directory, and attach both dist/* and sbom/* to the GitHub Release.

Test plan

  • Re-tag v6.1.16 once this merges and confirm the release workflow completes end to end (PyPI publish + GitHub Release with wheel/sdist/SBOM attached)

🤖 Generated with Claude Code

…on it

pypa/gh-action-pypi-publish globs dist/* by default and tries to upload
every file there to PyPI as a distribution artifact. The "Generate SBOM"
step (added in #368) wrote bom.json/bom.xml straight into dist/ alongside
the real wheel/sdist, so the publish step's pre-upload check failed on
bom.json ("InvalidDistribution: Unknown distribution format") and aborted
before uploading anything -- confirmed live on the v6.1.16 tag push, the
first release run since that step was added. Write the SBOM to its own
sbom/ directory instead, and attach both dist/* and sbom/* to the GitHub
Release.

Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Signed-off-by: Dan Wolfson <dan.wolfson@pdr-associates.com>
@dwolfson
dwolfson merged commit a46e5f3 into main Sep 16, 2026
5 checks passed
@dwolfson
dwolfson deleted the fix/release-sbom-dist-collision branch September 16, 2026 16:37
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