Fix TUI prompt corruption by using Log instead of UI.println (#180)

* fix: replace UI.println with Log in engine bootstrap to prevent TUI prompt corruption

Engine bootstrap messages (downloading uv, engine ready, etc.) were written via
UI.println() which writes to stderr. In TUI mode the framework captures stderr,
causing garbled text like "readyltimate-engine 0.4.0..." in the prompt input area.

Switched all status messages in engine.ts to use Log.Default.info() which routes
to the log file instead of stderr, keeping the TUI display clean.

https://claude.ai/code/session_01MBVNhRX6XKTHqtZ6zS1i6w

* chore: update bun.lock after dependency install

https://claude.ai/code/session_01MBVNhRX6XKTHqtZ6zS1i6w

* fix: replace console.log/error with Log.Default in TUI code to prevent prompt corruption

All console.log and console.error calls in TUI components and bridge
client wrote to stdout/stderr, which the TUI framework captures and
renders as garbled text in the prompt input area. Replace with
Log.Default.info/error which writes to the log file instead.

Affected: route navigation, bootstrap, theme resolution, route changes,
clipboard detection, session creation, auto-enhance, MCP toggle,
workspace creation, and altimate-engine stderr.

https://claude.ai/code/session_01MBVNhRX6XKTHqtZ6zS1i6w

* fix: resolve code review findings for TUI prompt corruption PR

- Move "workspace created" log inside success branch (was logging on failure)
- Use structured metadata in `client.ts` stderr handler instead of template string
- Downgrade noisy diagnostic logs to `debug` level (clipboard, theme, route, sync)
- Replace platform-dependent binaries (`echo`, `python3`, `tar`) with `process.execPath`
  in tests for CI portability
- Replace hollow Log test with real `Log.init({ print: false })` integration test
- Use regex matching instead of brittle exact-string matching in log assertion tests
- Add source-scanning regression tests for TUI files and `client.ts` to prevent
  future `console.log/error` regressions

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

---------

Co-authored-by: Claude <noreply@anthropic.com>

Author: anandgupta42

packages/opencode/src/altimate/bridge/client.ts

@@ -12,6 +12,7 @@ import path from "path"
 import { ensureEngine, enginePythonPath } from "./engine"
 import type { BridgeMethod, BridgeMethods } from "./protocol"
 import { Telemetry } from "../telemetry"
+import { Log } from "../../util/log"
 
 /** Resolve the Python interpreter to use for the engine sidecar.
  *  Exported for testing — not part of the public API. */
@@ -137,7 +138,7 @@ export namespace Bridge {
 
     child.stderr!.on("data", (data: Buffer) => {
       const msg = data.toString().trim()
-      if (msg) console.error(`[altimate-engine] ${msg}`)
+      if (msg) Log.Default.error("altimate-engine stderr", { message: msg })
     })
 
     child.on("exit", (code) => {

packages/opencode/src/altimate/bridge/engine.ts

@@ -16,7 +16,7 @@ import { existsSync } from "fs"
 import fs from "fs/promises"
 import path from "path"
 import { Global } from "../../global"
-import { UI } from "../../cli/ui"
+import { Log } from "../../util/log"
 import { Telemetry } from "@/telemetry"
 
 declare const ALTIMATE_ENGINE_VERSION: string
@@ -89,7 +89,7 @@ export async function ensureUv(): Promise<void> {
 
   const url = `https://github.com/astral-sh/uv/releases/latest/download/${asset}`
 
-  UI.println(`${UI.Style.TEXT_DIM}Downloading uv...${UI.Style.TEXT_NORMAL}`)
+  Log.Default.info("downloading uv")
 
   const dir = engineDir()
   await fs.mkdir(path.join(dir, "bin"), { recursive: true })
@@ -140,7 +140,7 @@ export async function ensureUv(): Promise<void> {
     await fs.chmod(uv, 0o755)
   }
 
-  UI.println(`${UI.Style.TEXT_SUCCESS}uv installed${UI.Style.TEXT_NORMAL}`)
+  Log.Default.info("uv installed")
 }
 
 /** Creates venv + installs altimate-engine. Upgrades on version mismatch.
@@ -170,7 +170,7 @@ async function ensureEngineImpl(): Promise<void> {
 
   // Create venv if it doesn't exist
   if (!existsSync(venvDir)) {
-    UI.println(`${UI.Style.TEXT_DIM}Creating Python environment...${UI.Style.TEXT_NORMAL}`)
+    Log.Default.info("creating python environment")
     try {
       execFileSync(uv, ["venv", "--python", "3.12", venvDir], { stdio: "pipe" })
     } catch (e: any) {
@@ -187,7 +187,7 @@ async function ensureEngineImpl(): Promise<void> {
 
   // Install/upgrade engine
   const pythonPath = enginePythonPath()
-  UI.println(`${UI.Style.TEXT_DIM}Installing altimate-engine ${ALTIMATE_ENGINE_VERSION}...${UI.Style.TEXT_NORMAL}`)
+  Log.Default.info("installing altimate-engine", { version: ALTIMATE_ENGINE_VERSION })
   try {
     execFileSync(uv, ["pip", "install", "--python", pythonPath, `altimate-engine==${ALTIMATE_ENGINE_VERSION}`], { stdio: "pipe" })
   } catch (e: any) {
@@ -224,7 +224,7 @@ async function ensureEngineImpl(): Promise<void> {
     duration_ms: Date.now() - startTime,
   })
 
-  UI.println(`${UI.Style.TEXT_SUCCESS}Engine ready${UI.Style.TEXT_NORMAL}`)
+  Log.Default.info("engine ready", { version: ALTIMATE_ENGINE_VERSION })
 }
 
 /** Returns current engine status */

packages/opencode/src/cli/cmd/tui/app.tsx

@@ -7,6 +7,7 @@ import { Switch, Match, createEffect, untrack, ErrorBoundary, createSignal, onMo
 import { win32DisableProcessedInput, win32FlushInputBuffer, win32InstallCtrlCGuard } from "./win32"
 import { Installation } from "@/installation"
 import { Flag } from "@/flag/flag"
+import { Log } from "@/util/log"
 import { DialogProvider, useDialog } from "@tui/ui/dialog"
 import { DialogProvider as DialogProviderList } from "@tui/component/dialog-provider"
 import { SDKProvider, useSDK } from "@tui/context/sdk"
@@ -239,7 +240,7 @@ export function tui(input: {
           keyBindings: [{ name: "y", ctrl: true, action: "copy-selection" }],
           onCopySelection: (text) => {
             Clipboard.copy(text).catch((error) => {
-              console.error(`Failed to copy console selection to clipboard: ${error}`)
+              Log.Default.error(`Failed to copy console selection to clipboard: ${error}`)
             })
           },
         },
@@ -306,7 +307,7 @@ function App() {
   const [terminalTitleEnabled, setTerminalTitleEnabled] = createSignal(kv.get("terminal_title_enabled", true))
 
   createEffect(() => {
-    console.log(JSON.stringify(route.data))
+    Log.Default.debug("route changed", { route: route.data })
   })
 
   // Update terminal window title based on current route and session

packages/opencode/src/cli/cmd/tui/component/dialog-mcp.tsx

@@ -7,6 +7,7 @@ import { useTheme } from "../context/theme"
 import { Keybind } from "@/util/keybind"
 import { TextAttributes } from "@opentui/core"
 import { useSDK } from "@tui/context/sdk"
+import { Log } from "@/util/log"
 
 function Status(props: { enabled: boolean; loading: boolean }) {
   const { theme } = useTheme()
@@ -61,10 +62,10 @@ export function DialogMcp() {
           if (status.data) {
             sync.set("mcp", status.data)
           } else {
-            console.error("Failed to refresh MCP status: no data returned")
+            Log.Default.error("Failed to refresh MCP status: no data returned")
           }
         } catch (error) {
-          console.error("Failed to toggle MCP:", error)
+          Log.Default.error("Failed to toggle MCP", { error })
         } finally {
           setLoading(null)
         }

packages/opencode/src/cli/cmd/tui/component/dialog-workspace-list.tsx

@@ -7,6 +7,7 @@ import type { Session } from "@opencode-ai/sdk/v2"
 import { useSDK } from "../context/sdk"
 import { useToast } from "../ui/toast"
 import { useKeybind } from "../context/keybind"
+import { Log } from "@/util/log"
 import { DialogSessionList } from "./workspace/dialog-session-list"
 import { createOpencodeClient } from "@opencode-ai/sdk/v2"
 
@@ -112,10 +113,9 @@ function DialogWorkspaceCreate(props: { onSelect: (workspaceID: string) => Promi
     setCreating(type)
 
     const result = await sdk.client.experimental.workspace.create({ type, branch: null }).catch((err) => {
-      console.log(err)
+      Log.Default.error("workspace creation failed", { error: err })
       return undefined
     })
-    console.log(JSON.stringify(result, null, 2))
     const workspace = result?.data
     if (!workspace) {
       setCreating(undefined)
@@ -125,6 +125,7 @@ function DialogWorkspaceCreate(props: { onSelect: (workspaceID: string) => Promi
       })
       return
     }
+    Log.Default.info("workspace created", { workspaceId: workspace.id })
     await sync.workspace.sync()
     await props.onSelect(workspace.id)
     setCreating(undefined)

packages/opencode/src/cli/cmd/tui/component/prompt/index.tsx

@@ -23,6 +23,7 @@ import { useExit } from "../../context/exit"
 import { Clipboard } from "../../util/clipboard"
 import type { FilePart } from "@opencode-ai/sdk/v2"
 import { TuiEvent } from "../../event"
+import { Log } from "@/util/log"
 import { iife } from "@/util/iife"
 import { Locale } from "@/util/locale"
 import { formatDuration } from "@/util/format"
@@ -603,7 +604,7 @@ export function Prompt(props: PromptProps) {
       })
 
       if (res.error) {
-        console.log("Creating a session failed:", res.error)
+        Log.Default.error("Creating a session failed", { error: res.error })
 
         toast.show({
           message: "Creating a session failed. Open console for more details.",
@@ -640,7 +641,7 @@ export function Prompt(props: PromptProps) {
         }
       } catch (err) {
         // Enhancement failure should never block prompt submission
-        console.error("auto-enhance failed, using original prompt", err)
+        Log.Default.error("auto-enhance failed, using original prompt", { error: err })
       } finally {
         enhancingInProgress = false
       }

packages/opencode/src/cli/cmd/tui/context/route.tsx

@@ -1,6 +1,7 @@
 import { createStore } from "solid-js/store"
 import { createSimpleContext } from "./helper"
 import type { PromptInfo } from "../component/prompt/history"
+import { Log } from "@/util/log"
 
 export type HomeRoute = {
   type: "home"
@@ -32,7 +33,7 @@ export const { use: useRoute, provider: RouteProvider } = createSimpleContext({
         return store
       },
       navigate(route: Route) {
-        console.log("navigate", route)
+        Log.Default.debug("navigate", { route })
         setStore(route)
       },
     }

packages/opencode/src/cli/cmd/tui/context/sync.tsx

@@ -356,7 +356,7 @@ export const { use: useSync, provider: SyncProvider } = createSimpleContext({
     const args = useArgs()
 
     async function bootstrap() {
-      console.log("bootstrapping")
+      Log.Default.debug("bootstrapping")
       const start = Date.now() - 30 * 24 * 60 * 60 * 1000
       const sessionListPromise = sdk.client.session
         .list({ start: start })

packages/opencode/src/cli/cmd/tui/context/theme.tsx

@@ -1,4 +1,5 @@
 import { SyntaxStyle, RGBA, type TerminalColors } from "@opentui/core"
+import { Log } from "@/util/log"
 import path from "path"
 import { createEffect, createMemo, onMount } from "solid-js"
 import { createSimpleContext } from "./helper"
@@ -319,13 +320,13 @@ export const { use: useTheme, provider: ThemeProvider } = createSimpleContext({
     onMount(init)
 
     function resolveSystemTheme() {
-      console.log("resolveSystemTheme")
+      Log.Default.debug("resolving system theme")
       renderer
         .getPalette({
           size: 16,
         })
         .then((colors) => {
-          console.log(colors.palette)
+          Log.Default.debug("system theme palette", { palette: colors.palette })
           if (!colors.palette[0]) {
             if (store.active === "system") {
               setStore(

packages/opencode/src/cli/cmd/tui/routes/session/index.tsx

@@ -13,6 +13,7 @@ import {
   useContext,
 } from "solid-js"
 import { Dynamic } from "solid-js/web"
+import { Log } from "@/util/log"
 import path from "path"
 import { useRoute, useRouteData } from "@tui/context/route"
 import { useSync } from "@tui/context/sync"
@@ -195,7 +196,7 @@ export function Session() {
         if (scroll) scroll.scrollBy(100_000)
       })
       .catch((e) => {
-        console.error(e)
+        Log.Default.error("session sync failed", { error: e })
         toast.show({
           message: `Session not found: ${route.sessionID}`,
           variant: "error",