Review
Nutzen Sie diesen Skill, um bereits existierende Arbeit zu bewerten. Review erstellt niemals selbst die Änderung, die es beurteilt.
Dies ist der kanonische Review-Workflow. merge-readiness leitet hierher, und ai-slop-cleaner ist ein optionaler Zweig innerhalb dieses Workflows.
Ziel
Finden, was tatsächlich falsch ist, geordnet nach Schweregrad, mit ausreichend Detail, um darauf reagieren zu können.
Workflow
- Feststellen, was geändert wurde und was es bewirken sollte.
- Die Änderung anhand dieser Absicht lesen.
- Zuerst Korrektheit prüfen, dann Risiko, dann Vereinfachung.
- Jeden Kandidatenbefund vor der Meldung verifizieren.
- Befunde nach Schweregrad (schwerste zuerst) melden.
Was zu prüfen ist
- Korrektheit — Logikfehler, Randfälle, Fehlerpfade, Nebenläufigkeit
- Risiko — Sicherheitsgrenzen, destruktive Operationen, Datenintegrität
- Wiederverwendung — vorhandene Hilfsmittel oder Muster, die die Änderung hätte verwenden sollen
- Vereinfachung — Code, der gelöscht oder zusammengefasst werden kann
- Abdeckung — Verhalten, das ungetestet ausgeliefert wird
Regeln
- Getrennte Arbeitskontexte: Der Prüfer darf nicht derselbe aktive Kontext wie der Autor sein.
- Vor dem Melden verifizieren. Ein plausibel klingender Befund, der sich nicht reproduzieren lässt, ist Rauschen.
- Schweregrad ehrlich angeben; die Liste nicht künstlich aufblähen, um gründlich zu wirken.
- "Keine Befunde" ist ein gültiges Ergebnis, wenn die Arbeit einwandfrei ist.
- Standardmäßig beratend — Review informiert, blockiert nicht. Harte Sperren (Release, Sicherheit, destruktive Operationen) bleiben getrennt und schließen standardmäßig fehl.
Ausgabe
- Befunde, schwerste zuerst, jeweils mit Datei, Zeile und konkretem Fehlerszenario
- Was geprüft und für unbedenklich befunden wurde
- Alles, was nicht bewertet werden konnte