uilint-eslint 0.2.67 → 0.2.68

package/README.md

@@ -27,7 +27,6 @@ export default [
   {
     plugins: { uilint: uilint.plugin },
     rules: {
-      "uilint/no-arbitrary-tailwind": "error",
       "uilint/consistent-spacing": [
         "warn",
         { scale: [0, 1, 2, 3, 4, 5, 6, 8, 10, 12, 16] },
@@ -48,7 +47,6 @@ export default [
 
 | Rule                           | Description                                             |
 | ------------------------------ | ------------------------------------------------------- |
-| `no-arbitrary-tailwind`        | Forbid arbitrary Tailwind values like `w-[123px]`       |
 | `consistent-spacing`           | Enforce spacing scale (no magic numbers in gap/padding) |
 | `no-direct-store-import`       | Forbid direct Zustand store imports (use hooks)         |
 | `no-mixed-component-libraries` | Forbid mixing shadcn and MUI components in same file    |
@@ -63,10 +61,6 @@ The `semantic` rule reads `.uilint/styleguide.md` and uses Ollama to analyze you
 
 ## Configuration
 
-### `no-arbitrary-tailwind`
-
-No options. Reports any use of Tailwind arbitrary values like `w-[100px]`, `bg-[#fff]`, etc.
-
 ### `consistent-spacing`
 
 ```javascript

package/dist/index.d.ts

@@ -72,7 +72,7 @@ interface RuleMigration {
  * making it easy to maintain and extend as new rules are added.
  */
 interface RuleMeta {
-    /** Rule identifier (e.g., "no-arbitrary-tailwind") - must match filename */
+    /** Rule identifier (e.g., "consistent-dark-mode") - must match filename */
     id: string;
     /** Semantic version of the rule (e.g., "1.0.0") */
     version: string;
@@ -672,9 +672,6 @@ declare function analyzeJSXElementCoverage(jsxNode: TSESTree.JSXElement, filePat
  * All available rules
  */
 declare const rules: {
-    "no-arbitrary-tailwind": _typescript_eslint_utils_ts_eslint.RuleModule<"noArbitraryValue", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener> & {
-        name: string;
-    };
     "consistent-dark-mode": _typescript_eslint_utils_ts_eslint.RuleModule<"inconsistentDarkMode" | "missingDarkMode", [({
         warnOnMissingDarkMode?: boolean;
     } | undefined)?], unknown, _typescript_eslint_utils_ts_eslint.RuleListener> & {
@@ -818,9 +815,6 @@ declare const plugin: {
         version: string;
     };
     rules: {
-        "no-arbitrary-tailwind": _typescript_eslint_utils_ts_eslint.RuleModule<"noArbitraryValue", [], unknown, _typescript_eslint_utils_ts_eslint.RuleListener> & {
-            name: string;
-        };
         "consistent-dark-mode": _typescript_eslint_utils_ts_eslint.RuleModule<"inconsistentDarkMode" | "missingDarkMode", [({
             warnOnMissingDarkMode?: boolean;
         } | undefined)?], unknown, _typescript_eslint_utils_ts_eslint.RuleListener> & {

package/dist/index.js

@@ -3,132 +3,12 @@ import { ESLintUtils } from "@typescript-eslint/utils";
 var createRule = ESLintUtils.RuleCreator(
   (name) => `https://github.com/peter-suggate/uilint/blob/main/packages/uilint-eslint/docs/rules/${name}.md`
 );
-function defineRuleMeta(meta18) {
-  return meta18;
-}
-
-// src/rules/no-arbitrary-tailwind.ts
-var meta = defineRuleMeta({
-  id: "no-arbitrary-tailwind",
-  version: "1.0.0",
-  name: "No Arbitrary Tailwind",
-  description: "Forbid arbitrary values like w-[123px], bg-[#fff]",
-  defaultSeverity: "error",
-  category: "static",
-  icon: "\u{1F3A8}",
-  hint: "Enforces design system tokens",
-  defaultEnabled: true,
-  docs: `
-## What it does
-
-Prevents the use of arbitrary Tailwind CSS values (bracket notation) in your codebase.
-Arbitrary values like \`w-[123px]\`, \`bg-[#ff5500]\`, or \`text-[14px]\` bypass Tailwind's
-design system and can lead to inconsistent UI.
-
-## Why it's useful
-
-- **Consistency**: Forces use of your design system's spacing, colors, and typography scales
-- **Maintainability**: Changes to design tokens automatically propagate everywhere
-- **Performance**: Arbitrary values generate extra CSS that can't be deduplicated
-
-## Examples
-
-### \u274C Incorrect
-
-\`\`\`tsx
-<div className="w-[123px] h-[456px]">     // Arbitrary dimensions
-<div className="bg-[#ff5500]">             // Arbitrary color
-<div className="text-[14px]">              // Arbitrary font size
-<div className="p-[7px]">                  // Arbitrary padding
-\`\`\`
-
-### \u2705 Correct
-
-\`\`\`tsx
-<div className="w-32 h-96">               // Use spacing scale
-<div className="bg-orange-500">           // Use color palette
-<div className="text-sm">                 // Use typography scale
-<div className="p-2">                     // Use spacing scale
-\`\`\`
-
-## Notes
-
-- Data attributes like \`data-[state=open]\` are allowed as they don't affect styling
-- If you need a truly custom value, consider extending your Tailwind config instead
-`
-});
-var ARBITRARY_VALUE_REGEX = /\b[\w-]+-\[[^\]]+\]/g;
-var no_arbitrary_tailwind_default = createRule({
-  name: "no-arbitrary-tailwind",
-  meta: {
-    type: "problem",
-    docs: {
-      description: "Forbid arbitrary Tailwind values like w-[123px]"
-    },
-    messages: {
-      noArbitraryValue: "Avoid arbitrary Tailwind value '{{value}}'. Use the spacing/color scale instead."
-    },
-    schema: []
-  },
-  defaultOptions: [],
-  create(context) {
-    return {
-      // Check className attributes in JSX
-      JSXAttribute(node) {
-        if (node.name.type === "JSXIdentifier" && (node.name.name === "className" || node.name.name === "class")) {
-          const value = node.value;
-          if (value?.type === "Literal" && typeof value.value === "string") {
-            checkClassString(context, value, value.value);
-          }
-          if (value?.type === "JSXExpressionContainer") {
-            const expr = value.expression;
-            if (expr.type === "Literal" && typeof expr.value === "string") {
-              checkClassString(context, expr, expr.value);
-            }
-            if (expr.type === "TemplateLiteral") {
-              for (const quasi of expr.quasis) {
-                checkClassString(context, quasi, quasi.value.raw);
-              }
-            }
-          }
-        }
-      },
-      // Check cn(), clsx(), classnames() calls
-      CallExpression(node) {
-        if (node.callee.type !== "Identifier") return;
-        const name = node.callee.name;
-        if (name === "cn" || name === "clsx" || name === "classnames") {
-          for (const arg of node.arguments) {
-            if (arg.type === "Literal" && typeof arg.value === "string") {
-              checkClassString(context, arg, arg.value);
-            }
-            if (arg.type === "TemplateLiteral") {
-              for (const quasi of arg.quasis) {
-                checkClassString(context, quasi, quasi.value.raw);
-              }
-            }
-          }
-        }
-      }
-    };
-  }
-});
-function checkClassString(context, node, classString) {
-  const matches = classString.matchAll(ARBITRARY_VALUE_REGEX);
-  for (const match of matches) {
-    if (match[0].startsWith("data-")) {
-      continue;
-    }
-    context.report({
-      node,
-      messageId: "noArbitraryValue",
-      data: { value: match[0] }
-    });
-  }
+function defineRuleMeta(meta17) {
+  return meta17;
 }
 
 // src/rules/consistent-dark-mode.ts
-var meta2 = defineRuleMeta({
+var meta = defineRuleMeta({
   id: "consistent-dark-mode",
   version: "1.0.0",
   name: "Consistent Dark Mode",
@@ -352,7 +232,7 @@ var consistent_dark_mode_default = createRule({
     let fileHasColorClasses = false;
     let fileHasDarkMode = false;
     const reportedNodes = /* @__PURE__ */ new Set();
-    function checkClassString2(node, classString) {
+    function checkClassString(node, classString) {
       const classes = classString.split(/\s+/).filter(Boolean);
       if (classes.length === 0) return;
       const prefixUsage = /* @__PURE__ */ new Map();
@@ -401,11 +281,11 @@ var consistent_dark_mode_default = createRule({
       }
     }
     function processStringValue(node, value) {
-      checkClassString2(node, value);
+      checkClassString(node, value);
     }
     function processTemplateLiteral(node) {
       for (const quasi of node.quasis) {
-        checkClassString2(quasi, quasi.value.raw);
+        checkClassString(quasi, quasi.value.raw);
       }
     }
     return {
@@ -466,7 +346,7 @@ var consistent_dark_mode_default = createRule({
 });
 
 // src/rules/no-direct-store-import.ts
-var meta3 = defineRuleMeta({
+var meta2 = defineRuleMeta({
   id: "no-direct-store-import",
   version: "1.0.0",
   name: "No Direct Store Import",
@@ -608,7 +488,7 @@ var no_direct_store_import_default = createRule({
 });
 
 // src/rules/prefer-zustand-state-management.ts
-var meta4 = defineRuleMeta({
+var meta3 = defineRuleMeta({
   id: "prefer-zustand-state-management",
   version: "1.0.0",
   name: "Prefer Zustand State Management",
@@ -1391,7 +1271,7 @@ function clearCache() {
 }
 
 // src/rules/no-mixed-component-libraries/index.ts
-var meta5 = defineRuleMeta({
+var meta4 = defineRuleMeta({
   id: "no-mixed-component-libraries",
   version: "1.0.0",
   name: "No Mixed Component Libraries",
@@ -1778,7 +1658,7 @@ function getStyleguide(startDir, explicitPath) {
 // src/rules/semantic/index.ts
 import { UILINT_DEFAULT_OLLAMA_MODEL } from "uilint-core";
 import { buildSourceScanPrompt } from "uilint-core";
-var meta6 = defineRuleMeta({
+var meta5 = defineRuleMeta({
   id: "semantic",
   version: "1.0.0",
   name: "Semantic Analysis",
@@ -2123,7 +2003,7 @@ function runSemanticAnalysisSync(sourceCode, styleguide, model, filePath) {
 // src/rules/semantic-vision.ts
 import { existsSync as existsSync5, readdirSync, readFileSync as readFileSync5 } from "fs";
 import { dirname as dirname5, join as join5, relative as relative2 } from "path";
-var meta7 = defineRuleMeta({
+var meta6 = defineRuleMeta({
   id: "semantic-vision",
   version: "1.0.0",
   name: "Vision Analysis",
@@ -2347,7 +2227,7 @@ var semantic_vision_default = createRule({
 });
 
 // src/rules/enforce-absolute-imports.ts
-var meta8 = defineRuleMeta({
+var meta7 = defineRuleMeta({
   id: "enforce-absolute-imports",
   version: "1.0.0",
   name: "Enforce Absolute Imports",
@@ -2523,7 +2403,7 @@ var enforce_absolute_imports_default = createRule({
 });
 
 // src/rules/no-any-in-props.ts
-var meta9 = defineRuleMeta({
+var meta8 = defineRuleMeta({
   id: "no-any-in-props",
   version: "1.0.0",
   name: "No Any in Props",
@@ -2873,7 +2753,7 @@ var no_any_in_props_default = createRule({
 });
 
 // src/rules/zustand-use-selectors.ts
-var meta10 = defineRuleMeta({
+var meta9 = defineRuleMeta({
   id: "zustand-use-selectors",
   version: "1.0.0",
   name: "Zustand Use Selectors",
@@ -3102,7 +2982,7 @@ var zustand_use_selectors_default = createRule({
 });
 
 // src/rules/no-prop-drilling-depth.ts
-var meta11 = defineRuleMeta({
+var meta10 = defineRuleMeta({
   id: "no-prop-drilling-depth",
   version: "1.0.0",
   name: "No Prop Drilling Depth",
@@ -3458,7 +3338,7 @@ var no_prop_drilling_depth_default = createRule({
 });
 
 // src/rules/no-secrets-in-code.ts
-var meta12 = defineRuleMeta({
+var meta11 = defineRuleMeta({
   id: "no-secrets-in-code",
   version: "1.0.0",
   name: "No Secrets in Code",
@@ -3813,7 +3693,7 @@ var no_secrets_in_code_default = createRule({
 });
 
 // src/rules/require-input-validation.ts
-var meta13 = defineRuleMeta({
+var meta12 = defineRuleMeta({
   id: "require-input-validation",
   version: "1.0.0",
   name: "Require Input Validation",
@@ -4155,7 +4035,7 @@ function log(message) {
   } catch {
   }
 }
-var meta14 = defineRuleMeta({
+var meta13 = defineRuleMeta({
   id: "no-semantic-duplicates",
   version: "1.0.0",
   name: "No Semantic Duplicates",
@@ -4316,8 +4196,8 @@ function loadIndex(projectRoot, indexPath) {
     log(`Loaded metadata.json: ${Object.keys(entries).length} entries`);
     const metadataStore = /* @__PURE__ */ new Map();
     const fileToChunks = /* @__PURE__ */ new Map();
-    for (const [id, meta18] of Object.entries(entries)) {
-      const m = meta18;
+    for (const [id, meta17] of Object.entries(entries)) {
+      const m = meta17;
       metadataStore.set(id, {
         filePath: m.filePath,
         startLine: m.startLine,
@@ -4407,9 +4287,9 @@ function findSimilarChunks(index, chunkId, threshold) {
   const sortedAll = allScores.sort((a, b) => b.score - a.score).slice(0, 10);
   log(`  Top 10 similarity scores (threshold=${threshold}):`);
   for (const { id, score } of sortedAll) {
-    const meta18 = index.metadataStore.get(id);
+    const meta17 = index.metadataStore.get(id);
     const meetsThreshold = score >= threshold ? "\u2713" : "\u2717";
-    log(`    ${meetsThreshold} ${(score * 100).toFixed(1)}% - ${id} (${meta18?.name || "anonymous"} in ${meta18?.filePath})`);
+    log(`    ${meetsThreshold} ${(score * 100).toFixed(1)}% - ${id} (${meta17?.name || "anonymous"} in ${meta17?.filePath})`);
   }
   log(`  Found ${results.length} chunks above threshold`);
   return results.sort((a, b) => b.score - a.score);
@@ -4498,20 +4378,20 @@ var no_semantic_duplicates_default = createRule({
           log(`  Chunk ${chunkId} already reported, skipping`);
           continue;
         }
-        const meta18 = index.metadataStore.get(chunkId);
-        if (!meta18) {
+        const meta17 = index.metadataStore.get(chunkId);
+        if (!meta17) {
           log(`  No metadata for chunk ${chunkId}`);
           continue;
         }
-        log(`  Checking chunk ${chunkId}: lines ${meta18.startLine}-${meta18.endLine} (node at line ${nodeLine})`);
-        if (nodeLine >= meta18.startLine && nodeLine <= meta18.endLine) {
+        log(`  Checking chunk ${chunkId}: lines ${meta17.startLine}-${meta17.endLine} (node at line ${nodeLine})`);
+        if (nodeLine >= meta17.startLine && nodeLine <= meta17.endLine) {
           log(`  Node is within chunk range, searching for similar chunks...`);
           const similar = findSimilarChunks(index, chunkId, threshold);
           if (similar.length > 0) {
             const best = similar[0];
             const bestMeta = index.metadataStore.get(best.id);
             if (bestMeta) {
-              const chunkLines = meta18.endLine - meta18.startLine + 1;
+              const chunkLines = meta17.endLine - meta17.startLine + 1;
               if (chunkLines < minLines) {
                 log(`  Skipping: chunk has ${chunkLines} lines, below minLines=${minLines}`);
                 continue;
@@ -4519,17 +4399,17 @@ var no_semantic_duplicates_default = createRule({
               reportedChunks.add(chunkId);
               const relPath = relative3(projectRoot, bestMeta.filePath);
               const similarity = Math.round(best.score * 100);
-              log(`  REPORTING: ${meta18.kind} '${name || meta18.name}' is ${similarity}% similar to '${bestMeta.name}' at ${relPath}:${bestMeta.startLine}`);
+              log(`  REPORTING: ${meta17.kind} '${name || meta17.name}' is ${similarity}% similar to '${bestMeta.name}' at ${relPath}:${bestMeta.startLine}`);
               context.report({
                 node,
                 loc: {
-                  start: { line: meta18.startLine, column: meta18.startColumn },
-                  end: { line: meta18.endLine, column: meta18.endColumn }
+                  start: { line: meta17.startLine, column: meta17.startColumn },
+                  end: { line: meta17.endLine, column: meta17.endColumn }
                 },
                 messageId: "semanticDuplicate",
                 data: {
-                  kind: meta18.kind,
-                  name: name || meta18.name || "(anonymous)",
+                  kind: meta17.kind,
+                  name: name || meta17.name || "(anonymous)",
                   similarity: String(similarity),
                   otherName: bestMeta.name || "(anonymous)",
                   otherLocation: `${relPath}:${bestMeta.startLine}`
@@ -4540,7 +4420,7 @@ var no_semantic_duplicates_default = createRule({
             log(`  No similar chunks found above threshold`);
           }
         } else {
-          log(`  Node line ${nodeLine} not in chunk range ${meta18.startLine}-${meta18.endLine}`);
+          log(`  Node line ${nodeLine} not in chunk range ${meta17.startLine}-${meta17.endLine}`);
         }
       }
     }
@@ -5786,7 +5666,7 @@ function simpleGlobMatch(pattern, path) {
   const regex = new RegExp(`^${regexStr}$`);
   return regex.test(normalizedPath);
 }
-var meta15 = defineRuleMeta({
+var meta14 = defineRuleMeta({
   id: "require-test-coverage",
   version: "1.0.0",
   name: "Require Test Coverage",
@@ -6499,7 +6379,7 @@ function checkForJSX(ast, visited = /* @__PURE__ */ new WeakSet()) {
 }
 
 // src/rules/prefer-tailwind.ts
-var meta16 = defineRuleMeta({
+var meta15 = defineRuleMeta({
   id: "prefer-tailwind",
   version: "1.0.0",
   name: "Prefer Tailwind",
@@ -6926,21 +6806,20 @@ var ruleRegistry = [
   meta4,
   meta5,
   meta6,
-  meta7,
   // New UI rules
+  meta7,
   meta8,
   meta9,
   meta10,
-  meta11,
   // New security rules
+  meta11,
   meta12,
-  meta13,
   // Semantic duplicate detection
-  meta14,
+  meta13,
   // Test coverage enforcement
-  meta15,
+  meta14,
   // Style preferences
-  meta16
+  meta15
 ];
 function getRuleMetadata(id) {
   return ruleRegistry.find((rule) => rule.id === id);
@@ -6959,7 +6838,6 @@ function getAllRuleIds() {
 // src/index.ts
 import { ResolverFactory as ResolverFactory3 } from "oxc-resolver";
 var rules = {
-  "no-arbitrary-tailwind": no_arbitrary_tailwind_default,
   "consistent-dark-mode": consistent_dark_mode_default,
   "no-direct-store-import": no_direct_store_import_default,
   "prefer-zustand-state-management": prefer_zustand_state_management_default,
@@ -6977,12 +6855,12 @@ var rules = {
   "prefer-tailwind": prefer_tailwind_default
 };
 var version = "0.1.0";
-var meta17 = {
+var meta16 = {
   name: "uilint",
   version
 };
 var plugin = {
-  meta: meta17,
+  meta: meta16,
   rules
 };
 var jsxLanguageOptions = {
@@ -7000,7 +6878,6 @@ var recommendedConfig = {
   },
   languageOptions: jsxLanguageOptions,
   rules: {
-    "uilint/no-arbitrary-tailwind": "error",
     "uilint/consistent-dark-mode": ["error", ...[
       {
         "warnOnMissingDarkMode": true
@@ -7129,7 +7006,6 @@ var strictConfig = {
   },
   languageOptions: jsxLanguageOptions,
   rules: {
-    "uilint/no-arbitrary-tailwind": "error",
     "uilint/consistent-dark-mode": ["error", ...[
       {
         "warnOnMissingDarkMode": true
@@ -7274,7 +7150,7 @@ var configs = {
   strict: strictConfig
 };
 var uilintEslint = {
-  meta: meta17,
+  meta: meta16,
   plugin,
   rules,
   configs
@@ -7312,7 +7188,7 @@ export {
   isEventHandlerAttribute,
   loadCache,
   loadStyleguide,
-  meta17 as meta,
+  meta16 as meta,
   plugin,
   ruleRegistry,
   rules,