Skip to content

Make plan interactive mode explicit via --interactive#62

Merged
MicheleBonus merged 2 commits intomainfrom
codex/adjust-interactive-prompting-in-cli.py
Mar 7, 2026
Merged

Make plan interactive mode explicit via --interactive#62
MicheleBonus merged 2 commits intomainfrom
codex/adjust-interactive-prompting-in-cli.py

Conversation

@MicheleBonus
Copy link
Copy Markdown
Owner

Motivation

  • Prevent accidental blocking prompts when invoking ambermeta plan with no mode selected by making interactive prompting opt-in.
  • Clarify CLI UX and docs so users explicitly choose manifest, recursive discovery, or interactive mode.

Description

  • Require one of --manifest, --recursive, or the new --interactive flag in _plan_command, printing actionable help and returning non-zero (2) when none are provided (file: ambermeta/cli.py).
  • Add a --interactive option to the plan parser and update plan help/usage examples to show explicit interactive invocation (file: ambermeta/cli.py).
  • Update user-facing documentation and examples to avoid implying interactive is the default (README.md, docs/cli.md).
  • Add tests/test_cli_plan.py with tests that assert the new default behavior, --interactive parsing, and opt-in interactive execution.

Testing

  • Ran pytest -q tests/test_cli_plan.py tests/test_cli_init.py tests/test_cli_protocol_output.py, and all tests passed (10 passed).
  • Unit tests added: test_plan_requires_explicit_mode, test_plan_parser_accepts_interactive_flag, and test_plan_interactive_mode_runs_only_with_opt_in, which all succeeded.

Codex Task

@MicheleBonus MicheleBonus merged commit 0bc1487 into main Mar 7, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant