mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-07-26 09:52:11 +03:00
888c872459
* fix(antigravity): preserve protocol fidelity and fail closed * chore: add PR-numbered changelog fragment * test: split oversized Antigravity suites * refactor(antigravity): align official IDE and CLI identities * fix(antigravity): align catalog with callable models * test(antigravity): update 2 test files to renamed version-cache API (#8013 fix) Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com> --------- Co-authored-by: nguyenha935 <208228297+nguyenha935@users.noreply.github.com> Co-authored-by: backryun <backryun@users.noreply.github.com> Co-authored-by: diegosouzapw <8016841+diegosouzapw@users.noreply.github.com> Co-authored-by: nguyenha935 <nguyenha935@users.noreply.github.com> Co-authored-by: Probe Test <probe@example.com>
40 lines
1.8 KiB
TypeScript
40 lines
1.8 KiB
TypeScript
/**
|
|
* #3229 — agy upstream 400 responses were MASKED as an empty
|
|
* `chat.completion` envelope.
|
|
*
|
|
* Two parts:
|
|
* (a) `gemini-3.1-pro-low` passes through verbatim; High uses `gemini-pro-agent`.
|
|
* (b) the non-stream branch fed the 4xx response into the SSE collector, producing a
|
|
* synthetic `{"object":"chat.completion","content":""}` instead of a real error →
|
|
* build a proper sanitized error body for non-ok upstream responses. (Still valid.)
|
|
*/
|
|
import test from "node:test";
|
|
import assert from "node:assert/strict";
|
|
|
|
import { resolveAntigravityModelId } from "../../open-sse/config/antigravityModelAliases.ts";
|
|
import { buildAntigravityUpstreamError } from "../../open-sse/executors/antigravityUpstreamError.ts";
|
|
|
|
test("(a) agy gemini-3.1-pro-low passes through to upstream unchanged (#3696)", () => {
|
|
assert.equal(resolveAntigravityModelId("gemini-3.1-pro-low"), "gemini-3.1-pro-low");
|
|
// plain id stays plain
|
|
assert.equal(resolveAntigravityModelId("gemini-3.1-pro"), "gemini-3.1-pro");
|
|
});
|
|
|
|
test("(b) a non-ok upstream response becomes a real error body, not an empty chat.completion", () => {
|
|
const body = buildAntigravityUpstreamError(
|
|
400,
|
|
"Bad Request",
|
|
JSON.stringify({ error: { code: 400, message: "Model not found: rejected-model" } })
|
|
);
|
|
assert.notEqual((body as { object?: string }).object, "chat.completion");
|
|
assert.ok(body.error, "must carry an error object");
|
|
assert.equal(typeof body.error.message, "string");
|
|
// sanitized: no raw stack traces leaked (hard rule #12)
|
|
assert.ok(!body.error.message.includes("at /"));
|
|
|
|
// non-JSON upstream body still yields a valid error envelope
|
|
const body2 = buildAntigravityUpstreamError(503, "Service Unavailable", "<html>oops</html>");
|
|
assert.ok(body2.error);
|
|
assert.notEqual((body2 as { object?: string }).object, "chat.completion");
|
|
});
|