这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@mknichel
Copy link
Member

@mknichel mknichel commented Oct 10, 2025

Description

The microfrontends local development proxy fails to start on Windows because it runs into: %1 is not a valid Win32 application. The issue is that Turborepo runs the .js script and relies on the shebang to be run correctly. That doesn't work on Windows. This PR changes the code to run node which will work on Windows.

Testing Instructions

In a microfrontends repository, such as https://github.com/vercel-labs/microfrontends-nextjs-app-multi-zone, run pnpm turbo run dev on Windows and ensure that it starts up.

@vercel
Copy link
Contributor

vercel bot commented Oct 10, 2025

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

Project Deployment Preview Comments Updated (UTC)
examples-basic-web Building Building Preview Comment Oct 15, 2025 1:32pm
examples-designsystem-docs Ready Ready Preview Comment Oct 15, 2025 1:32pm
examples-gatsby-web Ready Ready Preview Comment Oct 15, 2025 1:32pm
examples-kitchensink-blog Ready Ready Preview Comment Oct 15, 2025 1:32pm
examples-nonmonorepo Ready Ready Preview Comment Oct 15, 2025 1:32pm
examples-svelte-web Ready Ready Preview Comment Oct 15, 2025 1:32pm
examples-tailwind-web Ready Ready Preview Comment Oct 15, 2025 1:32pm
examples-vite-web Ready Ready Preview Comment Oct 15, 2025 1:32pm
turbo-site Ready Ready Preview Comment Oct 15, 2025 1:32pm

@anthonyshew anthonyshew marked this pull request as ready for review October 15, 2025 13:28
@anthonyshew anthonyshew requested a review from a team as a code owner October 15, 2025 13:28
@anthonyshew anthonyshew requested a review from tknickman October 15, 2025 13:28
@anthonyshew
Copy link
Contributor

Thanks, @mknichel! Tried out the original proposal and it broke macOS, so I'm going to try a slightly different approach that only changes the semantics of the binary call on Windows. Let's see how that does.

@anthonyshew anthonyshew enabled auto-merge (squash) October 15, 2025 13:32
@anthonyshew anthonyshew changed the title fix: use node to run microfrontends proxy on Windows fix: adjust binary call for microfrontends proxy on Windows Oct 15, 2025
@anthonyshew
Copy link
Contributor

The Windows failure seen in CI is not related to this branch. It can be observed on other branches as well. Merging past the failure; we will fix in another PR.

@anthonyshew anthonyshew disabled auto-merge October 15, 2025 14:53
@anthonyshew anthonyshew merged commit 14a9808 into main Oct 15, 2025
55 of 59 checks passed
@anthonyshew anthonyshew deleted the markknichel/mfes-1133-looks-like-the-microfrontends-proxy-doesnt-work-on-windows branch October 15, 2025 14:53
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.

4 participants