fix(no-undefined-types): treat param names as defined; fixes #1591

Author: brettz9

docs/rules/no-undefined-types.md

@@ -1115,5 +1115,15 @@ const Severities = {
  */
 
 export default Severities;
+
+/**
+ * @template {unknown} T
+ * @param {unknown} value
+ * @param {...T} validValues
+ * @returns {value is T}
+ */
+const checkIsOnOf = (value, ...validValues) => {
+  return validValues.includes(value);
+};
 ````
 

src/rules/noUndefinedTypes.js

@@ -466,6 +466,16 @@ export default iterateJsdoc(({
     return utils.isNamepathOrUrlReferencingTag(tag);
   }).map(tagToParsedType('namepathOrURL'));
 
+  const definedNamesAndNamepaths = new Set(utils.filterTags(({
+    tag,
+  }) => {
+    return utils.isNameOrNamepathDefiningTag(tag);
+  }).map(({
+    name,
+  }) => {
+    return name;
+  }));
+
   const tagsWithTypes = /** @type {TypeAndTagInfo[]} */ ([
     ...typeTags,
     ...namepathReferencingTags,
@@ -520,6 +530,7 @@ export default iterateJsdoc(({
       if (type === 'JsdocTypeName') {
         const structuredTypes = structuredTags[tag.tag]?.type;
         if (!allDefinedTypes.has(val) &&
+          !definedNamesAndNamepaths.has(val) &&
           (!Array.isArray(structuredTypes) || !structuredTypes.includes(val))
         ) {
           const parent =

test/rules/assertions/noUndefinedTypes.js

@@ -1891,5 +1891,18 @@ export default /** @type {import('../index.js').TestCases} */ ({
         export default Severities;
       `,
     },
+    {
+      code: `
+        /**
+         * @template {unknown} T
+         * @param {unknown} value
+         * @param {...T} validValues
+         * @returns {value is T}
+         */
+        const checkIsOnOf = (value, ...validValues) => {
+          return validValues.includes(value);
+        };
+      `,
+    },
   ],
 });