Skip to content

fix: add ./peer/supabase-js to jsr.json exports#94

Merged
mandarini merged 2 commits into
mainfrom
fix/jsr-score-peer-export
Jul 3, 2026
Merged

fix: add ./peer/supabase-js to jsr.json exports#94
mandarini merged 2 commits into
mainfrom
fix/jsr-score-peer-export

Conversation

@mandarini

@mandarini mandarini commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Two fixes to unblock JSR publishing and improve the JSR score.

jsr.json./peer/supabase-js was present in package.json exports but missing from jsr.json, so the entrypoint was invisible to the JSR registry. Added it to keep both export maps in sync.

release.yml — JSR has been silently failing to publish since v1.0.0. PR #62 split the release workflow into two jobs to isolate npm's OIDC token from the pnpm install step (supply chain hardening after the TanStack/router incident). As part of that split, id-token: write was removed from the build job on the assumption that JSR uses a separate OIDC binding that does not require it. That assumption was wrong: JSR authenticates CI publishes via Authorization: githuboidc <token>, and the JSR CLI obtains that token from GitHub's OIDC endpoint — which GitHub only exposes when the job has id-token: write. The audience is JSR-scoped (not npm), so restoring id-token: write to the build job does not re-open the npm supply chain path. The publish-npm job remains isolated with no pnpm install. Also removes continue-on-error: true from the JSR publish step so future failures are visible.

@pkg-pr-new

pkg-pr-new Bot commented Jul 3, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/@supabase/server@94

commit: d405acb

@mandarini mandarini marked this pull request as ready for review July 3, 2026 11:17
@mandarini mandarini requested review from a team as code owners July 3, 2026 11:17
@mandarini mandarini self-assigned this Jul 3, 2026
@mandarini mandarini merged commit 04276d0 into main Jul 3, 2026
7 checks passed
@mandarini mandarini deleted the fix/jsr-score-peer-export branch July 3, 2026 11:34
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