# Resolução pós-review — bipagem-codigo-barras-fator

Resolução das issues do `06-review.md`. Decisões tomadas com o solicitante.

## M1 (major) — i18n desconectado → RESOLVIDO (remoção das chaves órfãs)
Decisão do solicitante: **remover as 5 chaves i18n órfãs** e tratar a i18n da bipagem como débito.
- As 5 entradas (hashes `e574…`, `53e9…`, `0faa…`, `1d38…`, `ade1…`) foram revertidas dos 3 idiomas via `git checkout -- linguagens/{pt-br,en,es}.json` (diff agora vazio).
- **Razão**: as mensagens de bipagem sempre foram hardcoded em PT (zero regressão); o JS não tem mecanismo de tradução (`show()` é PHP-only) e o `MissoesBipagem` é inteiramente hardcoded. Cabear só 5 strings daria cobertura parcial/ilusória.

## Débito registrado — i18n da bipagem
Internacionalizar as mensagens de separação/carregamento/bipagem é uma **task dedicada** (fora do escopo "validação + fator"):
- PHP (`MissoesBipagem`): migrar mensagens para `show()` — varredura completa do arquivo, não parcial.
- JS (`item.js` sep/carreg): exige criar a passagem de strings traduzidas para o front (data-attributes ou objeto JS) — não existe hoje.
- Memória de projeto relacionada: `project_bipagem_fator_validacao`.

## Minors → LIMPOS
- **m1**: resíduo de seed no `FatorBipagemIntegrationTest.php` (item inserido com `produto_id` inválido e depois deletado) — removido.
- **m2**: expressão morta `? 0 : 0` no caso 7 — removida.
- Teste rerodado após a limpeza: **42/42 PASS**.

## Nits → aceitos
- **n1**: somem junto com a remoção das chaves i18n (eram referência às chaves).
- **n2**: `block_*` auxiliares nullable na migration — consciente/justificado (o framework injeta; só `block` é NOT NULL). Sem ação.

## Status final
0 blockers · M1 resolvido · minors limpos. Backend + frontend revisados; integração **42/42**. **Nada commitado** (aguarda ordem do solicitante). Pendência operacional: smoke manual do frontend (`item.js` não é coberto por automação).