Miguel PalhasandClaude Opus 4.6
88afd0df3f
Structured sessions ignored the --dangerously-skip-permissions flags in
preset commands because the daemon discards CLI flags when spawning the
ACP adapter binary. This adds an `auto_approve` bool to SpawnPreset and
CreateSessionRequest so the operator can declare it in config.toml:
{ id = "claude", command = "claude", auto_approve = true }
When set, the daemon's permission pump immediately responds AllowAlways
to every ACP permission request without surfacing it to clients.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
122 lines
3.6 KiB
TypeScript
122 lines
3.6 KiB
TypeScript
import { describe, expect, it } from "vitest";
|
|
|
|
import { buildSpawnRequest, type SpawnFormValues } from "./spawnForm.js";
|
|
|
|
const base: SpawnFormValues = {
|
|
command: "claude",
|
|
cwd: "/home/user/project",
|
|
name: "amber-fox",
|
|
structuredView: false,
|
|
autoApprove: false,
|
|
worktreeEnabled: false,
|
|
worktreeBranch: "",
|
|
worktreeBranchMode: "new",
|
|
worktreeBranchEdited: false,
|
|
};
|
|
|
|
describe("buildSpawnRequest", () => {
|
|
it("builds a plain request with the picked cwd and command", () => {
|
|
const res = buildSpawnRequest(base, null);
|
|
expect(res).toEqual({
|
|
req: { cwd: "/home/user/project", command: "claude", metadata: { name: "amber-fox" } },
|
|
});
|
|
});
|
|
|
|
it("omits metadata.name when the name is blank", () => {
|
|
const res = buildSpawnRequest({ ...base, name: " " }, null);
|
|
expect(res).toEqual({ req: { cwd: base.cwd, command: base.command, metadata: {} } });
|
|
});
|
|
|
|
it.each(["claude", "FOO=1 /usr/local/bin/codex", "env -i codex"])(
|
|
"requests structured view for resolved %s commands",
|
|
(command) => {
|
|
expect(buildSpawnRequest({ ...base, command, structuredView: true }, null)).toEqual({
|
|
req: {
|
|
cwd: base.cwd,
|
|
command,
|
|
metadata: { name: "amber-fox" },
|
|
view: "Structured",
|
|
},
|
|
});
|
|
},
|
|
);
|
|
|
|
it("does not request structured view for unsupported commands", () => {
|
|
expect(
|
|
buildSpawnRequest({ ...base, command: "opencode", structuredView: true }, null),
|
|
).toEqual({
|
|
req: { cwd: base.cwd, command: "opencode", metadata: { name: "amber-fox" } },
|
|
});
|
|
});
|
|
|
|
it("attaches a trimmed branch when worktree is enabled, and always cleans up", () => {
|
|
const res = buildSpawnRequest(
|
|
{ ...base, worktreeEnabled: true, worktreeBranch: " feat/foo " },
|
|
"/home/user/project",
|
|
);
|
|
expect(res).toEqual({
|
|
req: {
|
|
cwd: base.cwd,
|
|
command: base.command,
|
|
metadata: { name: "amber-fox" },
|
|
worktree: {
|
|
base_repo: "/home/user/project",
|
|
branch: "feat/foo",
|
|
create_branch: true,
|
|
},
|
|
},
|
|
});
|
|
});
|
|
|
|
it("checks out an existing branch when the mode is existing (#61)", () => {
|
|
const res = buildSpawnRequest(
|
|
{
|
|
...base,
|
|
worktreeEnabled: true,
|
|
worktreeBranch: "spawn-v2",
|
|
worktreeBranchMode: "existing",
|
|
},
|
|
"/home/user/project",
|
|
);
|
|
expect(res).toEqual({
|
|
req: {
|
|
cwd: base.cwd,
|
|
command: base.command,
|
|
metadata: { name: "amber-fox" },
|
|
worktree: {
|
|
base_repo: "/home/user/project",
|
|
branch: "spawn-v2",
|
|
create_branch: false,
|
|
},
|
|
},
|
|
});
|
|
});
|
|
|
|
it("rejects an existing-mode worktree with no branch picked", () => {
|
|
expect(
|
|
buildSpawnRequest(
|
|
{ ...base, worktreeEnabled: true, worktreeBranch: "", worktreeBranchMode: "existing" },
|
|
"/repo",
|
|
),
|
|
).toHaveProperty("error");
|
|
});
|
|
|
|
it("rejects a missing working directory", () => {
|
|
expect(buildSpawnRequest({ ...base, cwd: "" }, null)).toHaveProperty("error");
|
|
});
|
|
|
|
it("rejects a missing command", () => {
|
|
expect(buildSpawnRequest({ ...base, command: "" }, null)).toHaveProperty("error");
|
|
});
|
|
|
|
it("rejects a worktree without an inferred base repo", () => {
|
|
expect(buildSpawnRequest({ ...base, worktreeEnabled: true, worktreeBranch: "b" }, null))
|
|
.toHaveProperty("error");
|
|
});
|
|
|
|
it("rejects a worktree without a branch name", () => {
|
|
expect(buildSpawnRequest({ ...base, worktreeEnabled: true, worktreeBranch: " " }, "/repo"))
|
|
.toHaveProperty("error");
|
|
});
|
|
});
|