Skip to content

Conversation

@dtsong
Copy link
Owner

@dtsong dtsong commented Jan 23, 2026

Summary

  • Enhanced LLM prompt with detailed instructions for generating GitHub suggestion blocks (```suggestion)
  • Code suggestions are only included for critical/major severity issues (high-confidence filtering)
  • Fixed pre-existing circular import in metrics/__init__.py that broke 5 test files

Changes

  • src/pr_review_agent/review/llm_reviewer.py — expanded prompt with suggestion rules + severity filtering in inline comment builder
  • src/pr_review_agent/metrics/__init__.py — removed eager SupabaseLogger import to break circular dependency
  • tests/test_suggested_fixes.py — 8 new tests covering severity filtering and suggestion block rendering

Test plan

  • 8 new tests for suggested fixes (severity filtering, rendering)
  • All 171 tests passing
  • Lint clean

Closes #12

🤖 Generated with Claude Code

Enhance LLM prompt with detailed instructions for generating GitHub
suggestion blocks. Filter code suggestions to only appear for critical
and major severity issues, ensuring one-click "Apply suggestion" is
reserved for high-confidence fixes.

Also fix circular import in metrics/__init__.py that prevented several
test files from collecting.

Implements #12

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@dtsong dtsong merged commit 684ac0e into main Jan 23, 2026
1 of 2 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.

[Phase 2.2] Suggested Fixes

2 participants