snow-flow 10.0.129 → 10.0.131

package/package.json

@@ -1,6 +1,6 @@
 {
   "$schema": "https://json.schemastore.org/package.json",
-  "version": "10.0.129",
+  "version": "10.0.131",
   "name": "snow-flow",
   "description": "Snow-Flow - ServiceNow Multi-Agent Development Framework powered by AI",
   "license": "Elastic-2.0",

package/src/cli/cmd/tui/app.tsx

@@ -116,15 +116,20 @@ export function tui(input: {
 }) {
   _onUpgrade = input.onUpgrade
   const skipThemeDetection = !!process.env.OPENCODE_SKIP_THEME_DETECTION || !!process.env.OPENCODE_REMOTE_TUI
+  // In hosted mode, use stderr for pre-render logs to keep terminal clean for the renderer
+  const forceThread = !!process.env.OTUI_FORCE_THREAD
+  const log = forceThread
+    ? (msg: string) => process.stderr.write(`${msg}\n`)
+    : (msg: string) => console.log(msg)
   // promise to prevent immediate exit
   return new Promise<void>(async (resolve) => {
     let mode: "dark" | "light" = "dark"
     if (skipThemeDetection) {
-      console.log("[snow-flow] skipping theme detection")
+      log("[snow-flow] skipping theme detection")
     } else {
-      console.log("[snow-flow] detecting theme...")
+      log("[snow-flow] detecting theme...")
       mode = await getTerminalBackgroundColor()
-      console.log(`[snow-flow] theme: ${mode}`)
+      log(`[snow-flow] theme: ${mode}`)
     }
     const onExit = async () => {
       await input.onExit?.()
@@ -146,16 +151,11 @@ export function tui(input: {
       },
     } as Parameters<typeof render>[1]
 
-    // On Linux, @opentui disables Zig renderer threading which prevents PTY output.
-    // OTUI_FORCE_THREAD=1 temporarily spoofs platform so @opentui keeps threading enabled.
-    const forceThread = !!process.env.OTUI_FORCE_THREAD
-    if (forceThread) {
-      console.log("[snow-flow] forcing renderer threading (OTUI_FORCE_THREAD)")
-      Object.defineProperty(process, "platform", { value: "darwin", configurable: true })
-    }
+    // On Linux, @opentui sets useThread=false which may prevent the Zig renderer
+    // from flushing output. HostedRendererDiag handles starting the loop and flushing.
 
     try {
-      console.log("[snow-flow] starting tui...")
+      log("[snow-flow] starting tui...")
       await render(
         () => {
           return (
@@ -220,49 +220,47 @@ function HostedRendererDiag() {
   onMount(() => {
     const r = renderer as any
     const log = (msg: string) => process.stderr.write(`[sf-diag] ${msg}\n`)
-    log(`useThread=${r._useThread} isRunning=${r._isRunning} state=${r._controlState} live=${r.liveRequestCounter}`)
-    log(`platform=${process.platform} altScreen=${r._useAlternateScreen}`)
-    log(`termSetup=${r._terminalIsSetup} root=${!!r.root} buffer=${!!r.nextRenderBuffer}`)
+    log(`useThread=${r._useThread} isRunning=${r._isRunning} state=${r._controlState}`)
+    log(`platform=${process.platform} altScreen=${r._useAlternateScreen} termSetup=${r._terminalIsSetup}`)
+
+    // On Linux, useThread=false means Zig render() buffers ANSI but doesn't write to fd 1.
+    // Patch renderNative to manually flush the Zig stdout buffer after each render frame.
+    const realWrite = r.realStdoutWrite
+    const stdout = r.stdout
+    if (realWrite && r.lib && r.rendererPtr) {
+      const origRenderNative = r.renderNative.bind(r)
+      r.renderNative = () => {
+        origRenderNative()
+        // Try to flush Zig's internal stdout buffer by calling writeOut with empty data
+        try {
+          r.lib.writeOut(r.rendererPtr, new Uint8Array(0).buffer, 0)
+        } catch {}
+      }
+      log("patched renderNative with flush")
+    }
 
-    // Try 1: force-start the render loop if not running
+    // Clear screen and start the render loop
+    if (realWrite) {
+      realWrite.call(stdout, "\x1b[2J\x1b[H")
+    }
     if (!r._isRunning) {
-      log("calling renderer.start()")
       r.start()
-      log(`after start: isRunning=${r._isRunning} state=${r._controlState}`)
+      log(`started: isRunning=${r._isRunning} state=${r._controlState}`)
     }
 
-    // Try 2: after 2s, check state and try manual output
+    // After 3s, log state and try direct fd write test
     setTimeout(() => {
-      log(`2s check: isRunning=${r._isRunning} state=${r._controlState} rendering=${r.rendering}`)
-      // Test direct write via realStdoutWrite (saved reference to actual stdout.write)
-      const realWrite = r.realStdoutWrite
-      if (realWrite) {
-        realWrite.call(r.stdout, "\x1b[2J\x1b[H\x1b[1;31m[sf-diag] REAL STDOUT WRITE OK\x1b[0m\r\n")
-        log("realStdoutWrite sent")
-      }
-      // Test lib.writeOut
+      log(`3s: isRunning=${r._isRunning} rendering=${r.rendering}`)
+      // Direct write to fd 1 to verify terminal connectivity
       try {
-        r.writeOut("\x1b[3;1H\x1b[1;32m[sf-diag] WRITEOUT OK\x1b[0m\r\n")
-        log("writeOut sent")
-      } catch (e: any) {
-        log(`writeOut error: ${e.message}`)
-      }
-    }, 2000)
-
-    // Try 3: after 4s, try manual render cycle
-    setTimeout(() => {
-      log(`4s check: isRunning=${r._isRunning} rendering=${r.rendering}`)
-      try {
-        if (r.root && r.nextRenderBuffer) {
-          r.root.render(r.nextRenderBuffer, 0)
-          log("root.render done")
-        }
-        r.renderNative()
-        log("renderNative done")
-      } catch (e: any) {
-        log(`manual render error: ${e.message}`)
+        const fd1 = Bun.file("/dev/fd/1")
+        Bun.write(fd1, "\x1b[5;1H\x1b[1;31m[sf-diag] FD1 WRITE TEST\x1b[0m\r\n").catch(() => {})
+      } catch {}
+      // Also try realStdoutWrite
+      if (realWrite) {
+        realWrite.call(stdout, "\x1b[7;1H\x1b[1;32m[sf-diag] REAL WRITE TEST\x1b[0m\r\n")
       }
-    }, 4000)
+    }, 3000)
   })
   return null
 }