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

Merge pull request #10 from hobbyistlabs-coder/fix/cron-tool-reword-fix-comment-7066954326773042448

Browse source 
chore: reword "Fix:" comment to "Rationale:" in cron tool
This commit is contained in:
hobbyistlabs-coder 2026-03-13 15:59:24 -04:00 • committed by GitHub
commit 9970032d53
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
pkg/tools/cron.go
View file

@ -146,7 +146,7 @@ func (t *CronTool) addJob(ctx context.Context, args map[string]any) *ToolResult
everySeconds, hasEvery := args["every_seconds"].(float64) everySeconds, hasEvery := args["every_seconds"].(float64)
cronExpr, hasCron := args["cron_expr"].(string) cronExpr, hasCron := args["cron_expr"].(string)
// Fix: type assertions return true for zero values, need additional validity checks // Rationale: type assertions return true for zero values, need additional validity checks
// This prevents LLMs that fill unused optional parameters with defaults (0) from triggering wrong type // This prevents LLMs that fill unused optional parameters with defaults (0) from triggering wrong type
hasAt = hasAt && atSeconds > 0 hasAt = hasAt && atSeconds > 0
hasEvery = hasEvery && everySeconds > 0 hasEvery = hasEvery && everySeconds > 0