Agentic-Coding-Review-Bericht

Gegen Spec, Evals und Projekt-Constraints prüfen, nicht nur subjektives „sieht gut aus“.

Review-Scope

• PR / Commit:
• Spec:
• Risk Tier:
• Erforderliches Review-Level:

Zusammenfassung

• Entscheidung: Freigegeben / Änderungen angefordert / Blockiert

Spec-Treue

• [ ] Ziel erfüllt
• [ ] Scope eingehalten
• [ ] Out-of-Scope eingehalten
• [ ] Erwartetes Verhalten implementiert

Notizen:

Eval-Treue

• [ ] Erforderliche Evals existieren
• [ ] Evals laufen durch
• [ ] Edge Cases abgedeckt
• [ ] Bestehende Tests laufen weiterhin durch

Notizen:

Architektur-Review

• [ ] Bestehende Grenzen respektiert
• [ ] Bestehenden Patterns gefolgt
• [ ] Keine unnötigen Abstraktionen eingeführt
• [ ] Keine unbezogenen Refactorings

Notizen:

Security- und Daten-Review

• [ ] Keine Secrets gelesen oder offengelegt
• [ ] Keine Auth- / Berechtigungs-Änderungen außerhalb des Scopes
• [ ] Keine Produktionsdaten berührt
• [ ] Input-Validierung wo relevant berücksichtigt

Notizen:

Betriebs-Review

• [ ] CI-Auswirkung verstanden
• [ ] Deployment-Auswirkung verstanden
• [ ] Observability / Logging wo relevant berücksichtigt
• [ ] Rollback-Auswirkung wo relevant berücksichtigt

Notizen:

Blocker

• ...

Vorschläge

• ...

Endentscheidung

• Freigeben
• Änderungen anfordern
• Auf rigoroses Review eskalieren
• Blockieren