refactor(chatCore): extrai núcleo puro de persistCodexQuotaState (#3501)

[diegosouzapw]

O quê

Próximo incremento da decomposição do god-file chatCore.ts (QG v2 Fase 9 T5, #3501). Extrai o núcleo puro de persistCodexQuotaState para o novo leaf open-sse/handlers/chatCore/codexQuota.ts (buildCodexQuotaPersistence).

Stacked sobre o #4491 (que está stacked sobre o #4477). Base aponta para refactor/qg-chatcore-keyhealth; o GitHub re-aponta a cadeia automaticamente conforme os pais mergeiam.

Como (preservação de comportamento)

• Extração parcial pura: buildCodexQuotaPersistence parseia os headers de quota → snapshot codexQuotaState, faz passthrough do providerSpecificData existente e, num 429 cuja janela (dual-window) passou do threshold de exaustão, registra codexScopeRateLimitedUntil[scope] + codexExhaustedWindow, retornando a mensagem de debug-log (em vez de emiti-la).
• O handler mantém as partes impuras byte-idênticas e na mesma ordem: emite o exhaustionLog retornado → invalidateCodexQuotaCache em todo 429 (connectionId é garantido pelo early-return) → updateProviderConnection → mutação de credentials.providerSpecificData.
• Imports órfãos (parseCodexQuotaHeaders / getCodexModelScope / getCodexDualWindowCooldownMs) migram para o leaf; isCompactResponsesEndpoint permanece (usado no site de passthrough).

Gates

• file-size: chatCore.ts 5055→5019 (shrink −36); baseline ratchetado.
• complexity: 1905 = baseline 1905 (neutro).
• typecheck:core: exit 0.
• Testes: tests/unit/chatcore-codex-quota.test.ts — 5 casos (sem-headers→null, snapshot+passthrough do existing data, 429 com cooldown+janela+log, merge do codexScopeRateLimitedUntil preservando outros escopos, 429 abaixo do threshold = no-op). chatcore-imports-cleanly ✓ + integração codex (executor-codex + sse-auth) 107/107 + 139/139 na suíte chatcore (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 pure logic of persistCodexQuotaState into a new helper function buildCodexQuotaPersistence in open-sse/handlers/chatCore/codexQuota.ts, accompanied by unit tests in tests/unit/chatcore-codex-quota.test.ts. The review feedback highlights two key areas for improvement: first, protecting against a potential TypeError when assigning credentials.providerSpecificData if credentials is undefined; second, enhancing the purity and determinism of buildCodexQuotaPersistence by deriving the absolute reset time directly from the quota headers instead of recalculating it using Date.now() and cooldownMs.

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]

Se credentials for nulo ou indefinido, a atribuição direta a credentials.providerSpecificData lançará um TypeError e quebrará o handler da requisição. Como credentials é acessado defensivamente com encadeamento opcional (optional chaining) na linha 771, devemos também proteger esta atribuição para evitar possíveis falhas em tempo de execução.

      if (credentials) {
        credentials.providerSpecificData = built.nextProviderData;
      }

[gemini-code-assist]

Reconstruir scopeUntil adicionando Date.now() a cooldownMs introduz um pequeno desvio de relógio (clock drift) e não-determinismo, pois cooldownMs foi originalmente computado como resetTime - Date.now(). Como buildCodexQuotaPersistence deve ser uma função pura, podemos determinar o tempo de reset absoluto diretamente de quota.resetAt7d ou quota.resetAt5h com base em exhaustedWindow, recorrendo ao cálculo relativo apenas se necessário.

Suggested change

	const scopeUntil = new Date(Date.now() + cooldownMs).toISOString();
	const resetAt = exhaustedWindow === "7d" ? quota.resetAt7d : quota.resetAt5h;
	const scopeUntil = resetAt ? new Date(resetAt).toISOString() : new Date(Date.now() + cooldownMs).toISOString();