OmniRoute/changelog.d/maintenance/combo-harness-host-classification.md at feat/plugin-browser-pool

mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-07-26 09:52:11 +03:00

Files

Diego Rodrigues de Sa e Souza 25ef8d05c0 test(combo): classify dispatches by host and close the auto test pool

providerFromUrl() classified the upstream by URL path shape. With hundreds of
providers in the catalog that is wrong in both directions: '/chat/completions'
matched every OpenAI-compatible upstream (opencode.ai/zen was being reported as
'openai'), while OpenAI's own GPT-5.6 dispatches go to '/v1/responses' and came
back 'unknown'. Classify by hostname instead, and return 'host:<hostname>' for
unrecognised hosts so an unexpected dispatch names itself in the failure message.

combo-matrix/auto.test.ts additionally assumed the auto pool contained only the
connections it seeded, but no-auth providers (#6557/#7622) legitimately join the
pool without a seeded connection. Block those in beforeEach so the pool is closed
to the seeded connections — preserving what the assertions are actually about
(LKGP pinning, variant pool resolution) rather than relaxing them to accept
whatever an open pool picks.

Verified: auto.test.ts 2/2, and all 9 combo-matrix files 27/27 with no
regressions (no test was passing on the old helper's false 'openai' label).

2026-07-19 16:10:12 -03:00

397 B

Raw Permalink Blame History

Classify upstream dispatches by host instead of URL path shape in the combo-routing integration harness, and close the auto test pool to its seeded connections. The path heuristic mislabelled every OpenAI-compatible upstream as openai and stopped recognising OpenAI's own /v1/responses calls, which made combo-matrix/auto.test.ts fail on a release tip whose routing was in fact correct.

397 B Raw Permalink Blame History

397 B

Raw Permalink Blame History