Ein PR-Reviewer, der selbst mergt, ist sicher, sobald er weiß, wann er aufhören muss

Letzte Woche fragte mich ein Kollege, welche Automation in der Factory beim ersten Lesen am meisten Angst macht. Die Antwort fiel mir leicht: der PR Reviewer. Er mergt Code ganz ohne menschliche Freigabe.

Die Factory entsteht öffentlich auf software-factory.dev. Der PR Reviewer ist die Automation, die bei jedem Pull Request den Kreis schließt.

Zwei Trigger, eine Aktion pro Lauf

Er läuft auf zwei Wegen. Ein Pull-Request-Event weckt ihn, sobald sich etwas ändert. Zusätzlich geht eine Cron-Zeile, */30 * * * *, alle dreißig Minuten jeden offenen Pull Request durch, falls ein Event verloren ging.

Er arbeitet als kleine State Machine und führt pro Lauf genau eine Aktion aus:

• Ist die CI rot, liest er die Logs und pusht einen Fix.
• Ist die CI grün und der Pull Request noch ohne Review, prüft er ihn auf Korrektheit, Sicherheit, Konventionen und Design.
• Ist die CI grün und das Review sauber, löst er die offenen Review-Threads auf und mergt.

Er mergt nicht, solange auch nur ein Review-Thread ungelöst ist.

Bis hierhin ist das ein kompetenter Reviewer, bei dem eben kein Mensch in der Schleife sitzt. Dass man ihn unbeobachtet laufen lassen kann, liegt an einer einzigen Zahl.

Drei Versuche, dann ein Mensch

Hier steckt das Detail, das sich zu übernehmen lohnt. Der PR Reviewer zählt, wie oft er schon versucht hat, die CI auf genau diesem Pull Request zu reparieren. Die Markierung ist ein ci-fix-Tag in den Commit-Messages. Nach drei Versuchen hört er auf. Er schreibt einen Kommentar, dass die CI dreimal fehlgeschlagen ist und ein Mensch das untersuchen muss. Dann lässt er den Pull Request liegen.

Dieser Circuit Breaker trennt Autonomie von einer Schleife, die heißläuft. Ohne ihn bekommt ein Pull Request mit einem Problem, das die Automation nicht versteht, Fix-Versuch vier, fünf, sechs, endlos. Jeder Push löst den nächsten Lauf aus. Mit ihm hat die Automation eine bekannte Grenze: drei Versuche, dann ein Mensch.

Autonomie braucht einen Haltepunkt

Gefährlich ist die Automation, die keine Zahl kennt, ab der sie zugibt, dass sie feststeckt. Autonomie ist genau dann sicher, wenn sie einen Haltepunkt hat, den ihr laut benennen könnt.

Die ganze Automation könnt ihr in der Definition des PR Reviewers auf GitHub (öffnet in einem neuen Tab) nachlesen.

Bei welcher Zahl geben die Pull-Request-Automationen in eurer eigenen Factory das Problem zurück?

Weiterlesen

Die Artikel der Wissensbasis sind auf Englisch.

• Creator and verifier roles →
• Agentic Software Factory: Advanced Team Training →