fix: address cubic 4/5 review issues

- Preserve encoding/callback args in stdout.write wrapper (json-output.ts)
- Restore global console spy in afterEach (server-connection.test.ts)
- Restore console.error spy in afterEach (on-complete-hook.test.ts)

src/cli/run/json-output.ts

@@ -20,9 +20,9 @@ export function createJsonOutputManager(
   const originalWrite = stdout.write.bind(stdout)
 
   function redirectToStderr(): void {
-    stdout.write = function (chunk: string): boolean {
-      return stderr.write(chunk)
-    }
+    stdout.write = function (...args: Parameters<NodeJS.WriteStream["write"]>): boolean {
+      return (stderr.write as Function).apply(stderr, args)
+    } as NodeJS.WriteStream["write"]
   }
 
   function restore(): void {

src/cli/run/on-complete-hook.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, spyOn, beforeEach } from "bun:test"
+import { describe, it, expect, spyOn, beforeEach, afterEach } from "bun:test"
 import { executeOnCompleteHook } from "./on-complete-hook"
 
 describe("executeOnCompleteHook", () => {
@@ -9,8 +9,14 @@ describe("executeOnCompleteHook", () => {
     } as unknown as ReturnType<typeof Bun.spawn>
   }
 
+  let consoleErrorSpy: ReturnType<typeof spyOn<typeof console, "error">>
+
   beforeEach(() => {
-    spyOn(console, "error").mockImplementation(() => {})
+    consoleErrorSpy = spyOn(console, "error").mockImplementation(() => {})
+  })
+
+  afterEach(() => {
+    consoleErrorSpy.mockRestore()
   })
 
   it("executes command with correct env vars", async () => {

src/cli/run/server-connection.test.ts

@@ -1,5 +1,7 @@
 import { describe, it, expect, mock, beforeEach, afterEach } from "bun:test"
 
+const originalConsole = globalThis.console
+
 const mockServerClose = mock(() => {})
 const mockCreateOpencode = mock(() =>
   Promise.resolve({
@@ -36,6 +38,10 @@ describe("createServerConnection", () => {
     globalThis.console = { ...console, log: mockConsoleLog } as typeof console
   })
 
+  afterEach(() => {
+    globalThis.console = originalConsole
+  })
+
   it("attach mode returns client with no-op cleanup", async () => {
     // given
     const signal = new AbortController().signal