Skip to content

refactor(tjsp): decompõe download do CJPG - #334

Draft
bdcdo wants to merge 3 commits into
mainfrom
refactor/tjsp-cjpg-download
Draft

refactor(tjsp): decompõe download do CJPG#334
bdcdo wants to merge 3 commits into
mainfrom
refactor/tjsp-cjpg-download

Conversation

@bdcdo

@bdcdo bdcdo commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Resumo

  • Mantém a decomposição do download CJPG em primeira página, diagnóstico, normalização das páginas e gravação dos arquivos.
  • Preserva cjpg_debug, o HTML diagnóstico e o encadeamento da exceção quando a contagem falha.
  • Mantém listas esparsas, range com passo, URLs, ordem das requisições e pausas entre páginas.

Simplificação da revisão

  • Remove a injeção opcional de callback e a closure criada no client; cjpg_download chama diretamente a fonte canônica cjpg_n_pags.
  • Remove o estado inválido de callback ausente, sem criar fallback.
  • Usa Path internamente e converte para str somente na fronteira pública; consolida helpers duplicados nos testes.

Complexidade

  • cjpg_download: CCN 8 e complexidade cognitiva 6.
  • Nenhuma função alterada excede o limite 15.

Validação

  • uv run pytest: 1.751 testes passaram, 32 foram ignorados e 220 ficaram desmarcados.
  • Testes focados do fluxo e dos diagnósticos CJPG: passaram.
  • Pre-commit completo nos 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 1b3acf4.

  • Removi o callback opcional e a closure do client; o download usa diretamente cjpg_n_pags.
  • Preservei diagnóstico, causa da exceção, paginação e retorno público em str, com Path apenas internamente.
  • A suíte offline passou com 1.751 testes; cjpg_download ficou em CCN 8/cognitivo 6 e pre-commit passou.

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