Repository navigation
fix(nfse): reject obsolete fiscal configuration endpoints and scope ADN view (#419 #421) - #489
Merged
Merged
Conversation
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
…company Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
…file Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
vitormattos
marked this pull request as ready for review
October 14, 2026 10:33
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Justificativa
Épico #417 veda escrita na configuração fiscal
nfse.*antiga e perfis de certificado por CNPJ. A UI por empresa (#420) não usa esses formulários, mas as rotas HTTP de legado continuavam permitindo gravação de identidades fiscais fora do perfil e de certificados sem vínculocompany_id.Correção delimitada
410 Goneapós middleware nativo de permissão.Dependências de integração
A confirmação municipal passa a ser administrada pelo perfil no PR #486. Os PRs #483/#482/#484 complementam a remoção dos demais leitores legados. Não integrar esta mudança antes do #486 quando a confirmação municipal for relevante no ambiente.
Validação e governança
Dois commits incrementais Conventional Commits com DCO e assinatura Git Signing MCP. Sem emissão fiscal real. PR draft até CI da SHA final e aprovação independente exigida pelo ruleset; não mudar CODEOWNERS/proteções nem usar bypass/squash.
Revisão adicional de segurança (#421)
Falha intermitente identificada na integração
A CI de integração com a
mainapresentou falha emCompetenceClosingReportTest::testClosingAndCsvExcludeForeignCompanyFiscalDocuments. A falha foi reproduzida com chaves sintéticas iguais por colisão do gerador de fixtures (random_int(1, 8)repetido 50 vezes); não demonstrou vazamento fiscal. A correção isolada e determinística está no PR #494, cuja CI passou. Revalidar os checks de integração desta branch depois que #494 for integrado. Não aumentar timeouts, ignorar asserções nem aprovar este PR com teste falhando.