docs(tools/scheduling): fix stale edit_schedule interval_seconds doc
The edit_schedule MCP tool accepts interval_seconds as Option<u64> (positive values only), mapping None→leave-alone and Some(v)→set. The handler wraps this as .map(Some) before dispatching, so there is no way to express Some(None) = clear via the agent surface. The old doc said 'Clearing interval_seconds to null flips recurring → one-shot' which is only true for the dashboard PATCH endpoint (which uses the full double-Option form). Corrected: interval_seconds is positive-only via the MCP tool; toggling recurring→one-shot is operator-only via the dashboard.
This commit is contained in:
parent
dd67cfbe97
commit
2b3dfeb23b
1 changed files with 6 additions and 3 deletions
|
|
@ -26,9 +26,12 @@ audit.
|
||||||
Partial-update a schedule. Pass only the fields to change; absent
|
Partial-update a schedule. Pass only the fields to change; absent
|
||||||
fields are left alone. `targets_add` / `targets_remove` mutate the
|
fields are left alone. `targets_add` / `targets_remove` mutate the
|
||||||
recipient list in the same transaction — re-adding a previously
|
recipient list in the same transaction — re-adding a previously
|
||||||
cancelled target drops its tombstone and starts fresh. Clearing
|
cancelled target drops its tombstone and starts fresh.
|
||||||
`interval_seconds` to null flips recurring → one-shot. Refuses
|
`interval_seconds` accepts positive values only via this tool (omit
|
||||||
cancelled rows (terminal state).
|
to keep the existing cadence; pass a new positive value to change
|
||||||
|
it). Toggling recurring → one-shot (clearing the interval) is
|
||||||
|
operator-only via the dashboard PATCH endpoint. Refuses cancelled
|
||||||
|
rows (terminal state).
|
||||||
|
|
||||||
### `cancel_schedule(id, targets?)`
|
### `cancel_schedule(id, targets?)`
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue