mirror of
https://github.com/musistudio/claude-code-router.git
synced 2026-11-04 00:21:31 +08:00
Burlesque1
cfb6aae248
Large (>=256KB) JSON request bodies were fully JSON.parse'd and pretty-printed in the body formatter Web Worker, which then crashed (memory/CPU) on huge object graphs - surfacing "Body formatter worker failed." and sticking at "Loading body...". Preview truncation only applied to non-JSON bodies. Now, in preview mode, an over-length text body is truncated to a plain-text preview (createLogBodyPreviewText) instead of being parsed/pretty-printed, so the worker never chokes. Full formatting still runs in "full" mode. The judge is the real body.text length, not sizeBytes (which may be inflated metadata and should still render its JSON tree when lightweight). Fixes #1694
82 lines
2.8 KiB
TypeScript
82 lines
2.8 KiB
TypeScript
import assert from "node:assert/strict";
|
|
import test from "node:test";
|
|
import type { RequestLogBody } from "@ccr/core/contracts/app";
|
|
import { formatLogBodyForWorker } from "@ccr/ui/pages/home/shared/log-body-worker-protocol.ts";
|
|
import { formatRouteTracePath } from "@ccr/ui/pages/home/shared/network.ts";
|
|
|
|
test("route trace paths use request and response dotted notation", () => {
|
|
assert.equal(
|
|
formatRouteTracePath({ path: "/body/model", scope: "body" }),
|
|
"request.body.model"
|
|
);
|
|
assert.equal(
|
|
formatRouteTracePath({ path: "/headers/content-type", scope: "headers" }),
|
|
"request.header.content-type"
|
|
);
|
|
assert.equal(
|
|
formatRouteTracePath({ path: "/response/body/output/0/text", scope: "body" }),
|
|
"response.body.output.0.text"
|
|
);
|
|
assert.equal(
|
|
formatRouteTracePath({ path: "/response/headers/x-request-id", scope: "headers" }),
|
|
"response.header.x-request-id"
|
|
);
|
|
assert.equal(
|
|
formatRouteTracePath({ path: "/routing/model", scope: "routing" }),
|
|
"request.routing.model"
|
|
);
|
|
assert.equal(
|
|
formatRouteTracePath({ path: "/url", scope: "url" }),
|
|
"request.url"
|
|
);
|
|
});
|
|
|
|
test("request log preview bodies still format parseable JSON as JSON", () => {
|
|
const body: RequestLogBody = {
|
|
bodyRef: "preview-json-body",
|
|
contentType: "application/json",
|
|
encoding: "utf8",
|
|
preview: true,
|
|
sizeBytes: 512 * 1024,
|
|
text: JSON.stringify({ messages: [{ role: "user", content: "hello" }], model: "test-model" }),
|
|
truncated: false
|
|
};
|
|
|
|
const view = formatLogBodyForWorker(body, "preview", 256 * 1024, 160 * 1024);
|
|
assert.equal(view.preview, true);
|
|
assert.deepEqual(view.json, {
|
|
messages: [{ role: "user", content: "hello" }],
|
|
model: "test-model"
|
|
});
|
|
assert.match(view.text, /"model": "test-model"/);
|
|
});
|
|
|
|
test("preview mode truncates an over-large JSON text body instead of parsing it", () => {
|
|
const hugeText = JSON.stringify({
|
|
model: "test-model",
|
|
messages: Array.from({ length: 4000 }, (_, i) => ({
|
|
role: i % 2 === 0 ? "user" : "assistant",
|
|
content: `message body number ${i} `.repeat(20)
|
|
}))
|
|
});
|
|
// Far beyond previewTextLimit, simulating a real ~308KB request body (issue #1694)
|
|
assert.ok(hugeText.length > 300 * 1024, "fixture should exceed 300KB");
|
|
|
|
const body: RequestLogBody = {
|
|
bodyRef: "large-json-body",
|
|
contentType: "application/json",
|
|
encoding: "utf8",
|
|
preview: true,
|
|
sizeBytes: hugeText.length,
|
|
text: hugeText,
|
|
truncated: false
|
|
};
|
|
|
|
const view = formatLogBodyForWorker(body, "preview", 256 * 1024, 160 * 1024);
|
|
assert.equal(view.preview, true);
|
|
// Over-length text is truncated instead of fully parsed/pretty-printed, avoiding the worker crash
|
|
assert.equal(view.json, undefined);
|
|
assert.ok(view.text.length < hugeText.length, "should be truncated");
|
|
assert.match(view.text, /characters omitted from preview/);
|
|
});
|