{{ message }}
Repository navigation
Conversation
✅ Deploy Preview for changesets ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
jbgomez
marked this pull request as ready for review
October 14, 2026 16:10
This branch has not been deployed
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.

Bun workspaces currently fall back to npm for publishing. This can leave
workspace:dependency specs in published tarballs, causing npm consumers to fail withEUNSUPPORTEDPROTOCOL. This change selects Bun for package info, packing, and publishing so workspace dependency specs are resolved by Bun.This PR builds on @Andarist's #2181. Its five original commits have been rebased onto current
mainwith the original authorship preserved. The follow-up commits complete authentication handling, regression coverage, and documentation. These additions can also be incorporated into #2181 if that is the preferred landing path.Implementation
bun info,bun pm pack, andbun publish, including publishing prebuilt tarballs with--from-pack-dir.publishConfig.directorypoints to a missing build directory. Reject selecting a different directory for direct publishing or packing with an explicit error.NPM_CONFIG_TOKENor project.npmrc, and action v2 structured tag output.Bun >=1.3.1 is retained from #2181. That version fixed packing declared executables excluded by the
fileslist; the omission was reproduced on 1.3.0 and the positive regression is covered on both supported test versions.Includes a changeset for
@changesets/cli(minor) and@changesets/types(patch).Validation
Local validation used the repository scripts, invoked through Bun, with the repository's pnpm workspace and lockfile retained:
bun run test --run: 858 passed, with coverage enabled, 2 existing todo tests.scripts/e2e:fnm exec --using v24.15.0 -- bun run test --experimental.preParse=false -t 'bun|surfaces web-auth OTP|surfaces pre-publish errors|publishes a new version of a package|publishes from a pack directory|normalizes captured|preserves interactive'.-t 'publishes a new version of a package|publishes from a pack directory'and the same Node/preParse settings.bun.tsandgetPublishTool.ts, without coverage exclusions or threshold changes.Upstream CI run 38594001045 passed at
c08e68033c536d386b6fe0c63dfa055c57b7dade: build, lint, typecheck, unit tests on Linux/macOS/Windows, and the complete Linux e2e suite. Codecov patch and project checks passed; all modified, coverable lines are covered, and project coverage is 88.46%. Native Bun integrations were validated on macOS locally and Linux in CI; native Windows integration validation remains outstanding.Known limitations
changeset versiondoes not update Bun's lockfile. Users must refresh it withbun install --lockfile-only --ignore-scriptsafter versioning and before packing/publishing. Updating the lockfile after packing cannot repair existing artifacts.Direct
publishConfig.directoryoverrides are unsupported for Bun. Prebuilt tarball publishing is supported.This targets Changesets 3.x on
main; it does not backport support tomaintenance/v2. npm trusted publishing/provenance parity is not established by this adapter.Earlier full local e2e runs on macOS were not green: registry-negative cases produce timeouts or HTTP 502 instead of the expected HTTP 401. Four matching cases also failed in an independent checkout of unchanged
mainatc73949ba7b3160a4aa5729223335c190de1528f8, using Node 24.15.0. Captured failure snapshots now normalize only transient standalone progress rows; registry error content and interactive snapshots remain checked. The full Linux CI suite passes. The earlier local baseline selection was: