Skip to content

Conversation

@Eduard-Voiculescu
Copy link

@Eduard-Voiculescu Eduard-Voiculescu commented Nov 19, 2025

Should fix #64
Followed the instructions here: https://agentclientprotocol.com/protocol/session-setup#loading-a-session
Added a very simplistic in-memory approach.

Is there any requirement to add for testing (integration, e2e, unit, etc.)?

Thanks for the great work on this!! Really appreciate it.

@cla-bot
Copy link

cla-bot bot commented Nov 19, 2025

We require contributors to sign our Contributor License Agreement, and we don't have @Eduard-Voiculescu on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@Eduard-Voiculescu
Copy link
Author

@cla-bot check

@cla-bot
Copy link

cla-bot bot commented Nov 19, 2025

We require contributors to sign our Contributor License Agreement, and we don't have @Eduard-Voiculescu on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@cla-bot
Copy link

cla-bot bot commented Nov 19, 2025

The cla-bot has been summoned, and re-checked this pull request!

@Eduard-Voiculescu
Copy link
Author

@cla-bot check

@cla-bot cla-bot bot added the cla-signed label Nov 19, 2025
@cla-bot
Copy link

cla-bot bot commented Nov 19, 2025

The cla-bot has been summoned, and re-checked this pull request!

@benbrandt
Copy link
Member

@Eduard-Voiculescu thanks for your work here! There are some ongoing conversations/work happening around session load/list/resume/fork, so before committing to support load in this way, I want to see how those implementations go. So we'll have to wait on this PR for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Feature Request: Add loadSession support

2 participants