Skip to content

test(bot): verify PR-comment rendering on a hotspot touch#218

Open
RaghavChamadiya wants to merge 3 commits into
mainfrom
test/bot-comment-verification
Open

test(bot): verify PR-comment rendering on a hotspot touch#218
RaghavChamadiya wants to merge 3 commits into
mainfrom
test/bot-comment-verification

Conversation

@RaghavChamadiya
Copy link
Copy Markdown
Member

Touches `packages/core/src/repowise/core/ingestion/graph.py` (a top hotspot) with a debug-only `diagnose_graph()` helper that has intentionally nested branches. Verifies the post-merge PR-bot output exercises all sections:

  • File delta (with link to /s/{short}/files/...)
  • Refactoring suggestion blurb
  • Hotspot card + ownership
  • Hidden coupling, grouped by file, with strength badge
  • Suggested reviewers (@-mention)
  • Path truncation in tables and bullets
  • Footer opt-out hints + dashboard link

Will close after the comment is verified — `diagnose_graph` is not wired into the indexer hot path.

Temporary change to verify the post-merge PR-bot comment renders all
sections correctly on a real PR. Adds a debug-only diagnose_graph()
method to GraphBuilder with intentionally nested branches — the kind
of code the 12-biomarker scorer flags as nested complexity / brain
method.

Touches a hotspot file (graph.py) with co-change partners not in this
PR, so the comment should exercise: file delta, hotspot, ownership,
hidden coupling with strength tier, suggested-reviewers @-mention,
path truncation, and footer hints.

This PR will be closed after the bot comment is verified — diagnose_graph
is not wired into the indexer hot path.
@RaghavChamadiya RaghavChamadiya requested a review from swati510 as a code owner May 20, 2026 06:11
@repowise-bot
Copy link
Copy Markdown

repowise-bot Bot commented May 20, 2026

⚠️ Health: 7.0 → 6.9 (-0.1)
3 files moved · 3 hotspots · 5 hidden couplings · 5 dead-code findings

File Score Δ Why
.../generation/page_generator.py 4.5 → 2.5 ▼ -2.0 untested hotspot, brain method, nested complexity
.../ingestion/graph.py 3.3 → 1.8 ▼ -1.5 untested hotspot, dry violation, brain method
.../persistence/crud.py 3.3 → 1.8 ▼ -1.5 untested hotspot, nested complexity, dry violation

💡 .../generation/page_generator.py: Write tests before refactoring. This file is high-churn and high-dependents but lacks coverage — add characterization tests for the current behavior first, then refactor with confidence.

🔥 Hotspot touched

  • .../persistence/crud.py — 13 commits/90d, 39 dependents · primary owner: RaghavChamadiya (53%)
  • .../generation/page_generator.py — 18 commits/90d, 13 dependents · primary owner: Raghav Chamadiya (53%)
  • .../ingestion/graph.py — 21 commits/90d, 13 dependents · primary owner: Raghav Chamadiya (62%)

🔗 Hidden coupling

  • .../generation/page_generator.py co-changes with these files (not in this PR):
    • .../commands/init_cmd.py (9× — 🟢 routine)
    • .../commands/update_cmd.py (8× — 🟢 routine)
  • .../ingestion/graph.py co-changes with these files (not in this PR):
    • .../pipeline/orchestrator.py (8× — 🟢 routine)
    • .../ingestion/parser.py (7× — 🟢 routine)
  • .../persistence/crud.py co-changes with .../persistence/models.py (6× — 🟢 routine) — not in this PR.

Dead code

  • 💀 .../persistence/crud.py update_decision_metadata (confidence 1.00)
  • 💀 .../persistence/crud.py update_decision_by_id (confidence 1.00)
  • 💀 .../persistence/crud.py get_health_findings (confidence 1.00)
2 more
  • 💀 .../persistence/crud.py update_health_finding_status (confidence 1.00)
  • 💀 .../generation/page_generator.py _is_significant_file (confidence 0.65)

👀 Suggested reviewers @RaghavChamadiya


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

Extends the test PR so the bot's output exercises:
- Multiple file deltas in the table
- Multiple hotspots (graph.py, page_generator.py, crud.py)
- Hidden coupling list with >3 changed files (triggers <details> collapse)
- Dead-code section via _unused_layout_legacy_shim in page_generator.py
- Refactoring suggestion for the worst regressor

Each added function is plausibly-debugging code with nested branches —
the 12-biomarker scorer should flag nested complexity / brain method.
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.

1 participant