Finde die Fehler
Rate zuerst, ohne Punktabzug. Du hast einen KI-Agenten gebeten: „Füg fetch_user bis zu 3 Wiederholungen hinzu. Keine neuen Abhängigkeiten.“ Er meldet „Fertig, Tests laufen durch.“ Tipp auf die drei Zeilen seines Diffs, bei denen ein Reviewer anhalten sollte.
Der Diff, Zeile für Zeile
0 von 3 gefunden
Ein Diff ist die Liste der Zeilen, die eine Änderung hinzufügt und entfernt. Die Zusammenfassung eines Agenten sagt, was er vorhatte. Der Diff zeigt, was er getan hat. „Tests laufen durch“ beweist nur das, was die Tests prüfen.
Lies den Diff, bevor du etwas ausführst oder mergst, mit fünf Prüfungen:
- Umfang. Hat er nur geändert, worum du gebeten hast? Achte auf zusätzliche Dateien und neue Abhängigkeiten (externe Pakete, die dein Code jetzt braucht).
- Fehler. Werden Fehlschläge behandelt oder verschluckt, sodass niemand davon erfährt?
- Randfälle. Was passiert bei leerer Eingabe, einem Timeout oder einem fehlenden Feld?
- Deine Codebasis. Nutzt er eure Namen und Hilfsfunktionen oder erfindet er neue?
- Sicherheit. Geheimnisse im Code, SQL aus zusammengesetzten Strings, eine übersprungene Berechtigungsprüfung?
Ein Beispiel
Der Review-Kommentar zum Diff oben:
Drei Änderungen, bevor das gemergt werden kann:
1. Umfang: Entfern "import tenacity". Wir hatten keine neuen Abhängigkeiten vereinbart.
2. Sicherheit: Lies API_KEY aus der Umgebung. Tausch den Schlüssel aus, der
in diesem Diff steht, denn er ist jetzt in der Branch-Historie.
3. Fehler: Fang nur requests.RequestException ab und lös nach dem
dritten Fehlschlag einen Fehler aus, damit Aufrufer wissen, dass der Abruf scheiterte.
In Ordnung: die Wiederholungsschleife und das Timeout von 5 Sekunden.Lücken füllen
Füll die Lücken.
Review-Notiz
Bevor ich merge, lese ich den . Ich prüfe, dass er nur , dass Fehler und dass keine im Code stehen.
Probier es aus
Deine Schritte, etwa 2 Minuten
- 1.Lies den Diff der letzten Änderung, die eine KI für dich gemacht hat, mit den fünf Prüfungen, eine Zeile pro Prüfung. Füg die fünf Prüfungen deinem Kit hinzu.2 Min.