Skip to content

refactor(esaj): decompõe orquestrador de auto-chunk - #335

Draft
bdcdo wants to merge 3 commits into
mainfrom
refactor/esaj-auto-chunk
Draft

refactor(esaj): decompõe orquestrador de auto-chunk#335
bdcdo wants to merge 3 commits into
mainfrom
refactor/esaj-auto-chunk

Conversation

@bdcdo

@bdcdo bdcdo commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Resumo

  • Mantém a orquestração compartilhada de auto-chunk entre eSAJ cjsg e TJSP cjpg, com validação antes da primeira requisição e deduplicação por janela.
  • Normaliza datas uma única vez; cada alias de busca ou data emite exatamente um DeprecationWarning, tanto no caminho curto quanto no dividido.
  • Preserva query/termo como pesquisa canônica antes de consumir aliases, de modo que o valor chegue a todas as janelas.

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

  • Remove a supressão seguida de reemissão manual de warnings.
  • Incorpora _normalize_auto_chunk_search e _validate_auto_chunk_input, ambos usados uma vez, ao fluxo principal.
  • Registra em [Unreleased]/Fixed o alias de pesquisa que antes era validado e depois descartado no cjpg com janela superior a 366 dias.

Complexidade

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

Validação

  • Testes focados de auto-chunk: 34 testes passaram.
  • uv run pytest: 1.758 testes passaram, 32 foram ignorados e 220 ficaram desmarcados.
  • Pre-commit completo nos 6 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 66a0b53.

  • query/termo agora alimentam todas as janelas e cada alias emite um único DeprecationWarning.
  • Removi a supressão/reemissão manual de warnings e incorporei dois helpers usados uma vez.
  • A suíte offline passou com 1.758 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