-
-
Notifications
You must be signed in to change notification settings - Fork 52.9k
Closed
Labels
bugSomething isn't workingSomething isn't working
Description
Bug Description
Cannot update or remove cron jobs via the cron tool due to parameter name mismatch.
Expected Behavior
Should be able to update/remove jobs using jobId parameter as defined in the function schema.
Actual Behavior
- Outer function schema expects
jobId - Inner validation expects
id - Both reject the other's format
Steps to Reproduce
Attempt to remove a job:
Action: remove
jobId: cc1e9bd5-4e57-47ba-ba85-63bdd692d70e
Error:
invalid cron.remove params: data must have required property 'id'; data must NOT have additional properties
Attempt to update a job:
Action: update
jobId: cc1e9bd5-4e57-47ba-ba85-63bdd692d70e
patch: {"enabled": false}
Error:
invalid cron.update params: data must have required property 'id'; data must NOT have additional properties
Attempt using id instead:
Action: remove
id: cc1e9bd5-4e57-47ba-ba85-63bdd692d70e
Error:
Validation failed for tool "cron": jobId: must have required property 'jobId'
Root Cause (Likely)
Function definition exposes jobId but internal Zod/JSON schema validates against id. Parameter not being mapped/renamed before validation.
Impact
- Can add jobs ✅
- Cannot update jobs ❌
- Cannot remove jobs ❌
Suggested Fix
Either:
- Rename internal schema from
id→jobId - Map
jobId→idbefore internal validation
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working