uilint-eslint 0.2.21 → 0.2.23

package/dist/index.js

@@ -379,7 +379,7 @@ warns when a file uses color classes without any dark mode theming.
 
 - **Prevents broken dark mode**: Catches cases where some colors change in dark mode but others don't
 - **Encourages completeness**: Prompts you to add dark mode support where it's missing
-- **Supports semantic colors**: Automatically ignores shadcn/CSS variable colors that handle dark mode internally
+- **No false positives**: Only flags explicit Tailwind colors, not custom/CSS variable colors
 
 ## Examples
 
@@ -401,8 +401,10 @@ warns when a file uses color classes without any dark mode theming.
 // All color classes have dark variants
 <div className="bg-white dark:bg-slate-900 text-black dark:text-white">
 
-// Using semantic colors (automatically themed)
+// Using semantic/custom colors (automatically themed via CSS variables)
 <div className="bg-background text-foreground">
+<div className="bg-brand text-brand-foreground">
+<div className="bg-primary text-primary-foreground">
 
 // Consistent theming
 <button className="bg-blue-500 dark:bg-blue-600 border-gray-300 dark:border-gray-600">
@@ -419,7 +421,9 @@ warns when a file uses color classes without any dark mode theming.
 
 ## Notes
 
-- Semantic colors (like shadcn's \`background\`, \`foreground\`, \`primary\`, etc.) are exempt
+- Only explicit Tailwind colors (like \`blue-500\`, \`white\`, \`slate-900\`) require dark variants
+- Custom/semantic colors (\`background\`, \`foreground\`, \`brand\`, \`primary\`, etc.) are exempt
+- These are assumed to be CSS variables that handle dark mode automatically
 - Transparent, inherit, and current values are exempt
 - Non-color utilities (like \`text-lg\`, \`border-2\`) are correctly ignored
 `
@@ -450,129 +454,38 @@ var COLOR_PREFIXES = [
   "to-"
 ];
 var EXEMPT_SUFFIXES = ["transparent", "inherit", "current", "auto", "none"];
-var NON_COLOR_UTILITIES = /* @__PURE__ */ new Set([
-  // Exempt values (colorless or inherited) - don't need dark variants
-  "transparent",
-  "inherit",
-  "current",
-  "auto",
-  "none",
-  // text- utilities that aren't colors
-  "xs",
-  "sm",
-  "base",
-  "lg",
-  "xl",
-  "2xl",
-  "3xl",
-  "4xl",
-  "5xl",
-  "6xl",
-  "7xl",
-  "8xl",
-  "9xl",
-  "left",
-  "center",
-  "right",
-  "justify",
-  "start",
-  "end",
-  "wrap",
-  "nowrap",
-  "balance",
-  "pretty",
-  "ellipsis",
-  "clip",
-  // border- utilities that aren't colors
-  "0",
-  "2",
-  "4",
-  "8",
-  "solid",
-  "dashed",
-  "dotted",
-  "double",
-  "hidden",
-  "collapse",
-  "separate",
-  // shadow- utilities that aren't colors
-  // Note: "sm", "lg", "xl", "2xl" already included above
-  "md",
-  "inner",
-  // ring- utilities that aren't colors
-  // Note: "0", "2", "4", "8" already included above
-  "1",
-  "inset",
-  // outline- utilities that aren't colors
-  // Note: numeric values already included
-  "offset-0",
-  "offset-1",
-  "offset-2",
-  "offset-4",
-  "offset-8",
-  // decoration- utilities that aren't colors
-  // Note: "solid", "double", "dotted", "dashed" already included
-  "wavy",
-  "from-font",
-  "clone",
-  "slice",
-  // divide- utilities that aren't colors
-  "x",
-  "y",
-  "x-0",
-  "x-2",
-  "x-4",
-  "x-8",
-  "y-0",
-  "y-2",
-  "y-4",
-  "y-8",
-  "x-reverse",
-  "y-reverse",
-  // gradient direction utilities (from-, via-, to- prefixes)
-  "t",
-  "tr",
-  "r",
-  "br",
-  "b",
-  "bl",
-  "l",
-  "tl"
+var TAILWIND_COLOR_NAMES = /* @__PURE__ */ new Set([
+  // Special colors
+  "white",
+  "black",
+  // Gray scale palettes
+  "slate",
+  "gray",
+  "zinc",
+  "neutral",
+  "stone",
+  // Warm colors
+  "red",
+  "orange",
+  "amber",
+  "yellow",
+  // Green colors
+  "lime",
+  "green",
+  "emerald",
+  "teal",
+  // Blue colors
+  "cyan",
+  "sky",
+  "blue",
+  "indigo",
+  // Purple/Pink colors
+  "violet",
+  "purple",
+  "fuchsia",
+  "pink",
+  "rose"
 ]);
-var SEMANTIC_COLOR_NAMES = /* @__PURE__ */ new Set([
-  // Core shadcn colors
-  "background",
-  "foreground",
-  // Component colors
-  "card",
-  "card-foreground",
-  "popover",
-  "popover-foreground",
-  "primary",
-  "primary-foreground",
-  "secondary",
-  "secondary-foreground",
-  "muted",
-  "muted-foreground",
-  "accent",
-  "accent-foreground",
-  "destructive",
-  "destructive-foreground",
-  // Form/UI colors
-  "border",
-  "input",
-  "ring",
-  // Sidebar colors (shadcn sidebar component)
-  "sidebar",
-  "sidebar-foreground",
-  "sidebar-border",
-  "sidebar-primary",
-  "sidebar-primary-foreground",
-  "sidebar-accent",
-  "sidebar-accent-foreground",
-  "sidebar-ring"
-]);
-var CHART_COLOR_PATTERN = /^chart-\d+$/;
 function hasDarkVariant(className) {
   const parts = className.split(":");
   const variants = parts.slice(0, -1);
@@ -588,12 +501,31 @@ function getColorPrefix(baseClass) {
   );
   return sortedPrefixes.find((p) => baseClass.startsWith(p)) || null;
 }
-function isSemanticColor(value) {
-  if (SEMANTIC_COLOR_NAMES.has(value)) {
+function isTailwindColor(value) {
+  const valueWithoutOpacity = value.split("/")[0] || value;
+  if (TAILWIND_COLOR_NAMES.has(valueWithoutOpacity)) {
     return true;
   }
-  if (CHART_COLOR_PATTERN.test(value)) {
-    return true;
+  const match = valueWithoutOpacity.match(/^([a-z]+)-(\d+)$/);
+  if (match) {
+    const colorName = match[1];
+    const scale = match[2];
+    const validScales = [
+      "50",
+      "100",
+      "200",
+      "300",
+      "400",
+      "500",
+      "600",
+      "700",
+      "800",
+      "900",
+      "950"
+    ];
+    if (colorName && TAILWIND_COLOR_NAMES.has(colorName) && validScales.includes(scale || "")) {
+      return true;
+    }
   }
   return false;
 }
@@ -602,13 +534,7 @@ function isColorValue(baseClass, prefix) {
   if (!value) {
     return false;
   }
-  if (isSemanticColor(value)) {
-    return false;
-  }
-  if (NON_COLOR_UTILITIES.has(value)) {
-    return false;
-  }
-  return true;
+  return isTailwindColor(value);
 }
 function isExempt(baseClass) {
   return EXEMPT_SUFFIXES.some((suffix) => baseClass.endsWith(suffix));
@@ -2921,7 +2847,7 @@ function containsAnyType(node, allowInGenericDefaults) {
         }
       }
       for (const param of node.params) {
-        if (param.typeAnnotation?.typeAnnotation && param.type !== "RestElement") {
+        if (param.type !== "RestElement" && param.type !== "TSParameterProperty" && "typeAnnotation" in param && param.typeAnnotation?.typeAnnotation) {
           const result = containsAnyType(
             param.typeAnnotation.typeAnnotation,
             allowInGenericDefaults
@@ -3367,9 +3293,9 @@ var meta12 = defineRuleMeta({
       {
         key: "ignoredProps",
         label: "Ignored props",
-        type: "array",
-        defaultValue: ["className", "style", "children", "key", "ref", "id"],
-        description: "Prop names to ignore (common pass-through props)"
+        type: "text",
+        defaultValue: "className, style, children, key, ref, id",
+        description: "Comma-separated prop names to ignore (common pass-through props)"
       }
     ]
   },