Skip to content

feat(chat): add eca-chat-delete command#249

Merged
ericdallo merged 2 commits into
editor-code-assistant:masterfrom
rschmukler:rs/delete-tab
Jun 2, 2026
Merged

feat(chat): add eca-chat-delete command#249
ericdallo merged 2 commits into
editor-code-assistant:masterfrom
rschmukler:rs/delete-tab

Conversation

@rschmukler

Copy link
Copy Markdown
Contributor

Summary

  • Add eca-chat-delete: deletes the active chat from the server in one step, without the kill-buffer "Delete chat from server side?" prompt.
  • Resolves the current project's session and acts on its last visited chat, so it works from any buffer in the project — not just the chat buffer.
  • When the session has other chats, switches any window showing the deleted chat to another chat first, so the (dedicated) side window isn't left empty; then sends chat/delete and kills the buffer. The buffer is marked closed so the existing kill-buffer hook neither prompts nor sends a duplicate chat/delete.

Bound to C-c C-S-k (sibling to Reset's C-c C-k) and surfaced as a Delete entry in the transient menu. README + CHANGELOG updated.

Notes

Today the only one-step server delete is eca-chat-reset (delete + open a new chat). This adds a plain delete with no replacement chat, matching what you'd expect when closing a tab you no longer want.

Delete the active chat from the server in one step, without the
kill-buffer prompt. It resolves the current project's session and
acts on its last visited chat, so it works from any buffer in the
project, not only the chat buffer. When the session has other
chats, any window showing the deleted chat switches to another
chat first so the side window is not left empty.

Bound to C-c C-S-k and added as a Delete entry in the transient
menu.

@ericdallo ericdallo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good, thanks!

@ericdallo ericdallo merged commit d227cf0 into editor-code-assistant:master Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants