mirror of
https://github.com/diegosouzapw/OmniRoute.git
synced 2026-08-06 15:22:12 +03:00
Validated in local merge-train (devbox-vm-06-dev002) @ combined-tip (FAST gates: typecheck/complexity/cognitive/changelog/vitest — only pre-existing audit.test.ts flake). Evidence: /home/diegosouzapw/dev/proxys/OmniRoute/.claude/worktrees/merge-train-20260805-222248-suite.log
160 lines
6.3 KiB
TypeScript
160 lines
6.3 KiB
TypeScript
// Characterization of applyClientUsageBuffer — the non-streaming usage buffer/estimate block
|
|
// extracted from handleChatCore (chatCore god-file decomposition, #3501). Deps are injected so the
|
|
// buffer-vs-estimate branch and the in-place mutation of translatedResponse.usage are observable.
|
|
// Locks: usage present → buffer+filter; usage absent → estimate from content length; empty content
|
|
// (length 2 from JSON.stringify("")) still estimates; the mutation target is translatedResponse.
|
|
import { test } from "node:test";
|
|
import assert from "node:assert/strict";
|
|
|
|
const { applyClientUsageBuffer } =
|
|
await import("../../open-sse/handlers/chatCore/clientUsageBuffer.ts");
|
|
const { resolveChatCoreRequestFormat } =
|
|
await import("../../open-sse/handlers/chatCore/requestFormat.ts");
|
|
|
|
function makeDeps(overrides: Record<string, unknown> = {}) {
|
|
const calls = { buffer: [] as unknown[], estimate: [] as unknown[], filter: [] as unknown[] };
|
|
const deps = {
|
|
addBufferToUsage: (u: unknown) => {
|
|
calls.buffer.push(u);
|
|
return { ...(u as object), _buffered: true };
|
|
},
|
|
estimateUsage: (...a: unknown[]) => {
|
|
calls.estimate.push(a);
|
|
return { _estimated: true };
|
|
},
|
|
filterUsageForFormat: (u: unknown, _fmt: unknown) => {
|
|
calls.filter.push(u);
|
|
return { ...(u as object), _filtered: true };
|
|
},
|
|
...overrides,
|
|
} as Parameters<typeof applyClientUsageBuffer>[4];
|
|
return { deps, calls };
|
|
}
|
|
|
|
test("request format producer forwards its string contract to usage estimation", () => {
|
|
const { clientResponseFormat } = resolveChatCoreRequestFormat({
|
|
clientRawRequest: { endpoint: "/v1/chat/completions" },
|
|
body: { messages: [] },
|
|
provider: "openai",
|
|
userAgent: null,
|
|
});
|
|
const { deps, calls } = makeDeps();
|
|
const resp: Record<string, unknown> = {
|
|
choices: [{ message: { content: "hello" } }],
|
|
};
|
|
|
|
applyClientUsageBuffer(resp, { messages: [] }, clientResponseFormat, {}, deps);
|
|
|
|
const args = calls.estimate[0] as unknown[];
|
|
assert.equal(typeof clientResponseFormat, "string");
|
|
assert.equal(args[2], clientResponseFormat);
|
|
});
|
|
|
|
test("usage present → buffer then filter, mutates in place", () => {
|
|
const { deps, calls } = makeDeps();
|
|
const resp: Record<string, unknown> = { usage: { prompt_tokens: 5 } };
|
|
applyClientUsageBuffer(resp, { messages: [] }, "openai", {}, deps);
|
|
assert.equal(calls.buffer.length, 1);
|
|
assert.equal(calls.estimate.length, 0);
|
|
assert.equal((resp.usage as Record<string, unknown>)._buffered, true);
|
|
assert.equal((resp.usage as Record<string, unknown>)._filtered, true);
|
|
});
|
|
|
|
test("all-zero usage stub → estimate (not constant buffer-only 2000)", () => {
|
|
const { deps, calls } = makeDeps();
|
|
const resp: Record<string, unknown> = {
|
|
usage: { prompt_tokens: 0, completion_tokens: 0, total_tokens: 0 },
|
|
choices: [{ message: { content: "PONG" } }],
|
|
};
|
|
applyClientUsageBuffer(resp, { messages: [{ role: "user", content: "hi" }] }, "openai", {}, deps);
|
|
assert.equal(calls.buffer.length, 0, "must not buffer zeros into USAGE_TOKEN_BUFFER");
|
|
assert.equal(calls.estimate.length, 1);
|
|
assert.equal((resp.usage as Record<string, unknown>)._estimated, true);
|
|
});
|
|
|
|
test("no usage but content present → estimate then filter", () => {
|
|
const { deps, calls } = makeDeps();
|
|
const resp: Record<string, unknown> = {
|
|
choices: [{ message: { content: "hello world" } }],
|
|
};
|
|
applyClientUsageBuffer(resp, { messages: [] }, "openai", {}, deps);
|
|
assert.equal(calls.buffer.length, 0);
|
|
assert.equal(calls.estimate.length, 1);
|
|
assert.equal((resp.usage as Record<string, unknown>)._estimated, true);
|
|
assert.equal((resp.usage as Record<string, unknown>)._filtered, true);
|
|
// estimateUsage receives (body, contentLength, format)
|
|
const args = calls.estimate[0] as unknown[];
|
|
assert.equal(args[2], "openai");
|
|
assert.equal(typeof args[1], "number");
|
|
});
|
|
|
|
test("empty content → JSON.stringify('') length 2 > 0 still estimates", () => {
|
|
const { deps, calls } = makeDeps();
|
|
const resp: Record<string, unknown> = {};
|
|
applyClientUsageBuffer(resp, {}, "claude", {}, deps);
|
|
// content "" → JSON.stringify("") = '""' length 2 → contentLength 2 > 0
|
|
assert.equal(calls.estimate.length, 1);
|
|
const args = calls.estimate[0] as unknown[];
|
|
assert.equal(args[1], 2);
|
|
});
|
|
|
|
test("content length is computed from choices[0].message.content", () => {
|
|
const { deps, calls } = makeDeps();
|
|
const resp: Record<string, unknown> = {
|
|
choices: [{ message: { content: "abc" } }],
|
|
};
|
|
applyClientUsageBuffer(resp, {}, "openai", {}, deps);
|
|
// JSON.stringify("abc") = '"abc"' → length 5
|
|
const args = calls.estimate[0] as unknown[];
|
|
assert.equal(args[1], 5);
|
|
});
|
|
|
|
// #8331/#8356 added the `options` parameter between `clientResponseFormat` and `deps`,
|
|
// which is what silently broke the five call sites above (the injected spies landed in
|
|
// the `options` slot, so the real implementations ran and no spy was ever recorded).
|
|
// Cover the option itself so the new parameter is exercised, not just tolerated.
|
|
|
|
test("preserveContextBudgetInVisibleUsage folds context_budget_* back into visible fields", () => {
|
|
const { deps, calls } = makeDeps({
|
|
addBufferToUsage: (u: unknown) => ({
|
|
...(u as object),
|
|
context_budget_prompt_tokens: 2005,
|
|
context_budget_input_tokens: 2005,
|
|
context_budget_total_tokens: 2010,
|
|
}),
|
|
});
|
|
const resp: Record<string, unknown> = {
|
|
usage: { prompt_tokens: 5, input_tokens: 5, total_tokens: 10 },
|
|
};
|
|
|
|
applyClientUsageBuffer(
|
|
resp,
|
|
{ messages: [] },
|
|
"openai",
|
|
{
|
|
preserveContextBudgetInVisibleUsage: true,
|
|
},
|
|
deps
|
|
);
|
|
|
|
const filtered = calls.filter[0] as Record<string, unknown>;
|
|
assert.equal(filtered.prompt_tokens, 2005, "Claude-Code path re-folds the buffered value");
|
|
assert.equal(filtered.input_tokens, 2005);
|
|
assert.equal(filtered.total_tokens, 2010);
|
|
});
|
|
|
|
test("without the option the visible usage keeps the real unbuffered #8331 numbers", () => {
|
|
const { deps, calls } = makeDeps({
|
|
addBufferToUsage: (u: unknown) => ({
|
|
...(u as object),
|
|
context_budget_prompt_tokens: 2005,
|
|
}),
|
|
});
|
|
const resp: Record<string, unknown> = { usage: { prompt_tokens: 5 } };
|
|
|
|
applyClientUsageBuffer(resp, { messages: [] }, "openai", {}, deps);
|
|
|
|
const filtered = calls.filter[0] as Record<string, unknown>;
|
|
assert.equal(filtered.prompt_tokens, 5, "default path must not inflate client-visible metering");
|
|
});
|