Repository navigation
Consistently use the repository's preferred package manager - #2097
Merged
Merged
Conversation
🦋 Changeset detectedLatest commit: 0028822 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
1 task
Andarist
force-pushed
the
simplify-auth-flow
branch
2 times, most recently
from
June 23, 2026 21:22
7cdb754 to
3b9c86b
Compare
Andarist
force-pushed
the
use-host-package-manager
branch
from
July 1, 2026 11:33
bd3c4f5 to
07ccfb0
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #2097 +/- ##
==========================================
- Coverage 88.24% 82.27% -5.98%
==========================================
Files 72 76 +4
Lines 2604 3024 +420
Branches 698 845 +147
==========================================
+ Hits 2298 2488 +190
- Misses 283 485 +202
- Partials 23 51 +28 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
bluwy
reviewed
Jul 15, 2026
bluwy
approved these changes
Jul 15, 2026
|
Will these changes help solve this issue? #1468 |
Member
Author
|
@jperezrealini not yet - but we'll work on Bun support soon. This PR and #2155 prepare the groundwork for that |
This was referenced Jul 27, 2026
Closed
cursor Bot
pushed a commit
to thilllon/pixid
that referenced
this pull request
Aug 26, 2026
Switch the release workflow to npm trusted publishing: - Add id-token: write so the job can mint a GitHub OIDC token. - Drop the 'skip when NPM_TOKEN is empty' guard: the Version Packages PR and OIDC publishes need no secret. NPM_TOKEN stays wired through as a fallback only. - Keep changesets/action with 'publish: pnpm release' unchanged. The publish path is OIDC-capable end to end: @changesets/cli v3 routes the actual publish through the detected package manager (changesets/changesets#2097), i.e. 'pnpm publish' per package, and pnpm 11 implements the npm OIDC token exchange natively, preferring it over any static _authToken (pnpm/pnpm#11495). So no restructuring to 'pnpm -r publish' is needed and changesets keeps handling tags, GitHub releases, and skip-already-published logic. - No npm CLI upgrade step: the npm >= 11.5.1 trusted-publishing minimum is met by node 24's bundled npm (11.17.0 today), and npm is not on the publish path anyway since pnpm performs the publish itself. - No registry-url on setup-node: it would write an .npmrc token placeholder that shadows OIDC (npm/cli#8730). Publishing succeeds once a trusted publisher (owner thilllon, repo pixid, workflow release.yml) is configured on npmjs.com per package. Trusted publishing cannot create a brand-new package, so the unscoped 'pixid' needs a one-time manual/token first publish before its trusted publisher can be configured; until then the publish step fails on that package only. Co-authored-by: thilllon <[email protected]>
Draft
5 of 11 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
fixes #432
fixes #1454
fixes #598
fixes #813
fixes #1411