[diegosouzapw]

Problema (#2463)

NVIDIA NIM — e qualquer provedor — falhava com s.startsWith is not a function ("all the time" desde a 3.8.x). Rodando contra a fonte (stack não-minificado) com uma chave nvapi- real, o erro apareceu literalmente como errCode.startsWith is not a function.

Root cause

open-sse/utils/proxyFetch.ts protegia error.code apenas contra undefined antes de .startsWith("UND_ERR"):

(errCode !== undefined && errCode.startsWith("UND_ERR"))

Mas erros do dispatcher undici / sistema podem ter code numérico (errno). Aí .startsWith lança TypeError. Pior: o crash acontecia dentro do catch que deveria cair no fallback para fetch nativo — então uma falha recuperável do dispatcher virava fatal, e o NVIDIA quebrava sempre (o fetch nativo direto funciona, como comprovado isoladamente).

Mudanças

Arquivo	Mudança
open-sse/utils/proxyFetch.ts	O bug: typeof errCode === "string" antes do .startsWith.
open-sse/services/model.ts	parseModel rejeita não-string truthy (objeto/número) — evita cleanStr.endsWith crashar (mesma classe do #2359).
open-sse/handlers/chatCore.ts	Só faz strip de prefixo do model quando é string (defense-in-depth; a API já valida model via Zod).
open-sse/services/combo.ts	Coerção-por-guard do error.code/type do JSON upstream em vez de as string — a lógica de resiliência não pode crashar com code numérico.

Testes (red → green)

• tests/unit/proxyfetch-undici-retry.test.ts — erro com code numérico deve cair no fallback nativo, não crashar (RED reproduziu proxyFetch.ts:320 — errCode.startsWith is not a function).
• tests/unit/parse-model-defensive.test.ts — parseModel retorna resultado nulo (sem throw) para objeto/número/array.

Verificação

• Reproduzido contra o endpoint real da NVIDIA com chave nvapi- válida: antes do fix a validação retornava errCode.startsWith is not a function; depois, resultado limpo e sanitizado.
• npm run typecheck:core ✅ · eslint nos arquivos tocados ✅ · 34/34 testes relacionados ✅ · pre-push test:unit ✅

Auditoria da classe de bug

Auditei src/ e open-sse/ por chamadas de método de string sobre valores guardados só contra undefined/falsy. O único crash ativo era o proxyFetch:320 (camada de fetch compartilhada → afetava todos os provedores). As demais ocorrências eram latentes/não-alcançáveis pela API e foram endurecidas mesmo assim.

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

[gemini-code-assist]

Code Review

This pull request introduces defensive type-checking guards across several files (chatCore.ts, combo.ts, model.ts, and proxyFetch.ts) to prevent runtime crashes when handling unexpected non-string inputs, such as numeric error codes or object-type model strings. It also adds corresponding unit tests to verify these fixes. The reviewer suggests converting numeric or non-string error codes and types to strings using String() instead of discarding them as undefined in combo.ts, ensuring that valuable debugging information is preserved.

[gemini-code-assist]

Ao receber um erro do upstream com um code ou type numérico (por exemplo, { "code": 40001 }), a verificação estrita de tipo typeof ... === "string" descarta o valor definindo-o como undefined. Isso resulta na perda de informações úteis que poderiam ser usadas para classificação de erros ou depuração. Em vez de descartar valores não-string, é mais resiliente e informativo convertê-los para string usando String() caso existam e não sejam nulos.

Suggested change

	code:
	typeof (rawError as Record<string, unknown>).code === "string"
	? ((rawError as Record<string, unknown>).code as string)
	: undefined,
	type:
	typeof (rawError as Record<string, unknown>).type === "string"
	? ((rawError as Record<string, unknown>).type as string)
	: undefined,
	code:
	(rawError as Record<string, unknown>).code !== undefined && (rawError as Record<string, unknown>).code !== null
	? String((rawError as Record<string, unknown>).code)
	: undefined,
	type:
	(rawError as Record<string, unknown>).type !== undefined && (rawError as Record<string, unknown>).type !== null
	? String((rawError as Record<string, unknown>).type)
	: undefined,

[gemini-code-assist]

Ao receber um erro do upstream com um code ou type numérico (por exemplo, { "code": 40001 }), a verificação estrita de tipo typeof ... === "string" descarta o valor definindo-o como undefined. Isso resulta na perda de informações úteis que poderiam ser usadas para classificação de erros ou depuração. Em vez de descartar valores não-string, é mais resiliente e informativo convertê-los para string usando String() caso existam e não sejam nulos.

Suggested change

	code:
	typeof (rawError as Record<string, unknown>).code === "string"
	? ((rawError as Record<string, unknown>).code as string)
	: undefined,
	type:
	typeof (rawError as Record<string, unknown>).type === "string"
	? ((rawError as Record<string, unknown>).type as string)
	: undefined,
	code:
	(rawError as Record<string, unknown>).code !== undefined && (rawError as Record<string, unknown>).code !== null
	? String((rawError as Record<string, unknown>).code)
	: undefined,
	type:
	(rawError as Record<string, unknown>).type !== undefined && (rawError as Record<string, unknown>).type !== null
	? String((rawError as Record<string, unknown>).type)
	: undefined,

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (6 files)

• open-sse/handlers/chatCore.ts
• open-sse/services/combo.ts
• open-sse/services/model.ts
• open-sse/utils/proxyFetch.ts
• scripts/ad-hoc/nvidia-startswith-diag.ts (new)
• tests/unit/parse-model-defensive.test.ts (new)
• tests/unit/proxyfetch-undici-retry.test.ts (new)

---

_{Reviewed by nemotron-3-super-120b-a12b-20230311:free · 398,954 tokens}