fix: rule dynamic-import-chunkname crash due to ts migration (#346)

JounQin · web-flow · commit 2991f2e839fc · 2025-05-26T14:46:13.000+08:00
diff --git a/.changeset/pink-readers-design.md b/.changeset/pink-readers-design.md
@@ -0,0 +1,5 @@
+---
+"eslint-plugin-import-x": patch
+---
+
+fix: rule `dynamic-import-chunkname` crash due to ts migration
diff --git a/src/rules/dynamic-import-chunkname.ts b/src/rules/dynamic-import-chunkname.ts
@@ -232,8 +232,8 @@ export default createRule<[Options?], MessageId>({
         if (
           // @ts-expect-error - legacy parser type
           node.callee.type !== 'Import' &&
-          'name' in node.callee &&
-          !importFunctions.includes(node.callee.name)
+          (!('name' in node.callee) ||
+            !importFunctions.includes(node.callee.name))
         ) {
           return
         }
diff --git a/test/rules/dynamic-import-chunkname.spec.ts b/test/rules/dynamic-import-chunkname.spec.ts
@@ -1136,6 +1136,26 @@ describe('TypeScript', () => {
           )`,
         options,
       },
+      {
+        code: `import { useSyncExternalStore } from "use-sync-external-store/shim";
+
+export default function useAbortSignal(signal: AbortSignal): boolean {
+  return useSyncExternalStore(
+    (callback: () => void) => {
+      const unsubscribe = new AbortController();
+      signal.addEventListener("abort", callback, {
+        signal: unsubscribe.signal,
+        once: true,
+      });
+      return () => {
+        unsubscribe.abort();
+      };
+    },
+    () => signal.aborted,
+  );
+}`,
+        options,
+      },
     ],
     invalid: [
       {

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +---
 +"eslint-plugin-import-x": patch
 +---
++
 +fix: rule `dynamic-import-chunkname` crash due to ts migration