test(mcp-cli): add binary integration tests

- Add tests for mcp-cli --help, --version, and -c flag
- Fix loader to respect HOME env var for testability
- Tests skip gracefully if mcp-cli not installed

Author: ReinaMacCredy

src/features/mcp-cli-loader/loader.test.ts

@@ -1,10 +1,19 @@
 import { describe, it, expect, beforeEach, afterEach } from "bun:test"
 import { mkdtempSync, mkdirSync, rmSync, writeFileSync } from "node:fs"
 import { join } from "node:path"
-import { tmpdir } from "node:os"
+import { tmpdir, homedir } from "node:os"
 import { loadMcpCliConfigs } from "./loader"
 import type { McpCliConfig } from "./types"
 
+function isMcpCliInstalled(): boolean {
+  try {
+    const result = Bun.spawnSync(["which", "mcp-cli"])
+    return result.exitCode === 0
+  } catch {
+    return false
+  }
+}
+
 describe("mcp_servers.json config loader", () => {
   let tempDir: string
   let originalCwd: string
@@ -165,3 +174,55 @@ describe("mcp_servers.json config loader", () => {
     expect(result.loadedServers).toHaveLength(1)
   })
 })
+
+describe("mcp-cli binary integration", () => {
+  const MCP_CLI_INSTALLED = isMcpCliInstalled()
+
+  it.skipIf(!MCP_CLI_INSTALLED)("should show help with --help flag", async () => {
+    // #given - mcp-cli is installed
+
+    // #when - run mcp-cli --help
+    const proc = Bun.spawn(["mcp-cli", "--help"], {
+      stdout: "pipe",
+      stderr: "pipe",
+    })
+    const stdout = await new Response(proc.stdout).text()
+    const exitCode = await proc.exited
+
+    // #then - should show usage info
+    expect(exitCode).toBe(0)
+    expect(stdout).toContain("mcp-cli")
+  })
+
+  it.skipIf(!MCP_CLI_INSTALLED)("should show version with --version flag", async () => {
+    // #given - mcp-cli is installed
+
+    // #when - run mcp-cli --version
+    const proc = Bun.spawn(["mcp-cli", "--version"], {
+      stdout: "pipe",
+      stderr: "pipe",
+    })
+    const stdout = await new Response(proc.stdout).text()
+    const exitCode = await proc.exited
+
+    // #then - should show version
+    expect(exitCode).toBe(0)
+    expect(stdout).toMatch(/\d+\.\d+\.\d+/)
+  })
+
+  it.skipIf(!MCP_CLI_INSTALLED)("should accept -c flag for config path", async () => {
+    // #given - a non-existent config path
+    const fakePath = "/tmp/nonexistent-mcp-config.json"
+
+    // #when - run mcp-cli with -c flag (will error but shows flag is recognized)
+    const proc = Bun.spawn(["mcp-cli", "-c", fakePath], {
+      stdout: "pipe",
+      stderr: "pipe",
+    })
+    const stderr = await new Response(proc.stderr).text()
+    await proc.exited
+
+    // #then - should mention config file in error
+    expect(stderr).toContain("Config")
+  })
+})

src/features/mcp-cli-loader/loader.ts

@@ -16,7 +16,7 @@ interface McpCliConfigPath {
 
 function getMcpCliConfigPaths(): McpCliConfigPath[] {
   const cwd = process.cwd()
-  const home = homedir()
+  const home = process.env.HOME || homedir()
 
   return [
     { path: join(home, ".config", "opencode", "mcp_servers.json"), scope: "user" },