Make "args": "${command:pickArgs}" as default debug configuration

[TCPsoftware]

This PR is related to these issues:
microsoft/vscode#225834
microsoft/vscode#232560
#233
#465
#497

I searched repo:microsoft/vscode-python-debugger command:pickArgs filtered by code, and find four snippet in thee files.

vscode-python-debugger/src/extension/debugger/configuration/providers/fileLaunchWithArgs.ts

Lines 21 to 27 in 1b6ac0d

	request: 'launch',
	program: '${file}',
	console: 'integratedTerminal',
	args: ['${command:pickArgs}'],
	};
	sendTelemetryEvent(EventName.DEBUGGER_CONFIGURATION_PROMPTS, undefined, {
	configurationType: DebugConfigurationType.launchFileWithArgs,

vscode-python-debugger/src/test/unittest/configuration/providers/fileLaunchWithArgs.unit.test.ts

Lines 28 to 34 in 1b6ac0d

	request: 'launch',
	program: '${file}',
	console: 'integratedTerminal',
	args: ['${command:pickArgs}'],
	};
	expect(state.config).to.be.deep.equal(config);

vscode-python-debugger/package.json

Lines 286 to 306 in 1b6ac0d

	"args": {
	"default": [],
	"description": "Command line arguments passed to the program. For string type arguments, it will pass through the shell as is, and therefore all shell variable expansions will apply. But for the array type, the values will be shell-escaped.",
	"items": {
	"type": "string"
	},
	"anyOf": [
	{
	"default": "${command:pickArgs}",
	"enum": [
	"${command:pickArgs}"
	]
	},
	{
	"type": [
	"array",
	"string"
	]
	}
	]
	},

In lines 286 to 306 of package.json, it shows three usages of the args parameter. The first is that the value of the args parameter is "${command:pickArgs}", the second is to provide an array, and the third is to provide a string.

In the vscode documentation, it is also mentioned:

Note: There is a difference in how "${command:pickArgs}" and ["${command:pickArgs}"] are parsed, with specific notice to the usage of []. As an array, all arguments are passed as a single string, without brackets each argument is passed as its own string.

Therefore, I think the two files fileLaunchWithArgs.ts and fileLaunchWithArgs.unit.test.ts are the key factors affecting this problem. Then I delete the [ ] in these two code snippets.

---

When I wrote this PR, I checked the file modification history and found that the modification in PR #385 added [ ], but this was completely wrong (refer to the discussion of issue #497), and was not the correct way to solve issue #233 (PR #499 solved the problem), and led to a series of issues listed at the beginning.

Just consider this PR to have coincidentally performed a "revert" operation.

[TCPsoftware]

@microsoft-github-policy-service agree