oh-my-opencode 5.0.0-beta.33 → 5.0.0-beta.35

package/packages/omo-codex/plugin/components/codegraph/test/serve-provision.test.ts

@@ -1,147 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-
-import { runCodegraphServe } from "../src/serve.ts";
-
-describe("runCodegraphServe provisioning", () => {
-	it("#given CodeGraph is unresolved and daemon is omitted #when serving MCP #then provisions CodeGraph with daemon-on", async () => {
-		// given
-		const binPath = join("/tmp/home/.omo/codegraph", "bin", "codegraph");
-		const calls: Array<{
-			readonly args: readonly string[];
-			readonly command: string;
-			readonly env: Record<string, string | undefined>;
-		}> = [];
-		const stderr: string[] = [];
-
-		// when
-		const exitCode = await runCodegraphServe({
-			config: { codegraph: { enabled: true }, sources: [], warnings: [] },
-			env: { PATH: "/bin" },
-			homeDir: "/tmp/home",
-			nodeVersion: "22.14.0",
-			resolve: () => ({ argsPrefix: [], command: "codegraph", exists: false, source: "path" }),
-			ensureProvisioned: (options) =>
-				Promise.resolve({
-					binPath: join(options.installDir ?? "/tmp/home/.omo/codegraph", "bin", "codegraph"),
-					provisioned: true,
-				}),
-			runProcess: (command, args, options) => {
-				calls.push({ args, command, env: options.env });
-				return Promise.resolve(0);
-			},
-			stderr: { write: (chunk: string) => stderr.push(chunk) },
-		});
-
-		// then
-		expect(exitCode).toBe(0);
-		expect(stderr).toEqual([]);
-		expect(calls).toEqual([
-			{
-				args: ["serve", "--mcp"],
-				command: binPath,
-				env: {
-					CODEGRAPH_INSTALL_DIR: join("/tmp/home", ".omo", "codegraph"),
-					CODEGRAPH_NO_DOWNLOAD: "1",
-					CODEGRAPH_TELEMETRY: "0",
-					DO_NOT_TRACK: "1",
-					PATH: "/bin",
-				},
-			},
-		]);
-	});
-
-	it("#given a bundled runtime and a stale managed install #when serving MCP #then it upgrades and uses the managed runtime", async () => {
-		// given
-		const homeDir = mkdtempSync(join(tmpdir(), "omo-codegraph-serve-managed-upgrade-"));
-		const installDir = join(homeDir, ".omo", "codegraph");
-		const binPath = join(installDir, "bin", "codegraph");
-		mkdirSync(join(installDir, "bin"), { recursive: true });
-		mkdirSync(join(installDir, ".provisioned"), { recursive: true });
-		writeFileSync(binPath, "stale codegraph\n");
-		writeFileSync(
-			join(installDir, ".provisioned", "codegraph-1.4.1.json"),
-			`${JSON.stringify({ binPath, version: "1.4.1" })}\n`,
-		);
-		const provisionCalls: unknown[] = [];
-		const processCalls: Array<{ readonly args: readonly string[]; readonly command: string }> = [];
-
-		try {
-			// when
-			const exitCode = await runCodegraphServe({
-				config: { codegraph: { enabled: true }, sources: [], warnings: [] },
-				homeDir,
-				managedInstallExists: () => true,
-				nodeVersion: "22.14.0",
-				resolve: () => ({ argsPrefix: ["codegraph.js"], command: "/opt/node22/bin/node", exists: true, source: "bundled" }),
-				resolveManagedBin: () => null,
-				ensureProvisioned: (options) => {
-					provisionCalls.push(options);
-					return Promise.resolve({ binPath, provisioned: true });
-				},
-				runProcess: (command, args) => {
-					processCalls.push({ args, command });
-					return Promise.resolve(0);
-				},
-			});
-
-			// then
-			expect(exitCode).toBe(0);
-			expect(provisionCalls).toEqual([{ installDir, lockDir: join(installDir, ".locks"), version: "1.5.0" }]);
-			expect(processCalls).toEqual([{ args: ["serve", "--mcp"], command: binPath }]);
-		} finally {
-			rmSync(homeDir, { force: true, recursive: true });
-		}
-	});
-
-	it("#given CodeGraph is unresolved and codegraph.daemon=false #when serving MCP #then provisions CodeGraph with daemon-off", async () => {
-		// given
-		const binPath = join("/tmp/home/.omo/codegraph", "bin", "codegraph");
-		const calls: Array<{
-			readonly args: readonly string[];
-			readonly command: string;
-			readonly env: Record<string, string | undefined>;
-		}> = [];
-		const stderr: string[] = [];
-
-		// when
-		const exitCode = await runCodegraphServe({
-			config: { codegraph: { daemon: false, enabled: true }, sources: [], warnings: [] },
-			env: { PATH: "/bin" },
-			homeDir: "/tmp/home",
-			nodeVersion: "22.14.0",
-			resolve: () => ({ argsPrefix: [], command: "codegraph", exists: false, source: "path" }),
-			ensureProvisioned: (options) =>
-				Promise.resolve({
-					binPath: join(options.installDir ?? "/tmp/home/.omo/codegraph", "bin", "codegraph"),
-					provisioned: true,
-				}),
-			runProcess: (command, args, options) => {
-				calls.push({ args, command, env: options.env });
-				return Promise.resolve(0);
-			},
-			stderr: { write: (chunk: string) => stderr.push(chunk) },
-		});
-
-		// then
-		expect(exitCode).toBe(0);
-		expect(stderr).toEqual([]);
-		expect(calls).toEqual([
-			{
-				args: ["serve", "--mcp"],
-				command: binPath,
-				env: {
-					CODEGRAPH_INSTALL_DIR: join("/tmp/home", ".omo", "codegraph"),
-					CODEGRAPH_NO_DAEMON: "1",
-					CODEGRAPH_NO_DOWNLOAD: "1",
-					CODEGRAPH_TELEMETRY: "0",
-					DO_NOT_TRACK: "1",
-					PATH: "/bin",
-				},
-			},
-		]);
-		expect(calls[0]?.env["CODEGRAPH_NO_DAEMON"]).toBe("1");
-	});
-});

package/packages/omo-codex/plugin/components/codegraph/test/serve-unavailable-idle.test.ts

@@ -1,47 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import { PassThrough } from "node:stream";
-
-import { runUnavailableCodegraphMcpServer } from "../src/mcp-unavailable.ts";
-
-describe("unavailable codegraph MCP idle teardown", () => {
-	it("#given the codex host (no respawn for exited stdio servers) #when the unavailable stub starts #then the idle timeout is disabled", async () => {
-		// given — codex never respawns a user-configured stdio MCP server
-		// (codex-rs reconnects only its own codex_apps server), so if the stub
-		// inherited the mcp-stdio-core default idle timeout it would destroy its
-		// own stdin after 10 idle minutes and every later codegraph tool call in
-		// a long-lived session would fail on a dead connection instead of
-		// answering with the skip reason.
-		const stdin = new PassThrough();
-		const stdout = new PassThrough();
-		stdout.resume();
-		const lifecycle: Array<{ readonly event: string; readonly data?: unknown }> = [];
-
-		try {
-			// when
-			const served = runUnavailableCodegraphMcpServer({
-				input: stdin,
-				output: stdout,
-				reason: "CodeGraph MCP skipped: test reason",
-				serverVersion: "0.0.0-test",
-				lifecycleLog: (event, data) => {
-					lifecycle.push({ event, data });
-				},
-			});
-			const outcome = await Promise.race([
-				served.then(() => "settled" as const),
-				Bun.sleep(500).then(() => "parked" as const),
-			]);
-
-			// then — the server must park on the held-open pipe (no idle timer
-			// tore it down) and must have started with the timeout disabled.
-			expect(outcome).toBe("parked");
-			expect(lifecycle).toContainEqual({
-				event: "stdio_started",
-				data: expect.objectContaining({ idle_timeout_ms: 0 }),
-			});
-		} finally {
-			stdin.destroy();
-			stdout.destroy();
-		}
-	});
-});

package/packages/omo-codex/plugin/components/codegraph/test/serve-unavailable.test.ts

@@ -1,175 +0,0 @@
-import { describe, expect, it } from "bun:test";
-import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { PassThrough } from "node:stream";
-
-import { runCodegraphServe } from "../src/serve.ts";
-
-describe("runCodegraphServe unavailable CodeGraph paths", () => {
-	it("#given CodeGraph is unresolved #when serving MCP #then exposes an empty facade with a skip hint", async () => {
-		// given
-		const stderr: string[] = [];
-		const spawned: string[] = [];
-
-		// when
-		const exitCode = await runCodegraphServe({
-			...closedMcpStdio(),
-			config: { codegraph: { auto_provision: false, enabled: true }, sources: [], warnings: [] },
-			env: { PATH: "/bin" },
-			nodeVersion: "22.14.0",
-			buildEnv: () => ({}),
-			resolve: () => ({ argsPrefix: [], command: "codegraph", exists: false, source: "path" }),
-			runProcess: (command: string) => {
-				spawned.push(command);
-				return Promise.resolve(0);
-			},
-			stderr: { write: (chunk: string) => stderr.push(chunk) },
-		});
-
-		// then
-		expect(exitCode).toBe(0);
-		expect(spawned).toEqual([]);
-		expect(stderr).toEqual([
-			"CodeGraph MCP skipped: codegraph binary not found. Install CodeGraph or set OMO_CODEGRAPH_BIN.\n",
-		]);
-	});
-
-	it("#given an unsupported local Node #when serving MCP #then exposes an empty facade without spawning codegraph", async () => {
-		// given
-		const stderr: string[] = [];
-		const spawned: string[] = [];
-
-		// when
-		const exitCode = await runCodegraphServe({
-			...closedMcpStdio(),
-			env: {},
-			nodeVersion: "26.3.0",
-			buildEnv: () => ({}),
-			resolve: () => ({ argsPrefix: [], command: "codegraph", exists: true, source: "path" }),
-			runProcess: (command: string) => {
-				spawned.push(command);
-				return Promise.resolve(0);
-			},
-			stderr: { write: (chunk: string) => stderr.push(chunk) },
-		});
-
-		// then
-		expect(exitCode).toBe(0);
-		expect(spawned).toEqual([]);
-		expect(stderr).toHaveLength(1);
-		expect(stderr[0]).toContain("CodeGraph MCP skipped");
-		expect(stderr[0]).toContain("CODEGRAPH_ALLOW_UNSAFE_NODE");
-	});
-
-	it("#given OMO_CODEGRAPH_BIN points at a missing path #when serving MCP #then exposes an empty facade before spawn", async () => {
-		// given
-		const stderr: string[] = [];
-		const spawned: string[] = [];
-
-		// when
-		const exitCode = await runCodegraphServe({
-			...closedMcpStdio(),
-			buildEnv: () => ({}),
-			commandExists: () => false,
-			resolve: () => ({ argsPrefix: [], command: "/nonexistent", exists: true, source: "env" }),
-			runProcess: (command: string) => {
-				spawned.push(command);
-				return Promise.resolve(0);
-			},
-			stderr: { write: (chunk: string) => stderr.push(chunk) },
-		});
-
-		// then
-		expect(exitCode).toBe(0);
-		expect(spawned).toEqual([]);
-		expect(stderr).toEqual([
-			"CodeGraph MCP skipped: codegraph binary not found. Install CodeGraph or set OMO_CODEGRAPH_BIN.\n",
-		]);
-	});
-
-	it("#given Codex SOT disables CodeGraph #when serving MCP #then exposes an empty facade with a disabled hint", async () => {
-		// given
-		const homeDir = mkdtempSync(join(tmpdir(), "omo-codegraph-serve-disabled-home-"));
-		const workspace = mkdtempSync(join(tmpdir(), "omo-codegraph-serve-disabled-workspace-"));
-		const stderr: string[] = [];
-		const spawned: string[] = [];
-		try {
-			mkdirSync(join(homeDir, ".omo"), { recursive: true });
-			mkdirSync(join(workspace, ".omo"), { recursive: true });
-			writeFileSync(join(homeDir, ".omo", "omo.jsonc"), '{ "codegraph": { "enabled": true } }\n');
-			writeFileSync(join(workspace, ".omo", "omo.jsonc"), '{ "[codex]": { "codegraph": { "enabled": false } } }\n');
-
-			// when
-			const exitCode = await runCodegraphServe({
-				...closedMcpStdio(),
-				cwd: workspace,
-				env: { HOME: homeDir },
-				runProcess: (command: string) => {
-					spawned.push(command);
-					return Promise.resolve(0);
-				},
-				stderr: { write: (chunk: string) => stderr.push(chunk) },
-			});
-
-			// then
-			expect(exitCode).toBe(0);
-			expect(spawned).toEqual([]);
-			expect(stderr).toEqual([
-				"CodeGraph MCP skipped: disabled by OMO SOT config. Set [codex].codegraph.enabled=true to enable it.\n",
-			]);
-		} finally {
-			rmSync(homeDir, { recursive: true, force: true });
-			rmSync(workspace, { recursive: true, force: true });
-		}
-	});
-
-	it("#given the unavailable facade with held-open stdio and a dead parent #when the watchdog polls #then the placeholder server settles without waiting for stdin EOF", async () => {
-		// given
-		// win32: the real-timer watchdog + PassThrough.destroy() teardown race leaves
-		// the read loop's async iterator pending on Windows, so this held-open-stdin
-		// case cannot settle under bun:test (which has no fake timers). The watchdog
-		// settle path IS covered deterministically by mcp-stdio-core's fake-timer test
-		// (passes on win32) and by real-process QA (task-13-*); production settles via
-		// stdin EOF, exercised by the other tests in this file that run on win32.
-		if (process.platform === "win32") return;
-		// Client stdio is held open and never sees EOF, so the parent-liveness
-		// watchdog is the only settle path for the placeholder server.
-		const stdin = new PassThrough();
-		const stdout = new PassThrough();
-		stdout.resume();
-		const stderr: string[] = [];
-
-		try {
-			// when
-			const exitCode = await runCodegraphServe({
-				config: { codegraph: { enabled: false }, sources: [], warnings: [] },
-				env: {},
-				stdin,
-				stdout,
-				stderr: { write: (chunk: string) => stderr.push(chunk) },
-				parentWatchdog: { pollIntervalMs: 10, probeAlive: () => false },
-			});
-
-			// then
-			expect(exitCode).toBe(0);
-			expect(stderr).toEqual([
-				"CodeGraph MCP skipped: disabled by OMO SOT config. Set [codex].codegraph.enabled=true to enable it.\n",
-			]);
-		} finally {
-			// The held-open streams are never ended by the test; destroy them so the
-			// bun test runner's event loop can drain and exit (an undestroyed
-			// PassThrough keeps the process alive on win32 and hangs the suite).
-			stdin.destroy();
-			stdout.destroy();
-		}
-	});
-});
-
-function closedMcpStdio(): { readonly stdin: PassThrough; readonly stdout: PassThrough } {
-	const stdin = new PassThrough();
-	const stdout = new PassThrough();
-	stdout.resume();
-	stdin.end();
-	return { stdin, stdout };
-}