review-pr-strict

# 🔍 STRICT PR Alignment Audit (Full Coverage)
## {{repo}} — PR #{{number}}: {{title}}
**Audit Date:** {{current_date}}
**Auditor:** Claude (Strict Mode — Full Coverage via Swarm)
**Grading Mode:** STRICT (Default F, evidence required for upgrades)
**Coverage:** Every changed file, every linked issue, every cited ADR — no sampling

---

## ⚠️ STRICT MODE WARNING

This audit uses rigorous grading standards:

- **Every dimension starts at F** and must earn upgrades with concrete evidence
- **A grades are RARE**
- **Most real PRs score C–D range** — this is normal
- **"It exists / looks done" is not enough** — the diff must demonstrably satisfy each criterion
- **No credit for follow-up issues or TODOs** — only what THIS change delivers counts
- **Architecture docs are not ground truth — code is**

---

## 🎯 PR Under Review

| Field | Value |
|-------|-------|
| Number | #N |
| Title | ... |
| Author | @... |
| Base → Head | main ← feature/... |
| Head SHA | abc1234 |
| CI Status | ✅ / ❌ / ⏳ |
| Linked Issues | #A, #B, #C |
| Files Changed | X |
| LOC | +A / -B |

---

## 📊 Executive Scorecard

| # | Dimension | Grade | Score | Status |
|---|-----------|-------|-------|--------|
| 1 | Requirements Alignment | ? | ??% | 🟢/🟡/🔴 |
| 2 | Architecture Alignment | ? | ??% | 🟢/🟡/🔴 |
| 3 | Code Quality of Diff | ? | ??% | 🟢/🟡/🔴 |
| 4 | Test Coverage of Diff | ? | ??% | 🟢/🟡/🔴 |
| 5 | Security & Safety | ? | ??% | 🟢/🟡/🔴 |
| 6 | CI / Build / Hygiene | ? | ??% | 🟢/🟡/🔴 |
|   | **OVERALL** | **?** | **??%** | **🟢/🟡/🔴** |

🟢 A–B (healthy) | 🟡 C (needs attention) | 🔴 D–F (critical)

---

## 📋 Detailed Dimension Assessments

### Dimension 1: Requirements Alignment

**Grade: ? (??%)**

**Evidence Reviewed:**

- Issue(s): #N (read body, acceptance criteria, comments)
- PR description and commit messages
- Files changed: [list]
- Already-done check: [search results]

**Strengths:**

- [strength with file:line / issue field reference]

**Findings:**

- 🔴 CRITICAL: [finding with reference]
- 🟠 MAJOR: ...
- 🟡 MINOR: ...
- ⚪ NITPICK: ...

**Missing:**

- [acceptance criterion not addressed by the diff]

**Principle Compliance:**

- YAGNI: ...
- POLA: ...

---

[Repeat for all 6 dimensions]

---

## 🚨 Consolidated Findings

### Critical (Block Merge)

1. [DIM #] [finding with file:line] — [why blocking]

### Major (Fix Before Merge)

1. ...

### Minor (Fix Soon)

1. ...

### Nitpick (Optional)

1. ...

---

## 🔧 Triage Split

**Fix-now (in this PR):**

- [item — small, scoped, no design decision required]

**File-as-issue (separate work):**

- [item — requires design or cross-PR coordination]
  - Reconcile against existing backlog FIRST: [list of related open issues searched]

---

## 🏛️ Architecture Alignment Detail

| Architecture Source | Reviewed? | Aligned? | Notes |
|---------------------|-----------|----------|-------|
| CLAUDE.md | ✅ | 🟢/🟡/🔴 | ... |
| docs/architecture/ | ✅ | 🟢/🟡/🔴 | ... |
| ADRs | ✅ | 🟢/🟡/🔴 | ... |
| README architecture sections | ✅ | 🟢/🟡/🔴 | ... |
| Actual code ground truth | ✅ | 🟢/🟡/🔴 | ... |

**Doc-vs-code drift detected:** [yes/no — if yes, list]

---

## 📈 Development Principles Compliance

| Principle | Compliance | Evidence |
|-----------|------------|----------|
| KISS | 🟢/🟡/🔴 | file:line |
| YAGNI | 🟢/🟡/🔴 | file:line |
| TDD | 🟢/🟡/🔴 | file:line |
| DRY | 🟢/🟡/🔴 | file:line |
| SOLID | 🟢/🟡/🔴 | file:line |
| MODULARITY | 🟢/🟡/🔴 | file:line |
| POLA | 🟢/🟡/🔴 | file:line |

---

## 📊 Audit Methodology

**Files Examined:** X of Y changed (every changed file)
**Linked Issues Read:** N of N
**ADRs / Architecture Docs Read:** N of N
**Coverage Gaps:** [list or "none"]

**Evidence Standard:** Every grade backed by direct reading of the diff, the linked issues, and the cited architecture documentation. Architecture claims verified against current source code (ground truth).

---

## 📝 Summary

[2–3 paragraph honest narrative: does this change actually do what was asked, fit the architecture, and ship safely? What are the strongest signals and the most pressing concerns?]

---

## ✅ GO / CONDITIONAL / NO-GO Verdict

### Verdict: **[GO ✅ | CONDITIONAL 🟡 | NO-GO 🔴]**

**Rationale:**
[Evidence-based explanation citing specific blockers]

**Critical Blockers (if NO-GO):**

1. [issue with file:line]

**Conditions for GO (if CONDITIONAL):**

1. [specific, measurable]

**Recommended Next Steps:**

1. [highest priority]
2. ...
3. ...

---

## 📊 Grade Distribution Summary

- A grades: X
- B grades: X
- C grades: X
- D grades: X
- F grades: X
- N/A: X

**Reality Check:**
[If many A's/B's, explicitly question whether anti-inflation was applied]