CLI - Tasks History #3289
CLI - Tasks History #3289
Conversation
🦋 Changeset detectedLatest commit: 7998d7b The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Pull Request Overview
This PR introduces task history functionality to the CLI, enabling users to view, search, filter, and navigate through their task history directly from the command line. The implementation adds a new /tasks command with multiple subcommands, integrates task history state management, and includes terminal refresh capabilities.
Key Changes
- Added
/taskscommand with subcommands for viewing, searching, selecting, and filtering task history - Implemented task history state management using Jotai atoms
- Enhanced terminal clearing mechanism with a new refresh functionality
Reviewed Changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
| cli/src/commands/tasks.ts | New command implementing task history viewing, search, pagination, and filtering |
| cli/src/state/atoms/taskHistory.ts | Task history state atoms for data, filters, pagination, and loading states |
| cli/src/state/hooks/useTaskHistory.ts | Hook for managing task history operations and state updates |
| cli/src/state/atoms/ui.ts | Added refreshTerminalAtom for terminal clearing |
| cli/src/state/hooks/useTerminal.ts | Updated to use refreshTerminalCounter for terminal clearing |
| cli/src/state/hooks/useCommandContext.ts | Extended context with task history functionality |
| cli/src/state/atoms/effects.ts | Added handler for taskHistoryResponse messages |
| cli/src/ui/UI.tsx | Added task history fetching on mount |
| cli/src/commands/core/types.ts | Extended CommandContext with task history properties |
| cli/src/commands/new.ts | Added terminal refresh on new task |
| cli/src/commands/clear.ts | Added terminal refresh on clear |
| cli/src/commands/index.ts | Registered tasks command |
| cli/src/state/hooks/useCommandInput.ts | Added taskHistoryData to context |
| cli/src/services/autocomplete.ts | Added taskHistoryData to provider context |
| cli/src/ui/messages/MessageDisplay.tsx | Added logs import |
| .changeset/spicy-kings-appear.md | Changelog entry |
Co-authored-by: Copilot <[email protected]>
catrielmuller
force-pushed
the
catrielmuller/cli-tasks-list-and-load
branch
from
e6f290d to
7998d7b
Compare
| @@ -0,0 +1,8 @@ | |||
| export const generateMessage = () => { | |||
| const now = Date.now() | |||
| const uniqueSuffix = Math.floor(Math.random() * 10000) | |||
There was a problem hiding this comment.
We can be slightly more robust by having a counter variable above and then just using/incrementing the counter instead of generating a random suffix.
There was a problem hiding this comment.
We can add uuid to be more safe, this is a minimal workaround on the CLI. The extension shouls be send the Messages with a unique ID.
@RSO, What do you think of this?
Context
Resolve: #2995
Implementation
generateMessageto generate unique idsScreenshots