Skip to content

refactor(tjsc): decompõe parser de resultado - #333

Draft
bdcdo wants to merge 3 commits into
mainfrom
refactor/tjsc-result-item
Draft

refactor(tjsc): decompõe parser de resultado#333
bdcdo wants to merge 3 commits into
mainfrom
refactor/tjsc-result-item

Conversation

@bdcdo

@bdcdo bdcdo commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Resumo

  • Decompõe o item de resultado do TJSC em normalização de rótulos, extração de processo/classe e armazenamento de campos.
  • Usa um único mapa de rótulos normalizados, removendo as variantes acentuada e sem acento duplicadas.
  • Formata o CNJ com format_cnj(..., strict=False), removendo corretamente o sufixo /TJSC sem depender de rstrip por conjunto de caracteres.
  • Preserva decisao como coluna própria e só a usa como fallback de ementa quando não existe ementa explícita, independentemente da ordem do HTML.

Simplificação e correções da revisão

  • Remove o helper de precedência dependente da ordem dos campos.
  • O iterador de itens achata as páginas e reduz cjsg_parse_manager de complexidade cognitiva 16 para 13.
  • Registra os efeitos públicos em [Unreleased]/Fixed.

Complexidade

  • Maior CCN tocado: 12; maior complexidade cognitiva tocada: 13.
  • Nenhuma função alterada excede o limite 15.

Validação

  • uv run pytest tests/tjsc -q: 24 testes passaram.
  • uv run pytest: 1.751 testes passaram, 32 foram ignorados e 220 ficaram desmarcados.
  • Pre-commit completo nos 4 arquivos alterados e git diff --check: passaram.

Refs #307

@bdcdo bdcdo left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revisão pré-merge concluída no commit aa463c3.

  • O CNJ deixa de carregar /TJSC; decisao fica preservada e só preenche ementa quando ela não existe.
  • Unifiquei os rótulos normalizados e reduzi cjsg_parse_manager de complexidade cognitiva 16 para 13.
  • A suíte offline passou com 1.751 testes; Lizard, Complexipy e pre-commit passaram.

O GitHub não reporta checks para esta branch. Mantive o PR em draft e não fiz merge.

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