feat(input): accept alphanumeric CNPJ in OceanInputType (OR-98)

[FabioRolin]

Summary

• OceanInputType.CNPJ and OceanInputType.CpfCnpj now accept the alphanumeric CNPJ format defined by Receita Federal NT 49/2024 ([A-Z] allowed in positions 1..12, DVs stay numeric).
• Keyboard switches from KeyboardType.Number to KeyboardType.Ascii so the user can actually type letters. On POS hosts that disable the numeric virtual keyboard via OceanDS.disabledKeyboards, the Ascii keyboard still opens normally (only Number is disabled).
• New helper sanitizeWithAlphanumeric on the base OceanInputType interface — keeps [A-Z0-9], upper-cases letters, drops separators.
• CPF stays untouched (digits-only + numeric keyboard).
• Backward compatible: every numeric CNPJ that worked before keeps working.

Refs spec Pagnet/blu-mobile-android#3650 (OR-98) — delivers task T2 of the execution plan (Ocean DS Android is the first project in the queue, blocks consumer migrations T6/T7).

Test plan

• ./gradlew :ocean-components:testDebugUnitTest — new OceanInputTypeTest (10+ cases) + existing suite green.
• ./gradlew :ocean-components:ktlintCheck — no violations.
• Manual smoke on POS device — confirm Ascii virtual keyboard opens for CNPJ field even with N960KInputSetup active (covered by T8 in the spec).

Reviewers: @Pagnet/plataformas and mobile devs.

🤖 Generated with Claude Code

[gustavoalens]

Code Review

Resumo

PR implementa suporte ao formato alfanumérico de CNPJ (Receita Federal NT 49/2024) no OceanInputType, afetando CNPJ e CpfCnpj. A mudança é backward compatible, bem coberta por testes (10+ casos) e a lógica está correta. Dois arquivos modificados (+145/-7).

---

Apontamentos

🔴 Blockers

Nenhum.

🟡 Warnings

• UX degradation no campo CpfCnpj em modo CPF

  • Arquivo: OceanInputType.kt:231
  • Problema: CpfCnpj agora sempre abre teclado Ascii, mesmo quando o usuário está digitando um CPF (11 dígitos, sem letras). Teclados Ascii não têm layout numérico por padrão — usuários que digitam apenas CPF (maioria) vão encontrar um teclado menos conveniente.
  • Sugestão: Considerar retornar o tipo de teclado dinamicamente com base no estado atual do campo, ou aceitar o tradeoff e documentar claramente nos release notes. Vale validar com UX antes do merge nos consumidores.

• Lógica de detecção de letras divergente entre isCnpj e sanitize

  • Arquivo: OceanInputType.kt:258, 270
  • Problema: isCnpj usa currentValue.any { it in 'A'..'Z' } (valor já sanitizado, sempre uppercase). Já sanitize usa text.any { it.uppercaseChar() in 'A'..'Z' } (input bruto). Funcionalmente correto por conta da pipeline de sanitização, mas a inconsistência pode confundir mantenedores.
  • Sugestão: Centralizar a detecção de letras num único helper privado para garantir consistência.

🟢 Suggestions

• Comentários multi-linha em produção podem ser condensados — OceanInputType.kt:234-239, 251-253, 265-268: os blocos de 3-4 linhas de // explicam contexto não óbvio (POS, NT 49/2024), o que é válido, mas cabem em 2 linhas cada.

• KDoc no arquivo de teste é verboso — OceanInputTypeTest.kt:1-10: o KDoc de classe explica o contexto em 4 linhas; para teste, uma linha bastaria, pois os nomes dos testes já são autoexplicativos.

• CPF_LENGTH = 11 não compartilhado — existe apenas dentro de CpfCnpj. Se CPF precisar desse valor no futuro, haverá duplicação. Baixo risco agora, mas vale o registro.

---

Cobertura de Testes

• Sanitização — uppercase, separadores, símbolos
• Tipo de teclado (CNPJ → Ascii, CPF → Number)
• Max length e truncation
• Backward compatibility — CNPJ numérico continua funcionando
• CpfCnpj com letras roteado corretamente para CNPJ alfanumérico
• CPF permanece digits-only
• Edge case ausente: CpfCnpj com exatamente 11 dígitos (transformForInput("12345678909")) confirmando que permanece em modo CPF e não transborda para CNPJ — vale um teste explícito no boundary.

---

Veredito

APPROVED — lógica correta, backward compatible, bem testado. O warning de UX do teclado Ascii em modo CPF é o ponto mais relevante para validar com produto/UX antes do merge nos consumidores (T6/T7).

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud