horizon-code 0.6.0 → 0.6.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "horizon-code",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "AI-powered trading strategy terminal for Polymarket",
   "type": "module",
   "bin": {

package/src/ai/client.ts

@@ -32,7 +32,7 @@ export async function initMCP(): Promise<{ toolCount: number }> {
     });
     mcpTools = await mcpClient.tools();
     return { toolCount: Object.keys(mcpTools).length };
-  } catch { return { toolCount: 0 }; }
+  } catch (e) { console.error("[mcp] init failed:", e); return { toolCount: 0 }; }
 }
 
 export async function closeMCP(): Promise<void> {
@@ -141,7 +141,7 @@ async function* consumeSSE(res: Response): AsyncGenerator<Record<string, any>> {
       if (!line.startsWith("data: ")) continue;
       const data = line.slice(6);
       if (data === "[DONE]") return;
-      try { yield JSON.parse(data); } catch {}
+      try { yield JSON.parse(data); } catch (e) { console.error("[sse] malformed JSON:", data.slice(0, 100)); }
     }
   }
 }
@@ -162,7 +162,7 @@ function toolsToServerFormat(tools: Record<string, any>): Record<string, any> {
         const converted = zodSchema(tool.parameters);
         params = converted.jsonSchema ?? params;
       }
-    } catch {}
+    } catch (e) { failures++; console.error("[tools] schema conversion failed for", name, e); }
     result[name] = { description: tool.description ?? "", parameters: params };
   }
   return result;
@@ -273,6 +273,7 @@ export async function* chat(
     let emittedResponseLen = 0;
     let emittedCode: string | null = null;
     let structuredActive = useStructuredOutput;
+    let emittedAnyText = false;
     const pendingToolCalls: { toolName: string; args: Record<string, unknown> }[] = [];
     let eventCount = 0;
 
@@ -297,6 +298,7 @@ export async function* chat(
           if (trimmed.length > 3 && !trimmed.startsWith("{") && !trimmed.startsWith("[")) {
             structuredActive = false;
             yield { type: "text-delta", textDelta: jsonBuf };
+            emittedAnyText = true;
             jsonBuf = "";
             continue;
           }
@@ -304,6 +306,7 @@ export async function* chat(
           const response = extractJsonStringField(jsonBuf, "response");
           if (response && response.length > emittedResponseLen) {
             yield { type: "text-delta", textDelta: response.slice(emittedResponseLen) };
+            emittedAnyText = true;
             emittedResponseLen = response.length;
           }
 
@@ -314,6 +317,7 @@ export async function* chat(
           }
         } else {
           yield { type: "text-delta", textDelta: delta };
+          emittedAnyText = true;
         }
       } else if (event.type === "tool-call") {
         hasToolCalls = true;
@@ -339,15 +343,17 @@ export async function* chat(
     // the model likely responded with plain text — flush jsonBuf as text
     if (structuredActive && jsonBuf && emittedResponseLen === 0) {
       yield { type: "text-delta", textDelta: jsonBuf };
+      emittedAnyText = true;
     }
 
     // If no tool calls, we're done
     if (!hasToolCalls || pendingToolCalls.length === 0) {
       if (eventCount === 0) {
         yield { type: "error", message: "No response from server. Check your connection or try again." };
-      } else if (!hasToolCalls && jsonBuf === "" && emittedResponseLen === 0) {
-        // Server sent events (meta, usage) but no actual content
-        yield { type: "text-delta", textDelta: "*(Server returned no content. This may happen if the model is overloaded. Try again.)*" };
+      } else if (!emittedAnyText && !hasToolCalls && emittedCode === null) {
+        // Server sent events (meta, usage) but no actual text or code content
+        console.error("[chat] No content received. Events:", eventCount, "jsonBuf:", jsonBuf.slice(0, 200));
+        yield { type: "text-delta", textDelta: "*(No response received. Try again.)*" };
       }
       yield { type: "finish" };
       return;
@@ -391,6 +397,7 @@ export async function* chat(
     if (structuredActive) {
       jsonBuf = "";
       emittedResponseLen = 0;
+      emittedAnyText = false;
     }
 
     // Loop back — server will call OpenRouter again with the full conversation including tool results

package/src/ai/system-prompt.ts

@@ -131,7 +131,7 @@ export function getSystemPrompt(mode: Mode, verbosity: string = "normal"): strin
       const { buildProfileContext } = require("../platform/profile.ts");
       const profileCtx = buildProfileContext();
       if (profileCtx) prompt += "\n\n" + profileCtx;
-    } catch {}
+    } catch (e) { console.error("[prompt] profile context failed:", e); }
     // Inject active strategy context so research tools auto-scope
     const draft = store.getActiveSession()?.strategyDraft;
     if (draft?.code) {

package/src/app.ts

@@ -223,7 +223,7 @@ export class App {
         for (const s of dbSessions) {
           await deleteDbSession(s.id);
         }
-      } catch {}
+      } catch (e: any) { console.error("[app] delete chats failed:", e?.message); }
       this.showSystemMsg("All chats deleted.");
       this.settingsPanel.hide();
     });
@@ -461,7 +461,7 @@ export class App {
         this.keyHandler.codePanelVisible = this.codePanel.visible;
 
         this.renderer.requestRender();
-      } catch {}
+      } catch (e: any) { console.error("[app] state listener error:", e?.message); }
     });
 
     renderer.on("resize", () => renderer.requestRender());
@@ -578,7 +578,7 @@ export class App {
                   this.codePanel.appendLog(r.actionTaken);
                 }
               }
-            } catch {}
+            } catch (e: any) { console.error("[app] alert check failed:", e?.message); }
           })();
         })(localMetricsData);
 
@@ -590,7 +590,7 @@ export class App {
               for (const [pid] of runningProcesses) {
                 if (isRecording(pid)) recordMetrics(pid, metrics as any);
               }
-            } catch {}
+            } catch (e: any) { console.error("[app] replay record failed:", e?.message); }
           })();
         })(localMetricsData);
       } else if (this._hasLocalMetrics && alive === 0) {
@@ -720,11 +720,11 @@ export class App {
           this.showSystemMsg(`Logged in as ${loginResult.email}`);
           // Start session sync + platform sync now that we have a live session
           import("./platform/session-sync.ts").then(({ loadSessions, startAutoSave }) => {
-            loadSessions().catch(() => {});
+            loadSessions().catch((e: any) => console.error("[app] session/sync init failed:", e?.message));
             startAutoSave();
-          }).catch(() => {});
+          }).catch((e: any) => console.error("[app] session/sync init failed:", e?.message));
           import("./platform/sync.ts").then(({ platformSync }) => {
-            platformSync.start(30000).catch(() => {});
+            platformSync.start(30000).catch((e: any) => console.error("[app] platformSync start failed:", e?.message));
           });
         } else {
           this.showSystemMsg(`Login failed: ${loginResult.error}`);
@@ -1079,15 +1079,19 @@ export class App {
       const { hasLiveSession } = await import("./platform/supabase.ts");
       const live = await hasLiveSession();
 
+      const { loadSessions, startAutoSave } = await import("./platform/session-sync.ts");
       if (live) {
-        const { loadSessions, startAutoSave } = await import("./platform/session-sync.ts");
-        await loadSessions().catch(() => {});
+        await loadSessions().catch((e) => {
+          this.splash.setLoading(`Failed to load chats: ${e?.message ?? "unknown error"}`);
+        });
         startAutoSave();
+      } else {
+        this.splash.setLoading("Session expired -- type /login to restore your chats");
       }
 
       // Start platform sync (works with API key)
       const { platformSync } = await import("./platform/sync.ts");
-      platformSync.start(30000).catch(() => {});
+      platformSync.start(30000).catch((e: any) => console.error("[app] platformSync start failed:", e?.message));
 
       // Final status
       const firstTime = !cfg.has_launched;
@@ -1128,6 +1132,10 @@ export class App {
     store.update({ sessions });
     for (const [id] of this.messageRenderables) this.scrollBox.remove(`msg-${id}`);
     this.messageRenderables.clear();
+    this.codePanel.clearWidgets();
+    this.codePanel.setLogs("");
+    this.codePanel.setCode("", "none");
+    this.codePanel.setMetrics(null);
     this.updateContextMeter();
     this.inputBar.focus();
     this.renderer.requestRender();
@@ -1494,13 +1502,20 @@ export class App {
           }
         } else if (part.type === "tool-result") {
           // Replace the spinning tool-call with a completed tool-result (same line, not stacked)
+          // Include toolResult so renderer can detect errors and show brief explanations
+          const hasError = part.result && typeof part.result === "object" && "error" in (part.result as any);
+          const resultBlock: import("./chat/types.ts").ContentBlock = {
+            type: "tool-result",
+            toolName: part.toolName,
+            ...(hasError ? { toolResult: part.result } : {}),
+          };
           const callIdx = currentBlocks.findIndex(
             (b) => b.type === "tool-call" && b.toolName === part.toolName,
           );
           if (callIdx !== -1) {
-            currentBlocks[callIdx] = { type: "tool-result", toolName: part.toolName };
+            currentBlocks[callIdx] = resultBlock;
           } else {
-            currentBlocks.push({ type: "tool-result", toolName: part.toolName });
+            currentBlocks.push(resultBlock);
           }
 
           if (WIDGET_TOOLS.has(part.toolName)) {
@@ -1774,7 +1789,7 @@ export class App {
       const { saveActiveSession, stopAutoSave } = await import("./platform/session-sync.ts");
       stopAutoSave();
       await saveActiveSession();
-    } catch {}
+    } catch (e: any) { console.error("[app] shutdown save failed:", e?.message); }
     if (dashboard.running) dashboard.stop();
     cleanupStrategyProcesses();
     destroyTreeSitterClient();

package/src/chat/renderer.ts

@@ -33,7 +33,7 @@ const syntaxStyle = SyntaxStyle.fromStyles({
   "@punctuation.special": { fg: h("#D7BA7D") },
 });
 
-// Braille spinner frames
+// Braille spinner frames (tool calls)
 const BRAILLE = [
   "\u2801", "\u2803", "\u2807", "\u280f",
   "\u281f", "\u283f", "\u287f", "\u28ff",
@@ -41,6 +41,11 @@ const BRAILLE = [
   "\u28e0", "\u28c0", "\u2880", "\u2800",
 ];
 
+// Dot orbit spinner for header (different visual rhythm)
+const ORBIT = [
+  "\u25DC", "\u25DD", "\u25DE", "\u25DF",  // ◜ ◝ ◞ ◟
+];
+
 // Tool name → human-readable label
 function toolLabel(name: string): string {
   return name.replace(/_/g, " ");
@@ -62,9 +67,14 @@ export class ChatRenderer {
     this.spinnerTimer = setInterval(() => {
       if (this.spinnerNodes.length === 0) return;
       this.spinnerFrame = (this.spinnerFrame + 1) % BRAILLE.length;
-      const char = BRAILLE[this.spinnerFrame]!;
+      const brailleChar = BRAILLE[this.spinnerFrame]!;
+      const orbitChar = ORBIT[this.spinnerFrame % ORBIT.length]!;
       for (const node of this.spinnerNodes) {
-        node.content = `${char} `;
+        if (node.id.includes("stream-spin")) {
+          node.content = `  ${orbitChar}`;
+        } else {
+          node.content = `${brailleChar} `;
+        }
       }
       this.renderer.requestRender();
     }, 60);
@@ -80,6 +90,7 @@ export class ChatRenderer {
 
   renderMessage(message: Message): BoxRenderable {
     const isUser = message.role === "user";
+    const isStreaming = message.status === "streaming" || message.status === "thinking";
 
     const box = new BoxRenderable(this.renderer, {
       id: `msg-${message.id}`,
@@ -92,16 +103,60 @@ export class ChatRenderer {
     });
 
     if (message.role === "user" || message.role === "assistant") {
-      box.add(new TextRenderable(this.renderer, {
+      const headerBox = new BoxRenderable(this.renderer, {
+        id: `msg-header-${message.id}`,
+        flexDirection: "row",
+        width: "100%",
+      });
+      headerBox.add(new TextRenderable(this.renderer, {
         id: `msg-label-${message.id}`,
         content: isUser ? "U S E R" : "H O R I Z O N",
         fg: isUser ? COLORS.textMuted : COLORS.borderDim,
       }));
+
+      // Streaming indicator next to the role label — always visible at top
+      if (!isUser && isStreaming) {
+        const streamSpinner = new TextRenderable(this.renderer, {
+          id: `msg-stream-spin-${message.id}`,
+          content: `  ${ORBIT[0]}`,
+          fg: COLORS.accent,
+        });
+        this.spinnerNodes.push(streamSpinner);
+        headerBox.add(streamSpinner);
+      }
+
+      box.add(headerBox);
     }
 
+    // Render tool blocks first, then content blocks (text/markdown/widgets)
+    // This ensures tools are always at the top, generation text at the bottom
+    const toolBlocks: { block: ContentBlock; idx: number }[] = [];
+    const contentBlocks: { block: ContentBlock; idx: number }[] = [];
+
     for (let i = 0; i < message.content.length; i++) {
       const block = message.content[i]!;
-      const renderable = this.renderBlock(block, message, i);
+      if (block.type === "tool-call" || block.type === "tool-result") {
+        toolBlocks.push({ block, idx: i });
+      } else {
+        contentBlocks.push({ block, idx: i });
+      }
+    }
+
+    for (const { block, idx } of toolBlocks) {
+      const renderable = this.renderBlock(block, message, idx);
+      if (renderable) box.add(renderable);
+    }
+
+    // Add spacing between tool section and content
+    if (toolBlocks.length > 0 && contentBlocks.length > 0) {
+      box.add(new TextRenderable(this.renderer, {
+        id: `msg-tool-spacer-${message.id}`,
+        content: "",
+      }));
+    }
+
+    for (const { block, idx } of contentBlocks) {
+      const renderable = this.renderBlock(block, message, idx);
       if (renderable) box.add(renderable);
     }
 
@@ -144,12 +199,19 @@ export class ChatRenderer {
       case "thinking":
         return this.renderThinking(message.id, index);
 
-      case "error":
+      case "error": {
+        const rawErr = block.text ?? "Unknown error";
+        // Clean up error text: first line only, remove stack traces and verbose prefixes
+        const cleanErr = rawErr
+          .replace(/^Error:\s*/i, "")
+          .replace(/\n[\s\S]*/, "")
+          .slice(0, 120);
         return new TextRenderable(this.renderer, {
           id: `msg-err-${message.id}-${index}`,
-          content: `x ${block.text ?? "Unknown error"}`,
+          content: `\u2716 ${cleanErr}`,
           fg: COLORS.error,
         });
+      }
 
       default:
         return null;
@@ -179,7 +241,7 @@ export class ChatRenderer {
     return box;
   }
 
-  // Tool in-progress: braille spinner + name
+  // Tool in-progress: accent braille spinner + name
   private renderToolCall(block: ContentBlock, msgId: string, index: number): BoxRenderable {
     const box = new BoxRenderable(this.renderer, {
       id: `msg-tool-${msgId}-${index}`,
@@ -189,7 +251,7 @@ export class ChatRenderer {
     const spinner = new TextRenderable(this.renderer, {
       id: `msg-tool-icon-${msgId}-${index}`,
       content: `${BRAILLE[0]} `,
-      fg: COLORS.textMuted,
+      fg: COLORS.accent,
     });
     this.spinnerNodes.push(spinner);
     box.add(spinner);
@@ -197,34 +259,54 @@ export class ChatRenderer {
     box.add(new TextRenderable(this.renderer, {
       id: `msg-tool-name-${msgId}-${index}`,
       content: toolLabel(block.toolName ?? "tool"),
-      fg: COLORS.textMuted,
+      fg: COLORS.text,
     }));
 
     return box;
   }
 
-  // Tool completed: tick or error icon + name
+  // Tool completed: green check or red error with brief explanation
   private renderToolResult(block: ContentBlock, msgId: string, index: number): BoxRenderable {
     const box = new BoxRenderable(this.renderer, {
       id: `msg-result-${msgId}-${index}`,
       flexDirection: "row",
-      marginBottom: 1,
     });
 
     const hasError = block.toolResult && typeof block.toolResult === "object" && "error" in (block.toolResult as any);
-    box.add(new TextRenderable(this.renderer, {
-      id: `msg-result-icon-${msgId}-${index}`,
-      content: hasError ? "x " : ". ",
-      fg: hasError ? COLORS.error : COLORS.success,
-    }));
 
-    const label = toolLabel(block.toolName ?? "done");
-    const errorText = hasError ? ` — ${((block.toolResult as any).error ?? "").slice(0, 60)}` : "";
-    box.add(new TextRenderable(this.renderer, {
-      id: `msg-result-name-${msgId}-${index}`,
-      content: label + errorText,
-      fg: hasError ? COLORS.error : COLORS.textMuted,
-    }));
+    if (hasError) {
+      // Red x + tool name + brief, human-readable error
+      box.add(new TextRenderable(this.renderer, {
+        id: `msg-result-icon-${msgId}-${index}`,
+        content: "\u2716 ",
+        fg: COLORS.error,
+      }));
+
+      const label = toolLabel(block.toolName ?? "tool");
+      const rawError = String((block.toolResult as any).error ?? "failed");
+      // Clean up common error patterns to be user-friendly
+      const briefError = rawError
+        .replace(/^Error:\s*/i, "")
+        .replace(/\n[\s\S]*/, "")  // first line only
+        .slice(0, 80);
+      box.add(new TextRenderable(this.renderer, {
+        id: `msg-result-name-${msgId}-${index}`,
+        content: `${label} -- ${briefError}`,
+        fg: COLORS.error,
+      }));
+    } else {
+      // Green check + tool name
+      box.add(new TextRenderable(this.renderer, {
+        id: `msg-result-icon-${msgId}-${index}`,
+        content: "\u2713 ",
+        fg: COLORS.success,
+      }));
+      box.add(new TextRenderable(this.renderer, {
+        id: `msg-result-name-${msgId}-${index}`,
+        content: toolLabel(block.toolName ?? "done"),
+        fg: COLORS.textMuted,
+      }));
+    }
 
     return box;
   }

package/src/platform/auth.ts

@@ -1,7 +1,7 @@
 import { saveConfig, loadConfig, getApiKey } from "./config.ts";
 import { platform } from "./client.ts";
 
-export type AuthStatus = "authenticated" | "no_key" | "invalid_key";
+export type AuthStatus = "authenticated" | "no_key" | "invalid_key" | "network_error";
 
 export async function checkAuth(): Promise<AuthStatus> {
   const key = getApiKey();
@@ -13,8 +13,8 @@ export async function checkAuth(): Promise<AuthStatus> {
     return "authenticated";
   } catch (err: any) {
     if (err?.status === 401 || err?.status === 403) return "invalid_key";
-    // Network error or server down — assume key is fine
-    return "authenticated";
+    // Network error — can't verify, report as network error
+    return "network_error";
   }
 }
 

package/src/platform/session-sync.ts

@@ -84,7 +84,7 @@ export async function loadSessions(): Promise<void> {
         openTabIds: openIds,
       });
     }
-  } catch {}
+  } catch (e: any) { console.error("[session-sync] loadSessions failed:", e?.message ?? e); }
 }
 
 /**
@@ -132,7 +132,7 @@ export async function saveActiveSession(): Promise<void> {
 
     // Update session metadata
     await updateDbSession(dbId, { name: session.name }).catch(() => {});
-  } catch {}
+  } catch (e: any) { console.error("[session-sync] saveActiveSession failed:", e?.message ?? e); }
 }
 
 /**
@@ -140,7 +140,7 @@ export async function saveActiveSession(): Promise<void> {
  */
 export async function deleteSessionFromDb(sessionId: string): Promise<void> {
   if (!(await isLoggedIn())) return;
-  try { await deleteDbSession(sessionId); } catch {}
+  try { await deleteDbSession(sessionId); } catch (e: any) { console.error("[session-sync] deleteSessionFromDb failed:", e?.message ?? e); }
 }
 
 /**

package/src/platform/supabase.ts

@@ -34,7 +34,8 @@ export function getSupabase(): SupabaseClient {
             refresh_token: encryptEnvVar(session.refresh_token),
           };
           config.session_encrypted = true;
-        } catch {
+        } catch (e) {
+          console.error("[supabase] encryption failed, using plaintext:", e);
           // Fallback to plaintext if encryption fails
           config.supabase_session = {
             access_token: session.access_token,
@@ -67,7 +68,7 @@ function saveSessionTokens(config: ReturnType<typeof loadConfig>, accessToken: s
       config.supabase_session = { access_token: encryptEnvVar(accessToken), refresh_token: encryptEnvVar(refreshToken) };
       config.session_encrypted = true;
       return;
-    } catch {}
+    } catch (e) { console.error("[supabase] saveSessionTokens encryption failed:", e); }
   }
   config.supabase_session = { access_token: accessToken, refresh_token: refreshToken };
   config.session_encrypted = false;
@@ -125,7 +126,7 @@ export async function restoreSession(): Promise<boolean> {
         if (config.api_key) platform.setApiKey(config.api_key);
         return true;
       }
-    } catch {}
+    } catch (e) { console.error("[supabase] setSession failed:", e); }
 
     // Access token expired — try refreshing with just the refresh token
     try {
@@ -142,7 +143,7 @@ export async function restoreSession(): Promise<boolean> {
         if (config.api_key) platform.setApiKey(config.api_key);
         return true;
       }
-    } catch {}
+    } catch (e) { console.error("[supabase] refreshSession failed:", e); }
 
     // Both failed — don't delete the session, it might work next time
     // (network issue, Supabase outage, etc.). The API key still works for chat.
@@ -224,7 +225,7 @@ export async function loginWithBrowser(): Promise<{ success: boolean; error?: st
             try {
               const apiKey = await createApiKey();
               if (apiKey) { config.api_key = apiKey; platform.setApiKey(apiKey); }
-            } catch {}
+            } catch (e) { console.error("[supabase] createApiKey failed:", e); }
           } else {
             platform.setApiKey(config.api_key);
           }
@@ -235,10 +236,10 @@ export async function loginWithBrowser(): Promise<{ success: boolean; error?: st
       }
       if (res.status === 410) return { success: false, error: "Session expired. Type /login to sign in again." };
       if (res.status === 404) return { success: false, error: "Auth session not found." };
-    } catch {}
+    } catch (e) { console.error("[supabase] poll timeout:", e); }
   }
 
-  try { await sb.from("cli_auth_sessions").delete().eq("id", sessionId); } catch {}
+  try { await sb.from("cli_auth_sessions").delete().eq("id", sessionId); } catch (e) { console.error("[supabase] cleanup auth session failed:", e); }
   return { success: false, error: "Login timed out (90s). Try /login again." };
 }
 
@@ -260,7 +261,7 @@ export async function loginWithPassword(email: string, password: string): Promis
     try {
       const apiKey = await createApiKey();
       if (apiKey) { config.api_key = apiKey; platform.setApiKey(apiKey); }
-    } catch {}
+    } catch (e) { console.error("[supabase] createApiKey failed:", e); }
   } else {
     platform.setApiKey(config.api_key);
   }
@@ -295,7 +296,7 @@ async function createApiKey(): Promise<string | null> {
     });
     if (error) return null;
     return rawKey;
-  } catch { return null; }
+  } catch (e) { console.error("[supabase] createApiKey failed:", e); return null; }
 }
 
 // ── Logout ──

package/src/platform/sync.ts

@@ -8,8 +8,11 @@ const pnlHistories: Map<string, number[]> = new Map();
 
 export class PlatformSync {
   private timer: ReturnType<typeof setInterval> | null = null;
+  private _polling = false;
 
   async start(intervalMs = 30000): Promise<void> {
+    if (this.timer) return; // Prevent duplicate timers
+
     const loggedIn = await isLoggedIn();
     if (!loggedIn && !platform.authenticated) return;
 
@@ -22,6 +25,8 @@ export class PlatformSync {
   }
 
   private async poll(): Promise<void> {
+    if (this._polling) return;
+    this._polling = true;
     try {
       const strategies = await platform.listStrategies();
 
@@ -88,8 +93,11 @@ export class PlatformSync {
       });
 
       store.update({ deployments, connection: "connected" });
-    } catch {
+    } catch (e) {
+      console.error("[sync] poll failed:", e);
       store.update({ connection: "disconnected" });
+    } finally {
+      this._polling = false;
     }
   }
 }