Auth API specs two labels and uses both

[jrieken]

The current API allows to define a label in package.json and when registering. I believe the former is enough but I am sure that in case both are needed one should win

"authentication": [
	{
		"id": "autho",
		"label": "[Autho(PGJ)]"
	}
],

 vscode.authentication.registerAuthenticationProvider('autho', '[Autho]', new Autho());

[jrieken]

required for #88309

[RMacfarlane]

Augh I forgot about this problem.

My current thinking is we should get rid of the package.json registration. We needed that for exposing a providers property on authentication that was synchronous, but I'm not sure that is actually needed. I think it does have value for extension authors to learn what provider ids to use, but I think the use case of checking it at runtime is rare.

[RMacfarlane]

The label specified in registerAuthenticationProvider is now used everywhere

a5a8392