fix(engine): stop fabricating acceptance and funnel on degraded reads

[cpenaforte-clickbus]

Contexto e Motivação

Uma varredura antes do merge da série (#280 a #285), com o engine rodando sobre dados reais do GitHub e falhas simuladas em cada passo da leitura de PRs, mostrou que ainda saíam números inventados quando o enriquecimento falhava. Com a passada de GraphQL derrubada, a acceptance do repositório afirmava 0 commits em PR e 0% de PR rate, contra 30 commits e 33% no run limpo. E, uma vez sem acceptance, o funil de origem caía no valor padrão e mostrava 100% "em PR". Valores nulos no GraphQL ainda derrubavam a leitura inteira para fetch, e o gh api -F convertia tipos e lia arquivo a partir de valores das variáveis.

Resolver agora porque são exatamente o tipo de número que a #236 existe para não publicar, e a série ainda não entrou na main.

Refs #236

Mudanças

• Com o enriquecimento degradado, acceptance_by_origin e acceptance_by_tool são omitidos (casar commit com PR depende das commit refs), e também os campos de PR aberto parado, cuja última atividade usa os pushes de commit.
• O funil de origem sai inteiro sempre que a leitura degradou e a acceptance está ausente, em vez de supor que todo commit entrou por PR. Ausência por desenho — sem gh ou sem remote — mantém o comportamento de antes.
• O parser do enriquecimento trata repository, pullRequests ou nodes nulos como página incompleta e ignora nós e commits nulos, em vez de levantar exceção.
• Todas as variáveis de texto do GraphQL vão com gh api -f: com -F, um repositório de nome só numérico virava número e quebrava o enriquecimento em todo run, e um valor começando com @ era lido como caminho de arquivo.
• A linha de console diz quando a leitura veio incompleta, distingue falha da listagem de falha de passo secundário e conta PRs de todos os estados.
• Um helper único monta os grupos de acceptance, no lugar de dois blocos duplicados (commit de refatoração separado, sem mudança de comportamento).

Plano de Teste

• pytest tests/test_pr_fetch_degradation.py -v — acceptance, staleness e funil ausentes com o enriquecimento degradado e presentes no run limpo; funil ausente com basic e fetch; repository, pullRequests, nodes, nó e commit nulos sem exceção; -f para owner, name e cursor, inclusive nome 2048, true e null; as linhas de console — rodado em 0e0bf9d: 54 passed
• pytest tests/ -q — a suíte inteira, com o portão de determinismo da [FEAT] Comparador de paridade do metrics.json, campo a campo #214 — rodado em 0e0bf9d: local: 408 passed; a única falha é a test_status_reflects_state ([BUG] test_agent_settings_hook lê o spool real — falha em máquina com registros pendentes #146), que só ocorre fora do CI, e o test_hook_binders.py ficou de fora por depender de rede; no CI o comando literal passa

A verificação roda fora da máquina de quem construiu: o job cli do CI (RFC 0029, PM-4).

Impacto e Risco

• Áreas afetadas: agregador, funil de origem, leitor de PRs do GitHub, console e docs/METRICS.md.
• Compatibilidade: um run sem degradação gera o mesmo metrics.json e o mesmo relatório, byte a byte. Runs com o enriquecimento degradado deixam de trazer acceptance, staleness e funil.
• Performance: nenhuma mudança.
• Segurança: endurece a chamada ao gh: nenhum valor vindo da resposta do GitHub pode mais ser interpretado como caminho de arquivo.
• Acessibilidade e custo AWS: N/A — sem UI e sem infraestrutura.

Sobre o tamanho (RFC 0028 §2.4): 482 linhas, faixa 251–500. São correções da mesma varredura, cada uma num commit próprio; a parte de degradação por estado foi separada no PR seguinte justamente para caber.

Plano de Rollback

git revert do merge. Volta o comportamento da #285; nada a migrar.

Referências

• Issue: [DEBT] PR enrichment degrades silently when the GraphQL step fails; the report carries no signal #236
• Depends on feat(engine): flag repos with a degraded PR read in org outputs #285 — PR empilhado: a base é a branch dele
• Blocks fix(engine): gate degradation per PR state and retry transient errors #290
• Follow-up do teto de 500 PRs: [DEBT] Enriquecimento e reviews param em 500 PRs; os demais ficam sem dados e sem sinal #288
• RFC 0028, RFC 0029 PM-4

Autoria Assistida

• Agente: Claude Code — Claude Sonnet 5.5 e Claude Opus 5.5 nos commits; Claude Sonnet 5.5 nas revisões por task; Claude Opus 5.5 na varredura, na orquestração e nas revisões finais
• Escopo da assistência: varredura de bugs e validação com dados reais, implementação guiada por testes; a decisão de omitir em vez de estimar foi do autor
• Revisão humana: pendente — o autor revisa antes do merge

🤖 Generated with Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
clickbus-iris	Ready	Preview	Oct 8, 2026 1:44pm UTC