Write only the document to stdout from snapshot remove --json #256

Merged
clawbot merged 1 commits from snapshot-remove-json-warning-on-stderr into next 2026-10-07 04:46:14 +02:00
1 Commits
Author SHA1 Message Date
sneak ea91b78afd Write only the document to stdout from snapshot remove --json (closes #251)
check / check (push) Waiting to run
When the destination store cannot be reached, `snapshot remove` still
removes the snapshot from the local index and warns. The warning went
through the UI, which writes to stdout, so under `--json` it landed
ahead of the document and `| jq` failed on a command that exited 0.
Under `--json` the UI warning is now skipped; the logger's warning on
stderr carries the follow-up, with the snapshot ID as a field.

The follow-up, in the warning, the README and the command's help, said
`vaultik prune` would finish the cleanup, but `prune` never removes
snapshot metadata. They now say to run `vaultik snapshot remove` for the
snapshot again once the destination store is reachable.

The new test removes a snapshot from the local index against a missing
destination directory and checks both streams.

Model: opus-5-5
2026-10-07 02:08:53 +00:00