2 Commits

Author	SHA1	Message	Date
yinwm	ab20314882	Merge upstream/main into ralph/tool-result-refactor ... Resolved conflicts: - pkg/heartbeat/service.go: merged both 'started' field and 'onHeartbeatWithTools' - pkg/tools/edit.go: use validatePath() with ToolResult return - pkg/tools/filesystem.go: fixed return values to use ToolResult - cmd/picoclaw/main.go: kept active setupCronTool, fixed toolsPkg import - pkg/tools/cron.go: fixed Execute return value handling Fixed tests for new function signatures (NewEditFileTool, NewAppendFileTool, NewExecTool) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-02-13 01:00:26 +08:00
yinwm	e7e3f95ebe	feat: US-012 - Add ShellTool tests ... Added comprehensive test coverage for ShellTool (ExecTool) with 9 test cases: - TestShellTool_Success: Verifies successful command execution - TestShellTool_Failure: Verifies failed command execution with IsError flag - TestShellTool_Timeout: Verifies command timeout handling - TestShellTool_WorkingDir: Verifies custom working directory support - TestShellTool_DangerousCommand: Verifies safety guard blocks dangerous commands - TestShellTool_MissingCommand: Verifies error handling for missing command - TestShellTool_StderrCapture: Verifies stderr is captured and included - TestShellTool_OutputTruncation: Verifies long output is truncated - TestShellTool_RestrictToWorkspace: Verifies workspace restriction ShellTool implementation already conforms to ToolResult specification: - Success returns ForUser = command output - Failure returns IsError = true - ForLLM contains full output and exit code Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>	2026-02-12 19:52:16 +08:00