@gakim-digital/dexter-bridge 0.13.5 → 0.13.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gakim-digital/dexter-bridge",
-  "version": "0.13.5",
+  "version": "0.13.6",
   "description": "Local bridge for InstaWebAI and Dexter — runs Codex, Claude Code, or OpenCode on your machine.",
   "type": "module",
   "bin": {

package/src/framerRequirementContract.js

@@ -67,6 +67,20 @@ export function mergeRequirements(current, next) {
   return [...byId.values()];
 }
 
+// A requirement may name layers its own edit creates by their temporary ids,
+// or a breakpoint copy of one as the breakpoint id followed by that temporary
+// id. Once Framer gives those layers real ids, the requirement names them.
+export function renameRequirementNodes(requirements, renamedIds = {}) {
+  const renames = Object.entries(renamedIds || {}).filter(([from, to]) => from && to && from !== to);
+  if (!renames.length) return requirements;
+  const rename = (id) => {
+    if (renamedIds[id]) return renamedIds[id];
+    const copy = renames.find(([from]) => id.endsWith(from) && /^[A-Za-z0-9_-]{9}$/.test(id.slice(0, -from.length)));
+    return copy ? `${id.slice(0, -copy[0].length)}${copy[1]}` : id;
+  };
+  return requirements.map((item) => ({ ...item, nodeIds: [...new Set(item.nodeIds.map(rename))] }));
+}
+
 function finiteNumber(value) {
   const number = Number(value);
   return Number.isFinite(number) ? number : null;
@@ -312,22 +326,29 @@ export function summarizeRequirements(requirements, checkResults) {
 }
 
 // Compares two snapshots of layers that had to stay as they were. A snapshot
-// maps a layer id to { name, origin, rect, props } where props is the
-// serialized layer without children, origin is the layer a breakpoint or
-// variant copy follows, and rect is null when Dexter could not measure the
-// layer, so only its settings are compared. Sizes are compared only when both
-// readings came from the same source (the user's editor or Framer MCP), and
-// only size counts, not position: a section that grows moves every section
-// below it.
-// A change is edited when the run edited the layer, or the layer it follows;
-// any other change happened without the model touching the layer.
+// maps a layer id to { name, origin, rect, props, children } where props is
+// the serialized layer without children, origin is the layer a breakpoint or
+// variant copy follows, rect is null when Dexter could not measure the layer,
+// so only its settings are compared, and children lists its child layers in
+// order as { id, name, origin }. Sizes are compared only when both readings
+// came from the same source (the user's editor or Framer MCP), and only size
+// counts, not position: a section that grows moves every section below it.
+// The order of child layers does count: a layer inserted without an index
+// lands at the end of its parent.
+// A change is edited when the run edited the layer, the layer it follows, or
+// for a new order one of the child layers; any other change happened without
+// the model touching the layer.
+function childOrder(layer) {
+  return layer.children.map((child) => child.id).join(',');
+}
+
 export function unchangedViolations(before, after, edited = new Set(), tolerance = 1) {
   const changes = [];
   for (const [id, earlier] of Object.entries(before || {})) {
     const later = after?.[id];
     const name = earlier.name || id;
-    const source = edited.has(id) ? 'self' : earlier.origin && edited.has(earlier.origin) ? 'origin' : null;
-    const note = source === 'self' ? ' (you edited it)' : source === 'origin' ? ` (it follows your edit of ${earlier.origin})` : '';
+    const editedLayer = (layer) => edited.has(layer.id) || Boolean(layer.origin && edited.has(layer.origin));
+    let source = edited.has(id) ? 'self' : earlier.origin && edited.has(earlier.origin) ? 'origin' : null;
     const a = earlier.rect;
     const b = later?.rect;
     const comparable = a && b && (a.source || null) === (b.source || null);
@@ -335,7 +356,15 @@ export function unchangedViolations(before, after, edited = new Set(), tolerance
     if (!later) text = `${name} was removed`;
     else if (comparable && ['width', 'height'].some((key) => Math.abs(a[key] - b[key]) > tolerance)) {
       text = `${name} changed size (${a.width}×${a.height} → ${b.width}×${b.height})`;
+    } else if (earlier.children && later.children && childOrder(earlier) !== childOrder(later)) {
+      text = `${name} now holds its layers in this order: ${later.children.slice(0, 12).map((child) => child.name || child.id).join(', ')}`;
+      if (!source && [...earlier.children, ...later.children].some(editedLayer)) source = 'children';
     } else if (earlier.props !== later.props) text = `${name} has different settings`;
+    const note = source === 'self'
+      ? ' (you edited it)'
+      : source === 'origin'
+        ? ` (it follows your edit of ${earlier.origin})`
+        : source === 'children' ? ' (you edited layers inside it)' : '';
     if (text) changes.push({ text: `${text}${note}`, edited: Boolean(source) });
     if (changes.length >= 20) break;
   }

package/src/framerTools.js

@@ -9,6 +9,7 @@ import {
   normalizeChecks,
   normalizeRequirements,
   summarizeRequirements,
+  renameRequirementNodes,
   unchangedViolations,
   verifyChecks,
 } from './framerRequirementContract.js';
@@ -879,7 +880,12 @@ for (const id of ids) {
   const rect = (await __dexterLaidOut(id)) ? await __dexterRect(id) : null;
   let props = '';
   try { const { children, ...rest } = JSON.parse(JSON.stringify(node)); props = fingerprint(JSON.stringify(rest)); } catch {}
-  out[id] = { name: node.name || null, origin: node.$originalId || null, rect: rect ? { x: rect.x, y: rect.y, width: rect.width, height: rect.height } : null, props };
+  const children = (Array.isArray(node.children) ? node.children : []).slice(0, 60).map((child) => {
+    const childId = child && typeof child === 'object' ? child.id : child;
+    const childNode = childId ? tree.get(childId) : null;
+    return { id: childId, name: (childNode && childNode.name) || null, origin: (childNode && childNode.$originalId) || null };
+  }).filter((child) => child.id);
+  out[id] = { name: node.name || null, origin: node.$originalId || null, rect: rect ? { x: rect.x, y: rect.y, width: rect.width, height: rect.height } : null, props, children };
 }
 return out;`;
 }
@@ -1341,6 +1347,7 @@ export function createFramerAgentToolRuntime({
         await snapshotUnchanged();
         const result = await client.exec(wrapProgram(program));
         const applied = applyReceipt(result, screenshotNodeIds);
+        requirements = renameRequirementNodes(requirements, applied.receipt.renamedIds);
         const programError = programErrorOf(applied.output, wrappedLineOffset(program));
         // Layers created by this program only get their real ids once it
         // finishes, so screenshots are taken in a second call.
@@ -1648,7 +1655,7 @@ export function createFramerAgentToolRuntime({
     }
     const unchecked = uncheckedRequirements();
     if (unchecked.length) {
-      facts.push(`These requirements from your plan have not been checked since your last change, so they are not confirmed either way: ${unchecked.map((item) => item.text).join('; ')}.`);
+      facts.push(`These requirements from your plan have not been checked since your last change, so they are not confirmed either way: ${unchecked.map((item) => `${item.id}: ${item.text}`).join('; ')}.`);
     }
     if (unchangedChanges.length) {
       facts.push(`These layers you planned to leave as they were now read differently: ${unchangedChanges.map((change) => change.text).join('; ')}.`);