From 76856599a7898078cd491139834afdfbd8a24e16 Mon Sep 17 00:00:00 2001
From: YeonGyu-Kim <code.yeon.gyu@gmail.com>
Date: Sat, 7 Feb 2026 17:34:29 +0900
Subject: [PATCH] fix(test): mock SDK and port-utils in integration test to
 prevent CI failure

The 'port with available port starts server' test was calling
createOpencode from the SDK which spawns an actual opencode binary.
CI environments don't have opencode installed, causing ENOENT.

Mock @opencode-ai/sdk and port-utils (same pattern as
server-connection.test.ts) so the test verifies integration
logic without requiring the binary.
---
 src/cli/run/integration.test.ts | 34 +++++++++++++++++++++++++++++++--
 1 file changed, 32 insertions(+), 2 deletions(-)

diff --git a/src/cli/run/integration.test.ts b/src/cli/run/integration.test.ts
index afa6c0490..1cbfa0847 100644
--- a/src/cli/run/integration.test.ts
+++ b/src/cli/run/integration.test.ts
@@ -3,9 +3,32 @@ import type { RunResult } from "./types"
 import { createJsonOutputManager } from "./json-output"
 import { resolveSession } from "./session-resolver"
 import { executeOnCompleteHook } from "./on-complete-hook"
-import { createServerConnection } from "./server-connection"
 import type { OpencodeClient } from "./types"
 
+const mockServerClose = mock(() => {})
+const mockCreateOpencode = mock(() =>
+  Promise.resolve({
+    client: { session: {} },
+    server: { url: "http://127.0.0.1:9999", close: mockServerClose },
+  })
+)
+const mockCreateOpencodeClient = mock(() => ({ session: {} }))
+const mockIsPortAvailable = mock(() => Promise.resolve(true))
+const mockGetAvailableServerPort = mock(() => Promise.resolve({ port: 9999, wasAutoSelected: false }))
+
+mock.module("@opencode-ai/sdk", () => ({
+  createOpencode: mockCreateOpencode,
+  createOpencodeClient: mockCreateOpencodeClient,
+}))
+
+mock.module("../../shared/port-utils", () => ({
+  isPortAvailable: mockIsPortAvailable,
+  getAvailableServerPort: mockGetAvailableServerPort,
+  DEFAULT_SERVER_PORT: 4096,
+}))
+
+const { createServerConnection } = await import("./server-connection")
+
 interface MockWriteStream {
   write: (chunk: string) => boolean
   writes: string[]
@@ -228,6 +251,9 @@ describe("integration: server connection", () => {
 
   beforeEach(() => {
     consoleSpy = spyOn(console, "log").mockImplementation(() => {})
+    mockCreateOpencode.mockClear()
+    mockCreateOpencodeClient.mockClear()
+    mockServerClose.mockClear()
   })
 
   afterEach(() => {
@@ -245,11 +271,13 @@ describe("integration: server connection", () => {
     // then
     expect(result.client).toBeDefined()
     expect(result.cleanup).toBeDefined()
+    expect(mockCreateOpencodeClient).toHaveBeenCalledWith({ baseUrl: attachUrl })
     result.cleanup()
+    expect(mockServerClose).not.toHaveBeenCalled()
   })
 
   it("port with available port starts server", async () => {
-    // given - assuming port is available
+    // given
     const signal = new AbortController().signal
     const port = 9999
 
@@ -259,6 +287,8 @@ describe("integration: server connection", () => {
     // then
     expect(result.client).toBeDefined()
     expect(result.cleanup).toBeDefined()
+    expect(mockCreateOpencode).toHaveBeenCalled()
     result.cleanup()
+    expect(mockServerClose).toHaveBeenCalled()
   })
 })