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

fix: enhance cron job execution conditions with remote allowance

Browse source
This commit is contained in:
Zhang Rui 2026-04-14 21:39:13 +08:00
parent c8ac5e5ecc
commit 4ec24a0f63
1 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
pkg/tools/cron.go
View file

@ -40,7 +40,7 @@ func NewCronTool(
execEnabled := true
if config != nil {
allowCommand = config.Tools.Cron.AllowCommand
execEnabled = config.Tools.Exec.Enabled
execEnabled = config.Tools.Exec.Enabled && config.Tools.Exec.AllowRemote
}
var execTool *ExecTool
@ -189,9 +189,7 @@ func (t *CronTool) addJob(ctx context.Context, args map[string]any) *ToolResult
return ErrorResult("one of at_seconds, every_seconds, or cron_expr is required")
}
// GHSA-pv8c-p6jf-3fpp: command scheduling requires internal channel. When
// allow_command is disabled, explicit confirmation is required as an override.
// Non-command reminders remain open to all channels.
// If command is set, check if execution is allowed and if confirmation is provided when required
command, _ := args["command"].(string)
commandConfirm, _ := args["command_confirm"].(bool)
if command != "" {