fix(un-ts#123): uses exports of ExportMap

SukkaW · Aug 29, 2024 · 452cb13 · 452cb13
1 parent b56f705
commit 452cb13
Showing 1 changed file with 32 additions and 28 deletions.
diff --git a/src/rules/no-named-as-default.ts b/src/rules/no-named-as-default.ts
@@ -21,44 +21,48 @@ export = createRule<[], MessageId>({
   },
   defaultOptions: [],
   create(context) {
-    function checkDefault(
+    function createCheckDefault(
       nameKey: 'local' | 'exported',
-      defaultSpecifier: TSESTree.ImportDefaultSpecifier,
-      // | TSESTree.ExportDefaultSpecifier,
     ) {
-      // #566: default is a valid specifier
-      // @ts-expect-error - ExportDefaultSpecifier is unavailable yet
-      const nameValue = defaultSpecifier[nameKey].name as string
+      return function checkDefault(
+        defaultSpecifier: TSESTree.ImportDefaultSpecifier,
+        // | TSESTree.ExportDefaultSpecifier,
+      ) {
+        // #566: default is a valid specifier
+        // @ts-expect-error - ExportDefaultSpecifier is unavailable yet
+        const nameValue = defaultSpecifier[nameKey].name as string
 
-      if (nameValue === 'default') {
-        return
-      }
+        if (nameValue === 'default') {
+          return
+        }
 
-      const declaration = importDeclaration(context, defaultSpecifier)
+        const declaration = importDeclaration(context, defaultSpecifier)
 
-      const imports = ExportMap.get(declaration.source.value, context)
-      if (imports == null) {
-        return
-      }
+        const exportMapOfImported = ExportMap.get(declaration.source.value, context)
+        if (exportMapOfImported == null) {
+          return
+        }
 
-      if (imports.errors.length > 0) {
-        imports.reportErrors(context, declaration)
-        return
-      }
+        if (exportMapOfImported.errors.length > 0) {
+          exportMapOfImported.reportErrors(context, declaration)
+          return
+        }
 
-      if (imports.has('default') && imports.has(nameValue)) {
-        context.report({
-          node: defaultSpecifier,
-          messageId: 'default',
-          data: {
-            name: nameValue,
-          },
-        })
+        if (exportMapOfImported.exports.has('default') && exportMapOfImported.exports.has(nameValue)) {
+          console.log(nameValue, exportMapOfImported)
+          context.report({
+            node: defaultSpecifier,
+            messageId: 'default',
+            data: {
+              name: nameValue,
+            },
+          })
+        }
       }
     }
     return {
-      ImportDefaultSpecifier: checkDefault.bind(null, 'local'),
-      ExportDefaultSpecifier: checkDefault.bind(null, 'exported'),
+      ImportDefaultSpecifier: createCheckDefault('local'),
+      ExportDefaultSpecifier: createCheckDefault('exported'),
     }
   },
 })