fix(cjs): resolve directory import relative to parent (#42)

fixes #583
privatenumber · Jun 8, 2024 · 02d3856 · 02d3856
1 parent 89621bf
commit 02d3856
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/cjs/api/module-resolve-filename.ts b/src/cjs/api/module-resolve-filename.ts
@@ -164,7 +164,7 @@ export const createResolveFilename = (
 				 */
 				const resolved = (
 					tryExtensions(resolve, possiblePath)
-					|| tryExtensions(resolve, path.resolve(possiblePath, 'index'))
+					|| tryExtensions(resolve, `${request}${path.sep}index`)
 				);
 				if (resolved) {
 					return resolved + query;
@@ -187,7 +187,8 @@ export const createResolveFilename = (
 	} catch (error) {
 		const resolved = (
 			tryExtensions(resolve, request)
-			|| tryExtensions(resolve, path.resolve(request, 'index'))
+			// Default resolve handles resovling paths relative to the parent
+			|| tryExtensions(resolve, `${request}${path.sep}index`)
 		);
 		if (resolved) {
 			return resolved + query;

diff --git a/tests/fixtures.ts b/tests/fixtures.ts
@@ -136,6 +136,9 @@ export const files = {
 	${syntaxLowering}
 	${preserveName}
 	export const cjsContext = ${cjsContextCheck};
+
+	// Implicit directory import works outside of immedaite CWD child
+	import '../ts/'
 	`,
 
 	'json/index.json': JSON.stringify({ loaded: 'json' }),