pjandro/OmniRoute
1
0
Fork 0
mirror of https://github.com/diegosouzapw/OmniRoute.git synced 2026-08-07 15:52:52 +03:00
Code Issues Packages Projects Releases Wiki Activity

fix(open-sse): route GitHub Copilot gpt-5.6 sol/terra/luna to /responses (#9050)

Browse Source 
Validated in local merge-train (devbox-vm-06-dev002) @ combined-tip (FAST gates green: static + changed tests + vitest — only pre-existing audit.test.ts flake). Evidence: /home/diegosouzapw/dev/proxys/OmniRoute/.claude/worktrees/merge-train-20260805-213228-suite.log
This commit is contained in:
GreatLiu
2026-08-06 08:43:08 +08:00
committed by GitHub
parent 5f3f25e541
commit ea4bbdf7c0
2 changed files with 33 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
tests/unit/executor-github.test.ts
View File

@@ -98,6 +98,21 @@ test("GithubExecutor.buildUrl routes unlisted Codex models to /responses (9route
);
});
test("GithubExecutor.buildUrl routes gpt-5.6-sol/terra/luna to /responses (regression)", () => {
// These models were registered in the `gh` registry without targetFormat, so
// getModelTargetFormat returned null and requests fell through to
// /chat/completions -> upstream 400 "model is not accessible via the
// /chat/completions endpoint". They only support /responses upstream.
const executor = new GithubExecutor();
for (const model of ["gpt-5.6-sol", "gpt-5.6-terra", "gpt-5.6-luna"]) {
assert.equal(
executor.buildUrl(model, true),
"https://api.githubcopilot.com/responses",
`${model} must route to /responses`
);
}
});
test("GithubExecutor.transformRequest injects JSON response instructions for Claude and strips reasoning fields", () => {
const executor = new GithubExecutor();
const body = {