Context command to run playground

Author: award999

package.json

@@ -1527,6 +1527,11 @@
           "command": "swift.coverAllTests",
           "when": "view == projectPanel && viewItem == 'test_runnable'",
           "group": "inline@3"
+        },
+        {
+          "command": "swift.play",
+          "when": "view == projectPanel && viewItem == 'playground'",
+          "group": "inline@1"
         }
       ]
     },

src/commands.ts

@@ -51,7 +51,7 @@ import { switchPlatform } from "./commands/switchPlatform";
 import { extractTestItemsAndCount, runTestMultipleTimes } from "./commands/testMultipleTimes";
 import { SwiftLogger } from "./logging/SwiftLogger";
 import { SwiftToolchain } from "./toolchain/toolchain";
-import { PackageNode } from "./ui/ProjectPanelProvider";
+import { PackageNode, PlaygroundNode } from "./ui/ProjectPanelProvider";
 import { showToolchainSelectionQuickPick } from "./ui/ToolchainSelection";
 
 /**
@@ -153,7 +153,11 @@ export function register(ctx: WorkspaceContext): vscode.Disposable[] {
             if (!folder || !target) {
                 return false;
             }
-            return await runPlayground(folder, ctx.tasks, target);
+            return await runPlayground(
+                folder,
+                ctx.tasks,
+                PlaygroundNode.isPlaygroundNode(target) ? target.playground : target
+            );
         }),
         vscode.commands.registerCommand(Commands.CLEAN_BUILD, async () => await cleanBuild(ctx)),
         vscode.commands.registerCommand(

src/ui/ProjectPanelProvider.ts

@@ -400,21 +400,36 @@ class TargetNode {
     }
 }
 
-class PlaygroundNode {
+export class PlaygroundNode {
     constructor(
         public playground: Playground,
         private folder: FolderContext,
         private activeTasks: Set<string>
     ) {}
 
+    /**
+     * "instanceof" has a bad effect in our nightly tests when the VSIX
+     * bundled source is used. For example:
+     *
+     * ```
+     * vscode.commands.registerCommand(Commands.PLAY, async (item, folder) => {
+     *  if (item instanceof PlaygroundNode) {
+     *      return await runPlayground(item.playground);
+     *  }
+     * }),
+     * ```
+     *
+     * So instead we'll check for this set boolean property. Even if the implementation of the
+     * {@link PlaygroundNode} class changes, this property should not need to change
+     */
+    static isPlaygroundNode = (item: { __isPlaygroundNode?: boolean }) =>
+        item.__isPlaygroundNode ?? false;
+    __isPlaygroundNode = true;
+
     get name(): string {
         return this.playground.label ?? this.playground.id;
     }
 
-    get args(): string[] {
-        return [this.name];
-    }
-
     toTreeItem(): vscode.TreeItem {
         const name = this.name;
         const item = new vscode.TreeItem(this.name, vscode.TreeItemCollapsibleState.None);