fix: dynamic import warning with @vite-ignore

[patak-dev]

Description

#7357 update es-module-lexer to 0.10.1 (changelog), and we missed that the @vite-ignore change was broken after this.

---

What is the purpose of this pull request?

• Bug fix
• New Feature
• Documentation update
• Other

[bluwy]

Ah thanks for catching this

[adamdehaven]

@patak-dev this issue has surfaced again.

Related code:

await import(/* @vite-ignore */ url)

As of vite@4.0.4

[aevitas1]

Is there any workaround for this?

One of my pages gets 100+ errors now when it's loaded.

[patak-dev]

Please create a new issue with a minimal reproduction against latest, and you can link this PR as reference. Your reports will not be correctly tracked if you comment on a closed pull request.

[jcalfee]

I put the /* @vite-ignore */ after the code withing the "import()" call and see this warning is still happening in ^2.1.6. Until we have a reproducible repository, maybe there should be a way to disable the warning all together until it is stable.

[dev:solid] 146|  async function importMigration(folder, fileName) {
[dev:solid] 147|    const mg = (await import(resolveFrom(process.cwd(), folder + '/' + fileName) /* @vite-ignore */)).default;
[dev:solid]    |                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[dev:solid] 148|    if (typeof mg !== 'function') {
[dev:solid] 149|      throw new Error(`Expecting "export default [async] function([ddlClient])" in ${fileName}`);
[dev:solid] The above dynamic import cannot be analyzed by Vite.
[dev:solid] See https://github.com/rollup/plugins/tree/master/packages/dynamic-import-vars#limitations for supported dynamic import formats. If this is intended to be left as-is, you can use the /* @vite-ignore */ comment inside the import() call to suppress this warning.
[dev:solid] 
[dev:solid]   Plugin: vite:import-analysis