remove unnecessary label assignment (I think there used to be an icon there)

Author: bcpeinhardt

CHANGELOG.md

@@ -3,6 +3,7 @@
 ## Unreleased
 
 ### Added
+
 - Coder extension sidebar now displays available app statuses, and let's
   the user click them to drop into a session with a running AI Agent.
 

src/commands.ts

@@ -1,14 +1,14 @@
 import { Api } from "coder/site/src/api/api"
 import { getErrorMessage } from "coder/site/src/api/errors"
 import { User, Workspace, WorkspaceAgent } from "coder/site/src/api/typesGenerated"
+import path from "node:path"
 import * as vscode from "vscode"
 import { makeCoderSdk, needToken } from "./api"
 import { extractAgents } from "./api-helper"
 import { CertificateError } from "./error"
 import { Storage } from "./storage"
 import { toRemoteAuthority, toSafeHost } from "./util"
 import { OpenableTreeItem } from "./workspacesProvider"
-import path from "node:path"
 
 export class Commands {
   // These will only be populated when actively connected to a workspace and are
@@ -26,7 +26,7 @@ export class Commands {
     private readonly vscodeProposed: typeof vscode,
     private readonly restClient: Api,
     private readonly storage: Storage,
-  ) { }
+  ) {}
 
   /**
    * Find the requested agent if specified, otherwise return the agent if there
@@ -417,38 +417,46 @@ export class Commands {
   }): Promise<void> {
     // Launch and run command in terminal if command is provided
     if (app.command) {
-      return vscode.window.withProgress({
-        location: vscode.ProgressLocation.Notification,
-        title: `Connecting to AI Agent...`,
-        cancellable: false
-      }, async () => {
-        const terminal = vscode.window.createTerminal(app.name)
-
-        // If workspace_name is provided, run coder ssh before the command
-
-        let url = this.storage.getUrl()
-        if (!url) {
-          throw new Error("No coder url found for sidebar");
-        }
-        let binary = await this.storage.fetchBinary(this.restClient, toSafeHost(url))
-        const escape = (str: string): string => `"${str.replace(/"/g, '\\"')}"`
-        terminal.sendText(`${escape(binary)} ssh --global-config ${escape(
-          path.dirname(this.storage.getSessionTokenPath(toSafeHost(url))),
-        )} ${app.workspace_name}`)
-        await new Promise((resolve) => setTimeout(resolve, 5000))
-        terminal.sendText(app.command ?? "")
-        terminal.show(false)
-      });
+      return vscode.window.withProgress(
+        {
+          location: vscode.ProgressLocation.Notification,
+          title: `Connecting to AI Agent...`,
+          cancellable: false,
+        },
+        async () => {
+          const terminal = vscode.window.createTerminal(app.name)
+
+          // If workspace_name is provided, run coder ssh before the command
+
+          const url = this.storage.getUrl()
+          if (!url) {
+            throw new Error("No coder url found for sidebar")
+          }
+          const binary = await this.storage.fetchBinary(this.restClient, toSafeHost(url))
+          const escape = (str: string): string => `"${str.replace(/"/g, '\\"')}"`
+          terminal.sendText(
+            `${escape(binary)} ssh --global-config ${escape(
+              path.dirname(this.storage.getSessionTokenPath(toSafeHost(url))),
+            )} ${app.workspace_name}`,
+          )
+          await new Promise((resolve) => setTimeout(resolve, 5000))
+          terminal.sendText(app.command ?? "")
+          terminal.show(false)
+        },
+      )
     }
     // Check if app has a URL to open
     if (app.url) {
-      return vscode.window.withProgress({
-        location: vscode.ProgressLocation.Notification,
-        title: `Opening ${app.name || "application"} in browser...`,
-        cancellable: false
-      }, async () => {
-        await vscode.env.openExternal(vscode.Uri.parse(app.url!))
-      });
+      return vscode.window.withProgress(
+        {
+          location: vscode.ProgressLocation.Notification,
+          title: `Opening ${app.name || "application"} in browser...`,
+          cancellable: false,
+        },
+        async () => {
+          await vscode.env.openExternal(vscode.Uri.parse(app.url!))
+        },
+      )
     }
 
     // If no URL or command, show information about the app status

src/workspacesProvider.ts

@@ -251,7 +251,7 @@ export class WorkspaceProvider implements vscode.TreeDataProvider<vscode.TreeIte
             if (app.statuses && app.statuses.length > 0) {
               for (const status of app.statuses) {
                 // Show all statuses, not just ones needing attention.
-                // We need to do this for now because the reporting isn't super accurate 
+                // We need to do this for now because the reporting isn't super accurate
                 // yet.
                 appStatuses.push(
                   new AppStatusTreeItem({
@@ -437,11 +437,6 @@ class AgentTreeItem extends OpenableTreeItem {
       agent.expanded_directory,
       "coderAgent",
     )
-
-    if (agent.apps && agent.apps.length > 0) {
-      // Add an icon to indicate this agent has running apps
-      this.label = this.label
-    }
   }
 }