@drawpro/mcp 0.1.1 → 0.2.1

package/README.md

@@ -6,11 +6,18 @@ account from Claude Code, Claude Desktop, or any MCP client.
 ## Setup
 
 ```bash
-claude mcp add drawpro -e DRAWPRO_TOKEN="dp_live_..." -- npx -y @drawpro/mcp
+claude mcp add drawpro --scope user -e DRAWPRO_TOKEN="dp_live_..." -- npx -y @drawpro/mcp
 ```
 
 Mint the token in DrawPro under **Connect to Claude Code**.
 
+`--scope user` registers it for every project on your machine. The scope
+otherwise defaults to `local`, which loads the server only in the directory the
+command was run in — diagramming is not a property of one repo, so that default
+is almost never what you want here. Avoid `--scope project`: it writes
+`.mcp.json` into whichever repo you are standing in, token included, where it
+invites being committed.
+
 To read existing sheets, unlock the account once:
 
 ```bash
@@ -67,7 +74,34 @@ passcode itself.
 | `read_sheet` | yes | returns a readable outline, not raw scene JSON |
 | `validate_spec` | no | no side effects; check before writing |
 | `create_diagram` | no | returns a link to the new sheet |
-| `update_diagram` | no | replaces the sheet wholesale |
+| `update_diagram` | no | replaces the sheet wholesale — regenerates layout |
+| `edit_sheet_text` | yes | rewrites named text in place, preserving every other element |
+
+### Editing a sheet a person drew
+
+`update_diagram` builds the scene from a spec, so layout is derived and
+hand-placed content cannot survive it: text elements positioned by hand, region
+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 — 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.
 
 `read_sheet` deliberately returns shapes and edges rather than Excalidraw JSON.
 A real sheet's raw scene runs to tens of thousands of characters of coordinates,

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;
@@ -1328,7 +1353,7 @@ function api() {
     const token = process.env.DRAWPRO_TOKEN;
     if (!token) {
       throw new ConfigError(
-        'DRAWPRO_TOKEN is not set. Create a token in DrawPro under "Connect to Claude Code", then:\n  claude mcp add drawpro -e DRAWPRO_TOKEN="dp_live_..." -- npx -y @drawpro/mcp'
+        'DRAWPRO_TOKEN is not set. Create a token in DrawPro under "Connect to Claude Code", then:\n  claude mcp add drawpro --scope user -e DRAWPRO_TOKEN="dp_live_..." -- npx -y @drawpro/mcp'
       );
     }
     clientInstance = new DrawProClient(BASE_URL, token);
@@ -1551,6 +1576,68 @@ async function login(forget) {
   console.log(`Unlocked ${user.email}. Key stored in the ${location}.`);
   console.log("Claude can now read your sheets. The passcode was not stored or sent anywhere.");
 }
+server.tool(
+  "edit_sheet_text",
+  "Correct the wording on an existing sheet without redrawing it. Use this for any sheet a person drew: update_diagram regenerates layout from a spec, so it discards hand-placed elements, region containers, and unbound annotations. This rewrites only the text you name and leaves every other element, and every coordinate, exactly as it was.",
+  {
+    workspace_id: import_zod.z.string(),
+    sheet_id: import_zod.z.string(),
+    edits: import_zod.z.array(
+      import_zod.z.object({
+        find: import_zod.z.string().describe("The element's exact current text, as read_sheet reports it"),
+        replace: import_zod.z.string()
+      })
+    ).describe("Applied all-or-nothing: if any find has no match, nothing is written.")
+  },
+  async ({ workspace_id, sheet_id, edits }) => {
+    const unlocked = await unlockedKey();
+    if ("error" in unlocked) return text(unlocked.error);
+    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();
+      if (current === void 0) continue;
+      const edit = edits.find((e) => e.find.trim() === current);
+      if (!edit) continue;
+      el.text = edit.replace;
+      el.originalText = edit.replace;
+      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(
+        "Nothing was written. These strings matched no text element:\n" + missed.map((e) => `  ${JSON.stringify(e.find)}`).join("\n") + "\n\nRun read_sheet and copy the text exactly as it appears there."
+      );
+    }
+    const user = await currentUser();
+    await api().updateSheet(
+      workspace_id,
+      sheet_id,
+      { name: scene.name, elements, appState: scene.appState },
+      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}${grewNote}
+${sheetUrl(workspace_id, sheet_id)}`
+    );
+  }
+);
 async function main() {
   const command = process.argv[2];
   if (command === "login") {

package/package.json

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