fix(esm): resolve absolute paths (#544)

Co-authored-by: Hiroki Osame <hiroki.osame@gmail.com>
privatenumber · May 6, 2024 · 3a0ea18 · 3a0ea18
1 parent b407435
commit 3a0ea18
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/esm/hook/resolve.ts b/src/esm/hook/resolve.ts
@@ -123,6 +123,7 @@ export const resolve: resolve = async (
 
 	const isPath = (
 		specifier.startsWith(fileProtocol)
+		|| path.isAbsolute(specifier)
 		|| isRelativePathPattern.test(specifier)
 	);
 

diff --git a/tests/specs/smoke.ts b/tests/specs/smoke.ts
@@ -502,6 +502,7 @@ export default testSuite(async ({ describe }, { tsx }: NodeApis) => {
 						import './js/index.js?query=123';
 						import './js/index';
 						import './js/';
+						import { FIXTURE_PATH };
 
 						// No double .default.default in Dynamic Import
 						import('./js/index.js').then(m => {
@@ -638,6 +639,17 @@ export default testSuite(async ({ describe }, { tsx }: NodeApis) => {
 						// Comment at EOF: could be a sourcemap declaration. Edge case for inserting functions here
 						`.trim(),
 					});
+
+					const importFromTs = await fixture.readFile('import-from-ts.ts', 'utf8');
+					const importFromTsWithAbsolutePath = importFromTs.toString().replace(
+						'{ FIXTURE_PATH }',
+						JSON.stringify(
+							packageType === 'module'
+								? new URL('js/index.js', pathToFileURL(`${fixture.path}/`)).toString()
+								: path.resolve(fixture.path, 'js/index.js'),
+						),
+					);
+					await fixture.writeFile('import-from-ts.ts', importFromTsWithAbsolutePath);
 					onFinish(async () => await fixture.rm());
 
 					test('import all', async ({ onTestFail }) => {