fix(no-undefined-types): consider module scope variables as defined; fixes #1581

brettz9 · brettz9 · commit f938fdcc134e · 2025-10-24T13:59:17.000-07:00
diff --git a/docs/rules/no-undefined-types.md b/docs/rules/no-undefined-types.md
@@ -1097,5 +1097,23 @@ class Storage {
   /** @type {globalThis.localStorage} */
   #storage
 }
+
+/**
+ * La liste des sévérités.
+ *
+ * @type {Object<string, number>}
+ */
+const Severities = {
+    FATAL: 1,
+    ERROR: 2,
+    WARN: 3,
+    INFO: 4,
+};
+
+/**
+ * @typedef {Severities[keyof Severities]} Severity Le type des sévérités.
+ */
+
+export default Severities;
 ````
 
diff --git a/src/rules/noUndefinedTypes.js b/src/rules/noUndefinedTypes.js
@@ -342,7 +342,10 @@ export default iterateJsdoc(({
                 break;
               }
 
-              return [];
+              // Module scope names are also defined
+              return [
+                name,
+              ];
           }
 
           return [
diff --git a/test/rules/assertions/noUndefinedTypes.js b/test/rules/assertions/noUndefinedTypes.js
@@ -1870,5 +1870,26 @@ export default /** @type {import('../index.js').TestCases} */ ({
         },
       },
     },
+    {
+      code: `
+        /**
+         * La liste des sévérités.
+         *
+         * @type {Object<string, number>}
+         */
+        const Severities = {
+            FATAL: 1,
+            ERROR: 2,
+            WARN: 3,
+            INFO: 4,
+        };
+
+        /**
+         * @typedef {Severities[keyof Severities]} Severity Le type des sévérités.
+         */
+
+        export default Severities;
+      `,
+    },
   ],
 });

Original file line number	Diff line number	Diff line change
`@@ -342,7 +342,10 @@ export default iterateJsdoc(({`
`342`	`342`	`break;`
`343`	`343`	`}`
`344`	`344`
`345`		`- return [];`
	`345`	`+ // Module scope names are also defined`
	`346`	`+ return [`
	`347`	`+ name,`
	`348`	`+ ];`
`346`	`349`	`}`
`347`	`350`
`348`	`351`	`return [`