feat(cli): add verbose logging to workspace#944
Conversation
|
✅ Health: 7.5 (unchanged) 📋 At a glance 📌 Before you merge
🗺️ Change map flowchart LR
subgraph PR ["Changed in this PR (1 with dependents)"]
f_packages_cli_src_repowise_cli_commands_workspace_cmd_py[".../commands/workspace_cmd.py 🔥"]:::changed
end
f_packages_cli_src_repowise_cli_commands_init_cmd_command_py[".../init_cmd/command.py"]
f_packages_cli_src_repowise_cli_commands_workspace_cmd_py --> f_packages_cli_src_repowise_cli_commands_init_cmd_command_py
f_packages_cli_src_repowise_cli_commands_init_cmd_workspace_py[".../init_cmd/workspace.py"]
f_packages_cli_src_repowise_cli_commands_workspace_cmd_py --> f_packages_cli_src_repowise_cli_commands_init_cmd_workspace_py
f_packages_cli_src_repowise_cli_commands_update_cmd_command_py[".../update_cmd/command.py"]
f_packages_cli_src_repowise_cli_commands_workspace_cmd_py --> f_packages_cli_src_repowise_cli_commands_update_cmd_command_py
f_packages_cli_src_repowise_cli_commands_update_cmd_workspace_py[".../update_cmd/workspace.py"]
f_packages_cli_src_repowise_cli_commands_workspace_cmd_py --> f_packages_cli_src_repowise_cli_commands_update_cmd_workspace_py
more(["+1 more dependent"])
PR --> more
t_tests_unit_cli_test_distill_verdict_fingerprint_py(["✅ .../cli/test_distill_verdict_fingerprint.py"]):::guard
t_tests_unit_cli_test_distill_verdict_fingerprint_py -.-> f_packages_cli_src_repowise_cli_commands_workspace_cmd_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 (5 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)🔥 Hotspot touched (1)
📊 Full report · ⭐ Star Repowise · 📥 Install bot · Last updated 2026-07-19 20:20 UTC |
RaghavChamadiya
left a comment
There was a problem hiding this comment.
Approving the code. One correction to the description: it says the mode is propagated into the nested indexing and docs-generation calls, but the diff does not do that, and it does not need to. _run_index_for_repo and _generate_docs_for_added_repo run in the same process, so the boundary call already covers them. Could you reword that line so the PR history does not imply plumbing that is not there?
Part of #930
Summary
--verbose/-vtoworkspace addandworkspace scanThe nested indexing and docs helpers do not reconfigure logging, so boundary configuration carries through without an extra propagation parameter.
Tests
uv run pytest -q tests/unit/cli/test_workspace_logging.py tests/unit/cli/test_workspace_cmd.py tests/unit/cli/test_workspace_add_defaults.py(56 passed)uv run pytest -q tests/unit/cli(805 passed, 1 pre-existing mascot assertion failure already reproduced on cleanorigin/main)--follow-imports=skip)