Skip to content

feat(cron): Add support for direct shell command execution in scheduled jobs#74

Merged
lxowalle merged 6 commits intosipeed:mainfrom
SatyamDevv:main
Feb 12, 2026
Merged

feat(cron): Add support for direct shell command execution in scheduled jobs#74
lxowalle merged 6 commits intosipeed:mainfrom
SatyamDevv:main

Conversation

@SatyamDevv
Copy link
Contributor

@SatyamDevv SatyamDevv commented Feb 12, 2026

Description

This PR adds the ability for the Cron system to execute shell commands directly, bypassing the agent loop for simple tasks. It also improves Windows compatibility for shell execution.

Changes

  • pkg/cron: Added Command field to CronPayload.
  • pkg/tools/cron: Updated CronTool to handle command argument and execute via ExecTool.
  • pkg/tools/shell:
    • Added Windows support using PowerShell.
    • Added restrictToWorkspace validation logic.
  • cmd/picoclaw: Updated wiring to pass workspace path to CronTool.

How to test

  1. Schedule a command: Schedule a task after 5 sec to execute the command 'ls -la'
  2. Verify output is returned directly to the chat channel.

@lxowalle lxowalle merged commit e2862c7 into sipeed:main Feb 12, 2026
1 check passed
@lxowalle
Copy link
Collaborator

Thanks for the pr

@Zepan
Copy link
Contributor

Zepan commented Feb 13, 2026

Thanks for your contribution! We are forming the PicoClaw Dev Group to accelerate the evolution of the project. Any developer with more than one merged PR is invited to join.

Would you like to join the PicoClaw Dev Group? If so, please send an email to support@sipeed.com with the subject line: [Join PicoClaw Dev Group] + Your GitHub account. We will send the Discord invite link to your inbox.

emadomedher pushed a commit to emadomedher/picoclaw that referenced this pull request Feb 17, 2026
feat(cron): Add support for direct shell command execution in scheduled jobs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants