Skip to content

fix(v10/nuxt): Windows file:// for import-in-the-middle hook and isAbsolute for C:\ - #24026

Merged
s1gr1d merged 2 commits into
v10from
sig/v10-nuxt-filepath
Sep 4, 2026
Merged

fix(v10/nuxt): Windows file:// for import-in-the-middle hook and isAbsolute for C:\#24026
s1gr1d merged 2 commits into
v10from
sig/v10-nuxt-filepath

Conversation

@s1gr1d

@s1gr1d s1gr1d commented Sep 3, 2026

Copy link
Copy Markdown
Member

Backport of #23653

halillusion and others added 2 commits September 3, 2026 17:51
…te for C:\ (#23653)

- [x] tests added if needed
- [x] yarn lint passes
- [ ] no related issue, auto-link is fine

Windows fix for Nuxt dev on Node 24.

On Windows `npm run dev` was failing with
`ERR_UNSUPPORTED_ESM_URL_SCHEME` / `Received protocol 'c:'` because
`.nuxt/dev/index.mjs` generated `import 'C:\...'` instead of
`file:///C:/...`. Same `isExternal` check in `rollup.module.config.mjs`
treated `C:\` as external and broke the build.

Changed `addServerConfig.ts` (and the solidstart copy) to emit
`pathToFileURL(...).href` and fixed `isExternal` to use `isAbsolute()`
so Windows absolute paths work. Also handles POSIX fine.

Verified with `yarn nx run @sentry/nuxt:build:dev` -> now passes, and
`pathToFileURL` gives `file:///C:/...` on Windows.

Closes #XXXX

---------

Co-authored-by: Sigrid <32902192+s1gr1d@users.noreply.github.com>
@s1gr1d
s1gr1d requested a review from a team as a code owner September 3, 2026 16:06
@s1gr1d
s1gr1d requested review from chargome and mydea and removed request for a team September 3, 2026 16:06
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 27.76 kB added added
@sentry/browser - with treeshaking flags 26.2 kB added added
@sentry/browser (incl. Tracing) 46.64 kB added added
@sentry/browser (incl. Tracing + Span Streaming) 48.42 kB added added
@sentry/browser (incl. Tracing, Profiling) 51.39 kB added added
@sentry/browser (incl. Tracing, Replay) 85.9 kB added added
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 75.52 kB added added
@sentry/browser (incl. Tracing, Replay with Canvas) 90.62 kB added added
@sentry/browser (incl. Tracing, Replay, Feedback) 103.27 kB added added
@sentry/browser (incl. Feedback) 44.94 kB added added
@sentry/browser (incl. sendFeedback) 32.56 kB added added
@sentry/browser (incl. FeedbackAsync) 37.69 kB added added
@sentry/browser (incl. Metrics) 28.85 kB added added
@sentry/browser (incl. Logs) 29.07 kB added added
@sentry/browser (incl. Metrics & Logs) 29.77 kB added added
@sentry/react 29.57 kB added added
@sentry/react (incl. Tracing) 48.9 kB added added
@sentry/vue 33.19 kB added added
@sentry/vue (incl. Tracing) 48.61 kB added added
@sentry/svelte 27.79 kB added added
CDN Bundle 30.15 kB added added
CDN Bundle (incl. Tracing) 48.6 kB added added
CDN Bundle (incl. Logs, Metrics) 31.73 kB added added
CDN Bundle (incl. Tracing, Logs, Metrics) 49.9 kB added added
CDN Bundle (incl. Replay, Logs, Metrics) 71.02 kB added added
CDN Bundle (incl. Tracing, Replay) 86.12 kB added added
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 87.43 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) 91.92 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 93.2 kB added added
CDN Bundle - uncompressed 89.89 kB added added
CDN Bundle (incl. Tracing) - uncompressed 146.87 kB added added
CDN Bundle (incl. Logs, Metrics) - uncompressed 94.6 kB added added
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 150.85 kB added added
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 219.43 kB added added
CDN Bundle (incl. Tracing, Replay) - uncompressed 266.17 kB added added
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 270.14 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 279.87 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 283.83 kB added added
@sentry/nextjs (client) 51.44 kB added added
@sentry/sveltekit (client) 47.06 kB added added
@sentry/core/server 80.61 kB added added
@sentry/core/browser 66.77 kB added added
@sentry/node-core 63.28 kB added added
@sentry/node 125.81 kB added added
@sentry/node (incl. diagnostics channel injection) 169.89 kB added added
@sentry/node/import (ESM hook with diagnostics-channel injection) 166 B added added
@sentry/node/light 51.45 kB added added
@sentry/node - without tracing 74.97 kB added added
@sentry/aws-serverless 84.23 kB added added
@sentry/cloudflare (withSentry) - minified 202.61 kB added added
@sentry/cloudflare (withSentry) 498.9 kB added added

@mydea mydea changed the title fix(nuxt): Windows file:// for import-in-the-middle hook and isAbsolute for C:\ fix(v10/nuxt): Windows file:// for import-in-the-middle hook and isAbsolute for C:\ Sep 4, 2026
@s1gr1d
s1gr1d merged commit 2ce0ea1 into v10 Sep 4, 2026
49 checks passed
@s1gr1d
s1gr1d deleted the sig/v10-nuxt-filepath branch September 4, 2026 08:23
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.

3 participants