fix(no-restricted-syntax): avoid matching contexts when in different items of contexts array; fixes #818

Author: brettz9

README.md

@@ -9018,6 +9018,15 @@ function a () {}
  * @param cd
  */
 // "jsdoc/no-restricted-syntax": ["error"|"warn", {"contexts":[{"comment":"JsdocBlock:has(JsdocTag[name=/opt_/])","message":"Only allowing names not matching `/^opt_/i`."}]}]
+
+/**
+ * @enum {String}
+ * Object holding values of some custom enum
+ */
+const MY_ENUM = Object.freeze({
+  VAL_A: "myvala"
+} as const);
+// "jsdoc/no-restricted-syntax": ["error"|"warn", {"contexts":[{"comment":"JsdocBlock[postDelimiter=\"\"]:has(JsdocTag ~ JsdocTag[tag=/private|protected/])","context":"any","message":"Access modifier tags must come first"},{"comment":"JsdocBlock[postDelimiter=\"\"]:has(JsdocTag[tag=\"enum\"])","context":":declaration:not(TSEnumDeclaration):not(:has(ObjectExpression)), :function","message":"@enum is only allowed on potential enum types"}]}]
 ````
 
 

src/rules/noRestrictedSyntax.js

@@ -20,6 +20,12 @@ export default iterateJsdoc(({
         comment === cntxt.comment;
   });
 
+  // We are not on the *particular* matching context/comment, so don't assume
+  //   we need reporting
+  if (!foundContext) {
+    return;
+  }
+
   const contextStr = typeof foundContext === 'object' ?
     foundContext.context ?? 'any' :
     foundContext;

test/rules/assertions/noRestrictedSyntax.js

@@ -314,5 +314,31 @@ export default {
         ],
       }],
     },
+    {
+      code: `
+        /**
+         * @enum {String}
+         * Object holding values of some custom enum
+         */
+        const MY_ENUM = Object.freeze({
+          VAL_A: "myvala"
+        } as const);
+      `,
+      options: [{
+        contexts: [
+          {
+            comment: 'JsdocBlock[postDelimiter=""]:has(JsdocTag ~ JsdocTag[tag=/private|protected/])',
+            context: 'any',
+            message: 'Access modifier tags must come first',
+          },
+          {
+            comment: 'JsdocBlock[postDelimiter=""]:has(JsdocTag[tag="enum"])',
+            context: ':declaration:not(TSEnumDeclaration):not(:has(ObjectExpression)), :function',
+            message: '@enum is only allowed on potential enum types',
+          },
+        ],
+      }],
+      parser: require.resolve('@typescript-eslint/parser'),
+    },
   ],
 };