Agents View: support deletion of sessions

[rebornix]

Updated from @bpasero:

Today, deleting a session has gaps:

• it only really works for local sessions, not background or cloud who have their own actions and implementation
• it does not stop a running session
• it probably needs to call an API that the provider can do cleanup

Original from @rebornix:

Re #274477

In the grouped agents view, extensions contribute their own “delete” button to the session entry. This can easily fall apart, for example, when users delete a session that’s already opened in the Chat Editor, the editor still stays open even though the session is gone. Different extensions may also choose different icons for “delete”.

This makes me wonder if “Archive” / “Delete” should be part of the core API.

[bpasero]

Yes we need this. I wonder though if this is something we implement in core, or we implement as provider API. I would think archiving could maybe be core-level state, but deletion should probably be handled by providers to cleanup resources?

[bpasero]

@rebornix I am pushing changes to allow for archiving of sessions (note the toolbar on the top right of a session):

And a way to restore:

And this state is all client side and the provider is not made aware. Note that I still respect the archived state from the provider which is taken as the initial value so that we can still preserve the intent if provided.

I am however not 100% sure about "Delete": shouldn't that be something that we tell via API to the provider so that the provider can ensure to cleanup resources? Otherwise the session will likely always come back and we have to filter it out on our end.

//cc @mjbvz

[DonJayamanne]

elete": shouldn't that be something that we tell via API to the provider so that the provider

Agreed, feels weird for provider to do this
Also this way, core can delete all other related state if required.

Note that I still respect the archived state from the

Not sure how this gets used/set today. Its not used in chat extension
@osortega /cc

[bpasero]

As archiving is implemented, leaving this for the remainder: session deletion