fix(dynamic import): adding interop util at EOF (#383)

privatenumber · Nov 11, 2023 · 0039b60 · 0039b60
1 parent bf033b0
commit 0039b60
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/src/utils/transform/transform-dynamic-import.ts b/src/utils/transform/transform-dynamic-import.ts
@@ -3,7 +3,9 @@ import type { RawSourceMap } from '../../source-map';
 import { parseEsm } from '../es-module-lexer';
 
 const handlerName = '___tsxInteropDynamicImport';
-const handleEsModuleFunction = `function ${handlerName}${(function (imported: Record<string, unknown>) {
+
+// Prefixed with a newline since there could be a comment on the last line (e.g. sourcemap comment)
+const handleEsModuleFunction = `\nfunction ${handlerName}${(function (imported: Record<string, unknown>) {
 	const d = 'default';
 	const exports = Object.keys(imported);
 	if (

diff --git a/tests/specs/smoke.ts b/tests/specs/smoke.ts
@@ -417,7 +417,9 @@ export default testSuite(async ({ describe }, { tsx }: NodeApis) => {
 						}));
 		
 						// Could .js import TS files?
-						`,
+
+						// Comment at EOF: could be a sourcemap declaration. Edge case for inserting functions here
+						`.trim(),
 					});
 					onTestFinish(async () => await fixture.rm());
 
@@ -591,7 +593,9 @@ export default testSuite(async ({ describe }, { tsx }: NodeApis) => {
 							pkgCommonjs,
 							pkgModule,
 						}));
-						`,
+
+						// Comment at EOF: could be a sourcemap declaration. Edge case for inserting functions here
+						`.trim(),
 					});
 					onFinish(async () => await fixture.rm());