Skip to content
Sign up

flagon-io/g1t

public
00

Where people and agents ship software together. The open-source git platform for the whole job: issues, agents, checks and deploys to the edge.

gitagentscloudflarerustforgemcp
947 linesCodeBlameRaw
1// Tests for the plugin's hooks: node --test plugins/claude-code/test/
2import assert from "node:assert/strict";
3import { spawnSync } from "node:child_process";
4import { existsSync, mkdtempSync, readdirSync, readFileSync, rmSync, mkdirSync, writeFileSync } from "node:fs";
5import { tmpdir } from "node:os";
6import { join } from "node:path";
7import { afterEach, beforeEach, describe, test } from "node:test";
8import { fileURLToPath } from "node:url";
9
10import {
11 updateContext,
12 UPDATES_REPEAT_MS,
13 howToUpdate,
14 PROMPT_LIMIT,
15 RESULT_LIMIT,
16 agentResult,
17 branchFromCommand,
18 g1tTool,
19 hasTitle,
20 hook,
21 intentTitle,
22 repoPath,
23 MIN_CLI,
24 TITLE_LIMIT,
25 oldNode,
26 older,
27 typedSkill,
28 isG1tRemote,
29 isGitCommit,
30 isGitPush,
31 issueFromBranch,
32 issueFromCommand,
33 mayAddRemote,
34 pairSubagent,
35 plan,
36 rememberTask,
37 takeTask,
38 redact,
39 sessionFiles,
40 subagentHarnessId,
41 summarizeTool,
42 work,
43} from "../hooks/g1t.mjs";
44
45import { CONTRACT, refusal } from "./fake-g1t.mjs";
46
47const HERE = fileURLToPath(new URL(".", import.meta.url));
48const SCRIPT = join(HERE, "..", "hooks", "g1t.mjs");
49const FAKE = join(HERE, "fake-g1t.mjs");
50
51const base = { session_id: "abc-123", cwd: "/work/repo", transcript_path: "/t.jsonl", permission_mode: "default" };
52
53describe("plan", () => {
54 test("SessionStart starts the session", () => {
55 const job = plan({ ...base, hook_event_name: "SessionStart", source: "startup" }, {});
56 assert.equal(job.start, true);
57 assert.deepEqual(job.steps, []);
58 });
59
60 test("UserPromptSubmit logs the prompt, cut to the limit", () => {
61 const prompt = "x".repeat(PROMPT_LIMIT + 500);
62 const job = plan({ ...base, hook_event_name: "UserPromptSubmit", prompt }, {});
63 const args = job.steps[0].args;
64 assert.deepEqual(args.slice(0, 5), ["session", "log", "--kind", "prompt", "--"]);
65 assert.equal(args[5].length, PROMPT_LIMIT);
66 assert.ok(args[5].endsWith("…"));
67 });
68
69 test("a turn Claude Code starts itself for a background task is not a prompt", () => {
70 const notification = "<task-notification>\n<task-id>a1</task-id>\n<result>SECRET OUTPUT</result>\n</task-notification>";
71 assert.equal(plan({ ...base, hook_event_name: "UserPromptSubmit", prompt: notification }, {}), null);
72 assert.equal(plan({ ...base, hook_event_name: "UserPromptSubmit", prompt: "hi" }, {}).steps[0].args.at(-1), "hi");
73 });
74
75 test("a prompt's intent is its first line or sentence, plain, clipped on a word", () => {
76 assert.equal(intentTitle("fix the login redirect"), "Fix the login redirect");
77 assert.equal(intentTitle("Make preview builds faster. They time out on big repositories."), "Make preview builds faster");
78 assert.equal(intentTitle("Why does `src/app.ts` fail? It worked yesterday."), "Why does src/app.ts fail?");
79 assert.equal(intentTitle("```js\nconst a = 1;\n```\n## Clean **this** up\nmore"), "Clean this up");
80 assert.equal(intentTitle("\n\n- look at [the docs](https://docs.g1t.sh) for g1t.mjs"), "Look at the docs for g1t.mjs");
81 assert.equal(intentTitle(" spread over\tspaces "), "Spread over spaces");
82 const long = intentTitle("[TEST] Tidy up the README and then look at the CI pipeline, which keeps failing on the Windows runners");
83 assert.ok(long.length <= TITLE_LIMIT, long);
84 assert.ok(long.endsWith("…") && !long.endsWith(" …"), long);
85 assert.ok(long.startsWith("[TEST] Tidy up the README"), long);
86 assert.ok("[TEST] Tidy up the README and then look at the CI pipeline, which keeps failing on the Windows runners".startsWith(long.slice(0, -1)));
87 // What was pasted or attached is left out: the words typed around it title the session.
88 assert.equal(intentTitle('<pasted_content id="5c50">\nUncaught TypeError: x is undefined\n</pasted_content>\nAlso you broke the buttons'), "Also you broke the buttons");
89 assert.equal(intentTitle("[Image #1] the PR link is missing on Home"), "The PR link is missing on Home");
90 assert.equal(intentTitle("[Image: source: C:\\tmp\\1.png]\nmake it render"), "Make it render");
91 assert.equal(intentTitle('<pasted_content id="1">only a paste'), null);
92 // What VS Code puts ahead of what was typed is not a turn Claude Code started.
93 assert.equal(
94 intentTitle("<ide_opened_file>The user opened the file c:\\a\\Dockerfile in the IDE.</ide_opened_file>\nWhether it's the desktop app or the page, it crashes"),
95 "Whether it's the desktop app or the page, it crashes",
96 );
97 assert.equal(intentTitle("<ide_selection>const a = 1;</ide_selection>\n<ide_opened_file>a.ts</ide_opened_file>\nrename this"), "Rename this");
98 assert.equal(intentTitle("<ide_opened_file>a.ts</ide_opened_file>\n/clear"), null);
99 // Not intents: slash commands, turns Claude Code starts, nothing to say.
100 for (const prompt of ["/clear", "/compact keep the plan", "/g1t:status 4", "<command-name>/clear</command-name>", '<local-command-stdout lang="en">ok</local-command-stdout>', "```\ncode only\n```", "?", "", null]) {
101 assert.equal(intentTitle(prompt), null, String(prompt));
102 }
103 });
104
105 test("UserPromptSubmit titles the session with its intent, unless prompts are kept back", () => {
106 const job = plan({ ...base, hook_event_name: "UserPromptSubmit", prompt: "fix the login redirect" }, {});
107 assert.deepEqual(job.steps.at(-1), { title: "Fix the login redirect" });
108 assert.ok(!plan({ ...base, hook_event_name: "UserPromptSubmit", prompt: "/clear" }, {}).steps.some((s) => s.title));
109 assert.equal(plan({ ...base, hook_event_name: "UserPromptSubmit", prompt: "fix it" }, { G1T_RECORD_PROMPTS: "0" }), null);
110 assert.equal(hasTitle(JSON.stringify({ id: "ses_1", title: null })), false);
111 assert.equal(hasTitle(JSON.stringify({ id: "ses_1", title: " " })), false);
112 assert.equal(hasTitle(JSON.stringify({ id: "ses_1", title: "Fix it" })), true);
113 assert.equal(hasTitle("not json"), true, "unreadable: nothing is replaced");
114 });
115
116 test("G1T_RECORD_PROMPTS=0 keeps prompts back, and G1T_RECORD=0 everything", () => {
117 assert.equal(plan({ ...base, hook_event_name: "UserPromptSubmit", prompt: "hi" }, { G1T_RECORD_PROMPTS: "0" }), null);
118 assert.ok(plan({ ...base, hook_event_name: "Stop" }, { G1T_RECORD_PROMPTS: "0" }));
119 for (const hook_event_name of ["SessionStart", "UserPromptSubmit", "PostToolUse", "Stop", "SessionEnd"]) {
120 assert.equal(plan({ ...base, hook_event_name, prompt: "hi", tool_name: "Bash" }, { G1T_RECORD: "0" }), null);
121 }
122 });
123
124 test("PostToolUse sends the tool and its input's summary, never its response", () => {
125 const event = {
126 ...base,
127 hook_event_name: "PostToolUse",
128 tool_name: "Bash",
129 tool_input: { command: "npm test", description: "Run the tests" },
130 tool_response: { stdout: "SECRET OUTPUT 1234", stderr: "", interrupted: false },
131 tool_use_id: "toolu_1",
132 };
133 const job = plan(event, {});
134 assert.deepEqual(job.steps, [{ args: ["session", "log", "--kind", "tool_call", "--", "Bash: npm test"] }]);
135 assert.ok(!JSON.stringify(job).includes("SECRET OUTPUT"));
136 });
137
138 test("PostToolUse after a git push updates the head commit", () => {
139 const job = plan({ ...base, hook_event_name: "PostToolUse", tool_name: "Bash", tool_input: { command: "git push -u origin 42-fix" } }, {});
140 assert.deepEqual(job.steps[1], { headCommit: true });
141 const other = plan({ ...base, hook_event_name: "PostToolUse", tool_name: "Bash", tool_input: { command: "git status" } }, {});
142 assert.equal(other.steps.length, 1);
143 });
144
145 test("PostToolUse remembers the issue a command claims", () => {
146 const job = plan({ ...base, hook_event_name: "PostToolUse", tool_name: "Bash", tool_input: { command: "g1t work claim 42 --note x" } }, {});
147 assert.equal(job.issue, 42);
148 });
149
150 test("Stop sends a heartbeat and SessionEnd ends the session with its reason", () => {
151 assert.deepEqual(plan({ ...base, hook_event_name: "Stop", stop_hook_active: false }, {}).steps, [{ args: ["session", "heartbeat"] }]);
152 const end = plan({ ...base, hook_event_name: "SessionEnd", reason: "prompt_input_exit" }, {});
153 assert.deepEqual(end.steps, [{ args: ["session", "end", "--reason", "prompt_input_exit"] }]);
154 assert.equal(end.end, true);
155 });
156
157 test("events without a session, and other events, do nothing", () => {
158 assert.equal(plan({ hook_event_name: "Stop" }, {}), null);
159 assert.equal(plan({ ...base, hook_event_name: "PreToolUse", tool_name: "Bash" }, {}), null);
160 });
161});
162
163// Payloads as Claude Code sends them (code.claude.com/docs/en/hooks).
164const fixtures = {
165 subagentStart: {
166 session_id: "abc-123",
167 transcript_path: "/t.jsonl",
168 cwd: "/work/repo",
169 hook_event_name: "SubagentStart",
170 agent_id: "agent-def456",
171 agent_type: "Explore",
172 },
173 subagentStop: {
174 session_id: "abc-123",
175 transcript_path: "/t.jsonl",
176 cwd: "/work/repo",
177 permission_mode: "default",
178 hook_event_name: "SubagentStop",
179 stop_hook_active: false,
180 agent_id: "def456",
181 agent_type: "Explore",
182 agent_transcript_path: "/t/abc-123/subagents/agent-def456.jsonl",
183 last_assistant_message: "Analysis complete. Found 3 potential issues...",
184 background_tasks: [],
185 session_crons: [],
186 },
187 agentCompleted: {
188 ...base,
189 hook_event_name: "PostToolUse",
190 tool_name: "Agent",
191 tool_input: { description: "Find API endpoints", prompt: "Find all API endpoints", subagent_type: "Explore" },
192 tool_response: {
193 status: "completed",
194 agentId: "a4d2c8f1e0b3a297",
195 content: [{ type: "text", text: "Found 12 endpoints in src/api." }],
196 resolvedModel: "claude-sonnet-4-5",
197 totalTokens: 12450,
198 totalDurationMs: 48211,
199 totalToolUseCount: 7,
200 usage: { input_tokens: 8320, output_tokens: 410 },
201 },
202 tool_use_id: "toolu_9",
203 },
204 agentLaunched: {
205 ...base,
206 hook_event_name: "PostToolUse",
207 tool_name: "Agent",
208 tool_input: { description: "Review the diff", prompt: "Review it", subagent_type: "code-reviewer", run_in_background: true },
209 tool_response: { status: "async_launched", agentId: "b5", description: "Review the diff", prompt: "Review it", outputFile: "/tmp/b5.out", resolvedModel: "claude-sonnet-4-5" },
210 },
211 inSubagent: {
212 ...base,
213 hook_event_name: "PostToolUse",
214 agent_id: "agent-def456",
215 agent_type: "Explore",
216 tool_name: "Edit",
217 tool_input: { file_path: "/work/repo/src/api/routes.ts", old_string: "a", new_string: "b" },
218 tool_response: { filePath: "/work/repo/src/api/routes.ts", structuredPatch: "SECRET PATCH" },
219 },
220 skill: { ...base, hook_event_name: "PostToolUse", tool_name: "Skill", tool_input: { skill: "work-on-issue", args: "42" }, tool_response: { success: true, commandName: "work-on-issue" } },
221};
222
223describe("subagents", () => {
224 test("SubagentStart starts a session of its own for the subagent", () => {
225 const job = plan(fixtures.subagentStart, {});
226 assert.deepEqual(job.agent, { id: "agent-def456", type: "Explore", start: true });
227 assert.deepEqual(job.steps, []);
228 assert.equal(subagentHarnessId("abc-123", "agent-def456"), "abc-123/agent/def456");
229 assert.equal(subagentHarnessId("abc-123", "def456"), "abc-123/agent/def456", "the Agent tool's id names the same subagent");
230 });
231
232 test("SubagentStop ends it with its final message, cut to the limit", () => {
233 const job = plan(fixtures.subagentStop, {});
234 assert.deepEqual(job.agent, { id: "def456", type: "Explore" });
235 assert.deepEqual(job.steps, [{ args: ["session", "end", "--reason", "done", "--result=Analysis complete. Found 3 potential issues..."] }]);
236 const long = plan({ ...fixtures.subagentStop, last_assistant_message: "z".repeat(RESULT_LIMIT * 2) }, {});
237 assert.equal(long.steps[0].args.at(-1).length, "--result=".length + RESULT_LIMIT);
238 const dash = plan({ ...fixtures.subagentStop, last_assistant_message: "- Found 3 issues" }, {});
239 assert.equal(dash.steps[0].args.at(-1), "--result=- Found 3 issues", "a message that starts with a dash is still a value");
240 assert.equal(refusal(dash.steps[0].args), null);
241 });
242
243 test("an Agent call gives its subagent the task, and once finished its final message, and nothing else of its answer", () => {
244 const job = plan(fixtures.agentCompleted, {});
245 assert.deepEqual(job.subagent, {
246 id: "a4d2c8f1e0b3a297",
247 type: "Explore",
248 task: "Find API endpoints",
249 prompt: "Find all API endpoints",
250 model: "claude-sonnet-4-5",
251 result: "Found 12 endpoints in src/api.",
252 });
253 assert.deepEqual(job.steps, [{ args: ["session", "log", "--kind", "tool_call", "--", "Agent: Find API endpoints"] }]);
254 assert.ok(!JSON.stringify(job).includes("12450") && !JSON.stringify(job).includes("8320"));
255 const launched = plan(fixtures.agentLaunched, {});
256 assert.equal(launched.subagent.id, "b5");
257 assert.equal(launched.subagent.result, null, "a background subagent reports at SubagentStop");
258 assert.equal(plan(fixtures.agentCompleted, { G1T_RECORD_PROMPTS: "0" }).subagent.prompt, null);
259 assert.equal(agentResult({ content: "plain" }), "plain");
260 });
261
262 test("what a subagent does is its own session's, and the files it writes are touched", () => {
263 const job = plan(fixtures.inSubagent, {});
264 assert.deepEqual(job.agent, { id: "agent-def456", type: "Explore" });
265 assert.deepEqual(job.steps, [
266 { args: ["session", "log", "--kind", "tool_call", "--", "Edit: src/api/routes.ts"] },
267 { args: ["session", "touch", "--file=src/api/routes.ts"] },
268 ]);
269 assert.ok(!JSON.stringify(job).includes("SECRET PATCH"));
270 });
271
272 test("a skill is its name only", () => {
273 assert.deepEqual(plan(fixtures.skill, {}).steps, [{ args: ["session", "log", "--kind", "skill", "--", "work-on-issue"], skill: "work-on-issue" }]);
274 });
275
276 test("branches made, commits, claims and g1t's MCP tools are touched", () => {
277 const bash = (command) => plan({ ...base, hook_event_name: "PostToolUse", tool_name: "Bash", tool_input: { command } }, {});
278 assert.deepEqual(bash("git checkout -b 42-fix").steps.at(-1), { args: ["session", "touch", "--branch", "42-fix"] });
279 assert.deepEqual(bash('git commit -m "x" --trailer "G1t-Session: $G1T_SESSION"').steps.at(-1), { touchHead: true });
280 assert.deepEqual(bash("g1t work claim 42").steps.at(-1), { args: ["session", "touch", "--issue", "42"] });
281 const mcp = plan({ ...base, hook_event_name: "PostToolUse", tool_name: "mcp__plugin_g1t_g1t__issue", tool_input: { action: "claim", repo: "a/b", number: 7 } }, {});
282 assert.equal(mcp.issue, 7);
283 assert.equal(branchFromCommand("git switch -c feat/x && echo"), "feat/x");
284 assert.equal(branchFromCommand("git branch -d old"), null);
285 assert.equal(isGitCommit("git -C repo commit -am x"), true);
286 assert.equal(g1tTool("mcp__g1t__session"), "session");
287 assert.equal(g1tTool("mcp__github__issue"), null);
288 });
289
290 test("a hand-off's issue comes with the session", () => {
291 assert.equal(plan({ ...base, hook_event_name: "SessionStart" }, { G1T_ISSUE: "12" }).issue, 12);
292 });
293});
294
295describe("summaries", () => {
296 test("say what a tool was called with, in one short line", () => {
297 assert.equal(summarizeTool("Edit", { file_path: "/work/repo/src/a.ts", old_string: "x", new_string: "y" }, "/work/repo"), "Edit: src/a.ts");
298 assert.equal(summarizeTool("Grep", { pattern: "TODO", path: "src" }, "/w"), "Grep: TODO in src");
299 // From the top of the working copy, wherever the shell has moved to.
300 assert.equal(repoPath("/work/repo/apps/web/app/a.ts", "/work/repo/apps/web", "/work/repo"), "apps/web/app/a.ts");
301 assert.equal(repoPath("app/a.ts", "/work/repo/apps/web", "/work/repo"), "apps/web/app/a.ts");
302 assert.equal(repoPath("/elsewhere/a.ts", "/work/repo/apps/web", "/work/repo"), null);
303 assert.equal(repoPath("/work/repo/src/a.ts", "/work/repo", null), "src/a.ts");
304 assert.equal(summarizeTool("WebFetch", { url: "https://example.com", prompt: "p" }), "WebFetch: https://example.com");
305 assert.equal(summarizeTool("Bash", { command: "echo one\necho two" }), "Bash: echo one …");
306 assert.equal(summarizeTool("mcp__g1t__issue", { action: "get", number: 4 }), "mcp__g1t__issue: get");
307 assert.ok(summarizeTool("Bash", { command: "y".repeat(1000) }).length <= 200);
308 });
309
310 test("hide what looks like a secret", () => {
311 const line = redact('curl -H "Authorization: Bearer g1t_abcdefghijklmnop" https://u:pw@x.com GITHUB_TOKEN=ghp_123456789012 --token abc');
312 assert.ok(!line.includes("abcdefghijklmnop"));
313 assert.ok(!line.includes("ghp_123456789012"));
314 assert.ok(!line.includes("u:pw@"));
315 assert.ok(!line.includes("--token abc"));
316 });
317
318 test("find the issue in a branch name", () => {
319 assert.equal(issueFromBranch("42-fix-login"), 42);
320 assert.equal(issueFromBranch("issue-7"), 7);
321 assert.equal(issueFromBranch("fix/123-thing"), 123);
322 assert.equal(issueFromBranch("main"), null);
323 assert.equal(issueFromBranch("release/1.2"), null);
324 assert.equal(issueFromBranch("HEAD"), null);
325 assert.equal(issueFromCommand("g1t session attach --issue 9"), 9);
326 assert.equal(issueFromCommand("g1t pr create --issue=12 --draft"), 12);
327 assert.equal(isGitPush("cd x && git -C repo push"), true);
328 assert.equal(isGitPush("git pushd"), false);
329 assert.equal(isG1tRemote(["remote.origin.url https://g1t.sh/a/b.git"], {}), true);
330 assert.equal(isG1tRemote(["remote.origin.url https://example.com/a/b.git"], {}), false);
331 assert.equal(isG1tRemote(["remote.origin.url https://git.acme.dev/a/b.git"], { G1T_HOST: "git.acme.dev" }), true);
332 });
333});
334
335describe("the plugin's files", () => {
336 const root = join(HERE, "..", "..", "..");
337 test("parse as JSON", () => {
338 for (const file of [".claude-plugin/marketplace.json", "plugins/claude-code/.claude-plugin/plugin.json", "plugins/claude-code/.mcp.json", "plugins/claude-code/hooks/hooks.json"]) {
339 assert.doesNotThrow(() => JSON.parse(readFileSync(join(root, file), "utf8")), file);
340 }
341 });
342
343 test("the AGENTS.md snippet is the one in the docs", () => {
344 const snippet = readFileSync(join(root, "plugins/other-harnesses/AGENTS.snippet.md"), "utf8").replaceAll("\r\n", "\n");
345 const guide = readFileSync(join(root, "apps/docs/src/content/docs/guides/claude-code-plugin.md"), "utf8").replaceAll("\r\n", "\n");
346 const block = /```md title="AGENTS.md"\n([\s\S]*?)```/.exec(guide)?.[1];
347 assert.equal(block, snippet);
348 });
349});
350
351describe("with a working copy", () => {
352 let dir, env, log;
353
354 const git = (...args) => spawnSync("git", args, { cwd: dir, encoding: "utf8" });
355 const allCalls = () =>
356 existsSync(log)
357 ? readFileSync(log, "utf8")
358 .trim()
359 .split("\n")
360 .filter(Boolean)
361 .map((l) => JSON.parse(l))
362 : [];
363 // The calls that do something: not the version check before them.
364 const calls = () => allCalls().filter((c) => c.args[0] !== "--version");
365
366 beforeEach(() => {
367 dir = mkdtempSync(join(tmpdir(), "g1t-hook-"));
368 log = join(dir, "calls.jsonl");
369 git("init", "-q", "-b", "42-fix-login");
370 git("remote", "add", "origin", "https://g1t.sh/acme/app.git");
371 env = { PATH: process.env.PATH, G1T_BIN: FAKE, FAKE_G1T_LOG: log, CLAUDE_PLUGIN_DATA: join(dir, ".data") };
372 });
373 afterEach(() => {
374 // Every call the hooks made is one the real CLI takes (the stand-in refuses the rest).
375 const refused = allCalls().filter((c) => refusal(c.args));
376 // And none let the CLI claim every uncommitted file as this session's (the stand-in logs `touch` then).
377 const guessing = allCalls().filter((c) => c.touch !== undefined);
378 rmSync(dir, { recursive: true, force: true, maxRetries: 20, retryDelay: 100 });
379 assert.deepEqual(refused, [], "the CLI refuses these calls");
380 assert.deepEqual(guessing, [], "these calls let the CLI guess what was touched");
381 });
382
383 test("SessionStart gives Claude the branch's issue's work context", () => {
384 const { output, job } = hook({ ...base, cwd: dir, hook_event_name: "SessionStart", source: "startup" }, env);
385 const context = output.hookSpecificOutput.additionalContext;
386 assert.equal(output.hookSpecificOutput.hookEventName, "SessionStart");
387 assert.match(context, /issue #42/);
388 assert.match(context, /Definition of done: it works/);
389 assert.deepEqual(calls()[0].args, ["work", "context", "42"]);
390 assert.equal(job.start, true);
391 });
392
393 test("the background half starts the session once, then logs into it", () => {
394 work({ cwd: dir, sid: base.session_id, start: true, steps: [] }, env);
395 const { job } = hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "fix the login" }, env);
396 // No SessionStart marked this session as one in a g1t working copy.
397 assert.equal(job, null);
398 work(plan({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "fix the login" }, env), env);
399 const [start, logged] = calls().filter((call) => call.args[1] !== "get_session");
400 assert.deepEqual(start.args, ["session", "start", "--harness", "claude_code", "--harness-session-id", "abc-123", "--json"]);
401 assert.deepEqual(logged, { args: ["session", "log", "--kind", "prompt", "--", "fix the login"], session: "ses_fake01" });
402 assert.equal(readFileSync(sessionFiles(base.session_id, env).id, "utf8"), "ses_fake01");
403 });
404
405 test("a shell command's writes are this session's; what was uncommitted before it is not", () => {
406 const { job } = hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, env);
407 work(job, env);
408 // Another session's edit, uncommitted in the same working copy.
409 writeFileSync(join(dir, "theirs.md"), "theirs");
410 const command = { ...base, cwd: dir, tool_name: "Bash", tool_use_id: "tu_1", tool_input: { command: "sed -i s/a/b/ mine.md" } };
411 hook({ ...command, hook_event_name: "PreToolUse" }, env);
412 writeFileSync(join(dir, "mine.md"), "mine");
413 const after = hook({ ...command, hook_event_name: "PostToolUse" }, env).job;
414 const touched = after.steps.filter((step) => step.args?.[1] === "touch").flatMap((step) => step.args.filter((arg) => arg.startsWith("--file=")));
415 assert.ok(touched.includes("--file=mine.md"), String(touched));
416 assert.ok(!touched.includes("--file=theirs.md"), String(touched));
417 // Read once: a command run again under the same id finds nothing kept.
418 assert.equal(existsSync(sessionFiles(base.session_id, env).shell("tu_1")), false);
419 });
420
421 test("the CLI never guesses touched files from the whole working copy", () => {
422 const { job } = hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, env);
423 work(job, env);
424 work(plan({ ...base, cwd: dir, hook_event_name: "Stop" }, env), env);
425 const heartbeat = calls().find((call) => call.args[0] === "session" && call.args[1] === "heartbeat");
426 assert.ok(heartbeat, "a heartbeat was sent");
427 assert.equal(heartbeat.touch, undefined);
428 });
429
430 test("background retrieval reaches the next tool without auto-acknowledging", () => {
431 // A CLI that acknowledges: an older one is told to update instead (below).
432 env.FAKE_G1T_VERSION = "0.3.3";
433 env.FAKE_G1T_UPDATES = JSON.stringify([{ id: "upd_real", kind: "overlap.detected", data: { paths: ["src/api.ts"] } }]);
434 const event = { ...base, cwd: dir, hook_event_name: "SessionStart" };
435 const { job } = hook(event, env);
436 work(job, env);
437 const next = { ...event, hook_event_name: "PreToolUse", tool_name: "Read", tool_input: { file_path: "src/api.ts" } };
438 const first = hook(next, env).output;
439 assert.match(first.hookSpecificOutput.additionalContext, /upd_real/);
440 assert.equal(first.hookSpecificOutput.hookEventName, "PreToolUse");
441 // The next tool is not told again at once; the next prompt is.
442 assert.equal(hook(next, env).output, null);
443 const prompt = hook({ ...event, hook_event_name: "UserPromptSubmit", prompt: "go on" }, env).output;
444 assert.match(prompt.hookSpecificOutput.additionalContext, /upd_real/);
445 assert.ok(calls().some((call) => call.args[1] === "get_session" && call.args.includes("after=4294967295")));
446 assert.ok(!calls().some((call) => call.args[1] === "record_session_entries"));
447 });
448
449 test("hooks after SessionStart record; SessionEnd forgets the session", () => {
450 hook({ ...base, cwd: dir, hook_event_name: "SessionStart", source: "startup" }, env);
451 const files = sessionFiles(base.session_id, env);
452 assert.ok(existsSync(files.marker));
453 const { job } = hook({ ...base, cwd: dir, hook_event_name: "Stop" }, env);
454 work(job, env);
455 work(plan({ ...base, cwd: dir, hook_event_name: "SessionEnd", reason: "clear" }, env), env);
456 const args = calls().map((c) => c.args.join(" "));
457 assert.ok(args.includes("session heartbeat"));
458 assert.ok(args.includes("session end --reason clear"));
459 assert.ok(!existsSync(files.marker) && !existsSync(files.id));
460 });
461
462 test("a push that reached the remote updates the head commit", () => {
463 git("-c", "user.name=t", "-c", "user.email=t@t", "-c", "commit.gpgsign=false", "commit", "-q", "--allow-empty", "-m", "one");
464 git("update-ref", "refs/remotes/origin/42-fix-login", "HEAD");
465 const head = git("rev-parse", "HEAD").stdout.trim();
466 work(plan({ ...base, cwd: dir, hook_event_name: "PostToolUse", tool_name: "Bash", tool_input: { command: "git push" } }, env), env);
467 const update = calls().find((c) => c.args[0] === "api");
468 assert.deepEqual(update.args, ["api", "update_session", "-f", "id=ses_fake01", "-f", `head_commit=${head}`, "--silent"]);
469 });
470
471 test("each subagent is a session of its own under this one, whatever order its hooks arrive in", () => {
472 const at = (event) => ({ ...event, cwd: dir });
473 hook(at({ ...base, hook_event_name: "SessionStart", source: "startup" }), env);
474 work({ cwd: dir, sid: base.session_id, start: true, steps: [] }, env);
475 // Claude calls Agent; its task waits for the subagent.
476 const pre = hook(at({ ...base, hook_event_name: "PreToolUse", tool_name: "Agent", tool_input: fixtures.agentCompleted.tool_input }), env);
477 assert.equal(pre.output, null);
478 const started = hook(at(fixtures.subagentStart), env);
479 assert.match(started.output.hookSpecificOutput.additionalContext, /session of your own/);
480 assert.match(started.output.hookSpecificOutput.additionalContext, /g1t session list --issue 42 --subagents|g1t session list --subagents/);
481 assert.equal(started.job.pair, true, "paired with its Agent call in the background half");
482 work(started.job, env);
483 assert.equal(started.job.agent.task, "Find API endpoints", "no files of the subagent's: the oldest call of its type");
484 // What it does goes to its own session.
485 const edited = { ...fixtures.inSubagent.tool_input, file_path: join(dir, "src", "api", "routes.ts") };
486 work(hook(at({ ...fixtures.inSubagent, tool_input: edited }), env).job, env);
487 // Its g1t commands act as it.
488 const bash = hook(at({ ...base, hook_event_name: "PreToolUse", agent_id: "agent-def456", agent_type: "Explore", tool_name: "Bash", tool_input: { command: "g1t work claim 42" } }), env);
489 assert.equal(bash.output.hookSpecificOutput.updatedInput.command, "export G1T_SESSION=ses_sub_def456; g1t work claim 42");
490 const mcp = hook(at({ ...base, hook_event_name: "PreToolUse", agent_id: "agent-def456", tool_name: "mcp__plugin_g1t_g1t__issue", tool_input: { action: "claim", repo: "acme/app", number: 42 } }), env);
491 assert.equal(mcp.output.hookSpecificOutput.updatedInput.session_id, "ses_sub_def456");
492 // Outside a subagent, a command is left alone.
493 assert.equal(hook(at({ ...base, hook_event_name: "PreToolUse", tool_name: "Bash", tool_input: { command: "g1t work claim 42" } }), env).output, null);
494 work(hook(at(fixtures.subagentStop), env).job, env);
495 // A late SubagentStart starts it again, which g1t answers with the same, ended, session.
496 work(hook(at(fixtures.subagentStart), env).job, env);
497 const log = calls();
498 const childStarts = log.filter((c) => c.args[1] === "start" && c.args.includes("--parent"));
499 assert.ok(childStarts.length >= 2);
500 const first = childStarts[0].args;
501 assert.deepEqual(first.slice(0, 8), ["session", "start", "--harness", "claude_code", "--harness-session-id", "abc-123/agent/def456", "--parent", "ses_fake01"]);
502 assert.ok(first.includes("--no-remember"));
503 assert.ok(first.includes("--agent-type=Explore"));
504 assert.ok(first.includes("--task=Find API endpoints"));
505 const edit = log.find((c) => c.args[2] === "--kind" && c.args.at(-1) === "Edit: src/api/routes.ts");
506 assert.equal(edit.session, "ses_sub_def456");
507 const touched = log.find((c) => c.args[1] === "touch");
508 assert.deepEqual(touched, { args: ["session", "touch", "--file=src/api/routes.ts"], session: "ses_sub_def456" });
509 const ended = log.find((c) => c.args[1] === "end");
510 assert.deepEqual(ended, { args: ["session", "end", "--reason", "done", "--result=Analysis complete. Found 3 potential issues..."], session: "ses_sub_def456" });
511 });
512
513 test("five subagents of one type started at once each get their own Agent call's task, whatever order they start in", () => {
514 // Claude Code's transcript and, beside it, each subagent's files.
515 const transcript = join(dir, "projects", "abc-123.jsonl");
516 const subagents = join(dir, "projects", "abc-123", "subagents");
517 mkdirSync(subagents, { recursive: true });
518 writeFileSync(transcript, "");
519 const at = (event) => ({ ...event, cwd: dir, transcript_path: transcript });
520 hook(at({ ...base, hook_event_name: "SessionStart" }), env);
521 work({ cwd: dir, sid: base.session_id, start: true, steps: [] }, env);
522 const n = [1, 2, 3, 4, 5];
523 // One message, five Agent calls of the same type.
524 for (const i of n) {
525 const tool_input = { description: `Task ${i}`, prompt: `Do thing ${i}, and only thing ${i}.`, subagent_type: "general-purpose" };
526 hook(at({ ...base, hook_event_name: "PreToolUse", tool_name: "Agent", tool_input, tool_use_id: `toolu_${i}` }), env);
527 }
528 const files = (i, { meta = true, first = true } = {}) => {
529 if (meta) writeFileSync(join(subagents, `agent-a${i}.meta.json`), JSON.stringify({ agentType: "general-purpose", description: `Task ${i}`, toolUseId: `toolu_${i}` }));
530 if (first) writeFileSync(join(subagents, `agent-a${i}.jsonl`), `${JSON.stringify({ type: "user", agentId: `a${i}`, message: { role: "user", content: `Do thing ${i}, and only thing ${i}.` } })}\n{"type":"attachment"}\n`);
531 };
532 // They start in another order: one with its meta file, one with only its transcript (by prompt), the rest with both.
533 files(4);
534 files(2, { meta: false });
535 files(5);
536 files(1);
537 files(3, { first: false });
538 for (const i of [4, 2, 5, 1, 3]) work(hook(at({ ...base, hook_event_name: "SubagentStart", agent_id: `a${i}`, agent_type: "general-purpose" }), env).job, env);
539 const starts = calls().filter((c) => c.args[1] === "start" && c.args.includes("--parent"));
540 assert.equal(starts.length, 5);
541 for (const i of n) {
542 const start = starts.find((c) => c.args.includes(`abc-123/agent/a${i}`)).args;
543 assert.ok(start.includes(`--task=Task ${i}`), `subagent ${i}: ${start.join(" ")}`);
544 assert.ok(start.includes(`--prompt=Do thing ${i}, and only thing ${i}.`), `subagent ${i}'s own prompt`);
545 }
546 assert.equal(readdirSync(join(env.CLAUDE_PLUGIN_DATA, "sessions")).filter((f) => f.includes(".task.")).length, 0, "every task taken once");
547 });
548
549 test("a pending task is never handed to two subagents, and an exact subagent never takes another's", () => {
550 const files = sessionFiles(base.session_id, env);
551 const input = (i) => ({ description: `Task ${i}`, prompt: `Prompt ${i}`, subagent_type: "general-purpose" });
552 for (const i of [1, 2, 3]) rememberTask(files, input(i), `toolu_${i}`);
553 assert.equal(takeTask(files, { type: "general-purpose", toolUseId: "toolu_2" }).task, "Task 2");
554 assert.equal(takeTask(files, { type: "general-purpose", toolUseId: "toolu_2" }), null, "taken once");
555 assert.equal(takeTask(files, { type: "general-purpose", prompt: "Prompt 2" }), null, "nor by its prompt");
556 assert.equal(takeTask(files, { type: "general-purpose", toolUseId: "toolu_9" }), null, "an unknown call takes nothing rather than a guess");
557 assert.equal(takeTask(files, { type: "general-purpose", prompt: " Prompt 3\n" }).task, "Task 3", "the prompt as the transcript has it");
558 // With no files of the subagent's (an older Claude Code): the oldest of its type, as before.
559 assert.equal(takeTask(files, "general-purpose").task, "Task 1");
560 assert.equal(takeTask(files, "general-purpose"), null);
561 // An Agent call answered takes its own task back, so no later subagent guesses it.
562 rememberTask(files, input(4), "toolu_4");
563 work({ ...plan({ ...fixtures.agentLaunched, cwd: dir, tool_use_id: "toolu_4" }, env) }, env);
564 assert.equal(takeTask(files, "general-purpose"), null);
565 });
566
567 test("a subagent's files give its task even when its Agent call was not kept", () => {
568 const subagents = join(dir, "t", "subagents");
569 mkdirSync(subagents, { recursive: true });
570 writeFileSync(join(subagents, "agent-a7.meta.json"), JSON.stringify({ description: "Look around", toolUseId: "toolu_7" }));
571 writeFileSync(join(subagents, "agent-a7.jsonl"), `${JSON.stringify({ type: "user", message: { role: "user", content: [{ type: "text", text: "Look around the repo" }] } })}\n`);
572 const files = sessionFiles(base.session_id, env);
573 rememberTask(files, { description: "Someone else's", prompt: "not yours", subagent_type: "Explore" }, "toolu_8");
574 const agent = pairSubagent({ id: "agent-a7", type: "Explore" }, files, join(dir, "t.jsonl"), env);
575 assert.deepEqual(agent, { id: "agent-a7", type: "Explore", task: "Look around", prompt: "Look around the repo" });
576 assert.equal(takeTask(files, "Explore").task, "Someone else's", "left for its own subagent");
577 assert.equal(pairSubagent({ id: "a7", type: "Explore" }, files, join(dir, "t.jsonl"), { G1T_RECORD_PROMPTS: "0" }).prompt, undefined);
578 });
579
580 test("a session started with no g1t remote is recorded once a command adds one, titled by its first prompt", () => {
581 git("remote", "remove", "origin");
582 const at = (event) => ({ ...base, cwd: dir, ...event });
583 assert.deepEqual(hook(at({ hook_event_name: "SessionStart", source: "startup" }), env), { output: null, job: null });
584 assert.equal(hook(at({ hook_event_name: "UserPromptSubmit", prompt: "Build me a todo app. Use React." }), env).job, null);
585 assert.equal(hook(at({ hook_event_name: "UserPromptSubmit", prompt: "and make it blue" }), env).job, null);
586 const bash = (command) => at({ hook_event_name: "PostToolUse", tool_name: "Bash", tool_input: { command } });
587 assert.equal(hook(at({ hook_event_name: "PostToolUse", tool_name: "Write", tool_input: { file_path: join(dir, "a.ts") } }), env).job, null);
588 assert.equal(hook(bash("npm init -y"), env).job, null);
589 assert.equal(hook(bash("git remote -v"), env).job, null, "looked, and there is none yet");
590 assert.deepEqual(calls(), [], "nothing recorded without a g1t remote");
591 // The agent makes the repository and adds it as the remote.
592 git("remote", "add", "origin", "https://g1t.sh/acme/app.git");
593 const { output, job } = hook(bash("g1t repo create acme/app --private && git remote add origin https://g1t.sh/acme/app.git"), env);
594 assert.equal(job.start, true);
595 assert.equal(output.hookSpecificOutput.hookEventName, "PostToolUse");
596 assert.match(output.hookSpecificOutput.additionalContext, /issue #42/, "told what SessionStart would have told it");
597 assert.deepEqual(job.steps.at(-1), { title: "Build me a todo app" });
598 work(job, env);
599 const recorded = calls().filter((c) => c.args[0] === "session");
600 assert.deepEqual(recorded[0].args, ["session", "start", "--harness", "claude_code", "--harness-session-id", "abc-123", "--json"]);
601 assert.ok(recorded.some((c) => c.args[1] === "log" && c.args.at(-1).startsWith("Bash: g1t repo create acme/app")));
602 assert.ok(recorded.some((c) => c.args.join(" ") === "session attach --title=Build me a todo app"));
603 // From here on it is recorded like any other, and titled once.
604 const next = hook(at({ hook_event_name: "UserPromptSubmit", prompt: "Now add tests" }), env).job;
605 assert.ok(next && !next.steps.some((s) => s.title) && !next.start);
606 });
607
608 test("a session given a g1t remote between prompts is recorded at the next one", () => {
609 git("remote", "set-url", "origin", "https://example.com/acme/app.git");
610 const at = (event) => ({ ...base, cwd: dir, ...event });
611 hook(at({ hook_event_name: "SessionStart" }), env);
612 assert.equal(hook(at({ hook_event_name: "UserPromptSubmit", prompt: "/clear" }), env).job, null);
613 git("remote", "set-url", "origin", "https://g1t.sh/acme/app.git");
614 const { job } = hook(at({ hook_event_name: "UserPromptSubmit", prompt: "fix the login" }), env);
615 assert.equal(job.start, true);
616 assert.deepEqual(job.steps.at(-1), { title: "Fix the login" }, "no earlier prompt said what it is for: this one does");
617 assert.ok(existsSync(sessionFiles(base.session_id, env).marker));
618 assert.ok(!existsSync(sessionFiles(base.session_id, env).unrecorded));
619 });
620
621 test("only commands that may add a remote look for one", () => {
622 for (const command of ["git remote add origin https://g1t.sh/a/b.git", "git -C app push -u origin main", "git init", "g1t repo create a/b", "cd x && git clone https://g1t.sh/a/b"]) {
623 assert.equal(mayAddRemote(command), true, command);
624 }
625 for (const command of ["git status", "npm install", "g1t work list", "echo push", undefined]) assert.equal(mayAddRemote(command), false, String(command));
626 });
627
628 test("a subagent whose stop arrives first is started, then ended", () => {
629 hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, env);
630 work(hook({ ...fixtures.subagentStop, cwd: dir }, env).job, env);
631 const [, start, end] = calls().filter((c) => c.args[0] === "session");
632 assert.ok(start.args.includes("--parent"));
633 assert.equal(end.args[1], "end");
634 assert.equal(end.session, "ses_sub_def456");
635 });
636
637 test("an Agent call that finished ends its subagent with what it reported", () => {
638 hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, env);
639 work(hook({ ...fixtures.agentCompleted, cwd: dir }, env).job, env);
640 const log = calls().filter((c) => c.args[0] === "session");
641 assert.equal(log.find((c) => c.args[1] === "log").session, "ses_fake01", "the Agent call is the parent's");
642 const end = log.find((c) => c.args[1] === "end");
643 assert.deepEqual(end, { args: ["session", "end", "--reason", "done", "--result=Found 12 endpoints in src/api."], session: "ses_sub_a4d2c8f1e0b3a297" });
644 });
645
646 test("a hand-off names its issue at the start", () => {
647 git("checkout", "-q", "-b", "main");
648 const { output, job } = hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, { ...env, G1T_ISSUE: "12" });
649 assert.match(output.hookSpecificOutput.additionalContext, /issue #12 \(from the work handed to it\)/);
650 assert.match(output.hookSpecificOutput.additionalContext, /G1t-Session/);
651 work(job, env);
652 const start = calls().find((c) => c.args[1] === "start");
653 assert.ok(start.args.includes("--issue") && start.args.includes("12"));
654 });
655
656 test("a missing CLI records nothing and says so once", () => {
657 const missing = { ...env, G1T_BIN: join(dir, "no-such-g1t") };
658 const first = hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, missing);
659 assert.match(first.output.systemMessage, /not installed/);
660 assert.equal(first.job, null);
661 const second = hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, missing);
662 assert.equal(second.output, null);
663 assert.equal(hook({ ...base, cwd: dir, hook_event_name: "Stop" }, missing).job, null);
664 assert.deepEqual(calls(), []);
665 });
666
667 test("a signed-out CLI is said once, at the next prompt", () => {
668 hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, { ...env, CLAUDE_PLUGIN_DATA: env.CLAUDE_PLUGIN_DATA });
669 work({ cwd: dir, sid: base.session_id, start: true, steps: [] }, { ...env, FAKE_G1T_EXIT: "4" });
670 const first = hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "hi" }, env);
671 assert.match(first.output.systemMessage, /not signed in/);
672 const second = hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "hi" }, env);
673 assert.equal(second.output, null);
674 });
675
676 test("outside a g1t repository nothing happens", () => {
677 git("remote", "set-url", "origin", "https://example.com/acme/app.git");
678 const { output, job } = hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, env);
679 assert.equal(output, null);
680 assert.equal(job, null);
681 assert.equal(hook({ ...base, cwd: dir, hook_event_name: "Stop" }, env).job, null);
682 assert.deepEqual(calls(), []);
683 });
684
685 test("the script answers SessionStart on stdin quickly, as JSON", async () => {
686 const started = Date.now();
687 const out = spawnSync(process.execPath, [SCRIPT], {
688 input: JSON.stringify({ ...base, cwd: dir, hook_event_name: "SessionStart", source: "startup" }),
689 encoding: "utf8",
690 env: { ...process.env, ...env },
691 });
692 const took = Date.now() - started;
693 assert.equal(out.status, 0);
694 assert.match(JSON.parse(out.stdout).hookSpecificOutput.additionalContext, /issue #42/);
695 assert.ok(took < 3000, `took ${took} ms`);
696 // The detached half starts the session after the hook has returned.
697 const id = sessionFiles(base.session_id, env).id;
698 for (let i = 0; i < 100 && !existsSync(id); i++) await new Promise((done) => setTimeout(done, 50));
699 assert.equal(readFileSync(id, "utf8"), "ses_fake01");
700 await new Promise((done) => setTimeout(done, 200));
701 });
702
703 test("the script prints nothing for a prompt, and never fails on bad input", () => {
704 const prompt = spawnSync(process.execPath, [SCRIPT], {
705 input: JSON.stringify({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "hello" }),
706 encoding: "utf8",
707 env: { ...process.env, ...env },
708 });
709 assert.equal(prompt.status, 0);
710 assert.equal(prompt.stdout, "");
711 const bad = spawnSync(process.execPath, [SCRIPT], { input: "not json", encoding: "utf8", env: { ...process.env, ...env } });
712 assert.equal(bad.status, 0);
713 assert.equal(bad.stdout, "");
714 });
715
716 test("a CLI older than the plugin needs records nothing and says so once, with how to update it", () => {
717 const old = { ...env, FAKE_G1T_VERSION: "0.3.1" };
718 hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, old);
719 work({ cwd: dir, sid: base.session_id, start: true, steps: [] }, old);
720 work(plan({ ...base, cwd: dir, hook_event_name: "Stop" }, old), old);
721 // SessionStart still asks it for the issue's context, which every CLI gives.
722 assert.deepEqual(calls().map((c) => c.args.join(" ")), ["work context 42"], "nothing is recorded with it");
723 assert.equal(allCalls().filter((c) => c.args[0] === "--version").length, 1, "the version is asked once for each build of the CLI");
724 const first = hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "hi" }, old);
725 assert.match(first.output.systemMessage, /g1t CLI is 0\.3\.1, and the plugin needs 0\.3\.2 or later/);
726 assert.match(first.output.systemMessage, /curl -fsSL https:\/\/g1t\.sh\/install \| sh/);
727 assert.match(first.output.systemMessage, /irm https:\/\/g1t\.sh\/install\.ps1 \| iex/);
728 assert.equal(hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "hi" }, old).output, null);
729 assert.equal(older("0.3.1", MIN_CLI), true);
730 assert.equal(older("0.3.2", MIN_CLI), false);
731 assert.equal(older("0.10.0", MIN_CLI), false);
732 assert.equal(older("1.0.0", MIN_CLI), false);
733 });
734
735 test("a CLI that refuses a command (exit 2) is said once, as one to update", () => {
736 hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, env);
737 work({ cwd: dir, sid: base.session_id, start: true, steps: [] }, { ...env, FAKE_G1T_EXIT: "2" });
738 const first = hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "hi" }, env);
739 assert.match(first.output.systemMessage, /refused `g1t session start`/);
740 assert.match(first.output.systemMessage, /installer again/);
741 assert.equal(hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "hi" }, env).output, null);
742 });
743
744 test("the stand-in CLI refuses what the real one refuses", () => {
745 assert.equal(CONTRACT.minimum_version, MIN_CLI, "the hooks ask for the CLI the contract needs");
746 assert.match(refusal(["session", "log", "--kind", "tool-calls", "--", "x"]), /takes prompt, tool_call, skill/);
747 assert.match(refusal(["session", "log", "--kind", "note", "--", "x"]), /not note/, "the hooks never send a note");
748 assert.match(refusal(["session", "end", "--result", "- a list"]), /unexpected argument/);
749 assert.equal(refusal(["session", "end", "--result=- a list"]), null);
750 assert.match(refusal(["session", "touch"]), /needs one of/);
751 assert.match(refusal(["session", "heartbeat", "--now"]), /takes no --now/);
752 assert.match(refusal(["session", "start", "--harness", "claude-managed-agents", "--harness-session-id", "x"]), /takes claude_code/);
753 assert.match(refusal(["work", "context", "abc"]), /not abc/);
754 assert.match(refusal(["session", "pause"]), /takes 0 argument/);
755 assert.equal(refusal(["api", "update_session", "-f", "id=ses_1", "-f", "head_commit=abc", "--silent"]), null);
756 });
757
758 test("PowerShell commands are read as Bash ones are, and a subagent's act as its session", () => {
759 const ps = (command, extra = {}) => ({ ...base, cwd: dir, hook_event_name: "PostToolUse", tool_name: "PowerShell", tool_input: { command }, ...extra });
760 assert.equal(plan(ps("g1t work claim 42"), env).issue, 42);
761 assert.deepEqual(plan(ps("git checkout -b 42-fix; git status"), env).steps.at(-1), { args: ["session", "touch", "--branch", "42-fix"] });
762 assert.deepEqual(plan(ps("git commit -m x"), env).steps.at(-1), { touchHead: true });
763 assert.deepEqual(plan(ps("git push origin 42-fix"), env).steps.at(-1), { headCommit: true });
764 hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, env);
765 work({ cwd: dir, sid: base.session_id, start: true, steps: [] }, env);
766 work(hook({ ...fixtures.subagentStart, cwd: dir }, env).job, env);
767 const pre = (tool_name, command) =>
768 hook({ ...base, cwd: dir, hook_event_name: "PreToolUse", agent_id: "agent-def456", agent_type: "Explore", tool_name, tool_input: { command } }, env).output;
769 assert.equal(pre("PowerShell", "g1t work claim 42").hookSpecificOutput.updatedInput.command, "$env:G1T_SESSION='ses_sub_def456'; g1t work claim 42");
770 assert.equal(pre("PowerShell", "g1t.exe session view").hookSpecificOutput.updatedInput.command, "$env:G1T_SESSION='ses_sub_def456'; g1t.exe session view");
771 assert.equal(pre("Bash", "echo $G1T_SESSION").hookSpecificOutput.updatedInput.command, "export G1T_SESSION=ses_sub_def456; echo $G1T_SESSION");
772 assert.equal(pre("PowerShell", "$env:G1T_SESSION='ses_other'; g1t work claim 42"), null, "a session the command sets itself is kept");
773 assert.equal(pre("PowerShell", "Get-ChildItem"), null, "other commands are left alone");
774 // The tool call is recorded as Claude wrote it.
775 const after = plan(ps("$env:G1T_SESSION='ses_sub_def456'; g1t work claim 42", { agent_id: "agent-def456" }), env);
776 assert.deepEqual(after.steps[0].args.at(-1), "PowerShell: g1t work claim 42");
777 assert.equal(summarizeTool("Bash", { command: "export G1T_SESSION=ses_1; g1t pr view" }), "Bash: g1t pr view");
778 });
779
780 test("a skill typed as /plugin:name is recorded once in its turn, however Claude Code then runs it", () => {
781 assert.equal(typedSkill("/g1t:work-on-issue 42"), "g1t:work-on-issue");
782 assert.equal(typedSkill("/g1t:status"), "g1t:status");
783 assert.equal(typedSkill("/clear"), null, "Claude Code's own commands are not skills");
784 assert.equal(typedSkill("look at /g1t:status"), null);
785 hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, env);
786 work({ cwd: dir, sid: base.session_id, start: true, steps: [] }, env);
787 const typed = hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "/g1t:work-on-issue 42" }, env).job;
788 assert.deepEqual(typed.steps.map((s) => s.args.slice(0, 4)), [["session", "log", "--kind", "prompt"], ["session", "log", "--kind", "skill"]]);
789 work(typed, env);
790 const tool = hook({ ...base, cwd: dir, hook_event_name: "PostToolUse", tool_name: "Skill", tool_input: { skill: "g1t:work-on-issue", args: "42" } }, env).job;
791 work(tool, env);
792 const skills = calls().filter((c) => c.args.includes("skill"));
793 assert.equal(skills.length, 2);
794 assert.deepEqual(skills[0].args, ["session", "log", "--kind", "skill", "--key=skill:work-on-issue:1", "--", "g1t:work-on-issue"]);
795 assert.equal(skills[1].args[4], "--key=skill:work-on-issue:1", "the same key: g1t keeps the first");
796 // Another skill in the turn has a key of its own; the next prompt is another turn.
797 const next = hook({ ...base, cwd: dir, hook_event_name: "PostToolUse", tool_name: "Skill", tool_input: { skill: "g1t:coordinate" } }, env).job;
798 assert.equal(next.steps[0].key, "skill:coordinate:1");
799 hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "go on" }, { ...env, G1T_RECORD_PROMPTS: "0" });
800 const later = hook({ ...base, cwd: dir, hook_event_name: "PostToolUse", tool_name: "Skill", tool_input: { skill: "g1t:work-on-issue" } }, env).job;
801 assert.equal(later.steps[0].key, "skill:work-on-issue:2", "a turn whose prompt is not sent is still a turn");
802 // Prompts kept back: the typed skill's name is still recorded, the prompt is not.
803 const quiet = hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "/g1t:status" }, { ...env, G1T_RECORD_PROMPTS: "0" }).job;
804 assert.deepEqual(quiet.steps.map((s) => s.args.at(-1)), ["g1t:status"]);
805 });
806
807 test("with no issue in sight, Claude is told the session is recorded and when to open or claim one", () => {
808 git("symbolic-ref", "HEAD", "refs/heads/fix-previews");
809 const { output } = hook({ ...base, cwd: dir, hook_event_name: "SessionStart", source: "startup" }, env);
810 const context = output.hookSpecificOutput.additionalContext;
811 assert.match(context, /this session is recorded on g1t and shows to the team/);
812 assert.match(context, /Just work/);
813 assert.match(context, /`g1t work claim --title "…"`/);
814 assert.match(context, /Never open issues for questions, exploration or trivial fixes/);
815 assert.ok(context.length < 900, `injected every session, so short: ${context.length}`);
816 assert.ok(!calls().some((c) => c.args[0] === "work"), "no issue's context is asked for");
817 });
818
819 test("the session's first prompt with an intent titles it, once, and never over a title it has", () => {
820 hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, env);
821 work({ cwd: dir, sid: base.session_id, start: true, steps: [] }, env);
822 const prompt = (text, more = {}) => hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: text }, { ...env, ...more }).job;
823 // A slash command says nothing of what the session is for.
824 work(prompt("/clear"), env);
825 assert.ok(!calls().some((c) => c.args[1] === "attach" || c.args[1] === "heartbeat"));
826 work(prompt("[TEST] fix the preview builds. They time out."), env);
827 const titled = calls().filter((c) => c.args[1] === "heartbeat" || c.args[1] === "attach");
828 assert.deepEqual(titled, [
829 { args: ["session", "heartbeat", "--json"], session: "ses_fake01" },
830 { args: ["session", "attach", "--title=[TEST] fix the preview builds"], session: "ses_fake01" },
831 ]);
832 // Later prompts do not try again.
833 const next = prompt("now the tests");
834 assert.ok(!next.steps.some((s) => s.title));
835 });
836
837 test("a session titled already keeps its title", () => {
838 hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, env);
839 work({ cwd: dir, sid: base.session_id, start: true, steps: [] }, env);
840 const job = hook({ ...base, cwd: dir, hook_event_name: "UserPromptSubmit", prompt: "fix the login" }, env).job;
841 work(job, { ...env, FAKE_G1T_TITLE: "Rewrite the auth flow" });
842 assert.ok(calls().some((c) => c.args[1] === "heartbeat"));
843 assert.ok(!calls().some((c) => c.args[1] === "attach"), "a title set before is never replaced");
844 });
845
846 test("Claude's own commands get the session's id once it has started", () => {
847 const envFile = join(dir, "env.sh");
848 const { job } = hook({ ...base, cwd: dir, hook_event_name: "SessionStart" }, { ...env, CLAUDE_ENV_FILE: envFile });
849 assert.ok(!existsSync(envFile), "nothing is written before the session's id is known");
850 work(job, env);
851 // A plain assignment: Claude Code gives it to PowerShell commands too.
852 assert.equal(readFileSync(envFile, "utf8"), "export G1T_SESSION=ses_fake01\nexport G1T_TOUCH=0\n");
853 });
854
855 test("on a Node older than 18 nothing is recorded, and that is said once", () => {
856 const at = (hook_event_name) => ({ ...base, cwd: dir, hook_event_name });
857 assert.equal(oldNode(at("SessionStart"), env, "18.0.0"), undefined);
858 const first = oldNode(at("SessionStart"), env, "16.20.2");
859 assert.match(first.systemMessage, /need Node 18 or later, and `node` is v16\.20\.2/);
860 assert.equal(oldNode(at("UserPromptSubmit"), env, "16.20.2"), null);
861 assert.equal(oldNode(at("PostToolUse"), env, "16.20.2"), null);
862 assert.deepEqual(calls(), []);
863 });
864});
865
866test("pending updates survive delivery and stay isolated between parent and child", () => {
867 const dir = mkdtempSync(join(tmpdir(), "g1t-inbox-"));
868 try {
869 const env = { CLAUDE_PLUGIN_DATA: dir };
870 const files = sessionFiles("harness", env);
871 mkdirSync(files.dir, { recursive: true });
872 writeFileSync(files.id, "ses_parent");
873 writeFileSync(files.agent("child"), "ses_child");
874 writeFileSync(files.updates("ses_parent"), JSON.stringify([{ id: "parent-update", kind: "overlap.detected", data: {} }]));
875 writeFileSync(files.updates("ses_child"), JSON.stringify([{ id: "child-update", kind: "dependency.changed", data: {} }]));
876 const event = { session_id: "harness" };
877 const first = updateContext(event, env);
878 assert.match(first, /parent-update/);
879 assert.doesNotMatch(first, /child-update/);
880 assert.equal(updateContext(event, env), first);
881 // Before tools: shown once, again when it changes or after a while.
882 const files2 = sessionFiles("harness", env);
883 rmSync(files2.shown("ses_parent"), { force: true });
884 assert.equal(updateContext(event, env, { throttle: true, now: 1000 }), first);
885 assert.equal(updateContext(event, env, { throttle: true, now: 2000 }), null);
886 assert.equal(updateContext(event, env, { throttle: true, now: 1000 + UPDATES_REPEAT_MS }), first);
887 writeFileSync(files.updates("ses_parent"), JSON.stringify([{ id: "parent-update", kind: "overlap.detected", data: {} }, { id: "newer", kind: "work.claimed", data: {} }]));
888 assert.match(updateContext(event, env, { throttle: true, now: 1001 + UPDATES_REPEAT_MS }), /newer/);
889 writeFileSync(files.updates("ses_parent"), JSON.stringify([{ id: "parent-update", kind: "overlap.detected", data: {} }]));
890 const child = updateContext({ ...event, agent_id: "child" }, env);
891 assert.match(child, /child-update/);
892 assert.doesNotMatch(child, /parent-update/);
893 assert.equal(updateContext({ ...event, agent_id: "unknown" }, env), null);
894 writeFileSync(files.updates("ses_parent"), JSON.stringify([{ id: "oversized", data: "x".repeat(20000) }]));
895 const bounded = updateContext(event, env);
896 assert.ok(bounded.length < 10000);
897 assert.match(bounded, /read get_session first/);
898 // A payload that was omitted is not cleared from the cache.
899 assert.match(readFileSync(files.updates("ses_parent"), "utf8"), /oversized/);
900 writeFileSync(files.updates("ses_parent"), "[]");
901 assert.equal(updateContext(event, env), null);
902 assert.equal(updateContext({ ...event, agent_id: "child" }, env), child);
903 } finally { rmSync(dir, { recursive: true, force: true }); }
904});
905
906test("a CLI too old to acknowledge is told to update, once per set of updates", () => {
907 const dir = mkdtempSync(join(tmpdir(), "g1t-stuck-"));
908 try {
909 const env = { CLAUDE_PLUGIN_DATA: dir };
910 const files = sessionFiles("harness", env);
911 mkdirSync(files.dir, { recursive: true });
912 writeFileSync(files.id, "ses_old");
913 writeFileSync(join(dir, "cli.json"), JSON.stringify({ stamp: "x", version: "0.3.2" }));
914 writeFileSync(files.updates("ses_old"), JSON.stringify([{ id: "upd_1", kind: "session.ended", data: {} }]));
915 const event = { session_id: "harness" };
916 const first = updateContext(event, env);
917 assert.match(first, /upd_1/);
918 assert.match(first, /cannot acknowledge them \(0\.3\.3 can\)/);
919 assert.match(first, /install\.ps1/, "0.3.2 has no g1t upgrade: the installer");
920 assert.doesNotMatch(first, /g1t session acknowledge/);
921 // Nothing can clear them, so not again on every prompt or tool.
922 assert.equal(updateContext(event, env), null);
923 assert.equal(updateContext(event, env, { throttle: true, now: Date.now() + 10 * UPDATES_REPEAT_MS }), null);
924 writeFileSync(files.updates("ses_old"), JSON.stringify([{ id: "upd_1", data: {} }, { id: "upd_2", data: {} }]));
925 assert.match(updateContext(event, env), /upd_2/);
926 // A CLI that acknowledges is asked to.
927 writeFileSync(join(dir, "cli.json"), JSON.stringify({ stamp: "x", version: "0.3.3" }));
928 assert.match(updateContext(event, env), /g1t session acknowledge UPDATE_ID --session ses_old/);
929 assert.equal(howToUpdate("0.3.4"), "`g1t upgrade`");
930 assert.match(howToUpdate("0.3.3"), /curl -fsSL https:\/\/g1t\.sh\/install \| sh/);
931 assert.match(howToUpdate(null), /installer/);
932 } finally { rmSync(dir, { recursive: true, force: true }); }
933});
934
935test("an acknowledgement refreshes the inbox at once, from the shell or MCP", () => {
936 const dir = mkdtempSync(join(tmpdir(), "g1t-ack-"));
937 try {
938 const env = { CLAUDE_PLUGIN_DATA: dir };
939 const base = { session_id: "harness", cwd: dir, hook_event_name: "PostToolUse" };
940 const shell = plan({ ...base, tool_name: "Bash", tool_input: { command: "g1t session acknowledge upd_1 --note 'Separate scopes'" } }, env);
941 assert.equal(shell.refreshUpdates, true);
942 const mcp = plan({ ...base, tool_name: "mcp__plugin_g1t_g1t__session", tool_input: { action: "record", id: "ses_a", acknowledge: ["upd_1"] } }, env);
943 assert.equal(mcp.refreshUpdates, true);
944 const other = plan({ ...base, tool_name: "Bash", tool_input: { command: "git status" } }, env);
945 assert.equal(other.refreshUpdates, undefined);
946 } finally { rmSync(dir, { recursive: true, force: true }); }
947});