Feature flag request: Separate argument injection from snippets

[peteruhnak]

Would it be possible to separate argument injection into its own config independent of enable_snippets?

With enable_snippets enabled, accepting a function completion also injects the function's args and types,

e.g.

exe.addCS

is completed to

exe.addCSourceFile(file: []const u8, flags: []const []const u8)

instead of

exe.addCSourceFile()

I'm sure this is quite useful to some people and some code editors, but it seems like a separate functionality to be toggled independently (and to me it is quite annoying).

So basically change the code here to use different flag, and add docs for it & everything

zls/src/Server.zig

Line 498 in 8eeae02

const use_snippets = server.config.enable_snippets and server.client_capabilities.supports_snippets;

I can create a PR if you are ok with the change.

[Techatrix]

Sounds like a good idea.
Because its sometimes easy to forget editing all locations for a new config options, i will list them here which may help you:

1. zls/src/Config.zig

   Line 14 in 8cf93fc

   enable_snippets: bool = false,

2. zls/src/requests.zig

   Line 290 in 8cf93fc

   enable_snippets: ?bool,

3. zls/schema.json

   Line 7 in e776a5c

   "enable_snippets": {

4. zls/README.md

   Line 106 in a70beba

   | `enable_snippets` | `bool` | `false` | Enables snippet completions when the client also supports them. |

5. zls/src/setup.zig

   Line 172 in e776a5c

   const snippets = try askBool("Do you want to enable snippets?");