refactor(agents): remove video task status wrappers · ope...

Recent Commits to openclaw:main

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

Original file line number	Diff line number	Diff line change
`@@ -69,8 +69,6 @@ vi.mock("../video-generation-task-status.js", () => ({`
`69`	`69`	`buildVideoGenerationTaskStatusDetails: vi.fn(() => ({})),`
`70`	`70`	`buildVideoGenerationTaskStatusText: vi.fn(() => ""),`
`71`	`71`	`findActiveVideoGenerationTaskForSession: vi.fn(() => undefined),`
`72`		`-getVideoGenerationTaskProviderId: vi.fn(() => undefined),`
`73`		`-isActiveVideoGenerationTask: vi.fn(() => false),`
`74`	`72`	`}));`
`75`	`73`
`76`	`74`	`vi.mock("../image-generation-task-status.js", () => ({`

Original file line number	Diff line number	Diff line change
`@@ -17,8 +17,6 @@ const videoGenerationTaskStatusMocks = vi.hoisted(() => ({`
`17`	`17`	`buildVideoGenerationTaskStatusDetails: vi.fn(() => ({})),`
`18`	`18`	`buildVideoGenerationTaskStatusText: vi.fn(() => "Video generation task status"),`
`19`	`19`	`findActiveVideoGenerationTaskForSession: vi.fn(),`
`20`		`-getVideoGenerationTaskProviderId: vi.fn(),`
`21`		`-isActiveVideoGenerationTask: vi.fn(() => false),`
`22`	`20`	`VIDEO_GENERATION_TASK_KIND: "video_generation",`
`23`	`21`	`}));`
`24`	`22`	`const musicGenerationTaskStatusMocks = vi.hoisted(() => ({`

Original file line number	Diff line number	Diff line change
`@@ -24,8 +24,6 @@ const videoGenerationTaskStatusMocks = vi.hoisted(() => ({`
`24`	`24`	`buildVideoGenerationTaskStatusDetails: vi.fn(() => ({})),`
`25`	`25`	`buildVideoGenerationTaskStatusText: vi.fn(() => "Video generation task status"),`
`26`	`26`	`findActiveVideoGenerationTaskForSession: vi.fn(),`
`27`		`-getVideoGenerationTaskProviderId: vi.fn(),`
`28`		`-isActiveVideoGenerationTask: vi.fn(() => false),`
`29`	`27`	`VIDEO_GENERATION_TASK_KIND: "video_generation",`
`30`	`28`	`}));`
`31`	`29`

Original file line number	Diff line number	Diff line change
`@@ -7,8 +7,6 @@ import {`
`7`	`7`	`buildVideoGenerationTaskStatusDetails,`
`8`	`8`	`buildVideoGenerationTaskStatusText,`
`9`	`9`	`findActiveVideoGenerationTaskForSession,`
`10`		`-getVideoGenerationTaskProviderId,`
`11`		`-isActiveVideoGenerationTask,`
`12`	`10`	`VIDEO_GENERATION_TASK_KIND,`
`13`	`11`	`} from "./video-generation-task-status.js";`
`14`	`12`
`@@ -48,8 +46,8 @@ describe("video generation task status", () => {`
`48`	`46`	`});`
`49`	`47`
`50`	`48`	`it("recognizes active session-backed video generation tasks", () => {`
`51`		`-expect(`
`52`		`-isActiveVideoGenerationTask({`
	`49`	`+taskRuntimeInternalMocks.listTasksForOwnerKey.mockReturnValue([`
	`50`	`+{`
`53`	`51`	`taskId: "task-1",`
`54`	`52`	`runtime: "cli",`
`55`	`53`	`taskKind: VIDEO_GENERATION_TASK_KIND,`
`@@ -62,10 +60,8 @@ describe("video generation task status", () => {`
`62`	`60`	`deliveryStatus: "not_applicable",`
`63`	`61`	`notifyPolicy: "silent",`
`64`	`62`	`createdAt: Date.now(),`
`65`		`-}),`
`66`		`-).toBe(true);`
`67`		`-expect(`
`68`		`-isActiveVideoGenerationTask({`
	`63`	`+},`
	`64`	`+{`
`69`	`65`	`taskId: "task-2",`
`70`	`66`	`runtime: "cron",`
`71`	`67`	`taskKind: VIDEO_GENERATION_TASK_KIND,`
`@@ -78,8 +74,10 @@ describe("video generation task status", () => {`
`78`	`74`	`deliveryStatus: "not_applicable",`
`79`	`75`	`notifyPolicy: "silent",`
`80`	`76`	`createdAt: Date.now(),`
`81`		`-}),`
`82`		`-).toBe(false);`
	`77`	`+},`
	`78`	`+]);`
	`79`	`+`
	`80`	`+expect(findActiveVideoGenerationTaskForSession("agent:main")?.taskId).toBe("task-1");`
`83`	`81`	`});`
`84`	`82`
`85`	`83`	`it("prefers a running task over queued session siblings", () => {`
`@@ -121,7 +119,6 @@ describe("video generation task status", () => {`
`121`	`119`
`122`	`120`	`expect(task?.taskId).toBe("task-running");`
`123`	`121`	`const activeTask = expectActiveVideoGenerationTask(task);`
`124`		`-expect(getVideoGenerationTaskProviderId(activeTask)).toBe("openai");`
`125`	`122`	`expect(buildVideoGenerationTaskStatusText(activeTask, { duplicateGuard: true })).toContain(`
`126`	`123`	`"Do not call video_generate again for this request.",`
`127`	`124`	`);`

Original file line number	Diff line number	Diff line change
`@@ -11,27 +11,12 @@ import {`
`11`	`11`	`buildMediaGenerationTaskStatusText,`
`12`	`12`	`findActiveMediaGenerationTaskForSession,`
`13`	`13`	`findDuplicateGuardMediaGenerationTaskForSession,`
`14`		`-getMediaGenerationTaskProviderId,`
`15`		`-isActiveMediaGenerationTask,`
`16`	`14`	`} from "./media-generation-task-status-shared.js";`
`17`	`15`
`18`	`16`	`export const VIDEO_GENERATION_TASK_KIND = "video_generation";`
`19`	`17`	`const VIDEO_GENERATION_SOURCE_PREFIX = "video_generate";`
`20`	`18`	`const RECENT_VIDEO_GENERATION_DUPLICATE_GUARD_MS = 2 * 60_000;`
`21`	`19`
`22`		`-/** Returns whether a task is an active video generation task. */`
`23`		`-export function isActiveVideoGenerationTask(task: TaskRecord): boolean {`
`24`		`-return isActiveMediaGenerationTask({`
`25`		`- task,`
`26`		`-taskKind: VIDEO_GENERATION_TASK_KIND,`
`27`		`-});`
`28`		`-}`
`29`		`-`
`30`		`-/** Extracts the provider id from a video generation task source. */`
`31`		`-export function getVideoGenerationTaskProviderId(task: TaskRecord): string \| undefined {`
`32`		`-return getMediaGenerationTaskProviderId(task, VIDEO_GENERATION_SOURCE_PREFIX);`
`33`		`-}`
`34`		`-`
`35`	`20`	`/** Finds an active video generation task for a session. */`
`36`	`21`	`export function findActiveVideoGenerationTaskForSession(`
`37`	`22`	`sessionKey?: string,`

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

推荐订阅源

Recent Commits to openclaw:main