feat: add task archive functionality #703
Conversation
- Add archivedAt column to tasks table with migration - Add archive/restore/getArchivedTasks methods to DatabaseService - Add IPC handlers and expose via preload - Add Archive option to task context menu in sidebar - Add collapsible Archived section showing archived tasks per project - Add restore button on hover for archived tasks - Archived tasks are hidden from main list but preserved in database - Restored tasks appear at top of list (updatedAt is refreshed)
- Add delete button next to restore button in archived tasks section - Use same Trash icon and styling as other delete buttons - Add proper Radix UI tooltips for both restore and delete buttons
- Add archive button next to delete for individual tasks - Add bulk archive in select mode (Archive button next to Delete) - Pass onArchiveTask prop from App to ProjectMainView - Edge case already handled: archiving active task clears active state
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
![cursor[bot]](https://cdn.statically.io/img/avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
- Add silent option to handleArchiveTask for bulk operations - Pass silent: true from bulk archive to prevent duplicate toasts - Add restoringTaskIdsRef to prevent duplicate restore operations
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 3 potential issues.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| } | ||
| } catch (refreshError) { | ||
| log.error('Failed to refresh tasks after archive failure:', refreshError as any); | ||
| } |
There was a problem hiding this comment.
Archive error recovery lacks fallback for task restoration
Low Severity
The handleArchiveTask function optimistically removes the task from the UI, but if the archive operation fails AND the subsequent getTasks refresh also fails, the task disappears from the UI with no fallback restoration. Unlike handleDeleteTask which has a taskSnapshot fallback to restore the task if refresh fails, handleArchiveTask only logs the error. The task data remains safe in the database but becomes invisible until manual refresh.
| variant: 'destructive', | ||
| }); | ||
| } | ||
| }; |
There was a problem hiding this comment.
Archiving running tasks leaves orphaned terminal processes
High Severity
The handleArchiveTask function doesn't stop running agents or clean up terminal resources before archiving. Unlike handleDeleteTask which calls ptyKill, terminalSessionRegistry.dispose, and ptyClearSnapshot, handleArchiveTask performs no cleanup. If a task with active agents is archived, the PTY processes continue running in the background with no way for the user to stop them (the task is hidden from the UI). This causes resource leaks and potential unwanted file modifications.
| archivedNames.push(ws.name); | ||
| } catch { | ||
| // Continue archiving remaining tasks | ||
| } |
There was a problem hiding this comment.
Bulk archive counts failed tasks as successfully archived
Medium Severity
The handleBulkArchive function relies on onArchiveTask throwing an error to skip adding failed tasks to archivedNames. However, handleArchiveTask returns Promise<void> and catches errors internally without re-throwing, so it always returns normally. This means failed archives are still counted in the success summary toast, misleading users about which tasks were actually archived.
Summary
Adds the ability to archive tasks instead of permanently deleting them. Archived tasks are hidden from the main task list but remain accessible for reference or restoration.
Closes #695
Changes
archivedAtcolumn to tasks table with migrationTest Plan
Note
Medium Risk
Moderate risk: changes touch the persisted task schema and default task queries (archived filtering), plus new IPC surface and UI state updates that could cause tasks to disappear/reappear incorrectly if edge cases aren’t handled.
Overview
Adds task archiving as a soft-delete via a new
tasks.archived_atcolumn + Drizzle migration, and updates the DB layer sogetTasksexcludes archived tasks while newgetArchivedTasks/archiveTask/restoreTaskAPIs manage them.Exposes these operations over Electron IPC/preload and wires them into the UI: tasks can be archived from the sidebar context menu and from
ProjectMainView(including bulk archive), with a per-project collapsible Archived section that supports restore and permanent delete, plus telemetry/toast handling and optimistic UI updates.Written by Cursor Bugbot for commit 7d31f03. This will update automatically on new commits. Configure here.