fix(no-undefined-types): check class and interface globals; fixes #1420 (#1421)

brettz9 · web-flow · commit 3748bbeb3d2e · 2025-06-30T09:12:41.000+08:00
diff --git a/docs/rules/no-undefined-types.md b/docs/rules/no-undefined-types.md
@@ -971,5 +971,23 @@ export default [
  * @type {Abc['def']}
  */
 export const a = 'someString';
+
+export interface SomeInterface {
+  someProp: unknown;
+}
+
+/**
+ * {@link SomeInterface.someProp}
+ * @returns something
+ */
+
+class SomeClass {
+  someMethod () {}
+}
+
+/**
+ * {@link SomeClass.someMethod}
+ * @returns something
+ */
 ````
 
diff --git a/src/rules/noUndefinedTypes.js b/src/rules/noUndefinedTypes.js
@@ -269,24 +269,54 @@ export default iterateJsdoc(({
           variables,
         }) => {
           return variables;
-        }).map(({
+        }).flatMap(({
           identifiers,
           name,
         }) => {
-          if (
-            [
-              'ImportDefaultSpecifier',
-              'ImportNamespaceSpecifier',
-              'ImportSpecifier',
-            ].includes(
-              /** @type {import('estree').Identifier & {parent: {type: string}}} */ (
-                identifiers?.[0])?.parent?.type,
-            )
-          ) {
-            imports.push(name);
+          const globalItem = /** @type {import('estree').Identifier & {parent: import('@typescript-eslint/types').TSESTree.Node}} */ (
+            identifiers?.[0]
+          )?.parent;
+          switch (globalItem?.type) {
+            case 'ClassDeclaration':
+              return [
+                name,
+                ...globalItem.body.body.map((item) => {
+                  const property = /** @type {import('@typescript-eslint/types').TSESTree.Identifier} */ (
+                    /** @type {import('@typescript-eslint/types').TSESTree.PropertyDefinition} */ (
+                      item)?.key)?.name;
+                  /* c8 ignore next 3 -- Guard */
+                  if (!property) {
+                    return '';
+                  }
+
+                  return `${name}.${property}`;
+                }).filter(Boolean),
+              ];
+            case 'ImportDefaultSpecifier':
+            case 'ImportNamespaceSpecifier':
+            case 'ImportSpecifier':
+              imports.push(name);
+              break;
+            case 'TSInterfaceDeclaration':
+              return [
+                name,
+                ...globalItem.body.body.map((item) => {
+                  const property = /** @type {import('@typescript-eslint/types').TSESTree.Identifier} */ (
+                    /** @type {import('@typescript-eslint/types').TSESTree.TSPropertySignature} */ (
+                      item)?.key)?.name;
+                  /* c8 ignore next 3 -- Guard */
+                  if (!property) {
+                    return '';
+                  }
+
+                  return `${name}.${property}`;
+                }).filter(Boolean),
+              ];
           }
 
-          return name;
+          return [
+            name,
+          ];
         /* c8 ignore next */
         }) : [],
     )
diff --git a/test/rules/assertions/noUndefinedTypes.js b/test/rules/assertions/noUndefinedTypes.js
@@ -1692,5 +1692,32 @@ export default /** @type {import('../index.js').TestCases} */ ({
 
       `,
     },
+    {
+      code: `
+        export interface SomeInterface {
+          someProp: unknown;
+        }
+
+        /**
+         * {@link SomeInterface.someProp}
+         * @returns something
+         */
+      `,
+      languageOptions: {
+        parser: typescriptEslintParser,
+      },
+    },
+    {
+      code: `
+        class SomeClass {
+          someMethod () {}
+        }
+
+        /**
+         * {@link SomeClass.someMethod}
+         * @returns something
+         */
+      `,
+    },
   ],
 });
