fix(#123): make no-named-as-default ignores TS namespace

package.json

@@ -89,11 +89,13 @@
     "@types/is-glob": "^4.0.4",
     "@types/jest": "^29.5.12",
     "@types/json-schema": "^7.0.15",
+    "@types/klaw-sync": "^6.0.5",
     "@types/node": "^20.11.30",
     "@typescript-eslint/eslint-plugin": "^8.1.0",
     "@typescript-eslint/parser": "^8.1.0",
     "@typescript-eslint/rule-tester": "^8.1.0",
     "@unts/patch-package": "^8.0.0",
+    "classnames": "^2.5.1",
     "cross-env": "^7.0.3",
     "enhanced-resolve": "^5.16.0",
     "escope": "^4.0.0",
@@ -114,13 +116,15 @@
     "eslint9": "npm:eslint@^9.8.0",
     "hermes-eslint": "^0.23.1",
     "jest": "^29.7.0",
+    "klaw-sync": "^6.0.0",
     "npm-run-all2": "^6.1.2",
     "prettier": "^3.2.5",
     "redux": "^5.0.1",
     "rimraf": "^5.0.10",
     "svelte": "^4.2.12",
     "ts-node": "^10.9.2",
     "type-fest": "^4.14.0",
-    "typescript": "^5.5.4"
+    "typescript": "^5.5.4",
+    "zod": "^3.23.8"
   }
 }

test/package.spec.ts

@@ -52,7 +52,6 @@ describe('package', () => {
         continue
       }
       for (const rule of Object.keys(config.rules)) {
-        console.log({ rule, preamble })
         expect(() =>
           require(getRulePath(rule.slice(preamble.length))),
         ).not.toThrow()

test/rules/no-named-as-default.spec.ts

@@ -6,17 +6,54 @@ import rule from 'eslint-plugin-import-x/rules/no-named-as-default'
 
 const ruleTester = new TSESLintRuleTester()
 
-console.log({ babel: require(parsers.BABEL) })
+const ALL_EXTENSIONS = [
+  ".ts", ".cts", ".mts", ".tsx",
+  ".js",
+  ".cjs",
+  ".mjs",
+  ".jsx",
+];
 
 ruleTester.run('no-named-as-default', rule, {
   valid: [
+    // https://github.com/un-ts/eslint-plugin-import-x/issues/123
+    'import klawSync from "klaw-sync";',
+    test({
+      code: `/** TypeScript */ import klawSync from "klaw-sync";`,
+      settings: {
+        "import-x/extensions": [
+          ".ts", ".cts", ".mts", ".tsx",
+          ".js", ".cjs", ".mjs", ".jsx",
+        ],
+        "import-x/external-module-folders": [
+          "node_modules",
+          "node_modules/@types",
+        ],
+        "import-x/parsers": {
+          "@typescript-eslint/parser": [".ts", ".cts", ".mts", ".tsx"],
+        },
+        "import-x/resolver": {
+          typescript: true,
+          node: {
+            extensions: [
+              ".ts", ".cts", ".mts", ".tsx",
+              ".js", ".cjs", ".mjs", ".jsx",
+            ],
+          },
+        },
+      },
+    }),
+
+    'import ts from "typescript";',
+    `import classNames from 'classnames';`,
+
     test({
       code: 'import "./malformed.js"',
       languageOptions: { parser: require(parsers.ESPREE) },
     }),
 
-    test({ code: 'import bar, { foo } from "./bar";' }),
-    test({ code: 'import bar, { foo } from "./empty-folder";' }),
+    'import bar, { foo } from "./bar";',
+    'import bar, { foo } from "./empty-folder";',
 
     // es7
     test({
@@ -133,5 +170,16 @@ ruleTester.run('no-named-as-default', rule, {
         parserOptions: { ecmaVersion: 2022 },
       },
     }),
+
+    test({
+      code: `import z from 'zod';`,
+      errors: [
+        {
+          message:
+            "Using exported name 'z' as identifier for default export.",
+          type: 'ImportDefaultSpecifier',
+        },
+      ],
+    })
   ],
 })

test/utils.ts

@@ -109,29 +109,29 @@ export function testContext(settings?: PluginSettings) {
  * to crash at runtime
  */
 export const SYNTAX_CASES = [
-  test({ code: 'for (let { foo, bar } of baz) {}' }),
-  test({ code: 'for (let [ foo, bar ] of baz) {}' }),
+  'for (let { foo, bar } of baz) {}',
+  'for (let [ foo, bar ] of baz) {}',
 
-  test({ code: 'const { x, y } = bar' }),
+  'const { x, y } = bar',
   test({
     code: 'const { x, y, ...z } = bar',
     languageOptions: { parser: require(parsers.BABEL) },
   }),
 
   // all the exports
-  test({ code: 'let x; export { x }' }),
-  test({ code: 'let x; export { x as y }' }),
+  'let x; export { x }',
+  'let x; export { x as y }',
 
   // not sure about these since they reference a file
-  // test({ code: 'export { x } from "./y.js"'}),
-  // test({ code: 'export * as y from "./y.js"', languageOptions: { parser: require(parsers.BABEL) } }),
+  // 'export { x } from "./y.js"'}),
+  // 'export * as y from "./y.js"', languageOptions: { parser: require(parsers.BABEL) } }),
 
-  test({ code: 'export const x = null' }),
-  test({ code: 'export var x = null' }),
-  test({ code: 'export let x = null' }),
+  'export const x = null',
+  'export var x = null',
+  'export let x = null',
 
-  test({ code: 'export default x' }),
-  test({ code: 'export default class x {}' }),
+  'export default x',
+  'export default class x {}',
 
   // issue #267: parser opt-in extension list
   test({

yarn.lock

@@ -2285,6 +2285,13 @@
   resolved "https://registry.yarnpkg.com/@types/json-schema/-/json-schema-7.0.15.tgz#596a1747233694d50f6ad8a7869fcb6f56cf5841"
   integrity sha512-5+fP8P8MFNC+AyZCDxrB2pkZFPGzqQWUzpSeuuVLvm8VMcorNYavBqoFcxK8bQz4Qsbn4oUEEem4wDLfcysGHA==
 
+"@types/klaw-sync@^6.0.5":
+  version "6.0.5"
+  resolved "https://registry.yarnpkg.com/@types/klaw-sync/-/klaw-sync-6.0.5.tgz#0a87fa0762673a1a1d2e7b08a51d5d65f0c842cd"
+  integrity sha512-xlavCRyu5ibDjsOc7PSgeUbwOBZdnJsND8gFUVfBilplbBIWhLZVjwtqbZq0327ny3jNt7oviEh5NS9a4LudeQ==
+  dependencies:
+    "@types/node" "*"
+
 "@types/minimist@^1.2.0":
   version "1.2.5"
   resolved "https://registry.yarnpkg.com/@types/minimist/-/minimist-1.2.5.tgz#ec10755e871497bcd83efe927e43ec46e8c0747e"
@@ -2968,6 +2975,11 @@ cjs-module-lexer@^1.0.0:
   resolved "https://registry.yarnpkg.com/cjs-module-lexer/-/cjs-module-lexer-1.2.3.tgz#6c370ab19f8a3394e318fe682686ec0ac684d107"
   integrity sha512-0TNiGstbQmCFwt4akjjBg5pLRTSyj/PkWQ1ZoO2zntmg9yLqSRxwEa4iCfQLGjqhiqBfOJa7W/E8wfGrTDmlZQ==
 
+classnames@^2.5.1:
+  version "2.5.1"
+  resolved "https://registry.yarnpkg.com/classnames/-/classnames-2.5.1.tgz#ba774c614be0f016da105c858e7159eae8e7687b"
+  integrity sha512-saHYOzhIQs6wy2sVxTM6bUDsQO4F50V9RQ22qBpEdCW+I+/Wmke2HOl6lS6dTpdxVhb88/I6+Hs+438c3lfUow==
+
 clean-regexp@^1.0.0:
   version "1.0.0"
   resolved "https://registry.yarnpkg.com/clean-regexp/-/clean-regexp-1.0.0.tgz#8df7c7aae51fd36874e8f8d05b9180bc11a3fed7"
@@ -7121,3 +7133,8 @@ yocto-queue@^0.1.0:
   version "0.1.0"
   resolved "https://registry.yarnpkg.com/yocto-queue/-/yocto-queue-0.1.0.tgz#0294eb3dee05028d31ee1a5fa2c556a6aaf10a1b"
   integrity sha512-rVksvsnNCdJ/ohGc6xgPwyN8eheCxsiLM8mxuE/t/mOVqJewPuO1miLpTHQiRgTKCLexL4MeAFVagts7HmNZ2Q==
+
+zod@^3.23.8:
+  version "3.23.8"
+  resolved "https://registry.yarnpkg.com/zod/-/zod-3.23.8.tgz#e37b957b5d52079769fb8097099b592f0ef4067d"
+  integrity sha512-XBx9AXhXktjUqnepgTiE5flcKIYWi/rme0Eaj+5Y0lftuGBq+jyRu/md4WnuxqgP1ubdpNCsYEYPxrzVHD8d6g==