fix(win): use windows-safe paths

[avivkeller]

Currently, doc-kit does not work on Windows due to a Rolldown issue, and a paths issue.

This PR fixes the paths issue by using Windows-safe paths.

Additionally,

• When not minified, there is a chance that the code will end with an //#endregion comment. We need to add a newline before the return
• We set the variable code = to to dehydration, not our generated variable. TBH, I'm not sure how this worked at all previously...

@TheAlexLichter There appears to be a bug when building this on Windows:
It appears to import everything twice. Once using a Unix path, and once using a Windows path.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
api-docs-tooling	✅ Ready (Inspect)	Visit Preview	Jul 19, 2025 10:02pm

[Copilot]

Pull Request Overview

This PR fixes Windows compatibility issues in doc-kit by replacing new URL().pathname with Windows-safe path resolution using POSIX paths. The changes also correct variable naming inconsistencies and improve code generation formatting.

• Introduces Windows-safe path handling using node:path/posix and a computed ROOT constant
• Fixes variable naming bug where server-side rendering used incorrect variable references
• Updates external dependencies configuration and improves code formatting

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
src/generators/web/constants.mjs	Adds ROOT constant calculation and replaces URL.pathname with resolve() calls for Windows compatibility
src/generators/web/utils/generate.mjs	Updates import path resolution and fixes variable naming in server program generation
src/generators/web/utils/processing.mjs	Moves SSR variable declaration to module scope and fixes variable reference bug
src/generators/web/utils/bundle.mjs	Adds 'preact-render-to-string' to external dependencies list

[TheAlexLichter]

When not minified, there is a chance that the code will end with an //#endregion comment. We need to add a newline before the return

You can disable region comments via experimental.attachDebugData (rolldown/rolldown#4918)

@TheAlexLichter There appears to be a bug when building this on Windows: It appears to import everything twice. Once using a Unix path, and once using a Windows path.

Oh 👀
Would appreciate a minimal reproduction & issue in the https://github.com/rolldown/rolldown/ repo if possible 🙏🏻

[avivkeller]

Would appreciate a minimal reproduction & issue in the https://github.com/rolldown/rolldown/ repo if possible 🙏🏻

Will do! I'll work on getting a minimal repro of this, and the other issue we spoke about, today.

[avivkeller]

@nodejs/web-infra requesting fast track, as this is used in a reproduction for Rolldown.