agent-canvas 0.1.23 → 0.1.25

package/bin/lib/commands/push.ts

@@ -54,10 +54,14 @@ export async function handlePush(args: string[]) {
 
   let result = await response.json() as any;
 
-  // Bun.build() can corrupt internal state in long-running daemons,
-  // causing compilation failures that look like user errors. Restart
-  // the daemon and retry once — if the retry also fails, it's a real error.
-  if (!result.ok && result.error === "Canvas compilation failed") {
+  // Bun.build() / Bun.Transpiler can fail in long-running daemons due to
+  // internal state corruption, CWD unlinking, etc. Restart and retry once.
+  // Skip restart only for clear user errors (syntax errors with line/col,
+  // runtime validation errors) — those won't be fixed by a restart.
+  const isUserError = !result.ok && result.errors && Object.values(result.errors).every((e: any) =>
+    typeof e === "string" && (/\(line \d+, col \d+\)/.test(e) || e.startsWith("Runtime error:")));
+  const shouldRestart = !result.ok && result.error === "Canvas compilation failed" && !isUserError;
+  if (shouldRestart) {
     console.error("Compilation failed, restarting daemon and retrying...");
     stopDaemon();
     await startDaemon();

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 {

package/package.json

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