@drawpro/mcp 0.2.0 → 0.2.1

package/README.md

@@ -85,10 +85,20 @@ containers, and unbound annotation arrows all disappear, and the sheet comes
 back as an auto-laid-out graph. Correct content, different diagram.
 
 Use `edit_sheet_text` for those sheets. It reads the scene, rewrites only the
-strings you name, and writes every other element back byte for byte — so
-coordinates, groupings, and annotations are untouched. Edits apply
-all-or-nothing: if any `find` matches nothing, the sheet is left alone rather
-than half-updated.
+strings you name, and writes every other element back — so coordinates,
+groupings, and annotations are untouched. Edits apply all-or-nothing: if any
+`find` matches nothing, the sheet is left alone rather than half-updated.
+
+The one geometry change it makes is growing a box whose text no longer fits.
+Excalidraw regrows a container around text *bound* to it, but hand-drawn
+diagrams usually have text merely sitting on top of a shape, so longer text
+would otherwise spill out the bottom. Boxes are only ever grown, never shrunk,
+and nothing else moves — reflowing neighbours would be the wholesale rewrite
+this tool exists to avoid. Any box it grows is named in the result, so overlap
+with a neighbour is easy to spot and one drag to fix.
+
+It cannot add elements. A correction that needs a new box or a new annotation
+line is still a manual edit.
 
 `update_diagram` remains the right tool for sheets this package generated, where
 regenerating the layout is the point.

package/dist/server.js

@@ -1084,8 +1084,33 @@ function validateScene(elements) {
   return issues;
 }
 
-// ../diagram/src/describe.ts
+// ../diagram/src/fit.ts
 var SHAPES = /* @__PURE__ */ new Set(["rectangle", "ellipse", "diamond"]);
+var PADDING = 16;
+function growBoxesToFitText(elements, changedTextIds) {
+  const shapes = elements.filter((el) => SHAPES.has(el.type));
+  const resizes = [];
+  for (const text2 of elements) {
+    if (text2.type !== "text" || !changedTextIds.has(text2.id)) continue;
+    if (text2.containerId) continue;
+    const cx = text2.x + text2.width / 2;
+    const cy = text2.y + text2.height / 2;
+    const box = shapes.filter((sh) => cx >= sh.x && cx <= sh.x + sh.width && cy >= sh.y && cy <= sh.y + sh.height).sort((a, b) => a.width * a.height - b.width * b.height)[0];
+    if (!box) continue;
+    const needed = Math.ceil(text2.y - box.y + text2.height + PADDING);
+    if (needed > box.height) {
+      resizes.push({ shapeId: box.id, shapeType: box.type, from: box.height, to: needed });
+      box.height = needed;
+      box.version = (box.version ?? 1) + 1;
+      box.versionNonce = Math.floor(Math.random() * 2 ** 31);
+      box.updated = Date.now();
+    }
+  }
+  return resizes;
+}
+
+// ../diagram/src/describe.ts
+var SHAPES2 = /* @__PURE__ */ new Set(["rectangle", "ellipse", "diamond"]);
 var ARROW_LABEL_RADIUS = 60;
 function centre(b) {
   return { x: b.x + b.width / 2, y: b.y + b.height / 2 };
@@ -1126,7 +1151,7 @@ function describeScene(elements) {
     if (bound) return bound;
     const owner = byId.get(id);
     if (!owner) return "";
-    if (SHAPES.has(owner.type)) {
+    if (SHAPES2.has(owner.type)) {
       const hit = floating.find((t) => !adopted.has(t.id) && contains(owner, centre(t)));
       if (hit) {
         adopted.add(hit.id);
@@ -1157,7 +1182,7 @@ function describeScene(elements) {
   };
   const counts = {};
   for (const el of elements) counts[el.type] = (counts[el.type] ?? 0) + 1;
-  const shapes = elements.filter((el) => SHAPES.has(el.type)).map((el) => ({ id: el.id, type: el.type, label: labelFor(el.id) }));
+  const shapes = elements.filter((el) => SHAPES2.has(el.type)).map((el) => ({ id: el.id, type: el.type, label: labelFor(el.id) }));
   const shapeLabel = new Map(shapes.map((s) => [s.id, s.label]));
   const edges = elements.filter((el) => el.type === "arrow").map((el) => {
     const start = el.startBinding;
@@ -1570,6 +1595,7 @@ server.tool(
     const scene = await api().readSheet(workspace_id, sheet_id, unlocked.key);
     const elements = scene.elements;
     const counts = /* @__PURE__ */ new Map();
+    const changedTextIds = /* @__PURE__ */ new Set();
     for (const el of elements) {
       if (el.type !== "text") continue;
       const current = (el.originalText ?? el.text)?.trim();
@@ -1578,14 +1604,18 @@ server.tool(
       if (!edit) continue;
       el.text = edit.replace;
       el.originalText = edit.replace;
-      const metrics = measureText(edit.replace, el.fontSize ?? 20, 1e3);
+      const fontSize = el.fontSize ?? 20;
+      const wrapWidth = el.containerId ? el.width : 1e3;
+      const metrics = measureText(edit.replace, fontSize, wrapWidth);
       el.width = metrics.width;
       el.height = metrics.height;
+      changedTextIds.add(el.id);
       el.version = (el.version ?? 1) + 1;
       el.versionNonce = Math.floor(Math.random() * 2 ** 31);
       el.updated = Date.now();
       counts.set(edit.find, (counts.get(edit.find) ?? 0) + 1);
     }
+    const resizes = growBoxesToFitText(elements, changedTextIds);
     const missed = edits.filter((e) => !counts.has(e.find));
     if (missed.length > 0) {
       return text(
@@ -1600,9 +1630,10 @@ server.tool(
       user.publicKey
     );
     const applied = [...counts.entries()].map(([find, n]) => `  ${JSON.stringify(find)} -> ${n} element${n === 1 ? "" : "s"}`).join("\n");
+    const grewNote = resizes.length ? "\n\nBoxes grown so the new text fits (nothing else moved, so check for overlap):\n" + resizes.map((r) => `  a ${r.shapeType} grew ${Math.round(r.from)}px -> ${Math.round(r.to)}px`).join("\n") : "";
     return text(
       `Updated "${scene.name}". ${elements.length} elements preserved; only the text below changed.
-${applied}
+${applied}${grewNote}
 ${sheetUrl(workspace_id, sheet_id)}`
     );
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@drawpro/mcp",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "MCP server for DrawPro \u2014 read and create end-to-end encrypted Excalidraw diagrams from Claude",
   "license": "MIT",
   "repository": {