Also warn when absolute file paths include ? during build (relates to #14165)

[poppa]

Describe the bug

As per issue #14165 you should also warn when paths include ?, since those paths will fail miserably as well.

I guess the reason is that ESM uses URLs for imports, and what follows after ? and # are not part of the path in an URL.

Paths including any other URL-reserved character seems to be resolved correctly (but don't quote me on that).

Reproduction

https://stackblitz.com/edit/vitejs-vite-vxwonj?file=main.js

Steps to reproduce

No response

System Info

I guess all systems

Used Package Manager

npm

Logs

Build error output

[plugin:vite:import-analysis] Failed to resolve import "./some?dir/file.js" from "main.js". Does the file exist?

/home/projects/vitejs-vite-vxwonj/main.js:5:23

3  |  import viteLogo from '/vite.svg';
4  |  import { setupCounter } from './counter.js';
5  |  import { Test } from './some?dir/file.js';
   |                        ^
6  |  
7  |  console.log('Test:', Test);

    at formatError (/home/projects/vitejs-vite-vxwonj/node_modules/vite/dist/node/chunks/dep-9A4-l-43.js:63814:46)
    at error (/home/projects/vitejs-vite-vxwonj/node_modules/vite/dist/node/chunks/dep-9A4-l-43.js:63808:19)
    at normalizeUrl (/home/projects/vitejs-vite-vxwonj/node_modules/vite/dist/node/chunks/dep-9A4-l-43.js:62083:33

Validations

• Follow our Code of Conduct
• Read the Contributing Guidelines.
• Read the docs.
• Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
• Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
• Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
• The provided reproduction is a minimal reproducible example of the bug.

[stackblitz]

Start a new pull request in StackBlitz Codeflow.

• vitejs-vite-vxwonj (Open reproduction)