@sarj/eslint-plugin 2.3.1 → 2.3.2

package/dist/index.js

@@ -2223,7 +2223,15 @@ var PURE_CONSTRUCTORS = /* @__PURE__ */ new Set([
   "Object",
   "Headers",
   "URLSearchParams",
-  "FormData"
+  "FormData",
+  "TextEncoder",
+  "TextDecoder",
+  "Blob",
+  "ReadableStream",
+  "WritableStream",
+  "TransformStream",
+  "Response",
+  "AbortController"
 ]);
 function isNode3(value) {
   return typeof value === "object" && value !== null && typeof value.type === "string";
@@ -2413,7 +2421,49 @@ var INNOCUOUS_WORDS = /* @__PURE__ */ new Set([
   "invalid",
   "exists",
   "type",
-  "types"
+  "types",
+  "name",
+  "names",
+  "label",
+  "labels",
+  "title",
+  "expiry",
+  "expiration",
+  "expires",
+  "ttl",
+  "version",
+  "versions",
+  "policy",
+  "rotation",
+  "arn",
+  "path",
+  "paths",
+  "issuer",
+  "audience",
+  "strength",
+  "manager",
+  "service",
+  "services",
+  "repository",
+  "provider",
+  "providers",
+  "store",
+  "factory",
+  "handler",
+  "controller",
+  "bucket",
+  "url",
+  "uri",
+  "endpoint",
+  "endpoints",
+  "scope",
+  "scopes",
+  "event",
+  "events",
+  "format",
+  "at",
+  "len",
+  "length"
 ]);
 var REDACTION_RE = /prefix|suffix|redact|mask|hash|hint|_len|length/i;
 var WHOLE_TOKEN_REDACTION_MARKERS = /* @__PURE__ */ new Set(["tag"]);
@@ -2488,6 +2538,12 @@ function isLoggerExpr(expr) {
       return false;
   }
 }
+function isRawSecretValue(prop) {
+  if (prop.shorthand) {
+    return true;
+  }
+  return prop.value.type === "Identifier" || prop.value.type === "MemberExpression";
+}
 function propertyKeyName2(prop) {
   if (prop.computed) {
     return null;
@@ -2542,7 +2598,7 @@ var no_secret_in_log_default = ESLintUtils23.RuleCreator(
                 continue;
               }
               const keyName2 = propertyKeyName2(prop);
-              if (keyName2 !== null && isSecretKeyword(keyName2)) {
+              if (keyName2 !== null && isSecretKeyword(keyName2) && isRawSecretValue(prop)) {
                 context.report({
                   node: prop,
                   messageId: "noSecretInLog",
@@ -2578,6 +2634,10 @@ var CHOICE_TOKENS = /* @__PURE__ */ new Set([
 ]);
 var LOWER_TOKEN_RE = /^[a-z][a-z0-9_-]{0,30}$/;
 var MIN_CLUSTER_SIZE = 2;
+var BOOLEANISH = /* @__PURE__ */ new Set(["true", "false"]);
+function isEnumToken(lit) {
+  return LOWER_TOKEN_RE.test(lit) && lit.length >= 2 && !BOOLEANISH.has(lit);
+}
 var IGNORE_PATTERNS = [
   /[\\/]generated[\\/]/,
   /\.gen\.tsx?$/,
@@ -2607,14 +2667,89 @@ function keyName(key) {
   }
   return null;
 }
+function isStringLiteralMember(t) {
+  return t.type === AST_NODE_TYPES12.TSLiteralType && t.literal.type === AST_NODE_TYPES12.Literal && typeof t.literal.value === "string";
+}
 function isStringLiteralUnion(node) {
   if (node?.type !== AST_NODE_TYPES12.TSUnionType) {
     return false;
   }
-  const stringMembers = node.types.filter(
-    (t) => t.type === AST_NODE_TYPES12.TSLiteralType && t.literal.type === AST_NODE_TYPES12.Literal && typeof t.literal.value === "string"
-  );
-  return stringMembers.length >= MIN_CLUSTER_SIZE;
+  return node.types.filter(isStringLiteralMember).length >= MIN_CLUSTER_SIZE;
+}
+function isLikelyClosedUnion(node) {
+  return node?.type === AST_NODE_TYPES12.TSUnionType && node.types.some(isStringLiteralMember);
+}
+function unionMemberNames(typeNode, out) {
+  if (typeNode === void 0) {
+    return;
+  }
+  if (typeNode.type === AST_NODE_TYPES12.TSIntersectionType || typeNode.type === AST_NODE_TYPES12.TSUnionType) {
+    for (const t of typeNode.types) {
+      unionMemberNames(t, out);
+    }
+    return;
+  }
+  if (typeNode.type !== AST_NODE_TYPES12.TSTypeLiteral) {
+    return;
+  }
+  for (const member of typeNode.members) {
+    if (member.type === AST_NODE_TYPES12.TSPropertySignature && isLikelyClosedUnion(member.typeAnnotation?.typeAnnotation)) {
+      const propName2 = keyName(member.key);
+      if (propName2 !== null) {
+        out.add(propName2);
+      }
+    }
+  }
+}
+function bindingName(node) {
+  const id = node.type === AST_NODE_TYPES12.AssignmentPattern ? node.left : node;
+  return id.type === AST_NODE_TYPES12.Identifier ? id.name : null;
+}
+function addUnionRefsFromBinding(param, out) {
+  const binding = param.type === AST_NODE_TYPES12.AssignmentPattern ? param.left : param;
+  if (binding.type === AST_NODE_TYPES12.ObjectPattern) {
+    const members = /* @__PURE__ */ new Set();
+    unionMemberNames(binding.typeAnnotation?.typeAnnotation, members);
+    if (members.size === 0) {
+      return;
+    }
+    for (const prop of binding.properties) {
+      if (prop.type !== AST_NODE_TYPES12.Property) {
+        continue;
+      }
+      const propKey = keyName(prop.key);
+      const local = bindingName(prop.value);
+      if (propKey !== null && local !== null && members.has(propKey)) {
+        out.add(local);
+      }
+    }
+    return;
+  }
+  if (binding.type !== AST_NODE_TYPES12.Identifier) {
+    return;
+  }
+  const typeNode = binding.typeAnnotation?.typeAnnotation;
+  if (isLikelyClosedUnion(typeNode)) {
+    out.add(binding.name);
+    return;
+  }
+  if (typeNode?.type === AST_NODE_TYPES12.TSTypeLiteral) {
+    for (const member of typeNode.members) {
+      if (member.type === AST_NODE_TYPES12.TSPropertySignature && isLikelyClosedUnion(member.typeAnnotation?.typeAnnotation)) {
+        const propName2 = keyName(member.key);
+        if (propName2 !== null) {
+          out.add(`${binding.name}.${propName2}`);
+        }
+      }
+    }
+  }
+}
+function unionRefsFromParams(fn) {
+  const out = /* @__PURE__ */ new Set();
+  for (const param of fn.params) {
+    addUnionRefsFromBinding(param, out);
+  }
+  return out;
 }
 function refKey(node) {
   if (node.type === AST_NODE_TYPES12.Identifier) {
@@ -2661,16 +2796,25 @@ var prefer_string_literal_union_default = ESLintUtils24.RuleCreator(
     const validClusters = [];
     const bareChoiceProps = [];
     const containersWithUnion = /* @__PURE__ */ new Set();
-    function pushScope() {
-      scopeStack.push(/* @__PURE__ */ new Map());
+    function pushScope(fn) {
+      scopeStack.push({
+        clusters: /* @__PURE__ */ new Map(),
+        unionRefs: unionRefsFromParams(fn)
+      });
+    }
+    function isUnionTypedRef(key, current) {
+      if (current.has(key)) {
+        return true;
+      }
+      return scopeStack.some((s) => s.unionRefs.has(key));
     }
     function popScope() {
-      const clusters = scopeStack.pop();
-      if (clusters === void 0) {
+      const scope = scopeStack.pop();
+      if (scope === void 0) {
         return;
       }
-      for (const entry of clusters.values()) {
-        if (entry.allTokens && entry.literals.size >= MIN_CLUSTER_SIZE) {
+      for (const [key, entry] of scope.clusters) {
+        if (entry.allTokens && entry.literals.size >= MIN_CLUSTER_SIZE && !isUnionTypedRef(key, scope.unionRefs)) {
           validClusters.push(entry.node);
         }
       }
@@ -2680,10 +2824,10 @@ var prefer_string_literal_union_default = ESLintUtils24.RuleCreator(
       if (scope === void 0) {
         return;
       }
-      const allTokens = literals.every((lit) => LOWER_TOKEN_RE.test(lit));
-      const existing = scope.get(key);
+      const allTokens = literals.every((lit) => isEnumToken(lit));
+      const existing = scope.clusters.get(key);
       if (existing === void 0) {
-        scope.set(key, {
+        scope.clusters.set(key, {
           node,
           literals: new Set(literals),
           allTokens
@@ -2716,6 +2860,15 @@ var prefer_string_literal_union_default = ESLintUtils24.RuleCreator(
       "FunctionExpression:exit": popScope,
       ArrowFunctionExpression: pushScope,
       "ArrowFunctionExpression:exit": popScope,
+      VariableDeclarator(node) {
+        const scope = scopeStack[scopeStack.length - 1];
+        if (scope === void 0 || node.id.type !== AST_NODE_TYPES12.Identifier) {
+          return;
+        }
+        if (isLikelyClosedUnion(node.id.typeAnnotation?.typeAnnotation)) {
+          scope.unionRefs.add(node.id.name);
+        }
+      },
       BinaryExpression(node) {
         if (node.operator !== "===" && node.operator !== "!==" && node.operator !== "==" && node.operator !== "!=") {
           return;
@@ -2971,7 +3124,7 @@ var rules = {
 var plugin = {
   meta: {
     name: "@sarj/eslint-plugin",
-    version: "2.3.1"
+    version: "2.3.2"
   },
   rules,
   configs: {