updates baby

bcpeinhardt · bcpeinhardt · commit a7ed1ee9bf15 · 2024-09-09T15:12:26.000-05:00
diff --git a/src/api.ts b/src/api.ts
@@ -94,7 +94,7 @@ export async function makeCoderSdk(baseUrl: string, token: string | undefined, s
 /**
  * Start or update a workspace and return the updated workspace.
  */
-export async function startWorkspace(restClient: Api, workspace: Workspace): Promise<Workspace> {
+export async function startWorkspaceIfStoppedOrFailed(restClient: Api, workspace: Workspace): Promise<Workspace> {
   // If the workspace requires the latest active template version, we should attempt
   // to update that here.
   // TODO: If param set changes, what do we do??
@@ -107,7 +107,7 @@ export async function startWorkspace(restClient: Api, workspace: Workspace): Pro
   // Before we start a workspace, we make an initial request to check it's not already started
   const updatedWorkspace = await restClient.getWorkspace(workspace.id)
 
-  if (["starting", "running"].includes(updatedWorkspace.latest_build.status)) {
+  if (!["stopped", "failed"].includes(updatedWorkspace.latest_build.status)) {
     return updatedWorkspace
   }
 
diff --git a/src/remote.ts b/src/remote.ts
@@ -10,7 +10,7 @@ import * as path from "path"
 import prettyBytes from "pretty-bytes"
 import * as semver from "semver"
 import * as vscode from "vscode"
-import { makeCoderSdk, startWorkspace, waitForBuild } from "./api"
+import { makeCoderSdk, startWorkspaceIfStoppedOrFailed, waitForBuild } from "./api"
 import { extractAgents } from "./api-helper"
 import * as cli from "./cliManager"
 import { Commands } from "./commands"
@@ -105,7 +105,7 @@ export class Remote {
                   return undefined
                 }
                 this.storage.writeToCoderOutputChannel(`Starting ${workspaceName}...`)
-                workspace = await startWorkspace(restClient, workspace)
+                workspace = await startWorkspaceIfStoppedOrFailed(restClient, workspace)
                 break
               case "failed":
                 // On a first attempt, we will try starting a failed workspace
@@ -115,7 +115,7 @@ export class Remote {
                     return undefined
                   }
                   this.storage.writeToCoderOutputChannel(`Starting ${workspaceName}...`)
-                  workspace = await startWorkspace(restClient, workspace)
+                  workspace = await startWorkspaceIfStoppedOrFailed(restClient, workspace)
                   break
                 }
               // Otherwise fall through and error.
