canicode 0.3.2 → 0.3.3

package/README.md

@@ -194,7 +194,7 @@ canicode analyze <url> --config ./my-config.json
 
 | Option | Description |
 |--------|-------------|
-| `gridBase` | Spacing grid unit (default: 8) |
+| `gridBase` | Spacing grid unit (default: 4) |
 | `colorTolerance` | Color difference tolerance (default: 10) |
 | `excludeNodeTypes` | Node types to skip |
 | `excludeNodeNames` | Node name patterns to skip |

package/dist/cli/index.js

@@ -185,7 +185,7 @@ var RULE_CONFIGS = {
     score: -2,
     enabled: true,
     options: {
-      gridBase: 8
+      gridBase: 4
     }
   },
   "magic-number-spacing": {
@@ -193,7 +193,7 @@ var RULE_CONFIGS = {
     score: -4,
     enabled: true,
     options: {
-      gridBase: 8
+      gridBase: 4
     }
   },
   "raw-shadow": {
@@ -764,6 +764,12 @@ function transformNode(node) {
   if ("layoutPositioning" in node && node.layoutPositioning) {
     base.layoutPositioning = node.layoutPositioning;
   }
+  if ("layoutSizingHorizontal" in node && node.layoutSizingHorizontal) {
+    base.layoutSizingHorizontal = node.layoutSizingHorizontal;
+  }
+  if ("layoutSizingVertical" in node && node.layoutSizingVertical) {
+    base.layoutSizingVertical = node.layoutSizingVertical;
+  }
   if ("primaryAxisAlignItems" in node) {
     base.primaryAxisAlignItems = node.primaryAxisAlignItems;
   }
@@ -1552,7 +1558,7 @@ ${figmaToken ? `  <script>
         const res = await fetch('https://api.figma.com/v1/files/' + fileKey + '/comments', {
           method: 'POST',
           headers: { 'X-FIGMA-TOKEN': FIGMA_TOKEN, 'Content-Type': 'application/json' },
-          body: JSON.stringify({ message: commentBody, client_meta: { node_id: nodeId } }),
+          body: JSON.stringify({ message: commentBody, client_meta: { node_id: nodeId, node_offset: { x: 0, y: 0 } } }),
         });
         if (!res.ok) throw new Error(await res.text());
         btn.textContent = 'Sent \\u2713';
@@ -2315,6 +2321,12 @@ var ActivityLogger = class {
   }
 };
 
+// src/rules/excluded-names.ts
+var EXCLUDED_NAME_PATTERN = /(badge|close|dismiss|overlay|float|fab|dot|indicator|corner|decoration|tag|status|notification|icon|ico|image|asset|filter|dim|dimmed|bg|background|logo|avatar|divider|separator|nav|navigation|gnb|header|footer|sidebar|toolbar|modal|dialog|popup|toast|tooltip|dropdown|menu|sticky|spinner|loader|cursor|cta|chatbot|thumb|thumbnail|tabbar|tab-bar|statusbar|status-bar)/i;
+function isExcludedName(name) {
+  return EXCLUDED_NAME_PATTERN.test(name);
+}
+
 // src/agents/orchestrator.ts
 function selectNodes(summaries, strategy, maxNodes) {
   if (summaries.length === 0) return [];
@@ -2370,14 +2382,13 @@ var ELIGIBLE_NODE_TYPES = /* @__PURE__ */ new Set([
   "COMPONENT",
   "INSTANCE"
 ]);
-var EXCLUDED_NAME_PATTERN = /\b(icon|ico|badge|indicator|image|asset|chatbot|cta|gnb|navigation|nav|fab|modal|dialog|popup|overlay|toast|snackbar|tooltip|dropdown|menu|sticky|bg|background|divider|separator|logo|avatar|thumbnail|thumb|header|footer|sidebar|toolbar|tabbar|tab-bar|statusbar|status-bar|spinner|loader|cursor|dot|dim|dimmed|filter)\b/i;
 function filterConversionCandidates(summaries, documentRoot) {
   return summaries.filter((summary) => {
     const node = findNode(documentRoot, summary.nodeId);
     if (!node) return false;
     if (EXCLUDED_NODE_TYPES.has(node.type)) return false;
     if (!ELIGIBLE_NODE_TYPES.has(node.type)) return false;
-    if (EXCLUDED_NAME_PATTERN.test(node.name)) return false;
+    if (isExcludedName(node.name)) return false;
     const bbox = node.absoluteBoundingBox;
     if (bbox && (bbox.width < MIN_WIDTH || bbox.height < MIN_HEIGHT)) return false;
     if (!node.children || node.children.length < 3) return false;
@@ -2762,7 +2773,7 @@ Override canicode's default rule scores, severity, and filters.
 STRUCTURE
   - excludeNodeTypes: node types to skip (e.g. VECTOR, BOOLEAN_OPERATION)
   - excludeNodeNames: name patterns to skip (e.g. icon, ico)
-  - gridBase: spacing grid unit, default 8
+  - gridBase: spacing grid unit, default 4
   - colorTolerance: color diff tolerance, default 10
   - rules: per-rule overrides (score, severity, enabled)
 
@@ -2848,7 +2859,6 @@ var absolutePositionInAutoLayoutDef = {
   impact: "Element will not respond to sibling changes, may overlap unexpectedly",
   fix: "Remove absolute positioning or use proper Auto Layout alignment"
 };
-var INTENTIONAL_ABSOLUTE_PATTERNS = /^(badge|close|dismiss|overlay|float|fab|dot|indicator|corner|decoration|tag|status|notification|x|icon[-_ ]?(close|dismiss|x)|btn[-_ ]?(close|dismiss))/i;
 function isSmallRelativeToParent(node, parent) {
   const nodeBB = node.absoluteBoundingBox;
   const parentBB = parent.absoluteBoundingBox;
@@ -2862,7 +2872,8 @@ var absolutePositionInAutoLayoutCheck = (node, context) => {
   if (!context.parent) return null;
   if (!hasAutoLayout(context.parent)) return null;
   if (node.layoutPositioning !== "ABSOLUTE") return null;
-  if (INTENTIONAL_ABSOLUTE_PATTERNS.test(node.name)) return null;
+  if (node.type === "VECTOR" || node.type === "BOOLEAN_OPERATION" || node.type === "LINE" || node.type === "ELLIPSE" || node.type === "STAR" || node.type === "REGULAR_POLYGON") return null;
+  if (isExcludedName(node.name)) return null;
   if (isSmallRelativeToParent(node, context.parent)) return null;
   if (context.parent.type === "COMPONENT") return null;
   return {
@@ -2888,10 +2899,14 @@ var fixedWidthInResponsiveContextCheck = (node, context) => {
   if (!context.parent) return null;
   if (!hasAutoLayout(context.parent)) return null;
   if (!isContainerNode(node)) return null;
-  if (node.layoutAlign === "STRETCH") return null;
-  const bbox = node.absoluteBoundingBox;
-  if (!bbox) return null;
-  if (node.layoutAlign !== "INHERIT") return null;
+  if (node.layoutSizingHorizontal) {
+    if (node.layoutSizingHorizontal !== "FIXED") return null;
+  } else {
+    if (node.layoutAlign === "STRETCH") return null;
+    if (!node.absoluteBoundingBox) return null;
+    if (node.layoutAlign !== "INHERIT") return null;
+  }
+  if (isExcludedName(node.name)) return null;
   return {
     ruleId: fixedWidthInResponsiveContextDef.id,
     nodeId: node.id,
@@ -3164,7 +3179,7 @@ var inconsistentSpacingDef = {
   fix: "Use spacing values from the design system grid (e.g., 8pt increments)"
 };
 var inconsistentSpacingCheck = (node, context, options) => {
-  const gridBase = options?.["gridBase"] ?? getRuleOption("inconsistent-spacing", "gridBase", 8);
+  const gridBase = options?.["gridBase"] ?? getRuleOption("inconsistent-spacing", "gridBase", 4);
   const paddings = [
     node.paddingLeft,
     node.paddingRight,
@@ -3206,7 +3221,7 @@ var magicNumberSpacingDef = {
   fix: "Round spacing to the nearest grid value or use spacing tokens"
 };
 var magicNumberSpacingCheck = (node, context, options) => {
-  const gridBase = options?.["gridBase"] ?? getRuleOption("magic-number-spacing", "gridBase", 8);
+  const gridBase = options?.["gridBase"] ?? getRuleOption("magic-number-spacing", "gridBase", 4);
   const allSpacings = [
     node.paddingLeft,
     node.paddingRight,
@@ -3523,6 +3538,7 @@ var defaultNameDef = {
 };
 var defaultNameCheck = (node, context) => {
   if (!node.name) return null;
+  if (isExcludedName(node.name)) return null;
   if (!isDefaultName(node.name)) return null;
   return {
     ruleId: defaultNameDef.id,
@@ -3545,6 +3561,7 @@ var nonSemanticNameDef = {
 };
 var nonSemanticNameCheck = (node, context) => {
   if (!node.name) return null;
+  if (isExcludedName(node.name)) return null;
   if (!isNonSemanticName(node.name)) return null;
   if (!node.children || node.children.length === 0) {
     const shapeTypes = ["RECTANGLE", "ELLIPSE", "VECTOR", "LINE", "STAR", "REGULAR_POLYGON"];
@@ -3613,6 +3630,7 @@ var numericSuffixNameDef = {
 };
 var numericSuffixNameCheck = (node, context) => {
   if (!node.name) return null;
+  if (isExcludedName(node.name)) return null;
   if (isDefaultName(node.name)) return null;
   if (!hasNumericSuffix(node.name)) return null;
   return {