Poradnik / Praktyka
Recenzent nie pomylił się ani razu i to był problem
Automatyczny recenzent Codex w projekcie AgentCoop nie pomylił się ani razu, a mimo to sześć rund recenzji rozdęło funkcję z 28 do 42 linii, bo zasada „każda uwaga załatwiona przed mergem” po cichu zmieniła się w „każdą poprawiamy”, a tak kończyło się 91% uwag. Zespół odpowiedział procedurą liczącą koszt w godzinach zamiast etykietek P1 i P2 oraz skryptem chain-check, który zatrzymuje pracę, gdy dwie rundy z rzędu trafiają w kod napisany pod wcześniejszą uwagę tego samego recenzenta. Arytmetyki nie kupuję, ale ktoś wreszcie spisał drugą stronę rachunku, czyli to, co naprawdę traci użytkownik.
Napisała AI, żeby oszczędzić Ci czytania setek źródeł. · Jak to działa
Poradnik / Praktyka · 5 min ·
Funkcja miała 28 linii. Po sześciu rundach recenzji ma 42, a zabezpieczenie dopisane na samym końcu chroni przed plikiem konfiguracyjnym, w którym ktoś użyłby naraz 1 oraz "int:1" jako dwóch osobnych kluczy. Tak zapisać można, tyle że nikt nigdy tego nie zrobił. Napisanie tej ochrony zajęło jakieś trzy godziny, a ona sama kosztuje dalej, bo zostaje w kodzie na zawsze i każdy, kto ten plik później otworzy, musi ją zrozumieć i utrzymać.
Recenzent nie pomylił się ani razu i dokładnie to okazało się problemem.
Historię opisał na DEV agent AI, który publikuje jako HammerMei i razem z człowiekiem rozwija AgentCoop, projekt pozwalający agentom rozmawiać z ludźmi i między sobą na firmowych czatach. Każdy pull request przechodzi tam przez Codexa, automatycznego recenzenta od OpenAI, a zasada domowa brzmi ostro: każda uwaga musi zostać załatwiona przed mergem, czyli poprawiona albo odrzucona z pisemnym uzasadnieniem. Reguła jest dobra i to właśnie ona wyhodowała te sześć rund wokół podwójnego klucza, bo słowo "załatwiona" po cichu zmieniło się w "poprawiona". Kiedy recenzent praktycznie nigdy się nie myli, poprawka jest najtańszym sposobem na zamknięcie wątku.
Czytaj dalej za darmo
Podaj e-mail, aby odblokować pełną treść i cały serwis. Zapiszemy Cię do newslettera.
