Skip to content

fix(nextjs): Print unsupported Turbopack warning once - #24053

Merged
andreiborza merged 3 commits into
getsentry:developfrom
hafzism:feat/dedupe-turbopack-warning
Sep 4, 2026
Merged

fix(nextjs): Print unsupported Turbopack warning once#24053
andreiborza merged 3 commits into
getsentry:developfrom
hafzism:feat/dedupe-turbopack-warning

Conversation

@hafzism

@hafzism hafzism commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

The unsupported-Turbopack compatibility warning now appears once per Next.js command instead of once per config evaluation. This keeps the warning useful without repeating it as Next.js evaluates configuration in workers.

Root cause

Each config evaluation logged the warning independently. A process-scoped environment marker was chosen so child processes inherit the emitted state, while a new command starts cleanly without stale temporary files. Silent evaluations do not consume the warning, so a later non-silent evaluation can still surface it.

Fixes #15069

Co-Authored-By: OpenAI Codex <codex@openai.com>
@hafzism
hafzism marked this pull request as ready for review September 4, 2026 04:32
@hafzism
hafzism requested a review from a team as a code owner September 4, 2026 04:32
@hafzism
hafzism requested review from mydea and s1gr1d and removed request for a team September 4, 2026 04:32

@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 and found 1 potential issue.

Fix All in Cursor

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

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 0cd03f0. Configure here.

Comment thread packages/nextjs/test/config/withSentryConfig.test.ts
Co-Authored-By: OpenAI Codex <codex@openai.com>
Comment thread packages/nextjs/test/config/withSentryConfig.test.ts
Co-Authored-By: OpenAI Codex <codex@openai.com>

@andreiborza andreiborza left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi, looks good! Thanks for the contribution.

@andreiborza
andreiborza merged commit 52d28f3 into getsentry:develop Sep 4, 2026
143 of 145 checks passed
andreiborza added a commit that referenced this pull request Sep 4, 2026
This PR adds the external contributor to the CHANGELOG.md file, so that
they are credited for their contribution. See #24053

Co-authored-by: andreiborza <168741329+andreiborza@users.noreply.github.com>
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.

[Next.js] Print turbopack compatibility warning only once

2 participants