Luca Morelli
Software Engineer und Analyst
Teamleiter

Pull-Request-Review mit KI: Codequalität und die Entwicklung von Junior-Entwicklern.

Ein unübersichtlicher Pull Request erzwingt mehrere Runden, bevor er freigegeben wird, und mit Coding-Agenten wächst die Menge an Code, die geprüft werden muss. Ein Review mit KI kann helfen, die Qualität zu kontrollieren und Entwickler in der Ausbildung zu begleiten: Es muss aber ein unterstützendes Werkzeug bleiben, kein Richter.

Das gilt auch für Entwickler, die ohne KI arbeiten: Pull Requests und ihre Entwicklung nachzuverfolgen, Runde für Runde und über die Zeit, ist ohnehin nützlich. Coding-Agenten machen den Bedarf sichtbarer, sie schaffen ihn nicht.

Woher es kommt – und was ich noch nicht gemacht habe

Das Problem unübersichtlicher Pull Requests ist in Gesprächen mit Kollegen immer wieder aufgekommen, bei ganz unterschiedlichen Gelegenheiten.

Die hier beschriebene Methode ist daraus entstanden und aus meiner Erfahrung als Senior-Entwickler und Teamleiter, zu der es seit jeher gehört, Junior-Entwickler zu schulen und zu begleiten.

In einem Projekt habe ich sie noch nicht angewendet. Um sie wirklich zu erproben, braucht es ein reales Team: mehrere Entwickler, Aufgaben unterschiedlicher Komplexität und einen Zeitraum, der lang genug ist, um eine Entwicklung zu erkennen. Sie ist Teil einer gemeinsamen Harness des Teams, die ich, wie auf der Seite Qualität und Coding-Agenten beschrieben, noch nicht aufgebaut habe. Was folgt, ist daher ein methodischer Vorschlag.

Zwei Ziele

  • Die Qualität des Codes sichern, der ins Produkt gelangt.
  • Entwickler in der Ausbildung angemessen begleiten und erkennen, ob sie wirklich wachsen – auch wenn sie den Code mithilfe von KI schreiben.

Codequalität: dieselbe Schwelle für alle

Die Schwelle für die Freigabe eines Pull Requests wäre für alle gleich: Sicherheit, Korrektheit und wichtige Projektregeln blockieren die Freigabe, unabhängig davon, wer den Code geschrieben hat. Die KI würde ein erstes Review anhand der Richtlinien des Teams durchführen; das letzte Wort bliebe bei einem Menschen.

Autor und Prüfer sollten getrennt bleiben: Die Sitzung, die den Code prüft, darf nicht dieselbe sein, die ihn geschrieben hat. Ein Review ist nur so gut, wie die schriftlichen Regeln des Projekts klar sind.

Entwickler in der Ausbildung begleiten: Der Weg ändert sich, nicht die Schwelle

Was sich ändert, ist der Weg zur Freigabe. Bei Junior-Entwicklern erklären die Kommentare das Warum und verweisen auf die Regel; kleinere Probleme wie Benennung oder Stil werden gemeldet, blockieren aber nicht. Bei Senior-Entwicklern genügen kurze Kommentare. Zu jedem blockierenden Problem erklärt die Person, die den Code geschrieben hat, in wenigen Zeilen, was sie geändert hat und warum; ist sie mit einem Kommentar nicht einverstanden, kann sie widersprechen – und ein begründeter Widerspruch ist ein gutes Zeichen.

Innerhalb eines einzelnen Pull Requests würde ich eine Höchstzahl an Runden festlegen: Ist sie erreicht, setzt man sich besser mit einem Senior-Entwickler zusammen, statt weiter zu iterieren.

Zu vermeiden ist ein Entwickler, der das Review besteht, ohne zu verstehen. Eine kontrollierte Studie von Anthropic mit 52 Entwicklern, überwiegend Junior-Entwicklern, ergab schlechtere Ergebnisse im Verständnistest bei denjenigen, die eine neue Bibliothek mit dem Assistenten lernten (50 % gegenüber 67 %), ohne signifikanten Zeitgewinn. Wer sich von der KI Konzepte erklären ließ, erzielte ähnliche Ergebnisse wie die Gruppe ohne KI. Die Studie ist klein und vorläufig, zeigt aber, worauf zu achten ist: nicht, ob ein Entwickler KI nutzt, sondern wie.

Wann Begleitung normal ist und wann sie ein Signal ist

Mit jedem weiteren Pull Request, oder mit jedem Monat, erwartet man, dass neue Teammitglieder schneller zur Freigabe kommen. Am Anfang kann es normal sein, dass ein Senior-Entwickler unterstützen muss: Das hängt vom Projekt ab, von seiner Komplexität und davon, wie gut es dokumentiert ist. Wiederholt sich die Situation nach fünf oder sechs Pull Requests immer noch, gibt es etwas zu verstehen. Die Zahlen sind Richtwerte.

Das Problem liegt nicht unbedingt bei der Person: Auch die zugewiesenen Aufgaben, ein schlecht dokumentierter Teil des Codes oder eine unklare Regel können eine Rolle spielen.

Warum Review-Daten sammeln

Wenn mehrere Entwickler gleichzeitig eingearbeitet werden, lässt sich der Weg jedes Einzelnen kaum aus dem Gedächtnis verfolgen. Ein Review-Protokoll mit den gemeldeten Problemen, dem Bereich der Anwendung und den nötigen Runden zeigt, ob dieselben Fehler wiederkehren oder verschwinden und wie lange es dauert, bei den verschiedenen Aufgabenarten selbstständig zu werden.

Nach Anwendungsbereichen gelesen, verrät dasselbe Protokoll noch mehr: Stoßen mehrere Junior-Entwickler im selben Teil des Codes auf dasselbe Problem, liegt die Schwachstelle wahrscheinlich nicht bei ihnen, sondern in der Harness, also in der Dokumentation, den Regeln oder den Prüfungen dieses Teils. So lässt sich über die Zeit auch erkennen, ob die Harness funktioniert.

Review-Werkzeuge auf dem Markt erfassen diese Daten bereits: Die Dokumentation von CodeRabbit beschreibt zum Beispiel Kennzahlen, die sich nach einzelnen Entwicklern filtern lassen. Die Frage ist also nicht, ob es die Daten gibt, sondern wie man sie nutzt.

Arbeit vorbereiten: klarere Pull Requests und geringere Kosten

Ein unübersichtlicher Pull Request beginnt oft mit einer vagen Aufgabe. Die Arbeit vorzubereiten, bevor man sie zuweist, bringt zweierlei: Der Entwickler weiß, was er liefern soll und wie er es prüft, und der Agent arbeitet mit präzisen Anfragen, statt den Code ziellos zu durchsuchen, und verbraucht weniger. Das wird immer wichtiger, denn viele Unternehmen begrenzen inzwischen ihre Ausgaben für KI.

Werkzeuge und Zuständigkeiten im Team

Wer bewertet

Das Werkzeug bewertet keine Menschen. Es liefert technische Daten zu den Pull Requests: Probleme, Runden, wiederkehrende Fehler, nach der Freigabe entdeckte Mängel und ihre Ursache. Diese Daten fließen in die Prozesse zur Bewertung der beruflichen Entwicklung ein, die das Unternehmen bereits hat; das Urteil bleibt bei einem Menschen.

Einige Bedingungen halte ich für notwendig:

  • Entwickler sehen alles, was ihre Führungskraft über sie sieht, und können darauf antworten;
  • keine Ranglisten zwischen Personen und keine Gesamtpunktzahlen;
  • erfasst werden nur Daten zum technischen Inhalt der Pull Requests, weder Arbeitszeiten noch Gespräche mit der KI, und die ausgeschlossenen Daten werden schriftlich festgehalten;
  • die Daten werden für einen festgelegten Zeitraum aufbewahrt.

Quellen zur Einordnung

Die Methode ist ein Vorschlag aus eigener Erfahrung und Gesprächen mit Kollegen; die Quellen stützen einzelne Punkte, nicht die Methode als Ganzes.

Inhalt zuletzt aktualisiert am

Sollen Ihre Entwickler lernen, Coding-Agenten methodisch einzusetzen?

Ausführlicher Lebenslauf und Referenzen auf Anfrage verfügbar.