unguard 0.13.0 → 0.13.2

package/dist/{chunk-KNKWDZ6H.js → chunk-2QER6GG7.js}

@@ -632,6 +632,7 @@ var optionalArgAlwaysUsed = {
 
 // src/rules/cross-file/repeated-literal-property.ts
 import * as ts8 from "typescript";
+var MODULE_SCOPE = -1;
 var repeatedLiteralProperty = {
   id: "repeated-literal-property",
   severity: "warning",
@@ -651,7 +652,8 @@ var repeatedLiteralProperty = {
               valueMap.set(result.literalText, list);
             }
             const line = ts8.getLineAndCharacterOfPosition(sourceFile, prop.getStart(sourceFile)).line + 1;
-            list.push({ line, isAsConst: result.isAsConst });
+            const scopeId = findEnclosingFunctionStart(prop, sourceFile);
+            list.push({ line, isAsConst: result.isAsConst, scopeId });
           }
         }
         ts8.forEachChild(node, visit2);
@@ -661,31 +663,40 @@ var repeatedLiteralProperty = {
       ts8.forEachChild(sourceFile, visit2);
       for (const [value, occurrences] of valueMap) {
         const hasAsConst = occurrences.some((o) => o.isAsConst);
+        const uniqueScopes = new Set(occurrences.map((o) => o.scopeId));
         const threshold = hasAsConst ? 3 : 5;
-        if (occurrences.length < threshold) continue;
+        if (uniqueScopes.size < threshold) continue;
         const sorted = [...occurrences].sort((a, b) => a.line - b.line);
-        for (const occ of sorted) {
-          const otherLines = sorted.filter((o) => o !== occ).map((o) => o.line).join(", ");
-          diagnostics.push({
-            ruleId: this.id,
-            severity: this.severity,
-            message: `${JSON.stringify(value)}${hasAsConst ? " as const" : ""} repeated ${occurrences.length} times as property value (also at lines ${otherLines})`,
-            file,
-            line: occ.line,
-            column: 1
-          });
-        }
+        const first = sorted[0];
+        if (first === void 0) continue;
+        const otherLines = sorted.slice(1).map((o) => o.line).join(", ");
+        diagnostics.push({
+          ruleId: this.id,
+          severity: this.severity,
+          message: `${JSON.stringify(value)}${hasAsConst ? " as const" : ""} repeated across ${uniqueScopes.size} scopes as property value (also at lines ${otherLines})`,
+          file,
+          line: first.line,
+          column: 1
+        });
       }
     }
     return diagnostics;
   }
 };
+function findEnclosingFunctionStart(node, sourceFile) {
+  let current = node.parent;
+  while (current) {
+    if (ts8.isFunctionDeclaration(current) || ts8.isFunctionExpression(current) || ts8.isArrowFunction(current) || ts8.isMethodDeclaration(current)) {
+      return current.getStart(sourceFile);
+    }
+    current = current.parent;
+  }
+  return MODULE_SCOPE;
+}
 function extractLiteral(node, sourceFile) {
-  if (ts8.isStringLiteral(node) || ts8.isNumericLiteral(node) || ts8.isNoSubstitutionTemplateLiteral(node)) {
+  if (ts8.isStringLiteral(node) || ts8.isNoSubstitutionTemplateLiteral(node)) {
     return { literalText: node.text, isAsConst: false };
   }
-  if (node.kind === ts8.SyntaxKind.TrueKeyword) return { literalText: "true", isAsConst: false };
-  if (node.kind === ts8.SyntaxKind.FalseKeyword) return { literalText: "false", isAsConst: false };
   if (ts8.isAsExpression(node) && node.type.getText(sourceFile).trim() === "const") {
     const inner = extractLiteral(node.expression, sourceFile);
     if (inner.literalText !== null) {
@@ -763,17 +774,17 @@ var repeatedReturnShape = {
       const unique = [...byFunction.values()];
       if (unique.length < THRESHOLD) continue;
       const sorted = unique.sort((a, b) => a.file.localeCompare(b.file) || a.line - b.line);
-      for (const entry of sorted) {
-        const others = sorted.filter((e) => e !== entry).map((e) => `${e.functionName} (${e.file}:${e.line})`).join(", ");
-        diagnostics.push({
-          ruleId: this.id,
-          severity: this.severity,
-          message: `${unique.length} functions return shape {${entry.props.join(", ")}}; consider a shared return type (${others})`,
-          file: entry.file,
-          line: entry.line,
-          column: 1
-        });
-      }
+      const first = sorted[0];
+      if (first === void 0) continue;
+      const others = sorted.slice(1).map((e) => `${e.functionName} (${e.file}:${e.line})`).join(", ");
+      diagnostics.push({
+        ruleId: this.id,
+        severity: this.severity,
+        message: `${unique.length} functions return shape {${first.props.join(", ")}}; consider a shared return type (${others})`,
+        file: first.file,
+        line: first.line,
+        column: 1
+      });
     }
     return diagnostics;
   }
@@ -817,7 +828,7 @@ function collectReturnShapes(funcNode, file, funcLine, functionName, sourceFile,
 }
 function addShape(objLiteral, file, funcLine, functionName, sourceFile, shapeMap) {
   const props = extractPropertyNames(objLiteral);
-  if (props === null || props.length === 0) return;
+  if (props === null || props.length < 2) return;
   const { sorted, list } = getShapeGroup(shapeMap, props);
   list.push({ file, line: funcLine, functionName, props: sorted });
 }
@@ -2918,4 +2929,4 @@ export {
   executeScan,
   scan
 };
-//# sourceMappingURL=chunk-KNKWDZ6H.js.map
+//# sourceMappingURL=chunk-2QER6GG7.js.map