pch/picoclaw
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(agent): add ok checks for startup info type assertions

Browse source 
GetStartupInfo returns map[string]any, and type-asserting tools/skills entries without checking ok is fragile. While the current implementation always stores the correct types, a future refactor could cause silent nil dereference. Add ok checks with explicit nil fallback.
This commit is contained in:
程智超0668000959 2026-06-07 21:28:45 +08:00
parent 67eaa984c7
commit 639f700c15
1 changed files with 8 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
cmd/picoclaw/internal/agent/helpers.go
View file

@ -56,8 +56,14 @@ func agentCmd(message, sessionKey, model string, debug bool) error {
// Print agent startup info (only for interactive mode) // Print agent startup info (only for interactive mode)
startupInfo := agentLoop.GetStartupInfo() startupInfo := agentLoop.GetStartupInfo()
toolsInfo, _ := startupInfo["tools"].(map[string]any) toolsInfo, ok := startupInfo["tools"].(map[string]any)
skillsInfo, _ := startupInfo["skills"].(map[string]any) if !ok {
toolsInfo = nil
}
skillsInfo, ok := startupInfo["skills"].(map[string]any)
if !ok {
skillsInfo = nil
}
logFields := map[string]any{} logFields := map[string]any{}
if toolsInfo != nil { if toolsInfo != nil {
logFields["tools_count"] = toolsInfo["count"] logFields["tools_count"] = toolsInfo["count"]