refactor(chatCore): extrai resolução de endpoint/formato para leaf puro (#3501)

[diegosouzapw]

O quê

Próximo incremento da decomposição do god-file chatCore.ts (QG v2 Fase 9 T5, #3501). Move o bloco de resolução de endpoint/formato do topo de handleChatCore para o novo leaf puro open-sse/handlers/chatCore/requestFormat.ts (resolveChatCoreRequestFormat), ao lado de resolveChatCoreRequestSetup (fase de request-setup).

Stacked sobre o #4492 (4º da cadeia #4477 → #4491 → #4492 → este). Base aponta para refactor/qg-chatcore-codexquota; o GitHub re-aponta a cadeia automaticamente conforme os pais mergeiam.

Como (preservação de comportamento)

• Os 7 derivados (endpointPath / sourceFormat / isResponsesEndpoint / nativeCodexPassthrough / isDroidCLI / copilotCompatibleReasoning / clientResponseFormat) movem byte-idênticos para o leaf puro; o handler desestrutura o resultado, então todo uso downstream fica inalterado.
• A função local isCopilotClient (usada só por esse bloco) vira helper privado do leaf.
• Import órfão detectFormatFromEndpoint migra para o leaf (getTargetFormat permanece no import de provider.ts). shouldUseNativeCodexPassthrough e FORMATS seguem no handler (re-exportado / usado em 26 pontos).
• Tipo via ReturnType<typeof resolveChatCoreRequestFormat> (sem anotação manual divergente).

Gates

• file-size: chatCore.ts 5019→4993 (shrink −26, cruza abaixo de 5000); baseline ratchetado.
• complexity: 1905 = baseline 1905 (neutro).
• typecheck:core: exit 0.
• Testes: tests/unit/chatcore-request-format.test.ts — 7 casos (incl. o downgrade OpenAI-Responses → OpenAI quando a forma Responses chega por endpoint não-/responses e não-Droid, e a supressão desse downgrade pelo Droid CLI; delegação do nativeCodexPassthrough; detecção Copilot por header/UA). chatcore-imports-cleanly ✓ + 207/207 na suíte chatcore + sse-auth (excluído só o translation-paths flaky pré-existente).

Extração de god-file sem mudança de comportamento em runtime.

[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 refactors open-sse/handlers/chatCore.ts by extracting the endpoint and format resolution logic into a new module open-sse/handlers/chatCore/requestFormat.ts and adding corresponding unit tests. Feedback is provided to improve type safety by replacing an unsafe type assertion with the standard headers.entries() method when iterating over headers.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

Instead of using an unsafe type assertion (as unknown as Iterable<[string, string]>), you can use the standard headers.entries() method which returns an IterableIterator<[string, string]>. This is fully type-safe and cleaner.

  if (headers instanceof Headers) {
    for (const [key, value] of headers.entries()) {
      if (isMatch(key) || isMatch(value)) return true;
    }
  }