admin/ki-praxis
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity
Files
main
ki-praxis/checklists/review-agent-diff.md
T

1.2 KiB
Raw Permalink Blame History

Review — Agent-Diff

Gegenmittel zu Accept-and-Forget (Kap. 19.4).
Für Mensch oder Review-Agent. Kopierbar; an deinen Stack anpassen.
Severity (Kommentar): Blocker vor Merge · Should nachziehen · Nit optional.
Nicht verwechseln mit der Reife-Ampel in Kapitel 21 (Low-Maturity) — das hier ist Diff-Triage, keine Repo-Diagnose.

Review-Budget: Ein Agent-PR, den niemand in der verfügbaren Review-Zeit lesen kann, war der Slice zu groß — nicht „Review war zu streng“.

  • Diff matcht Auftrag (kein Drive-by, kein Scope-Creep)?
  • Dateiliste: gehört jede Datei zum Slice?
  • Öffentliche API / Schema / Migration: Intent korrekt?
  • AuthZ / Validierung an neuen oder geänderten Grenzen?
  • Keine Secrets, keine Debug-Backdoors, keine Secrets in Instructions?
  • Dependencies: nötig und lizenziert?
  • Tests vorhanden und nicht nur „compiles“ / leere Asserts?
  • DoD-Commands lokal oder in CI nachvollziehbar (gleiche Commands wie im Repo)?
  • Permissions/YOLO-Spuren im PR-Text unnötig? (kein „musste allow-all“ als Dauerzustand)
  • Findings mit Severity (Blocker / Should / Nit) oder explizit „keine Blocker“?
Reference in new issue View Git Blame Copy Permalink