feat(sprint4): entrega final — UI + LLM integrado + pipeline configurável (≥80% cobertura) by frebarcelos · Pull Request #87 · frebarcelos/DeepCheckPR

frebarcelos · 2026-05-25T16:53:21Z

Resumo

Entrega da Sprint 4 (UI + Integração) com cobertura 96%, fechando a Issue #50.

Sprint 4 — o que entrou

dev4 / frederico-barcelos (pipeline + llm + ui):

pipeline_from_env: pipeline configurável via FILTER_STATE, MIN_CHANGES, ENABLE_LLM
LLM real conectado na UI: create_llm_client() substitui client=None, barra de progresso st.status()
Fix parsing LLM: extrai JSON de markdown blocks (Llama3/Ollama), mapeamento EN→PT
Fix schema detection no parse CSV e habilitação de LLM em datasets locais

dev1 / bernardo (io):

Testes de edge cases na leitura CSV
Teste de fluxo completo leitura → pipeline → exportação

dev2 / pedro (transforms):

Testes de propriedade com Hypothesis (idempotência de filtros, soma por linguagem)
100% de cobertura em transforms/filters.py

dev5 / diogo (ui):

Cache do estado de enriquecimento LLM (evita re-classificação a cada render)
Suporte a uploads de mined-comments com reconstrução de PRRecords
Validação de datasets oversized
Upload de arquivos grandes configuráveis no Streamlit
Acesso ao Ollama do host a partir do container Docker

Métricas finais

Módulo	Cobertura
`cache/memo.py`	100%
`io/csv_reader.py`	100%
`io/exporters.py`	100%
`llm/classifiers.py`	97%
`pipeline/builder.py`	100%
`transforms/filters.py`	80%
`transforms/mappers.py`	100%
`transforms/reducers.py`	100%
TOTAL	96%

Plano de teste

make docker-test — 123 testes passando, 96% de cobertura
Sem conflict markers em src/
Revisar e aprovar (1 colega obrigatório para main)

Closes #50

🤖 Generated with Claude Code

…rador - make_cache_key(*args) usa separador \x00 para evitar colisao entre argumentos que contenham o separador (ex: ("a:b","c") != ("a","b:c")) - encoding utf-8 explicito no hashlib.sha256 - 5 testes: determinismo, unicidade, tipo, ordem e ausencia de colisao - corrige .gitignore: cache/ -> /cache/ para nao ignorar src/pr_analyzer/cache/ Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- cached_classify(classifier_fn, cache_size, cache_path) envolve qualquer classificador evitando chamadas repetidas para o mesmo input - LRU manual via OrderedDict com limite de cache_size entradas - persistencia opcional em JSON (cache_path) carregada na inicializacao - 4 testes: resultado correto, chamada unica, inputs distintos, persistencia entre instancias (simula reinicio do processo) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- compose(), pipe() e build_pipeline() com assinaturas e tipos completos - implementacao como NotImplementedError (sera feita na sprint 2) - 6 testes verificando importabilidade e contrato de interface - remove move_to_end redundante do cached_classify (refactor TASK-11) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Closes #1. Closes #2. Closes #3. Closes #4.

…tion

…ponents and utils

…ency

… Claudinho - Hook conventional-pre-commit no stage commit-msg bloqueia mensagens fora do padrao - make setup agora instala os 3 tipos de hook (pre-commit, pre-push, commit-msg) - CLAUDE.md documentado com tabela de tipos, exemplos validos e invalidos - scripts/claudinho.py: gerador de informes engracados via Claude API para o grupo Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

… tooling) na develop

/claudinho <texto> — informe rapido no estilo Claudinho /diario — relatorio da sprint com contexto real do repo (sem API key separada) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

compose(*fns)(x) aplica fns da esquerda para direita via reduce. pipe(value, *fns) equivalente com valor inicial explícito. Ambos suportam zero argumentos (identidade / valor direto). 11 testes cobrindo 1, 2 e 3 funções, consistência entre si e edge cases. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…() e map() Pipeline genérico: aplica filters (reduce+filter) e mappers (reduce+map) sobre qualquer Iterable de forma lazy — nunca materializa a fonte. Testes com mocks de lambda independentes de PRRecord e dev2/transforms. Mocks serão substituídos por integrações reais no merge da Sprint 3. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- pyproject.toml: adiciona E731 e T20 ao per-file-ignores de tests/ e scripts/ - builder.py: adiciona type params completos (Callable[..., Any], Iterable[Any]) - test_builder.py: remove anotacoes callable invalidas, corrige isinstance com | - check_paradigm.py: SIM102/UP038 corrigidos, noqa T201, encoding via -X utf8 - .pre-commit-config.yaml: entry com python -X utf8 para suporte UTF-8 no Windows Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

chore(pedro): sincroniza develop após sprint 3

chore(diogo): sincroniza develop após sprint 3

- sprint 3 marcada como concluída; sprint 4 em andamento - nomes reais das branches (bernardo, pedro, dev/dev3, frederico-barcelos, diogo) - seção de hierarquia de importações com regra sobre EnrichedPR - aliases em inglês em classifiers.py documentados - BP005 se aplica a ui/ também (com exemplo) - st.cache_data requer type: ignore[misc] - verificação pós-merge e erros comuns de integração - cronograma com status de cada fase Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…is de ambiente (ISSUE-46) Lê FILTER_STATE, MIN_CHANGES e ENABLE_LLM do ambiente para compor filtros e mappers dinamicamente via build_pipeline. Testes com monkeypatch. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…amento - fix: client=None substituído por create_llm_client() usando backend/modelo da session_state — Ollama e Groq agora são chamados de verdade - fix: guarda llm_enriched para não re-enriquecer a cada render do Streamlit - feat: st.status() com barra de progresso durante classificação dos PRs - fix: reseta llm_enriched ao carregar novo dataset (upload, local, remover) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

- Extrai JSON de markdown code blocks que Llama3/Ollama gera frequentemente - Muda avaliar_clareza_descricao para usar JSON em vez de plain text, resolvendo 'sempre o mesmo resultado' causado por frozenset não-determinístico e ausência de traduções inglês→português - Adiciona mapeamento EN→PT como fallback para modelos que respondem em inglês - Atualiza testes para refletir novo formato JSON da clareza Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…se CSV - _maybe_enrich: mostra aviso explícito quando LLM toggle está ativo mas raw_prs é None (demo/archive/CSV incompatível) — usuário sabia que LLM estava ativo mas não percebia que os dados simulados continuavam sendo usados - parse_csv_bytes: usa detect_schema + schema_adapter antes de apply_schema, resolvendo body sempre vazio em CSVs no formato github_export (description→body) - _load_local: passa CSVs locais pelo load_uploaded para popular raw_prs, habilitando enriquecimento LLM também em datasets locais compatíveis Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

… reducers Implementa testes matemáticos respeitando as regras puramente funcionais, garantindo idempotência de filtros e soma consistente para as linguagens.

…0% de cobertura

… (frederico-barcelos) feat(sprint4): pipeline_from_env, LLM real conectado e fixes de parsing

test(io): cobertura de edge cases e fluxo pipeline-exportação (Sprint 4)

test(transforms): testes de propriedade Hypothesis e cobertura 100% nos filtros (Sprint 4)

- pipeline_bridge.py: mescla imports — detect_schema/schema_adapter (dev4) com backend_enrich_prs/LLMClient (dev5); descarta cached_classify e classifiers individuais que não eram usados no corpo do arquivo - app.py: usa fluxo dev4 para _maybe_enrich (create_llm_client explícito, warning quando raw_prs=None, st.status com progress bar) adaptado à API do dev5 — enrich_prs(prs, client=client) sem parâmetro cache counter - sidebar.py: combina reset llm_enriched=False do dev4 com except Exception mais amplo e mensagem de erro com filename do dev5 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(ui): cache de enriquecimento, suporte mined-comments e validações de upload (Sprint 4)

frebarcelos and others added 30 commits May 1, 2026 22:00

feat(io): implementa leitura lazy de PRs

f4b1ac7

Closes #1. Closes #2. Closes #3. Closes #4.

feat(transforms): implementa filtros por estado e linguagem (TASK-05)

1638f12

feat(transforms): adiciona filtro por data (TASK-06)

084e572

feat(llm): TASK-08 e TASK-09 - client Agno + stubs classificadores

ae65512

feat(transforms): implementa filtros auxiliares e combinador (07)

789138b

[FIX] Updating pandas to pyproject.toml

338256a

[NEW] Dashboard initial version

2a8bd45

fix: omitting streamlit toolbar buttons

f49409b

refactor(config): remove hardcoded settings and externalize configura…

3350272

…tion

refactor(ui): modularize Streamlit app and separate concerns into com…

b12fc19

…ponents and utils

chore(deps): update streamlit version and add streamlit-extras depend…

8a3eead

…ency

build(setup): adiciona setup.ps1 para Windows sem make instalado

8f17a1e

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

chore(merge): integra branch bernardo (io/) na develop

906e691

chore(merge): integra branch pedro (transforms/) na develop

06356e3

chore(merge): integra branch dev/dev3 (llm/) na develop

a8783a2

chore(merge): integra branch frederico-barcelos (cache/ + pipeline/ +…

6df4b1a

… tooling) na develop

chore(merge): integra branch diogo (ui/) na develop

9728bd0

chore(merge): sincroniza develop na branch bernardo

bc98892

chore(merge): sincroniza develop na branch pedro

2ce1825

chore(merge): sincroniza develop na branch dev/dev3

c4fff55

chore(merge): sincroniza develop na branch diogo

87b6881

chore(merge): sincroniza develop na branch frederico-barcelos

9aaa12c

feat(claudinho): slash commands /claudinho e /diario para Claude Code

cc35dca

/claudinho <texto> — informe rapido no estilo Claudinho /diario — relatorio da sprint com contexto real do repo (sem API key separada) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

frebarcelos and others added 29 commits May 18, 2026 11:20

chore: sincroniza develop após sprint 3

828c196

chore(pedro): sincroniza develop após sprint 3

chore: sincroniza develop após sprint 3

09cb300

chore(diogo): sincroniza develop após sprint 3

test(io): cobre edge cases da leitura csv

bfa817b

test(io): valida fluxo leitura pipeline exportacao

46ee9f4

test(transforms): adiciona testes de propriedades com hypothesis para…

b05fa61

… reducers Implementa testes matemáticos respeitando as regras puramente funcionais, garantindo idempotência de filtros e soma consistente para as linguagens.

test(transforms): adiciona testes faltantes nos filtros garantindo 10…

8a84e6f

…0% de cobertura

build(docker): enable host ollama access from container

13ea7e1

feat(streamlit): support configurable large file uploads

8d4f6b7

fix(upload): reset streamlit file cursor before parsing

1021aec

feat(data): add oversized dataset validation

4f7ce1a

style(docker): remove inline compose profile comment

debbaca

docs(ollama): improve docker setup guidance in sidebar

4bfa498

style(code): clean comments and normalize docstrings

f697c36

refactor(llm): delegate enrichment to backend implementation

c5cbab8

feat(data): support mined-comments uploads and PR reconstruction

766a5a6

feat(ui): cache enrichment state and rebuild PR records

59363d9

ci(pre-commit): migrate hooks to push stage

7c41d9b

style(lint): apply ruff auto-fixes

4ad9c85

style(mypy): remove unnecessary ignore comments

2846ba0

chore(develop): integra sprint 4 do módulo pipeline/ + llm fixes + ui…

492af9a

… (frederico-barcelos) feat(sprint4): pipeline_from_env, LLM real conectado e fixes de parsing

chore(develop): integra sprint 4 do módulo io/ (bernardo)

708dbb8

test(io): cobertura de edge cases e fluxo pipeline-exportação (Sprint 4)

chore(develop): integra sprint 4 do módulo transforms/ (pedro)

dcb6d58

test(transforms): testes de propriedade Hypothesis e cobertura 100% nos filtros (Sprint 4)

chore(develop): integra sprint 4 do módulo ui/ (diogo)

5b20dea

feat(ui): cache de enriquecimento, suporte mined-comments e validações de upload (Sprint 4)

frebarcelos merged commit 210a459 into main May 25, 2026
1 of 2 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(sprint4): entrega final — UI + LLM integrado + pipeline configurável (≥80% cobertura)#87

feat(sprint4): entrega final — UI + LLM integrado + pipeline configurável (≥80% cobertura)#87
frebarcelos merged 124 commits into
mainfrom
develop

frebarcelos commented May 25, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

frebarcelos commented May 25, 2026

Resumo

Sprint 4 — o que entrou

Métricas finais

Plano de teste

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants