fix: wrong path resolution in multiple eslintrc configurations (#51)

close #50
Co-authored-by: Alex Vasiuro <[email protected]>

package.json

@@ -35,10 +35,11 @@
     "lint:tsc": "tsc --incremental false --noEmit",
     "prepublishOnly": "yarn build",
     "pretest": "r",
+    "test": "run-p test:*",
+    "test:multipleEslintrcs": "eslint --ext ts,tsx tests/multipleEslintrcs",
+    "test:multipleTsconfigs": "eslint --ext ts,tsx tests/multipleTsconfigs",
     "test:withPaths": "eslint --ext ts,tsx tests/withPaths",
     "test:withoutPaths": "eslint --ext ts,tsx tests/withoutPaths",
-    "test:multipleTsconfigs": "eslint --ext ts,tsx tests/multipleTsconfigs",
-    "test": "run-p test:*",
     "type-coverage": "type-coverage --cache --detail --ignore-catch --strict"
   },
   "peerDependencies": {

src/index.ts

@@ -111,6 +111,7 @@ function packageFilter(pkg: Record<string, string>) {
   return pkg
 }
 
+let mappersBuildForOptions: TsResolverOptions
 let mappers:
   | Array<(source: string, file: string) => string | undefined>
   | undefined
@@ -133,7 +134,7 @@ function getMappedPath(source: string, file: string) {
 }
 
 function initMappers(options: TsResolverOptions) {
-  if (mappers) {
+  if (mappers && mappersBuildForOptions === options) {
     return
   }
 
@@ -171,6 +172,8 @@ function initMappers(options: TsResolverOptions) {
         return matchPath(source, undefined, undefined, extensions)
       }
     })
+
+  mappersBuildForOptions = options
 }
 
 function isConfigLoaderSuccessResult(

tests/multipleEslintrcs/module-a/.eslintrc.js

@@ -0,0 +1 @@
+module.exports = require('../../baseEslintConfig')(__dirname);

tests/multipleEslintrcs/module-a/index.ts

@@ -0,0 +1,15 @@
+// import relative
+import './tsImportee'
+import './tsxImportee'
+import './subfolder/tsImportee'
+import './subfolder/tsxImportee'
+
+// import using tsconfig.json path mapping
+import 'folder/tsImportee'
+import 'folder/tsxImportee'
+import 'folder/subfolder/tsImportee'
+import 'folder/subfolder/tsxImportee'
+
+// import from node_module
+import 'typescript'
+import 'dummy.js'

tests/multipleEslintrcs/module-a/subfolder/tsImportee.ts

@@ -0,0 +1 @@
+export default 'yes'

tests/multipleEslintrcs/module-a/subfolder/tsxImportee.tsx

@@ -0,0 +1 @@
+export default 'React Component'

tests/multipleEslintrcs/module-a/tsImportee.ts

@@ -0,0 +1 @@
+export default 'yes'

tests/multipleEslintrcs/module-a/tsconfig.json

@@ -0,0 +1,11 @@
+{
+  "compilerOptions": {
+    "baseUrl": ".",
+    "jsx": "react",
+    "paths": {
+      "folder/*": ["*"],
+      "*": ["../../../../node_modules/*"]
+    }
+  },
+  "files": ["index.ts", "tsImportee.ts", "tsxImportee.tsx"]
+}

tests/multipleEslintrcs/module-a/tsxImportee.tsx

@@ -0,0 +1 @@
+export default 'React Component'

tests/multipleEslintrcs/module-b/.eslintrc.js

@@ -0,0 +1 @@
+module.exports = require('../../baseEslintConfig')(__dirname);

tests/multipleEslintrcs/module-b/index.ts

@@ -0,0 +1,15 @@
+// import relative
+import './tsImportee'
+import './tsxImportee'
+import './subfolder/tsImportee'
+import './subfolder/tsxImportee'
+
+// import using tsconfig.json path mapping
+import 'folder/tsImportee'
+import 'folder/tsxImportee'
+import 'folder/subfolder/tsImportee'
+import 'folder/subfolder/tsxImportee'
+
+// import from node_module
+import 'typescript'
+import 'dummy.js'

tests/multipleEslintrcs/module-b/subfolder/tsImportee.ts

@@ -0,0 +1 @@
+export default 'yes'

tests/multipleEslintrcs/module-b/subfolder/tsxImportee.tsx

@@ -0,0 +1 @@
+export default 'React Component'

tests/multipleEslintrcs/module-b/tsImportee.ts

@@ -0,0 +1 @@
+export default 'yes'

tests/multipleEslintrcs/module-b/tsconfig.json

@@ -0,0 +1,11 @@
+{
+  "compilerOptions": {
+    "baseUrl": ".",
+    "jsx": "react",
+    "paths": {
+      "folder/*": ["*"],
+      "*": ["../../../../node_modules/*"]
+    }
+  },
+  "files": ["index.ts", "tsImportee.ts", "tsxImportee.tsx"]
+}

tests/multipleEslintrcs/module-b/tsxImportee.tsx

@@ -0,0 +1 @@
+export default 'React Component'