Conversation
Entire-Checkpoint: ca22077c95e2
PR SummaryLow Risk Overview Extends Written by Cursor Bugbot for commit 7df74ed. Configure here. |
There was a problem hiding this comment.
Pull request overview
This PR fixes the display order of multiple sessions during resume operations. Previously, sessions were displayed in subdirectory index order which didn't reflect their actual chronological creation order. Now sessions are sorted by their CreatedAt timestamp in ascending order, ensuring the most recent session is displayed last.
Changes:
- Added
CreatedAtfield toRestoredSessionstruct to track session creation time - Populated
CreatedAtfrom checkpoint metadata inManualCommitStrategy.RestoreLogsOnly - Sorted sessions by
CreatedAtin ascending order before displaying resume commands
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| cmd/entire/cli/strategy/strategy.go | Added CreatedAt field to RestoredSession struct with documentation |
| cmd/entire/cli/strategy/manual_commit_rewind.go | Populated CreatedAt field from session metadata during restore |
| cmd/entire/cli/resume.go | Added sorting logic to order sessions by creation time before display |
No description provided.