uilint 0.2.4 → 0.2.6

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) {
@@ -2923,10 +2991,34 @@ import { createRequire } from "module";
 var __filename2 = fileURLToPath3(import.meta.url);
 var __dirname2 = dirname8(__filename2);
 var require2 = createRequire(import.meta.url);
+function findNodeModulesPackageRoot(pkgName, startDir) {
+  let dir = startDir;
+  while (true) {
+    const candidate = join10(dir, "node_modules", pkgName);
+    if (existsSync11(join10(candidate, "package.json"))) return candidate;
+    const parent = dirname8(dir);
+    if (parent === dir) break;
+    dir = parent;
+  }
+  return null;
+}
 function getUilintEslintPackageRoot() {
-  const entry = require2.resolve("uilint-eslint");
-  const entryDir = dirname8(entry);
-  return dirname8(entryDir);
+  const fromCwd = findNodeModulesPackageRoot("uilint-eslint", process.cwd());
+  if (fromCwd) return fromCwd;
+  const fromHere = findNodeModulesPackageRoot("uilint-eslint", __dirname2);
+  if (fromHere) return fromHere;
+  try {
+    const entry = require2.resolve("uilint-eslint");
+    const entryDir = dirname8(entry);
+    return dirname8(entryDir);
+  } catch (e) {
+    const msg = e instanceof Error ? e.message : String(e);
+    throw new Error(
+      `Unable to locate uilint-eslint in node_modules (searched upwards from cwd and uilint's install path).
+Resolver error: ${msg}
+Fix: ensure uilint-eslint is installed in the target project (or workspace) and try again.`
+    );
+  }
 }
 function getUilintEslintSrcDir() {
   const devPath = join10(
@@ -3016,7 +3108,7 @@ function loadRule(ruleId, options = { typescript: true }) {
     const implPath = join10(rulesDir, `${ruleId}.js`);
     if (!existsSync11(implPath)) {
       throw new Error(
-        `Rule "${ruleId}" not found at ${implPath}. Make sure uilint-eslint has been built.`
+        `Rule "${ruleId}" not found at ${implPath}. For JavaScript-only projects, uilint-eslint must be built to include compiled rule files in dist/rules/. If you're developing uilint-eslint, run 'pnpm build' in packages/uilint-eslint. If you're using a published package, ensure it includes the dist/ directory.`
       );
     }
     const content = readFileSync7(implPath, "utf-8");
@@ -3264,28 +3356,25 @@ function createPlan(state, choices, options = {}) {
         path: rulesDir
       });
       const isTypeScript = pkgInfo?.isTypeScript ?? true;
-      try {
-        const ruleFiles = loadSelectedRules(
-          selectedRules.map((r) => r.id),
-          {
-            typescript: isTypeScript
-          }
-        );
-        for (const ruleFile of ruleFiles) {
+      const ruleFiles = loadSelectedRules(
+        selectedRules.map((r) => r.id),
+        {
+          typescript: isTypeScript
+        }
+      );
+      for (const ruleFile of ruleFiles) {
+        actions.push({
+          type: "create_file",
+          path: join11(rulesDir, ruleFile.implementation.relativePath),
+          content: ruleFile.implementation.content
+        });
+        if (ruleFile.test && isTypeScript) {
           actions.push({
             type: "create_file",
-            path: join11(rulesDir, ruleFile.implementation.relativePath),
-            content: ruleFile.implementation.content
+            path: join11(rulesDir, ruleFile.test.relativePath),
+            content: ruleFile.test.content
           });
-          if (ruleFile.test && isTypeScript) {
-            actions.push({
-              type: "create_file",
-              path: join11(rulesDir, ruleFile.test.relativePath),
-              content: ruleFile.test.content
-            });
-          }
         }
-      } catch {
       }
       dependencies.push({
         packagePath: pkgPath,