Skip to content

create-next-app: Add ignoreScripts for unrs-resolver on bun#86324

Merged
bgw merged 1 commit intocanaryfrom
bgw/bun-ignore-scripts
Nov 19, 2025
Merged

create-next-app: Add ignoreScripts for unrs-resolver on bun#86324
bgw merged 1 commit intocanaryfrom
bgw/bun-ignore-scripts

Conversation

@bgw
Copy link
Member

@bgw bgw commented Nov 19, 2025

unrs-resolver's postinstall script is only needed for legacy versions of npm, so it's safe to ignore: unrs/unrs-resolver#193 (comment)

Without this, bun blocks the postinstall script (okay) and logs a warning (not ideal, confusing to end users). This prevents it from running and suppresses the warning. This matches what we now do in pnpm (#83168) but for bun.

Context:

Before

CleanShot 2025-11-19 at 08.46.56@2x.png

After

Screenshot 2025-11-19 at 1.02.01 PM.png

Copy link
Member Author

bgw commented Nov 19, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@bgw bgw marked this pull request as ready for review November 19, 2025 21:05
@@ -340,6 +340,19 @@ export const installTemplate = async ({
);
Copy link
Member Author

Choose a reason for hiding this comment

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

More context up here!

@bgw bgw requested a review from icyJoseph November 19, 2025 21:10
@bgw bgw requested a review from eps1lon November 19, 2025 21:10
@bgw bgw merged commit d53a7a7 into canary Nov 19, 2025
289 of 292 checks passed
@bgw bgw deleted the bgw/bun-ignore-scripts branch November 19, 2025 22:34
Moroshima pushed a commit to Moroshima/next.js that referenced this pull request Nov 20, 2025
…el#86324)

`unrs-resolver`'s postinstall script is only needed for legacy versions
of npm, so it's safe to ignore:
unrs/unrs-resolver#193 (comment)

Without this, bun blocks the postinstall script (okay) and logs a
warning (not ideal, confusing to end users). This prevents it from
running and suppresses the warning. This matches what we now do in
`pnpm` (vercel#83168) but for `bun`.

Context:
- https://vercel.slack.com/archives/C03KAR5DCKC/p1763570835510729
- https://vercel.slack.com/archives/C06DNAH5LSG/p1763580178004169

## Before

![CleanShot 2025-11-19 at
08.46.56@2x.png](https://app.graphite.com/user-attachments/assets/4979ca3c-0a05-4c05-831a-93c358832ed4.png)

## After

![Screenshot 2025-11-19 at
1.02.01 PM.png](https://app.graphite.com/user-attachments/assets/f99b419b-a30e-449a-9118-77621b15fcd9.png)
@github-actions github-actions bot added the locked label Dec 4, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants