Skip to content

ci: PEP 740 attestations via astral-sh/attest-action; middleware 0.0.3 - #878

Merged
blove merged 1 commit into
mainfrom
blove/pypi-attestations
Aug 30, 2026
Merged

ci: PEP 740 attestations via astral-sh/attest-action; middleware 0.0.3#878
blove merged 1 commit into
mainfrom
blove/pypi-attestations

Conversation

@blove

@blove blove commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Resolves the attestation trade-off from #873 — it turns out there wasn't one.

What I had wrong in #873

I wrote "uv publish doesn't support attestations yet." Half-wrong: uv publish uploads attestation files found beside the dists by default (--no-attestations is the opt-out) — it just doesn't mint them. Astral's own GHA publishing example pairs it with their attest-action for the minting step. So we can have one toolchain end to end and attestations; no version skew re-enters because nothing validates metadata twice with different tools.

Changes

  • Generate PEP 740 attestations step (real release only — a dry run uploads nothing for them to accompany), pinned astral-sh/attest-action@f589a42a # v0.0.6, SHA verified against the tag per the action-pin discipline. The input is paths (whitespace-separated globs) — caught by reading action.yml at the pinned SHA rather than trusting my first guess of path.
  • Middleware bumped 0.0.2 → 0.0.3, no code change: PyPI files are immutable, so attestations are only provable on a fresh upload. Consumers (examples/chat, cockpit/langgraph/client-tools) pin >=0.0.2/>=0.0.1 and are unaffected.

Verification plan (after merge)

  1. Dry-run dispatch (attest step correctly skipped, tests + build green)
  2. Real dispatch for 0.0.3
  3. The check that counts: PyPI simple API files[].provenance for the 0.0.3 wheel + sdist must be non-null — the same query that showed none for 0.0.2

Also verified while investigating: current twine passes our Metadata-Version 2.5 artifacts, so the pypa-action path would work again today too — but it re-arms the two-toolchain skew, which is how #873 happened.

🤖 Generated with Claude Code

…e 0.0.3

Restores what #873 traded away, without reintroducing the toolchain skew.
uv publish UPLOADS attestation files found beside the dists by default
(--no-attestations is the opt-out) — it just doesn't MINT them. Astral's
own publishing example pairs it with astral-sh/attest-action for exactly
that. One toolchain end to end, attestations back.

- attest step gated to the real release (a dry run uploads nothing for
  attestations to accompany); SHA pin f589a42a verified against tag v0.0.6
  (lightweight tag, commit direct); the action's input is  (glob) —
  caught by reading action.yml at the pinned SHA, not the README
- middleware bumped to 0.0.3: attestations are only provable on a fresh
  upload, and PyPI versions are immutable. No code change; consumers pin
  >=0.0.2 and are unaffected. The 0.0.3 upload doubles as the verification
  (simple-API provenance must flip from 'none')

Verified locally: 0.0.2 artifacts pass current twine check (Metadata 2.5),
YAML parses.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@vercel

vercel Bot commented Aug 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview Aug 30, 2026 8:26pm

Request Review

@github-actions github-actions Bot 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.

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@blove
blove enabled auto-merge (squash) August 30, 2026 20:16
@github-actions

github-actions Bot commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@blove
blove merged commit 0ea5bd9 into main Aug 30, 2026
28 checks passed
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