feat: clear key mapping; default to <C-l>

[teocns]

Uses Config.windows.ask.start_insert to determine whether to go insert mode after clearing. Is this something we want, or should we create a separate config?

Clear mapping is set on mappings.sidebar.clear, but feel free to suggest a better one if this doesn't fit the sidebar context

[teocns]

@yetone have my back with the linter please

[yetone]

‌‌‌‌Should we confirm before clearing? Because in my opinion, deletion is always the most dangerous operation.

[teocns]

<C-l> is a pretty standard clear command. Perhaps one doesn't just randomly press that. Usually no confirmation is prompted (hence the "shortcut" name).

Tip

CopilotChat.nvim behaves the same way

[aarnphm]

I actually don't agree with this. This should be done manually instead of this.

delete is dangerous, share same sentiment with yetone (but it seems like you are saving it to history before deleting it, which seems a bit misleading)

I'm fine doing this through a cmd or sth

[teocns]

How about double <C-l> <C-l> + warning at first key stroke?
I understand deleting is dangerous and might require some sort of confirmation, though I wouldn't see the point in having an extra confirmation step such as pressing Y: it'd consume as much time as typing /clear, and Y is also a painful key to reach

but it seems like you are saving it to history before deleting it

how weird - I'll take a look

[aarnphm]

L1245

[teocns]

Right, I just replicated the same behavior of /clear buying a safe ticket to consistency in behaviors:

avante.nvim/lua/avante/sidebar.lua

Lines 1199 to 1208 in a8e2b9a

	clear = function(args, cb)
	local chat_history = Path.history.load(self.code.bufnr)
	if next(chat_history) ~= nil then
	chat_history = {}
	Path.history.save(self.code.bufnr, chat_history)
	self:update_content("Chat history cleared", { focus = false, scroll = false })
	vim.defer_fn(function()
	self:close()
	if cb then cb(args) end
	end, 1000)

You might wanna look at the blame history.

[teocns]

Added double keystroke for <C-l> + warning showing to press again before deleting

4778e1e

This behavior lines up with my expectation, but if you don't vision it merged feel free to leave it aside or modify it at will