fix(ssr): load sourcemaps alongside modules

[sapphi-red]

Description

This PR reapplies #11576 with a fix and tests.
Also this PR adds some comments why this works.

fixes #3288

cc @patak-dev @Vap0r1ze @benmccann

Additional context

#11576 requires source map support for new Function which landed in Node.js "^16.17.0 || >=18.6.0".
Adding the offset seems to be required due to V8's bug (nodejs/node#43047 (comment)). When the bug is fixed, that part of the code needs to be removed.

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

Before submitting the PR, please make sure you do the following

• Read the Contributing Guidelines.
• Read the Pull Request Guidelines and follow the PR Title Convention.
• Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
• Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
• Ideally, include relevant tests that fail without this PR but pass with it.

[sapphi-red]

/ecosystem-ci run

[vite-ecosystem-ci]

📝 Ran ecosystem CI: Open

suite	result
astro	✅ success
histoire	❌ failure
iles	✅ success
ladle	✅ success
laravel	✅ success
marko	✅ success
nuxt-framework	✅ success
previewjs	❌ failure
qwik	❌ failure
rakkas	✅ success
sveltekit	✅ success
vite-plugin-ssr	✅ success
vite-plugin-react	✅ success
vite-plugin-react-pages	❌ failure
vite-plugin-react-swc	✅ success
vite-plugin-svelte	✅ success
vite-plugin-vue	✅ success
vite-setup-catalogue	✅ success
vitepress	✅ success
vitest	✅ success
windicss	✅ success

[Vap0r1ze]

Will this work if I set a breakpoint on the first line of a file?

[sapphi-red]

I think it doesn't. I guess we can append 'use strict' in ssrTransformScript. But I'm not sure if we can introduce that change in a minor version bump.
related: #9199 (comment)

[sapphi-red]

This won't work if something called process.setSourceMapsEnabled after this line. (I mean the isSourceMapEnabled will not be synced with the actual value.)

I've created a feature request to Node to fix this: nodejs/node#46304

[Vap0r1ze]

Ah I see that makes sense, that's fine I think

[xania]

Hi, I've added a fix to the PR, you can check it out in
sapphi-red#5

[patak-dev]

Let's merge this one so we can test it during the current beta.

[mengdu]

How to break points in vscode? Doesn't work on JavaScript Debug Terminal