Skip to content

Add gitcron integration: native tool + skills#1

Draft
patel-lyzr wants to merge 1 commit into
open-gitagent:mainfrom
patel-lyzr:add-gitcron-integration
Draft

Add gitcron integration: native tool + skills#1
patel-lyzr wants to merge 1 commit into
open-gitagent:mainfrom
patel-lyzr:add-gitcron-integration

Conversation

@patel-lyzr
Copy link
Copy Markdown
Collaborator

Summary

  • Add native gitcron built-in tool (src/tools/gitcron.ts) that wraps the gitcron CLI for scheduling, task management, and reminders
  • Register gitcron tool in src/tools/index.ts
  • Add three skills: scheduling, task-management, reminders with SKILL.md definitions
  • Update agent.yaml to declare the gitcron tool and skills

Test plan

  • Verify npm run build compiles without errors
  • Verify gitcron tool is registered and available in agent runtime
  • Verify skills are loaded from skills/ directory

Author: Khush Patel (khush@lyzr.ai) @patel-lyzr

@patel-lyzr patel-lyzr marked this pull request as draft March 5, 2026 11:07
- New built-in tool: gitcron — agents can schedule jobs, manage tasks,
  set reminders directly through the tool interface
- Skills: scheduling, task-management, reminders — teach agents when
  and how to use gitcron capabilities
- Registered in agent.yaml and createBuiltinTools()

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@patel-lyzr patel-lyzr force-pushed the add-gitcron-integration branch from cb05b4e to 9622930 Compare March 5, 2026 11:09
@patel-lyzr patel-lyzr requested review from shreyas-lyzr and removed request for shreyas-lyzr March 5, 2026 16:46
Copy link
Copy Markdown
Contributor

@shreyas-lyzr shreyas-lyzr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The gitcron integration adds scheduling skills and a native tool. However:

  1. This modifies `agent.yaml` at the repo root — that file is a template/scaffold for new agents. Adding `gitcron` as a required tool there means every new agent gets it by default, which may not be desired.
  2. The skills are well-structured but this should be opt-in via the plugin system or skills marketplace, not baked into the core scaffold.

Suggest making this a plugin or installable skill pack instead of modifying the core agent template.

@shreyas-lyzr
Copy link
Copy Markdown
Contributor

Apologies for the very late review @patel-lyzr.

State of play: gitcron is not in main today. The PR is technically still relevant, but a few questions before I'd merge:

  1. Is gitcron the canonical scheduling tool we want bundled as a built-in? Today the codebase has src/schedules.ts and src/schedule-runner.ts doing scheduling internally. If those already cover the use cases this PR addresses, gitcron is redundant. If they cover different concerns (e.g., gitcron is for cross-agent scheduling, internal scheduling is for per-agent), then both can coexist — but the README / docs need to explain the split.
  2. Built-in vs optional? Even if gitcron is the right tool, I'd lean toward shipping it as an installable skill/plugin rather than a built-in src/tools/gitcron.ts. The skill ecosystem (skills/scheduling, skills/task-management, skills/reminders) is exactly the right place for it — and you already created those skills in this PR. Worth considering whether the built-in tool wrapper adds enough on top.
  3. Rebase needed regardless — two months of drift, 446 lines diff.

If gitcron is still the planned direction, please confirm and rebase; I'll merge same-day. If it makes more sense as a separate gitclaw-gitcron-skill plugin package, I'd close this PR and you could ship that independently.

Real apologies for the wait. Either direction is fine — just need a call on which.

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.

2 participants