canicode 0.12.2 → 0.12.4

package/dist/mcp/server.js

@@ -852,6 +852,7 @@ var RuleEngine = class {
       analysisState,
       scope,
       rootNodeType,
+      rootNode,
       void 0,
       void 0
     );
@@ -888,7 +889,7 @@ var RuleEngine = class {
   /**
    * Recursively traverse the tree and run rules
    */
-  traverseAndCheck(node, file, rules, maxDepth, issues, failedRules, depth, path, ancestorTypes, componentDepth, analysisState, scope, rootNodeType, parent, siblings) {
+  traverseAndCheck(node, file, rules, maxDepth, issues, failedRules, depth, path, ancestorTypes, componentDepth, analysisState, scope, rootNodeType, analysisRoot, parent, siblings) {
     const nodePath = [...path, node.name];
     const isComponentBoundary = node.type === "COMPONENT" || node.type === "COMPONENT_SET" || node.type === "INSTANCE";
     const currentComponentDepth = isComponentBoundary ? 0 : componentDepth;
@@ -911,6 +912,7 @@ var RuleEngine = class {
       analysisState,
       scope,
       rootNodeType,
+      analysisRoot,
       findAcknowledgment: (nodeId, ruleId) => acknowledgmentsByKey.get(`${normalizeNodeId(nodeId)}::${ruleId}`)
     };
     for (const rule of rules) {
@@ -960,6 +962,7 @@ var RuleEngine = class {
           analysisState,
           scope,
           rootNodeType,
+          analysisRoot,
           node,
           node.children
         );
@@ -1943,7 +1946,7 @@ function computeApplyContext(violation, instanceContext) {
 }
 
 // package.json
-var version = "0.12.2";
+var version = "0.12.4";
 
 // src/core/engine/scoring.ts
 var GRADE_ORDER = ["S", "A+", "A", "B+", "B", "C+", "C", "D", "F"];
@@ -2655,7 +2658,11 @@ function mapToQuestion(issue, file) {
     ...applyContext.annotationProperties !== void 0 ? { annotationProperties: applyContext.annotationProperties } : {},
     ...suggestedName !== void 0 ? { suggestedName } : {},
     isInstanceChild: applyContext.isInstanceChild,
-    ...applyContext.sourceChildId !== void 0 ? { sourceChildId: applyContext.sourceChildId } : {}
+    ...applyContext.sourceChildId !== void 0 ? { sourceChildId: applyContext.sourceChildId } : {},
+    // #560 / Phase 3 delta 4a: thread groupMembers through from the
+    // violation. Currently only populated by `missing-component`
+    // Stage 3; non-group rules pass undefined and the field is omitted.
+    ...issue.violation.groupMembers !== void 0 ? { groupMembers: issue.violation.groupMembers } : {}
   };
 }
 function deduplicateBySourceComponent(questions) {
@@ -3684,6 +3691,22 @@ body {
   border: 1px solid rgba(239,68,68,0.15);
 }
 
+/* ---- Warning (skipped nodes, etc.) ---- */
+.warning-msg {
+  background: var(--amber-bg);
+  color: #92400e;
+  padding: 10px 12px;
+  border-radius: var(--radius-sm);
+  font-size: 12px;
+  margin-bottom: 16px;
+  border: 1px solid rgba(245,158,11,0.2);
+}
+.warning-msg details { margin-top: 8px; }
+.warning-msg summary { cursor: pointer; font-weight: 500; }
+.warning-msg ul { margin: 8px 0 0 16px; padding: 0; }
+.warning-msg li { margin: 4px 0; }
+.warning-msg a { color: inherit; text-decoration: underline; }
+
 /* ---- Layout (plugin) ---- */
 .container { padding: 16px; }
 .header {
@@ -3909,7 +3932,24 @@ var EVENTS = {
   // has a single place to wire it up.
   ROUNDTRIP_DEFINITION_WRITE_SKIPPED: `${EVENT_PREFIX}roundtrip_definition_write_skipped`,
   /** CLI `canicode roundtrip-tally` completed successfully. */
-  ROUNDTRIP_TALLY: `${EVENT_PREFIX}roundtrip_tally`
+  ROUNDTRIP_TALLY: `${EVENT_PREFIX}roundtrip_tally`,
+  /**
+   * Phase 3 (#508 / ADR-023): `applyComponentize` outcome — either a
+   * successful `createComponentFromNode` call or one of the guard rejections
+   * (instance-child, free-form parent, error) that route to the Strategy C
+   * annotate-fallback. Surfaces under `props.outcome` so a Node-side reader
+   * can split the funnel.
+   */
+  ROUNDTRIP_COMPONENTIZE: `${EVENT_PREFIX}roundtrip_componentize`,
+  /**
+   * Phase 3 (#508 / ADR-023, #554): `applyReplaceWithInstance` outcome —
+   * either a successful instance swap (`outcome: "replaced"`) or one of the
+   * guard rejections (`"skipped-free-form-parent"`, `"skipped-prereq-missing"`,
+   * `"error"`). The primitive pairs with `ROUNDTRIP_COMPONENTIZE` so a
+   * Node-side reader can correlate componentize+swap pairs across a single
+   * Phase 3 batch.
+   */
+  ROUNDTRIP_REPLACE_WITH_INSTANCE: `${EVENT_PREFIX}roundtrip_replace_with_instance`
 };
 
 // src/core/monitoring/capture.ts
@@ -4309,8 +4349,8 @@ var missingComponentMsg = {
     message: `"${name}" appears ${count} times`,
     suggestion: `Extract as a reusable component`
   }),
-  structureRepetition: (name, siblingCount) => ({
-    message: `"${name}" and ${siblingCount} sibling frame(s) share the same internal structure`,
+  structureRepetition: (name, matchCount) => ({
+    message: `"${name}" and ${matchCount} other frame(s) share the same internal structure`,
     suggestion: `Extract a shared component from the repeated structure`
   }),
   styleOverride: (componentName, overrides) => ({
@@ -4736,6 +4776,10 @@ defineRule({
 function isOnGrid(value, gridBase) {
   return value % gridBase === 0;
 }
+function hasOwnBoundVariables(obj) {
+  const bv = obj["boundVariables"];
+  return bv !== void 0 && bv !== null && Object.keys(bv).length > 0;
+}
 var rawValueDef = {
   id: "raw-value",
   name: "Raw Value",
@@ -4750,6 +4794,7 @@ var rawValueCheck = (node, context) => {
     if (!hasStyleReference(node, "fill") && !hasBoundVariable(node, "fills")) {
       for (const fill of node.fills) {
         const fillObj = fill;
+        if (hasOwnBoundVariables(fillObj)) continue;
         if (fillObj["type"] === "SOLID" && fillObj["color"]) {
           const c = fillObj["color"];
           const hex = `#${Math.round((c["r"] ?? 0) * 255).toString(16).padStart(2, "0")}${Math.round((c["g"] ?? 0) * 255).toString(16).padStart(2, "0")}${Math.round((c["b"] ?? 0) * 255).toString(16).padStart(2, "0")}`.toUpperCase();
@@ -4784,9 +4829,10 @@ var rawValueCheck = (node, context) => {
     }
   }
   if (node.effects && Array.isArray(node.effects) && node.effects.length > 0) {
-    if (!hasStyleReference(node, "effect")) {
+    if (!hasStyleReference(node, "effect") && !hasBoundVariable(node, "effects")) {
       for (const effect of node.effects) {
         const effectObj = effect;
+        if (hasOwnBoundVariables(effectObj)) continue;
         if (effectObj["type"] === "DROP_SHADOW" || effectObj["type"] === "INNER_SHADOW") {
           const shadowType = effectObj["type"] === "DROP_SHADOW" ? "drop shadow" : "inner shadow";
           const offset = effectObj["offset"];
@@ -4817,6 +4863,7 @@ var rawValueCheck = (node, context) => {
   }
   const spacingKeys = ["paddingLeft", "paddingRight", "paddingTop", "paddingBottom", "itemSpacing"];
   for (const key of spacingKeys) {
+    if (key === "itemSpacing" && node.primaryAxisAlignItems === "SPACE_BETWEEN") continue;
     const value = node[key];
     if (value !== void 0 && value > 0 && !hasBoundVariable(node, key)) {
       const label = key === "itemSpacing" ? "gap" : key.replace("padding", "padding-").toLowerCase();
@@ -4849,14 +4896,15 @@ var irregularSpacingCheck = (node, context, options) => {
   const spacingEntries = [];
   for (const key of ["paddingLeft", "paddingRight", "paddingTop", "paddingBottom"]) {
     const v = node[key];
-    if (v !== void 0 && v > 0) spacingEntries.push({ value: v, subType: "padding" });
+    if (v !== void 0 && v > 0) spacingEntries.push({ key, value: v, subType: "padding" });
   }
-  if (node.itemSpacing !== void 0 && node.itemSpacing > 0) {
-    spacingEntries.push({ value: node.itemSpacing, subType: "gap" });
+  if (node.itemSpacing !== void 0 && node.itemSpacing > 0 && node.primaryAxisAlignItems !== "SPACE_BETWEEN") {
+    spacingEntries.push({ key: "itemSpacing", value: node.itemSpacing, subType: "gap" });
   }
   const commonValues = [1, 2];
   for (const entry of spacingEntries) {
     if (commonValues.includes(entry.value)) continue;
+    if (hasBoundVariable(node, entry.key)) continue;
     if (!isOnGrid(entry.value, gridBase)) {
       return {
         ruleId: irregularSpacingDef.id,
@@ -4921,6 +4969,41 @@ function getSeenStage1(context) {
 function getSeenStage4(context) {
   return getAnalysisState(context, SEEN_STAGE4_KEY, () => /* @__PURE__ */ new Set());
 }
+function stage3GroupsKey(maxDepth) {
+  return `missing-component:stage3Groups:depth=${maxDepth}`;
+}
+function nodeQualifiesForStage3(node, parent, insideInstance) {
+  if (insideInstance) return false;
+  if (node.type !== "FRAME") return false;
+  if (parent?.type === "COMPONENT_SET") return false;
+  if (!node.children || node.children.length === 0) return false;
+  return true;
+}
+function buildStage3Groups(root, maxFingerprintDepth) {
+  const groups = /* @__PURE__ */ new Map();
+  const walk = (node, parent, ancestorIsInstance) => {
+    const insideInstance = ancestorIsInstance || node.type === "INSTANCE";
+    if (nodeQualifiesForStage3(node, parent, ancestorIsInstance)) {
+      const fp = buildFingerprint(node, maxFingerprintDepth);
+      const existing = groups.get(fp);
+      if (existing) existing.memberIds.push(node.id);
+      else groups.set(fp, { memberIds: [node.id] });
+    }
+    if (node.children) {
+      for (const child of node.children) walk(child, node, insideInstance);
+    }
+  };
+  walk(root, null, false);
+  return groups;
+}
+function getStage3Groups(context, maxFingerprintDepth) {
+  const root = context.analysisRoot ?? context.file.document;
+  return getAnalysisState(
+    context,
+    stage3GroupsKey(maxFingerprintDepth),
+    () => buildStage3Groups(root, maxFingerprintDepth)
+  );
+}
 var missingComponentDef = {
   id: "missing-component",
   name: "Missing Component",
@@ -4935,7 +5018,8 @@ var missingComponentCheck = (node, context, options) => {
     const matchingComponent = Object.values(components).find(
       (c) => c.name.toLowerCase() === node.name.toLowerCase()
     );
-    const frameNames = collectFrameNames(context.file.document);
+    const scopeRoot = context.analysisRoot ?? context.file.document;
+    const frameNames = collectFrameNames(scopeRoot);
     const sameNameFrames = frameNames.get(node.name);
     const firstFrame = sameNameFrames?.[0];
     if (matchingComponent) {
@@ -4963,37 +5047,32 @@ var missingComponentCheck = (node, context, options) => {
         };
       }
     }
-    if (isInsideInstance(context)) return null;
-    if (context.parent?.type === "COMPONENT_SET") return null;
-    if (!node.children || node.children.length === 0) return null;
+    if (!nodeQualifiesForStage3(node, context.parent ?? null, isInsideInstance(context))) {
+      return null;
+    }
     const structureMinRepetitions = options?.["structureMinRepetitions"] ?? getRuleOption("missing-component", "structureMinRepetitions", 2);
     const maxFingerprintDepth = options?.["maxFingerprintDepth"] ?? getRuleOption("missing-component", "maxFingerprintDepth", 3);
+    const groups = getStage3Groups(context, maxFingerprintDepth);
     const fingerprint = buildFingerprint(node, maxFingerprintDepth);
-    const siblings = context.siblings ?? [];
-    const qualifyingSiblings = siblings.filter(
-      (s) => s.type === "FRAME" && s.children !== void 0 && s.children.length > 0
-    );
-    const matchingNodes = qualifyingSiblings.filter(
-      (s) => buildFingerprint(s, maxFingerprintDepth) === fingerprint
-    );
-    const selfIsInSiblings = qualifyingSiblings.some((s) => s.id === node.id);
-    const count = selfIsInSiblings ? matchingNodes.length : matchingNodes.length + 1;
-    if (count >= structureMinRepetitions) {
-      const firstMatch = qualifyingSiblings.find(
-        (s) => buildFingerprint(s, maxFingerprintDepth) === fingerprint
-      );
-      const firstMatchId = firstMatch?.id ?? node.id;
-      if (firstMatchId === node.id) {
-        return {
-          ruleId: missingComponentDef.id,
-          subType: "structure-repetition",
-          nodeId: node.id,
-          nodePath: context.path.join(" > "),
-          ...missingComponentMsg.structureRepetition(node.name, count - 1)
-        };
-      }
-    }
-    return null;
+    const group = groups.get(fingerprint);
+    if (!group) return null;
+    if (group.memberIds.length < structureMinRepetitions) return null;
+    if (group.memberIds[0] !== node.id) return null;
+    return {
+      ruleId: missingComponentDef.id,
+      subType: "structure-repetition",
+      nodeId: node.id,
+      nodePath: context.path.join(" > "),
+      ...missingComponentMsg.structureRepetition(
+        node.name,
+        group.memberIds.length - 1
+      ),
+      // #560 / delta 4a: surface the full group so the apply step can drive
+      // the componentize+swap loop from a single user answer. `nodeId` is
+      // the document-order first member; the rest are siblings or cross-
+      // parent matches found by the scope-wide pass (#557).
+      groupMembers: [...group.memberIds]
+    };
   }
   if (node.type === "INSTANCE" && node.componentId) {
     const seenStage4 = getSeenStage4(context);
@@ -5083,17 +5162,28 @@ defineRule({
 var CODE_CONNECT_SETUP_KEY = "unmapped-component:setup-detected";
 var CODE_CONNECT_MAPPINGS_KEY = "unmapped-component:mappings";
 var SEEN_MAIN_IDS_KEY = "unmapped-component:seen-main-ids";
+function browserCwd() {
+  return typeof process !== "undefined" && typeof process.cwd === "function" ? process.cwd() : null;
+}
 function codeConnectIsSetUp(context) {
   return getAnalysisState(context, CODE_CONNECT_SETUP_KEY, () => {
-    return existsSync(join(process.cwd(), "figma.config.json"));
+    const cwd = browserCwd();
+    if (cwd === null) return false;
+    return existsSync(join(cwd, "figma.config.json"));
   });
 }
 function codeConnectMappings(context) {
-  return getAnalysisState(
-    context,
-    CODE_CONNECT_MAPPINGS_KEY,
-    () => parseCodeConnectMappings(process.cwd())
-  );
+  return getAnalysisState(context, CODE_CONNECT_MAPPINGS_KEY, () => {
+    const cwd = browserCwd();
+    if (cwd === null) {
+      return {
+        mappedNodeIds: /* @__PURE__ */ new Set(),
+        scannedFiles: [],
+        skipReason: "no-config"
+      };
+    }
+    return parseCodeConnectMappings(cwd);
+  });
 }
 function seenMainIds(context) {
   return getAnalysisState(context, SEEN_MAIN_IDS_KEY, () => /* @__PURE__ */ new Set());