feat: add --resume to resume sessions by ID #355
+412
−1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Anthonyeef
force-pushed
the
yifen/session-resume
branch
from
832b553 to
fdf945e
Compare
Open
Anthonyeef
force-pushed
the
yifen/session-resume
branch
from
fdf945e to
3b747ad
Compare
Anthonyeef
changed the title
Contributor
|
nice job! but if you run For logic, at least codex have no comment. |
- Add session_to_workdir mapping in Metadata for fast session lookup - Implement Session.load_by_id() with metadata lookup and filesystem fallback - Add --resume CLI option (short form: -r) to resume any session by its UUID - Validate session ID format and ensure mutual exclusivity with --continue - Auto-populate metadata when sessions are found via filesystem scan - Use session's original work directory when resuming by ID - Add comprehensive tests for Session.create(), Session.continue_(), and Session.load_by_id() - Update all Session methods to use KaosPath for async filesystem operations
Anthonyeef
force-pushed
the
yifen/session-resume
branch
from
3b747ad to
fbebd25
Compare
Author
|
hey @Eric-Guo thanks for the callout. fixed |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related Issue
Resolve #213
Description
Adds
--resumeoption to resume sessions by their UUID, complementing the existing--continueoption.Changes
Session.load_by_id()method with metadata lookup and filesystem fallbacksession_to_workdirmapping inMetadatafor efficient session lookup--resumeCLI option (short form:-r) with UUID validationSession.create()to populatesession_to_workdirmappingKaosPathfor async filesystem operationsTesting
All tests pass. Test coverage includes:
Usage
Resume a specific session by ID:
or