openAgent

1import { describe, expect, test } from "bun:test"
2import path from "path"
3import { Shell } from "@opencode-ai/core/shell"
4import { FSUtil } from "@opencode-ai/core/fs-util"
5import { which } from "@opencode-ai/core/util/which"
6 
7const withShell = async (shell: string | undefined, fn: () => void | Promise<void>) => {
8  const prev = process.env.SHELL
9  if (shell === undefined) delete process.env.SHELL
10  else process.env.SHELL = shell
11  Shell.acceptable.reset()
12  Shell.preferred.reset()
13  try {
14    await fn()
15  } finally {
16    if (prev === undefined) delete process.env.SHELL
17    else process.env.SHELL = prev
18    Shell.acceptable.reset()
19    Shell.preferred.reset()
20  }
21}
22 
23describe("shell", () => {
24  test("normalizes shell names", () => {
25    expect(Shell.name("/bin/bash")).toBe("bash")
26    if (process.platform === "win32") {
27      expect(Shell.name("C:/tools/NU.EXE")).toBe("nu")
28      expect(Shell.name("C:/tools/PWSH.EXE")).toBe("pwsh")
29    }
30  })
31 
32  test("detects login shells", () => {
33    expect(Shell.login("/bin/bash")).toBe(true)
34    expect(Shell.login("C:/tools/pwsh.exe")).toBe(false)
35  })
36 
37  test("detects posix shells", () => {
38    expect(Shell.posix("/bin/bash")).toBe(true)
39    expect(Shell.posix("/bin/fish")).toBe(false)
40    expect(Shell.posix("C:/tools/pwsh.exe")).toBe(false)
41  })
42 
43  test("falls back when configured shell cannot be resolved", async () => {
44    await withShell(undefined, async () => {
45      const preferred = Shell.preferred()
46      const acceptable = Shell.acceptable()
47      expect(Shell.preferred("opencode-missing-shell")).toBe(preferred)
48      expect(Shell.acceptable("opencode-missing-shell")).toBe(acceptable)
49    })
50  })
51 
52  test("falls back for terminal-only acceptable shells", () => {
53    expect(Shell.name(Shell.acceptable("fish"))).not.toBe("fish")
54    expect(Shell.name(Shell.acceptable("nu"))).not.toBe("nu")
55  })
56 
57  test("builds command args per shell family", () => {
58    expect(Shell.args("/bin/sh", "echo hi", "/tmp")).toEqual(["-c", "echo hi"])
59    expect(Shell.args("/usr/bin/fish", "echo hi", "/tmp")).toEqual(["-c", "echo hi"])
60    const zsh = Shell.args("/bin/zsh", "echo hi", "/tmp")
61    expect(zsh[0]).toBe("-l")
62    expect(zsh[1]).toBe("-c")
63    expect(zsh.at(-1)).toBe("/tmp")
64  })
65 
66  if (process.platform === "win32") {
67    test("rejects blacklisted shells case-insensitively", async () => {
68      await withShell("NU.EXE", async () => {
69        expect(Shell.name(Shell.acceptable())).not.toBe("nu")
70      })
71    })
72 
73    test("normalizes Git Bash shell paths from env", async () => {
74      const shell = "/cygdrive/c/Program Files/Git/bin/bash.exe"
75      await withShell(shell, async () => {
76        expect(Shell.preferred()).toBe(FSUtil.windowsPath(shell))
77      })
78    })
79 
80    test("resolves /usr/bin/bash from env to Git Bash", async () => {
81      const bash = Shell.gitbash()
82      if (!bash) return
83      await withShell("/usr/bin/bash", async () => {
84        expect(Shell.acceptable()).toBe(bash)
85        expect(Shell.preferred()).toBe(bash)
86      })
87    })
88 
89    test("resolves bare bash to Git Bash before PATH", async () => {
90      const bash = Shell.gitbash()
91      if (!bash) return
92      expect(Shell.acceptable("bash")).toBe(bash)
93      expect(Shell.preferred("bash")).toBe(bash)
94      await withShell("bash", async () => {
95        expect(Shell.acceptable()).toBe(bash)
96        expect(Shell.preferred()).toBe(bash)
97      })
98    })
99 
100    test("resolves bare PowerShell shells", async () => {
101      const shell = which("pwsh") || which("powershell")
102      if (!shell) return
103      await withShell(path.win32.basename(shell), async () => {
104        expect(Shell.preferred()).toBe(shell)
105      })
106    })
107  }
108})
109