canicode 0.10.2 → 0.10.3

package/README.md

@@ -44,7 +44,7 @@ Rule scores aren't guesswork. The calibration pipeline converts real Figma desig
    - ✅ **scene write succeeded** — the property was written directly to the scene node or instance override.
    - 📝 **annotated the scene node** — the skill left a structured annotation instead of writing the property. This is the [ADR-012](.claude/docs/ADR.md) default for instance-child layout writes, because propagating a property to the component definition (and therefore every instance of it) is almost never what the user wants. **A summary full of 📝 markers is correct behavior, not failure.**
    - 🌐 **definition write propagated** — the property was written to the component definition and every instance inherited it. Only happens when the user opted in up front with `allowDefinitionWrite`.
-4. **Re-analyze** — verify the grade improved. Repeat step 2 if new gotchas surface.
+4. **Re-analyze** — verify the gotchas were addressed (grade movement is a side-effect). Repeat step 2 if new gotchas surface.
 5. **Hand off** to `figma-implement-design` — canicode's scope ends here ([ADR-013](.claude/docs/ADR.md)). Figma's official code-generation skill takes the now-clean design and produces code.
 
 ---

package/dist/cli/index.js

@@ -3286,6 +3286,15 @@ function hasStateInComponentMaster(node, context, statePattern) {
   if (!master) return false;
   return hasStateInVariantProps(master, statePattern);
 }
+function canDetermineVariants(node, context) {
+  if (node.type === "COMPONENT") return true;
+  if (node.componentPropertyDefinitions !== void 0) return true;
+  if (node.componentId !== void 0) {
+    const defs = context.file.componentDefinitions;
+    if (defs && defs[node.componentId] !== void 0) return true;
+  }
+  return false;
+}
 var missingInteractionStateDef = {
   id: "missing-interaction-state",
   name: "Missing Interaction State",
@@ -3300,6 +3309,7 @@ var missingInteractionStateCheck = (node, context) => {
   if (!interactiveType) return null;
   const expectedStates = EXPECTED_STATES[interactiveType];
   if (!expectedStates) return null;
+  if (!canDetermineVariants(node, context)) return null;
   const seen = getSeen(context);
   const nodePath = context.path.join(" > ");
   for (const state of expectedStates) {
@@ -4015,7 +4025,7 @@ function computeApplyContext(violation, instanceContext) {
 }
 
 // package.json
-var version2 = "0.10.2";
+var version2 = "0.10.3";
 
 // src/core/engine/scoring.ts
 function computeTotalScorePerCategory(configs) {
@@ -5730,7 +5740,8 @@ function generateGotchaSurvey(result, scores) {
   );
   const deduped = deduplicateSiblingIssues(relevantIssues);
   const sorted = stableSortBySeverity(deduped);
-  const questions = sorted.map((issue) => mapToQuestion(issue, result.file)).filter((q) => q !== null);
+  const mapped = sorted.map((issue) => mapToQuestion(issue, result.file)).filter((q) => q !== null);
+  const questions = deduplicateBySourceComponent(mapped);
   return {
     designGrade: grade,
     isReadyForCodeGen: isReadyForCodeGen(grade),
@@ -5800,6 +5811,50 @@ function mapToQuestion(issue, file) {
     ...applyContext.sourceChildId !== void 0 ? { sourceChildId: applyContext.sourceChildId } : {}
   };
 }
+function deduplicateBySourceComponent(questions) {
+  const groups = /* @__PURE__ */ new Map();
+  const order = [];
+  let uniqueCounter = 0;
+  for (const q of questions) {
+    const ic = q.instanceContext;
+    let key;
+    if (ic && ic.sourceComponentId && ic.sourceNodeId) {
+      key = `${ic.sourceComponentId}::${ic.sourceNodeId}::${q.ruleId}`;
+    } else {
+      key = `__unique__${uniqueCounter++}`;
+    }
+    const bucket = groups.get(key);
+    if (bucket) {
+      bucket.push(q);
+    } else {
+      groups.set(key, [q]);
+      order.push(key);
+    }
+  }
+  return order.map((key) => {
+    const group = groups.get(key);
+    const first = group[0];
+    if (group.length === 1) return first;
+    const otherIds = group.slice(1).map((q) => q.nodeId);
+    const sourceComponentName = first.instanceContext?.sourceComponentName;
+    const template = GOTCHA_QUESTIONS[first.ruleId];
+    const renamed = {
+      ...first,
+      replicas: group.length,
+      replicaNodeIds: otherIds
+    };
+    if (sourceComponentName) {
+      renamed.nodeName = sourceComponentName;
+      if (template) {
+        renamed.question = template.question.replace(
+          "{nodeName}",
+          sourceComponentName
+        );
+      }
+    }
+    return renamed;
+  });
+}
 function buildInstanceContext(nodeId, file) {
   const parts = parseInstanceChildNodeId(nodeId);
   if (!parts) return null;