LSP: Add parentheses and cursor position when completing functions

[themartdev]

Hi!
I'm not sure how opinionated this is, but I would like for function completion to add parentheses and position the cursor inside/outside of those depending on the arity of the function:

pub fn main() {
  io.debug(<cursor>)
  wisp.configure_logger()<cursor>
}

The only case where I can think it could be a downgrade is with pipes, but maybe we can have different behavior in that context

[themartdev]

For reference, here's the behavior with gopls

Screencast.from.2024-10-06.14-34-14.webm

[lpil]

Please see prior discussion for why we do not implement this presently.

[themartdev]

@lpil Can you point me to the discussion in question? I found the older issues regarding autocompletion, but I can't find anything related to this

[giacomocavalieri]

Hello! The prior discussion happened here #3308 (comment)

[themartdev]

Thanks. I understand the point of view. Personally I think that since the frequency of using a function as a value is often much lower than calling it directly, in the end it's more annoying to add brackets most of the time vs deleting it sometimes. But it does seem like a completely subjective view, and maybe since Gleam is a functional language the proportion is not as one-sided as a language like go.

Of course the best case would be to make it context-dependant, but I don't think it's something we can expect in the short term.

Thank you!