fix: auto-restart daemon on any compilation failure, not just "Unknown Error"

Bun.build() corruption in long-running daemons can manifest as
`{ success: false }` with empty error logs (not just a thrown
"Unknown Error"). Broaden the CLI retry logic to restart the daemon
on any compilation failure — if the retry also fails, it's a real
user error.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: matej21

bin/lib/commands/push.ts

@@ -55,9 +55,10 @@ 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...");
+  // 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") {
+    console.error("Compilation failed, restarting daemon and retrying...");
     stopDaemon();
     await startDaemon();
     const retryResponse = await fetch(`${BASE_URL}/api/session/${sessionId}/plan`, {

daemon/src/compiler.ts

@@ -166,7 +166,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();