Skip to content

Fix PyPI publish workflow trusted-publisher claim mismatch - #11

Merged
mzargham merged 2 commits into
mainfrom
copilot/fix-publish-job
Sep 19, 2026
Merged

mzargham merged 2 commits into
mainfrom
copilot/fix-publish-job

Conversation

Copilot AI commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

The publish GitHub Actions job failed during trusted publishing with invalid-publisher because the OIDC token subject included an environment-scoped claim that did not match the configured PyPI publisher. This change aligns the workflow identity claims with PyPI trusted publisher matching.

  • Workflow identity alignment

    • Removed environment: pypi from the publish job in .github/workflows/publish.yml.
    • This drops the :environment:pypi suffix from the OIDC sub claim used by pypa/gh-action-pypi-publish, allowing publisher matching against the repository/workflow identity.
  • No behavioral changes outside publish auth

    • Build and upload steps remain unchanged.
    • Only the claim shape used for PyPI trusted publishing is adjusted.
jobs:
  publish:
    runs-on: ubuntu-latest
    permissions:
      id-token: write

Co-authored-by: mzargham <10465438+mzargham@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job publish Fix PyPI publish workflow trusted-publisher claim mismatch Sep 19, 2026
Copilot AI requested a review from mzargham September 19, 2026 19:05
@mzargham
mzargham marked this pull request as ready for review September 19, 2026 19:15

@mzargham mzargham 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.

this fix makes sense to me.

@mzargham
mzargham merged commit 0e2acab into main Sep 19, 2026
2 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.

2 participants