Skip to content

feat(web): switch build and dev back to Turbopack - #4119

Merged
amitsingh-007 merged 1 commit into
mainfrom
feat-web-turbopack-build
Aug 1, 2026
Merged

feat(web): switch build and dev back to Turbopack#4119
amitsingh-007 merged 1 commit into
mainfrom
feat-web-turbopack-build

Conversation

@amitsingh-007

@amitsingh-007 amitsingh-007 commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Check if the Pull Request fulfils these requirements

  • Does the extension require a version change?

🤖 Generated with Claude Code

Greptile Summary

Switches the web application’s development and production builds from webpack back to Next.js’s default Turbopack bundler.

  • Removes the explicit --webpack flags from the web build and development scripts.
  • Updates configuration comments explaining the TypeScript CLI checker requirement.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The pinned Next.js version and documented validation address the prior Turbopack tracing failure, while the remaining configuration and script changes are internally consistent.

Reviews (1): Last reviewed commit: "feat(web): switch build and dev back to ..." | Re-trigger Greptile

fd0f1e5 pinned the web build to `next build --webpack` because
/api/upload-file 500'd on Vercel with ERR_DLOPEN_FAILED for sharp's
libvips. Root cause was @vercel/nft's sharp special case being gated on
`sharp/lib/index.js`, which sharp 0.35 removed, so @img/sharp-libvips-*
never got traced into the lambda (lovell/sharp#4567, vercel/next.js#96064).
Fixed upstream in vercel/next.js#95507, which 16.3.0-preview.10 includes.

Verified the Turbopack trace locally: the upload-file nft.json now lists
libvips-cpp.8.18.3 and 47 sharp files (webpack traced 38), and both
hash-suffixed aliases carry their store target files, so the
vercel/next.js#87737 failure mode is absent too. Persons e2e passes 17/17,
including both image-upload specs.

0c2b412 had put dev on --webpack only to match build, so it follows.

Also drop the stale "Requires Next >= 16.3" note from useTypeScriptCli and
keep it marked TODO with its removal condition.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

@sourcery-ai sourcery-ai 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.

Hey - I've reviewed your changes and they look great!


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@amitsingh-007
amitsingh-007 merged commit 6a910a2 into main Aug 1, 2026
6 checks passed
@amitsingh-007
amitsingh-007 deleted the feat-web-turbopack-build branch August 1, 2026 18:26
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.

1 participant