feat(cli): add verbose logging to watch - #941
Conversation
|
✅ Health: 7.5 (unchanged) 📋 At a glance Files & modules (2)
📌 Before you merge
🗺️ Change map flowchart LR
subgraph PR ["Changed in this PR (2 with dependents)"]
f_packages_cli_src_repowise_cli_commands_watch_cmd_py[".../commands/watch_cmd.py 🔥"]:::changed
f_tests_unit_cli_test_commands_py[".../cli/test_commands.py"]:::changed
end
f_packages_cli_src_repowise_cli_main_py[".../cli/main.py"]
f_packages_cli_src_repowise_cli_commands_watch_cmd_py --> f_packages_cli_src_repowise_cli_main_py
f_packages_cli_src_repowise_cli_commands_init_cmd_command_py[".../init_cmd/command.py"]
f_tests_unit_cli_test_commands_py --> f_packages_cli_src_repowise_cli_commands_init_cmd_command_py
f_packages_cli_src_repowise_cli_helpers_py[".../cli/helpers.py"]
f_tests_unit_cli_test_commands_py --> f_packages_cli_src_repowise_cli_helpers_py
f_packages_cli_src_repowise_cli_mcp_config_py[".../cli/mcp_config.py"]
f_tests_unit_cli_test_commands_py --> f_packages_cli_src_repowise_cli_mcp_config_py
t_tests_integration_test_cli_py(["✅ tests/integration/test_cli.py"]):::guard
t_tests_integration_test_cli_py -.-> f_tests_unit_cli_test_commands_py
classDef changed fill:#dbeafe,stroke:#1d4ed8,color:#1e3a5f
classDef warn fill:#fef3c7,stroke:#b45309,color:#78350f
classDef guard fill:#dcfce7,stroke:#15803d,color:#14532d
Solid arrows: code that imports the changed files (4 direct dependents, from the last indexed snapshot). Dashed: history/tests. 🩹 Review priority (files here with the most recent bug-fix history — defects cluster, so review these first)
🔎 More signals (1)🔥 Hotspots touched (2)
📊 Full report · ⭐ Star Repowise · 📥 Install bot · Last updated 2026-07-19 19:40 UTC |
RaghavChamadiya
left a comment
There was a problem hiding this comment.
Approving. Forwarding --verbose into the nested update_command invocation is necessary, not decorative: that child run re-enters configure_cli_logging and would otherwise reset to quiet.
Two notes, neither blocking. The find_workspace_root / resolve_repo_path import removal is correct, they were genuinely unused, but the drive-by formatting churn in _watch_workspace makes the diff harder to read than it needs to be. And for the record, _watch_workspace not taking verbose is fine: it calls update_workspace in-process, so the process-wide config already applies. Worth knowing so nobody "fixes" it later.
Summary
--verbose/-vtorepowise watchupdateinvocations so it is not reset to quietcallabletype annotationRelated Issues
Part of #930 (
watchslice).Test Plan
watch_cmd.pygit diff --checkThe sole CLI-suite failure is the existing mascot tagline assertion
test_banner_at_wide_width_stays_compact_with_long_tagline; it reproduces unchanged on cleanorigin/main.Checklist