@sarj/eslint-plugin 13.0.0 → 13.1.0

package/README.md

@@ -8,7 +8,7 @@ Custom ESLint rules for hypermodern TypeScript / React / Next.js projects
 
 | Field | Value |
 | --- | --- |
-| Version | `13.0.0` |
+| Version | `13.1.0` |
 | Registry | npm |
 | License | MIT |
 | Runtime | `node >=22.13.0` |
@@ -47,7 +47,7 @@ Custom ESLint rules for hypermodern TypeScript / React / Next.js projects
 | `eslint:no-insecure-random-id` | Disallow using `Math.random()` to generate identifiers, tokens, or secrets; use `crypto.randomUUID()` or `crypto.getRandomValues(...)` instead. | `error` | `none` |
 | `eslint:no-json-stringify-error` | Disallow `JSON.stringify` on an Error value; it yields `{}` because `message`/`stack` are non-enumerable. | `error` | `none` |
 | `eslint:no-log-only-catch` | Disallow `catch` clauses that only log (or silently do nothing) and then swallow the error; rethrow or handle it instead. | `error` | `none` |
-| `eslint:no-long-comment` | Flag unusually large unstructured prose blocks in implementation code. | `error` | `none` |
+| `eslint:no-long-comment` | Flag unusually large unstructured JSDoc blocks in implementation code. | `error` | `none` |
 | `eslint:no-offset-pagination` | Disallow OFFSET pagination in embedded SQL; it is O(N) per page and drops or repeats rows under concurrent writes. Use a keyset cursor. | `error` | `none` |
 | `eslint:no-positional-tuple-return` | Disallow returning a multi-field tuple from an exported function; return a named object so call sites cannot mismatch slots. | `error` | `none` |
 | `eslint:no-raw-env` | Disallow direct `process.env` and `import.meta.env` reads outside validated boundaries. | `error` | `none` |

package/dist/index.cjs

@@ -1142,6 +1142,7 @@ var STEP_NARRATION_RE = /^(?:first(?:ly)?|second(?:ly)?|third(?:ly)?|then|next|a
 var META_COMMENTARY_RE = /\b(?:keeping (?:it|this) simple|could be (?:refactored|improved|cleaned up|simplified)|refactor(?:ed|ing)? (?:later|this)|not sure (?:if|whether|why|how)|quick[- ](?:and[- ]dirty|fix)|(?:a |bit of a )?hacky|is a hack|temporary (?:solution|workaround|fix|hack)|revisit (?:this|later|below)|clean (?:this|it) up|not ideal|placeholder for now)\b/i;
 var EDITORIAL_PLACEHOLDER_RE = /^(?:(?:implementation omitted|existing code here|your code here|rest of (?:the )?code (?:is )?unchanged|same as above|placeholder implementation)\s*[.!]?|in a real (?:app(?:lication)?|implementation),?\s+(?:this|we|you|it)\s+would\s+(?:call|fetch|generate|download|persist|save|send|store|write)\b[^,;]*[.!]?)$/i;
 var FOR_NOW_RE = /\bfor now\b/i;
+var JSDOC_DEBT_RE = /^@?(?:todo|fixme)\b/i;
 var DEFERRAL_STOPWORDS = /* @__PURE__ */ new Set([
   "a",
   "an",
@@ -1270,6 +1271,7 @@ var DIAGRAM_ARROW_RE = /[-=~]{2,}>|<[-=~]{2,}/;
 var CODE_KEYWORD_RE = /^(import |export |const |let |var |function\b|class |interface |type \w|enum |return\b|throw |await |async |if\s*\(|for\s*\(|while\s*\(|switch\s*\(|new |console\.)/;
 var CODE_TAIL_RE = /[;{}()]\s*$|=>\s*$|,\s*$/;
 var ASSIGN_RE = /^[A-Za-z_$][\w.$[\]]*\s*(?:=(?![=>])|\+=|-=|\*=)\s*\S.*[;)}\]]\s*$/;
+var DECLARATION_RE = /^(?:export\s+)?(?:declare\s+)?(?:const|let|var)\s+[A-Za-z_$][\w$]*(?:\s*:\s*[^=]+)?\s*=\s*(?:[A-Za-z_$][\w.$]*(?:\s*\(|\s*$)|["'`]|\[|\{|\d|true\b|false\b|null\b|undefined\b|new\b|await\b|async\b|function\b|class\b)/;
 var CALL_RE = /^[A-Za-z_$][\w.$]*\([^)]*\)\s*;?\s*$/;
 var ASSERTION_CODE_RE = /^(?:await\s+)?(?:expect(?:TypeOf)?|assert(?:\.\w+)?)\s*\(/;
 var HTTP_CONTRACT_RE = /\b(?:GET|HEAD|OPTIONS|PATCH|POST|PUT|DELETE)\s+(?:https?:\/\/|\/|\{[A-Za-z_$])/;
@@ -1297,6 +1299,7 @@ function looksLikeCode(text, allowCall = true) {
   if (!t) return false;
   if (PROSE_ASSIGNMENT_RE.test(t)) return false;
   if (CODE_KEYWORD_RE.test(t) && CODE_TAIL_RE.test(t)) return true;
+  if (DECLARATION_RE.test(t)) return true;
   if (ASSIGN_RE.test(t)) return true;
   if (ASSERTION_CODE_RE.test(t)) return true;
   return allowCall && CALL_RE.test(t);
@@ -1626,6 +1629,11 @@ var no_comment_cruft_default = createRule({
           }
           if (wallMembers.has(comment)) continue;
           if (isJsDoc(comment)) {
+            const debt = comment.value.split("\n").map(stripCommentMarker).find((line) => JSDOC_DEBT_RE.test(line));
+            if (debt !== void 0 && !runCitesAReference(comments, i)) {
+              context.report({ node: comment, messageId: "untrackedTodo" });
+              continue;
+            }
             if (isStandalone(comment) && isSectionJsDoc(comment)) {
               context.report({ node: comment, messageId: "sectionBanner" });
             }
@@ -4432,11 +4440,11 @@ function typedFunction(node) {
 
 // src/rules/no-long-comment.ts
 var noLongCommentDocumentation = {
-  summary: "Flag unusually large unstructured prose blocks in implementation code.",
+  summary: "Flag unusually large unstructured JSDoc blocks in implementation code.",
   rationale: "Large narrative comments become stale and obscure the local facts that belong beside the code.",
   remediation: "Keep only durable local constraints and express the remaining behavior in code.",
   category: "maintainability",
-  limitations: ["Structured API docs, tests, scripts, generated files, and versioned dependencies are excluded."],
+  limitations: ["Only JSDoc blocks are inspected; structured API docs, tests, scripts, generated files, and versioned dependencies are excluded."],
   examples: [
     { id: "local-fact", title: "Keep a concise local fact", outcome: "no-match", files: [{ path: "src/cache.ts", source: "// The cache is process local.\nconst cache = new Map();" }], focusPath: "src/cache.ts", expectedCount: 0, public: true },
     { id: "prose-wall", title: "Avoid an unstructured prose wall", outcome: "match", files: [{ path: "src/chart.ts", source: "/** One. Two. Three. Four. Five. Six. Seven. Eight. */\nconst chart = createChart();" }], focusPath: "src/chart.ts", expectedCount: 1, public: true }
@@ -4464,7 +4472,7 @@ var no_long_comment_default = createRule({
   documentation: noLongCommentDocumentation,
   meta: {
     type: "suggestion",
-    docs: { description: "Flag unusually large unstructured prose blocks in implementation code." },
+    docs: { description: "Flag unusually large unstructured JSDoc blocks in implementation code." },
     schema: [],
     messages: {
       tooLong: "Comment is an unusually large prose block \u2014 keep the local facts and clarify the code itself."
@@ -5409,6 +5417,7 @@ var MIN_DISTINCT_SCOPES = 2;
 var PREVIEW_LENGTH = 40;
 var SQL_KEYWORD_RE = /\b(SELECT|INSERT|UPDATE|DELETE|FROM|WHERE|JOIN|VALUES|ON CONFLICT|RETURNING|GROUP BY|ORDER BY)\b/;
 var IDENTIFIER_RE = /^[a-z_][a-z0-9_.]*$/;
+var URL_PATH_RE = /^\/(?=[^\s]*[A-Za-z0-9])[A-Za-z0-9._~!$&'()*+,;=:@%/?#{}\u005B\u005D-]+$/;
 var FUNCTION_TYPES4 = /* @__PURE__ */ new Set([
   import_utils29.AST_NODE_TYPES.FunctionDeclaration,
   import_utils29.AST_NODE_TYPES.FunctionExpression,
@@ -5416,7 +5425,7 @@ var FUNCTION_TYPES4 = /* @__PURE__ */ new Set([
 ]);
 var noRepeatedStringLiteralDocumentation = {
   summary: "Disallow a long structured string literal repeated across functions; the copies drift when one is edited. Extract a module-level constant.",
-  rationale: "Independent copies of a structured value can diverge and silently change behavior.",
+  rationale: "Independent copies of a query, route template, or identifier can diverge and silently change behavior.",
   remediation: "Extract the repeated value to one module-level constant and reference it from each function.",
   category: "maintainability",
   limitations: ["Test files, short strings, prose, substitutions, module sources, JSX attributes, and repetition within one function are excluded."],
@@ -5426,7 +5435,7 @@ var noRepeatedStringLiteralDocumentation = {
   ]
 };
 function isStructured(value) {
-  return value.includes("\n") || SQL_KEYWORD_RE.test(value) || IDENTIFIER_RE.test(value);
+  return value.includes("\n") || SQL_KEYWORD_RE.test(value) || IDENTIFIER_RE.test(value) || URL_PATH_RE.test(value);
 }
 function preview(value) {
   const oneLine = value.replaceAll("\n", " ").trim();
@@ -5445,8 +5454,9 @@ function isScaffolding(node) {
   if (parent === void 0) {
     return true;
   }
+  const isNonComputedPropertyKey = (parent.type === import_utils29.AST_NODE_TYPES.Property || parent.type === import_utils29.AST_NODE_TYPES.PropertyDefinition || parent.type === import_utils29.AST_NODE_TYPES.MethodDefinition || parent.type === import_utils29.AST_NODE_TYPES.AccessorProperty) && parent.key === node && !parent.computed;
   const isRequireSource = parent.type === import_utils29.AST_NODE_TYPES.CallExpression && parent.callee.type === import_utils29.AST_NODE_TYPES.Identifier && parent.callee.name === "require";
-  return parent.type === import_utils29.AST_NODE_TYPES.ImportDeclaration || parent.type === import_utils29.AST_NODE_TYPES.ImportExpression || parent.type === import_utils29.AST_NODE_TYPES.ExportNamedDeclaration || parent.type === import_utils29.AST_NODE_TYPES.ExportAllDeclaration || parent.type === import_utils29.AST_NODE_TYPES.TSImportType || parent.type === import_utils29.AST_NODE_TYPES.JSXAttribute || parent.type === import_utils29.AST_NODE_TYPES.TSLiteralType || isRequireSource;
+  return parent.type === import_utils29.AST_NODE_TYPES.ImportDeclaration || parent.type === import_utils29.AST_NODE_TYPES.ImportExpression || parent.type === import_utils29.AST_NODE_TYPES.ExportNamedDeclaration || parent.type === import_utils29.AST_NODE_TYPES.ExportAllDeclaration || parent.type === import_utils29.AST_NODE_TYPES.TSImportType || parent.type === import_utils29.AST_NODE_TYPES.JSXAttribute || parent.type === import_utils29.AST_NODE_TYPES.TSLiteralType || isNonComputedPropertyKey || isRequireSource;
 }
 var no_repeated_string_literal_default = createRule({
   name: "no-repeated-string-literal",
@@ -5463,7 +5473,7 @@ var no_repeated_string_literal_default = createRule({
   },
   defaultOptions: [],
   create(context) {
-    if (isTestFile(context.filename)) {
+    if (isTestFile(context.filename) || isGeneratedFile(context.filename, context.sourceCode.text)) {
       return {};
     }
     const occurrences = /* @__PURE__ */ new Map();
@@ -5783,10 +5793,14 @@ var no_restated_jsdoc_default = createRule({
         for (const comment of sourceCode.getAllComments()) {
           if (comment.type !== "Block" || !comment.value.startsWith("*")) continue;
           const { description, tags } = parseJsDoc(comment.value);
-          if (DIRECTIVE_RE4.test(description)) continue;
+          const describedText = [
+            description,
+            ...tags.filter((tag) => tag.name === "description").map((tag) => tag.text)
+          ].filter((text) => text.length > 0).join("\n");
+          if (DIRECTIVE_RE4.test(describedText)) continue;
           const tagNames = new Set(tags.map((tag) => tag.name));
           if ([...tagNames].some((name) => !MODELLED_TAGS.has(name))) continue;
-          if (isProtected(description)) continue;
+          if (isProtected(describedText)) continue;
           const token = sourceCode.getTokenAfter(comment, { includeComments: false });
           if (token === null || token.loc.start.line !== comment.loc.end.line + 1) continue;
           let node = sourceCode.getNodeByRangeIndex(token.range[0]);
@@ -5799,13 +5813,14 @@ var no_restated_jsdoc_default = createRule({
           if (declaration === null) continue;
           const paramTags = tags.filter((tag) => PARAM_TAGS.has(tag.name));
           const returnTags = tags.filter((tag) => RETURN_TAGS.has(tag.name));
-          if (description.length === 0 && paramTags.length === 0 && returnTags.length === 0) {
+          if (describedText.length === 0 && paramTags.length === 0 && returnTags.length === 0) {
             continue;
           }
+          if ((paramTags.length > 0 || returnTags.length > 0) && documentsTypedFunction(sourceCode, comment)) continue;
           const nameTokens = tokensOf([declaration.name]);
           const paramTokens = tokensOf(declaration.params);
           const known = /* @__PURE__ */ new Set([...nameTokens, ...paramTokens]);
-          let addsNothing = covered(description, known);
+          let addsNothing = covered(describedText, known);
           for (const tag of paramTags) {
             const text = tag.text.replace(/^\{[^}]*\}\s*/, "");
             const match = /^\[?([A-Za-z_$][\w.$]*)\]?\s*-?\s*([\s\S]*)$/.exec(text);
@@ -5813,7 +5828,13 @@ var no_restated_jsdoc_default = createRule({
               addsNothing = false;
               break;
             }
-            const own = /* @__PURE__ */ new Set([...splitIdentifier(match[1]?.split(".").pop() ?? ""), ...nameTokens]);
+            const path = match[1] ?? "";
+            const root = path.split(".")[0] ?? "";
+            if (!declaration.params.includes(root)) {
+              addsNothing = false;
+              break;
+            }
+            const own = /* @__PURE__ */ new Set([...splitIdentifier(path.split(".").pop() ?? ""), ...nameTokens]);
             if (!covered(match[2] ?? "", own)) {
               addsNothing = false;
               break;
@@ -6638,10 +6659,10 @@ var no_sentinel_return_on_catch_default = createRule({
           return false;
         }
         if (matcher.isLoggingCall(current)) {
-          return true;
+          return caughtName === null || argsIncludeBinding(current.arguments, caughtName);
         }
         const name = calleeName2(current.callee);
-        return name !== null && REPORT_NAME_RE.test(name) && argsIncludeBinding(current.arguments, caughtName);
+        return name !== null && REPORT_NAME_RE.test(name) && (caughtName === null || argsIncludeBinding(current.arguments, caughtName));
       });
     }
     return {
@@ -8110,7 +8131,7 @@ var no_type_member_comment_wall_default = createRule({
         claimed.add(comment);
         commented += 1;
         const body2 = commentBody(comment);
-        if (body2.length === 0 || carriesValue(body2)) continue;
+        if (body2.length === 0 || carriesValue(body2) || isTagsOnly(body2)) continue;
         if (novelWords(body2, knownTokens(sourceCode.getText(member))) <= options.maxNovelWords) {
           restated += 1;
         }
@@ -14016,7 +14037,7 @@ var rules = {
 };
 var meta = {
   name: "@sarj/eslint-plugin",
-  version: "13.0.0"
+  version: "13.1.0"
 };
 var applicationOnlyRules = [
   "no-restricted-library-load",

package/dist/index.d.cts

@@ -399,7 +399,7 @@ type FlatPreset = {
 declare const plugin: {
     readonly meta: {
         readonly name: "@sarj/eslint-plugin";
-        readonly version: "13.0.0";
+        readonly version: "13.1.0";
     };
     readonly rules: {
         readonly "duplicate-test-body": DocumentedRule<readonly [], "duplicateTestBody">;

package/dist/index.d.ts

@@ -399,7 +399,7 @@ type FlatPreset = {
 declare const plugin: {
     readonly meta: {
         readonly name: "@sarj/eslint-plugin";
-        readonly version: "13.0.0";
+        readonly version: "13.1.0";
     };
     readonly rules: {
         readonly "duplicate-test-body": DocumentedRule<readonly [], "duplicateTestBody">;

package/dist/index.js

@@ -1099,6 +1099,7 @@ var STEP_NARRATION_RE = /^(?:first(?:ly)?|second(?:ly)?|third(?:ly)?|then|next|a
 var META_COMMENTARY_RE = /\b(?:keeping (?:it|this) simple|could be (?:refactored|improved|cleaned up|simplified)|refactor(?:ed|ing)? (?:later|this)|not sure (?:if|whether|why|how)|quick[- ](?:and[- ]dirty|fix)|(?:a |bit of a )?hacky|is a hack|temporary (?:solution|workaround|fix|hack)|revisit (?:this|later|below)|clean (?:this|it) up|not ideal|placeholder for now)\b/i;
 var EDITORIAL_PLACEHOLDER_RE = /^(?:(?:implementation omitted|existing code here|your code here|rest of (?:the )?code (?:is )?unchanged|same as above|placeholder implementation)\s*[.!]?|in a real (?:app(?:lication)?|implementation),?\s+(?:this|we|you|it)\s+would\s+(?:call|fetch|generate|download|persist|save|send|store|write)\b[^,;]*[.!]?)$/i;
 var FOR_NOW_RE = /\bfor now\b/i;
+var JSDOC_DEBT_RE = /^@?(?:todo|fixme)\b/i;
 var DEFERRAL_STOPWORDS = /* @__PURE__ */ new Set([
   "a",
   "an",
@@ -1227,6 +1228,7 @@ var DIAGRAM_ARROW_RE = /[-=~]{2,}>|<[-=~]{2,}/;
 var CODE_KEYWORD_RE = /^(import |export |const |let |var |function\b|class |interface |type \w|enum |return\b|throw |await |async |if\s*\(|for\s*\(|while\s*\(|switch\s*\(|new |console\.)/;
 var CODE_TAIL_RE = /[;{}()]\s*$|=>\s*$|,\s*$/;
 var ASSIGN_RE = /^[A-Za-z_$][\w.$[\]]*\s*(?:=(?![=>])|\+=|-=|\*=)\s*\S.*[;)}\]]\s*$/;
+var DECLARATION_RE = /^(?:export\s+)?(?:declare\s+)?(?:const|let|var)\s+[A-Za-z_$][\w$]*(?:\s*:\s*[^=]+)?\s*=\s*(?:[A-Za-z_$][\w.$]*(?:\s*\(|\s*$)|["'`]|\[|\{|\d|true\b|false\b|null\b|undefined\b|new\b|await\b|async\b|function\b|class\b)/;
 var CALL_RE = /^[A-Za-z_$][\w.$]*\([^)]*\)\s*;?\s*$/;
 var ASSERTION_CODE_RE = /^(?:await\s+)?(?:expect(?:TypeOf)?|assert(?:\.\w+)?)\s*\(/;
 var HTTP_CONTRACT_RE = /\b(?:GET|HEAD|OPTIONS|PATCH|POST|PUT|DELETE)\s+(?:https?:\/\/|\/|\{[A-Za-z_$])/;
@@ -1254,6 +1256,7 @@ function looksLikeCode(text, allowCall = true) {
   if (!t) return false;
   if (PROSE_ASSIGNMENT_RE.test(t)) return false;
   if (CODE_KEYWORD_RE.test(t) && CODE_TAIL_RE.test(t)) return true;
+  if (DECLARATION_RE.test(t)) return true;
   if (ASSIGN_RE.test(t)) return true;
   if (ASSERTION_CODE_RE.test(t)) return true;
   return allowCall && CALL_RE.test(t);
@@ -1583,6 +1586,11 @@ var no_comment_cruft_default = createRule({
           }
           if (wallMembers.has(comment)) continue;
           if (isJsDoc(comment)) {
+            const debt = comment.value.split("\n").map(stripCommentMarker).find((line) => JSDOC_DEBT_RE.test(line));
+            if (debt !== void 0 && !runCitesAReference(comments, i)) {
+              context.report({ node: comment, messageId: "untrackedTodo" });
+              continue;
+            }
             if (isStandalone(comment) && isSectionJsDoc(comment)) {
               context.report({ node: comment, messageId: "sectionBanner" });
             }
@@ -4391,11 +4399,11 @@ function typedFunction(node) {
 
 // src/rules/no-long-comment.ts
 var noLongCommentDocumentation = {
-  summary: "Flag unusually large unstructured prose blocks in implementation code.",
+  summary: "Flag unusually large unstructured JSDoc blocks in implementation code.",
   rationale: "Large narrative comments become stale and obscure the local facts that belong beside the code.",
   remediation: "Keep only durable local constraints and express the remaining behavior in code.",
   category: "maintainability",
-  limitations: ["Structured API docs, tests, scripts, generated files, and versioned dependencies are excluded."],
+  limitations: ["Only JSDoc blocks are inspected; structured API docs, tests, scripts, generated files, and versioned dependencies are excluded."],
   examples: [
     { id: "local-fact", title: "Keep a concise local fact", outcome: "no-match", files: [{ path: "src/cache.ts", source: "// The cache is process local.\nconst cache = new Map();" }], focusPath: "src/cache.ts", expectedCount: 0, public: true },
     { id: "prose-wall", title: "Avoid an unstructured prose wall", outcome: "match", files: [{ path: "src/chart.ts", source: "/** One. Two. Three. Four. Five. Six. Seven. Eight. */\nconst chart = createChart();" }], focusPath: "src/chart.ts", expectedCount: 1, public: true }
@@ -4423,7 +4431,7 @@ var no_long_comment_default = createRule({
   documentation: noLongCommentDocumentation,
   meta: {
     type: "suggestion",
-    docs: { description: "Flag unusually large unstructured prose blocks in implementation code." },
+    docs: { description: "Flag unusually large unstructured JSDoc blocks in implementation code." },
     schema: [],
     messages: {
       tooLong: "Comment is an unusually large prose block \u2014 keep the local facts and clarify the code itself."
@@ -5368,6 +5376,7 @@ var MIN_DISTINCT_SCOPES = 2;
 var PREVIEW_LENGTH = 40;
 var SQL_KEYWORD_RE = /\b(SELECT|INSERT|UPDATE|DELETE|FROM|WHERE|JOIN|VALUES|ON CONFLICT|RETURNING|GROUP BY|ORDER BY)\b/;
 var IDENTIFIER_RE = /^[a-z_][a-z0-9_.]*$/;
+var URL_PATH_RE = /^\/(?=[^\s]*[A-Za-z0-9])[A-Za-z0-9._~!$&'()*+,;=:@%/?#{}\u005B\u005D-]+$/;
 var FUNCTION_TYPES4 = /* @__PURE__ */ new Set([
   AST_NODE_TYPES21.FunctionDeclaration,
   AST_NODE_TYPES21.FunctionExpression,
@@ -5375,7 +5384,7 @@ var FUNCTION_TYPES4 = /* @__PURE__ */ new Set([
 ]);
 var noRepeatedStringLiteralDocumentation = {
   summary: "Disallow a long structured string literal repeated across functions; the copies drift when one is edited. Extract a module-level constant.",
-  rationale: "Independent copies of a structured value can diverge and silently change behavior.",
+  rationale: "Independent copies of a query, route template, or identifier can diverge and silently change behavior.",
   remediation: "Extract the repeated value to one module-level constant and reference it from each function.",
   category: "maintainability",
   limitations: ["Test files, short strings, prose, substitutions, module sources, JSX attributes, and repetition within one function are excluded."],
@@ -5385,7 +5394,7 @@ var noRepeatedStringLiteralDocumentation = {
   ]
 };
 function isStructured(value) {
-  return value.includes("\n") || SQL_KEYWORD_RE.test(value) || IDENTIFIER_RE.test(value);
+  return value.includes("\n") || SQL_KEYWORD_RE.test(value) || IDENTIFIER_RE.test(value) || URL_PATH_RE.test(value);
 }
 function preview(value) {
   const oneLine = value.replaceAll("\n", " ").trim();
@@ -5404,8 +5413,9 @@ function isScaffolding(node) {
   if (parent === void 0) {
     return true;
   }
+  const isNonComputedPropertyKey = (parent.type === AST_NODE_TYPES21.Property || parent.type === AST_NODE_TYPES21.PropertyDefinition || parent.type === AST_NODE_TYPES21.MethodDefinition || parent.type === AST_NODE_TYPES21.AccessorProperty) && parent.key === node && !parent.computed;
   const isRequireSource = parent.type === AST_NODE_TYPES21.CallExpression && parent.callee.type === AST_NODE_TYPES21.Identifier && parent.callee.name === "require";
-  return parent.type === AST_NODE_TYPES21.ImportDeclaration || parent.type === AST_NODE_TYPES21.ImportExpression || parent.type === AST_NODE_TYPES21.ExportNamedDeclaration || parent.type === AST_NODE_TYPES21.ExportAllDeclaration || parent.type === AST_NODE_TYPES21.TSImportType || parent.type === AST_NODE_TYPES21.JSXAttribute || parent.type === AST_NODE_TYPES21.TSLiteralType || isRequireSource;
+  return parent.type === AST_NODE_TYPES21.ImportDeclaration || parent.type === AST_NODE_TYPES21.ImportExpression || parent.type === AST_NODE_TYPES21.ExportNamedDeclaration || parent.type === AST_NODE_TYPES21.ExportAllDeclaration || parent.type === AST_NODE_TYPES21.TSImportType || parent.type === AST_NODE_TYPES21.JSXAttribute || parent.type === AST_NODE_TYPES21.TSLiteralType || isNonComputedPropertyKey || isRequireSource;
 }
 var no_repeated_string_literal_default = createRule({
   name: "no-repeated-string-literal",
@@ -5422,7 +5432,7 @@ var no_repeated_string_literal_default = createRule({
   },
   defaultOptions: [],
   create(context) {
-    if (isTestFile(context.filename)) {
+    if (isTestFile(context.filename) || isGeneratedFile(context.filename, context.sourceCode.text)) {
       return {};
     }
     const occurrences = /* @__PURE__ */ new Map();
@@ -5742,10 +5752,14 @@ var no_restated_jsdoc_default = createRule({
         for (const comment of sourceCode.getAllComments()) {
           if (comment.type !== "Block" || !comment.value.startsWith("*")) continue;
           const { description, tags } = parseJsDoc(comment.value);
-          if (DIRECTIVE_RE4.test(description)) continue;
+          const describedText = [
+            description,
+            ...tags.filter((tag) => tag.name === "description").map((tag) => tag.text)
+          ].filter((text) => text.length > 0).join("\n");
+          if (DIRECTIVE_RE4.test(describedText)) continue;
           const tagNames = new Set(tags.map((tag) => tag.name));
           if ([...tagNames].some((name) => !MODELLED_TAGS.has(name))) continue;
-          if (isProtected(description)) continue;
+          if (isProtected(describedText)) continue;
           const token = sourceCode.getTokenAfter(comment, { includeComments: false });
           if (token === null || token.loc.start.line !== comment.loc.end.line + 1) continue;
           let node = sourceCode.getNodeByRangeIndex(token.range[0]);
@@ -5758,13 +5772,14 @@ var no_restated_jsdoc_default = createRule({
           if (declaration === null) continue;
           const paramTags = tags.filter((tag) => PARAM_TAGS.has(tag.name));
           const returnTags = tags.filter((tag) => RETURN_TAGS.has(tag.name));
-          if (description.length === 0 && paramTags.length === 0 && returnTags.length === 0) {
+          if (describedText.length === 0 && paramTags.length === 0 && returnTags.length === 0) {
             continue;
           }
+          if ((paramTags.length > 0 || returnTags.length > 0) && documentsTypedFunction(sourceCode, comment)) continue;
           const nameTokens = tokensOf([declaration.name]);
           const paramTokens = tokensOf(declaration.params);
           const known = /* @__PURE__ */ new Set([...nameTokens, ...paramTokens]);
-          let addsNothing = covered(description, known);
+          let addsNothing = covered(describedText, known);
           for (const tag of paramTags) {
             const text = tag.text.replace(/^\{[^}]*\}\s*/, "");
             const match = /^\[?([A-Za-z_$][\w.$]*)\]?\s*-?\s*([\s\S]*)$/.exec(text);
@@ -5772,7 +5787,13 @@ var no_restated_jsdoc_default = createRule({
               addsNothing = false;
               break;
             }
-            const own = /* @__PURE__ */ new Set([...splitIdentifier(match[1]?.split(".").pop() ?? ""), ...nameTokens]);
+            const path = match[1] ?? "";
+            const root = path.split(".")[0] ?? "";
+            if (!declaration.params.includes(root)) {
+              addsNothing = false;
+              break;
+            }
+            const own = /* @__PURE__ */ new Set([...splitIdentifier(path.split(".").pop() ?? ""), ...nameTokens]);
             if (!covered(match[2] ?? "", own)) {
               addsNothing = false;
               break;
@@ -6597,10 +6618,10 @@ var no_sentinel_return_on_catch_default = createRule({
           return false;
         }
         if (matcher.isLoggingCall(current)) {
-          return true;
+          return caughtName === null || argsIncludeBinding(current.arguments, caughtName);
         }
         const name = calleeName2(current.callee);
-        return name !== null && REPORT_NAME_RE.test(name) && argsIncludeBinding(current.arguments, caughtName);
+        return name !== null && REPORT_NAME_RE.test(name) && (caughtName === null || argsIncludeBinding(current.arguments, caughtName));
       });
     }
     return {
@@ -8069,7 +8090,7 @@ var no_type_member_comment_wall_default = createRule({
         claimed.add(comment);
         commented += 1;
         const body2 = commentBody(comment);
-        if (body2.length === 0 || carriesValue(body2)) continue;
+        if (body2.length === 0 || carriesValue(body2) || isTagsOnly(body2)) continue;
         if (novelWords(body2, knownTokens(sourceCode.getText(member))) <= options.maxNovelWords) {
           restated += 1;
         }
@@ -13987,7 +14008,7 @@ var rules = {
 };
 var meta = {
   name: "@sarj/eslint-plugin",
-  version: "13.0.0"
+  version: "13.1.0"
 };
 var applicationOnlyRules = [
   "no-restricted-library-load",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sarj/eslint-plugin",
-  "version": "13.0.0",
+  "version": "13.1.0",
   "packageManager": "npm@11.19.0",
   "description": "Custom ESLint rules for hypermodern TypeScript / React / Next.js projects",
   "type": "module",