fix: ensure permissive settings are converted for jsdoc-type-pratt-parser

Also:
- chore: update jsdoccomment and devDeps.

Author: brettz9

docs/rules/prefer-import-tag.md

@@ -266,6 +266,32 @@ The following patterns are considered problems:
  */
 // "jsdoc/prefer-import-tag": ["error"|"warn", {"exemptTypedefs":true}]
 // Message: Inline `import()` found; prefer `@import`
+
+/**
+ * @type {import('eslint').anchors[keyof DataMap.anchors]}
+ */
+// Message: Inline `import()` found; prefer `@import`
+
+/** @typedef {import('eslint').Rule[keyof import('eslint').Rule]} Rule  */
+/**
+ * @type {import('eslint').Abc.Rule}
+ */
+// "jsdoc/prefer-import-tag": ["error"|"warn", {"exemptTypedefs":true}]
+// Message: Inline `import()` found; prefer `@import`
+
+/** @typedef {import('eslint').Rule[keyof import('eslint').Rule]} Rule  */
+/**
+ * @type {import('eslint').Rule[keyof import('eslint').Rule]}
+ */
+// "jsdoc/prefer-import-tag": ["error"|"warn", {"exemptTypedefs":true}]
+// Message: Inline `import()` found; using `@typedef`
+
+/** @typedef {import('eslint').Rule} Rule  */
+/**
+ * @type {import('eslint').Rule[keyof import('eslint').Rule]}
+ */
+// "jsdoc/prefer-import-tag": ["error"|"warn", {"exemptTypedefs":true}]
+// Message: Inline `import()` found; using `@typedef`
 ````
 
 

package.json

@@ -5,7 +5,7 @@
     "url": "http://gajus.com"
   },
   "dependencies": {
-    "@es-joy/jsdoccomment": "~0.60.0",
+    "@es-joy/jsdoccomment": "~0.61.0",
     "are-docs-informative": "^0.0.2",
     "comment-parser": "1.4.1",
     "debug": "^4.4.3",
@@ -58,10 +58,10 @@
     "glob": "^11.0.3",
     "globals": "^16.4.0",
     "husky": "^9.1.7",
-    "jsdoc-type-pratt-parser": "^5.4.0",
+    "jsdoc-type-pratt-parser": "^5.8.0",
     "json-schema": "^0.4.0",
     "json-schema-to-typescript": "^15.0.4",
-    "lint-staged": "^16.2.0",
+    "lint-staged": "^16.2.1",
     "mocha": "^11.7.2",
     "open-editor": "^5.1.0",
     "replace": "^1.2.2",

pnpm-lock.yaml

@@ -68,7 +68,6 @@ import textEscaping from './rules/textEscaping.js';
 import typeFormatting from './rules/typeFormatting.js';
 import validTypes from './rules/validTypes.js';
 
-/* eslint-disable jsdoc/valid-types -- Bug */
 /**
  * @typedef {"recommended" | "stylistic" | "contents" | "logical" | "requirements"} ConfigGroups
  * @typedef {"" | "-typescript" | "-typescript-flavor"} ConfigVariants
@@ -86,7 +85,6 @@ import validTypes from './rules/validTypes.js';
  * }}
  */
 const index = {};
-/* eslint-enable jsdoc/valid-types -- Bug */
 index.configs = {};
 index.rules = {
   'check-access': checkAccess,

src/index-cjs.js

@@ -74,7 +74,6 @@ import textEscaping from './rules/textEscaping.js';
 import typeFormatting from './rules/typeFormatting.js';
 import validTypes from './rules/validTypes.js';
 
-/* eslint-disable jsdoc/valid-types -- Bug */
 /**
  * @typedef {"recommended" | "stylistic" | "contents" | "logical" | "requirements"} ConfigGroups
  * @typedef {"" | "-typescript" | "-typescript-flavor"} ConfigVariants
@@ -92,7 +91,6 @@ import validTypes from './rules/validTypes.js';
  * }}
  */
 const index = {};
-/* eslint-enable jsdoc/valid-types -- Bug */
 index.configs = {};
 index.rules = {
   'check-access': checkAccess,

src/index.js

@@ -2284,7 +2284,10 @@ const iterateAllJsdocs = (iterator, ruleConfig, contexts, additiveCommentContext
       }
 
       if (contexts) {
-        handler = commentHandler(settings);
+        handler = commentHandler({
+          ...settings,
+          mode: settings.mode === 'permissive' ? 'typescript' : settings.mode,
+        });
       }
 
       const state = {};
@@ -2537,7 +2540,10 @@ export default function iterateJsdoc (iterator, ruleConfig) {
         contextObject = jsdocUtils.getContextObject(
           contexts,
           checkJsdoc,
-          commentHandler(settings),
+          commentHandler({
+            ...settings,
+            mode: settings.mode === 'permissive' ? 'typescript' : settings.mode,
+          }),
         );
       } else {
         for (const prop of [

src/iterateJsdoc.js

@@ -117,10 +117,18 @@ export default iterateJsdoc(({
           break;
         }
 
-        pathSegments.unshift(currentNode.right.value);
+        pathSegments.unshift(
+          currentNode.right.type === 'JsdocTypeIndexedAccessIndex' ?
+            stringify(currentNode.right.right) :
+            currentNode.right.value,
+        );
         nodes.unshift(currentNode);
         propertyOrBrackets.unshift(currentNode.pathType);
-        quotes.unshift(currentNode.right.meta.quote);
+        quotes.unshift(
+          currentNode.right.type === 'JsdocTypeIndexedAccessIndex' ?
+            undefined :
+            currentNode.right.meta.quote,
+        );
       }
 
       /**
@@ -218,7 +226,12 @@ export default iterateJsdoc(({
               break;
             }
 
-            if (typedefNode.right.value !== pathSegment) {
+            if (
+              (typedefNode.right.type === 'JsdocTypeIndexedAccessIndex' &&
+                stringify(typedefNode.right.right) !== pathSegment) ||
+              (typedefNode.right.type !== 'JsdocTypeIndexedAccessIndex' &&
+                typedefNode.right.value !== pathSegment)
+            ) {
               if (namepathMatch === true) {
                 // It stopped matching, so stop
                 break;

src/rules/preferImportTag.js

@@ -35,6 +35,35 @@ export default /** @type {import('../index.js').TestCases} */ ({
        */
       function quux () {
 
+      }
+      `,
+      errors: [
+        {
+          line: 2,
+          message: 'Syntax is restricted: FunctionDeclaration',
+        },
+      ],
+      ignoreReadme: true,
+      options: [
+        {
+          contexts: [
+            'FunctionDeclaration',
+          ],
+        },
+      ],
+      settings: {
+        jsdoc: {
+          mode: 'permissive',
+        },
+      },
+    },
+    {
+      code: `
+      /**
+       *
+       */
+      function quux () {
+
       }
       `,
       errors: [
@@ -780,6 +809,32 @@ export default /** @type {import('../index.js').TestCases} */ ({
         },
       ],
     },
+    {
+      code: `
+        /**
+         * @param ab
+         * @param cd
+         */
+        function a () {}
+      `,
+      ignoreReadme: true,
+      options: [
+        {
+          contexts: [
+            {
+              comment: 'JsdocBlock:has(JsdocTag[name=/opt_/])',
+              context: 'any',
+              message: 'Only allowing names not matching `/^opt_/i`.',
+            },
+          ],
+        },
+      ],
+      settings: {
+        jsdoc: {
+          mode: 'permissive',
+        },
+      },
+    },
     {
       code: `
         /**