uilint 0.2.5 → 0.2.7

package/dist/index.js

@@ -1913,6 +1913,35 @@ function hasSpreadProperties(obj) {
     (p2) => p2 && (p2.type === "SpreadElement" || p2.type === "SpreadProperty")
   );
 }
+var IGNORED_AST_KEYS = /* @__PURE__ */ new Set([
+  "loc",
+  "start",
+  "end",
+  "extra",
+  "leadingComments",
+  "trailingComments",
+  "innerComments"
+]);
+function normalizeAstForCompare(node) {
+  if (node === null) return null;
+  if (node === void 0) return void 0;
+  if (typeof node !== "object") return node;
+  if (Array.isArray(node)) return node.map(normalizeAstForCompare);
+  const out = {};
+  const keys = Object.keys(node).filter((k) => !IGNORED_AST_KEYS.has(k)).sort();
+  for (const k of keys) {
+    if (k.startsWith("$")) continue;
+    out[k] = normalizeAstForCompare(node[k]);
+  }
+  return out;
+}
+function astEquivalent(a, b) {
+  try {
+    return JSON.stringify(normalizeAstForCompare(a)) === JSON.stringify(normalizeAstForCompare(b));
+  } catch {
+    return false;
+  }
+}
 function collectUilintRuleIdsFromRulesObject(rulesObj) {
   const ids = /* @__PURE__ */ new Set();
   if (!rulesObj || rulesObj.type !== "ObjectExpression") return ids;
@@ -2209,8 +2238,46 @@ function extractConfiguredUilintRuleIds(source) {
 function getMissingSelectedRules(selectedRules, configuredIds) {
   return selectedRules.filter((r) => !configuredIds.has(r.id));
 }
-function getRulesNeedingUpdate(selectedRules, configuredIds) {
-  return selectedRules.filter((r) => configuredIds.has(r.id));
+function buildDesiredRuleValueExpression(rule) {
+  if (rule.defaultOptions && rule.defaultOptions.length > 0) {
+    return `["${rule.defaultSeverity}", ...${JSON.stringify(
+      rule.defaultOptions,
+      null,
+      2
+    )}]`;
+  }
+  return `"${rule.defaultSeverity}"`;
+}
+function collectUilintRuleValueNodesFromConfigArray(arrayExpr) {
+  const out = /* @__PURE__ */ new Map();
+  if (!arrayExpr || arrayExpr.type !== "ArrayExpression") return out;
+  for (const el of arrayExpr.elements ?? []) {
+    if (!el || el.type !== "ObjectExpression") continue;
+    const rules = getObjectPropertyValue(el, "rules");
+    if (!rules || rules.type !== "ObjectExpression") continue;
+    for (const prop of rules.properties ?? []) {
+      if (!prop) continue;
+      if (prop.type !== "ObjectProperty" && prop.type !== "Property") continue;
+      const key = prop.key;
+      if (!isStringLiteral(key)) continue;
+      const k = key.value;
+      if (typeof k !== "string" || !k.startsWith("uilint/")) continue;
+      const id = k.slice("uilint/".length);
+      if (!out.has(id)) out.set(id, prop.value);
+    }
+  }
+  return out;
+}
+function getRulesNeedingUpdate(selectedRules, configuredIds, arrayExpr) {
+  const existingVals = collectUilintRuleValueNodesFromConfigArray(arrayExpr);
+  return selectedRules.filter((r) => {
+    if (!configuredIds.has(r.id)) return false;
+    const existing = existingVals.get(r.id);
+    if (!existing) return true;
+    const desiredExpr = buildDesiredRuleValueExpression(r);
+    const desiredAst = parseExpression(desiredExpr).$ast;
+    return !astEquivalent(existing, desiredAst);
+  });
 }
 async function installEslintPlugin(opts) {
   const configPath = findEslintConfigFile(opts.projectPath);
@@ -2243,7 +2310,8 @@ async function installEslintPlugin(opts) {
   );
   const rulesToUpdate = getRulesNeedingUpdate(
     opts.selectedRules,
-    configuredIds
+    configuredIds,
+    arrayExpr
   );
   let rulesToApply = [];
   if (!info.configured) {