From 711e300f1b87b03073c33b5d61d1e89bb8a4e33c Mon Sep 17 00:00:00 2001 From: murashit Date: Mon, 22 Jun 2026 13:56:21 +0900 Subject: [PATCH] Remove unused threads refresh scheduling --- src/features/threads-view/view-lifecycle.ts | 7 ------- tests/features/threads-view/view-lifecycle.test.ts | 12 +----------- 2 files changed, 1 insertion(+), 18 deletions(-) diff --git a/src/features/threads-view/view-lifecycle.ts b/src/features/threads-view/view-lifecycle.ts index 8e19a8c4..c4151162 100644 --- a/src/features/threads-view/view-lifecycle.ts +++ b/src/features/threads-view/view-lifecycle.ts @@ -9,26 +9,19 @@ export type ThreadsViewRefreshLifecycleEvent = export interface ThreadsViewDeferredTasks { scheduleRender(callback: () => void): void; - scheduleRefresh(callback: () => void): void; clearAll(): void; } export function createThreadsViewDeferredTasks(getWindow: () => DeferredTaskWindow): ThreadsViewDeferredTasks { const renderTask = new DeferredTask(getWindow, 0); - const refreshTask = new DeferredTask(getWindow, 250); return { scheduleRender(callback): void { renderTask.schedule(callback); }, - scheduleRefresh(callback): void { - refreshTask.schedule(callback); - }, - clearAll(): void { renderTask.clear(); - refreshTask.clear(); }, }; } diff --git a/tests/features/threads-view/view-lifecycle.test.ts b/tests/features/threads-view/view-lifecycle.test.ts index dc37cff7..b6c6b6f8 100644 --- a/tests/features/threads-view/view-lifecycle.test.ts +++ b/tests/features/threads-view/view-lifecycle.test.ts @@ -9,24 +9,17 @@ import { } from "../../../src/features/threads-view/view-lifecycle"; describe("createThreadsViewDeferredTasks", () => { - it("coalesces render and refresh callbacks", () => { + it("coalesces render callbacks", () => { vi.useFakeTimers(); try { const tasks = createThreadsViewDeferredTasks(() => window); const render = vi.fn(); - const refresh = vi.fn(); tasks.scheduleRender(render); tasks.scheduleRender(render); - tasks.scheduleRefresh(refresh); - tasks.scheduleRefresh(refresh); vi.advanceTimersByTime(0); expect(render).toHaveBeenCalledTimes(1); - expect(refresh).not.toHaveBeenCalled(); - - vi.advanceTimersByTime(250); - expect(refresh).toHaveBeenCalledTimes(1); } finally { vi.useRealTimers(); } @@ -37,15 +30,12 @@ describe("createThreadsViewDeferredTasks", () => { try { const tasks = createThreadsViewDeferredTasks(() => window); const render = vi.fn(); - const refresh = vi.fn(); tasks.scheduleRender(render); - tasks.scheduleRefresh(refresh); tasks.clearAll(); vi.runAllTimers(); expect(render).not.toHaveBeenCalled(); - expect(refresh).not.toHaveBeenCalled(); } finally { vi.useRealTimers(); }