fix(sse): pin the ok variant of the non-streaming leg result in chatCore

[diegosouzapw]

Base-red vivo

API Route Typecheck falha no tip de release/v3.8.51:

apiTypecheckErrors=302
[api-typecheck] FAIL — 1 new/regressed TypeScript error(s) under src/app/api/ not covered by the frozen baseline:
  ✗ open-sse/handlers/chatCore.ts TS2339 (baseline 0, live 13)

Introduzido pelo #12867, que eu mergeei. Erro meu: validei a leva com typecheck:core e dei por coberto. O tsconfig.typecheck-core.json não puxa o chatCore.ts; o tsconfig.typecheck-api.json puxa, através da rota. São dois escopos diferentes e eu só rodei um.

Causa

legResult é declarado como a união NonStreamingProviderLegResult inteira. O guard kind === "error" com return antecipado estreita para a variante ok, mas a reatribuição condicional logo abaixo devolve o tipo declarado — e as 13 leituras de campo seguintes (upstreamResponse, headers, providerRequest, requestUrl, providerBody, response, …) perdem a narrowing.

Correção

loopApply.leg já é tipado NonStreamingProviderLegResult & { kind: "ok" }, então basta fixar a variante num binding próprio. Sem cast, sem as, sem any.

let okLeg: NonStreamingProviderLegResult & { kind: "ok" } = legResult;
if (loopApply.kind === "ok") {
  toolLoopRan = true;
  toolLoopUsage = loopApply.usage;
  okLeg = loopApply.leg;
}

Também caiu o import morto de ChatCoreErrorResult (o no-unused-vars que o pre-commit acusava) e a entrada de supressão do chatCore.ts no eslint-suppressions.json, que declarava 26 violações quando restava 1 — resíduo do split do #12867. Entrada obsoleta suprime defeito futuro, então saiu.

Dois commits, de propósito

O chatCore.ts chegou ao tip fora do padrão do Prettier: 4523 linhas divergem da saída do formatador. Qualquer PR que toque o arquivo carrega essa reformatação, porque o lint-staged formata no commit. Separei:

1. style(sse) — Prettier puro sobre o arquivo do tip. Verifiquei que o resultado é byte-idêntico a prettier(tip) com a config do projeto: nenhuma edição minha nesse commit.
2. fix(sse) — a mudança semântica, 39 linhas.

Revise o segundo; o primeiro é mecânico.

Evidência

gate	antes	depois
check-api-typecheck	FAIL — 302 erros, 13 novos	OK — 289, todos na baseline
typecheck:core	limpo	limpo
ESLint em chatCore.ts	1 erro	0

Achado separado

O shard 1/4 de unit tests no tip puro tem 9 falhas; 5 são de tempo sob carga, 4 não são: uma chave Google não estava sendo redigida em corpos de erro. Tratado à parte em fix/release-v3.8.51-google-key-redaction.

[ggiak]

Verified locally on the current tip — the gate goes green with this PR.

• PR head 09cb0766f8 merged clean onto release/v3.8.51 @ fcc2dcd1a6 (docs(changelog): reconcile the v3.8.51 living section — fold 366 fragments, cover every cycle commit, credit every contributor #12971 reconcile tip)
• node scripts/check/check-api-typecheck.mjs → exit 0: [api-typecheck] OK — 289 pre-existing error(s), all within frozen baseline.
• Same gate on the bare tip (no PR) reproduces CI exactly: ✗ open-sse/handlers/chatCore.ts TS2339 (baseline 0, live 13), exit 1

Node v26.4.0. This is the red every open PR against the branch currently inherits under API Route Typecheck (e.g. #13007, #12987, #13036), so it clears all of them at once.

[ggiak]

LGTM