fix(require-jsdoc): avoid erring on static blocks

gajus · Sep 13, 2024 · 8222262 · 8222262
1 parent 288f0ae
commit 8222262
Show file tree

Hide file tree

Showing 3 changed files with 30 additions and 0 deletions.
diff --git a/docs/rules/require-jsdoc.md b/docs/rules/require-jsdoc.md
@@ -1894,5 +1894,12 @@ export class MyClass {
   #myPrivateProp = 5;
 }
 // "jsdoc/require-jsdoc": ["error"|"warn", {"publicOnly":true,"contexts":["PropertyDefinition"],"require":{"MethodDefinition":true}}]
+
+class Abc {
+  static {
+    this.x = '2'
+  }
+}
+// "jsdoc/require-jsdoc": ["error"|"warn", {"publicOnly":true,"require":{"ClassDeclaration":true}}]
 ````
 
diff --git a/src/exportParser.js b/src/exportParser.js
@@ -212,6 +212,9 @@ const getSymbol = function (node, globals, scope, opt) {
   case 'ClassBody': {
     const val = createNode();
     for (const method of node.body) {
+      if (!('key' in method)) { // StaticBlock
+        continue;
+      }
       val.props[
         /** @type {import('estree').Identifier} */ (
           /** @type {import('estree').MethodDefinition} */ (

diff --git a/test/rules/assertions/requireJsdoc.js b/test/rules/assertions/requireJsdoc.js
@@ -6293,5 +6293,25 @@ function quux (foo) {
         },
       ],
     },
+    {
+      code: `
+        class Abc {
+          static {
+            this.x = '2'
+          }
+        }
+      `,
+      languageOptions: {
+        parser: babelEslintParser,
+      },
+      options: [
+        {
+          publicOnly: true,
+          require: {
+            ClassDeclaration: true
+          }
+        }
+      ],
+    }
   ],
 };