Skip to content

ci: fix publish job by moving npm release to OIDC trusted publishing#444

Draft
Copilot wants to merge 2 commits into
masterfrom
copilot/fix-publish-job-failure
Draft

ci: fix publish job by moving npm release to OIDC trusted publishing#444
Copilot wants to merge 2 commits into
masterfrom
copilot/fix-publish-job-failure

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 25, 2026

Description

The publish GitHub Actions job was failing on release tags with npm publish returning E404/permission-not-found for @mia-platform/lc39. This updates the workflow to use npm trusted publishing (GitHub OIDC) for the release publish path.

  • Root cause

    • Publish relied on token-based auth in the workflow; npm rejected publish for the scoped package during tag release.
  • Workflow changes

    • Added explicit job permissions required for OIDC:
      • contents: read
      • id-token: write
  • Publish command update

    • Switched publish invocation to provenance-enabled trusted publishing and removed explicit token env on the step.
permissions:
  contents: read
  id-token: write

- run: npm publish --access public --provenance --tag ${{ steps.npm-tag.outputs.tag }}

Checklist

  • your branch will not cause merge conflict with master
  • run npm test
  • tests are included
  • the documentation is updated or integrated accordingly with your changes
  • the code will follow the lint rules and style of the project
  • you are not committing extraneous files or sensible data

Agent-Logs-Url: https://github.com/mia-platform/lc39/sessions/2d1b6a4a-78ff-4b3c-9e8e-8094ff9be7f0

Co-authored-by: ThisIsDemetrio <5429953+ThisIsDemetrio@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job publish ci: fix publish job by moving npm release to OIDC trusted publishing May 25, 2026
Copilot AI requested a review from ThisIsDemetrio May 25, 2026 16:15
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