fix: validate together video durations · openclaw/opencla...

Recent Commits to openclaw:main

steipete · 2026-05-29 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -128,6 +128,42 @@ describe("together video generation provider", () => {`
`128`	`128`	`expect(request.url).toBe("https://api.together.xyz/v2/videos");`
`129`	`129`	`});`
`130`	`130`
	`131`	`+it("drops out-of-range duration values before creating videos", async () => {`
	`132`	`+postJsonRequestMock.mockResolvedValue({`
	`133`	`+response: {`
	`134`	`+json: async () => ({`
	`135`	`+id: "video_123",`
	`136`	`+}),`
	`137`	`+},`
	`138`	`+release: vi.fn(async () => {}),`
	`139`	`+});`
	`140`	`+fetchWithTimeoutMock`
	`141`	`+.mockResolvedValueOnce({`
	`142`	`+json: async () => ({`
	`143`	`+id: "video_123",`
	`144`	`+status: "completed",`
	`145`	`+outputs: { video_url: "https://example.com/together.mp4" },`
	`146`	`+}),`
	`147`	`+})`
	`148`	`+.mockResolvedValueOnce({`
	`149`	`+headers: new Headers({ "content-type": "video/mp4" }),`
	`150`	`+arrayBuffer: async () => Buffer.from("mp4-bytes"),`
	`151`	`+});`
	`152`	`+`
	`153`	`+const provider = buildTogetherVideoGenerationProvider();`
	`154`	`+await provider.generateVideo({`
	`155`	`+provider: "together",`
	`156`	`+model: "Wan-AI/Wan2.2-T2V-A14B",`
	`157`	`+prompt: "A bicycle weaving through a rainy neon street",`
	`158`	`+durationSeconds: 99,`
	`159`	`+cfg: {},`
	`160`	`+});`
	`161`	`+`
	`162`	`+const request = requireFirstPostJsonRequest("Together request");`
	`163`	`+const body = requireRecord(request.body, "Together request body");`
	`164`	`+expect(body).not.toHaveProperty("seconds");`
	`165`	`+});`
	`166`	`+`
`131`	`167`	`it("rejects reference images for Together text-to-video models before calling the API", async () => {`
`132`	`168`	`const provider = buildTogetherVideoGenerationProvider();`
`133`	`169`

Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,10 @@ import {`
`12`	`12`	`resolveProviderHttpRequestConfig,`
`13`	`13`	`type ProviderOperationTimeoutMs,`
`14`	`14`	`} from "openclaw/plugin-sdk/provider-http";`
`15`		`-import { normalizeOptionalString } from "openclaw/plugin-sdk/string-coerce-runtime";`
	`15`	`+import {`
	`16`	`+asSafeIntegerInRange,`
	`17`	`+normalizeOptionalString,`
	`18`	`+} from "openclaw/plugin-sdk/string-coerce-runtime";`
`16`	`19`	`import type {`
`17`	`20`	`GeneratedVideoAsset,`
`18`	`21`	`VideoGenerationProvider,`
`@@ -26,6 +29,8 @@ const TOGETHER_VIDEO_BASE_URL = "https://api.together.xyz/v2";`
`26`	`29`	`const DEFAULT_TIMEOUT_MS = 120_000;`
`27`	`30`	`const POLL_INTERVAL_MS = 5_000;`
`28`	`31`	`const MAX_POLL_ATTEMPTS = 120;`
	`32`	`+const TOGETHER_MIN_DURATION_SECONDS = 1;`
	`33`	`+const TOGETHER_MAX_DURATION_SECONDS = 10;`
`29`	`34`
`30`	`35`	`type TogetherVideoResponse = {`
`31`	`36`	`id?: string;`
`@@ -81,6 +86,17 @@ function extractTogetherVideoUrl(payload: TogetherVideoResponse): string \| undef`
`81`	`86`	`);`
`82`	`87`	`}`
`83`	`88`
	`89`	`+function resolveTogetherDurationSeconds(value: unknown): string \| undefined {`
	`90`	`+if (typeof value !== "number" \|\| !Number.isFinite(value)) {`
	`91`	`+return undefined;`
	`92`	`+}`
	`93`	`+const duration = asSafeIntegerInRange(Math.round(value), {`
	`94`	`+min: TOGETHER_MIN_DURATION_SECONDS,`
	`95`	`+max: TOGETHER_MAX_DURATION_SECONDS,`
	`96`	`+});`
	`97`	`+return duration === undefined ? undefined : String(duration);`
	`98`	`+}`
	`99`	`+`
`84`	`100`	`async function pollTogetherVideo(params: {`
`85`	`101`	`videoId: string;`
`86`	`102`	`headers: Headers;`
`@@ -151,15 +167,15 @@ export function buildTogetherVideoGenerationProvider(): VideoGenerationProvider`
`151`	`167`	`capabilities: {`
`152`	`168`	`generate: {`
`153`	`169`	`maxVideos: 1,`
`154`		`-maxDurationSeconds: 12,`
	`170`	`+maxDurationSeconds: TOGETHER_MAX_DURATION_SECONDS,`
`155`	`171`	`supportsSize: true,`
`156`	`172`	`},`
`157`	`173`	`imageToVideo: {`
`158`	`174`	`enabled: true,`
`159`	`175`	`maxInputImagesByModel: {`
`160`	`176`	`"Wan-AI/Wan2.2-I2V-A14B": 1,`
`161`	`177`	`},`
`162`		`-maxDurationSeconds: 12,`
	`178`	`+maxDurationSeconds: TOGETHER_MAX_DURATION_SECONDS,`
`163`	`179`	`supportsSize: true,`
`164`	`180`	`},`
`165`	`181`	`videoToVideo: {`
`@@ -203,8 +219,9 @@ export function buildTogetherVideoGenerationProvider(): VideoGenerationProvider`
`203`	`219`	`prompt: req.prompt,`
`204`	`220`	`};`
`205`	`221`	`const model = String(body.model);`
`206`		`-if (typeof req.durationSeconds === "number" && Number.isFinite(req.durationSeconds)) {`
`207`		`-body.seconds = String(Math.max(1, Math.round(req.durationSeconds)));`
	`222`	`+const duration = resolveTogetherDurationSeconds(req.durationSeconds);`
	`223`	`+if (duration !== undefined) {`
	`224`	`+body.seconds = duration;`
`208`	`225`	`}`
`209`	`226`	`const size = normalizeOptionalString(req.size);`
`210`	`227`	`if (size) {`

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

推荐订阅源

Recent Commits to openclaw:main