chore: make workspace ID required when deleting workspaces

thedadams · thedadams · commit 8ef041284cbe · 2024-10-28T07:51:17.000-04:00
Signed-off-by: Donnie Adams &lt;donnie@acorn.io&gt;
diff --git a/src/gptscript.ts b/src/gptscript.ts
@@ -473,10 +473,11 @@ export class GPTScript {
         return out.trim()
     }
 
-    async deleteWorkspace(workspaceID?: string): Promise<void> {
+    async deleteWorkspace(workspaceID: string): Promise<void> {
         if (!workspaceID) {
-            workspaceID = process.env.GPTSCRIPT_WORKSPACE_ID ?? ""
+            return Promise.reject("workspace ID cannot be empty")
         }
+
         await this.runBasicCommand("workspaces/delete", {
             id: workspaceID,
             workspaceTool: this.opts.WorkspaceTool,

Original file line number	Diff line number	Diff line change
`@@ -473,10 +473,11 @@ export class GPTScript {`
`473`	`473`	`return out.trim()`
`474`	`474`	`}`
`475`	`475`
`476`		`- async deleteWorkspace(workspaceID?: string): Promise<void> {`
	`476`	`+ async deleteWorkspace(workspaceID: string): Promise<void> {`
`477`	`477`	`if (!workspaceID) {`
`478`		`- workspaceID = process.env.GPTSCRIPT_WORKSPACE_ID ?? ""`
	`478`	`+ return Promise.reject("workspace ID cannot be empty")`
`479`	`479`	`}`
	`480`	`+`
`480`	`481`	`await this.runBasicCommand("workspaces/delete", {`
`481`	`482`	`id: workspaceID,`
`482`	`483`	`workspaceTool: this.opts.WorkspaceTool,`