Codex Review

Was macht Codex Review?

Führt die integrierte Code-Review von Codex als Abschlussprüfung durch. Dies ist die Code-Review (codex review), nicht die Guardian auto_review-Genehmigungsweiterleitung.

Verwenden bei:

• Benutzer wünscht Codex-Review / Autoreview / Zweitmodell-Review
• nach nicht-trivialen Code-Änderungen, vor finalem Commit / Auslieferung
• Überprüfung eines lokalen Branches oder PR-Branches nach Korrekturen

Vereinbarung

• Behandle die Review-Ausgabe als beratend. Wende sie niemals blind an.
• Überprüfe jeden Fund, indem du den tatsächlichen Codepfad und die benachbarten Dateien liest.
• Lies die Dokumentation / Quellen / Typen von Abhängigkeiten, wenn der Fund von externem Verhalten abhängt.
• Lehne unrealistische Randfälle, spekulative Risiken, großflächige Umschreibungen und Korrekturen ab, die die Codebasis übermäßig verkomplizieren.
• Bevorzuge kleine Korrekturen an der richtigen Zuständigkeitsgrenze; kein Refactoring, es sei denn, es verbessert eindeutig die Fehlerklasse.
• Fahre fort, bis die Codex-Review keine akzeptierten/umsetzbaren Funde mehr meldet.
• Wenn eine durch die Review ausgelöste Korrektur Code ändert, führe fokussierte Tests und die Codex-Review erneut aus.
• Wechsle oder überschreibe das Review-Modell niemals. Sollte die Review die Modellkapazität erreichen, wiederhole denselben Befehl einige Male mit demselben Modell. Das Hilfsskript führt die verschachtelte Review standardmäßig im YOLO-/Vollzugriffsmodus aus; verwende --no-yolo nur, wenn du bewusst das Sandbox-Verhalten testen möchtest.
• Beende, sobald der Review-Befehl/das Hilfsskript mit Exit-Code 0 und ohne akzeptierte/umsetzbare Funde beendet wird. Führe keinen zusätzlichen direkten codex review-Befehl aus, nur um eine schönere „saubere“ Ausgabe, eine zweite Meinung oder eine klarere Abschlussformulierung zu erhalten.
• Behandle den erfolgreichen Exit des Hilfsskripts zusammen mit dem Fehlen umsetzbarer Funde als das saubere Review-Ergebnis, auch wenn die zugrunde liegende Codex-CLI-Ausgabe knapp ist.
• Wenn du einen Fund als beabsichtigt oder nicht korrekturbedürftig ablehnst, füge nur dann einen kurzen Inline-Kommentar hinzu, wenn er eine echte Invariante oder Zuständigkeitsentscheidung erklärt, die zukünftige Prüfer kennen sollten.
• Pushe nicht nur, um zu reviewen. Pushe nur, wenn der Benutzer Push/Auslieferung/PR-Update angefordert hat.

Ziel auswählen

Lokale unversionierte Arbeit:

codex review --uncommitted

Verwende dies nur, wenn der Patch tatsächlich unstaged/staged/untracked im aktuellen Checkout ist. Für committete, gepushte oder PR-Arbeit weise Codex stattdessen auf den Commit- oder Branch-Diff an; erzwinge nicht --mode local / --uncommitted, nur weil die Hilfsskript-Dokumentation zuerst die unversionierte Arbeit erwähnt. Ein sauberer --uncommitted-Review beweist nur, dass es keinen lokalen Patch gibt.

Branch-/PR-Arbeit:

git fetch origin
codex review --base origin/main

Übergib keinen Inline-Prompt mit --base; die aktuelle CLI lehnt --base + [PROMPT] ab, auch wenn der Hilfetext mehrdeutig ist. Wenn benutzerdefinierte Anweisungen benötigt werden, führe zuerst den einfachen --base-Review durch, dann einen lokalen/manuellen Folge-Durchlauf.

Wenn ein offener PR existiert, verwende seinen tatsächlichen Base-Branch:

base=$(gh pr view --json baseRefName --jq .baseRefName)
codex review --base "origin/$base"

Committete einzelne Änderung:

codex review --commit HEAD

oder mit dem Hilfsskript:

/Users/steipete/Projects/agent-scripts/skills/codex-review/scripts/codex-review --mode commit --commit HEAD

Verwende den Commit-Review für bereits gelandete oder gepushte Arbeit auf main. Das Überprüfen von sauberem main gegen origin/main ist nach dem Push normalerweise ein leerer Diff. Bei einem kleinen Stack überprüfe jeden Commit explizit oder überprüfe den Branch vor dem Merge mit --base.

Paralleler Abschluss

Formatiere zuerst, wenn die Formatierung die Zeilenpositionen ändern kann. Danach ist es in Ordnung, Tests und Review parallel auszuführen:

scripts/codex-review --parallel-tests "<fokussierter Testbefehl>"

Abwägung: Tests können Code-Änderungen erzwingen, die den Review ungültig machen. Wenn Tests oder Review zu Code-Änderungen führen, führe die betroffenen Tests und den Review erneut aus, bis keine akzeptierten/umsetzbaren Funde mehr vorhanden sind. Sobald dieser erneute Durchlauf sauber beendet wird, stoppe; investiere keinen weiteren langen Review-Zyklus in redundante Bestätigung.

Kontexteffizienz

Codex-Review ist normalerweise verrauscht. Verwende standardmäßig einen Subagenten-Filter, wenn Subagenten verfügbar sind. Bitte ihn, die Review durchzuführen und nur zurückzugeben:

• akzeptierte/umsetzbare Funde
• abgelehnte Funde mit einzeiliger Begründung
• genaue Dateien/Tests, die erneut ausgeführt werden müssen

Führe den Inline-Review nur bei winzigen Änderungen oder wenn Subagenten nicht verfügbar sind, durch.

Hilfsskript

Gebündeltes Hilfsskript:

~/.codex/skills/codex-review/scripts/codex-review --help

Wenn aus agent-scripts installiert, lautet der Pfad:

/Users/steipete/Projects/agent-scripts/skills/codex-review/scripts/codex-review --help

Das Hilfsskript:

• wählt zuerst --uncommitted, wenn unversionierte Änderungen vorliegen
• verwendet andernfalls den aktuellen PR-Base-Branch, wenn gh pr view funktioniert
• verwendet andernfalls origin/main für Nicht-main-Branches
• verwende --mode commit --commit <ref> für bereits committete Arbeit, insbesondere sauberes main nach dem Landen
• sollte im Modus --mode auto belassen oder für PR-/Branch-Arbeit auf --mode branch gesetzt werden; erzwinge --mode local nicht nach dem Commit
• schreibt nur nach stdout, es sei denn, --output oder CODEX_REVIEW_OUTPUT ist gesetzt
• unterstützt --dry-run, --parallel-tests und Commit-Referenzen
• führt die verschachtelte Review standardmäßig mit --dangerously-bypass-approvals-and-sandbox aus
• akzeptiert weiterhin --full-access; verwende --no-yolo oder CODEX_REVIEW_YOLO=0, um abzulehnen
• gibt codex-review clean: no accepted/actionable findings reported aus, wenn der ausgewählte Review-Befehl mit Exit-Code 0 beendet wird

Abschlussbericht

Enthalten:

• verwendeter Review-Befehl
• ausgeführte Tests/Nachweise
• akzeptierte/abgelehnte Funde mit kurzer Begründung
• das saubere Review-Ergebnis des letzten Hilfsskript-/Review-Durchlaufs oder warum ein verbleibender Fund bewusst abgelehnt wurde

Führe keinen weiteren Codex-Review nur durch, um die Formulierung des Abschlussberichts zu verbessern. Wenn der letzte Hilfsskript-Durchlauf mit Exit-Code 0 beendet wurde und keine akzeptierten/umsetzbaren Funde erzeugte, melde genau diesen Durchlauf als sauber.