fix(sse): register quota batch after imports resolve

[wedreamer]

Summary

• Cold POST /v1/chat/completions, /v1/messages, and /v1/responses return an empty HTTP 500 after process start. The console error is TypeError: (0 , e.Zd) is not a function.
• e.Zd is registerQuotaFetcher in quotaPreflight.ts, a real Map.set wrapper, not a missing export. Webpack emits that module as an async module. quotaTrackersBatch.ts called registerQuotaTrackersBatch() at module scope, so the first callee (registerAgentrouterQuotaFetcher) touched the live binding before it was a function. Next caches the rejected chat-route module, so every later chat request 500s. Restarting the same image does not help. GET /v1/models does not load this barrel and stays up.
• Remove the module-load side effect. Call registerQuotaTrackersBatch() from the chat.ts module body (same point as the sibling registrars that already survive init) and from instrumentation-node.ts after await import. chat.ts is frozen at 2586 lines, so the new call replaces a blank line.

Related Issues

• Closes fix(backend): Cold chat route returns empty HTTP 500: (0 , e.Zd) is not a function #15547
• Related to fix(domain): qwen-cloud-token-plan parked as "all accounts exhausted" after a credit-pack top-up #14359 (sibling ESM init cycle around quotaPreflight; that one deadlocks the esbuild MCP bundle, this one rejects the Next chat chunk)

Validation

• Change type: routing
• Focused tests and category gates from the golden path
• npm run lint — not run in this sparse checkout (no node_modules). CI lint is the gate.
• Reconciled with the current active release base (release/v3.8.52 @ 23a11484); focused check rerun afterward
• Production-code changes include a new or updated automated test in this PR
• SonarQube is temporarily opt-in while the private project has no quota; it is not a PR gate.

Tests Added Or Updated

• tests/unit/quota-trackers-batch-init.test.ts (added)

Local run, from a sparse checkout of this branch:

node --experimental-strip-types --test tests/unit/quota-trackers-batch-init.test.ts
# tests 3, pass 3, fail 0

The existing tests/unit/instrumentation-import-graph-12074.test.ts case "startup still registers bespoke, batch, and generic quota fetchers" still calls registerQuotaFetchers() and asserts getQuotaFetcher("agentrouter") is a function. That path now goes through the explicit call.

Coverage Notes

• open-sse/services/quotaTrackersBatch.ts, src/sse/handlers/chat.ts, and src/instrumentation-node.ts are covered by the new source-contract test: no module-scope call, chat body calls the registrar before registerGenericQuotaFetchers(), instrumentation calls it only after the dynamic import resolves.
• No coverage drop expected. The production change deletes a side effect and moves the same function call to the two callers that already owned registration.

Reviewer Notes

• Reproduced on published image diegosouzapw/omniroute:latest sha256:8bd462c9f60d8eda79329cfbb6ea7ea723505fe7721beb944f3d43835409e218 (package 3.8.51, image created 2026-09-24). The side effect is unchanged on release/v3.8.52.
• Throw site in that image: function k(){(0,e.Zd)("agentrouter",j),(0,f.registerMonitorFetcher)("agentrouter",j)} while chunk 80010.js module 502008 is still inside webpack's async dependency resolve. No source maps in the image.
• chat.ts line count stays 2586, equal to config/quality/file-size-baseline.json.
• Registration order is unchanged relative to registerGenericQuotaFetchers(), so bespoke batch fetchers still win over the generic registrar. Both call sites are idempotent (Map.set).

[wedreamer]

CI for this fork PR finished as action_required with zero jobs — GitHub is waiting on a maintainer to approve workflows from a first-time contributor. I only have pull access, so I cannot approve them.

Semgrep cloud scan already passed. chat.ts stays at the frozen 2586-line baseline. Focused test run locally: node --experimental-strip-types --test tests/unit/quota-trackers-batch-init.test.ts (3/3).

Once the workflows are approved I will watch the gates and fix anything this change breaks.

[wedreamer]

@diegosouzapw could you review this when you have a moment?

This is a production outage, not a cosmetic fix: on diegosouzapw/omniroute:latest (package 3.8.51, image sha256:8bd462c9f60d), a cold start makes every /v1/chat/completions, /v1/messages and /v1/responses return an empty HTTP 500. The throw is TypeError: (0 , e.Zd) is not a function while registerQuotaTrackersBatch() runs at module scope; e.Zd is registerQuotaFetcher, which is still an unresolved webpack async live binding. /v1/models is unaffected, and restarting the same image does not clear it because webpack caches the rejected module.

I am a first-time contributor, so the required workflows (CI, Quality Gates, API Route Typecheck, semgrep) are stuck at action_required with zero jobs. Mergify also will not queue this until you apply the queue label.

Could you approve those workflow runs and, if the fix looks right, queue the PR? Happy to adjust the patch if you want a different init boundary.