Skip to content

feat(cli): add verbose logging to health#943

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

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

Conversation

@lntutor

@lntutor lntutor commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Part of #930

Summary

  • add --verbose/-v to repowise health
  • configure CLI logging before target resolution and analysis/provider work
  • preserve stricter log silencing for JSON and Markdown output
  • document the new flag and cover quiet/verbose forwarding

Tests

  • uv run pytest -q tests/unit/cli/test_health_cmd_logging.py tests/unit/cli/test_analysis_cmds_include_submodules.py (6 passed)
  • uv run pytest -q tests/unit/cli (803 passed, 1 pre-existing mascot assertion failure reproduced on clean origin/main)
  • targeted Ruff lint and formatting checks
  • targeted mypy (--follow-imports=skip)

Repository-wide lint/type checking currently reports unrelated baseline issues, including existing Ruff violations and two mypy errors in health_cmd/refactoring_targets.py.

@repowise-bot

repowise-bot Bot commented Jul 19, 2026

Copy link
Copy Markdown

✅ Health: 7.5 (unchanged)

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

🗺️ Change map

flowchart LR
  subgraph PR ["Changed in this PR (1 with dependents)"]
    f_packages_cli_src_repowise_cli_commands_health_cmd_command_py[".../health_cmd/command.py 🔥"]:::changed
  end
  f_packages_cli_src_repowise_cli_commands_health_cmd___init___py[".../health_cmd/__init__.py"]
  f_packages_cli_src_repowise_cli_commands_health_cmd_command_py --> f_packages_cli_src_repowise_cli_commands_health_cmd___init___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 (1 direct dependent, 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)

  • .../health_cmd/command.py — 4 commits/90d, 1 dependents · primary owner: Raghav Chamadiya (98%)

📊 Full report · ⭐ Star Repowise · 📥 Install bot · Last updated 2026-07-19 20:13 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. Docs updated, the test drives the real Click parser, and it asserts ordering rather than just arrival, which is the part that actually matters here.

Calling out one thing for anyone reading this later: configure_cli_logging lands before the existing silence_logs_for_machine_output(), so repowise health --format json | jq stays pipe-safe under -v. Reversing those two would let structlog into stdout and break the JSON. Good catch.

@RaghavChamadiya
RaghavChamadiya merged commit a80b831 into repowise-dev:main Jul 20, 2026
7 checks 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