Skip to content

fix(danfse): align QR vertical placement with NT008 2.4.3 (#118) - #141

Merged
vitormattos merged 2 commits into
mainfrom
fix/danfse-qr-y-nt008-118-20261013
Oct 13, 2026
Merged

vitormattos merged 2 commits into
mainfrom
fix/danfse-qr-y-nt008-118-20261013

Conversation

@vitormattos

@vitormattos vitormattos commented Oct 13, 2026 •

Copy link
Copy Markdown
Member

Contraprova objetiva — NT008 v1.02, item 2.4.3

A especificação fixa QR Code com tamanho mínimo de 1,52 × 1,52 cm e posição X=17,48 cm / Y=1,67 cm no DANFSe. O artefato real danfse-visual-evidence do workflow 38306909004 (commit aprovado f344818c, árvore idêntica à main) contém 10 PDFs sintéticos.

A leitura geométrica de seus objetos vetoriais no PDF encontrou:

  • QR Code de 1,644 × 1,644 cm (mínimo atendido);
  • X=17,528 cm (próximo à referência);
  • Y=1,363 cm (aproximadamente 3,1 mm acima da coordenada indicada), repetido nos casos padrão.

A suíte existente verifica conteúdo, fontes e paginação, mas não detectou essa diferença de posição. Não se deve declarar conformidade visual integral apenas pelos checks anteriores.

Correção mínima isolada

Dois commits incrementais assinados ajustam apenas o CSS do QR: padding-top: 8.7pt na .qr-container e position: relative; left: -1.4pt no elemento da imagem. XML, conteúdo do QR, dimensões, textos, fontes, tributos, API pública, Composer e cache permanecem inalterados.

Evidência final e gate de governança

  • Executar as suítes existentes e DANFSe Visual Evidence na SHA final do PR.
  • Baixar o novo artefato da SHA e medir novamente a caixa vetorial do QR em centímetros; comparar posição X/Y e dimensão com o item 2.4.3.
  • Confirmar pdfinfo: 10/10 A4 retrato, 1 página, sem cortes/colisões no cabeçalho; pdffonts: Liberation Sans embutida, fontes customizadas isoladas; pdftotext: chave, identificadores e rodapé íntegros, serviço extenso com reticências visíveis.
  • Se Y permanecer divergente ou houver regressão, corrigir o mínimo e testar novamente antes de considerar o PR pronto.
  • Exigir aprovação independente elegível de CODEOWNER e todas as proteções da main, sem bypass e sem squash automático.

Resultado definitivo na SHA 014e6a9c59708fb5d21a467c38b12fe6136b2066

  • 12 checks concluídos com sucesso e 1 job de aprovação automática skipped (esperado; não é aprovação manual).
  • GitHub Actions run 38343315048, artefato 11787616994: 10/10 PDFs sintéticos A4 retrato, uma página, fonte incorporada e texto fiscal extraível sem regressão observada.
  • Objetos vetoriais do QR (9 casos padrão): X=17,4791 cm, Y=1,6701 cm, 1,6439 × 1,6439 cm. Distâncias da coordenada nominal: aproximadamente 0,009 mm em X e 0,001 mm em Y. Caso de fonte personalizada: Y=1,6825 cm (diferença +0,125 mm), sem deslocar a fonte padrão após o teste.
  • Inspeção do serviço extenso: reticências presentes, rodapé íntegro, QR sem colisão visual observada.
  • main e akaunting-nfse permanecem em seus SHAs anteriores; nenhuma correção foi integrada nem repinada no consumidor.

Essa correção não altera a divergência tipográfica nominal Liberation Sans versus Arial/Microsoft Sans Serif, ainda documentada sob #118. A correção exclusivamente documental de IBS/CBS tramita de forma independente no PR #140. Não encerrar #118 automaticamente.

Refs #118.

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@vitormattos
vitormattos merged commit 484831e into main Oct 13, 2026
13 checks passed
@vitormattos
vitormattos deleted the fix/danfse-qr-y-nt008-118-20261013 branch October 13, 2026 11:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant