feat: added support for dynamic imports

src/index.ts

@@ -94,10 +94,23 @@ export default function externalAssets(pattern: FilterPattern): Plugin {
 		async renderChunk(code, chunk, outputOptions) {
 			const chunk_id = getOutputId(chunk.fileName, outputOptions);
 			const chunk_basename = path.basename(chunk_id);
-
-			const ast = parse(code, { sourceFileName: chunk_basename });
 			const rollup_context = this;
 
+			const customParser = {
+				parse(source: string) {
+					// Use rollup's internal acorn instance to parse `source`.
+					return rollup_context.parse(source, {
+						ecmaVersion: "latest",
+						locations: true, // Needed by `recast` to preserve code formatting.
+					});
+				},
+			};
+
+			const ast = parse(code, {
+				sourceFileName: chunk_basename,
+				parser: customParser,
+			});
+
 			visit(ast, {
 				visitLiteral(nodePath) {
 					const value = nodePath.node.value;

tests/fixtures/src/index5.js

@@ -0,0 +1,9 @@
+import text from "./file1";
+import sayHi from "./file2";
+
+import("../assets/image.png")
+	.then(png => console.log(png))
+	.catch(err => console.error(err));
+
+console.log(text);
+sayHi();

tests/output.test.js

@@ -148,6 +148,11 @@ test("treeshake.moduleSideEffects = false", outputSnapshotMacro,
 	}
 );
 
-// TODO: recast throws an error when a chunk contains dynamic imports,
-// I tried to use acorn instead of the default esprima parser, but the error is still the same.
-// I'll invistigate the source of that error later, but for now I'll only test static imports.
+test("dynamic imports", outputSnapshotMacro,
+	{
+		input: "tests/fixtures/src/index5.js",
+		plugins: [
+			externalAssets("tests/fixtures/assets/*"),
+		],
+	}
+);