@depup/eslint-plugin-jsdoc 65.3.2-depup.0 → 65.3.4-depup.0

package/README.md

@@ -13,8 +13,8 @@ npm install @depup/eslint-plugin-jsdoc
 
 | Field | Value |
 |-------|-------|
-| Original | [eslint-plugin-jsdoc](https://www.npmjs.com/package/eslint-plugin-jsdoc) @ 65.3.2 |
-| Processed | 2026-10-13 |
+| Original | [eslint-plugin-jsdoc](https://www.npmjs.com/package/eslint-plugin-jsdoc) @ 65.3.4 |
+| Processed | 2026-10-15 |
 | Smoke test | passed |
 | Deps updated | 2 |
 

package/changes.json

@@ -9,6 +9,6 @@
       "to": "^8.71.2"
     }
   },
-  "timestamp": "2026-10-13T16:10:58.556Z",
+  "timestamp": "2026-10-15T00:29:27.540Z",
   "totalUpdated": 2
 }

package/dist/rules.d.ts

@@ -334,16 +334,23 @@ export interface Rules {
            * class
            * enum
            * implements
-           * private
            * property
-           * protected
-           * public
            * readonly
            * this
            * type
            * typedef
            * ```
            *
+           * These tags are unnecessary on class members (where TypeScript has access
+           * modifiers), but are allowed elsewhere (e.g., the `@public` release tag of
+           * TSDoc/API Extractor on an exported function):
+           *
+           * ```
+           * private
+           * protected
+           * public
+           * ```
+           *
            * These tags are unnecessary except when inside a TypeScript `declare` context:
            *
            * ```

package/package.json

@@ -168,7 +168,7 @@
     "test-cov": "TIMING=1 c8 --reporter text pnpm run test-no-cov",
     "test-index": "pnpm run test-no-cov test/rules/index.js"
   },
-  "version": "65.3.2-depup.0",
+  "version": "65.3.4-depup.0",
   "depup": {
     "changes": {
       "@es-joy/jsdoccomment": {
@@ -182,8 +182,8 @@
     },
     "depsUpdated": 2,
     "originalPackage": "eslint-plugin-jsdoc",
-    "originalVersion": "65.3.2",
-    "processedAt": "2026-10-13T16:11:24.215Z",
+    "originalVersion": "65.3.4",
+    "processedAt": "2026-10-15T00:29:54.759Z",
     "smokeTest": "passed"
   }
 }

package/src/jsdocUtils.js

@@ -1083,18 +1083,34 @@ const isNameOrNamepathDefiningTag = (tag, tagMap = tagStructure) => {
     tagStruct.get('namepathRole')));
 };
 
+/**
+ * The parsed JSDoc blocks of each file (and fix pass), keyed by its
+ * `SourceCode` object. Rules such as `no-undefined-types` and
+ * `require-returns-check` ask for all blocks of the file once per JSDoc
+ * block they check, so without this, every comment of the file was parsed
+ * again for each block (quadratic time on larger files).
+ * @type {WeakMap<import('eslint').SourceCode, import('@es-joy/jsdoccomment').JsdocBlockWithInline[]>}
+ */
+const jsdocCommentBlocksCache = new WeakMap();
+
 /**
  * @param {import('eslint').SourceCode} sourceCode
  * @returns {import('@es-joy/jsdoccomment').JsdocBlockWithInline[]}
  */
 const getJSDocCommentBlocks = (sourceCode) => {
-  return sourceCode.getAllComments()
-    .filter((comment) => {
-      return (/^\*(?!\*)/v).test(comment.value);
-    })
-    .map((commentNode) => {
-      return parseComment(commentNode, '');
-    });
+  let blocks = jsdocCommentBlocksCache.get(sourceCode);
+  if (!blocks) {
+    blocks = sourceCode.getAllComments()
+      .filter((comment) => {
+        return (/^\*(?!\*)/v).test(comment.value);
+      })
+      .map((commentNode) => {
+        return parseComment(commentNode, '');
+      });
+    jsdocCommentBlocksCache.set(sourceCode, blocks);
+  }
+
+  return blocks;
 };
 
 /**

package/src/rules/checkTagNames.js

@@ -29,6 +29,20 @@ const typedTagsNeedingName = new Set([
   'template',
 ]);
 
+const accessTags = new Set([
+  'private',
+  'protected',
+  'public',
+]);
+
+/**
+ * @param {import('eslint').Rule.Node|null} node
+ * @returns {boolean}
+ */
+const isClassMember = (node) => {
+  return node?.parent?.type === 'ClassBody';
+};
+
 const typedTagsUnnecessaryOutsideDeclare = new Set([
   'abstract',
   'access',
@@ -201,7 +215,13 @@ export default iterateJsdoc(({
    * @returns {boolean}
    */
   const checkTagForTypedValidity = (jsdocTag, tagIndex) => {
-    if (typedTagsAlwaysUnnecessary.has(jsdocTag.tag)) {
+    if (
+      typedTagsAlwaysUnnecessary.has(jsdocTag.tag) &&
+      // TypeScript only has syntax for these on class members; elsewhere they
+      //   are not redundant (e.g., TSDoc/API Extractor release tag `@public`,
+      //   TypeDoc's `@private`/`@protected` visibility, Knip's `@public`)
+      (!accessTags.has(jsdocTag.tag) || isClassMember(node))
+    ) {
       reportWithTagRemovalFixer(
         `'@${jsdocTag.tag}' is redundant when using a type system.`,
         jsdocTag,
@@ -361,16 +381,23 @@ callback
 class
 enum
 implements
-private
 property
-protected
-public
 readonly
 this
 type
 typedef
 \`\`\`
 
+These tags are unnecessary on class members (where TypeScript has access
+modifiers), but are allowed elsewhere (e.g., the \`@public\` release tag of
+TSDoc/API Extractor on an exported function):
+
+\`\`\`
+private
+protected
+public
+\`\`\`
+
 These tags are unnecessary except when inside a TypeScript \`declare\` context:
 
 \`\`\`

package/src/rules.d.ts

@@ -334,16 +334,23 @@ export interface Rules {
            * class
            * enum
            * implements
-           * private
            * property
-           * protected
-           * public
            * readonly
            * this
            * type
            * typedef
            * ```
            *
+           * These tags are unnecessary on class members (where TypeScript has access
+           * modifiers), but are allowed elsewhere (e.g., the `@public` release tag of
+           * TSDoc/API Extractor on an exported function):
+           *
+           * ```
+           * private
+           * protected
+           * public
+           * ```
+           *
            * These tags are unnecessary except when inside a TypeScript `declare` context:
            *
            * ```