🤖 fix: delete plan file locally for local runtimes

Change-Id: I9aa88d6979c49c8e68926f85e7bc82421d67dca9
Signed-off-by: Thomas Kosiewski <tk@coder.com>

Author: ThomasK33

src/node/services/workspaceService.ts

@@ -1110,21 +1110,32 @@ export class WorkspaceService extends EventEmitter {
       ? expandTildeForSSH(legacyPlanPath)
       : shellQuote(expandTilde(legacyPlanPath));
 
-    // Delete plan files through runtime (supports both local and SSH)
-    const runtime = createRuntime(metadata.runtimeConfig, {
-      projectPath: metadata.projectPath,
-    });
-
-    try {
-      // Use exec to delete files since runtime doesn't have a deleteFile method
-      // Delete both paths in one command for efficiency
-      await runtime.exec(`rm -f ${quotedPlanPath} ${quotedLegacyPlanPath}`, {
-        cwd: metadata.projectPath,
-        timeout: 10,
+    // SSH runtime: delete via remote shell so $HOME expands on the remote.
+    if (isSSHRuntime(metadata.runtimeConfig)) {
+      const runtime = createRuntime(metadata.runtimeConfig, {
+        projectPath: metadata.projectPath,
       });
-    } catch {
-      // Plan files don't exist or can't be deleted - ignore
+
+      try {
+        await runtime.exec(`rm -f ${quotedPlanPath} ${quotedLegacyPlanPath}`, {
+          cwd: metadata.projectPath,
+          timeout: 10,
+        });
+      } catch {
+        // Plan files don't exist or can't be deleted - ignore
+      }
+
+      return;
     }
+
+    // Local runtimes: delete directly on the local filesystem.
+    const planPathAbs = expandTilde(planPath);
+    const legacyPlanPathAbs = expandTilde(legacyPlanPath);
+
+    await Promise.allSettled([
+      fsPromises.rm(planPathAbs, { force: true }),
+      fsPromises.rm(legacyPlanPathAbs, { force: true }),
+    ]);
   }
 
   async truncateHistory(workspaceId: string, percentage?: number): Promise<Result<void>> {