fix(openrouter): bound video response reads · openclaw/op...

Recent Commits to openclaw:main

Alix-007 · 2026-06-29 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -546,6 +546,7 @@ describe("openrouter video generation provider", () => {`
`546`	`546`	`provider: "openrouter",`
`547`	`547`	`capability: "video",`
`548`	`548`	`baseUrl: "https://custom.openrouter.test/api/v1",`
	`549`	`+allowPrivateNetwork: false,`
`549`	`550`	`request: requestOverrides,`
`550`	`551`	`},`
`551`	`552`	`);`
`@@ -672,11 +673,10 @@ describe("openrouter video generation provider", () => {`
`672`	`673`
`673`	`674`	`it("wraps non-JSON successful OpenRouter submit responses", async () => {`
`674`	`675`	`postJsonRequestMock.mockResolvedValue({`
`675`		`-response: {`
`676`		`-json: async () => {`
`677`		`-throw new SyntaxError("Unexpected token < in JSON");`
`678`		`-},`
`679`		`-},`
	`676`	`+response: new Response("<html></html>", {`
	`677`	`+status: 200,`
	`678`	`+headers: { "content-type": "text/html" },`
	`679`	`+}),`
`680`	`680`	`release: vi.fn(async () => {}),`
`681`	`681`	`});`
`682`	`682`
`@@ -691,6 +691,22 @@ describe("openrouter video generation provider", () => {`
`691`	`691`	`).rejects.toThrow("OpenRouter video generation response malformed");`
`692`	`692`	`});`
`693`	`693`
	`694`	`+it("bounds oversized successful OpenRouter submit responses", async () => {`
	`695`	`+const oversized = releasedOversizedJsonStream();`
	`696`	`+postJsonRequestMock.mockResolvedValue(oversized);`
	`697`	`+`
	`698`	`+const provider = buildOpenRouterVideoGenerationProvider();`
	`699`	`+await expect(`
	`700`	`+provider.generateVideo({`
	`701`	`+provider: "openrouter",`
	`702`	`+model: "google/veo-3.1",`
	`703`	`+prompt: "oversized body",`
	`704`	`+cfg: {} as never,`
	`705`	`+}),`
	`706`	`+).rejects.toThrow("OpenRouter video generation: JSON response exceeds 16777216 bytes");`
	`707`	`+expect(oversized.wasCanceled()).toBe(true);`
	`708`	`+});`
	`709`	`+`
`694`	`710`	`it("rejects unknown OpenRouter poll statuses without waiting for timeout", async () => {`
`695`	`711`	`postJsonRequestMock.mockResolvedValue(`
`696`	`712`	`releasedJson({`

Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,7 @@ import {`
`6`	`6`	`assertOkOrThrowHttpError,`
`7`	`7`	`createProviderOperationDeadline,`
`8`	`8`	`postJsonRequest,`
	`9`	`+readProviderJsonResponse,`
`9`	`10`	`resolveProviderHttpRequestConfig,`
`10`	`11`	`resolveProviderOperationTimeoutMs,`
`11`	`12`	`sanitizeConfiguredModelProviderRequest,`
`@@ -68,9 +69,12 @@ type OpenRouterFrameImagePart = OpenRouterImagePart & {`
`68`	`69`	`async function readOpenRouterVideoJson(response: Response): Promise<Record<string, unknown>> {`
`69`	`70`	`let payload: unknown;`
`70`	`71`	`try {`
`71`		`-payload = await response.json();`
`72`		`-} catch {`
`73`		`-throw new Error(OPENROUTER_VIDEO_MALFORMED_RESPONSE);`
	`72`	`+payload = await readProviderJsonResponse<unknown>(response, "OpenRouter video generation");`
	`73`	`+} catch (error) {`
	`74`	`+if (error instanceof Error && error.message.endsWith(": malformed JSON response")) {`
	`75`	`+throw new Error(OPENROUTER_VIDEO_MALFORMED_RESPONSE, { cause: error });`
	`76`	`+}`
	`77`	`+throw error;`
`74`	`78`	`}`
`75`	`79`	`if (!isRecord(payload)) {`
`76`	`80`	`throw new Error(OPENROUTER_VIDEO_MALFORMED_RESPONSE);`

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

推荐订阅源

Recent Commits to openclaw:main