Skip to content

feat(cli): add verbose logging to workspace#944

Merged
RaghavChamadiya merged 1 commit into
repowise-dev:mainfrom
lntutor:feat/workspace-verbose-930
Jul 20, 2026
Merged

feat(cli): add verbose logging to workspace#944
RaghavChamadiya merged 1 commit into
repowise-dev:mainfrom
lntutor:feat/workspace-verbose-930

Conversation

@lntutor

@lntutor lntutor commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Part of #930

Summary

  • add --verbose/-v to workspace add and workspace scan
  • configure CLI logging before workspace/path resolution, indexing, docs generation, or scanning
  • retain quiet-by-default behavior while exposing pipeline diagnostics on demand
  • document both flags and add command-boundary ordering tests

The 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 clean origin/main)
  • targeted Ruff lint and formatting checks
  • targeted mypy (--follow-imports=skip)

@repowise-bot

repowise-bot Bot commented Jul 19, 2026

Copy link
Copy Markdown

✅ Health: 7.5 (unchanged)

📋 At a glance
1 hotspot touched · 1 new finding introduced · 1 file with recent fix history. Scoped to packages.

📌 Before you merge

  • Run .../cli/test_distill_verdict_fingerprint.py, .../cli/test_workspace_add_defaults.py, .../cli/test_workspace_curated_grouping.py: they depend on the changed files
  • Docs that usually track this code: docs/CLI_REFERENCE.md, docs/USER_GUIDE.md

🗺️ 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
Loading

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)

  • .../commands/workspace_cmd.py — 15 commits/90d, 8 dependents · primary owner: Raghav Chamadiya (98%)

📊 Full report · ⭐ Star Repowise · 📥 Install bot · Last updated 2026-07-19 20:20 UTC
Silence on a single PR with [skip repowise] in the title · Per-repo toggle on repowise.dev/settings?tab=bot

@RaghavChamadiya RaghavChamadiya left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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?

@RaghavChamadiya
RaghavChamadiya merged commit 69f14a8 into repowise-dev:main Jul 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants