Skip to content

fix(ci): use pnpm for publishing - #9052

Merged
avivkeller merged 2 commits into
mainfrom
pnpm-publish
Jul 30, 2026
Merged

fix(ci): use pnpm for publishing#9052
avivkeller merged 2 commits into
mainfrom
pnpm-publish

Conversation

@avivkeller

@avivkeller avivkeller commented Jul 30, 2026

Copy link
Copy Markdown
Member

Signed-off-by: Aviv Keller <me@aviv.sh>
@avivkeller
avivkeller requested a review from a team as a code owner July 30, 2026 19:54
Copilot AI review requested due to automatic review settings July 30, 2026 19:54

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vercel

vercel Bot commented Jul 30, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nodejs-org Ready Ready Preview Jul 30, 2026 8:01pm

Request Review

@cursor

cursor Bot commented Jul 30, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Touches the release/publish path for public packages; a misconfiguration could affect npm publishes, though the change is limited to one script line.

Overview
Updates the root release script so publishing runs through pnpm publish -r --access=public instead of changeset publish, after the existing filtered package build step.

This matches pnpm’s recommended Changesets release flow and keeps public registry access explicit on the publish command.

Reviewed by Cursor Bugbot for commit 7a8feeb. Bugbot is set up for automated code reviews on this repo. Configure here.

@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.09%. Comparing base (0ab03a2) to head (7a8feeb).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9052      +/-   ##
==========================================
- Coverage   75.13%   75.09%   -0.05%     
==========================================
  Files         102      102              
  Lines        9105     9105              
  Branches      372      372              
==========================================
- Hits         6841     6837       -4     
- Misses       2260     2264       +4     
  Partials        4        4              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@cursor cursor Bot 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.

Cursor Bugbot has reviewed your changes using default effort and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 074d07a. Configure here.

Comment thread package.json Outdated
Comment thread package.json Outdated
Signed-off-by: Aviv Keller <me@aviv.sh>
Copilot AI review requested due to automatic review settings July 30, 2026 20:00

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@avivkeller
avivkeller enabled auto-merge July 30, 2026 20:05
@avivkeller
avivkeller added this pull request to the merge queue Jul 30, 2026
@github-actions

Copy link
Copy Markdown
Contributor

📦 Build Size Comparison

Summary

Metric Value
Old Total First Load JS 7.16 MB
New Total First Load JS 7.16 MB
Delta 0 B (0.00%)

Merged via the queue into main with commit efbdb7b Jul 30, 2026
16 checks passed
@avivkeller
avivkeller deleted the pnpm-publish branch July 30, 2026 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track Fast Tracking PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants