fix: ensure TSDeclareFunction is checked by default where function contexts are expected; closes #897

Author: brettz9

README.md

@@ -3280,6 +3280,12 @@ function quux (foo) {
 }
 // "jsdoc/check-param-names": ["error"|"warn", {"enableFixer":true}]
 // Message: Duplicate @param "foo"
+
+/**
+ * @param barr This is the description of bar. Oops, we misspelled "bar" as "barr".
+ */
+declare function foo(bar: number) {}
+// Message: Expected @param names to be "bar". Got "barr".
 ````
 
 The following patterns are not considered problems:

src/iterateJsdoc.js

@@ -1353,6 +1353,7 @@ export default function iterateJsdoc (iterator, ruleConfig) {
           'ArrowFunctionExpression',
           'FunctionDeclaration',
           'FunctionExpression',
+          'TSDeclareFunction',
         ]) {
           contextObject[prop] = checkJsdoc.bind(null, {
             selector: prop,

src/jsdocUtils.js

@@ -973,6 +973,7 @@ const enforcedContexts = (context, defaultContexts) => {
       'ArrowFunctionExpression',
       'FunctionDeclaration',
       'FunctionExpression',
+      'TSDeclareFunction',
     ] : defaultContexts,
   } = context.options[0] || {};
 

test/rules/assertions/checkParamNames.js

@@ -1162,6 +1162,21 @@ export default {
           }
       `,
     },
+    {
+      code: `
+      /**
+       * @param barr This is the description of bar. Oops, we misspelled "bar" as "barr".
+       */
+      declare function foo(bar: number) {}
+      `,
+      errors: [
+        {
+          line: 3,
+          message: 'Expected @param names to be "bar". Got "barr".',
+        },
+      ],
+      parser: require.resolve('@typescript-eslint/parser'),
+    },
   ],
   valid: [
     {