fix(sourcemap): preserve original sourcesContent

[sun0day]

Description

• fix missing source files
  • fixes @rollup/plugin-inject incompatible despite being in the list #12916 (The @rollup/plugin-inject plugin causes null print path for the console #13657 happens though)
  • refs Vite chokes outputting sourcemaps for modules with query parameters when there are certain kinds of plugins outputting sourcemaps from transform #11860 (not fixed, similar to The @rollup/plugin-inject plugin causes null print path for the console #13657)
  • fixes Lazy loaded React-Router reports sourcemap issue #11238
  • fixes Sourcemap for '...bootstrap.min.css' points to missing source files #10068
  • close fix(pluginContainer): should not exclude client sourcemap source content #13092
• refs fix(pluginContainer): should not exclude client sourcemap source content #13092 (comment)
• discuss https://discord.com/channels/804011606160703521/1106421226689265684

Additional context

• excludeContent needs to be false unconditionally to pass sourcesContent all along the way.

  • Personally, I think there is no need to include the source content when source file exists. But we could exclude it for now in 4.4 beta to see how it works.

---

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.

[stackblitz]

Run & review this pull request in StackBlitz Codeflow.

[sun0day]

@sapphi-red cc

[sapphi-red]

Thank you!
I tested this PR on some issues' reproductions and added the results on the OP.

[sapphi-red]

Personally, I think there is no need to include the source content when source file exists.

We need to include it for now. If we exclude it, the browser will fetch the content. But because Vite cannot differentiate that request and a normal JS import request, Vite returns a transformed request. This results in showing a transformed file as a original source (related: #13391).

[patak-dev]

Let's merge it and do one more beta patch before releasing 4.4. cc @bmeurer