[BUG] comment-line sometimes does not comment filetype=sh

[sdothum]

[BUG] comment-line sometimes does not comment filetype=sh

[krobelus]

there is at least one issue when the filetype is first something like markdown and then sh

Can you reproduce wiht this in your kakrc:

hook global BufSetOption filetype=.* %{
    echo -debug setting %val{hook_param}
}
hook global BufSetOption comment_line=.* %{
    echo -debug setting %val{hook_param}
}

and post the contents of the debug buffer.

The conf hook is already a hack; ideally resetting the option when a filetype gets unset works for all filetypes, even the empty one.
We have the same problem with some other hooks.
One problem is hook ordering, maybe we should have a hook -first option so "scope end" style hooks run before others.

[sdothum]

Here is the requested debug output..

*** This is the debug buffer, where debug info will be written ***
setting comment_line=
setting filetype=html
setting filetype=sh
setting filetype=sh
setting comment_line=
kak-lsp: config-change detected: 
. . .

with the stock comment.kak (without the hook for filetype=sh that i added) opening a script which sets the filetype with modeline-prompt.

i opened this file and attempted to immediately comment the first line.

[krobelus]

I'm working on a fix, my current approach is to add a BufSetOptionPre hook so each filetype hook can unset what it did without BufSetOption hooks having to worry.