Allow users to delete chat entries.

[Prezombie]

It's rather annoying how there's no way to delete single chat entries, only edit your own or regenerate the AI response. Especially considering loading an old chat involves fiddling with every node that was edited or regenerated to get to the current position.

[nsarrazin]

Hi, I think that makes sense, if someone feels like tackling it feel free! Otherwise I might take a look when I have time

[omesh-omg]

hello @nsarrazin and @Prezombie can I work on this

[nsarrazin]

Sure thing! Would be a nice feature to see.

How about showing a trash icon if a message has more than 1 children ?

Then you would need a deleteBranch form action in src/routes/conversation/[id]/+page.server.ts which sends the message ID in the form

The delete should delete every message in conversation.messages that contain the message ID in their ancestors property, as well as the message itself. That should do the trick! Let me know if you need more details

[omesh-omg]

just built the project locally and went through the code do we have a icon library to use for the delete icon or should I use some random icon there?

[nsarrazin]

I think you can use the trashcan icon we use for deleting conversations

[krakenftw]

I have added the basic things needed and created the action, my question is how do i send the Message.id form page to deleteBranch. stuck here : /

[nsarrazin]

I have added the basic things needed and created the action, my question is how do i send the Message.id form page to deleteBranch. stuck here : /

You could wrap the button in a form with an input type='hidden' with name messageId and value message.id to send the data to the form action.

Feel free to open a draft PR and I can help review there 😄

[krakenftw]

wanted a bit of clarification on this, does deleteBranch delete the whole message including the regenerated ones?

[nsarrazin]

Just the current branch, so if you're on 1/12 for example, afterwards there should only be 11 branches left.

But it should delete any message that comes after the one you're deleting in the DB (since they can't be shown anyway, that's why you filter on the ancestors property). And the option shouldn't be visible if you only have one branch. (only show if you see the < 1/2 > indicator)

[krakenftw]

can you have a look at this once? @nsarrazin
#1239

[omesh-omg]

okay @krakenftw you can work on this