Skip to content

RFD: Session Delete #395

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
benbrandt merged 5 commits into from
Feb 4, 2026
Merged

RFD: Session Delete #395

benbrandt merged 5 commits into from
Feb 4, 2026
+151 −1

Conversation

@chazcb
Copy link
Contributor

@chazcb chazcb commented Jan 24, 2026

Summary

  • Proposes a capability-gated session/delete method to complement session/list
  • Gives users control over which sessions appear in their session history
  • Implementation-defined storage behavior (soft/hard delete) and load behavior for deleted sessions

Follows up on the session/list RFD which explicitly deferred deletion.

🤖 Generated with Claude Code

@chazcb chazcb marked this pull request as ready for review January 26, 2026 16:55
@chazcb chazcb requested a review from a team as a code owner January 26, 2026 16:55
nikomatsakis
nikomatsakis approved these changes Jan 27, 2026
View reviewed changes
Copy link
Contributor

@nikomatsakis nikomatsakis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've definitely wanted this.

benbrandt
benbrandt requested changes Feb 3, 2026
View reviewed changes
Copy link
Member

@benbrandt benbrandt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small note, otherwise I think looks good and we can move this one forward.

@chazcb chazcb requested a review from benbrandt February 3, 2026 16:40
@benbrandt benbrandt enabled auto-merge (squash) February 4, 2026 11:12
@benbrandt benbrandt merged commit 158f01d into agentclientprotocol:main Feb 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nikomatsakis nikomatsakis nikomatsakis approved these changes

@benbrandt benbrandt benbrandt approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@chazcb @nikomatsakis @benbrandt