Compatibility issue with Nuxt 3.13.2 / Nitro 2.10.2 / PNPM : Worker init failed

[simplg]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Self-hosted/on-premise

Which SDK are you using?

@sentry/nuxt

SDK Version

8.36.0

Framework Version

Nuxt 3.13.2

Link to Sentry event

No response

Reproduction Example/SDK Setup

https://stackblitz.com/edit/nuxt-starter-j3ktug?file=sentry.client.config.ts

It seems using pnpm as default package manager with the last nuxt/nitro version, breaks dev environment. Above I have a barebone reproduction example with nuxt + sentry.
If this project is installed with "npm" instead, it works : "rm -rf node_modules && rm -rf .nuxt && npm install && npm run dev"

Steps to Reproduce

1. Run "pnpm install"
2. Run "pnpm dev"
3. Get "ERROR [worker reload] [worker init] /home/projects/nuxt-starter-j3ktug/.nuxt/dev/index.mjs failed"
4. No page rendered

Expected Result

1. Have no errors and pages should be rendered

Actual Result

Not much, just a blank page with the nuxt logo due to the worker not being initialized (failing). I'm not able to get more information about this error.

[andreiborza]

Hi @simplg, thanks for filing this. We'll take a look.

[s1gr1d]

The server-side of Sentry has a import-in-the-middle dependency and Sentry adds some code in the server index.mjs that imports import-in-the-middle/hook.mjs (a hint for node bundlers to really include this file).

As pnpm enforces stricter dependency isolation with their hoisting behavior, you can either add import-in-the-middle as a direct dependency like this:

  "dependencies": {
    "import-in-the-middle": "1.11.2"
  }

Or you add a .npmrc with this (pnpm docs here):

...but you probably don't want to do this as it defeats the purpose of the security intent of pnpm.

shamefully-hoist=true

I'll add some docs for this.

[simplg]

Yes! Thank you, it seems to work by adding the dependency.

[s1gr1d]

Perfect, I'm gonna close this issue :)

Ideally, Sentry users don't have to do this anymore in the future, so please check the docs for future changes.