@clinebot/core 0.0.34 → 0.0.36

package/src/team/spawn-agent-tool.test.ts

@@ -1,387 +0,0 @@
-import type { AgentConfig } from "@clinebot/shared";
-import { beforeEach, describe, expect, it, vi } from "vitest";
-import { createDelegatedAgentConfigProvider } from "./delegated-agent";
-
-type AgentExtension = NonNullable<AgentConfig["extensions"]>[number];
-
-const runMock = vi.fn();
-const getAgentIdMock = vi.fn(() => "sub-agent-1");
-const getConversationIdMock = vi.fn(() => "conv-sub-1");
-const agentConstructorSpy = vi.fn();
-
-vi.mock("@clinebot/agents", async () => {
-	const actual =
-		await vi.importActual<typeof import("@clinebot/agents")>(
-			"@clinebot/agents",
-		);
-
-	return {
-		...actual,
-		Agent: class MockAgent {
-			constructor(config: unknown) {
-				agentConstructorSpy(config);
-			}
-
-			getAgentId(): string {
-				return getAgentIdMock();
-			}
-
-			getConversationId(): string {
-				return getConversationIdMock();
-			}
-
-			async run(input: string): Promise<unknown> {
-				return runMock(input);
-			}
-		},
-	};
-});
-
-describe("createSpawnAgentTool", () => {
-	beforeEach(() => {
-		vi.clearAllMocks();
-	});
-
-	it("creates a sub-agent, forwards callbacks, and returns normalized output", async () => {
-		const { createSpawnAgentTool } = await import("./spawn-agent-tool.js");
-		runMock.mockResolvedValue({
-			text: "sub-agent result",
-			iterations: 2,
-			finishReason: "completed",
-			usage: { inputTokens: 11, outputTokens: 7 },
-		});
-
-		const onSubAgentStart = vi.fn();
-		const onSubAgentEnd = vi.fn();
-		const createSubAgentTools = vi.fn().mockResolvedValue([]);
-		const extensions = [
-			{
-				name: "sample-ext",
-				manifest: { capabilities: ["hooks"], hookStages: ["runtime_event"] },
-				onRuntimeEvent: vi.fn(),
-			} as AgentExtension,
-		];
-
-		const tool = createSpawnAgentTool({
-			configProvider: createDelegatedAgentConfigProvider({
-				providerId: "anthropic",
-				modelId: "mock-model",
-				extensions,
-			}),
-			defaultMaxIterations: 4,
-			createSubAgentTools,
-			onSubAgentStart,
-			onSubAgentEnd,
-		});
-
-		const output = await tool.execute(
-			{
-				systemPrompt: "You are focused",
-				task: "Do delegated work",
-			},
-			{
-				agentId: "parent-1",
-				conversationId: "conv-parent",
-				iteration: 3,
-			},
-		);
-
-		expect(createSubAgentTools).toHaveBeenCalledTimes(1);
-		expect(runMock).toHaveBeenCalledWith("Do delegated work");
-		expect(onSubAgentStart).toHaveBeenCalledTimes(1);
-		expect(onSubAgentEnd).toHaveBeenCalledTimes(1);
-		expect(output).toEqual({
-			text: "sub-agent result",
-			iterations: 2,
-			finishReason: "completed",
-			usage: {
-				inputTokens: 11,
-				outputTokens: 7,
-			},
-		});
-		expect(agentConstructorSpy).toHaveBeenCalledWith(
-			expect.objectContaining({
-				parentAgentId: "parent-1",
-				maxIterations: 4,
-				extensions,
-			}),
-		);
-		expect(agentConstructorSpy).toHaveBeenCalledWith(
-			expect.not.objectContaining({
-				prepareTurn: expect.anything(),
-			}),
-		);
-	});
-
-	it("passes extension hooks through delegated config", async () => {
-		const { createSpawnAgentTool } = await import("./spawn-agent-tool.js");
-		runMock.mockResolvedValue({
-			text: "sub-agent result",
-			iterations: 1,
-			finishReason: "completed",
-			usage: { inputTokens: 1, outputTokens: 1 },
-		});
-
-		const extensions = [
-			{
-				name: "before-start-ext",
-				manifest: {
-					capabilities: ["hooks"],
-					hookStages: ["before_agent_start"],
-				},
-				onBeforeAgentStart: vi.fn(),
-			} as AgentExtension,
-		];
-
-		const tool = createSpawnAgentTool({
-			configProvider: createDelegatedAgentConfigProvider({
-				providerId: "anthropic",
-				modelId: "mock-model",
-				extensions,
-			}),
-			subAgentTools: [],
-		});
-
-		await tool.execute(
-			{
-				systemPrompt: "You are focused",
-				task: "Do delegated work",
-			},
-			{
-				agentId: "parent-1",
-				conversationId: "conv-parent",
-				iteration: 3,
-			},
-		);
-
-		expect(agentConstructorSpy).toHaveBeenCalledWith(
-			expect.objectContaining({
-				extensions,
-			}),
-		);
-	});
-
-	it("propagates sub-agent errors and still reports onSubAgentEnd", async () => {
-		const { createSpawnAgentTool } = await import("./spawn-agent-tool.js");
-		runMock.mockRejectedValue(new Error("sub-agent failed"));
-		const onSubAgentEnd = vi.fn();
-
-		const tool = createSpawnAgentTool({
-			configProvider: createDelegatedAgentConfigProvider({
-				providerId: "anthropic",
-				modelId: "mock-model",
-			}),
-			subAgentTools: [],
-			onSubAgentEnd,
-		});
-
-		await expect(
-			tool.execute(
-				{
-					systemPrompt: "System",
-					task: "Fail task",
-					maxIterations: 6,
-				},
-				{
-					agentId: "parent-2",
-					conversationId: "conv-parent",
-					iteration: 1,
-				},
-			),
-		).rejects.toThrow("sub-agent failed");
-
-		expect(onSubAgentEnd).toHaveBeenCalledTimes(1);
-		expect(onSubAgentEnd).toHaveBeenCalledWith(
-			expect.objectContaining({
-				parentAgentId: "parent-2",
-				error: expect.any(Error),
-			}),
-		);
-		expect(agentConstructorSpy).toHaveBeenCalledWith(
-			expect.objectContaining({
-				maxIterations: 6,
-			}),
-		);
-	});
-
-	it("leaves maxIterations unset when neither input nor default is provided", async () => {
-		const { createSpawnAgentTool } = await import("./spawn-agent-tool.js");
-		runMock.mockResolvedValue({
-			text: "sub-agent result",
-			iterations: 1,
-			finishReason: "completed",
-			usage: { inputTokens: 1, outputTokens: 1 },
-		});
-
-		const tool = createSpawnAgentTool({
-			configProvider: createDelegatedAgentConfigProvider({
-				providerId: "anthropic",
-				modelId: "mock-model",
-			}),
-			subAgentTools: [],
-		});
-
-		await tool.execute(
-			{
-				systemPrompt: "System",
-				task: "Do task",
-			},
-			{
-				agentId: "parent-3",
-				conversationId: "conv-parent",
-				iteration: 1,
-			},
-		);
-
-		expect(agentConstructorSpy).toHaveBeenCalledWith(
-			expect.objectContaining({
-				maxIterations: undefined,
-			}),
-		);
-	});
-
-	it("appends workspace metadata for cline sub-agents when missing", async () => {
-		const { createSpawnAgentTool } = await import("./spawn-agent-tool.js");
-		runMock.mockResolvedValue({
-			text: "ok",
-			iterations: 1,
-			finishReason: "completed",
-			usage: { inputTokens: 1, outputTokens: 1 },
-		});
-
-		const workspaceMetadata = `# Workspace Configuration
-{
-  "workspaces": {
-    "/repo/demo": {
-      "hint": "demo"
-    }
-  }
-}`;
-
-		const tool = createSpawnAgentTool({
-			configProvider: createDelegatedAgentConfigProvider({
-				providerId: "cline",
-				modelId: "anthropic/claude-sonnet-4.6",
-				cwd: "/repo/demo",
-				workspaceMetadata,
-			}),
-			subAgentTools: [],
-		});
-
-		await tool.execute(
-			{
-				systemPrompt: "You are a specialist teammate.",
-				task: "Investigate module boundaries",
-			},
-			{
-				agentId: "parent-4",
-				conversationId: "conv-parent",
-				iteration: 1,
-			},
-		);
-
-		expect(agentConstructorSpy).toHaveBeenCalledWith(
-			expect.objectContaining({
-				systemPrompt: expect.stringContaining(workspaceMetadata),
-			}),
-		);
-	});
-
-	it("does not duplicate workspace metadata for cline sub-agents", async () => {
-		const { createSpawnAgentTool } = await import("./spawn-agent-tool.js");
-		runMock.mockResolvedValue({
-			text: "ok",
-			iterations: 1,
-			finishReason: "completed",
-			usage: { inputTokens: 1, outputTokens: 1 },
-		});
-
-		const inputSystemPrompt = `You are a specialist teammate.
-
-# Workspace Configuration
-{
-  "workspaces": {
-    "/repo/demo": {
-      "hint": "demo"
-    }
-  }
-}`;
-
-		const tool = createSpawnAgentTool({
-			configProvider: createDelegatedAgentConfigProvider({
-				providerId: "cline",
-				modelId: "anthropic/claude-sonnet-4.6",
-				cwd: "/repo/demo",
-				workspaceMetadata: "# Workspace Configuration\n{}",
-			}),
-			subAgentTools: [],
-		});
-
-		await tool.execute(
-			{
-				systemPrompt: inputSystemPrompt,
-				task: "Investigate module boundaries",
-			},
-			{
-				agentId: "parent-5",
-				conversationId: "conv-parent",
-				iteration: 1,
-			},
-		);
-
-		expect(agentConstructorSpy).toHaveBeenCalledWith(
-			expect.objectContaining({
-				systemPrompt: inputSystemPrompt,
-			}),
-		);
-	});
-
-	it("resolves connection settings lazily at execution time", async () => {
-		const { createSpawnAgentTool } = await import("./spawn-agent-tool.js");
-		runMock.mockResolvedValue({
-			text: "ok",
-			iterations: 1,
-			finishReason: "completed",
-			usage: { inputTokens: 1, outputTokens: 1 },
-		});
-
-		const configProvider = createDelegatedAgentConfigProvider({
-			providerId: "cline",
-			modelId: "stale-model",
-			apiKey: "oauth-access-old",
-		});
-		const updateConnectionDefaults = vi.spyOn(
-			configProvider,
-			"updateConnectionDefaults",
-		);
-		configProvider.updateConnectionDefaults({
-			apiKey: "oauth-access-new",
-			modelId: "updated-model",
-		});
-
-		const tool = createSpawnAgentTool({
-			configProvider,
-			subAgentTools: [],
-		});
-
-		await tool.execute(
-			{
-				systemPrompt: "System",
-				task: "Do task",
-			},
-			{
-				agentId: "parent-6",
-				conversationId: "conv-parent",
-				iteration: 1,
-			},
-		);
-
-		expect(updateConnectionDefaults).toHaveBeenCalledTimes(1);
-		expect(agentConstructorSpy).toHaveBeenCalledWith(
-			expect.objectContaining({
-				apiKey: "oauth-access-new",
-				modelId: "updated-model",
-			}),
-		);
-	});
-});