Code action constantly suggests 'Update the Table of Contents', even when it is updated

[David-Else]

Cool language server! :)

With marksman 2022-08-19 on Linux and Neovim I create a table of contents. I have nvim-lightbulb installed. Now the lightbulb is constantly lit (wherever my cursor is) telling me there is a code action to 'Update the Table of Contents', even when non of the headings have changed.

Is it possible to only announce this code action when a relevant item has changed? Thanks.

[artempyanykh]

@David-Else yeah, we definitely should do it! @keynmol do you think you'll get to #22 soonish? If not, I can take a stab at this issue myself.

[keynmol]

Once #22 is merged and no further regressions are found (the whole damn thing is a regression), I can take a look at this
I agree in principle that code action shouldn't be offered if both statements are true:

a) it was detected in the workspace
b) the text and in-memory TOC result are the same

It will be a bit harder than I initially assumed because on first opening of the document one will need to parse and reconstruct the TOC, and do that on every document change (because nothing stops you from manually modifying the TOC)

[David-Else]

So it will auto-update the TOC on save? That would be better, then there is no need for the update TOC code action. Nobody wants a wrong TOC, and having to remember to update it all the time, auto-update is a much better solution :)

[artempyanykh]

So it will auto-update the TOC on save?

That's a good point. I guess we could hook into willSaveWaitUntil request to do the necessary TOC update on save. But I think we'll still need a standalone code action too.

[keynmol]

Yeah, it can be a configurable thing - I'm about 50/50 on automatically updated TOC, as I prefer to be in control of edits sent from the LSP, but also I like the convenience.

[artempyanykh]

@keynmol you gotta let go and submit to the machines man 😉

Seriously tho, I'm fine with a config in this case but we need to keep in mind that every config option adds more variability to the server's behavior, hence less testing and eyeballing for a particular set of configurations.

[artempyanykh]

The original problem with constantly suggesting to "Update TOC" is resolved now. Updating TOC "on save" feels like a separate feature. @David-Else feel free to open an issue for this if it's something you care about.

[David-Else]

Cheers! I will test it when it gets released in a new version. You can close the issue, or I will after I test the fix.