fix: disable twoslash for cloudflare build#8920
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryLow Risk Overview The VFS Twoslash path is removed for this target: Other deployment targets (e.g. default/Vercel) are unchanged and still use Twoslash. Reviewed by Cursor Bugbot for commit 15cb87e. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
Why? I'm not sure I follow removing functionality instead of fixing it... And, was this PR generated with AI assistance that you haven't declared? Your PR body originally had escaped newline characters which is usually a tell-tale sign of that. |
Is fix of other PR |
|
I don't think removing functionality is particularly a fix? |
|
That generation was specifically added because it works on CF deployments. Removing it would not make sense. |
Targets #8917.
The Cloudflare OpenNext build fails while collecting page data because the Cloudflare MDX config imports a VFS Twoslash setup, which pulls Node.js filesystem modules into the Worker bundle. This keeps syntax highlighting enabled on Cloudflare but disables Twoslash for that deployment target, and removes the now-unused VFS generator/dependencies.