Skip to content

fix(publish): honor provenance in PNPM 11+ - #141

Merged
pascal-botpress merged 1 commit into
masterfrom
pb/provenance
Aug 27, 2026
Merged

pascal-botpress merged 1 commit into
masterfrom
pb/provenance

Conversation

@pascal-botpress

@pascal-botpress pascal-botpress commented Aug 27, 2026 •

Copy link
Copy Markdown
Member

Pnpm 11+ no longer honors NPM_CONFIG_PROVENANCE, so repos upgrading to pnpm 11 would no longer be able to publish.

contributes to KKN-927

Copilot AI lite review requested due to automatic review settings August 27, 2026 14:52

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the publish-if-not-exists composite action so provenance publishing continues to work when callers use pnpm 11+, which no longer reads NPM_CONFIG_PROVENANCE.

Changes:

  • Adds a new optional provenance input to explicitly control provenance (true / false / empty to defer).
  • Introduces a “Resolve provenance” step that normalizes provenance preference across inputs.provenance, NPM_CONFIG_PROVENANCE, and PNPM_CONFIG_PROVENANCE, and exports both env vars for the publish step.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@pascal-botpress
pascal-botpress merged commit 3d4ab13 into master Aug 27, 2026
2 checks passed
@linear-code

linear-code Bot commented Aug 27, 2026

Copy link
Copy Markdown

KKN-927

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.

3 participants