test: fix Windows CI failures in install and bridge tests

- Replace `/bin/echo` with `process.execPath` in bridge test — /bin/echo
  does not exist on Windows, causing ENOENT on spawn; process.execPath
  (the current Bun binary) exists on all platforms and exits quickly
  without speaking JSON-RPC as expected
- Add `unixtest` guard to postinstall, bin-wrapper, and integration tests
  — on Windows, postinstall.mjs takes a different early-exit path that
  skips hard-link setup; dummy binaries are Unix shell scripts that
  cannot be executed on Windows; skip all Unix-specific test paths using
  the same `process.platform !== "win32" ? test : test.skip` pattern
  already used in fsmonitor.test.ts

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: anandgupta42

packages/opencode/test/bridge/client.test.ts

@@ -140,9 +140,10 @@ describe("Bridge.start integration", () => {
   test("ensureEngine is called when bridge starts", async () => {
     const { Bridge } = await import("../../src/altimate/bridge/client")
 
-    // /bin/echo exists and will spawn successfully but won't respond to
-    // the JSON-RPC ping, so start() will eventually fail on verification.
-    process.env.OPENCODE_PYTHON = "/bin/echo"
+    // process.execPath (the current Bun/Node binary) exists on all platforms.
+    // When spawned as a Python replacement it exits quickly without speaking
+    // JSON-RPC, so start() fails on the ping verification as expected.
+    process.env.OPENCODE_PYTHON = process.execPath
 
     try {
       await Bridge.call("ping", {} as any)

packages/opencode/test/install/bin-wrapper.test.ts

@@ -11,6 +11,10 @@ import {
   CURRENT_ARCH,
 } from "./fixture"
 
+// Dummy binaries created by the fixture are Unix shell scripts (#!/bin/sh).
+// Tests that run these binaries can only pass on Unix platforms.
+const unixtest = process.platform !== "win32" ? test : test.skip
+
 let cleanup: (() => void) | undefined
 
 afterEach(() => {
@@ -27,7 +31,7 @@ function copyBinWrapper(destDir: string): string {
 }
 
 describe("bin/altimate-code wrapper", () => {
-  test("uses ALTIMATE_CODE_BIN_PATH env var when set", () => {
+  unixtest("uses ALTIMATE_CODE_BIN_PATH env var when set", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 
@@ -41,7 +45,7 @@ describe("bin/altimate-code wrapper", () => {
     expect(result.stdout).toContain("altimate-code-test-ok")
   })
 
-  test("uses cached .opencode when present", () => {
+  unixtest("uses cached .opencode when present", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 
@@ -54,7 +58,7 @@ describe("bin/altimate-code wrapper", () => {
     expect(result.stdout).toContain("altimate-code-test-ok")
   })
 
-  test("finds binary in sibling node_modules package", () => {
+  unixtest("finds binary in sibling node_modules package", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 
@@ -73,7 +77,7 @@ describe("bin/altimate-code wrapper", () => {
     expect(result.stdout).toContain("altimate-code-test-ok")
   })
 
-  test("finds binary in parent node_modules (hoisted)", () => {
+  unixtest("finds binary in parent node_modules (hoisted)", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 

packages/opencode/test/install/integration.test.ts

@@ -11,6 +11,11 @@ import {
   CURRENT_PLATFORM,
 } from "./fixture"
 
+// These integration tests combine postinstall (Unix hard-link path) with
+// bin-wrapper execution of a Unix shell-script dummy binary. Skip on Windows
+// where both behave differently.
+const unixtest = process.platform !== "win32" ? test : test.skip
+
 let cleanup: (() => void) | undefined
 
 afterEach(() => {
@@ -19,7 +24,7 @@ afterEach(() => {
 })
 
 describe("install pipeline integration", () => {
-  test("full flow: layout -> postinstall -> bin wrapper executes dummy binary", () => {
+  unixtest("full flow: layout -> postinstall -> bin wrapper executes dummy binary", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 
@@ -44,7 +49,7 @@ describe("install pipeline integration", () => {
     expect(wrapperResult.stdout).toContain("altimate-code-test-ok")
   })
 
-  test("missing optional dep: postinstall fails, bin wrapper also fails gracefully", () => {
+  unixtest("missing optional dep: postinstall fails, bin wrapper also fails gracefully", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 
@@ -70,7 +75,7 @@ describe("install pipeline integration", () => {
     expect(wrapperResult.stderr).toContain("package manager failed to install")
   })
 
-  test("wrong-platform-only install: both scripts fail with clear errors", () => {
+  unixtest("wrong-platform-only install: both scripts fail with clear errors", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 

packages/opencode/test/install/postinstall.test.ts

@@ -9,6 +9,11 @@ import {
   CURRENT_PLATFORM,
 } from "./fixture"
 
+// On Windows, postinstall.mjs takes a different early-exit path that skips
+// binary setup entirely (the .exe is packaged separately). Skip all tests that
+// depend on the Unix hard-link / error-path behavior.
+const unixtest = process.platform !== "win32" ? test : test.skip
+
 let cleanup: (() => void) | undefined
 
 afterEach(() => {
@@ -17,7 +22,7 @@ afterEach(() => {
 })
 
 describe("postinstall.mjs", () => {
-  test("finds binary and creates hard link in bin/", () => {
+  unixtest("finds binary and creates hard link in bin/", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 
@@ -34,7 +39,7 @@ describe("postinstall.mjs", () => {
     expect(stat.mode & 0o111).toBeGreaterThan(0)
   })
 
-  test("replaces existing stale binary", () => {
+  unixtest("replaces existing stale binary", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 
@@ -54,7 +59,7 @@ describe("postinstall.mjs", () => {
     expect(content).toContain("altimate-code-test-ok")
   })
 
-  test("creates bin/ dir if missing", () => {
+  unixtest("creates bin/ dir if missing", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 
@@ -84,7 +89,7 @@ describe("postinstall.mjs", () => {
     expect(result.stdout).toContain("altimate-code v2.5.0 installed")
   })
 
-  test("exits 1 when platform binary package is missing", () => {
+  unixtest("exits 1 when platform binary package is missing", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 
@@ -96,7 +101,7 @@ describe("postinstall.mjs", () => {
     expect(result.stderr).toContain("Failed to setup altimate-code binary")
   })
 
-  test("exits 1 when package exists but binary file is missing", () => {
+  unixtest("exits 1 when package exists but binary file is missing", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c
 
@@ -108,7 +113,7 @@ describe("postinstall.mjs", () => {
     expect(result.stderr).toContain("Failed to setup altimate-code binary")
   })
 
-  test("exits 1 when only wrong-platform package is present", () => {
+  unixtest("exits 1 when only wrong-platform package is present", () => {
     const { dir, cleanup: c } = installTmpdir()
     cleanup = c