Repository navigation
Handoff: vollständigen Codex-Abschluss prüfen und Fehler verständlich melden - #79
Merged
DeepWolf42 merged 1 commit intoNov 1, 2026
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Der Codex-Adapter konnte eine vorhandene Vorschlagsdatei auch ohne erfolgreich abgeschlossenen CLI-Turn akzeptieren. Dieser Kandidat verlangt einen vollständigen, widerspruchsfreien Ereignisstrom und bindet die finale Nachricht an den geprüften ChangeSet. Fehler, unvollständige Turns, Werkzeuge und abweichende Vorschläge werden vor Änderungen verworfen und über den bestehenden Handoff als FAILED_WORKER gespeichert.
Der vorhandene Runner erfasst stdout/stderr optional getrennt, mit gemeinsamer Ausgabegrenze und unveränderter Prozessbegrenzung. Nur stdout ist der Codex-Ereignisstrom. Standardverhalten und Ergebnisschema anderer Prüfer bleiben unverändert. Neue feste Diagnosecodes und --human liefern deutsche Bedienhinweise ohne rohe Workerdiagnostik; Erfolgs-JSON und Exit-Codes bleiben erhalten, Eingabefehler ergänzen diagnostic_code.
Basis: #78 / 28c6552. Eigener Head: 5ab32fd. Die parallele Historien-/Indexprüfung des Parents bleibt erhalten. Nur neun eigene Dateien geändert, einschließlich synchroner EN/DE-Dokumentation.
Validierung: 25 neue Tests; gesamte lokale Windows-Suite320 Tests PASS/5 Capability-Skips. Beide Completion-Gates auf exakt5ab32fd PASS. Unabhängiger Review prüfte Prozess-/Ausgabegrenzen, finale25 Gegenfälle, einen separaten Statusprozess und tatsächliche Vorschlags-/Diffbindung. Drei reale CLI-Aufrufe mit synthetischen freigegebenen Eingaben: erster Versuch korrekt vor Apply gestoppt wegen Kanalmischung, zweiter nach Fix erfolgreich, dritter auf neuer Parent-Basis erfolgreich. Keine alten Versuche überschrieben. Finaler Status/Duplicate/Human-Aufruf erfolgreich; Quelle unverändert. Codex CLI0.166.0-alpha.5.1, vorhandener ChatGPT-Login; Modellidentität und Geldkosten UNKNOWN. Private Pfade, Rohlogs und Profile bleiben außerhalb GitHub.
Gehostete CI41319744112 SUCCESS: Ubuntu320Tests/1Skip, Windows320Tests/2Skips, beide Completion-Gates und repository-policy PASS. Getesteter synthetischer Mergecheckout864004b2d0f156858660f3afb45866e419811565 aus exakt Head5ab32fd und Parent28c6552; kein Produktmerge. Der auf gestapeltem Ziel sonst nicht automatisch laufende bestehende Secret-Scan41319852988 wurde einmal auf dem eigenen Zweig separat gestartet und bestand auf exakt5ab32fd. Weiterhin eigener Draft auf #78. Kein Merge, Release, neuer Dienst, Rechte-/Consumer-Pinwechsel oder allgemeiner Modell-/Cloud-Qualitätsnachweis. Der Runner dekodiert weiter mit UTF-8-Ersetzung: keine Rohbyte-Attestation. Aufbaufehler vor dem geschützten Parser können weiterhin bestehendes UNKNOWN/Recovery ergeben. Geänderte Prüferquellen verlangen neue Belegqualifikation.