botholomew 0.3.1 → 0.3.2

package/src/tui/components/ToolPanel.tsx

@@ -1,6 +1,6 @@
 import { Box, Text, useInput, useStdout } from "ink";
-import { useEffect, useMemo, useState } from "react";
-import { theme } from "../theme.ts";
+import { memo, useEffect, useMemo, useState } from "react";
+import { ansi, theme } from "../theme.ts";
 import { resolveToolDisplay, type ToolCallData } from "./ToolCall.tsx";
 
 interface ToolPanelProps {
@@ -10,16 +10,6 @@ interface ToolPanelProps {
 
 const SIDEBAR_WIDTH = 42;
 
-// ANSI escape helpers
-const RESET = "\x1b[0m";
-const BOLD = "\x1b[1m";
-const DIM = "\x1b[2m";
-const CYAN = "\x1b[36m";
-const GREEN = "\x1b[32m";
-const YELLOW = "\x1b[33m";
-const MAGENTA = "\x1b[35m";
-const BLUE = "\x1b[34m";
-
 /** Try to parse a string as JSON; returns the parsed value or undefined on failure */
 function tryParseJson(str: string): unknown | undefined {
   try {
@@ -37,10 +27,10 @@ function colorizeJson(str: string): string {
 }
 
 function colorizeValue(value: unknown, indent: number): string {
-  if (value === null) return `${MAGENTA}null${RESET}`;
+  if (value === null) return `${ansi.toolName}null${ansi.reset}`;
   if (typeof value === "boolean")
-    return `${MAGENTA}${value ? "true" : "false"}${RESET}`;
-  if (typeof value === "number") return `${YELLOW}${value}${RESET}`;
+    return `${ansi.toolName}${value ? "true" : "false"}${ansi.reset}`;
+  if (typeof value === "number") return `${ansi.accent}${value}${ansi.reset}`;
   if (typeof value === "string") {
     // Try to unwrap stringified JSON (common in tool results)
     const inner = tryParseJson(value);
@@ -48,7 +38,7 @@ function colorizeValue(value: unknown, indent: number): string {
       return colorizeValue(inner, indent);
     }
     const escaped = JSON.stringify(value);
-    return `${GREEN}${escaped}${RESET}`;
+    return `${ansi.success}${escaped}${ansi.reset}`;
   }
 
   const pad = "  ".repeat(indent);
@@ -67,7 +57,7 @@ function colorizeValue(value: unknown, indent: number): string {
     if (entries.length === 0) return "{}";
     const lines = entries.map(
       ([k, v]) =>
-        `${innerPad}${CYAN}${JSON.stringify(k)}${RESET}: ${colorizeValue(v, indent + 1)}`,
+        `${innerPad}${ansi.info}${JSON.stringify(k)}${ansi.reset}: ${colorizeValue(v, indent + 1)}`,
     );
     return `{\n${lines.join(",\n")}\n${pad}}`;
   }
@@ -88,26 +78,36 @@ function buildDetailAnsi(tool: ToolCallData): string {
     tool.name,
     tool.input,
   );
-  lines.push(`${BOLD}${CYAN}${displayName}${RESET}`);
+  lines.push(`${ansi.bold}${ansi.info}${displayName}${ansi.reset}`);
   if (tool.name === "mcp_exec") {
-    lines.push(`${DIM}via mcp_exec${RESET}`);
+    lines.push(`${ansi.dim}via mcp_exec${ansi.reset}`);
   }
-  lines.push(`${DIM}Time: ${time}${RESET}`);
+  lines.push(`${ansi.dim}Time: ${time}${ansi.reset}`);
   if (tool.running) {
-    lines.push(`${YELLOW}⟳ running${RESET}`);
+    lines.push(`${ansi.accent}⟳ running${ansi.reset}`);
   }
   lines.push("");
 
-  lines.push(`${BOLD}${BLUE}Input${RESET}`);
+  lines.push(`${ansi.bold}${ansi.primary}Input${ansi.reset}`);
   lines.push(colorizeJson(displayInput));
   lines.push("");
 
   if (tool.output) {
-    lines.push(`${BOLD}${BLUE}Output${RESET}`);
-    lines.push(colorizeJson(tool.output));
+    if (tool.isError) {
+      lines.push(`${ansi.bold}${ansi.error}Error${ansi.reset}`);
+      lines.push(`${ansi.error}${colorizeJson(tool.output)}${ansi.reset}`);
+    } else {
+      lines.push(`${ansi.bold}${ansi.primary}Output${ansi.reset}`);
+      if (tool.largeResult) {
+        lines.push(
+          `${ansi.accent}Paginated for LLM: ${tool.largeResult.chars.toLocaleString()} chars, ${tool.largeResult.pages} page(s) — stored as ${tool.largeResult.id}${ansi.reset}`,
+        );
+      }
+      lines.push(colorizeJson(tool.output));
+    }
   } else if (!tool.running) {
-    lines.push(`${BOLD}${BLUE}Output${RESET}`);
-    lines.push(`${DIM}(no output)${RESET}`);
+    lines.push(`${ansi.bold}${ansi.primary}Output${ansi.reset}`);
+    lines.push(`${ansi.dim}(no output)${ansi.reset}`);
   }
 
   return lines.join("\n");
@@ -115,7 +115,10 @@ function buildDetailAnsi(tool: ToolCallData): string {
 
 const PAGE_SCROLL_LINES = 10;
 
-export function ToolPanel({ toolCalls, isActive }: ToolPanelProps) {
+export const ToolPanel = memo(function ToolPanel({
+  toolCalls,
+  isActive,
+}: ToolPanelProps) {
   const { stdout } = useStdout();
   const termRows = stdout?.rows ?? 24;
   const [selectedIndex, setSelectedIndex] = useState(0);
@@ -261,7 +264,7 @@ export function ToolPanel({ toolCalls, isActive }: ToolPanelProps) {
         {sidebarVisible.map((tc, vi) => {
           const i = vi + sidebarScrollOffset;
           const isSelected = i === selectedIndex;
-          const icon = tc.running ? "⟳" : "✔";
+          const icon = tc.running ? "⟳" : tc.isError ? "✘" : "✔";
           const time = tc.timestamp.toLocaleTimeString([], {
             hour: "2-digit",
             minute: "2-digit",
@@ -273,7 +276,7 @@ export function ToolPanel({ toolCalls, isActive }: ToolPanelProps) {
               ? `${displayName.slice(0, maxName - 1)}…`
               : displayName;
           return (
-            <Box key={`${i}-${tc.name}`} paddingX={1}>
+            <Box key={tc.id} paddingX={1}>
               <Text
                 backgroundColor={isSelected ? theme.selectionBg : undefined}
                 bold={isSelected}
@@ -288,7 +291,13 @@ export function ToolPanel({ toolCalls, isActive }: ToolPanelProps) {
               >
                 {isSelected ? "▸" : " "}{" "}
                 <Text
-                  color={tc.running ? theme.accent : theme.muted}
+                  color={
+                    tc.running
+                      ? theme.accent
+                      : tc.isError
+                        ? theme.error
+                        : theme.muted
+                  }
                   bold={false}
                 >
                   {icon}
@@ -330,4 +339,4 @@ export function ToolPanel({ toolCalls, isActive }: ToolPanelProps) {
       </Box>
     </Box>
   );
-}
+});

package/src/tui/theme.ts

@@ -29,8 +29,11 @@ function detectDarkBackground(): boolean {
         ["read", "-g", "AppleInterfaceStyle"],
         { encoding: "utf-8", timeout: 500 },
       );
-      // Returns "Dark" in dark mode; exits non-zero in light mode
-      return result.stdout?.trim() === "Dark";
+      // Returns "Dark" in dark mode, "Light" or exit 1 in light mode
+      // Only trust the result if the command succeeded (status 0)
+      if (result.status === 0) {
+        return result.stdout?.trim() === "Dark";
+      }
     } catch {
       // fall through to default
     }
@@ -44,7 +47,7 @@ const isDark = detectDarkBackground();
 export const theme = {
   accent: isDark ? "yellow" : "#B8860B",
   accentBorder: isDark ? "yellow" : "#B8860B",
-  userBg: isDark ? "#2a4a6c" : "#d0e0f0",
+  userBg: isDark ? "#2a5a8c" : "#d0e0f0",
   selectionBg: isDark ? "#333" : "#ddd",
   success: "green",
   error: "red",
@@ -54,5 +57,19 @@ export const theme = {
   muted: "gray",
 } as const;
 
+/** ANSI escape codes for raw string building (detail panes, etc.) */
+export const ansi = {
+  reset: "\x1b[0m",
+  bold: "\x1b[1m",
+  dim: "\x1b[2m",
+  success: "\x1b[32m",
+  error: "\x1b[31m",
+  info: "\x1b[36m",
+  primary: "\x1b[34m",
+  accent: "\x1b[33m",
+  toolName: "\x1b[35m",
+  muted: "\x1b[2m",
+} as const;
+
 // Exported for testing
 export { detectDarkBackground };

package/src/utils/title.ts

@@ -0,0 +1,47 @@
+import Anthropic from "@anthropic-ai/sdk";
+import type { BotholomewConfig } from "../config/schemas.ts";
+import type { DbConnection } from "../db/connection.ts";
+import { updateThreadTitle } from "../db/threads.ts";
+import { logger } from "./logger.ts";
+
+/**
+ * Generate a short title for a thread using the chunker model (Haiku).
+ * Fire-and-forget — errors are logged at debug level and never propagated.
+ */
+export async function generateThreadTitle(
+  config: Required<BotholomewConfig>,
+  conn: DbConnection,
+  threadId: string,
+  context: string,
+): Promise<void> {
+  try {
+    const client = new Anthropic({
+      apiKey: config.anthropic_api_key || undefined,
+    });
+
+    const response = await client.messages.create({
+      model: config.chunker_model,
+      max_tokens: 50,
+      system:
+        "You are a title generator. The user will provide the first message from a conversation. Output a short descriptive title (5-8 words). Output ONLY the title, nothing else.",
+      messages: [
+        {
+          role: "user",
+          content: `Generate a title for this message:\n\n"${context}"`,
+        },
+      ],
+    });
+
+    const title = response.content
+      .filter((b) => b.type === "text")
+      .map((b) => b.text)
+      .join("")
+      .trim();
+
+    if (title) {
+      await updateThreadTitle(conn, threadId, title);
+    }
+  } catch (err) {
+    logger.warn(`Failed to generate thread title: ${err}`);
+  }
+}