eslint-plugin-jsdoc - npm Package Compare versions

Comparing version 46.4.1 to 46.4.2

dist/exportParser.js

@@ -690,2 +690,3 @@ "use strict";
 };
+const accessibilityNodes = new Set(['PropertyDefinition', 'MethodDefinition']);
 
@@ -695,2 +696,11 @@ /**
  * @param {import('eslint').Rule.Node} node
+ * @returns {boolean}
+ */
+const hasAccessibility = node => {
+  return accessibilityNodes.has(node.type) && 'accessibility' in node && node.accessibility !== 'public';
+};
+
+/**
+ *
+ * @param {import('eslint').Rule.Node} node
  * @param {import('eslint').SourceCode} sourceCode
@@ -705,3 +715,3 @@ * @param {import('./rules/requireJsdoc.js').RequireJsdocOpts} opt
   if (opt.esm) {
-    if (node.type === 'PropertyDefinition' && 'accessibility' in node && node.accessibility !== 'public') {
+    if (hasAccessibility(node) || node.parent && hasAccessibility(node.parent)) {
       return false;
@@ -708,0 +718,0 @@ }

docs/rules/require-jsdoc.md

@@ -642,3 +642,3 @@ <a name="user-content-require-jsdoc"></a>
 }
-// "jsdoc/require-jsdoc": ["error"|"warn", {"contexts":["MethodDefinition:not([accessibility=\"private\"]) > FunctionExpression"],"publicOnly":true,"require":{"ArrowFunctionExpression":false,"ClassDeclaration":false,"ClassExpression":false,"FunctionDeclaration":false,"FunctionExpression":false,"MethodDefinition":false}}]
+// "jsdoc/require-jsdoc": ["error"|"warn", {"contexts":["MethodDefinition > FunctionExpression"],"publicOnly":true,"require":{"ArrowFunctionExpression":false,"ClassDeclaration":false,"ClassExpression":false,"FunctionDeclaration":false,"FunctionExpression":false,"MethodDefinition":false}}]
 // Message: Missing JSDoc comment.
@@ -1679,3 +1679,3 @@
 }
-// "jsdoc/require-jsdoc": ["error"|"warn", {"contexts":["MethodDefinition:not([accessibility=\"private\"]) > FunctionExpression"],"publicOnly":true,"require":{"ArrowFunctionExpression":false,"ClassDeclaration":false,"ClassExpression":false,"FunctionDeclaration":false,"FunctionExpression":false,"MethodDefinition":false}}]
+// "jsdoc/require-jsdoc": ["error"|"warn", {"contexts":["MethodDefinition > FunctionExpression"],"publicOnly":true,"require":{"ArrowFunctionExpression":false,"ClassDeclaration":false,"ClassExpression":false,"FunctionDeclaration":false,"FunctionExpression":false,"MethodDefinition":false}}]
 
@@ -1851,3 +1851,10 @@ /**
 // "jsdoc/require-jsdoc": ["error"|"warn", {"contexts":[{"context":"MethodDefinition","minLineCount":4}],"require":{"ClassDeclaration":false,"FunctionExpression":false,"MethodDefinition":false}}]
+
+export default class Test {
+  private abc(a) {
+    this.a = a;
+  }
+}
+// "jsdoc/require-jsdoc": ["error"|"warn", {"publicOnly":true,"require":{"ArrowFunctionExpression":false,"ClassDeclaration":false,"ClassExpression":false,"FunctionDeclaration":false,"FunctionExpression":false,"MethodDefinition":true}}]
 ````
 

package.json

@@ -139,3 +139,3 @@ {
   },
-  "version": "46.4.1"
+  "version": "46.4.2"
 }

No alert changes

Improved metrics

Total package byte prevSize

1997698

increased by0.06%

Lines of code

13964

increased by0.06%

No dependency changes