fix(chat): keep the request alive while its answer streams, so a stop still reaches the provider

A Request's signal follows the signal it was created with through a weak link inside undici: once the Request object is garbage collected, the link is gone and request.signal never aborts (nodejs/undici#3644). The chat route passes req.signal to streamText and holds nothing else from the request, so a GC pause during a stream could leave the model call running after the client stopped or disconnected, with no onAbort accounting. The route now keeps each request in a WeakMap keyed by its response, which Next holds while the body is piped. This is also the cause of the unit test chat-route-abort hanging on CI (three times this week, 60 ms locally): the test's Request is dropped as soon as the route returns, and a GC in that window lost the abort. With a forced GC the test hangs on Node 20 and 24 without this change and passes with it.
2026-10-10 19:49:52 +08:00 · 2026-10-10 09:10:58 +09:00
20 changed files with 230 additions and 1543 deletions
@@ -55,16 +55,12 @@ import { allowPrivateUrls, isPrivateUrl } from "@/lib/ssrf-protection"
 import { getSystemPrompt } from "@/lib/system-prompts"
 import { normalizeBaseUrl } from "@/lib/types/model-config"
 import { getUserIdFromRequest } from "@/lib/user-id"
-import { foldCells } from "@/packages/mcp-server/src/compact-cells.ts"
 import { hasCells } from "@/packages/mcp-server/src/pages.ts"
 import {
    getShapeLibrary,
    SHAPE_LIBRARY_LIST,
 } from "@/packages/mcp-server/src/shape-library.ts"
-import {
-    STYLE_CLASS_EXAMPLE,
-    SWIMLANE_EXAMPLE,
-} from "@/packages/mcp-server/src/xml-examples.ts"
+import { SWIMLANE_EXAMPLE } from "@/packages/mcp-server/src/xml-examples.ts"

 // No explicit cap: a reasoning model can spend minutes planning before it emits
 // the tool call, so take whatever the host allows. Vercel's own default is 300s,
@@ -524,20 +520,18 @@ ${userInputText}
        SINGLE_SYSTEM_PROVIDERS.has(resolvedProvider) || isCustomOpenAIEndpoint

    const selectionContext = formatSelectionContext(body.selectedCells)
-    // The model reads the diagram in the compact notation it writes
-    // (compact-cells.ts); the canvas itself keeps the full XML
    const xmlContext = `${
        previousXml
            ? `Previous diagram XML (before user's last message):
 """xml
-${foldCells(previousXml)}
+${previousXml}
 """

 `
            : ""
    }Current diagram XML (AUTHORITATIVE - the source of truth):
 """xml
-${foldCells(xml || "")}
+${xml || ""}
 """

 IMPORTANT: The "Current diagram XML" is the SINGLE SOURCE OF TRUTH for what's on the canvas right now. The user can manually add, delete, or modify shapes directly in draw.io. Always count and describe elements based on the CURRENT XML, not on what you previously generated. If both previous and current XML are shown, compare them to understand what the user changed.${selectionContext ? `\n\n${selectionContext}` : ""}`
@@ -665,24 +659,19 @@ IMPORTANT: The "Current diagram XML" is the SINGLE SOURCE OF TRUTH for what's on
        tools: {
            // Client-side tool that will be executed on the client
            display_diagram: {
-                description: `Display a diagram on draw.io. Pass ONLY the mxStyle definitions and the mxCell elements - wrapper tags and root cells are added automatically.
+                description: `Display a diagram on draw.io. Pass ONLY the mxCell elements - wrapper tags and root cells are added automatically.

 VALIDATION RULES (XML will be rejected if violated):
-1. Generate ONLY mxStyle definitions and mxCell elements - NO wrapper tags (<mxfile>, <mxGraphModel>, <root>)
+1. Generate ONLY mxCell elements - NO wrapper tags (<mxfile>, <mxGraphModel>, <root>)
 2. Do NOT include root cells (id="0" or id="1") - they are added automatically
 3. All mxCell elements must be siblings - never nested
 4. Every mxCell needs a unique id (start from "2")
-5. parent defaults to "1"; write parent="<container-id>" only for shapes inside a container
+5. Every mxCell needs a valid parent attribute (use "1" for top-level)
 6. Escape special chars in values: &lt; &gt; &amp; &quot;

-A shape is one self-closing mxCell with x, y, w and h; an edge is one with source and target. vertex="1", edge="1", parent="1" and the mxGeometry element are added automatically, so write them only when needed: parent for a shape inside a container, an mxGeometry element for edge waypoints or a label placed on an edge.
-
 Example (generate ONLY this - no wrapper tags):
 ${SWIMLANE_EXAMPLE}

-Styles: define a style used by several cells ONCE with <mxStyle name="..." value="..."/> before the cells and use the name in style like a CSS class; overrides after the name win. Name only styles that two or more cells share; names must not be draw.io's own style names (text, ellipse, rhombus, swimlane, label, image, blue, green, red, gray, yellow, orange, purple, pink). A definition applies to the call it is in. html=1 and whiteSpace=wrap are added automatically, never write them. Labels are HTML: use &lt;br&gt; for a line break, never \\n; a literal < or > is written &amp;lt; or &amp;gt;.
-${STYLE_CLASS_EXAMPLE}
-
 Notes:
 - For AWS diagrams, use **AWS 2025 icons**.
 - For animated connectors, add "flowAnimation=1" to edge style.
@@ -701,12 +690,12 @@ Operations:
 - add: Add a new cell. Provide cell_id (new unique id) and new_xml.
 - delete: Remove a cell. Cascade is automatic: children AND edges (source/target) are auto-deleted. Only specify ONE cell_id.

-For update/add, new_xml is the complete mxCell in the compact form (a shape with x, y, w, h; an edge with source and target), with its complete style (named styles are not available in edit_diagram).
+For update/add, new_xml must be a complete mxCell element including mxGeometry.

 ⚠️ JSON ESCAPING: Every " inside new_xml MUST be escaped as \\". Example: id=\\"5\\" value=\\"Label\\"

 Example - Add a rectangle:
-{"operations": [{"operation": "add", "cell_id": "rect-1", "new_xml": "<mxCell id=\\"rect-1\\" value=\\"Hello\\" style=\\"rounded=1;\\" x=\\"100\\" y=\\"100\\" w=\\"120\\" h=\\"60\\"/>"}]}
+{"operations": [{"operation": "add", "cell_id": "rect-1", "new_xml": "<mxCell id=\\"rect-1\\" value=\\"Hello\\" style=\\"rounded=0;\\" vertex=\\"1\\" parent=\\"1\\"><mxGeometry x=\\"100\\" y=\\"100\\" width=\\"120\\" height=\\"60\\" as=\\"geometry\\"/></mxCell>"}]}

 Example - Delete container (children & edges auto-deleted):
 {"operations": [{"operation": "delete", "cell_id": "2"}]}`,
@@ -10,15 +10,8 @@ import {
    extractCompleteMxCells,
    replaceNodes,
 } from "@/lib/utils"
-import { expandCompactCells } from "@/packages/mcp-server/src/compact-cells.ts"
 import { applyDiagramOperations } from "@/packages/mcp-server/src/diagram-operations.ts"
 import { BLANK_MXFILE } from "@/packages/mcp-server/src/pages.ts"
-import {
-    addDefaultStyles,
-    applyStyleClasses,
-    edgeIdsOf,
-    readStyleClasses,
-} from "@/packages/mcp-server/src/style-classes.ts"

 // Helper to extract complete operations from streaming input
 function getCompleteOperations(
@@ -78,17 +71,9 @@ export function useDiagramStreaming({
    // so far. The tool handler validates and loads the final diagram.
    const handleDisplayChart = useCallback(
        (xml: string) => {
-            // Named styles come before the cells, so the ones written so
-            // far are known by the time their cells stream in
-            const { classes, xml: cellsXml } = readStyleClasses(xml || "")
-            const completeCells = extractCompleteMxCells(cellsXml)
+            const completeCells = extractCompleteMxCells(xml || "")
            if (!completeCells) return
-            const convertedXml = addDefaultStyles(
-                applyStyleClasses(
-                    expandCompactCells(convertToLegalXml(completeCells)),
-                    classes,
-                ),
-            )
+            const convertedXml = convertToLegalXml(completeCells)
            if (convertedXml === previousXML.current) return

            // Skip this update while the cells written so far don't parse
@@ -217,20 +202,9 @@ export function useDiagramStreaming({
            if (lastProcessedXmlRef.current.get(opsKey) === ops) continue
            lastProcessedXmlRef.current.set(opsKey, ops)
            try {
-                const edges = edgeIdsOf(originalXml)
                const { result } = applyDiagramOperations(
                    originalXml,
-                    completeOps.map((op) =>
-                        op.new_xml
-                            ? {
-                                  ...op,
-                                  new_xml: addDefaultStyles(
-                                      expandCompactCells(op.new_xml),
-                                      edges,
-                                  ),
-                              }
-                            : op,
-                    ),
+                    completeOps,
                )
                if (lastProcessedXmlRef.current.get(resultKey) === result) {
                    continue
@@ -8,10 +8,8 @@ import type { LoadMode } from "@/contexts/diagram-context"
 import type { ValidationResult } from "@/lib/diagram-validator"
 import { formatValidationFeedback } from "@/lib/diagram-validator"
 import { isMxCellXmlComplete } from "@/lib/utils"
-import { foldCells } from "@/packages/mcp-server/src/compact-cells.ts"
 import { editDiagram } from "@/packages/mcp-server/src/edit-diagram.ts"
 import { prepareNewDiagram } from "@/packages/mcp-server/src/new-diagram.ts"
-import { hasCells } from "@/packages/mcp-server/src/pages.ts"

 const DEBUG = process.env.NODE_ENV === "development"

@@ -518,7 +516,7 @@ ${finalXml}

 Current diagram XML:
 \`\`\`xml
-${foldCells(currentXml)}
+${currentXml}
 \`\`\`

 Please check the cell IDs and retry.`,
@@ -547,7 +545,7 @@ Please check the cell IDs and retry.`,

 Current diagram XML:
 \`\`\`xml
-${currentXml ? foldCells(currentXml) : "No XML available"}
+${currentXml || "No XML available"}
 \`\`\`

 Please check cell IDs and retry, or use display_diagram to regenerate.`,
@@ -577,10 +575,7 @@ Use display_diagram to create the complete diagram, or edit_diagram to change th
        // Detect if LLM incorrectly started fresh instead of continuing
        // LLM should only output bare mxCells now, so wrapper tags indicate error
        const trimmed = xml.trim()
-        // Definitions at the start are a restart only once cells were written
        const isFreshStart =
-            (trimmed.startsWith("<mxStyle") &&
-                hasCells(partialXmlRef.current)) ||
            trimmed.startsWith("<mxGraphModel") ||
            trimmed.startsWith("<root") ||
            trimmed.startsWith("<mxfile") ||
@@ -740,154 +740,146 @@ export const CACHED_EXAMPLE_RESPONSES: CachedResponse[] = [
    {
        promptText: "Draw a cat for me",
        hasImage: false,
-        xml: `<mxCell id="2" edge="1" parent="1" style="curved=1;endArrow=none;html=1;strokeWidth=14;strokeColor=#F4A460;" value="">
-    <mxGeometry relative="1" as="geometry">
+        xml: `<mxCell id="2" value="" style="ellipse;whiteSpace=wrap;html=1;aspect=fixed;fillColor=#FFE6CC;strokeColor=#000000;strokeWidth=2;" vertex="1" parent="1">
+    <mxGeometry x="300" y="150" width="120" height="120" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="3" value="" style="triangle;whiteSpace=wrap;html=1;fillColor=#FFE6CC;strokeColor=#000000;strokeWidth=2;rotation=30;" vertex="1" parent="1">
+    <mxGeometry x="280" y="120" width="50" height="60" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="4" value="" style="triangle;whiteSpace=wrap;html=1;fillColor=#FFE6CC;strokeColor=#000000;strokeWidth=2;rotation=-30;" vertex="1" parent="1">
+    <mxGeometry x="390" y="120" width="50" height="60" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="5" value="" style="triangle;whiteSpace=wrap;html=1;fillColor=#FFB6C1;strokeColor=none;rotation=30;" vertex="1" parent="1">
+    <mxGeometry x="290" y="135" width="30" height="35" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="6" value="" style="triangle;whiteSpace=wrap;html=1;fillColor=#FFB6C1;strokeColor=none;rotation=-30;" vertex="1" parent="1">
+    <mxGeometry x="400" y="135" width="30" height="35" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="7" value="" style="ellipse;whiteSpace=wrap;html=1;aspect=fixed;fillColor=#000000;strokeColor=#000000;" vertex="1" parent="1">
+    <mxGeometry x="325" y="185" width="15" height="15" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="8" value="" style="ellipse;whiteSpace=wrap;html=1;aspect=fixed;fillColor=#000000;strokeColor=#000000;" vertex="1" parent="1">
+    <mxGeometry x="380" y="185" width="15" height="15" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="9" value="" style="triangle;whiteSpace=wrap;html=1;fillColor=#FFB6C1;strokeColor=#000000;strokeWidth=1;rotation=180;" vertex="1" parent="1">
+    <mxGeometry x="350" y="210" width="20" height="15" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="10" value="" style="curved=1;endArrow=none;html=1;strokeColor=#000000;strokeWidth=2;exitX=0.5;exitY=1;exitDx=0;exitDy=0;" edge="1" parent="1">
+    <mxGeometry width="50" height="50" relative="1" as="geometry">
+      <mxPoint x="360" y="220" as="sourcePoint"/>
+      <mxPoint x="340" y="235" as="targetPoint"/>
      <Array as="points">
-        <mxPoint x="580" y="480"/>
-        <mxPoint x="600" y="380"/>
+        <mxPoint x="355" y="230"/>
      </Array>
-      <mxPoint x="490" y="480" as="sourcePoint"/>
-      <mxPoint x="560" y="330" as="targetPoint"/>
    </mxGeometry>
  </mxCell>


-  <mxCell id="3" parent="1" style="ellipse;whiteSpace=wrap;html=1;fillColor=#F4A460;strokeColor=#8B5A2B;strokeWidth=2;" value="" vertex="1">
-    <mxGeometry height="190" width="200" x="320" y="320" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="4" parent="1" style="ellipse;whiteSpace=wrap;html=1;fillColor=#FFE4C4;strokeColor=none;" value="" vertex="1">
-    <mxGeometry height="120" width="100" x="370" y="360" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="5" parent="1" style="ellipse;whiteSpace=wrap;html=1;fillColor=#F4A460;strokeColor=#8B5A2B;strokeWidth=2;" value="" vertex="1">
-    <mxGeometry height="40" width="60" x="350" y="480" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="6" parent="1" style="ellipse;whiteSpace=wrap;html=1;fillColor=#F4A460;strokeColor=#8B5A2B;strokeWidth=2;" value="" vertex="1">
-    <mxGeometry height="40" width="60" x="430" y="480" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="7" parent="1" style="triangle;direction=north;whiteSpace=wrap;html=1;fillColor=#F4A460;strokeColor=#8B5A2B;strokeWidth=2;rotation=-15;" value="" vertex="1">
-    <mxGeometry height="80" width="70" x="310" y="110" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="8" parent="1" style="triangle;direction=north;whiteSpace=wrap;html=1;fillColor=#F4A460;strokeColor=#8B5A2B;strokeWidth=2;rotation=15;" value="" vertex="1">
-    <mxGeometry height="80" width="70" x="460" y="110" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="9" parent="1" style="triangle;direction=north;whiteSpace=wrap;html=1;fillColor=#FFB6C1;strokeColor=none;rotation=-15;" value="" vertex="1">
-    <mxGeometry height="45" width="36" x="327" y="135" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="10" parent="1" style="triangle;direction=north;whiteSpace=wrap;html=1;fillColor=#FFB6C1;strokeColor=none;rotation=15;" value="" vertex="1">
-    <mxGeometry height="45" width="36" x="477" y="135" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="11" parent="1" style="ellipse;whiteSpace=wrap;html=1;fillColor=#F4A460;strokeColor=#8B5A2B;strokeWidth=2;" value="" vertex="1">
-    <mxGeometry height="200" width="240" x="300" y="140" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="12" parent="1" style="ellipse;whiteSpace=wrap;html=1;fillColor=#333333;strokeColor=none;" value="" vertex="1">
-    <mxGeometry height="44" width="36" x="355" y="200" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="13" parent="1" style="ellipse;whiteSpace=wrap;html=1;fillColor=#333333;strokeColor=none;" value="" vertex="1">
-    <mxGeometry height="44" width="36" x="449" y="200" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="14" parent="1" style="ellipse;whiteSpace=wrap;html=1;fillColor=#FFFFFF;strokeColor=none;" value="" vertex="1">
-    <mxGeometry height="13" width="13" x="363" y="207" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="15" parent="1" style="ellipse;whiteSpace=wrap;html=1;fillColor=#FFFFFF;strokeColor=none;" value="" vertex="1">
-    <mxGeometry height="13" width="13" x="457" y="207" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="16" parent="1" style="ellipse;whiteSpace=wrap;html=1;fillColor=#FFB6C1;strokeColor=none;opacity=70;" value="" vertex="1">
-    <mxGeometry height="22" width="40" x="325" y="255" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="17" parent="1" style="ellipse;whiteSpace=wrap;html=1;fillColor=#FFB6C1;strokeColor=none;opacity=70;" value="" vertex="1">
-    <mxGeometry height="22" width="40" x="475" y="255" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="18" parent="1" style="triangle;direction=south;whiteSpace=wrap;html=1;fillColor=#FF69B4;strokeColor=none;" value="" vertex="1">
-    <mxGeometry height="16" width="24" x="408" y="252" as="geometry"/>
-  </mxCell>
-
-
-  <mxCell id="19" edge="1" parent="1" style="curved=1;endArrow=none;html=1;strokeWidth=2;strokeColor=#333333;" value="">
-    <mxGeometry relative="1" as="geometry">
+  <mxCell id="11" value="" style="curved=1;endArrow=none;html=1;strokeColor=#000000;strokeWidth=2;" edge="1" parent="1">
+    <mxGeometry width="50" height="50" relative="1" as="geometry">
+      <mxPoint x="360" y="220" as="sourcePoint"/>
+      <mxPoint x="380" y="235" as="targetPoint"/>
      <Array as="points">
-        <mxPoint x="410" y="288"/>
+        <mxPoint x="365" y="230"/>
      </Array>
-      <mxPoint x="400" y="278" as="sourcePoint"/>
-      <mxPoint x="420" y="270" as="targetPoint"/>
    </mxGeometry>
  </mxCell>


-  <mxCell id="20" edge="1" parent="1" style="curved=1;endArrow=none;html=1;strokeWidth=2;strokeColor=#333333;" value="">
-    <mxGeometry relative="1" as="geometry">
+  <mxCell id="12" value="" style="endArrow=none;html=1;strokeColor=#000000;strokeWidth=1.5;" edge="1" parent="1">
+    <mxGeometry width="50" height="50" relative="1" as="geometry">
+      <mxPoint x="310" y="200" as="sourcePoint"/>
+      <mxPoint x="260" y="195" as="targetPoint"/>
+    </mxGeometry>
+  </mxCell>
+
+
+  <mxCell id="13" value="" style="endArrow=none;html=1;strokeColor=#000000;strokeWidth=1.5;" edge="1" parent="1">
+    <mxGeometry width="50" height="50" relative="1" as="geometry">
+      <mxPoint x="310" y="210" as="sourcePoint"/>
+      <mxPoint x="260" y="210" as="targetPoint"/>
+    </mxGeometry>
+  </mxCell>
+
+
+  <mxCell id="14" value="" style="endArrow=none;html=1;strokeColor=#000000;strokeWidth=1.5;" edge="1" parent="1">
+    <mxGeometry width="50" height="50" relative="1" as="geometry">
+      <mxPoint x="310" y="220" as="sourcePoint"/>
+      <mxPoint x="260" y="225" as="targetPoint"/>
+    </mxGeometry>
+  </mxCell>
+
+
+  <mxCell id="15" value="" style="endArrow=none;html=1;strokeColor=#000000;strokeWidth=1.5;" edge="1" parent="1">
+    <mxGeometry width="50" height="50" relative="1" as="geometry">
+      <mxPoint x="410" y="200" as="sourcePoint"/>
+      <mxPoint x="460" y="195" as="targetPoint"/>
+    </mxGeometry>
+  </mxCell>
+
+
+  <mxCell id="16" value="" style="endArrow=none;html=1;strokeColor=#000000;strokeWidth=1.5;" edge="1" parent="1">
+    <mxGeometry width="50" height="50" relative="1" as="geometry">
+      <mxPoint x="410" y="210" as="sourcePoint"/>
+      <mxPoint x="460" y="210" as="targetPoint"/>
+    </mxGeometry>
+  </mxCell>
+
+
+  <mxCell id="17" value="" style="endArrow=none;html=1;strokeColor=#000000;strokeWidth=1.5;" edge="1" parent="1">
+    <mxGeometry width="50" height="50" relative="1" as="geometry">
+      <mxPoint x="410" y="220" as="sourcePoint"/>
+      <mxPoint x="460" y="225" as="targetPoint"/>
+    </mxGeometry>
+  </mxCell>
+
+
+  <mxCell id="18" value="" style="ellipse;whiteSpace=wrap;html=1;fillColor=#FFE6CC;strokeColor=#000000;strokeWidth=2;" vertex="1" parent="1">
+    <mxGeometry x="285" y="250" width="150" height="180" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="19" value="" style="ellipse;whiteSpace=wrap;html=1;fillColor=#FFFFFF;strokeColor=none;" vertex="1" parent="1">
+    <mxGeometry x="315" y="280" width="90" height="120" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="20" value="" style="ellipse;whiteSpace=wrap;html=1;fillColor=#FFE6CC;strokeColor=#000000;strokeWidth=2;" vertex="1" parent="1">
+    <mxGeometry x="300" y="410" width="40" height="50" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="21" value="" style="ellipse;whiteSpace=wrap;html=1;fillColor=#FFE6CC;strokeColor=#000000;strokeWidth=2;" vertex="1" parent="1">
+    <mxGeometry x="380" y="410" width="40" height="50" as="geometry"/>
+  </mxCell>
+
+
+  <mxCell id="22" value="" style="curved=1;endArrow=none;html=1;strokeColor=#000000;strokeWidth=3;fillColor=#FFE6CC;" edge="1" parent="1">
+    <mxGeometry width="50" height="50" relative="1" as="geometry">
+      <mxPoint x="285" y="340" as="sourcePoint"/>
+      <mxPoint x="240" y="260" as="targetPoint"/>
      <Array as="points">
-        <mxPoint x="430" y="288"/>
+        <mxPoint x="260" y="350"/>
+        <mxPoint x="240" y="320"/>
+        <mxPoint x="235" y="290"/>
      </Array>
-      <mxPoint x="420" y="270" as="sourcePoint"/>
-      <mxPoint x="440" y="278" as="targetPoint"/>
    </mxGeometry>
-  </mxCell>
-
-
-  <mxCell id="21" edge="1" parent="1" style="endArrow=none;html=1;strokeWidth=2;strokeColor=#5C3A1A;" value="">
-    <mxGeometry relative="1" as="geometry">
-      <mxPoint x="370" y="262" as="sourcePoint"/>
-      <mxPoint x="270" y="245" as="targetPoint"/>
-    </mxGeometry>
-  </mxCell>
-
-
-  <mxCell id="22" edge="1" parent="1" style="endArrow=none;html=1;strokeWidth=2;strokeColor=#5C3A1A;" value="">
-    <mxGeometry relative="1" as="geometry">
-      <mxPoint x="370" y="272" as="sourcePoint"/>
-      <mxPoint x="268" y="278" as="targetPoint"/>
-    </mxGeometry>
-  </mxCell>
-
-
-  <mxCell id="23" edge="1" parent="1" style="endArrow=none;html=1;strokeWidth=2;strokeColor=#5C3A1A;" value="">
-    <mxGeometry relative="1" as="geometry">
-      <mxPoint x="470" y="262" as="sourcePoint"/>
-      <mxPoint x="570" y="245" as="targetPoint"/>
-    </mxGeometry>
-  </mxCell>
-
-
-  <mxCell id="24" edge="1" parent="1" style="endArrow=none;html=1;strokeWidth=2;strokeColor=#5C3A1A;" value="">
-    <mxGeometry relative="1" as="geometry">
-      <mxPoint x="470" y="272" as="sourcePoint"/>
-      <mxPoint x="572" y="278" as="targetPoint"/>
-    </mxGeometry>
-  </mxCell>
-
-
-  <mxCell id="25" parent="1" style="text;html=1;align=center;verticalAlign=middle;fontSize=22;fontStyle=1;fontColor=#FF69B4;" value="Meow~ ♥" vertex="1">
-    <mxGeometry height="40" width="120" x="530" y="120" as="geometry"/>
  </mxCell>`,
    },
 ]
@@ -4,7 +4,6 @@
 */

 import {
-    STYLE_CLASS_EXAMPLE,
    SWIMLANE_EXAMPLE,
    TWO_EDGES_EXAMPLE,
    WAYPOINT_EXAMPLE,
@@ -101,40 +100,45 @@ Note that:

 When using edit_diagram tool:
 - Use operations: update (modify cell by id), add (new cell), delete (remove cell by id)
- For update/add: provide cell_id and the complete new_xml in the same compact form (a shape with x, y, w, h; an edge with source and target)
+- For update/add: provide cell_id and complete new_xml (full mxCell element including mxGeometry)
 - For delete: only cell_id is needed
- Named styles are not available in edit_diagram: write each cell's complete style
 - Find the cell_id from "Current diagram XML" in system context
- Example update: {"operations": [{"operation": "update", "cell_id": "3", "new_xml": "<mxCell id=\\"3\\" value=\\"New Label\\" style=\\"rounded=1;\\" x=\\"100\\" y=\\"100\\" w=\\"120\\" h=\\"60\\"/>"}]}
+- Example update: {"operations": [{"operation": "update", "cell_id": "3", "new_xml": "<mxCell id=\\"3\\" value=\\"New Label\\" style=\\"rounded=1;\\" vertex=\\"1\\" parent=\\"1\\">\\n  <mxGeometry x=\\"100\\" y=\\"100\\" width=\\"120\\" height=\\"60\\" as=\\"geometry\\"/>\\n</mxCell>"}]}
 - Example delete: {"operations": [{"operation": "delete", "cell_id": "5"}]}
- Example add: {"operations": [{"operation": "add", "cell_id": "new1", "new_xml": "<mxCell id=\\"new1\\" value=\\"New Box\\" style=\\"rounded=1;\\" x=\\"400\\" y=\\"200\\" w=\\"120\\" h=\\"60\\"/>"}]}
+- Example add: {"operations": [{"operation": "add", "cell_id": "new1", "new_xml": "<mxCell id=\\"new1\\" value=\\"New Box\\" style=\\"rounded=1;\\" vertex=\\"1\\" parent=\\"1\\">\\n  <mxGeometry x=\\"400\\" y=\\"200\\" width=\\"120\\" height=\\"60\\" as=\\"geometry\\"/>\\n</mxCell>"}]}

 ⚠️ JSON ESCAPING: Every " inside new_xml MUST be escaped as \\". Example: id=\\"5\\" value=\\"Label\\"

 ## Draw.io XML Structure Reference

-**IMPORTANT:** You only generate the named styles and the mxCell elements. The wrapper structure and root cells (id="0", id="1") are added automatically. A named style is written before the cells as <mxStyle name="n" value="...style pairs..."/>; a cell uses it by putting the name among its style tokens (see Styles). A shape is one self-closing mxCell with x, y, w and h; an edge is one with source and target. vertex="1", edge="1", parent="1" and the mxGeometry element are added automatically, so write them only when needed: parent for a shape inside a container, an mxGeometry element for edge waypoints or a label placed on an edge.
+**IMPORTANT:** You only generate the mxCell elements. The wrapper structure and root cells (id="0", id="1") are added automatically.

 Example - generate ONLY this:
 \`\`\`xml
-<mxCell id="2" value="Label" style="rounded=1;" x="100" y="100" w="120" h="60"/>
+<mxCell id="2" value="Label" style="rounded=1;" vertex="1" parent="1">
+  <mxGeometry x="100" y="100" width="120" height="60" as="geometry"/>
+</mxCell>
 \`\`\`

 CRITICAL RULES:
-1. Generate ONLY mxStyle definitions and mxCell elements - NO wrapper tags (<mxfile>, <mxGraphModel>, <root>)
+1. Generate ONLY mxCell elements - NO wrapper tags (<mxfile>, <mxGraphModel>, <root>)
 2. Do NOT include root cells (id="0" or id="1") - they are added automatically
 3. ALL mxCell elements must be siblings - NEVER nest mxCell inside another mxCell
 4. Use unique sequential IDs starting from "2"
-5. Write parent="<container-id>" only for shapes inside a container; top-level cells need no parent
+5. Set parent="1" for top-level shapes, or parent="<container-id>" for grouped elements

 Shape (vertex) example:
 \`\`\`xml
-<mxCell id="2" value="Label" style="rounded=1;" x="100" y="100" w="120" h="60"/>
+<mxCell id="2" value="Label" style="rounded=1;whiteSpace=wrap;html=1;" vertex="1" parent="1">
+  <mxGeometry x="100" y="100" width="120" height="60" as="geometry"/>
+</mxCell>
 \`\`\`

 Connector (edge) example:
 \`\`\`xml
-<mxCell id="3" style="edgeStyle=orthogonalEdgeStyle;" source="2" target="4"/>
+<mxCell id="3" style="endArrow=classic;html=1;" edge="1" parent="1" source="2" target="4">
+  <mxGeometry relative="1" as="geometry"/>
+</mxCell>

 ### Edge Routing Rules:
 When creating edges/connectors, you MUST follow these rules to avoid overlapping lines:
@@ -149,7 +153,7 @@ When creating edges/connectors, you MUST follow these rules to avoid overlapping

 **Rule 3: Always specify exitX, exitY, entryX, entryY explicitly**
 - Every edge MUST have these 4 attributes set in the style
- Example: style="edgeStyle=orthogonalEdgeStyle;exitX=1;exitY=0.3;entryX=0;entryY=0.3;"
+- Example: style="edgeStyle=orthogonalEdgeStyle;exitX=1;exitY=0.3;entryX=0;entryY=0.3;endArrow=classic;"

 **Rule 4: Route edges AROUND intermediate shapes (obstacle avoidance) - CRITICAL!**
 - Before creating an edge, identify ALL shapes positioned between source and target
@@ -191,14 +195,10 @@ When creating edges/connectors, you MUST follow these rules to avoid overlapping

 // Style instructions - only included when minimalStyle is false
 const STYLE_INSTRUCTIONS = `
-## Styles
-Define each style used by several cells ONCE, as a named style before the cells, and use the name in the cells like a CSS class. A cell's style can combine a shape token, a name and overrides; later pairs win. Name only styles that two or more cells share; a style used by one cell stays inline. Names must not be draw.io's own style names: shapes such as text, ellipse, rhombus, swimlane, label, image, and colors such as blue, green, red, gray, yellow, orange, purple, pink. A definition applies to the call it is in: each display_diagram call defines the names it uses. The app expands the names, so the saved file is standard draw.io XML.
-\`\`\`xml
-${STYLE_CLASS_EXAMPLE}
-\`\`\`
- NEVER write html=1 or whiteSpace=wrap: the app adds html=1 to every cell and whiteSpace=wrap to shapes. Labels are HTML: use &lt;br&gt; for a line break and &lt;b&gt; for bold, never \\n; a literal < or > in a label is written &amp;lt; or &amp;gt;.
- Do NOT repeat what draw.io already uses. For a plain shape: rounded=0, align=center, verticalAlign=middle, fontSize=12, strokeWidth=1, fillColor=#ffffff, strokeColor=#000000, fontColor=#000000. For an edge: endArrow=classic, strokeColor=#000000. Writing one of them is right only when it overrides what a name or the shape sets: an edge is rounded by default, so rounded=0 on an edge is a real setting, and a text cell is left/top aligned by default, so there align=center or verticalAlign=middle are real settings.
- Keys: shapes rounded=1, fillColor=#hex, strokeColor=#hex; edges endArrow=block/open/none, startArrow=classic, curved=1, dashed=1, edgeStyle=orthogonalEdgeStyle; text fontSize=14, fontStyle=1 (bold), align=center/right.
+Common styles:
+- Shapes: rounded=1 (rounded corners), fillColor=#hex, strokeColor=#hex
+- Edges: endArrow=classic/block/open/none, startArrow=none/classic, curved=1, edgeStyle=orthogonalEdgeStyle
+- Text: fontSize=14, fontStyle=1 (bold), align=center/left/right
 `

 // Minimal style instruction - skip styling and focus on layout (prepended to prompt for emphasis)
@@ -208,13 +208,13 @@ const MINIMAL_STYLE_INSTRUCTION = `
 ### No Styling - Plain Black/White Only
 - NO fillColor, NO strokeColor, NO rounded, NO fontSize, NO fontStyle
 - NO color attributes (no hex colors like #ff69b4)
- Shapes: no style, or only the shape (ellipse, rhombus). Edges: edgeStyle=orthogonalEdgeStyle plus the exit/entry points from the Edge Routing Rules, nothing else. html=1 and whiteSpace=wrap are added automatically.
+- Style: "whiteSpace=wrap;html=1;" for shapes, "html=1;endArrow=classic;" for edges
 - IGNORE all color/style examples below

 ### Container/Group Shapes - MUST be Transparent
 - For container shapes (boxes that contain other shapes): use "fillColor=none;" to make background transparent
 - This prevents containers from covering child elements
- Example: style="fillColor=none;" for container rectangles
+- Example: style="whiteSpace=wrap;html=1;fillColor=none;" for container rectangles

 ### Focus on Layout Quality
 Since we skip styling, STRICTLY follow the "Edge Routing Rules" section below:
@@ -235,10 +235,10 @@ const EXTENDED_ADDITIONS = `
 ### display_diagram Details

 **VALIDATION RULES** (XML will be rejected if violated):
-1. Generate ONLY mxStyle definitions and mxCell elements - wrapper tags and root cells are added automatically
+1. Generate ONLY mxCell elements - wrapper tags and root cells are added automatically
 2. All mxCell elements must be siblings - never nested inside other mxCell elements
 3. Every mxCell needs a unique id attribute (start from "2")
-4. parent defaults to "1"; write it only for a shape inside a container (the container's id)
+4. Every mxCell needs a valid parent attribute (use "1" for top-level, or container-id for grouped)
 5. Edge source/target attributes must reference existing cell IDs
 6. Escape special characters in values: &lt; for <, &gt; for >, &amp; for &, &quot; for "

@@ -257,7 +257,7 @@ ${SWIMLANE_EXAMPLE}
 3. Complete the remaining mxCell elements
 4. If still truncated, call append_diagram again with the next fragment

-**Example:** If previous output ended with \`<mxCell id="x" style="rounded=1\`, continue with \`;" x="40" y="40" w="120" h="60"/>\` and complete the remaining elements.
+**Example:** If previous output ended with \`<mxCell id="x" style="rounded=1\`, continue with \`;" vertex="1">...\` and complete the remaining elements.

 ### edit_diagram Details

@@ -283,12 +283,12 @@ edit_diagram uses ID-based operations to modify cells directly by their id attri

 Change label:
 \`\`\`json
-{"operations": [{"operation": "update", "cell_id": "3", "new_xml": "<mxCell id=\\"3\\" value=\\"New Label\\" style=\\"rounded=1;\\" x=\\"100\\" y=\\"100\\" w=\\"120\\" h=\\"60\\"/>"}]}
+{"operations": [{"operation": "update", "cell_id": "3", "new_xml": "<mxCell id=\\"3\\" value=\\"New Label\\" style=\\"rounded=1;\\" vertex=\\"1\\" parent=\\"1\\">\\n  <mxGeometry x=\\"100\\" y=\\"100\\" width=\\"120\\" height=\\"60\\" as=\\"geometry\\"/>\\n</mxCell>"}]}
 \`\`\`

 Add new shape:
 \`\`\`json
-{"operations": [{"operation": "add", "cell_id": "new1", "new_xml": "<mxCell id=\\"new1\\" value=\\"New Box\\" style=\\"rounded=1;fillColor=#dae8fc;\\" x=\\"400\\" y=\\"200\\" w=\\"120\\" h=\\"60\\"/>"}]}
+{"operations": [{"operation": "add", "cell_id": "new1", "new_xml": "<mxCell id=\\"new1\\" value=\\"New Box\\" style=\\"rounded=1;fillColor=#dae8fc;\\" vertex=\\"1\\" parent=\\"1\\">\\n  <mxGeometry x=\\"400\\" y=\\"200\\" width=\\"120\\" height=\\"60\\" as=\\"geometry\\"/>\\n</mxCell>"}]}
 \`\`\`

 Delete container (children & edges auto-deleted):
@@ -312,7 +312,7 @@ ${TWO_EDGES_EXAMPLE}

 ### Edge with single waypoint (simple detour):
 \`\`\`xml
-<mxCell id="edge1" style="edgeStyle=orthogonalEdgeStyle;exitX=0.5;exitY=1;entryX=0.5;entryY=0;" edge="1" parent="1" source="a" target="b">
+<mxCell id="edge1" style="edgeStyle=orthogonalEdgeStyle;exitX=0.5;exitY=1;entryX=0.5;entryY=0;endArrow=classic;" edge="1" parent="1" source="a" target="b">
  <mxGeometry relative="1" as="geometry">
    <Array as="points">
      <mxPoint x="300" y="150"/>
@@ -2,7 +2,6 @@ import { type ClassValue, clsx } from "clsx"
 import * as pako from "pako"
 import { twMerge } from "tailwind-merge"
 import { hasCells } from "@/packages/mcp-server/src/pages.ts"
-import { readStyleClasses } from "@/packages/mcp-server/src/style-classes.ts"

 export function cn(...inputs: ClassValue[]) {
    return twMerge(clsx(inputs))
@@ -42,9 +41,7 @@ export function isRealDiagram(xml: string | undefined | null): boolean {
 * @returns true if XML appears complete, false if truncated or empty
 */
 export function isMxCellXmlComplete(xml: string | undefined | null): boolean {
-    // Named style definitions before the cells are not cells: output cut off
-    // right after them is incomplete
-    const trimmed = readStyleClasses(xml || "").xml.trim()
+    const trimmed = xml?.trim() || ""
    if (!trimmed) return false

    // Find position of last complete mxCell ending (either /> or </mxCell>)
@@ -1,12 +1,12 @@
 {
    "name": "next-ai-draw-io",
-    "version": "0.5.0",
+    "version": "0.4.16",
    "lockfileVersion": 3,
    "requires": true,
    "packages": {
        "": {
            "name": "next-ai-draw-io",
-            "version": "0.5.0",
+            "version": "0.4.16",
            "license": "Apache-2.0",
            "dependencies": {
                "@ai-sdk/amazon-bedrock": "^4.0.191",
@@ -1,6 +1,6 @@
 {
    "name": "next-ai-draw-io",
-    "version": "0.5.0",
+    "version": "0.4.16",
    "license": "Apache-2.0",
    "private": true,
    "main": "dist-electron/main/index.js",
@@ -1,166 +0,0 @@
-/**
- * Compact cells: the short way the model writes shapes and edges.
- *
- * A shape is one self-closing mxCell with its position and size as x, y, w
- * and h attributes; an edge is one with source and target. The fixed parts
- * of draw.io's XML are left out and filled in here:
- *
- *   <mxCell id="2" value="Start" style="step;" x="40" y="40" w="120" h="60"/>
- *   <mxCell id="5" style="down;" source="2" target="3"/>
- *
- * becomes
- *
- *   <mxCell id="2" value="Start" style="step;" vertex="1" parent="1">
- *     <mxGeometry x="40" y="40" width="120" height="60" as="geometry"/>
- *   </mxCell>
- *   <mxCell id="5" style="down;" edge="1" parent="1" source="2" target="3">
- *     <mxGeometry relative="1" as="geometry"/>
- *   </mxCell>
- *
- * The long form stays accepted, and anything beyond the plain case (edge
- * waypoints, a label on an edge, a parent other than "1") is written the
- * long way, so a cell may mix both: compact attributes with an explicit
- * parent, for example.
- *
- * foldCells is the reverse, for the diagram shown to the model: it writes
- * the plain shapes and edges compactly and leaves everything else as it is,
- * so the model reads the notation it is asked to write. Folding then
- * expanding gives the same cells back.
- */
-import { readAttributes, type TagAttribute } from "./xml-attributes.ts"
-
-// A whole mxCell: self-closing, or with its children (cells never nest)
-const CELL_BLOCK =
-    /<mxCell\b((?:[^<>"']|"[^"]*"|'[^']*')*?)\s*(?:\/>|>([\s\S]*?)<\/mxCell>)/g
-
-// The four compact attributes and the names draw.io uses for them
-const SIZE_ATTRS: Record<string, string> = {
-    x: "x",
-    y: "y",
-    w: "width",
-    h: "height",
-    width: "width",
-    height: "height",
-}
-
-const attributeText = (attrs: TagAttribute[]) =>
-    attrs.map((a) => ` ${a.name}="${a.value}"`).join("")
-
-// Attributes a cell is written with, in draw.io's usual order
-const FIRST = ["id", "value", "style"]
-const LAST = ["vertex", "edge", "parent", "source", "target"]
-
-/** Turn compact shapes and edges into standard draw.io cells */
-export function expandCompactCells(xml: string): string {
-    return xml.replace(CELL_BLOCK, (block, attrText: string, body?: string) => {
-        const attrs = readAttributes(attrText)
-        const byName = new Map(attrs.map((a) => [a.name, a.value]))
-        const size = attrs.filter((a) => a.name in SIZE_ATTRS)
-        const isEdge =
-            byName.get("edge") === "1" ||
-            (size.length === 0 &&
-                byName.get("vertex") !== "1" &&
-                byName.has("source") &&
-                byName.has("target"))
-        const isVertex =
-            !isEdge && (byName.get("vertex") === "1" || size.length > 0)
-        // Root cells and anything else the model wrote in full are left alone
-        if (!isEdge && !isVertex) return block
-        const inner = body ?? ""
-        const hasGeometry = /<mxGeometry\b/.test(inner)
-        const complete =
-            size.length === 0 &&
-            byName.has(isEdge ? "edge" : "vertex") &&
-            byName.has("parent") &&
-            hasGeometry
-        if (complete) return block
-
-        // Rebuild the attributes in the usual order: id, value, style, the
-        // rest, then the flags and the connections
-        const rest = attrs.filter(
-            (a) =>
-                !FIRST.includes(a.name) &&
-                !LAST.includes(a.name) &&
-                !(a.name in SIZE_ATTRS),
-        )
-        const ordered = [
-            ...FIRST.flatMap((n) =>
-                byName.has(n) ? [[n, byName.get(n) ?? ""]] : [],
-            ),
-            ...rest.map((a) => [a.name, a.value]),
-            [isEdge ? "edge" : "vertex", "1"],
-            ["parent", byName.get("parent") ?? "1"],
-            ...["source", "target"].flatMap((n) =>
-                byName.has(n) ? [[n, byName.get(n) ?? ""]] : [],
-            ),
-        ]
-        const written = ordered.map(([n, v]) => ` ${n}="${v}"`).join("")
-        let geometry = ""
-        if (!hasGeometry) {
-            if (isEdge) {
-                geometry = '<mxGeometry relative="1" as="geometry"/>'
-            } else {
-                const get = (short: string, long: string, fallback: string) =>
-                    byName.get(short) ?? byName.get(long) ?? fallback
-                geometry = `<mxGeometry x="${get("x", "x", "0")}" y="${get("y", "y", "0")}" width="${get("w", "width", "120")}" height="${get("h", "height", "60")}" as="geometry"/>`
-            }
-        }
-        return `<mxCell${written}>${geometry}${inner}</mxCell>`
-    })
-}
-
-/** Attributes of a single self-closing mxGeometry, or null for anything else */
-function plainGeometry(body: string): Map<string, string> | null {
-    const trimmed = body.trim()
-    const m = trimmed.match(
-        /^<mxGeometry\b((?:[^<>"']|"[^"]*"|'[^']*')*?)\s*\/>$/,
-    )
-    if (!m) return null
-    const attrs = new Map(readAttributes(m[1]).map((a) => [a.name, a.value]))
-    // Anything odd, such as a wrong "as", stays as written so folding and
-    // expanding give the same cells back
-    return attrs.get("as") === "geometry" ? attrs : null
-}
-
-/**
- * Write plain shapes and edges compactly, for the diagram shown to the
- * model. A shape is plain when its only child is an mxGeometry with x, y,
- * width and height; an edge when its only child is the relative geometry.
- * Everything else, including wrapped cells' extra data and edge labels,
- * is left as written.
- */
-export function foldCells(xml: string): string {
-    return xml.replace(CELL_BLOCK, (block, attrText: string, body?: string) => {
-        if (body === undefined) return block
-        const attrs = readAttributes(attrText)
-        const byName = new Map(attrs.map((a) => [a.name, a.value]))
-        if (attrs.some((a) => a.name in SIZE_ATTRS)) return block
-        const geometry = plainGeometry(body)
-        if (!geometry) return block
-        const geometryKeys = [...geometry.keys()]
-            .filter((k) => k !== "as")
-            .sort()
-        const kept = attrs.filter(
-            (a) =>
-                a.name !== "vertex" &&
-                a.name !== "edge" &&
-                !(a.name === "parent" && a.value === "1"),
-        )
-        if (byName.get("vertex") === "1" && byName.get("edge") !== "1") {
-            if (geometryKeys.join() !== "height,width,x,y") return block
-            const size = ` x="${geometry.get("x")}" y="${geometry.get("y")}" w="${geometry.get("width")}" h="${geometry.get("height")}"`
-            return `<mxCell${attributeText(kept)}${size}/>`
-        }
-        if (byName.get("edge") === "1" && byName.get("vertex") !== "1") {
-            if (
-                geometryKeys.join() !== "relative" ||
-                geometry.get("relative") !== "1"
-            ) {
-                return block
-            }
-            if (!byName.has("source") || !byName.has("target")) return block
-            return `<mxCell${attributeText(kept)}/>`
-        }
-        return block
-    })
-}
@@ -9,7 +9,6 @@

 import {
    indent,
-    STYLE_CLASS_EXAMPLE,
    SWIMLANE_EXAMPLE,
    TWO_EDGES_EXAMPLE,
    WAYPOINT_EXAMPLE,
@@ -27,17 +26,21 @@ export const DRAWING_GUIDE = `# Draw.io drawing guide
 - The preview page has History (it saves a snapshot before every AI change and can restore any of the last 20 versions) and Download. You can make changes freely; nothing is lost.

 ## The XML you send
-Single page (create_new_diagram, add_page): send ONLY the named styles and the mxCell elements. The server adds <mxfile>, <mxGraphModel>, <root> and the root cells id="0" and id="1", expands named styles (see Styles), adds html=1 to every cell and whiteSpace=wrap to shapes, and fills in vertex, edge, parent="1" and the mxGeometry element. A shape is one self-closing mxCell with x, y, w and h; an edge is one with source and target. Write parent only for a shape inside a container, and an mxGeometry element only for edge waypoints or a label placed on an edge.
+Single page (create_new_diagram, add_page): send ONLY the mxCell elements. The server adds <mxfile>, <mxGraphModel>, <root> and the root cells id="0" and id="1".

-    <mxCell id="2" value="Label" style="rounded=1;" x="100" y="100" w="120" h="60"/>
-    <mxCell id="3" style="edgeStyle=orthogonalEdgeStyle;exitX=1;exitY=0.5;entryX=0;entryY=0.5;" source="2" target="4"/>
+    <mxCell id="2" value="Label" style="rounded=1;whiteSpace=wrap;html=1;" vertex="1" parent="1">
+      <mxGeometry x="100" y="100" width="120" height="60" as="geometry"/>
+    </mxCell>
+    <mxCell id="3" style="edgeStyle=orthogonalEdgeStyle;exitX=1;exitY=0.5;entryX=0;entryY=0.5;endArrow=classic;html=1;" edge="1" parent="1" source="2" target="4">
+      <mxGeometry relative="1" as="geometry"/>
+    </mxCell>

 Several pages at once (create_new_diagram only): send a full <mxfile> with one <diagram id="..." name="..."> per page. Every page's <root> must start with <mxCell id="0"/><mxCell id="1" parent="0"/>.

 Rules (XML that breaks them is rejected):
 1. All mxCell elements are siblings. NEVER nest an mxCell inside another mxCell.
 2. Ids are unique within a page and start from "2" ("0" and "1" are the root cells).
-3. parent="<container id>" for shapes inside a container; top-level cells need no parent.
+3. parent="1" for top-level shapes, parent="<container id>" for shapes inside a container.
 4. Edge source and target must reference existing cell ids.
 5. Escape special characters in attribute values: &lt; for <, &gt; for >, &amp; for &, &quot; for ".
 6. NEVER include XML comments (<!-- -->). draw.io strips them.
@@ -56,7 +59,7 @@ ${indent(SWIMLANE_EXAMPLE)}
 ## Edge routing rules
 Rule 1: Never let two edges share a path. Two edges between the same nodes exit and enter at different points (exitY=0.3 for the first, exitY=0.7 for the second, not both 0.5).
 Rule 2: For bidirectional connections (A to B and B to A), use opposite sides: A exits right (exitX=1) into the left of B (entryX=0); B exits left (exitX=0) into the right of A (entryX=1).
-Rule 3: Always set exitX, exitY, entryX and entryY in the edge style, e.g. style="edgeStyle=orthogonalEdgeStyle;exitX=1;exitY=0.3;entryX=0;entryY=0.3;".
+Rule 3: Always set exitX, exitY, entryX and entryY in the edge style, e.g. style="edgeStyle=orthogonalEdgeStyle;exitX=1;exitY=0.3;entryX=0;entryY=0.3;endArrow=classic;".
 Rule 4: Route edges AROUND shapes in the way. Before drawing an edge, find every shape between source and target; if one is in the path, add waypoints. Route diagonal connections along the outside of the diagram, not through the middle. Keep 20-30px clearance from shapes. An edge must never cross another shape's box.
 Rule 5: Plan the layout first. Organize shapes into rows or columns following the flow, space them 150-200px apart so edges have room, and prefer one flow direction (left to right or top to bottom).
 Rule 6: Use 2-3 waypoints for L-shaped or U-shaped paths. Each change of direction needs a waypoint, and segments should be horizontal or vertical.
@@ -77,33 +80,27 @@ Waypoints go inside <Array as="points"> in the edge geometry. Example: Hotfix (r
 ${indent(WAYPOINT_EXAMPLE)}

 ## Styles
-Define each style used by several cells ONCE, as a named style before the cells, and use the name in the cells like a CSS class. A cell's style can combine a shape token, a name and overrides; later pairs win. Name only styles that two or more cells share; a style used by one cell stays inline. Names must not be draw.io's own style names: shapes such as text, ellipse, rhombus, swimlane, label, image, and colors such as blue, green, red, gray, yellow, orange, purple, pink. A definition applies to the call it is in: each create_new_diagram or add_page call defines the names it uses. The server expands the names, so the saved file is standard draw.io XML.
-
-${indent(STYLE_CLASS_EXAMPLE)}
-
- NEVER write html=1 or whiteSpace=wrap: the server adds html=1 to every cell and whiteSpace=wrap to shapes. Labels are HTML: use &lt;br&gt; for a line break and &lt;b&gt; for bold, never \\n; a literal < or > in a label is written &amp;lt; or &amp;gt;.
- Do NOT repeat what draw.io already uses. For a plain shape: rounded=0, align=center, verticalAlign=middle, fontSize=12, strokeWidth=1, fillColor=#ffffff, strokeColor=#000000, fontColor=#000000. For an edge: endArrow=classic, strokeColor=#000000. Writing one of them is right only when it overrides what a name or the shape sets: an edge is rounded by default, so rounded=0 on an edge is a real setting, and a text cell is left/top aligned by default, so there align=center or verticalAlign=middle are real settings.
- Shapes: rounded=1, fillColor=#hex, strokeColor=#hex
- Edges: endArrow=block, open or none; startArrow=classic; curved=1; dashed=1; edgeStyle=orthogonalEdgeStyle
- Text: fontSize=14, fontStyle=1 (bold), align=center or right
+- Shapes: rounded=1, fillColor=#hex, strokeColor=#hex, whiteSpace=wrap;html=1;
+- Edges: endArrow=classic, block, open or none; startArrow=none or classic; curved=1; edgeStyle=orthogonalEdgeStyle
+- Text: fontSize=14, fontStyle=1 (bold), align=center, left or right
 - Animated connectors: add flowAnimation=1 to the edge style.

 ## Minimal style
 When the user asks for a minimal, plain, black-and-white or unstyled diagram, use these rules instead of the styles above:
 - No fillColor, strokeColor, rounded, fontSize, fontStyle or hex colors.
- Shapes have no style, or only the shape (ellipse, rhombus); edges have edgeStyle=orthogonalEdgeStyle plus the exit and entry points from the routing rules, nothing else.
- Containers that hold other shapes use "fillColor=none;" so they do not cover their children.
+- Shapes use style "whiteSpace=wrap;html=1;", edges use "html=1;endArrow=classic;".
+- Containers that hold other shapes use "whiteSpace=wrap;html=1;fillColor=none;" so they do not cover their children.
 - Keep at least 50px between elements, and follow all edge routing rules strictly.

 ## Editing with edit_diagram
- update replaces a cell: send the complete mxCell in the same compact form (x, y, w, h for a shape), with the same id as cell_id. Named styles are not available here: write each cell's complete style.
+- update replaces a cell: send the complete mxCell including mxGeometry, with the same id as cell_id.
 - add inserts a new cell with a new id. One cell per operation.
 - delete removes a cell. Its children and every edge connected to it are deleted too, so give only the container's id.
 - All-or-nothing: if any operation fails, nothing is applied. A rejected call includes the current XML of the page; rebuild your operations on it and retry.
 - If the diagram is large, change it with edit_diagram instead of redrawing it.

-    {"operations": [{"operation": "update", "cell_id": "3", "new_xml": "<mxCell id=\\"3\\" value=\\"New Label\\" style=\\"rounded=1;\\" x=\\"100\\" y=\\"100\\" w=\\"120\\" h=\\"60\\"/>"}]}
-    {"page_name": "CNN", "operations": [{"operation": "add", "cell_id": "conv-1", "new_xml": "<mxCell id=\\"conv-1\\" value=\\"Conv\\" x=\\"40\\" y=\\"40\\" w=\\"120\\" h=\\"60\\"/>"}]}
+    {"operations": [{"operation": "update", "cell_id": "3", "new_xml": "<mxCell id=\\"3\\" value=\\"New Label\\" style=\\"rounded=1;\\" vertex=\\"1\\" parent=\\"1\\"><mxGeometry x=\\"100\\" y=\\"100\\" width=\\"120\\" height=\\"60\\" as=\\"geometry\\"/></mxCell>"}]}
+    {"page_name": "CNN", "operations": [{"operation": "add", "cell_id": "conv-1", "new_xml": "<mxCell id=\\"conv-1\\" value=\\"Conv\\" vertex=\\"1\\" parent=\\"1\\"><mxGeometry x=\\"40\\" y=\\"40\\" width=\\"120\\" height=\\"60\\" as=\\"geometry\\"/></mxCell>"}]}
    {"page_index": 1, "operations": [{"operation": "delete", "cell_id": "5"}]}

 Pages: list_pages shows every page's id, name and index. edit_diagram, get_diagram and export_diagram take an optional page_id, page_name or page_index; without one they use the first page.
@@ -7,13 +7,11 @@
 * so the model never builds on a half-applied edit.
 */

-import { expandCompactCells } from "./compact-cells.ts"
 import {
    applyDiagramOperations,
    type DiagramOperation,
 } from "./diagram-operations.ts"
 import { type PageSelector, projectPage } from "./pages.ts"
-import { addDefaultStyles, edgeIdsOf } from "./style-classes.ts"
 import { validateAndFixXml, validateMxCellStructure } from "./xml-validation.ts"

 export type EditOutcome =
@@ -50,19 +48,11 @@ export function editDiagram(
    const fixes: string[] = []
    const prepared: DiagramOperation[] = []

-    // Edges already on the page, so a label added to one is not wrapped
-    const edges = edgeIdsOf(xml)
    for (const op of operations) {
        if (op.operation === "delete" || !op.new_xml) {
            prepared.push(op)
            continue
        }
-        if (/<mxStyle\b/i.test(op.new_xml)) {
-            errors.push(
-                `${op.operation} ${op.cell_id}: named styles (mxStyle) are not available in edit_diagram; write the cell's complete style`,
-            )
-            continue
-        }
        // Checked before validation: several cells fail the strict parser
        // with a misleading "only one root" syntax error.
        if (countTopLevelElements(op.new_xml) > 1) {
@@ -81,15 +71,7 @@ export function editDiagram(
        if (check.fixed) {
            fixes.push(`${op.cell_id}: ${check.fixes.join(", ")}`)
        }
-        // The defaults the model is told not to write (style-classes.ts),
-        // added after the fixes so a repaired cell gets them too
-        prepared.push({
-            ...op,
-            new_xml: addDefaultStyles(
-                expandCompactCells(check.fixed ?? op.new_xml),
-                edges,
-            ),
-        })
+        prepared.push({ ...op, new_xml: check.fixed ?? op.new_xml })
    }
    if (errors.length > 0) return { ok: false, errors, pageError: false }

@@ -23,7 +23,6 @@ import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js"
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js"
 import open from "open"
 import { z } from "zod"
-import { expandCompactCells, foldCells } from "./compact-cells.ts"
 import type { DiagramOperation } from "./diagram-operations.ts"
 import { installDomPolyfill } from "./dom.ts"
 import { DRAWING_GUIDE } from "./drawing-guide.ts"
@@ -49,11 +48,7 @@ import {
 } from "./http-server.ts"
 import { parseDrawioFileContent } from "./load-diagram.ts"
 import { log } from "./logger.ts"
-import {
-    prepareNewDiagram,
-    reservedIdError,
-    takeStyleDefinitions,
-} from "./new-diagram.ts"
+import { prepareNewDiagram, reservedIdError } from "./new-diagram.ts"
 import {
    addPageToDoc,
    deletePageFromDoc,
@@ -71,7 +66,6 @@ import {
 } from "./pages.ts"
 import { Autosaver, defaultDataDir, expandHome } from "./persistence.ts"
 import { getShapeLibrary, SHAPE_LIBRARY_LIST } from "./shape-library.ts"
-import { addDefaultStyles, applyStyleClasses } from "./style-classes.ts"
 import { validateAndFixXml } from "./xml-validation.ts"

 // DOMParser/XMLSerializer globals for the XML helpers (Node has neither)
@@ -342,11 +336,11 @@ Before using icon shapes (AWS, Azure, GCP, Kubernetes, Cisco...), call get_shape

 Accepted xml:
 1) Only the mxCell elements of one page (recommended). The server adds <mxfile>, <mxGraphModel>, <root> and the root cells "0" and "1":
-<mxCell id="2" value="Shape" style="rounded=1;" x="40" y="40" w="120" h="60"/>
+<mxCell id="2" value="Shape" style="rounded=1;whiteSpace=wrap;html=1;" vertex="1" parent="1"><mxGeometry x="40" y="40" width="120" height="60" as="geometry"/></mxCell>
 2) A bare <mxGraphModel> with <root> (one page).
 3) A full <mxfile> with one or more <diagram> pages. Every page's <root> must start with <mxCell id="0"/><mxCell id="1" parent="0"/>.

-Rules: cells are siblings (never nested), ids are unique per page and start from "2", parent="1" for top-level shapes, no XML comments, and shapes stay within x 0 to 800 and y 0 to 600. A style used by several cells is defined once with <mxStyle name="..." value="..."/> before the cells and used by name (see the drawing guide); html=1 and whiteSpace=wrap are added automatically.`,
+Rules: cells are siblings (never nested), ids are unique per page and start from "2", parent="1" for top-level shapes, no XML comments, and shapes stay within x 0 to 800 and y 0 to 600.`,
        inputSchema: {
            xml: z
                .string()
@@ -595,10 +589,10 @@ registerWriteTool(
            "- add: Add a new cell. Provide cell_id (new unique id within the page) and new_xml. One cell per operation.\n" +
            "- update: Replace an existing cell by its id. Provide cell_id and complete new_xml.\n" +
            "- delete: Remove a cell by its id. Only cell_id is needed. Its children and connected edges are deleted too, so give only a container's id.\n\n" +
-            "For add/update, new_xml is the complete mxCell in the compact form (a shape with x, y, w, h; an edge with source and target). No XML comments. " +
+            "For add/update, new_xml must be a complete mxCell element including mxGeometry. No XML comments. " +
            'Every " inside new_xml must be escaped as \\" in the JSON.\n\n' +
            "Example - Add a rectangle on the default (first) page:\n" +
-            '{"operations": [{"operation": "add", "cell_id": "rect-1", "new_xml": "<mxCell id=\\"rect-1\\" value=\\"Hello\\" style=\\"rounded=1;\\" x=\\"100\\" y=\\"100\\" w=\\"120\\" h=\\"60\\"/>"}]}\n\n' +
+            '{"operations": [{"operation": "add", "cell_id": "rect-1", "new_xml": "<mxCell id=\\"rect-1\\" value=\\"Hello\\" style=\\"rounded=0;\\" vertex=\\"1\\" parent=\\"1\\"><mxGeometry x=\\"100\\" y=\\"100\\" width=\\"120\\" height=\\"60\\" as=\\"geometry\\"/></mxCell>"}]}\n\n' +
            "Example - Delete a cell on the default page:\n" +
            '{"operations": [{"operation": "delete", "cell_id": "rect-1"}]}',
        inputSchema: {
@@ -875,7 +869,7 @@ server.registerTool(
                    content: [
                        {
                            type: "text",
-                            text: `Current diagram XML:\n\n${foldCells(session.xml)}\n\n${pageList}${staleNote}`,
+                            text: `Current diagram XML:\n\n${session.xml}\n\n${pageList}${staleNote}`,
                        },
                    ],
                }
@@ -1593,28 +1587,14 @@ registerWriteTool(

            // If caller provided XML, validate it before splicing it in so we
            // never get a half-broken mxfile written to the session.
-            // Named style definitions come out first and are expanded on the
-            // validated XML, like prepareNewDiagram
-            const {
-                classes,
-                xml: startXml,
-                error: styleError,
-            } = takeStyleDefinitions(xml ?? "")
-            if (styleError) {
-                return {
-                    content: [{ type: "text", text: `Error: ${styleError}` }],
-                    isError: true,
-                }
-            }
-            const reserved = startXml && reservedIdError(startXml)
+            const reserved = xml && reservedIdError(xml)
            if (reserved) {
                return {
                    content: [{ type: "text", text: `Error: ${reserved}` }],
                    isError: true,
                }
            }
-            let cleanXml: string | undefined =
-                startXml && wrapCellsInModel(startXml)
+            let cleanXml: string | undefined = xml && wrapCellsInModel(xml)
            if (cleanXml) {
                const { valid, error, fixed, fixes } =
                    validateAndFixXml(cleanXml)
@@ -1635,9 +1615,6 @@ registerWriteTool(
                        isError: true,
                    }
                }
-                cleanXml = addDefaultStyles(
-                    applyStyleClasses(expandCompactCells(cleanXml), classes),
-                )
            }

            let info
@@ -2,14 +2,7 @@
 * A whole new diagram written by the model, for the create_new_diagram tool
 * and the web app's display_diagram tool.
 */
-import { expandCompactCells } from "./compact-cells.ts"
-import { hasCells, normalizeToMxfile, wrapCellsInModel } from "./pages.ts"
-import {
-    addDefaultStyles,
-    applyStyleClasses,
-    readStyleClasses,
-    type StyleClasses,
-} from "./style-classes.ts"
+import { normalizeToMxfile, wrapCellsInModel } from "./pages.ts"
 import { readAttributes } from "./xml-attributes.ts"
 import { validateAndFixXml } from "./xml-validation.ts"

@@ -49,55 +42,23 @@ export function reservedIdError(input: string): string | null {
 }

 /**
- * The named style definitions are taken out first (style-classes.ts). Bare
- * cells then get the wrapper and root cells, since the strict parser rejects
- * several top-level elements. Then the XML is validated and auto-fixed while
- * it is still a bare model, where duplicate ids are renamed. Compact cells
- * (compact-cells.ts) are expanded, the names expanded and the default styles
- * added on the fixed XML, so repaired cells get them too, and finally it is
- * turned into an <mxfile>.
+ * Bare cells get the wrapper and root cells first, since the strict parser
+ * rejects several top-level elements. Then the XML is validated and
+ * auto-fixed while it is still a bare model, where duplicate ids are
+ * renamed, and finally turned into an <mxfile>.
 */
-/**
- * Take the named style definitions out of the model's XML (style-classes.ts).
- * Returns the error for the model when a definition never closed or when
- * nothing but definitions was sent.
- */
-export function takeStyleDefinitions(input: string): {
-    classes: StyleClasses
-    xml: string
-    error: string | null
-} {
-    const { classes, xml } = readStyleClasses(input)
-    let error: string | null = null
-    if (/<mxStyle\b/i.test(xml)) {
-        error =
-            'A named style definition is not closed. Write it as <mxStyle name="..." value="..."/> before the cells.'
-    } else if (classes.size > 0 && !hasCells(xml)) {
-        error =
-            "Only named style definitions were sent, no cells. Send the mxCell elements after the definitions."
-    }
-    return { classes, xml, error }
-}
-
 export function prepareNewDiagram(
    input: string,
    page: { pageId?: string; pageName?: string } = {},
 ): NewDiagram {
-    const {
-        classes,
-        xml: cells,
-        error: styleError,
-    } = takeStyleDefinitions(input)
-    if (styleError) return { ok: false, error: styleError }
-    const reserved = reservedIdError(cells)
+    const reserved = reservedIdError(input)
    if (reserved) return { ok: false, error: reserved }
-    let xml = wrapCellsInModel(cells)
+    let xml = wrapCellsInModel(input)
    const { valid, error, fixed, fixes } = validateAndFixXml(xml)
    if (fixed) xml = fixed
    if (!valid) {
        return { ok: false, error: `XML validation failed - ${error}` }
    }
-    xml = addDefaultStyles(applyStyleClasses(expandCompactCells(xml), classes))
    const normalized = normalizeToMxfile(xml, page)
    if (!normalized) {
        return {
@@ -1,232 +0,0 @@
-/**
- * Named styles, written once and used by name like CSS classes.
- *
- * The model defines a style before the cells and refers to it from any
- * number of cells, so a style string is not repeated in every cell:
- *
- *   <mxStyle name="blue" value="fillColor=#dae8fc;strokeColor=#6c8ebf;"/>
- *   <mxCell id="2" value="A" style="rounded=1;blue;" vertex="1" parent="1">...</mxCell>
- *   <mxCell id="3" value="B" style="rhombus;blue;fontSize=14;" vertex="1" parent="1">...</mxCell>
- *
- * draw.io resolves a style token without "=" through its stylesheet, which
- * is not saved in the file, so the names are expanded here and the saved
- * XML is plain draw.io XML. A name without a definition is left as it is:
- * draw.io ignores it, unless its own stylesheet knows the name (text,
- * ellipse, label, blue, green, ...), which then keeps its meaning.
- *
- * html=1 and whiteSpace=wrap, which draw.io itself puts on every new shape,
- * are added here too, so the model never has to write them.
- *
- * The callers validate and auto-fix the XML first and rewrite the styles
- * after, so a repaired cell (quotes written as entities, a lowercase tag)
- * gets its defaults too, and the rewrite only ever sees proper attributes.
- */
-import { readAttributes } from "./xml-attributes.ts"
-
-export type StyleClasses = Map<string, string>
-
-// <mxStyle name="..." value="..."/> with the whitespace after it, also when
-// written as <mxStyle ...> or <mxStyle ...></mxStyle>; quoted values may hold ">"
-const STYLE_DEFINITION =
-    /<mxStyle\b((?:[^<>"']|"[^"]*"|'[^']*')*?)\s*\/?>(?:\s*<\/mxStyle>)?\s*/gi
-
-// An mxCell opening tag; the attributes stop before a closing "/" so a
-// self-closing cell keeps its "/>"
-const CELL_TAG = /<mxCell\b((?:[^<>"']|"[^"]*"|'[^']*')*?)\s*(\/?)>/g
-
-// draw.io reads keys and tokens as written, " html" is not "html", so the
-// checks are exact too
-/** Whether the style sets the key (as `key=` at the start or after a ";") */
-const hasKey = (style: string, key: string) =>
-    new RegExp(`(^|;)${key}=`).test(style)
-
-/** Whether the style holds this exact token */
-const hasToken = (style: string, token: string) =>
-    style.split(";").includes(token)
-
-/**
- * Replace the names among the tokens with their definitions. The tokens are
- * kept exactly as written, empty ones included: a leading ";" is draw.io's
- * "no default style" marker, and an entity such as &quot; ends in ";" too.
- * A name whose definition is empty is dropped, so it does not leave such a
- * marker behind.
- */
-function applyToStyle(style: string, classes: StyleClasses): string {
-    const out: string[] = []
-    for (const token of style.split(";")) {
-        const definition = token.includes("=")
-            ? undefined
-            : classes.get(token.trim())
-        if (definition === undefined) out.push(token)
-        else if (definition !== "") out.push(definition)
-    }
-    return out.join(";")
-}
-
-/**
- * A definition's value goes into a style attribute unchecked by the XML
- * validator (the definitions are taken out before it runs), so the two
- * characters that would break the attribute are escaped here, the way the
- * validator repairs them in cells.
- */
-function escapeForAttribute(value: string): string {
-    return value
-        .replace(
-            /&(?!(?:lt|gt|amp|quot|apos|#[0-9]+|#x[0-9a-fA-F]+);)/g,
-            "&amp;",
-        )
-        .replace(/</g, "&lt;")
-}
-
-/**
- * The definitions, resolved against each other, and the XML without them.
- * A definition may use names defined before or after it; a name using
- * itself stays a bare token.
- */
-export function readStyleClasses(xml: string): {
-    classes: StyleClasses
-    xml: string
-} {
-    const classes: StyleClasses = new Map()
-    const rest = xml.replace(STYLE_DEFINITION, (_match, attrText: string) => {
-        const attrs = new Map(
-            readAttributes(attrText).map((a) => [a.name, a.value]),
-        )
-        // "name" may end in the ";" the model is used to; "style" is the
-        // attribute the model may reach for instead of "value"
-        const name = attrs.get("name")?.trim().replace(/;+$/, "")
-        const value = attrs.get("value") ?? attrs.get("style")
-        if (name && value !== undefined) {
-            // Without the trailing ";", so a name followed by ";" in a cell
-            // expands to "...;" and not to "...;;"
-            classes.set(name, escapeForAttribute(value).replace(/;+$/, ""))
-        }
-        return ""
-    })
-    // Two passes resolve names used by other definitions, in either order
-    for (let pass = 0; pass < 2; pass++) {
-        for (const [name, value] of classes) {
-            const others = new Map(classes)
-            others.delete(name)
-            classes.set(name, applyToStyle(value, others))
-        }
-    }
-    return { classes, xml: rest }
-}
-
-/**
- * Rewrite the style of every mxCell. The callback returns the new style, or
- * undefined to leave the cell alone. A cell without a style attribute gets
- * one when the callback returns a style for it.
- */
-function rewriteCellStyles(
-    xml: string,
-    rewrite: (
-        style: string | undefined,
-        attrs: Map<string, string>,
-    ) => string | undefined,
-): string {
-    return xml.replace(CELL_TAG, (tag, attrText: string, selfClose: string) => {
-        const attributes = readAttributes(attrText)
-        const attrs = new Map(attributes.map((a) => [a.name, a.value]))
-        const style = attributes.find((a) => a.name === "style")
-        const next = rewrite(style?.value, attrs)
-        if (next === undefined || next === style?.value) return tag
-        // The value is copied from attributes as written, so only a quote
-        // from a single-quoted source needs escaping
-        const written = `style="${next.replace(/"/g, "&quot;")}"`
-        if (!style) {
-            return `<mxCell${attrText} ${written}${selfClose}>`
-        }
-        // Keep the attribute where it was, with the whitespace before it
-        const space =
-            attrText.slice(style.start, style.end).match(/^\s*/)?.[0] ?? " "
-        return `<mxCell${attrText.slice(0, style.start)}${space}${written}${attrText.slice(style.end)}${selfClose}>`
-    })
-}
-
-/** Expand the names in every cell's style; the overrides after a name still win */
-export function applyStyleClasses(xml: string, classes: StyleClasses): string {
-    if (classes.size === 0) return xml
-    return rewriteCellStyles(xml, (style) =>
-        style === undefined ? undefined : applyToStyle(style, classes),
-    )
-}
-
-/** The ids of the edges in the XML, whose child vertices are edge labels */
-export function edgeIdsOf(xml: string): Set<string> {
-    const ids = new Set<string>()
-    for (const [, attrText] of xml.matchAll(CELL_TAG)) {
-        const attrs = new Map(
-            readAttributes(attrText).map((a) => [a.name, a.value]),
-        )
-        const id = attrs.get("id")
-        if (id && attrs.get("edge") === "1") ids.add(id)
-    }
-    return ids
-}
-
-/**
- * Whether the label sits outside the shape, as with library icons (their
- * styles position the label below) and images, so wrapping it to the
- * shape's width would be wrong.
- */
-function hasLabelOutside(style: string): boolean {
-    return (
-        hasKey(style, "verticalLabelPosition") ||
-        hasKey(style, "labelPosition") ||
-        hasKey(style, "image") ||
-        hasToken(style, "shape=image")
-    )
-}
-
-/**
- * draw.io's own defaults for a new shape: html=1 (so labels may hold <br>
- * and <b>) and whiteSpace=wrap. Wrapping is not added to shapes whose label
- * sits outside (icons, images), to text that sizes itself (autosize=1), to
- * edge labels (their label box has no width of its own) or next to html=0
- * (draw.io renders a wrapping label as HTML). Edges get html=1. Root cells
- * and cells that already set a key are left alone.
- *
- * An edit adds cells to a page the XML does not show, so the caller passes
- * the ids of the edges already on it (edgeIdsOf), or new edge labels would
- * be wrapped like shapes.
- */
-export function addDefaultStyles(
-    xml: string,
-    knownEdgeIds: Iterable<string> = [],
-): string {
-    const edges = new Set([...knownEdgeIds, ...edgeIdsOf(xml)])
-    return rewriteCellStyles(xml, (style, attrs) => {
-        // A cell marked as both is drawn as an edge
-        const isEdge = attrs.get("edge") === "1"
-        const isVertex = attrs.get("vertex") === "1" && !isEdge
-        if (!isVertex && !isEdge) return undefined
-        const current = style ?? ""
-        const isEdgeLabel =
-            hasToken(current, "edgeLabel") ||
-            attrs.get("connectable") === "0" ||
-            edges.has(attrs.get("parent") ?? "")
-        const added: string[] = []
-        if (
-            isVertex &&
-            !hasKey(current, "whiteSpace") &&
-            !hasLabelOutside(current) &&
-            !isEdgeLabel &&
-            !hasToken(current, "autosize=1") &&
-            !hasToken(current, "html=0")
-        ) {
-            added.push("whiteSpace=wrap")
-        }
-        if (!hasKey(current, "html")) added.push("html=1")
-        if (added.length === 0) return undefined
-        const separator = current === "" || current.endsWith(";") ? "" : ";"
-        return `${current}${separator}${added.join(";")};`
-    })
-}
-
-/** Expand the named styles of a model's XML and add the default styles */
-export function expandStyles(xml: string): string {
-    const { classes, xml: cells } = readStyleClasses(xml)
-    return addDefaultStyles(applyStyleClasses(cells, classes))
-}
@@ -3,16 +3,30 @@
 * prompt and tool descriptions, so both teach the model the same thing.
 */

-export const SWIMLANE_EXAMPLE = `<mxCell id="lane1" value="Frontend" style="swimlane;" x="40" y="40" w="200" h="200"/>
-<mxCell id="step1" value="Step 1" style="rounded=1;" parent="lane1" x="20" y="60" w="160" h="40"/>
-<mxCell id="lane2" value="Backend" style="swimlane;" x="280" y="40" w="200" h="200"/>
-<mxCell id="step2" value="Step 2" style="rounded=1;" parent="lane2" x="20" y="60" w="160" h="40"/>
-<mxCell id="edge1" style="edgeStyle=orthogonalEdgeStyle;" source="step1" target="step2"/>`
+export const SWIMLANE_EXAMPLE = `<mxCell id="lane1" value="Frontend" style="swimlane;" vertex="1" parent="1">
+  <mxGeometry x="40" y="40" width="200" height="200" as="geometry"/>
+</mxCell>
+<mxCell id="step1" value="Step 1" style="rounded=1;" vertex="1" parent="lane1">
+  <mxGeometry x="20" y="60" width="160" height="40" as="geometry"/>
+</mxCell>
+<mxCell id="lane2" value="Backend" style="swimlane;" vertex="1" parent="1">
+  <mxGeometry x="280" y="40" width="200" height="200" as="geometry"/>
+</mxCell>
+<mxCell id="step2" value="Step 2" style="rounded=1;" vertex="1" parent="lane2">
+  <mxGeometry x="20" y="60" width="160" height="40" as="geometry"/>
+</mxCell>
+<mxCell id="edge1" style="edgeStyle=orthogonalEdgeStyle;endArrow=classic;" edge="1" parent="1" source="step1" target="step2">
+  <mxGeometry relative="1" as="geometry"/>
+</mxCell>`

-export const TWO_EDGES_EXAMPLE = `<mxCell id="e1" value="A to B" style="edgeStyle=orthogonalEdgeStyle;exitX=1;exitY=0.3;entryX=0;entryY=0.3;" source="a" target="b"/>
-<mxCell id="e2" value="B to A" style="edgeStyle=orthogonalEdgeStyle;exitX=0;exitY=0.7;entryX=1;entryY=0.7;" source="b" target="a"/>`
+export const TWO_EDGES_EXAMPLE = `<mxCell id="e1" value="A to B" style="edgeStyle=orthogonalEdgeStyle;exitX=1;exitY=0.3;entryX=0;entryY=0.3;endArrow=classic;" edge="1" parent="1" source="a" target="b">
+  <mxGeometry relative="1" as="geometry"/>
+</mxCell>
+<mxCell id="e2" value="B to A" style="edgeStyle=orthogonalEdgeStyle;exitX=0;exitY=0.7;entryX=1;entryY=0.7;endArrow=classic;" edge="1" parent="1" source="b" target="a">
+  <mxGeometry relative="1" as="geometry"/>
+</mxCell>`

-export const WAYPOINT_EXAMPLE = `<mxCell id="hotfix_to_main" style="edgeStyle=orthogonalEdgeStyle;exitX=0.5;exitY=0;entryX=1;entryY=0.5;" edge="1" parent="1" source="hotfix" target="main">
+export const WAYPOINT_EXAMPLE = `<mxCell id="hotfix_to_main" style="edgeStyle=orthogonalEdgeStyle;exitX=0.5;exitY=0;entryX=1;entryY=0.5;endArrow=classic;" edge="1" parent="1" source="hotfix" target="main">
  <mxGeometry relative="1" as="geometry">
    <Array as="points">
      <mxPoint x="750" y="80"/>
@@ -21,15 +35,6 @@ export const WAYPOINT_EXAMPLE = `<mxCell id="hotfix_to_main" style="edgeStyle=or
  </mxGeometry>
 </mxCell>`

-/** Named styles defined once and used by name, like CSS classes */
-export const STYLE_CLASS_EXAMPLE = `<mxStyle name="step" value="fillColor=#dae8fc;strokeColor=#6c8ebf;"/>
-<mxStyle name="down" value="edgeStyle=orthogonalEdgeStyle;exitX=0.5;exitY=1;entryX=0.5;entryY=0;"/>
-<mxCell id="2" value="Start" style="rounded=1;step;" x="40" y="40" w="120" h="60"/>
-<mxCell id="3" value="Check" style="rhombus;step;fontStyle=1;" x="40" y="160" w="120" h="80"/>
-<mxCell id="4" value="Done" style="rounded=1;step;" x="40" y="300" w="120" h="60"/>
-<mxCell id="5" style="down;" source="2" target="3"/>
-<mxCell id="6" style="down;" source="3" target="4"/>`
-
 /** Indent every line, for an indented code block in Markdown */
 export const indent = (text: string, prefix = "    ") =>
    text
@@ -657,15 +657,6 @@ export function autoFixXml(xml: string): { fixed: string; fixes: string[] } {
        fixes.push("Fixed malformed attribute quotes")
    }

-    // 6b. Missing closing quote on a numeric attribute right before the
-    // self-closing slash (h="112/>), which a compact cell ends with. Numbers
-    // only, so a quote that belongs to a label is never touched.
-    const quoteBeforeSlash = /=("-?\d+(?:\.\d+)?)\/>/g
-    if (quoteBeforeSlash.test(fixed)) {
-        fixed = fixed.replace(quoteBeforeSlash, '=$1"/>')
-        fixes.push("Added a missing closing quote before />")
-    }
-
    // 7. Fix malformed closing tags
    const malformedClosingTag = /<\/([a-zA-Z][a-zA-Z0-9]*)\s*\/>/g
    if (malformedClosingTag.test(fixed)) {
@@ -1,193 +0,0 @@
-import { describe, expect, it } from "vitest"
-import {
-    expandCompactCells,
-    foldCells,
-} from "@/packages/mcp-server/src/compact-cells.ts"
-import { editDiagram } from "@/packages/mcp-server/src/edit-diagram.ts"
-import { prepareNewDiagram } from "@/packages/mcp-server/src/new-diagram.ts"
-import {
-    normalizeToMxfile,
-    wrapCellsInModel,
-} from "@/packages/mcp-server/src/pages.ts"
-
-const shape =
-    '<mxCell id="2" value="Start" style="rounded=1;" x="40" y="40" w="120" h="60"/>'
-const shapeLong =
-    '<mxCell id="2" value="Start" style="rounded=1;" vertex="1" parent="1"><mxGeometry x="40" y="40" width="120" height="60" as="geometry"/></mxCell>'
-const edge = '<mxCell id="5" style="down;" source="2" target="3"/>'
-const edgeLong =
-    '<mxCell id="5" style="down;" edge="1" parent="1" source="2" target="3"><mxGeometry relative="1" as="geometry"/></mxCell>'
-
-/** Cells compared by what they mean, not by attribute order */
-function canonical(xml: string): string {
-    const doc = new DOMParser().parseFromString(
-        `<root>${xml}</root>`,
-        "text/xml",
-    )
-    const describe = (el: Element): string => {
-        const attrs = Array.from(el.attributes)
-            .map((a) => `${a.name}=${a.value}`)
-            .sort()
-            .join(" ")
-        const children = Array.from(el.children).map(describe).join("")
-        return `<${el.tagName} ${attrs}>${children}</${el.tagName}>`
-    }
-    return Array.from(doc.documentElement.children).map(describe).join("\n")
-}
-
-describe("expandCompactCells", () => {
-    it("gives a compact shape its flags, parent and geometry", () => {
-        expect(expandCompactCells(shape)).toBe(shapeLong)
-    })
-
-    it("gives a compact edge its flags, parent and relative geometry", () => {
-        expect(expandCompactCells(edge)).toBe(edgeLong)
-    })
-
-    it("keeps an explicit parent and accepts width/height as names", () => {
-        const inLane =
-            '<mxCell id="3" value="Step" style="rounded=1;" parent="lane1" x="20" y="60" width="160" height="40"/>'
-        expect(expandCompactCells(inLane)).toBe(
-            '<mxCell id="3" value="Step" style="rounded=1;" vertex="1" parent="lane1"><mxGeometry x="20" y="60" width="160" height="40" as="geometry"/></mxCell>',
-        )
-    })
-
-    it("fills in a default position and size when some are missing", () => {
-        expect(expandCompactCells('<mxCell id="2" value="A" w="100"/>')).toBe(
-            '<mxCell id="2" value="A" vertex="1" parent="1"><mxGeometry x="0" y="0" width="100" height="60" as="geometry"/></mxCell>',
-        )
-    })
-
-    it("lets an explicit geometry win over compact attributes", () => {
-        const both =
-            '<mxCell id="2" x="1" y="1" w="1" h="1" vertex="1" parent="1"><mxGeometry x="40" y="40" width="120" height="60" as="geometry"/></mxCell>'
-        expect(expandCompactCells(both)).toBe(
-            shapeLong.replace(' value="Start" style="rounded=1;"', ""),
-        )
-    })
-
-    it("adds the relative geometry to a long-form edge that has none", () => {
-        const noGeometry =
-            '<mxCell id="5" style="down;" edge="1" parent="1" source="2" target="3"/>'
-        expect(expandCompactCells(noGeometry)).toBe(edgeLong)
-    })
-
-    it("leaves root cells, long-form cells and edge labels alone", () => {
-        const roots = '<mxCell id="0"/><mxCell id="1" parent="0"/>'
-        const label =
-            '<mxCell id="9" value="yes" style="edgeLabel;" vertex="1" connectable="0" parent="5"><mxGeometry x="-0.5" relative="1" as="geometry"/></mxCell>'
-        expect(expandCompactCells(roots + shapeLong + edgeLong + label)).toBe(
-            roots + shapeLong + edgeLong + label,
-        )
-    })
-
-    it("works on a cell inside a UserObject wrapper", () => {
-        const wrapped =
-            '<UserObject id="4" label="Docs" link="https://example.com"><mxCell style="rounded=1;" x="580" y="40" w="120" h="60"/></UserObject>'
-        expect(expandCompactCells(wrapped)).toBe(
-            '<UserObject id="4" label="Docs" link="https://example.com"><mxCell style="rounded=1;" vertex="1" parent="1"><mxGeometry x="580" y="40" width="120" height="60" as="geometry"/></mxCell></UserObject>',
-        )
-    })
-})
-
-describe("foldCells", () => {
-    it("writes plain shapes and edges compactly", () => {
-        expect(foldCells(shapeLong)).toBe(shape)
-        expect(foldCells(edgeLong)).toBe(edge)
-    })
-
-    it("keeps a parent other than 1 and leaves special cells as written", () => {
-        const inLane =
-            '<mxCell id="3" value="Step" style="rounded=1;" vertex="1" parent="lane1"><mxGeometry x="20" y="60" width="160" height="40" as="geometry"/></mxCell>'
-        expect(foldCells(inLane)).toBe(
-            '<mxCell id="3" value="Step" style="rounded=1;" parent="lane1" x="20" y="60" w="160" h="40"/>',
-        )
-        const waypoints =
-            '<mxCell id="6" edge="1" parent="1" source="2" target="3"><mxGeometry relative="1" as="geometry"><Array as="points"><mxPoint x="750" y="80"/></Array></mxGeometry></mxCell>'
-        const label =
-            '<mxCell id="9" value="yes" style="edgeLabel;" vertex="1" connectable="0" parent="5"><mxGeometry x="-0.5" relative="1" as="geometry"/></mxCell>'
-        const sourcePoint =
-            '<mxCell id="7" edge="1" parent="1" target="3"><mxGeometry relative="1" as="geometry"><mxPoint x="10" y="10" as="sourcePoint"/></mxGeometry></mxCell>'
-        const roots = '<mxCell id="0"/><mxCell id="1" parent="0"/>'
-        const wrongAs =
-            '<mxCell id="8" vertex="1" parent="1"><mxGeometry x="1" y="1" width="2" height="2" as="33"/></mxCell>'
-        for (const xml of [waypoints, label, sourcePoint, roots, wrongAs]) {
-            expect(foldCells(xml)).toBe(xml)
-        }
-    })
-
-    it("round-trips through expandCompactCells", () => {
-        const page = `<mxGraphModel><root><mxCell id="0"/><mxCell id="1" parent="0"/>${shapeLong}${edgeLong}<mxCell id="lane1" value="Lane" style="swimlane;" vertex="1" parent="1"><mxGeometry x="40" y="40" width="200" height="200" as="geometry"/></mxCell><mxCell id="3" value="Step" style="rounded=1;" vertex="1" parent="lane1"><mxGeometry x="20" y="60" width="160" height="40" as="geometry"/></mxCell><mxCell id="6" edge="1" parent="1" source="2" target="3"><mxGeometry relative="1" as="geometry"><Array as="points"><mxPoint x="750" y="80"/></Array></mxGeometry></mxCell></root></mxGraphModel>`
-        const folded = foldCells(page)
-        expect(folded).not.toContain("vertex=")
-        expect(canonical(expandCompactCells(folded))).toBe(canonical(page))
-    })
-})
-
-describe("compact cells through the pipeline", () => {
-    const geometryOf = (xml: string, id: string) =>
-        xml.match(
-            new RegExp(`<mxCell id="${id}"[^>]*>(<mxGeometry[^>]*/>)`),
-        )?.[1]
-
-    it("prepareNewDiagram expands them and still adds the defaults", () => {
-        const prepared = prepareNewDiagram(`${shape}\n${edge}`)
-        expect(prepared.ok).toBe(true)
-        if (!prepared.ok) return
-        expect(prepared.xml).toContain(
-            'style="rounded=1;whiteSpace=wrap;html=1;" vertex="1" parent="1"',
-        )
-        expect(geometryOf(prepared.xml, "2")).toBe(
-            '<mxGeometry x="40" y="40" width="120" height="60" as="geometry"/>',
-        )
-        expect(prepared.xml).toContain(
-            '<mxCell id="5" style="down;html=1;" edge="1" parent="1" source="2" target="3"><mxGeometry relative="1" as="geometry"/></mxCell>',
-        )
-    })
-
-    it("editDiagram accepts a compact cell in new_xml", () => {
-        const file =
-            normalizeToMxfile(wrapCellsInModel(shapeLong), {
-                pageId: "p1",
-                pageName: "Page-1",
-            }) ?? ""
-        const outcome = editDiagram(
-            file,
-            [
-                {
-                    operation: "add",
-                    cell_id: "3",
-                    new_xml:
-                        '<mxCell id="3" value="Next" style="rounded=1;" x="200" y="40" w="120" h="60"/>',
-                },
-                {
-                    operation: "update",
-                    cell_id: "2",
-                    new_xml:
-                        '<mxCell id="2" value="Begin" style="rounded=1;" x="40" y="40" w="120" h="60"/>',
-                },
-            ],
-            {},
-        )
-        expect(outcome.ok).toBe(true)
-        if (!outcome.ok) return
-        expect(outcome.xml).toContain('value="Next"')
-        expect(outcome.xml).toContain('value="Begin"')
-        expect(geometryOf(outcome.xml, "3")).toBe(
-            '<mxGeometry x="200" y="40" width="120" height="60" as="geometry"/>',
-        )
-    })
-})
-
-describe("a compact cell with a slip", () => {
-    it("repairs a missing closing quote before the slash", () => {
-        const prepared = prepareNewDiagram(
-            '<mxCell id="12" value="Coin Flip" style="rounded=1;" x="535" y="276" w="225" h="112/>',
-        )
-        expect(prepared.ok).toBe(true)
-        if (!prepared.ok) return
-        expect(prepared.xml).toContain(
-            '<mxGeometry x="535" y="276" width="225" height="112" as="geometry"/>',
-        )
-    })
-})
@@ -1,499 +0,0 @@
-import { describe, expect, it } from "vitest"
-import { isMxCellXmlComplete } from "@/lib/utils"
-import { editDiagram } from "@/packages/mcp-server/src/edit-diagram.ts"
-import { prepareNewDiagram } from "@/packages/mcp-server/src/new-diagram.ts"
-import {
-    normalizeToMxfile,
-    wrapCellsInModel,
-} from "@/packages/mcp-server/src/pages.ts"
-import {
-    addDefaultStyles,
-    applyStyleClasses,
-    expandStyles,
-    readStyleClasses,
-} from "@/packages/mcp-server/src/style-classes.ts"
-
-const geometry =
-    '<mxGeometry x="0" y="0" width="80" height="40" as="geometry"/>'
-const cell = (id: string, style: string, extra = "") =>
-    `<mxCell id="${id}" value="${id}" style="${style}" vertex="1" parent="1"${extra}>${geometry}</mxCell>`
-const edge = (id: string, style: string) =>
-    `<mxCell id="${id}" style="${style}" edge="1" parent="1" source="2" target="3"><mxGeometry relative="1" as="geometry"/></mxCell>`
-
-const BLUE =
-    '<mxStyle name="blue" value="fillColor=#dae8fc;strokeColor=#6c8ebf;"/>'
-
-describe("readStyleClasses", () => {
-    it("reads the definitions and removes them from the XML", () => {
-        const xml = `${BLUE}\n<mxStyle name="flow" value="edgeStyle=orthogonalEdgeStyle;"></mxStyle>\n${cell("2", "blue;")}`
-        const { classes, xml: rest } = readStyleClasses(xml)
-        expect(classes.get("blue")).toBe(
-            "fillColor=#dae8fc;strokeColor=#6c8ebf",
-        )
-        expect(classes.get("flow")).toBe("edgeStyle=orthogonalEdgeStyle")
-        expect(rest).toBe(cell("2", "blue;"))
-    })
-
-    it("resolves a definition that uses an earlier name", () => {
-        const xml = `${BLUE}<mxStyle name="pill" value="rounded=1;blue;arcSize=50;"/>`
-        const { classes } = readStyleClasses(xml)
-        expect(classes.get("pill")).toBe(
-            "rounded=1;fillColor=#dae8fc;strokeColor=#6c8ebf;arcSize=50",
-        )
-    })
-
-    it("leaves a definition that is still streaming in place", () => {
-        const xml = `${BLUE}<mxStyle name="flow" value="edgeSt`
-        const { classes, xml: rest } = readStyleClasses(xml)
-        expect(classes.size).toBe(1)
-        expect(rest).toBe('<mxStyle name="flow" value="edgeSt')
-    })
-})
-
-describe("applyStyleClasses", () => {
-    const classes = readStyleClasses(BLUE).classes
-
-    it("expands a name in place, so the overrides after it still win", () => {
-        const out = applyStyleClasses(
-            cell("2", "rhombus;blue;fontSize=14;"),
-            classes,
-        )
-        expect(out).toBe(
-            cell(
-                "2",
-                "rhombus;fillColor=#dae8fc;strokeColor=#6c8ebf;fontSize=14;",
-            ),
-        )
-    })
-
-    it("leaves unknown names, key=value tokens and other attributes alone", () => {
-        const xml = `${cell("2", "text;fontSize=blue;")}<mxCell id="3" vertex="1" parent="1" value="blue">${geometry}</mxCell>`
-        expect(applyStyleClasses(xml, classes)).toBe(xml)
-    })
-
-    it("works inside a wrapped model and keeps the attribute order", () => {
-        const xml = `<mxGraphModel><root><mxCell id="0"/><mxCell id="1" parent="0"/><mxCell id="2" style="blue;" value="A" vertex="1" parent="1">${geometry}</mxCell></root></mxGraphModel>`
-        expect(applyStyleClasses(xml, classes)).toBe(
-            xml.replace(
-                'style="blue;"',
-                'style="fillColor=#dae8fc;strokeColor=#6c8ebf;"',
-            ),
-        )
-    })
-})
-
-describe("addDefaultStyles", () => {
-    it("adds whiteSpace=wrap and html=1 to a shape", () => {
-        expect(addDefaultStyles(cell("2", "rounded=1;"))).toBe(
-            cell("2", "rounded=1;whiteSpace=wrap;html=1;"),
-        )
-    })
-
-    it("gives a shape without a style attribute one", () => {
-        const xml = `<mxCell id="2" value="A" vertex="1" parent="1">${geometry}</mxCell>`
-        expect(addDefaultStyles(xml)).toBe(
-            `<mxCell id="2" value="A" vertex="1" parent="1" style="whiteSpace=wrap;html=1;">${geometry}</mxCell>`,
-        )
-    })
-
-    it("keeps html=0 and an existing whiteSpace", () => {
-        const xml = cell("2", "html=0;whiteSpace=nowrap;")
-        expect(addDefaultStyles(xml)).toBe(xml)
-    })
-
-    it("adds only html=1 to icon shapes, images and edge labels", () => {
-        const icon = cell(
-            "2",
-            "shape=mxgraph.aws4.resourceIcon;resIcon=mxgraph.aws4.ec2;verticalLabelPosition=bottom;",
-        )
-        const image = cell("3", "image=data:image/png,abc;")
-        const label = cell("4", "edgeLabel;align=center;")
-        const unconnectable = cell("5", "align=center;", ' connectable="0"')
-        expect(addDefaultStyles(icon)).toBe(
-            cell(
-                "2",
-                "shape=mxgraph.aws4.resourceIcon;resIcon=mxgraph.aws4.ec2;verticalLabelPosition=bottom;html=1;",
-            ),
-        )
-        expect(addDefaultStyles(image)).toBe(
-            cell("3", "image=data:image/png,abc;html=1;"),
-        )
-        expect(addDefaultStyles(label)).toBe(
-            cell("4", "edgeLabel;align=center;html=1;"),
-        )
-        expect(addDefaultStyles(unconnectable)).toBe(
-            cell("5", "align=center;html=1;", ' connectable="0"'),
-        )
-    })
-
-    it("adds html=1 to edges and leaves the root cells alone", () => {
-        const roots = '<mxCell id="0"/><mxCell id="1" parent="0"/>'
-        expect(addDefaultStyles(roots + edge("4", "endArrow=block;"))).toBe(
-            roots + edge("4", "endArrow=block;html=1;"),
-        )
-    })
-})
-
-describe("expandStyles", () => {
-    it("expands the names, then adds the defaults", () => {
-        const xml = `${BLUE}${cell("2", "blue;")}`
-        expect(expandStyles(xml)).toBe(
-            cell(
-                "2",
-                "fillColor=#dae8fc;strokeColor=#6c8ebf;whiteSpace=wrap;html=1;",
-            ),
-        )
-    })
-})
-
-describe("prepareNewDiagram with named styles", () => {
-    it("saves plain draw.io XML without the definitions", () => {
-        const xml = `${BLUE}\n<mxStyle name="flow" value="edgeStyle=orthogonalEdgeStyle;exitX=0.5;exitY=1;entryX=0.5;entryY=0;"/>\n${cell("2", "blue;")}\n${cell("3", "rhombus;blue;")}\n${edge("4", "flow;")}`
-        const prepared = prepareNewDiagram(xml, { pageId: "p1" })
-        expect(prepared.ok).toBe(true)
-        if (!prepared.ok) return
-        expect(prepared.xml).not.toContain("mxStyle")
-        expect(prepared.xml).toContain(
-            'style="fillColor=#dae8fc;strokeColor=#6c8ebf;whiteSpace=wrap;html=1;"',
-        )
-        expect(prepared.xml).toContain(
-            'style="rhombus;fillColor=#dae8fc;strokeColor=#6c8ebf;whiteSpace=wrap;html=1;"',
-        )
-        expect(prepared.xml).toContain(
-            'style="edgeStyle=orthogonalEdgeStyle;exitX=0.5;exitY=1;entryX=0.5;entryY=0;html=1;"',
-        )
-    })
-})
-
-describe("editDiagram adds the default styles", () => {
-    it("adds html=1 and whiteSpace=wrap to an added cell", () => {
-        const file =
-            normalizeToMxfile(wrapCellsInModel(cell("2", "")), {
-                pageId: "p1",
-                pageName: "Page-1",
-            }) ?? ""
-        const outcome = editDiagram(
-            file,
-            [
-                {
-                    operation: "add",
-                    cell_id: "3",
-                    new_xml: cell("3", "rounded=1;"),
-                },
-            ],
-            {},
-        )
-        expect(outcome.ok).toBe(true)
-        if (!outcome.ok) return
-        expect(outcome.xml).toContain(
-            'style="rounded=1;whiteSpace=wrap;html=1;"',
-        )
-    })
-})
-
-describe("cells written in other shapes", () => {
-    it("keeps a self-closing cell without a style self-closing", () => {
-        const edge =
-            '<mxCell id="4" edge="1" parent="1" source="2" target="3"/>'
-        expect(addDefaultStyles(edge)).toBe(
-            '<mxCell id="4" edge="1" parent="1" source="2" target="3" style="html=1;"/>',
-        )
-        const shape = '<mxCell id="5" value="A" vertex="1" parent="1" />'
-        expect(addDefaultStyles(shape)).toBe(
-            '<mxCell id="5" value="A" vertex="1" parent="1" style="whiteSpace=wrap;html=1;"/>',
-        )
-    })
-
-    it("expands a name in a cell wrapped in a UserObject", () => {
-        const xml = `${BLUE}<UserObject id="4" label="Docs" link="https://example.com"><mxCell style="blue;" vertex="1" parent="1">${geometry}</mxCell></UserObject>`
-        expect(expandStyles(xml)).toBe(
-            `<UserObject id="4" label="Docs" link="https://example.com"><mxCell style="fillColor=#dae8fc;strokeColor=#6c8ebf;whiteSpace=wrap;html=1;" vertex="1" parent="1">${geometry}</mxCell></UserObject>`,
-        )
-    })
-
-    it("leaves text that sizes itself unwrapped", () => {
-        const xml = cell("2", "text;autosize=1;")
-        expect(addDefaultStyles(xml)).toBe(cell("2", "text;autosize=1;html=1;"))
-    })
-
-    it("reads a definition written without the closing slash", () => {
-        const { classes, xml } = readStyleClasses(
-            `<mxStyle name="a" value="x=1">\n${cell("2", "a;")}`,
-        )
-        expect(classes.get("a")).toBe("x=1")
-        expect(xml).toBe(cell("2", "a;"))
-    })
-
-    it("adds the defaults inside a full mxfile", () => {
-        const file = `<mxfile><diagram id="p" name="Page-1"><mxGraphModel><root><mxCell id="0"/><mxCell id="1" parent="0"/>${cell("2", "rounded=1;")}</root></mxGraphModel></diagram></mxfile>`
-        const prepared = prepareNewDiagram(file)
-        expect(prepared.ok).toBe(true)
-        if (!prepared.ok) return
-        expect(prepared.xml).toContain(
-            '<mxCell id="0"/><mxCell id="1" parent="0"/>',
-        )
-        expect(prepared.xml).toContain(
-            'style="rounded=1;whiteSpace=wrap;html=1;"',
-        )
-    })
-})
-
-describe("truncation check with named styles", () => {
-    it("treats output cut off right after the definitions as incomplete", () => {
-        expect(isMxCellXmlComplete(BLUE)).toBe(false)
-        expect(
-            isMxCellXmlComplete(`${BLUE}<mxStyle name="flow" value="edgeSt`),
-        ).toBe(false)
-        expect(isMxCellXmlComplete(`${BLUE}${cell("2", "blue;")}`)).toBe(true)
-    })
-})
-
-describe("styles are rewritten as written", () => {
-    it("keeps entities, a leading semicolon and a missing trailing one", () => {
-        const quoted = cell(
-            "2",
-            "fontFamily=&quot;Times New Roman&quot;;fillColor=red;",
-        )
-        expect(addDefaultStyles(quoted)).toBe(
-            cell(
-                "2",
-                "fontFamily=&quot;Times New Roman&quot;;fillColor=red;whiteSpace=wrap;html=1;",
-            ),
-        )
-        const noDefaults = cell("3", ";shape=rectangle;strokeColor=#000000")
-        expect(addDefaultStyles(noDefaults)).toBe(
-            cell(
-                "3",
-                ";shape=rectangle;strokeColor=#000000;whiteSpace=wrap;html=1;",
-            ),
-        )
-        const classes = readStyleClasses(BLUE).classes
-        expect(applyStyleClasses(cell("4", "blue"), classes)).toBe(
-            cell("4", "fillColor=#dae8fc;strokeColor=#6c8ebf"),
-        )
-    })
-
-    it("does not add wrapping next to html=0, since wrapping turns HTML on", () => {
-        const xml = cell("2", "html=0;")
-        expect(addDefaultStyles(xml)).toBe(xml)
-    })
-
-    it("wraps ordinary shape= shapes and leaves library icons alone", () => {
-        expect(
-            addDefaultStyles(
-                cell(
-                    "2",
-                    "shape=parallelogram;perimeter=parallelogramPerimeter;",
-                ),
-            ),
-        ).toBe(
-            cell(
-                "2",
-                "shape=parallelogram;perimeter=parallelogramPerimeter;whiteSpace=wrap;html=1;",
-            ),
-        )
-        expect(
-            addDefaultStyles(
-                cell(
-                    "3",
-                    "shape=mxgraph.cisco19.router;verticalLabelPosition=bottom;",
-                ),
-            ),
-        ).toBe(
-            cell(
-                "3",
-                "shape=mxgraph.cisco19.router;verticalLabelPosition=bottom;html=1;",
-            ),
-        )
-        // Library shapes whose label sits inside wrap like any shape
-        expect(
-            addDefaultStyles(cell("5", "shape=mxgraph.flowchart.process;")),
-        ).toBe(
-            cell(
-                "5",
-                "shape=mxgraph.flowchart.process;whiteSpace=wrap;html=1;",
-            ),
-        )
-        expect(addDefaultStyles(cell("4", "shape=image;image=a.png;"))).toBe(
-            cell("4", "shape=image;image=a.png;html=1;"),
-        )
-    })
-
-    it("escapes a quote that a single-quoted definition brings in", () => {
-        const xml = `<mxStyle name="a" value='fontFamily="Arial";'/>${cell("2", "a;")}`
-        expect(expandStyles(xml)).toBe(
-            cell("2", "fontFamily=&quot;Arial&quot;;whiteSpace=wrap;html=1;"),
-        )
-    })
-
-    it("resolves references in either order; duplicates and built-in names", () => {
-        const { classes } = readStyleClasses(
-            `<mxStyle name="b" value="c;x=1;"/><mxStyle name="c" value="y=2;"/><mxStyle name="c" value="y=3;"/><mxStyle name="loop" value="loop;z=1;"/>`,
-        )
-        // A name defined later still resolves; the last definition wins
-        expect(classes.get("b")).toBe("y=3;x=1")
-        // A name using itself stays a bare token
-        expect(classes.get("loop")).toBe("loop;z=1")
-        // The last definition of a name wins
-        expect(classes.get("c")).toBe("y=3")
-        // A definition named like a built-in replaces that built-in's meaning
-        const shadow = readStyleClasses(
-            '<mxStyle name="ellipse" value="fillColor=red;"/>',
-        ).classes
-        expect(applyStyleClasses(cell("2", "ellipse;"), shadow)).toBe(
-            cell("2", "fillColor=red;"),
-        )
-    })
-})
-
-describe("the auto-fix runs before the defaults", () => {
-    it("repairs quotes written as entities in an edit and then adds the defaults", () => {
-        const file = normalizeToMxfile(wrapCellsInModel(cell("2", "")), {
-            pageId: "p1",
-            pageName: "Page-1",
-        })
-        const outcome = editDiagram(
-            file ?? "",
-            [
-                {
-                    operation: "add",
-                    cell_id: "3",
-                    new_xml: `<mxCell id="3" value="B" style=&quot;rounded=1;&quot; vertex="1" parent="1">${geometry}</mxCell>`,
-                },
-            ],
-            {},
-        )
-        expect(outcome.ok).toBe(true)
-        if (!outcome.ok) return
-        expect(outcome.xml).toContain(
-            'style="rounded=1;whiteSpace=wrap;html=1;"',
-        )
-    })
-
-    it("repairs a new diagram the same way", () => {
-        const prepared = prepareNewDiagram(
-            `${BLUE}<mxCell id="2" value="A" style=&quot;blue;&quot; vertex="1" parent="1">${geometry}</mxCell>`,
-        )
-        expect(prepared.ok).toBe(true)
-        if (!prepared.ok) return
-        expect(prepared.xml).toContain(
-            'style="fillColor=#dae8fc;strokeColor=#6c8ebf;whiteSpace=wrap;html=1;"',
-        )
-    })
-})
-
-describe("definitions and cells the model may write imperfectly", () => {
-    it("escapes & and < in a definition, which the validator never sees", () => {
-        const xml = `<mxStyle name="f" value="fontFamily=A&B <C;"/>${cell("2", "f;")}`
-        const prepared = prepareNewDiagram(xml)
-        expect(prepared.ok).toBe(true)
-        if (!prepared.ok) return
-        expect(prepared.xml).toContain(
-            'style="fontFamily=A&amp;B &lt;C;whiteSpace=wrap;html=1;"',
-        )
-    })
-
-    it("accepts a lowercase tag, a style attribute and a name ending in ;", () => {
-        const { classes, xml } = readStyleClasses(
-            `<mxstyle name="blue;" style="fillColor=red;"/>${cell("2", "blue;")}`,
-        )
-        expect(classes.get("blue")).toBe("fillColor=red")
-        expect(xml).toBe(cell("2", "blue;"))
-    })
-
-    it("reads keys exactly, like draw.io, so a key after a space does not count", () => {
-        const xml = cell("2", "rounded=1; html=1; whiteSpace=wrap;")
-        expect(addDefaultStyles(xml)).toBe(
-            cell(
-                "2",
-                "rounded=1; html=1; whiteSpace=wrap;whiteSpace=wrap;html=1;",
-            ),
-        )
-        const classes = readStyleClasses(BLUE).classes
-        expect(
-            applyStyleClasses(
-                cell("3", "rounded=1; blue; fontSize=14;"),
-                classes,
-            ),
-        ).toBe(
-            cell(
-                "3",
-                "rounded=1;fillColor=#dae8fc;strokeColor=#6c8ebf; fontSize=14;",
-            ),
-        )
-    })
-
-    it("treats a vertex whose parent is an edge as an edge label", () => {
-        const xml = `${edge("e1", "")}<mxCell id="lbl" value="yes" style="align=center;" vertex="1" parent="e1"><mxGeometry relative="1" as="geometry"/></mxCell>`
-        expect(addDefaultStyles(xml)).toContain('style="align=center;html=1;"')
-    })
-
-    it("reports a definition that never closed and definitions without cells", () => {
-        const open = prepareNewDiagram(
-            `${BLUE}<mxStyle name="flow" value="edgeSt`,
-        )
-        expect(open.ok).toBe(false)
-        if (!open.ok) expect(open.error).toContain("not closed")
-        const only = prepareNewDiagram(BLUE)
-        expect(only.ok).toBe(false)
-        if (!only.ok) expect(only.error).toContain("no cells")
-    })
-})
-
-describe("edge labels and empty definitions", () => {
-    it("does not wrap a label added to an edge the XML does not show", () => {
-        const file =
-            normalizeToMxfile(
-                wrapCellsInModel(cell("2", "") + cell("3", "") + edge("4", "")),
-                {
-                    pageId: "p1",
-                    pageName: "Page-1",
-                },
-            ) ?? ""
-        const label = `<mxCell id="9" value="yes" style="text;" vertex="1" parent="4"><mxGeometry x="-0.5" relative="1" as="geometry"/></mxCell>`
-        const outcome = editDiagram(
-            file,
-            [{ operation: "add", cell_id: "9", new_xml: label }],
-            {},
-        )
-        expect(outcome.ok).toBe(true)
-        if (!outcome.ok) return
-        expect(outcome.xml).toContain('id="9" value="yes" style="text;html=1;"')
-        // The same through the function the preview uses
-        expect(addDefaultStyles(label, ["4"])).toContain('style="text;html=1;"')
-    })
-
-    it("treats a cell marked edge and vertex as an edge", () => {
-        const both = `<mxCell id="5" style="rounded=1;" edge="1" vertex="1" parent="1" source="2" target="3"><mxGeometry relative="1" as="geometry"/></mxCell>`
-        expect(addDefaultStyles(both)).toContain('style="rounded=1;html=1;"')
-    })
-
-    it("drops a name whose definition is empty instead of leaving a leading ;", () => {
-        const xml = `<mxStyle name="plain" value=""/>${cell("2", "plain;rounded=1;")}`
-        expect(expandStyles(xml)).toBe(
-            cell("2", "rounded=1;whiteSpace=wrap;html=1;"),
-        )
-    })
-
-    it("tells the model that edit_diagram has no named styles", () => {
-        const file =
-            normalizeToMxfile(wrapCellsInModel(cell("2", "")), {
-                pageId: "p1",
-                pageName: "Page-1",
-            }) ?? ""
-        const outcome = editDiagram(
-            file,
-            [
-                {
-                    operation: "add",
-                    cell_id: "3",
-                    new_xml: `${BLUE}${cell("3", "blue;")}`,
-                },
-            ],
-            {},
-        )
-        expect(outcome.ok).toBe(false)
-        if (!outcome.ok)
-            expect(outcome.errors[0]).toContain("not available in edit_diagram")
-    })
-})
@@ -11,15 +11,11 @@ const box = (id: string) =>
 const BEFORE_FIRST_EDIT = page(box("a"))
 const AFTER_FIRST_EDIT = page(box("a") + box("b"))

-const mocks = vi.hoisted(() => ({
-    loadDiagram: vi.fn((_xml: string, ..._rest: unknown[]) => null),
-}))
-
 vi.mock("@/contexts/diagram-context", () => ({
    useDiagram: () => ({
        chartXML: BEFORE_FIRST_EDIT,
        chartXMLRef: { current: AFTER_FIRST_EDIT },
-        loadDiagram: mocks.loadDiagram,
+        loadDiagram: vi.fn(() => null),
    }),
 }))

@@ -63,85 +59,3 @@ describe("the streaming preview of a second edit", () => {
        )
    })
 })
-
-describe("the streaming preview of display_diagram", () => {
-    it("expands the named styles and adds the defaults to the cells so far", () => {
-        mocks.loadDiagram.mockClear()
-        // A complete definition and cell, then a cell still being written
-        const xml = `<mxStyle name="blue" value="fillColor=#dae8fc;strokeColor=#6c8ebf;"/>
-<mxCell id="a" value="a" style="rounded=1;blue;" vertex="1" parent="1"><mxGeometry x="0" y="0" width="80" height="40" as="geometry"/></mxCell>
-<mxCell id="b" value="b" style="blue;" vertex="1" parent="1"><mxGeometry x="0" y="0" wid`
-        const messages = [
-            {
-                id: "m1",
-                role: "assistant",
-                parts: [
-                    {
-                        type: "tool-display_diagram",
-                        toolCallId: "draw-1",
-                        state: "input-streaming",
-                        input: { xml },
-                    },
-                ],
-            },
-        ] as any
-        renderHook(() =>
-            useDiagramStreaming({
-                messages,
-                processedToolCallsRef: { current: new Set() },
-                editDiagramOriginalXmlRef: { current: new Map() },
-                loadedMessageIdsRef: { current: new Set() },
-            }),
-        )
-        expect(mocks.loadDiagram).toHaveBeenCalledTimes(1)
-        const loaded = mocks.loadDiagram.mock.calls[0][0]
-        expect(loaded).toContain(
-            'style="rounded=1;fillColor=#dae8fc;strokeColor=#6c8ebf;whiteSpace=wrap;html=1;"',
-        )
-        expect(loaded).not.toContain("mxStyle")
-        expect(loaded).not.toContain('id="b"')
-    })
-})
-
-describe("the streaming preview of edit_diagram", () => {
-    it("adds the default styles to the cell being added, like the final edit", () => {
-        mocks.loadDiagram.mockClear()
-        const messages = [
-            {
-                id: "m1",
-                role: "assistant",
-                parts: [
-                    {
-                        type: "tool-edit_diagram",
-                        toolCallId: "edit-3",
-                        state: "input-streaming",
-                        input: {
-                            operations: [
-                                {
-                                    operation: "add",
-                                    cell_id: "c",
-                                    new_xml: box("c").replace(
-                                        'vertex="1"',
-                                        'style="rounded=1;" vertex="1"',
-                                    ),
-                                },
-                            ],
-                        },
-                    },
-                ],
-            },
-        ] as any
-        renderHook(() =>
-            useDiagramStreaming({
-                messages,
-                processedToolCallsRef: { current: new Set() },
-                editDiagramOriginalXmlRef: { current: new Map() },
-                loadedMessageIdsRef: { current: new Set() },
-            }),
-        )
-        expect(mocks.loadDiagram).toHaveBeenCalledTimes(1)
-        expect(mocks.loadDiagram.mock.calls[0][0]).toContain(
-            'style="rounded=1;whiteSpace=wrap;html=1;"',
-        )
-    })
-})