openAgent

1import { describe, expect, test } from "bun:test"
2import { posix } from "path"
3import { configEntryNameFromPath } from "@/config/entry-name"
4 
5// Use POSIX semantics so the test is deterministic regardless of host OS —
6// production code passes paths through `path.relative` on the runtime
7// platform, but the helper normalizes via `replaceAll("\\", "/")`, so the
8// regression assertion ("the helper returns the bare name") holds on either
9// platform as long as we feed it a relative path. Using `posix.relative`
10// keeps the intermediate values stable across CI runners.
11 
12// The prefixes shipped by config/agent.ts after the relative-path refactor.
13const AGENT_PREFIXES = ["agent/", "agents/"]
14 
15describe("configEntryNameFromPath", () => {
16  test("strips an `agents/` prefix and returns the bare name", () => {
17    expect(configEntryNameFromPath("agents/build.md", AGENT_PREFIXES)).toBe("build")
18  })
19 
20  test("strips an `agent/` (singular) prefix", () => {
21    expect(configEntryNameFromPath("agent/build.md", AGENT_PREFIXES)).toBe("build")
22  })
23 
24  test("preserves nested subdirectories in the key", () => {
25    expect(configEntryNameFromPath("agents/team/build.md", AGENT_PREFIXES)).toBe("team/build")
26  })
27 
28  test("normalizes Windows-style backslashes", () => {
29    expect(configEntryNameFromPath("agents\\team\\build.md", AGENT_PREFIXES)).toBe("team/build")
30  })
31 
32  test("falls back to basename when no prefix matches", () => {
33    expect(configEntryNameFromPath("orphaned.md", AGENT_PREFIXES)).toBe("orphaned")
34    expect(configEntryNameFromPath("anywhere/orphaned.md", [])).toBe("orphaned")
35  })
36 
37  // Regression for #25713: a username (or any parent segment) containing
38  // `agent` or `agents` used to win the substring match before the real
39  // `agents/` directory could match, leaking the entire intervening path into
40  // the agent key (e.g. `.config/opencode/agents/build`). Anchoring at the
41  // caller via `path.relative(dir, item)` makes this impossible — the relative
42  // path is always rooted at `agent/` or `agents/`.
43  test("regression #25713: caller passes relative path; parent /agent/ segment is irrelevant", () => {
44    const dir = "/home/agent/.config/opencode"
45    const item = "/home/agent/.config/opencode/agents/build.md"
46    const relative = posix.relative(dir, item)
47    expect(relative).toBe("agents/build.md")
48    expect(configEntryNameFromPath(relative, AGENT_PREFIXES)).toBe("build")
49  })
50 
51  test("regression #25713: parent /agents/ segment is irrelevant", () => {
52    const dir = "/srv/agents/team/.config/opencode"
53    const item = "/srv/agents/team/.config/opencode/agents/build.md"
54    const relative = posix.relative(dir, item)
55    expect(configEntryNameFromPath(relative, AGENT_PREFIXES)).toBe("build")
56  })
57})
58