fix(require-param): skip destructuring; fixes gajus#352 (though still need to handle gajus#11 )

Author: brettz9

README.md

@@ -6237,6 +6237,17 @@ export class SomeClass {
    */
   constructor(private property: string) {}
 }
+
+/**
+ * Assign the project to an employee.
+ *
+ * @param {object} employee - The employee who is responsible for the project.
+ * @param {string} employee.name - The name of the employee.
+ * @param {string} employee.department - The employee's department.
+ */
+function assign({name, department}) {
+  // ...
+}
 ````
 
 

src/rules/requireParam.js

@@ -21,6 +21,9 @@ export default iterateJsdoc(({
   }
 
   functionParameterNames.forEach((functionParameterName) => {
+    if (['<ObjectPattern>', '<ArrayPattern>'].includes(functionParameterName)) {
+      return;
+    }
     if (!jsdocParameterNames.includes(functionParameterName)) {
       report(`Missing JSDoc @${utils.getPreferredTagName({tagName: 'param'})} "${functionParameterName}" declaration.`);
     }

test/rules/assertions/requireParam.js

@@ -788,6 +788,20 @@ export default {
       parserOptions: {
         sourceType: 'module'
       }
+    },
+    {
+      code: `
+      /**
+       * Assign the project to an employee.
+       *
+       * @param {object} employee - The employee who is responsible for the project.
+       * @param {string} employee.name - The name of the employee.
+       * @param {string} employee.department - The employee's department.
+       */
+      function assign({name, department}) {
+        // ...
+      }
+      `
     }
   ]
 };