fix(qa): reject non-object mock Anthropic JSON · openclaw/openclaw@5e86c7e

Recent Commits to openclaw:main

vincentkoc · 2026-06-21 · via Recent Commits to openclaw:main

File tree

extensions/qa-lab/src/providers/mock-openai

Original file line number	Diff line number	Diff line change
`@@ -4240,7 +4240,7 @@ describe("qa mock openai server", () => {`
`4240`	`4240`	`expect(body).not.toContain("HEARTBEAT_OK");`
`4241`	`4241`	`});`
`4242`	`4242`
`4243`		`-it("rejects malformed Anthropic /v1/messages JSON with an invalid_request_error", async () => {`
	`4243`	`+it("rejects malformed or non-object Anthropic /v1/messages JSON", async () => {`
`4244`	`4244`	`const server = await startQaMockOpenAiServer({`
`4245`	`4245`	`host: "127.0.0.1",`
`4246`	`4246`	`port: 0,`
`@@ -4249,20 +4249,25 @@ describe("qa mock openai server", () => {`
`4249`	`4249`	`await server.stop();`
`4250`	`4250`	`});`
`4251`	`4251`
`4252`		-const response = await fetch(`${server.baseUrl}/v1/messages`, {
`4253`		`-method: "POST",`
`4254`		`-headers: { "content-type": "application/json" },`
`4255`		`-body: '{"model":"claude-opus-4-8","messages":[',`
`4256`		`-});`
	`4252`	`+for (const rawBody of ['{"model":"claude-opus-4-8","messages":[', "null", "[]", '"text"']) {`
	`4253`	+const response = await fetch(`${server.baseUrl}/v1/messages`, {
	`4254`	`+method: "POST",`
	`4255`	`+headers: { "content-type": "application/json" },`
	`4256`	`+body: rawBody,`
	`4257`	`+});`
	`4258`	`+`
	`4259`	`+expect(response.status).toBe(400);`
	`4260`	`+const body = (await response.json()) as {`
	`4261`	`+type: string;`
	`4262`	`+error: { type: string; message: string };`
	`4263`	`+};`
	`4264`	`+expect(body.type).toBe("error");`
	`4265`	`+expect(body.error.type).toBe("invalid_request_error");`
	`4266`	`+expect(body.error.message).toContain("Malformed JSON body");`
	`4267`	`+}`
`4257`	`4268`
`4258`		`-expect(response.status).toBe(400);`
`4259`		`-const body = (await response.json()) as {`
`4260`		`-type: string;`
`4261`		`-error: { type: string; message: string };`
`4262`		`-};`
`4263`		`-expect(body.type).toBe("error");`
`4264`		`-expect(body.error.type).toBe("invalid_request_error");`
`4265`		`-expect(body.error.message).toContain("Malformed JSON body");`
	`4269`	+const health = await fetch(`${server.baseUrl}/healthz`);
	`4270`	`+expect(health.status).toBe(200);`
`4266`	`4271`	`});`
`4267`	`4272`
`4268`	`4273`	`it("rejects malformed OpenAI-compatible JSON without crashing the mock server", async () => {`

Original file line number	Diff line number	Diff line change
`@@ -228,7 +228,7 @@ function readBody(req: IncomingMessage): Promise<string> {`
`228`	`228`	`});`
`229`	`229`	`}`
`230`	`230`
`231`		`-function parseOpenAiJsonBody(raw: string): Record<string, unknown> \| null {`
	`231`	`+function parseJsonObjectBody(raw: string): Record<string, unknown> \| null {`
`232`	`232`	`try {`
`233`	`233`	`const parsed = raw ? (JSON.parse(raw) as unknown) : {};`
`234`	`234`	`return parsed && typeof parsed === "object" && !Array.isArray(parsed)`
`@@ -3438,7 +3438,7 @@ export async function startQaMockOpenAiServer(params?: { host?: string; port?: n`
`3438`	`3438`	`}`
`3439`	`3439`	`if (req.method === "POST" && url.pathname === "/v1/images/generations") {`
`3440`	`3440`	`const raw = await readBody(req);`
`3441`		`-const body = parseOpenAiJsonBody(raw);`
	`3441`	`+const body = parseJsonObjectBody(raw);`
`3442`	`3442`	`if (!body) {`
`3443`	`3443`	`writeOpenAiMalformedJsonError(res, "OpenAI Images");`
`3444`	`3444`	`return;`
`@@ -3466,7 +3466,7 @@ export async function startQaMockOpenAiServer(params?: { host?: string; port?: n`
`3466`	`3466`	`}`
`3467`	`3467`	`if (req.method === "POST" && url.pathname === "/v1/embeddings") {`
`3468`	`3468`	`const raw = await readBody(req);`
`3469`		`-const body = parseOpenAiJsonBody(raw);`
	`3469`	`+const body = parseJsonObjectBody(raw);`
`3470`	`3470`	`if (!body) {`
`3471`	`3471`	`writeOpenAiMalformedJsonError(res, "OpenAI Embeddings");`
`3472`	`3472`	`return;`
`@@ -3492,7 +3492,7 @@ export async function startQaMockOpenAiServer(params?: { host?: string; port?: n`
`3492`	`3492`	`}`
`3493`	`3493`	`if (req.method === "POST" && url.pathname === "/v1/responses") {`
`3494`	`3494`	`const raw = await readBody(req);`
`3495`		`-const body = parseOpenAiJsonBody(raw);`
	`3495`	`+const body = parseJsonObjectBody(raw);`
`3496`	`3496`	`if (!body) {`
`3497`	`3497`	`writeOpenAiMalformedJsonError(res, "OpenAI Responses");`
`3498`	`3498`	`return;`
`@@ -3534,10 +3534,8 @@ export async function startQaMockOpenAiServer(params?: { host?: string; port?: n`
`3534`	`3534`	`}`
`3535`	`3535`	`if (req.method === "POST" && url.pathname === "/v1/messages") {`
`3536`	`3536`	`const raw = await readBody(req);`
`3537`		`-let body: AnthropicMessagesRequest;`
`3538`		`-try {`
`3539`		`-body = raw ? (JSON.parse(raw) as AnthropicMessagesRequest) : {};`
`3540`		`-} catch {`
	`3537`	`+const body = parseJsonObjectBody(raw) as AnthropicMessagesRequest \| null;`
	`3538`	`+if (!body) {`
`3541`	`3539`	`writeJson(res, 400, {`
`3542`	`3540`	`type: "error",`
`3543`	`3541`	`error: {`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main

File tree