AIChat broken with nobuflisted set

[Konfekt]

See #203 for the discussion and #207 for a possible fix. To recall, :AIChat,:close,:AIChat should reopen the closed buffer instead of creating a new one, also with

let &rtp = '~/.vim/plugged/vim-ai' . ',' . &rtp
filetype plugin indent on
syntax on
set hidden
let g:vim_ai_chat = {
      \  "ui": {
      \    "scratch_buffer_keep_open": 1,
      \  },
      \ }

[madox2]

Please describe and provide steps how to reproduce this issue, it is not clear from referenced pull requests what is it exactly about

[Konfekt]

With the following ~/.vimrc

let &rtp = '~/.vim/plugged/vim-ai' . ',' . &rtp
filetype plugin indent on
syntax on
set hidden
let g:vim_ai_chat = {
      \  "ui": {
      \    "scratch_buffer_keep_open": 1,
      \  },
      \ }

run

vim --clean -u $HOME/.vimrc

and enter :AIChat<cr>,:close<cr>,:AIChat<cr>.

Expected: same buffer as after first invocation of :AIChat opened
Observed: another buffer is opened.

See for a record (with additional "open_chat_command": "preset_right", in "ui" and iaaa<esc> before :close.)

[madox2]

Cannot reproduce with the steps provided. When "scratch_buffer_keep_open": 1 is set, vim-ai always re-use the last (or tab-local) buffer.

The video is obviously not recorded on the .vimrc example provided above. Please double check and try it with defautl vim-ai configuration with only "scratch_buffer_keep_open": 1 to avoid misconfiguration issues.

[Konfekt]

with

let &rtp = '~/.vim/plugged/vim-ai' . ',' . &rtp
filetype plugin indent on
syntax on
set hidden
" options go here
let g:vim_ai_chat = {
      \  "ui": {
      \    "scratch_buffer_keep_open": 1,
      \    "open_chat_command": "preset_right",
      \  },
      \ }

and ZQ = :q! to close the window

[Konfekt]

Note that in actual non-minimal config :q! is not necessary but :close reproduces the same issue, just a bit tired of tracking down that setting and fixing this could already suffice

[madox2]

when you close with :q! vim unloads the buffer and it becomes gone to vim-ai, and that is the expected behavior. you should close the window with :q instead. regarding :close, by normal behaviour (and according to docs) it shouldn't unload the buffer as well.

You can also see that in your minimal config it opens the buffer with the same name ">>> AI chat" so the unloaded buffer is actually recreated.

However in your non-minimal config it opens a new instance ">>> AI chat 2" which makes me think there is some kind of misconfiguration.

[Konfekt]

Yes, I found the setting to be autocmd Filetype aichat setlocal nobuflisted as it would keep the buffer list clean when working with different aichat buffers on different tabs, as they're resurfaced by :AIChat and :ls! would show them up if need be. Is that supported?

[Konfekt]

The &hidden setting was a red herring, pardon

[Konfekt]

Since there's a ftdetect file, I suppose aichat buffers are meant to be saved as .aichat files, so in fact

let g:vim_ai_scratch_buffer_name = ">>> AI chat"
autocmd Filetype aichat autocmd BufFilePost <buffer> ++once if bufname('%') =~# '\V\^'..escape(g:vim_ai_scratch_buffer_name, '\') | setlocal nobuflisted | endif

is the more precise autocmd (the above simplified) to restrict to scratch buffers.

[madox2]

Is that supported?

No, these kinds of chat tweaks are not supported. Of course you can do it, but on your own risk. Closing