Alan-Jowett / sonde / 23565342407

81%

push

github

Admin CLI: prompt for confirmation on destructive commands (#503)

* admin: prompt for confirmation on destructive commands

Add a confirmation prompt before executing destructive sonde-admin
commands: \
ode remove\, \program remove\, \state import\,
\pairing stop\, and \pairing revoke-phone\.

A \--yes\ / \-y\ global flag skips the prompt for scripted use.
The prompt is also auto-skipped when stdin is not a terminal (piped
input), detected via \std::io::IsTerminal\.

Closes #449

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: reject non-interactive mode without --yes, fix confirm() indentation

- Non-interactive stdin (piped input) now returns an error instead of
  auto-confirming, preventing silent bypass of safety prompts.
  Users must pass --yes explicitly for scripted/piped usage.
- Fix rustfmt indentation of confirm() call in NodeAction::Remove.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Address review findings: align --yes help text with non-interactive behavior

Update the --yes/-y flag doc comment to mention it is required when
running non-interactively (piped stdin). PR description also updated
to reflect the refusal-to-run behavior in non-interactive mode.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

0 of 84 new or added lines in 1 file covered. (0.0%)

22164 of 25900 relevant lines covered (85.58%)

162.92 hits per line