Zrób z review bramkę
Przegląd i wdrożenie z Codex
ENBezpieczne wdrażanie kodu napisanego przez Codex polega na nałożeniu kilku kontroli między diffem a produkcją: /review przed commitem i codex exec review przed pushem, @codex review na pull requeście, openai/codex-action w CI oraz wdrożenie z przećwiczonym rollbackiem. Review od Codex to jeden z sygnałów; o scaleniu decydują dowody ze wszystkich bramek razem.
Ten przegląd jest dla programistów i tech leadów, u których sesje Codex produkują więcej pull requestów, niż zespół jest w stanie przeczytać linijka po linijce. Diffy wyglądają schludnie, styl jest jednolity, a błędy chowają się tam, gdzie przestałeś zaglądać: brakujące sprawdzenie uprawnień, pominięty test, migracja bez backfillu. Strony poniżej zastępują czytanie każdej linijki bramkami, które da się zmierzyć. Polecenia sprawdzono na Codex CLI 0.157.1.
Która strona o wdrażaniu z Codex odpowiada na twoje pytanie?
Dział zatytułowany „Która strona o wdrażaniu z Codex odpowiada na twoje pytanie?”| Twoje pytanie | Strona |
|---|---|
| Jak zrobić z review od Codex bramkę dla zespołu i wiedzieć, że łapie prawdziwe błędy? | Code review z Codex |
| Jak głęboko człowiek ma patrzeć w każdą zmianę i jaki tryb zatwierdzania wybrać? | Przegląd pracy agenta bez stawania się wąskim gardłem |
| Jak uruchomić Codex w GitHub Actions, nie wystawiając sekretów? | Codex w CI/CD z GitHub Action |
| Jak pull request przechodzi z worktree do scalenia z pomocą Codex? | Automatyzacja pull requestów z Codex |
| Jak wdrożyć to, co zbudował Codex, i jak to wycofać? | Automatyzacja wdrożeń z Codex |
Jeśli nie robiłeś jeszcze review zmiany od Codex, zacznij od przeglądu zmian Codex przed commitem. Bramka jest tak mocna jak testy, które za nią stoją; zobacz testowanie i debugowanie z Codex.
W jakiej kolejności zmiana od Codex ma trafić na produkcję?
Dział zatytułowany „W jakiej kolejności zmiana od Codex ma trafić na produkcję?”Tanie, prywatne kontrole idą pierwsze, dzięki czemu publiczne review nadal jest warte czytania.
-
Przed commitem. Uruchom
/revieww sesji albocodex review --uncommittedw terminalu i popraw każde znalezisko P0 i P1 (Codex oznacza każde znalezisko priorytetem od P0 do P3, gdzie P0 jest najpoważniejsze). -
Przed pushem. Hook pre-push uruchamia review jako skrypt i blokuje push przy znaleziskach P0 lub P1. W terminalu:
Okno terminala codex exec review --base origin/main --ephemeral -o .codex-review.mdSam hook i sposób sprawdzenia formatu jego wyniku znajdziesz na stronie o code review.
-
Na pull requeście. Skomentuj
@codex review, a przy uwierzytelnianiu, parsowaniu wejścia i sekretach także@codex security review. Automatyczne review włącz dopiero wtedy, gdy reguły review wAGENTS.mdsą dostrojone. -
W CI. Testy, typy i lint uruchamiają się na wypchniętej gałęzi. Uruchom
openai/codex-action@v1zpermission-profile: ":read-only"i promptem do review, a osobny krok niech kończy job błędem, gdy wynik zawiera znalezisko P0. CI blokuje tylko na P0, luźniej niż hook pre-push, bo znaleziska P1 i tak trafiają na pull request, gdzie decyduje o nich człowiek, właściciel zmiany; tę bramkę buduje strona o CI/CD. Tryb tylko do odczytu oznacza, że Codex nie może edytować kodu ani testów, żeby przejść kontrole. -
Przy scaleniu. Wskazana osoba zatwierdza ścieżki eskalacji z
CODEOWNERS: migracje, uwierzytelnianie, pieniądze, kontrakty publicznego API oraz zmiany w testach i CI. -
Przy wdrożeniu. Wdrażaj za feature flagą albo przez canary, z rollbackiem, który przynajmniej raz uruchomiłeś.
Skąd wiesz, że zmianę od Codex można wdrożyć, bez czytania każdej linijki?
Dział zatytułowany „Skąd wiesz, że zmianę od Codex można wdrożyć, bez czytania każdej linijki?”Każda bramka daje dowód, za który ktoś odpowiada. Czytaj dowody, a do diffu zaglądaj tam, gdzie one wskazują.
| Dowód | Co udowadnia | Właściciel |
|---|---|---|
| Odsetek wykrytych celowo wprowadzonych błędów (seeded bugs) w review od Codex | Review znajduje klasy błędów, na których ci zależy | Tech lead |
| Zielone CI na wypchniętej gałęzi: testy, typy, lint | Dotychczasowe zachowanie nadal działa | CI |
| Bramka znalezisk Codex bez P0 (P1 ocenia człowiek, właściciel zmiany) | Brak blokujących problemów według twoich reguł z AGENTS.md | Właściciel promptu i workflow review w CI (w CODEOWNERS) |
Zatwierdzenie ścieżek eskalacji z CODEOWNERS | Człowiek przyjął ryzyko tam, gdzie przeoczenie jest drogie | Wskazany code owner |
| Metryki canary i przećwiczony rollback | Produkcja potwierdza zmianę albo szybko ją cofniesz | Dyżurny inżynier |
Co powinien zawierać pull request od agenta, opisuje pakiet dowodów (evidence bundle), a triaż, który z niego korzysta, review pull requesta od agenta bez czytania każdej linijki.
Co psuje się między zatwierdzonym diffem a działającym wdrożeniem?
Dział zatytułowany „Co psuje się między zatwierdzonym diffem a działającym wdrożeniem?”| Objaw | Przyczyna | Co zrobić |
|---|---|---|
| Zespół ignoruje komentarze Codex | Za dużo znalezisk P2 i P3 | Na pull requestach raportuj tylko P0 i P1; dodaj do AGENTS.md linię „Do not flag” dla każdej kategorii szumu |
codex exec review --base main "focus on auth" kończy się błędem | W 0.157.1 każda z flag --base, --commit i --uncommitted odrzuca własny prompt | Usuń prompt, a to, na czym review ma się skupić, opisz w AGENTS.md |
| Pull request osłabia własne review w CI | Prompty i workflowy ładują się z merge commita pull requesta | Obejmij .github/ regułą w CODEOWNERS |
| Sekret wycieka z joba uruchamianego przez PR | Niezaufany kod działał z sekretami w środowisku | Zanim włączysz ten trigger, przejdź przez tożsamość agentów i sekrety |
| Recenzent (Codex) zatwierdza kod, który sam napisał | Ten sam model, ta sama sesja | Ustaw review_model na inny model i uruchamiaj review w osobnym procesie |
| Rollback nie działa na produkcji | Migracja usunęła dane, których potrzebuje stary kod | Rozbij ją według wzorca expand and contract; nie wdrażaj destrukcyjnej migracji razem z kodem, który jej wymaga |
Dokąd dalej przy wdrażaniu zmian od Codex
Dział zatytułowany „Dokąd dalej przy wdrażaniu zmian od Codex”Zabezpiecz CI
Utrzymuj to, co wdrożone
Dla wszystkich narzędzi
Korzystasz też z innego agenta? Ten sam etap opisują przegląd i wdrożenie z Claude Code oraz przegląd i wdrożenie z Cursor.