fix: signature help spam by caching active parameter

[warcholjakub]

Summary

Fixes excessive signature help requests by implementing caching of the active parameter index.

[rochala]

The PR looks good and we can merge it but the current solution looks like could be greatly simplified just by adding check here pseudocode:

if (tr.changedText.contains("(",")"....) then getSignatureTooltips(tr.state) else tooltips

[rochala]

Do you mind checking if that is the case ? If not let me know and we will merge current solution

[warcholjakub]

Yeah, the current solution needs to be changed imo. I'm working on one.

[warcholjakub]

Do you mind checking if that is the case ? If not let me know and we will merge current solution

Ok, I'll check if that's the case.

[rochala]

Did it work ? I like the current solution but parsing them seems like and overkill

[warcholjakub]

I can't remember exactly, but I do recall, that afaik it did not fix the underlying problem of the previous solution. It didn't work well with the nested braces. But I can test it again if you want - just to be sure.

[warcholjakub]

Ah now I remember what was the problem. This solution doesn't work well with nested brackets. Another issue was that it would not display signatureHelp when moving text cursor without actually inserting brackets.