[webpack.FileSystemInfo] dynamic import() expects string?

[luwes]

Hi,

I was wondering about this code here in Webpack:

webpack/lib/FileSystemInfo.js

Lines 1731 to 1733 in 412ae54

	// import()
	let expr = source.substring(imp.s, imp.e).trim();
	dependency = parseString(expr);

I'm running into a bug because the parsing only handles strings in the dynamic import argument it seems.
Is this correct? Webpack only handles a string, not a variable as imported URL?
Is there a workaround for this?

    [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Node.js doesn't offer a (nice) way to introspect the ESM dependency graph yet.
    Until a full solution is available webpack uses an experimental ESM tracking based on parsing.
    As best effort webpack parses the ESM files to guess dependencies. But this can lead to expensive and incorrect tracking.
<w> [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] Parsing of /Users/wes/Sites/mux/next-video/dist/config.js for build dependencies failed at 'import(fileUrl
<w>   )'.
<w> Build dependencies behind this expression are ignored and might cause incorrect cache invalidation.
    [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo]  at file dependencies /Users/wes/Sites/mux/next-video/dist/config.js
     at file /Users/wes/Sites/mux/next-video/dist/config.js
     at resolve esm file ./config.js
     at file dependencies /Users/wes/Sites/mux/next-video/dist/with-next-video.js
     at file /Users/wes/Sites/mux/next-video/dist/with-next-video.js
     at resolve esm file ./with-next-video.js
     at file dependencies /Users/wes/Sites/mux/next-video/dist/process.js
     at file /Users/wes/Sites/mux/next-video/dist/process.js
     at resolve esm file next-video/process
     at file dependencies /Users/wes/Sites/sandbox/next-video-app/next.config.mjs
     at file /Users/wes/Sites/sandbox/next-video-app/next.config.mjs
     at resolve commonjs /Users/wes/Sites/sandbox/next-video-app/next.config.mjs
    [webpack.cache.PackFileCacheStrategy/webpack.FileSystemInfo] SyntaxError: Unexpected token 'i', "fileUrl" is not valid JSON
        at JSON.parse (<anonymous>)
        at parseString (/Users/wes/Sites/sandbox/next-video-app/node_modules/next/dist/compiled/webpack/bundle5.js:28:259326)
        at /Users/wes/Sites/sandbox/next-video-app/node_modules/next/dist/compiled/webpack/bundle5.js:28:275556
        at /Users/wes/Sites/sandbox/next-video-app/node_modules/next/dist/compiled/webpack/bundle5.js:1:267649
        at FSReqCallback.readFileAfterClose [as oncomplete] (node:internal/fs/read/context:68:3)

Original issue here muxinc/next-video#176

[alexander-akait]

We can't analyze variables there

[luwes]

that makes sense.
in that case should any non-string import() argument be ignored instead of throwing?
I can help create a PR for this.

[alexander-akait]

Yeah, let's ignore them, feel free to send a PR