@sarj/eslint-plugin 16.22.2 → 16.23.0

package/dist/index.cjs

@@ -866,7 +866,7 @@ function isSearchParamReceiver(call, services) {
 // package.json
 var package_default = {
   name: "@sarj/eslint-plugin",
-  version: "16.22.2",
+  version: "16.23.0",
   packageManager: "npm@12.2.0",
   description: "Custom ESLint rules for hypermodern TypeScript / React / Next.js projects",
   type: "module",
@@ -901,7 +901,7 @@ var package_default = {
   scripts: {
     build: "tsup",
     dogfood: "npm run build && npm run dogfood:built",
-    lint: "eslint . ../../.github/scripts ../standards/src ../standards/eslint.config.mts --max-warnings 0",
+    lint: 'node ../standards/src/sarj_standards/configs/eslint-selected-rules.mts "{\\"full\\":true,\\"config\\":null,\\"rules\\":[],\\"consoleOutput\\":true,\\"failOnWarnings\\":true}" -- . ../../.github/scripts ../standards/src ../standards/eslint.config.mts',
     prepack: "npm run build && npm run verify-package",
     test: "vitest run",
     typecheck: "tsc --noEmit && tsc --project tsconfig.tools.json",
@@ -11273,7 +11273,7 @@ var no_long_comment_default = createRule({
 // src/rules/no-vague-suppression-description.ts
 var ESLINT_DIRECTIVE_WITH_DESCRIPTION_RE = /^eslint-(?:disable|disable-next-line|disable-line)\b[^:\n]*?\s*(?::|--)\s*(.+?)\s*$/iu;
 var TS_EXPECT_ERROR_WITH_DESCRIPTION_RE = /^@ts-expect-error\b(?:(?:\s*(?::|--)\s*)|\s+)(.+?)\s*$/iu;
-var VAGUE_RE = /^(?:needed|required|intentional(?:ly)?|ignore(?:d)?|false positive|type error|typescript|to satisfy (?:the )?(?:linter|typescript|type checker))\.?$/iu;
+var VAGUE_RE = /^(?:needed|required|same|n\/a|legacy|intentional(?:ly)?|ignore(?:d)?|false positive|type error|typescript|to satisfy (?:the )?(?:linter|typescript|type checker))\.?$/iu;
 var NO_VAGUE_SUPPRESSION_DESCRIPTION_DOCUMENTATION = {
   summary: "Require suppression descriptions to name the concrete mismatch or invariant instead of a generic non-reason.",
   rationale: "Generic phrases satisfy require-description mechanically while leaving reviewers unable to audit the risk or remove stale debt.",
@@ -11282,7 +11282,7 @@ var NO_VAGUE_SUPPRESSION_DESCRIPTION_DOCUMENTATION = {
   limitations: [
     "Only ESLint disable comments and TypeScript expect-error directives are checked.",
     "The rule uses a small anchored vocabulary and does not score prose quality generally.",
-    "Generated files and descriptions outside that exact vocabulary are excluded; an unflagged reason is not proof of a justified suppression. Missing descriptions remain owned by the upstream description requirement."
+    "Descriptions outside that exact vocabulary are excluded; an unflagged reason is not proof of a justified suppression. Missing descriptions remain owned by the upstream description requirement. Explicit path exclusions own generated-source boundaries; a source marker does not waive suppression accountability."
   ],
   examples: [
     {
@@ -11333,15 +11333,13 @@ var no_vague_suppression_description_default = createRule({
   },
   defaultOptions: [],
   create(context) {
-    if (isGeneratedFile(context.filename, context.sourceCode.text)) {
-      return {};
-    }
     return {
       Program() {
         for (const comment of context.sourceCode.getAllComments()) {
           const text = comment.value.trim();
           const description = suppressionDescription(text);
-          if (description === void 0 || !VAGUE_RE.test(description)) continue;
+          if (description === void 0 || !VAGUE_RE.test(description))
+            continue;
           context.report({
             loc: comment.loc,
             messageId: "vagueDescription",
@@ -27229,6 +27227,17 @@ function stringValue(node) {
   }
   return null;
 }
+function staticPathSuffix(pathArguments, builder) {
+  const trailing = [];
+  for (const argument of pathArguments.toReversed()) {
+    if (argument.type === import_utils143.AST_NODE_TYPES.SpreadElement) return null;
+    const segment = stringValue(argument);
+    if (segment === null) break;
+    trailing.unshift(segment);
+    if (builder === "resolve" && import_node_path3.posix.isAbsolute(segment.replaceAll("\\", "/"))) break;
+  }
+  return trailing.length > 0 ? trailing.join("/") : null;
+}
 function importSource(node) {
   return typeof node.source.value === "string" ? node.source.value : null;
 }
@@ -27291,26 +27300,18 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
         const value = stringValue(current);
         if (value !== null) return sourcePathValue(value);
         if (current.type === import_utils143.AST_NODE_TYPES.Identifier) return visible("paths", current);
-        if (current.type === import_utils143.AST_NODE_TYPES.CallExpression || current.type === import_utils143.AST_NODE_TYPES.NewExpression) {
-          const callee = unwrapExpression(current.callee);
-          const first = current.arguments[0];
-          if (first === void 0 || first.type === import_utils143.AST_NODE_TYPES.SpreadElement) return false;
-          const pathBuilder = callee.type === import_utils143.AST_NODE_TYPES.Identifier ? nativePathBuilder(callee) : null;
-          if (current.type === import_utils143.AST_NODE_TYPES.CallExpression && pathBuilder !== null) {
-            if (temporaryDescendant(current, pathBuilder)) return false;
-            const trailing = [];
-            for (const argument of current.arguments.toReversed()) {
-              if (argument.type === import_utils143.AST_NODE_TYPES.SpreadElement) return false;
-              const segment = stringValue(argument);
-              if (segment === null) break;
-              trailing.unshift(segment);
-              if (pathBuilder === "resolve" && import_node_path3.posix.isAbsolute(segment.replaceAll("\\", "/"))) break;
-            }
-            return trailing.length > 0 && sourcePathValue(trailing.join("/"));
-          }
-          if (current.type === import_utils143.AST_NODE_TYPES.NewExpression && callee.type === import_utils143.AST_NODE_TYPES.Identifier && callee.name === "URL" && (bindingOf(callee)?.defs.length ?? 0) === 0) return sourcePath(first);
-          if (callee.type === import_utils143.AST_NODE_TYPES.Identifier && bindingOf(callee)?.defs.some((definition) => definition.node.type === import_utils143.AST_NODE_TYPES.ImportSpecifier && definition.node.imported.type === import_utils143.AST_NODE_TYPES.Identifier && definition.node.imported.name === "fileURLToPath" && definition.node.parent.type === import_utils143.AST_NODE_TYPES.ImportDeclaration && ["node:url", "url"].includes(String(definition.node.parent.source.value)))) return sourcePath(first);
-        }
+        if (current.type !== import_utils143.AST_NODE_TYPES.CallExpression && current.type !== import_utils143.AST_NODE_TYPES.NewExpression) return false;
+        const callee = unwrapExpression(current.callee);
+        const first = current.arguments[0];
+        if (first === void 0 || first.type === import_utils143.AST_NODE_TYPES.SpreadElement) return false;
+        const pathBuilder = callee.type === import_utils143.AST_NODE_TYPES.Identifier ? nativePathBuilder(callee) : null;
+        if (current.type === import_utils143.AST_NODE_TYPES.CallExpression && pathBuilder !== null) {
+          if (temporaryDescendant(current, pathBuilder)) return false;
+          const suffix = staticPathSuffix(current.arguments, pathBuilder);
+          return suffix !== null && sourcePathValue(suffix);
+        }
+        if (current.type === import_utils143.AST_NODE_TYPES.NewExpression && callee.type === import_utils143.AST_NODE_TYPES.Identifier && callee.name === "URL" && (bindingOf(callee)?.defs.length ?? 0) === 0) return sourcePath(first);
+        if (callee.type === import_utils143.AST_NODE_TYPES.Identifier && bindingOf(callee)?.defs.some((definition) => definition.node.type === import_utils143.AST_NODE_TYPES.ImportSpecifier && definition.node.imported.type === import_utils143.AST_NODE_TYPES.Identifier && definition.node.imported.name === "fileURLToPath" && definition.node.parent.type === import_utils143.AST_NODE_TYPES.ImportDeclaration && ["node:url", "url"].includes(String(definition.node.parent.source.value)))) return sourcePath(first);
         return false;
       };
       const nativePathBuilder = (node) => {

package/dist/index.js

@@ -819,7 +819,7 @@ function isSearchParamReceiver(call, services) {
 // package.json
 var package_default = {
   name: "@sarj/eslint-plugin",
-  version: "16.22.2",
+  version: "16.23.0",
   packageManager: "npm@12.2.0",
   description: "Custom ESLint rules for hypermodern TypeScript / React / Next.js projects",
   type: "module",
@@ -854,7 +854,7 @@ var package_default = {
   scripts: {
     build: "tsup",
     dogfood: "npm run build && npm run dogfood:built",
-    lint: "eslint . ../../.github/scripts ../standards/src ../standards/eslint.config.mts --max-warnings 0",
+    lint: 'node ../standards/src/sarj_standards/configs/eslint-selected-rules.mts "{\\"full\\":true,\\"config\\":null,\\"rules\\":[],\\"consoleOutput\\":true,\\"failOnWarnings\\":true}" -- . ../../.github/scripts ../standards/src ../standards/eslint.config.mts',
     prepack: "npm run build && npm run verify-package",
     test: "vitest run",
     typecheck: "tsc --noEmit && tsc --project tsconfig.tools.json",
@@ -11258,7 +11258,7 @@ var no_long_comment_default = createRule({
 // src/rules/no-vague-suppression-description.ts
 var ESLINT_DIRECTIVE_WITH_DESCRIPTION_RE = /^eslint-(?:disable|disable-next-line|disable-line)\b[^:\n]*?\s*(?::|--)\s*(.+?)\s*$/iu;
 var TS_EXPECT_ERROR_WITH_DESCRIPTION_RE = /^@ts-expect-error\b(?:(?:\s*(?::|--)\s*)|\s+)(.+?)\s*$/iu;
-var VAGUE_RE = /^(?:needed|required|intentional(?:ly)?|ignore(?:d)?|false positive|type error|typescript|to satisfy (?:the )?(?:linter|typescript|type checker))\.?$/iu;
+var VAGUE_RE = /^(?:needed|required|same|n\/a|legacy|intentional(?:ly)?|ignore(?:d)?|false positive|type error|typescript|to satisfy (?:the )?(?:linter|typescript|type checker))\.?$/iu;
 var NO_VAGUE_SUPPRESSION_DESCRIPTION_DOCUMENTATION = {
   summary: "Require suppression descriptions to name the concrete mismatch or invariant instead of a generic non-reason.",
   rationale: "Generic phrases satisfy require-description mechanically while leaving reviewers unable to audit the risk or remove stale debt.",
@@ -11267,7 +11267,7 @@ var NO_VAGUE_SUPPRESSION_DESCRIPTION_DOCUMENTATION = {
   limitations: [
     "Only ESLint disable comments and TypeScript expect-error directives are checked.",
     "The rule uses a small anchored vocabulary and does not score prose quality generally.",
-    "Generated files and descriptions outside that exact vocabulary are excluded; an unflagged reason is not proof of a justified suppression. Missing descriptions remain owned by the upstream description requirement."
+    "Descriptions outside that exact vocabulary are excluded; an unflagged reason is not proof of a justified suppression. Missing descriptions remain owned by the upstream description requirement. Explicit path exclusions own generated-source boundaries; a source marker does not waive suppression accountability."
   ],
   examples: [
     {
@@ -11318,15 +11318,13 @@ var no_vague_suppression_description_default = createRule({
   },
   defaultOptions: [],
   create(context) {
-    if (isGeneratedFile(context.filename, context.sourceCode.text)) {
-      return {};
-    }
     return {
       Program() {
         for (const comment of context.sourceCode.getAllComments()) {
           const text = comment.value.trim();
           const description = suppressionDescription(text);
-          if (description === void 0 || !VAGUE_RE.test(description)) continue;
+          if (description === void 0 || !VAGUE_RE.test(description))
+            continue;
           context.report({
             loc: comment.loc,
             messageId: "vagueDescription",
@@ -27253,6 +27251,17 @@ function stringValue(node) {
   }
   return null;
 }
+function staticPathSuffix(pathArguments, builder) {
+  const trailing = [];
+  for (const argument of pathArguments.toReversed()) {
+    if (argument.type === AST_NODE_TYPES122.SpreadElement) return null;
+    const segment = stringValue(argument);
+    if (segment === null) break;
+    trailing.unshift(segment);
+    if (builder === "resolve" && posix2.isAbsolute(segment.replaceAll("\\", "/"))) break;
+  }
+  return trailing.length > 0 ? trailing.join("/") : null;
+}
 function importSource(node) {
   return typeof node.source.value === "string" ? node.source.value : null;
 }
@@ -27315,26 +27324,18 @@ function createSourceCoupledRule(name, documentation, sourceSuffixRe) {
         const value = stringValue(current);
         if (value !== null) return sourcePathValue(value);
         if (current.type === AST_NODE_TYPES122.Identifier) return visible("paths", current);
-        if (current.type === AST_NODE_TYPES122.CallExpression || current.type === AST_NODE_TYPES122.NewExpression) {
-          const callee = unwrapExpression(current.callee);
-          const first = current.arguments[0];
-          if (first === void 0 || first.type === AST_NODE_TYPES122.SpreadElement) return false;
-          const pathBuilder = callee.type === AST_NODE_TYPES122.Identifier ? nativePathBuilder(callee) : null;
-          if (current.type === AST_NODE_TYPES122.CallExpression && pathBuilder !== null) {
-            if (temporaryDescendant(current, pathBuilder)) return false;
-            const trailing = [];
-            for (const argument of current.arguments.toReversed()) {
-              if (argument.type === AST_NODE_TYPES122.SpreadElement) return false;
-              const segment = stringValue(argument);
-              if (segment === null) break;
-              trailing.unshift(segment);
-              if (pathBuilder === "resolve" && posix2.isAbsolute(segment.replaceAll("\\", "/"))) break;
-            }
-            return trailing.length > 0 && sourcePathValue(trailing.join("/"));
-          }
-          if (current.type === AST_NODE_TYPES122.NewExpression && callee.type === AST_NODE_TYPES122.Identifier && callee.name === "URL" && (bindingOf(callee)?.defs.length ?? 0) === 0) return sourcePath(first);
-          if (callee.type === AST_NODE_TYPES122.Identifier && bindingOf(callee)?.defs.some((definition) => definition.node.type === AST_NODE_TYPES122.ImportSpecifier && definition.node.imported.type === AST_NODE_TYPES122.Identifier && definition.node.imported.name === "fileURLToPath" && definition.node.parent.type === AST_NODE_TYPES122.ImportDeclaration && ["node:url", "url"].includes(String(definition.node.parent.source.value)))) return sourcePath(first);
-        }
+        if (current.type !== AST_NODE_TYPES122.CallExpression && current.type !== AST_NODE_TYPES122.NewExpression) return false;
+        const callee = unwrapExpression(current.callee);
+        const first = current.arguments[0];
+        if (first === void 0 || first.type === AST_NODE_TYPES122.SpreadElement) return false;
+        const pathBuilder = callee.type === AST_NODE_TYPES122.Identifier ? nativePathBuilder(callee) : null;
+        if (current.type === AST_NODE_TYPES122.CallExpression && pathBuilder !== null) {
+          if (temporaryDescendant(current, pathBuilder)) return false;
+          const suffix = staticPathSuffix(current.arguments, pathBuilder);
+          return suffix !== null && sourcePathValue(suffix);
+        }
+        if (current.type === AST_NODE_TYPES122.NewExpression && callee.type === AST_NODE_TYPES122.Identifier && callee.name === "URL" && (bindingOf(callee)?.defs.length ?? 0) === 0) return sourcePath(first);
+        if (callee.type === AST_NODE_TYPES122.Identifier && bindingOf(callee)?.defs.some((definition) => definition.node.type === AST_NODE_TYPES122.ImportSpecifier && definition.node.imported.type === AST_NODE_TYPES122.Identifier && definition.node.imported.name === "fileURLToPath" && definition.node.parent.type === AST_NODE_TYPES122.ImportDeclaration && ["node:url", "url"].includes(String(definition.node.parent.source.value)))) return sourcePath(first);
         return false;
       };
       const nativePathBuilder = (node) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sarj/eslint-plugin",
-  "version": "16.22.2",
+  "version": "16.23.0",
   "packageManager": "npm@12.2.0",
   "description": "Custom ESLint rules for hypermodern TypeScript / React / Next.js projects",
   "type": "module",
@@ -35,7 +35,7 @@
   "scripts": {
     "build": "tsup",
     "dogfood": "npm run build && npm run dogfood:built",
-    "lint": "eslint . ../../.github/scripts ../standards/src ../standards/eslint.config.mts --max-warnings 0",
+    "lint": "node ../standards/src/sarj_standards/configs/eslint-selected-rules.mts \"{\\\"full\\\":true,\\\"config\\\":null,\\\"rules\\\":[],\\\"consoleOutput\\\":true,\\\"failOnWarnings\\\":true}\" -- . ../../.github/scripts ../standards/src ../standards/eslint.config.mts",
     "prepack": "npm run build && npm run verify-package",
     "test": "vitest run",
     "typecheck": "tsc --noEmit && tsc --project tsconfig.tools.json",