feat(platform): mark incomplete PR data on compare and dashboard

[cpenaforte-clickbus]

Contexto e Motivação

O badge de dados de PR incompletos (#282) só aparece na página do repositório. Na tabela de comparação e no dashboard da organização, um run cuja leitura de PRs degradou aparece igual a um completo — e o dashboard agrega PR de todos os repositórios, então um só degradado distorce os números da organização sem aviso.

Há também uma dependência de ordem: com o passo reviews degradado, o engine vai deixar de emitir single_pass_rate e median_review_rounds nos grupos de acceptance_by_* (#284), e org-summary.ts e tool-comparison.ts multiplicam esses valores por peso — campo ausente viraria NaN% no dashboard.

Refs #236

Mudanças

• Primeiro commit — tolerância: single_pass_rate e median_review_rounds passam a opcionais em AcceptanceMetrics, e org-summary.ts e tool-comparison.ts só somam um grupo quando o valor é número. É o que precisa estar em produção antes do fix(engine): omit review-derived metrics when the reviews read fails #284.
• Comparação: cada repositório cujo último run degradou ganha um marcador âmbar, com texto para leitor de tela, e a tabela ganha uma legenda visível quando há algum marcado. O dado vem de payload->pr_enrichment_degraded na view repo_metric_summaries (migração 023), sem trazer o payload inteiro. Em banco sem essa coluna, a consulta repete sem ela e a tabela segue preenchida, sem marcador — o mesmo fallback que org-summary.ts já usa.
• Dashboard: uma nota acima dos painéis de PR lista os repositórios cujo run mais recente tem dado de PR incompleto, a partir dos payloads que o dashboard já carrega. Some quando nenhum degradou.
• Strings em en-US e pt-BR; o es-ES cai no en-US.

Plano de Teste

• npx vitest run tests/pr-degraded-repos.test.ts tests/compare-pr-degraded.test.ts tests/org-summary.test.ts tests/tool-comparison.test.ts — a lista de repositórios degradados a partir dos payloads; o mapeamento da comparação com campo válido, ausente e lixo, inclusive o fallback sem payload; e as agregações com um grupo sem single_pass_rate, sem NaN e iguais às de quem não tem o grupo — rodado em 237e3b5: 29 passed
• npm run lint, npx tsc --noEmit, npm run test:coverage, npm run build — os passos do job platform do CI — rodado em 237e3b5: lint 0 erros, tsc ok, test:coverage 417 passed, build ok
• Manual: com um repositório degradado, ver o marcador e a legenda na comparação e a nota no dashboard; sem nenhum, nada aparece — verificado de ponta a ponta no topo da pilha (fix(platform): make the PR-data marker readable on touch and keyboard #293, em 4911e6d, com o rótulo corrigido conferido depois em 8e83ca4), que contém este PR: plataforma rodando sobre as migrações reais e um PostgREST, com payloads do engine em sete cenários de falha, no Playwright em desktop, celular (390 px), só teclado, leitor de tela, tema escuro, pt-BR, en-US e es-ES; marcador, legenda e nota do dashboard presentes nos 7 repositórios degradados e ausentes no limpo

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

Impacto e Risco

• Áreas afetadas: a consulta e a tela de comparação, o dashboard da organização e as agregações de single-pass por origem e por ferramenta.
• Compatibilidade: nada muda quando o campo está ausente. Payloads sem as chaves de review nos grupos de acceptance deixam de gerar NaN.
• Performance: uma chave a mais lida da view; em banco sem a migração 023, uma segunda consulta, como no fallback existente.
• Segurança: nenhuma rota nova, nenhuma migração; a leitura do campo passa por prDegradedSteps, que ignora o que não reconhece.
• Acessibilidade: a nota do dashboard é texto visível com role="note"; o marcador da comparação tem texto para leitor de tela e uma legenda visível. Limitação conhecida: a lista em cards da comparação, no mobile, não mostra o marcador; ali o aviso fica no badge da página do repositório.
• Custo AWS: N/A.

Sobre o tamanho (RFC 0028 §2.4): 424 linhas, faixa 251–500. A tolerância e as marcas formam a parte de plataforma da mesma mudança de engine; separar a tolerância deixaria a comparação e o dashboard sem o aviso que explica os números ausentes. Este PR já é a divisão do PR de plataforma original, que passava de 500.

Plano de Rollback

git revert do merge. Se o #284 já estiver em produção, reverter este PR traz de volta o NaN% para repositórios com reviews degradado — reverta os dois juntos.

Referências

• Issue: [DEBT] PR enrichment degrades silently when the GraphQL step fails; the report carries no signal #236
• Depends on feat(platform): flag repo runs whose PR data is incomplete #282 — PR empilhado: a base é a branch dele
• Blocks fix(engine): omit review-derived metrics when the reviews read fails #284 — o primeiro commit precisa estar em produção antes dele
• Blocks fix(platform): make the PR-data query opt-in and localize its steps #292 — correções da varredura pré-merge da plataforma
• RFC 0025, RFC 0028, RFC 0029 PM-4

Autoria Assistida

• Agente: Claude Code — Claude Sonnet 5.5 nos commits e nas revisões por task; Claude Opus 5.5 na orquestração e na revisão final do conjunto
• Escopo da assistência: desenho e implementação guiada por testes; a decisão de omitir as métricas de review 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:24pm UTC