src/goDebugConfiguration: let vscode resolve pick process
There was a bug that used the command specification as a "name"
argument to pass into the pick process command. This change
makes sure not to attempt to resolve the commands in
resolveDebugConfiguration.
Change-Id: Iaeb99d2447c8eaebb215babfc27489e46d024b3b
Reviewed-on: https://go-review.googlesource.com/c/vscode-go/+/322294
Trust: Suzy Mueller <suzmue@golang.org>
Trust: Hyang-Ah Hana Kim <hyangah@gmail.com>
Run-TryBot: Suzy Mueller <suzmue@golang.org>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>
diff --git a/src/goDebugConfiguration.ts b/src/goDebugConfiguration.ts
index ca42792..78a66fc 100644
--- a/src/goDebugConfiguration.ts
+++ b/src/goDebugConfiguration.ts
@@ -297,7 +297,11 @@
if (!debugConfiguration['processId'] || debugConfiguration['processId'] === 0) {
// The processId is not valid, offer a quickpick menu of all processes.
debugConfiguration['processId'] = parseInt(await pickProcess(), 10);
- } else if (typeof debugConfiguration['processId'] === 'string') {
+ } else if (
+ typeof debugConfiguration['processId'] === 'string' &&
+ debugConfiguration['processId'] !== '${command:pickProcess}' &&
+ debugConfiguration['processId'] !== '${command:pickGoProcess}'
+ ) {
debugConfiguration['processId'] = parseInt(
await pickProcessByName(debugConfiguration['processId']),
10
