Reviewer: review-first, fix-later flow — address all review feedback #74

Summary

The PR reviewer currently treats "no review score yet" as "all good" and skips the PR. When it does fix, it only addresses CI failures and merge conflicts — ignoring code quality bugs, performance issues, and other review feedback.

Desired Behavior

1. No review yet → Post a review (score the PR), exit without fixing
2. Review exists, score < threshold → Fix ALL flagged issues (bugs, code quality, performance, CI, merge conflicts), push, exit
3. After fixing → Exit. Next scheduled run (or GH Actions on push) re-scores
4. Score >= threshold → Skip (unchanged)

PRD

See docs/prds/reviewer-review-first-fix-later.md

Key Files

• scripts/night-watch-pr-reviewer-cron.sh — cron orchestration
• instructions/night-watch-pr-reviewer.md — reviewer/fixer prompt
• .github/workflows/pr-review.yml — GH Actions review trigger
• .github/prompts/pr-review.md — scoring criteria