feat: auto-delete task history on extension load by hannesrudolph · Pull Request #9262 · RooCodeInc/Roo-Code

-Original file line number
+Diff line change
@@ Expand Up / @@ -281,7 +281,7 @@ describe("CustomToolRegistry", () => { @@
     			const result = await registry.loadFromDirectory(TEST_FIXTURES_DIR)
     			expect(result.loaded).toContain("cached")
-    		}, 30000)
+    		}, 120_000)
     	})
     	describe.sequential("loadFromDirectories", () => {
@@ Expand Down @@

packages/types/src/global-settings.ts

-Original file line number
+Diff line change
@@ Expand Up / @@ -44,6 +44,14 @@ export const MAX_CHECKPOINT_TIMEOUT_SECONDS = 60 @@
      */
     export const DEFAULT_CHECKPOINT_TIMEOUT_SECONDS = 15
+    /**
+     * Allowed values for the task history retention setting.
+     * Stored as strings in most UI/extension flows.
+     */
+    export const TASK_HISTORY_RETENTION_OPTIONS = ["never", "90", "60", "30", "7", "3"] as const
+    export type TaskHistoryRetentionSetting = (typeof TASK_HISTORY_RETENTION_OPTIONS)[number]
     /**
      * GlobalSettings
      */
@@ Expand Down Expand Up / @@ -181,6 +189,16 @@ export const globalSettingsSchema = z.object({ @@
     	customSupportPrompts: customSupportPromptsSchema.optional(),
     	enhancementApiConfigId: z.string().optional(),
     	includeTaskHistoryInEnhance: z.boolean().optional(),
+    	// Auto-delete task history on extension reload.
+    	taskHistoryRetention: z.enum(TASK_HISTORY_RETENTION_OPTIONS).optional(),
+    	// Calculated task history count for the Settings > About page
+    	// Note: Size calculation was removed for performance reasons - with large numbers of
+    	// tasks (e.g., 9000+), recursively stat'ing every file caused significant delays.
+    	taskHistorySize: z
+    		.object({
+    			taskCount: z.number(),
+    		})
+    		.optional(),
     	historyPreviewCollapsed: z.boolean().optional(),
     	reasoningBlockCollapsed: z.boolean().optional(),
     	/**
@@ Expand Down @@

packages/types/src/vscode-extension-host.ts

-Original file line number
+Diff line change
@@ Expand Up / @@ -335,6 +335,7 @@ export type ExtensionState = Pick< @@
     	| "maxGitStatusFiles"
     	| "requestDelaySeconds"
     	| "showWorktreesInHomeScreen"
+    	| "taskHistoryRetention"
     > & {
     	version: string
     	clineMessages: ClineMessage[]
@@ Expand Down Expand Up / @@ -395,6 +396,14 @@ export type ExtensionState = Pick< @@
     	marketplaceInstalledMetadata?: { project: Record<string, any>; global: Record<string, any> }
     	profileThresholds: Record<string, number>
     	hasOpenedModeSelector: boolean
+    	/** Task history count for the Settings > About page
+    	 * Note: Size calculation was removed for performance reasons - with large numbers of
+    	 * tasks (e.g., 9000+), recursively stat'ing every file caused significant delays.
+    	 */
+    	taskHistorySize?: {
+    		/** Number of task directories */
+    		taskCount: number
+    	}
     	openRouterImageApiKey?: string
     	messageQueue?: QueuedMessage[]
     	lastShownAnnouncementId?: string
@@ Expand Down Expand Up / @@ -592,6 +601,7 @@ export interface WebviewMessage { @@
     		| "requestModes"
     		| "switchMode"
     		| "debugSetting"
+    		| "refreshTaskHistorySize"
     		// Worktree messages
     		| "listWorktrees"
     		| "createWorktree"
@@ Expand Down @@

src/__tests__/extension.spec.ts

-Original file line number
+Diff line change
@@ Expand Up / @@ -114,6 +114,8 @@ vi.mock("../core/config/ContextProxy", () => ({ @@
     			setValue: vi.fn(),
     			getValues: vi.fn().mockReturnValue({}),
     			getProviderSettings: vi.fn().mockReturnValue({}),
+    			// Needed by retention purge on activation
+    			globalStorageUri: { fsPath: "/tmp/roo-retention-test" },
     		}),
     	},
     }))
@@ Expand Down Expand Up / @@ -157,6 +159,17 @@ vi.mock("../utils/autoImportSettings", () => ({ @@
     	autoImportSettings: vi.fn().mockResolvedValue(undefined),
     }))
+    // Avoid filesystem access during activation by stubbing background purge
+    vi.mock("../utils/task-history-retention", () => ({
+    	startBackgroundRetentionPurge: vi.fn(),
+    	startBackgroundCheckpointPurge: vi.fn(),
+    }))
+    // Ensure storage base path resolves to provided path to avoid touching VS Code config
+    vi.mock("../utils/storage", () => ({
+    	getStorageBasePath: (p: string) => Promise.resolve(p),
+    }))
     vi.mock("../extension/api", () => ({
     	API: vi.fn().mockImplementation(() => ({})),
     }))
@@ Expand Down @@

Provide feedback