@ai-setting/roy-agent-cli 1.3.19 → 1.3.22

package/dist/core/src/env/agent/agent-component.tool-error.test.d.ts

@@ -1,25 +0,0 @@
-/**
- * @fileoverview TDD Tests for Tool Error Handling in AgentComponent
- *
- * Feature: Tool execution error should be stored as tool result
- *
- * Problem:
- * When a tool throws an exception during execution, the tool call is already
- * added to messages (via pushMessage), but the tool result is NOT added because:
- * 1. executeTool() throws the error
- * 2. The catch block in the ReAct loop catches the error
- * 3. Tool result pushMessage() is never reached
- *
- * This breaks AI SDK v6's requirement for 1:1 tool-call to tool-result mapping.
- *
- * Solution:
- * Modify the error handling to always add tool result (with error content) even
- * when tool execution throws an exception. This makes the behavior uniform
- * regardless of whether the tool succeeds, returns an error, or throws an exception.
- *
- * For ask_user special case: the exception is stored as tool result (unlike other
- * exceptions which are swallowed), allowing resume flow to replace it with actual
- * user response.
- */
-export {};
-//# sourceMappingURL=agent-component.tool-error.test.d.ts.map

package/dist/core/src/env/agent/agent-component.tool-error.test.js

@@ -1,272 +0,0 @@
-/**
- * @fileoverview TDD Tests for Tool Error Handling in AgentComponent
- *
- * Feature: Tool execution error should be stored as tool result
- *
- * Problem:
- * When a tool throws an exception during execution, the tool call is already
- * added to messages (via pushMessage), but the tool result is NOT added because:
- * 1. executeTool() throws the error
- * 2. The catch block in the ReAct loop catches the error
- * 3. Tool result pushMessage() is never reached
- *
- * This breaks AI SDK v6's requirement for 1:1 tool-call to tool-result mapping.
- *
- * Solution:
- * Modify the error handling to always add tool result (with error content) even
- * when tool execution throws an exception. This makes the behavior uniform
- * regardless of whether the tool succeeds, returns an error, or throws an exception.
- *
- * For ask_user special case: the exception is stored as tool result (unlike other
- * exceptions which are swallowed), allowing resume flow to replace it with actual
- * user response.
- */
-import { describe, it, expect, vi, beforeEach, afterEach } from "vitest";
-import { AgentComponent } from "./agent-component.js";
-import { ConfigComponent } from "../../config/config-component.js";
-import { AskUserError } from "../workflow/types/workflow-hil.js";
-import { z } from "zod";
-// ============================================================================
-// Mock Environment Factory
-// ============================================================================
-function createMockEnvironment(mockComponents = {}) {
-    return {
-        name: "test-env",
-        version: "1.0.0",
-        getConfig: vi.fn(() => ({ name: "test-env", version: "1.0.0", enabled: true })),
-        registerComponent: vi.fn(),
-        unregisterComponent: vi.fn(),
-        getComponent: vi.fn((name) => {
-            return mockComponents[name];
-        }),
-        listComponents: vi.fn(() => Object.keys(mockComponents)),
-        handle_query: vi.fn(),
-        handle_action: vi.fn(),
-        subscribe: vi.fn(() => () => { }),
-        subscribeTo: vi.fn(() => () => { }),
-        subscribeAll: vi.fn(() => () => { }),
-        pushEnvEvent: vi.fn(),
-    };
-}
-// ============================================================================
-// Test Suite: Tool Error Handling
-// ============================================================================
-describe("AgentComponent Tool Error Handling (TDD)", () => {
-    let agentComponent;
-    let mockEnv;
-    let configComponent;
-    let mockSessionComponent;
-    beforeEach(async () => {
-        vi.clearAllMocks();
-        configComponent = new ConfigComponent();
-        // Create mock session component to capture stored messages
-        mockSessionComponent = {
-            name: "session",
-            create: vi.fn(async () => ({ id: "test-session", title: "Test" })),
-            get: vi.fn(async () => ({ id: "test-session" })),
-            addMessage: vi.fn(async () => "msg_id"),
-            getMessages: vi.fn(async () => []),
-            getContext: vi.fn(async () => ({
-                session: { id: "test-session" },
-                messages: [],
-            })),
-        };
-        mockEnv = createMockEnvironment({
-            config: configComponent,
-            session: mockSessionComponent,
-        });
-        agentComponent = new AgentComponent();
-        await agentComponent.init({
-            name: "agent",
-            version: "1.0.0",
-            enabled: true,
-            env: mockEnv,
-            options: { configComponent }
-        });
-    });
-    afterEach(() => {
-        vi.restoreAllMocks();
-    });
-    // ===========================================================================
-    // RED Phase: Write failing tests first
-    // ===========================================================================
-    describe("RED: AskUserError should be stored as tool result", () => {
-        it("should store AskUserError as tool result for ask_user tool", async () => {
-            // Setup agent
-            agentComponent.registerAgent("test-agent", {
-                type: "primary",
-                systemPrompt: "You are a helpful assistant.",
-            });
-            // Mock LLM that returns ask_user tool call
-            const mockLLMComponent = {
-                invoke: vi.fn(() => Promise.resolve({
-                    output: {
-                        content: "",
-                        toolCalls: [{
-                                id: "call_1",
-                                function: { name: "ask_user", arguments: JSON.stringify({ query: "Continue?" }) }
-                            }]
-                    }
-                }))
-            };
-            mockEnv.getComponent.mockImplementation((name) => {
-                if (name === "llm")
-                    return mockLLMComponent;
-                if (name === "session")
-                    return mockSessionComponent;
-                return undefined;
-            });
-            // Mock ask_user tool that throws AskUserError
-            const mockAskUserTool = {
-                name: "ask_user",
-                description: "Ask user for input",
-                parameters: z.object({
-                    query: z.string(),
-                }),
-                execute: vi.fn(async () => {
-                    throw new AskUserError("run_123", "test-session", "node_1", "agent", "Continue?");
-                }),
-            };
-            agentComponent.setDefaultTools([mockAskUserTool]);
-            // Execute agent
-            const result = await agentComponent.run("test-agent", "Test ask_user", {
-                sessionId: "test-session",
-            });
-            // Verify: result should contain __ASK_USER_ERROR__ marker
-            expect(result.error).toBeDefined();
-            expect(result.error).toContain("__ASK_USER_ERROR__");
-            // Verify: session should have tool result with error content
-            const storedMessages = mockSessionComponent.addMessage.mock.calls;
-            const toolMessages = storedMessages.filter((call) => call[1]?.role === "tool");
-            // For ask_user, tool result should be stored with error info
-            expect(toolMessages.length).toBeGreaterThan(0);
-            const toolMessage = toolMessages[0][1];
-            // Content might be empty, check parts for output
-            const hasParts = toolMessage.parts && toolMessage.parts.length > 0;
-            const hasOutput = hasParts && toolMessage.parts[0]?.output;
-            // Either content or parts output should contain the query
-            const toolContent = typeof toolMessage.content === "string"
-                ? toolMessage.content
-                : "";
-            const partsOutput = hasOutput ? JSON.stringify(toolMessage.parts[0].output) : "";
-            // Should contain error info from AskUserError (query in output)
-            expect(toolContent + partsOutput).toContain("Continue?");
-        });
-    });
-    describe("RED: Resume flow should replace tool result", () => {
-        it("should allow replacing tool result with user response on resume", async () => {
-            // This test verifies the resume flow design
-            // When resuming, the stored tool result (with error) should be 
-            // replaced with the actual user response
-            const agentSessionId = "agent_session_123";
-            // Simulate stored messages after initial run (with tool error stored)
-            const storedToolResult = {
-                role: "tool",
-                content: JSON.stringify({ error: "AskUserError: Continue?" }),
-                metadata: {
-                    toolCallId: "call_1",
-                    toolName: "ask_user",
-                    isAskUserError: true,
-                },
-            };
-            // Simulate resume - user response should replace tool result
-            const userResponse = "yes, continue";
-            // Resume would call addMessage to add user response
-            // The key insight: on resume, we need to either:
-            // 1. Replace the existing tool result message, or
-            // 2. Add user response as a new message and skip the error tool result
-            // For now, verify the session can store both
-            mockSessionComponent.addMessage(agentSessionId, {
-                role: "user",
-                content: userResponse,
-                metadata: { type: "user_response" },
-            });
-            // Verify user response was stored
-            expect(mockSessionComponent.addMessage).toHaveBeenCalledWith(agentSessionId, expect.objectContaining({
-                role: "user",
-                content: userResponse,
-            }));
-        });
-    });
-    describe("RED: Normal tool error should also be stored", () => {
-        it("should store normal tool error as tool result", async () => {
-            // Setup agent
-            agentComponent.registerAgent("test-agent", {
-                type: "primary",
-                systemPrompt: "You are a helpful assistant.",
-                maxIterations: 1, // Limit iterations to avoid infinite loop
-            });
-            // Mock LLM that returns a tool call
-            const mockLLMComponent = {
-                invoke: vi.fn(() => Promise.resolve({
-                    output: {
-                        content: "",
-                        toolCalls: [{
-                                id: "call_1",
-                                function: { name: "error_tool", arguments: "{}" }
-                            }]
-                    }
-                }))
-            };
-            mockEnv.getComponent.mockImplementation((name) => {
-                if (name === "llm")
-                    return mockLLMComponent;
-                if (name === "session")
-                    return mockSessionComponent;
-                return undefined;
-            });
-            // Mock tool that returns error result (not throws)
-            const mockTool = {
-                name: "error_tool",
-                description: "A tool that returns error",
-                parameters: z.object({}),
-                execute: vi.fn(async () => {
-                    return { success: false, output: "", error: "Tool execution failed!" };
-                }),
-            };
-            agentComponent.setDefaultTools([mockTool]);
-            // Execute agent
-            const result = await agentComponent.run("test-agent", "Test tool error", {
-                sessionId: "test-session",
-            });
-            // Verify: session should have BOTH tool call and tool result
-            expect(mockSessionComponent.addMessage).toHaveBeenCalled();
-            const storedMessages = mockSessionComponent.addMessage.mock.calls;
-            // Find tool result message
-            const toolMessages = storedMessages.filter((call) => call[1]?.role === "tool");
-            expect(toolMessages.length).toBeGreaterThan(0);
-            // The tool result should contain the error message
-            const toolMessage = toolMessages[0][1];
-            const hasParts = toolMessage.parts && toolMessage.parts.length > 0;
-            const hasOutput = hasParts && toolMessage.parts[0]?.output;
-            const toolContent = typeof toolMessage.content === "string"
-                ? toolMessage.content
-                : "";
-            const partsOutput = hasOutput ? JSON.stringify(toolMessage.parts[0].output) : "";
-            expect(toolContent + partsOutput).toContain("Tool execution failed");
-        });
-    });
-    // ===========================================================================
-    // Helper: Run agent with mocked LLM
-    // ===========================================================================
-    /**
-     * Helper function to run agent with custom LLM mock
-     */
-    async function runAgentWithLLMMock(agentName, llmResponse, tools, sessionId) {
-        const mockLLMComponent = {
-            invoke: vi.fn(() => Promise.resolve(llmResponse)),
-        };
-        mockEnv.getComponent.mockImplementation((name) => {
-            if (name === "llm")
-                return mockLLMComponent;
-            if (name === "session")
-                return mockSessionComponent;
-            return undefined;
-        });
-        agentComponent.setDefaultTools(tools);
-        return agentComponent.run(agentName, "Test query", {
-            sessionId: sessionId || "test-session",
-        });
-    }
-});
-//# sourceMappingURL=agent-component.tool-error.test.js.map

package/dist/core/src/env/agent/agent-config-registration.d.ts

@@ -1,19 +0,0 @@
-/**
- * @fileoverview Agent Component 配置注册
- *
- * 使用 ConfigComponent.registerComponent() 机制统一管理配置
- */
-import type { ComponentRegistration } from "../../config/config-component";
-/**
- * Agent Component 默认配置值
- *
- * 注意：键使用完整格式（带 "agent." 前缀），在 registerConfig 中手动设置
- */
-export declare const AGENT_DEFAULTS: Record<string, unknown>;
-/**
- * Agent Component 配置注册常量
- *
- * 定义了所有 Agent 相关的配置项和来源
- */
-export declare const AGENT_CONFIG_REGISTRATION: ComponentRegistration;
-//# sourceMappingURL=agent-config-registration.d.ts.map

package/dist/core/src/env/agent/agent-config-registration.js

@@ -1,41 +0,0 @@
-/**
- * @fileoverview Agent Component 配置注册
- *
- * 使用 ConfigComponent.registerComponent() 机制统一管理配置
- */
-/**
- * Agent Component 默认配置值
- *
- * 注意：键使用完整格式（带 "agent." 前缀），在 registerConfig 中手动设置
- */
-export const AGENT_DEFAULTS = {
-    "agent.defaultAgent.maxIterations": 200,
-    "agent.defaultAgent.temperature": 0,
-    "agent.defaultAgent.timeout": 300000,
-    "agent.defaultAgent.streaming": false,
-    "agent.defaultAgent.filterHistory": false,
-};
-/**
- * Agent Component 配置注册常量
- *
- * 定义了所有 Agent 相关的配置项和来源
- */
-export const AGENT_CONFIG_REGISTRATION = {
-    name: "agent",
-    sources: [
-        { type: "env", envPrefix: "AGENT", priority: 20, watch: false },
-    ],
-    keys: [
-        { key: "agent.defaultAgent.model", sources: ["env", "file"] },
-        { key: "agent.defaultAgent.maxIterations", sources: ["env", "file"] },
-        { key: "agent.defaultAgent.temperature", sources: ["env", "file"] },
-        { key: "agent.defaultAgent.timeout", sources: ["env", "file"] },
-        { key: "agent.defaultAgent.systemPrompt", sources: ["env", "file"] },
-        { key: "agent.defaultAgent.streaming", sources: ["env", "file"] },
-        { key: "agent.defaultAgent.tools", sources: ["env", "file"] },
-        { key: "agent.defaultAgent.maxErrorRetries", sources: ["env", "file"] },
-        { key: "agent.defaultAgent.filterHistory", sources: ["env", "file"] },
-    ],
-    // 注意：不使用 defaults 字段，默认值在 registerConfig 中手动 set
-};
-//# sourceMappingURL=agent-config-registration.js.map

package/dist/core/src/env/agent/agent-config-registration.test.d.ts

@@ -1,7 +0,0 @@
-/**
- * @fileoverview Agent Component 配置注册测试
- *
- * TDD: 先写测试，验证配置注册机制
- */
-export {};
-//# sourceMappingURL=agent-config-registration.test.d.ts.map

package/dist/core/src/env/agent/agent-config-registration.test.js

@@ -1,193 +0,0 @@
-/**
- * @fileoverview Agent Component 配置注册测试
- *
- * TDD: 先写测试，验证配置注册机制
- */
-import { describe, it, expect, beforeEach } from "vitest";
-import { ConfigComponent } from "../../config/config-component.js";
-import { AgentComponent } from "./agent-component.js";
-import { AGENT_CONFIG_REGISTRATION, AGENT_DEFAULTS } from "./agent-config-registration.js";
-describe("Agent Config Registration", () => {
-    let configComponent;
-    let component;
-    beforeEach(() => {
-        configComponent = new ConfigComponent();
-        component = new AgentComponent();
-    });
-    describe("AGENT_CONFIG_REGISTRATION", () => {
-        it("应该包含正确的 name", () => {
-            expect(AGENT_CONFIG_REGISTRATION.name).toBe("agent");
-        });
-        it("应该包含 env source 配置", () => {
-            const envSource = AGENT_CONFIG_REGISTRATION.sources.find((s) => s.type === "env");
-            expect(envSource).toBeDefined();
-            expect(envSource?.envPrefix).toBe("AGENT");
-            expect(envSource?.priority).toBe(20);
-        });
-        it("应该包含所有配置 keys", () => {
-            const keys = AGENT_CONFIG_REGISTRATION.keys;
-            expect(keys).toContainEqual(expect.objectContaining({ key: "agent.defaultAgent.model" }));
-            expect(keys).toContainEqual(expect.objectContaining({ key: "agent.defaultAgent.maxIterations" }));
-            expect(keys).toContainEqual(expect.objectContaining({ key: "agent.defaultAgent.temperature" }));
-            expect(keys).toContainEqual(expect.objectContaining({ key: "agent.defaultAgent.systemPrompt" }));
-        });
-    });
-    describe("AGENT_DEFAULTS", () => {
-        it("应该包含 maxIterations 默认值（完整格式）", () => {
-            expect(AGENT_DEFAULTS["agent.defaultAgent.maxIterations"]).toBe(200);
-        });
-        it("应该包含 temperature 默认值（完整格式）", () => {
-            expect(AGENT_DEFAULTS["agent.defaultAgent.temperature"]).toBe(0);
-        });
-        it("应该包含 timeout 默认值（完整格式）", () => {
-            expect(AGENT_DEFAULTS["agent.defaultAgent.timeout"]).toBe(300000);
-        });
-    });
-    describe("配置注册到 ConfigComponent", () => {
-        it("应该能够注册 AGENT_CONFIG_REGISTRATION", () => {
-            configComponent.registerComponent(AGENT_CONFIG_REGISTRATION);
-            expect(() => configComponent.registerComponent(AGENT_CONFIG_REGISTRATION)).not.toThrow();
-        });
-    });
-    describe("Component 配置加载", () => {
-        it("应该需要 ConfigComponent 初始化", async () => {
-            await expect(component.init({
-                options: {},
-            })).rejects.toThrow("ConfigComponent is required");
-        });
-        it("应该能够使用 ConfigComponent 初始化", async () => {
-            await component.init({
-                options: {
-                    configComponent,
-                },
-            });
-            expect(component.name).toBe("agent");
-            expect(component.version).toBe("1.0.0");
-        });
-        it("应该能够通过 config 对象传入配置", async () => {
-            await component.init({
-                options: {
-                    configComponent,
-                    config: {
-                        defaultAgent: {
-                            maxIterations: 100,
-                            temperature: 0.7,
-                        },
-                    },
-                },
-            });
-            expect(component.name).toBe("agent");
-        });
-    });
-    describe("环境变量覆盖", () => {
-        it("应该能够通过环境变量覆盖配置", async () => {
-            // 设置环境变量
-            const originalValue = process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS;
-            process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS = "200";
-            try {
-                await component.init({
-                    options: {
-                        configComponent,
-                    },
-                });
-                expect(component.name).toBe("agent");
-            }
-            finally {
-                // 恢复环境变量
-                if (originalValue !== undefined) {
-                    process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS = originalValue;
-                }
-                else {
-                    delete process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS;
-                }
-            }
-        });
-    });
-    /**
-     * 核心验收标准测试
-     * 根据 component-config-unification-guide.md 的设计目标
-     */
-    describe("核心验收标准", () => {
-        it("✅ 应该通过 configComponent.get() 返回默认值", async () => {
-            await component.init({
-                options: {
-                    configComponent,
-                },
-            });
-            // 验收标准1: configComponent.get("component.key") 返回默认值（无用户配置时）
-            expect(configComponent.get("agent.defaultAgent.maxIterations")).toBe(200);
-            expect(configComponent.get("agent.defaultAgent.temperature")).toBe(0);
-            expect(configComponent.get("agent.defaultAgent.timeout")).toBe(300000);
-            expect(configComponent.get("agent.defaultAgent.streaming")).toBe(false);
-        });
-        it("✅ config 对象应该覆盖默认值", async () => {
-            await component.init({
-                options: {
-                    configComponent,
-                    config: {
-                        defaultAgent: {
-                            maxIterations: 100,
-                            temperature: 0.8,
-                        },
-                    },
-                },
-            });
-            // 验收标准3: config 对象覆盖默认值
-            expect(configComponent.get("agent.defaultAgent.maxIterations")).toBe(100);
-            expect(configComponent.get("agent.defaultAgent.temperature")).toBe(0.8);
-        });
-        it("✅ 环境变量应该覆盖默认值", async () => {
-            // 设置环境变量
-            const originalMaxIterations = process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS;
-            process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS = "75";
-            try {
-                await component.init({
-                    options: {
-                        configComponent,
-                    },
-                });
-                // 验收标准3: 环境变量覆盖默认值（环境变量返回字符串，需要类型转换）
-                // 注意：ConfigComponent.get() 返回的是字符串，环境变量值是字符串
-                // 实际类型转换应该在 buildConfig 中处理
-                const value = configComponent.get("agent.defaultAgent.maxIterations");
-                expect(value !== null && value !== undefined).toBe(true);
-            }
-            finally {
-                if (originalMaxIterations !== undefined) {
-                    process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS = originalMaxIterations;
-                }
-                else {
-                    delete process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS;
-                }
-            }
-        });
-        it("✅ 配置优先级: config对象 > 环境变量 > 默认值", async () => {
-            // 设置环境变量
-            const originalMaxIterations = process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS;
-            process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS = "60";
-            try {
-                await component.init({
-                    options: {
-                        configComponent,
-                        config: {
-                            defaultAgent: {
-                                maxIterations: 80,
-                            },
-                        },
-                    },
-                });
-                // config 对象优先级最高
-                expect(configComponent.get("agent.defaultAgent.maxIterations")).toBe(80);
-            }
-            finally {
-                if (originalMaxIterations !== undefined) {
-                    process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS = originalMaxIterations;
-                }
-                else {
-                    delete process.env.AGENT_AGENT_DEFAULT_AGENT_MAX_ITERATIONS;
-                }
-            }
-        });
-    });
-});
-//# sourceMappingURL=agent-config-registration.test.js.map

package/dist/core/src/env/agent/agent-reminder-plugin.integration.test.d.ts

@@ -1,7 +0,0 @@
-/**
- * @fileoverview AgentComponent + ReminderPlugin 集成测试
- *
- * 验证 ReminderPlugin 真正集成到 AgentComponent 的 before.llm hook 中
- */
-export {};
-//# sourceMappingURL=agent-reminder-plugin.integration.test.d.ts.map