agent-canvas 0.1.22 → 0.1.24

package/bin/lib/commands/push.ts

@@ -55,9 +55,14 @@ export async function handlePush(args: string[]) {
   let result = await response.json() as any;
 
   // Bun.build() can corrupt internal state in long-running daemons,
-  // returning "Unknown Error, TODO". Restart daemon and retry once.
-  if (!result.ok && typeof result.error === "string" && result.error.includes("Unknown Error")) {
-    console.error("Daemon bundler crashed, restarting...");
+  // throwing opaque errors like "Bundle failed" or "Unknown Error".
+  // Restart and retry once — but not for clear syntax errors (which
+  // now have precise line/col info from the transpiler pre-check).
+  const isBundlerCrash = !result.ok && result.error === "Canvas compilation failed"
+    && result.errors && Object.values(result.errors).some((e: any) =>
+      typeof e === "string" && (e === "Bundle failed" || e.includes("Unknown Error")));
+  if (isBundlerCrash) {
+    console.error("Compilation failed, restarting daemon and retrying...");
     stopDaemon();
     await startDaemon();
     const retryResponse = await fetch(`${BASE_URL}/api/session/${sessionId}/plan`, {

package/daemon/src/compiler.ts

@@ -20,6 +20,9 @@ const { Section, Item, Task, FilePreview, CodeBlock, Callout,
         Markdown, useFeedback, useAnnotations } = C;
 `;
 
+/** Number of lines in COMPONENT_IMPORTS — used to adjust error line numbers */
+const COMPONENT_IMPORTS_LINES = COMPONENT_IMPORTS.split("\n").length - 1;
+
 // --- Validation via Preact --------------------------------------------------
 // After Bun.build() succeeds we strip ESM syntax from the compiled output,
 // provide Preact as the React implementation with lightweight stub components,
@@ -121,7 +124,11 @@ async function validateCompiledPlan(js: string): Promise<{ ok: true } | { ok: fa
     try {
       await mermaid.parse(source);
     } catch (e: any) {
-      return { ok: false, error: `Mermaid syntax error: ${e?.message || String(e)}` };
+      const msg = e?.message || String(e);
+      // DOMPurify doesn't work in Bun (no DOM) — mermaid.parse() triggers
+      // it for some diagram types even though it's just parsing. Skip these.
+      if (msg.includes("DOMPurify")) continue;
+      return { ok: false, error: `Mermaid syntax error: ${msg}` };
     }
   }
 
@@ -148,6 +155,20 @@ export async function compilePlan(jsx: string, projectRoot?: string): Promise<Co
     ? `${COMPONENT_IMPORTS}\n${jsx}`
     : `${COMPONENT_IMPORTS}\nexport default function Plan() {\n  return (<>${jsx}</>);\n}\n`;
 
+  // Pre-validate syntax with Bun.Transpiler — gives precise error positions,
+  // unlike Bun.build() which throws opaque "Bundle failed" for syntax errors.
+  try {
+    const transpiler = new Bun.Transpiler({ loader: "jsx" });
+    transpiler.transformSync(source);
+  } catch (syntaxError: any) {
+    const pos = syntaxError?.position;
+    // Offset: COMPONENT_IMPORTS lines + wrapper lines (export default function Plan() + return)
+    const wrapperLines = hasDefaultExport ? 1 : 2;
+    const lineOffset = COMPONENT_IMPORTS_LINES + wrapperLines;
+    const loc = pos ? ` (line ${pos.line - lineOffset}, col ${pos.column})` : "";
+    return { ok: false, error: `${syntaxError?.message || "Syntax error"}${loc}` };
+  }
+
   const tmpFile = join(COMPILE_TEMP_DIR, `plan-${randomUUID()}.jsx`);
 
   try {
@@ -166,7 +187,7 @@ export async function compilePlan(jsx: string, projectRoot?: string): Promise<Co
           .filter((l) => l.level === "error")
           .map((l) => l.message)
           .join("\n");
-        return { ok: false, error: errors || "Compilation failed" };
+        return { ok: false, error: errors || "Bundle failed" };
       }
 
       const js = await result.outputs[0].text();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-canvas",
-  "version": "0.1.22",
+  "version": "0.1.24",
   "description": "Interactive visual canvas for Claude Code — rich annotatable documents in the browser",
   "bin": {
     "agent-canvas": "bin/agent-canvas.js"