fix(combo): ignore benign empty error fields in streaming quality validation (502 false-positive on opencode tool calls)

[diegosouzapw]

Causa-raiz

A qualidade de streaming (validateResponseQuality → isStreamingUpstreamError em open-sse/services/combo/validateQuality.ts) usava uma checagem de presença de chave em vez de conteúdo:

if (parsed.error != null) return true;

parsed.error != null é true para {}, "", false, 0 — qualquer valor exceto null/undefined. Quando um cliente como o opencode faz uma chamada de tool call, o upstream SSE abre com frames de role-only (sem conteúdo reconhecido) e um chunk posterior que carrega tool_calls reais mais um campo error vazio benigno (que alguns backends emitem em todo chunk). O gate de erro roda ANTES dos reconhecedores de conteúdo, então esse único frame vira "error" → 502 "streaming upstream error".

O mesmo combo via kilocode funciona porque o wire format do kilocode não emite o campo error vazio → o gate nunca dispara. Por isso o bug parece tool-call-specific e client-specific.

Fix

Novo helper isSubstantiveError(): só trata error como falha real quando tem conteúdo substantivo:

• String não-vazia → erro
• Objeto com chaves (ex.: {message, code}) → erro
• true explícito → erro
• {}, "", false, 0 → benigno (não erro)

Aplicado ao gate de top-level (parsed.error). O caso nested (parsed.response.status === "failed") permanece intacto por já ser guardado pelo status.

TDD

• Teste novo: tests/unit/quality-validation-benign-error.test.ts — 3 casos:
  1. tool_calls + error:{} → valid (antes: inválido/502)
  2. tool_calls + error:"" → valid (antes: inválido/502)
  3. erro real {message, code} → invalid (garante que erro verdadeiro ainda falha)
• Suíte existente: 19/19 verdes (streaming-empty-content-block, combo-quality-validator-reasoning, combo-responses-sse-failure-fallback)
• combo-routing-engine + typecheck em validação

Escopo

Toca apenas open-sse/services/combo/validateQuality.ts (1 helper novo + 1 linha) + 1 teste novo.

⚠️ base-red inherited: #9737

[diegosouzapw]

Thanks for the contribution. PR looks structurally viable from diff-level metadata. Please confirm gates with /merge-prs flow before merge.

[diegosouzapw]

Thanks for the contribution. PR looks structurally viable from diff-level metadata. Please confirm gates with /merge-prs flow before merge.

[diegosouzapw]

Thanks for the contribution. PR looks structurally viable from diff-level metadata. Please confirm gates with /merge-prs flow before merge.