Align "editor.suggest.snippetsPreventQuickSuggestions" with VSCode defaults

[sebnyberg]

Setup

No Go settings. The global setting editor.suggest.snippetsPreventQuickSuggestions set to true.

Bug description

While inside placeholders of snippets in Go, VSCode suggest auto-completions even though the global setting editor.suggest.snippetsPreventQuickSuggestions is set to true.

Steps to reproduce:

Create the snippet

	"Example": {
		"scope": "go",
		"prefix": "ex",
		"body": [
			"type $1 struct {}$0",
		],
	}

Open a Go file, type "ex" and tab three times, yielding:

type type type  struct {} struct {} struct {}

Discussion

By default in VSCode editor.suggest.snippetsPreventQuickSuggestions is set to true. However, in version 0.25, vscode-go introduced an override of the global default value to false as per #839.

The confusion is two-fold:

• The behaviour of vscode-go does not align with the behaviour of VSCode and other language packages
• Setting the global value has no effect

In principle, default behaviour should be coherent between languages. I don't see why snippet expansion should work differently for Go compared to other languages. For those who use only Go, there's potentially a 50/50 split between those who are happy about the setting and those who are not. In reality however, most VSCode users work with more than one language, and this change forces all of them to learn about this setting.

I therefore propose that the default setting override introduced in v0.25 is reverted.

Edit: changed proposed solution to align with title and actual concern.

[hyangah]

Like editor.formatOnSave, editor.suggest.snippetsPreventQuickSuggestions is owned and used by the vscode, so this extension has no control over how this setting is used, other than suggesting the best setting for Go development using configurationDefaults which is what the change for #839 did.

Please override the Go-specific setting by using

"[go]": {
   "editor.suggest.snippetsPreventQUickSuggestsions": false
 }  

In the mean time, how about upvoting microsoft/vscode#58995 to make this language-specific setting default more visible to users?

[sebnyberg]

This bug probably failed to describe the real issue. Let me add some context.

My personal experience was that snippets suddenly started expanding stuff when while tabbing between fields. I had never seen this behaviour before. I assumed it to be a bug.

After months of mild frustration I realised that the "bug" was not being addressed, so I started googling for answers. I found the setting editor.suggest.snippetsPreventQuickSuggestions, and that it can be toggled on/off in the settings. Toggling the setting did not affect the behaviour, which confirmed my hypothesis that it was a bug.

I tried other languages to locate the bug, and I noticed that only Go had it. I dissected VSCode versions until I found the breaking release. That's when I found #839.

If you set up projects in Rust, JS, Python, C# and Go, then the only language that behaves in this manner is Go.

No matter how many people would've preferred this setting to diverge from the global default, I still do not think that such changes are warranted.

And even if an extension's custom default values were more visible in the settings UI, it would not have helped. This was so orthogonal with my usual experience in VSCode that it never once crossed my mind that Go might have custom default values.

I highly suggest that this change is reverted so that the default value is aligned with the global default.

[sebnyberg]

Here's some more reasoning in case it helps make my case.

For all (boolean) preferences in VSCode, there are two types of people: those who prefer it on, and those who prefer it off. When the default value aligns with a person's preference - things just happen to work out of the box.

For those who prefer it the other way, if the setting is important / annoying enough, they might Google their way to the setting's name and find out how it can be changed. This is what happened in #839 .

When the default value of a toggle is flipped, all users who had never known about the setting are forced to learn about it. On top of that, in this scenario those users also have to somehow know to search for the Go issue tracker to find out that Go's values diverge from the rest of the VSCode ecosystem.

Another side-effect of having a Go-specific default is that any user who uses snippets in Go and another language would by necessity have to learn about this setting.

I hope that you see my point here. It is applicable to not only this setting but the vast majority of preferences. It is also independent of how many think that setting should have a certain default or not - a global "bad" consistent behaviour is better than inconsistent "sometimes good" behaviour.

[hyangah]

@sebnyberg I am sorry for your frustration, and thanks for sharing your thought. We changed the default after observing many users who struggled with completion behavior and thinking about it for months. We don't take the default change lightly.

As discussed in microsoft/vscode#5844 and eventually in microsoft/vscode#58995, surfacing this language-specific change better in VS Code UI could help users who use snippets and know about this setting understand what is going on immediately. Once the language specific defaults are understood, this becomes the "consistent" behavior.

#839 landed in many months ago. Changes in other settings were many years old. Reverting them now would affect many more users. Instead, we will look for a way to improve the visibility from the extension side.

Thanks for reporting the issue - I hope users like you now might google their way and find the culprit easily through this issue.