Commit 976269e
docs(cli): document import commit/cancel in the confirmation protocol
This branch routes `import commit` and `import cancel` through
`confirmOrAbort`, so both now return exit code 2 and a
`confirmation_required` envelope in agent sessions. The protocol reference
listed the confirmation-gated write commands as a closed set —
`incidents create/update/resolve`, `deploy`, `destroy` — which no longer
matches the CLI. Add the two import commands to that list.
Also document the pinning behavior the same commands introduce. Both write
their resolved plan back into the `confirmCommand`, so a bare
`checkly import commit` confirms as:
checkly import commit --plan-id="abc123" --force
carrying a flag the caller never passed. That is deliberate — it stops the
approved run from re-resolving to a plan the user was never shown — but it
looks like noise to an agent that assumes the confirmCommand only ever
repeats what it typed, and the reference gave it no reason to think
otherwise. Say so explicitly, and say not to strip the flag.
Reference-only change: `skills/checkly/SKILL.md` carries the action list,
not reference bodies, so no `sync:skills` regeneration is needed.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>1 parent 4a5c739 commit 976269e
1 file changed
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
| 7 | + | |
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
28 | 32 | | |
29 | 33 | | |
30 | 34 | | |
| |||
0 commit comments