Fix import path in function when building on Windows

[mrstork]

Description

On Windows, run ntl build --offline on demos/vite-functions or demos/vite-edge and you will receive some version of the following error:

There was an error when loading the '...uildserverserver.js' npm module.
Support for npm modules in edge functions is an experimental feature. 
Refer to https://ntl.fyi/edge-functions-npm for more information.

This happens because export { default } from "..\..\build\server\server.js"; is generated at the top of demos/vite-functions/.netlify/functions-internal/remix-server.mjs. The backslashes are then read as escape characters, hence the '...uildserverserver.js' with missing characters in the error message.

The fix is to convert the path to posix before generating the file to create the following output instead:

export { default } from "../../build/server/server.js";

Related Tickets & Documents

• Related Issue [Bug] Functions Entrypoint generates invalid Import on Windows #275
• Closes [Bug] Functions Entrypoint generates invalid Import on Windows #275

QA Instructions, Screenshots, Recordings

1. Run ntl build --offline on a Windows machine or VM targeting the vite-functions or vite-edge demo site.

For us to review and ship your PR efficiently, please perform the following steps:

• Open a bug/issue before writing your code 🧑‍💻. This
  ensures we can discuss the changes and get feedback from everyone that should be involved. If you`re fixing a
  typo or something that`s on fire 🔥 (e.g. incident related), you can skip this step.
• Read the contribution guidelines 📖. This ensures your code follows our style
  guide and passes our tests.
• Update or add tests (if any source code was changed or added) 🧪
• Update or add documentation (if features were changed or added) 📝
• Make sure the status checks below are successful ✅

[netlify]

✅ Deploy Preview for remix-edge ready!

Name	Link
🔨 Latest commit	aecf292
🔍 Latest deploy log	https://app.netlify.com/sites/remix-edge/deploys/660d7eaa2a0e9b0008ef46ba
😎 Deploy Preview	https://deploy-preview-281--remix-edge.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[netlify]

✅ Deploy Preview for remix-serverless ready!

Name	Link
🔨 Latest commit	aecf292
🔍 Latest deploy log	https://app.netlify.com/sites/remix-serverless/deploys/660d7eaa03793000089e31c2
😎 Deploy Preview	https://deploy-preview-281--remix-serverless.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[Skn0tt]

Looks perfect! Ideally, we'd also add a test for this, but it looks like we don't have a test setup in this repository yet.

[pschaeffer]

As best I can tell, the problem still exists. See the attached files. Note that I am using 'netlify serve' (without the quotes) to get the error.

[Skn0tt]

Peter, could you provide a minimal reproduction repository where this is broken?

[pschaeffer]

I am assuming you are referring to my-remix-udemy-section3-netlify4, not my-remix-udemy-section3-netlify3. Both 'netlify dev' and 'netlify serve' work great with my-remix-udemy-section3-netlify4. I upgraded to remix-adapter (2.3.1) and my problems with my-remix-udemy-section3-netlify4 went away.

[pschaeffer]

Thank you folks for all of your help