implement snapshot delete

Author: williamstein

src/packages/frontend/project/explorer/action-bar.tsx

@@ -166,8 +166,13 @@ export function ActionBar({
   }
 
   function render_action_button(name: FileAction) {
-    if (isSnapshotPath(current_path) && isDisabledSnapshots(name)) {
-      return null;
+    if (isSnapshotPath(current_path)) {
+      if (isDisabledSnapshots(name)) {
+        return null;
+      }
+      if (current_path != SNAPSHOTS && name == "delete") {
+        return null;
+      }
     }
     const obj = file_actions[name];
     const handle_click = (_e: React.MouseEvent) => {

src/packages/frontend/project/snapshots.tsx

@@ -105,6 +105,7 @@ function CreateSnapshot() {
         }}
         footer={[
           <Button
+            key="cancel"
             onClick={() => {
               setOpen(false);
               setName("");
@@ -113,6 +114,7 @@ function CreateSnapshot() {
             Cancel
           </Button>,
           <Button
+            key="create"
             type="primary"
             onClick={createSnapshot}
             disabled={!name.trim()}
@@ -229,6 +231,7 @@ function EditSchedule() {
         }}
         footer={[
           <Button
+            key="cancel"
             onClick={() => {
               setOpen(false);
               setName("");
@@ -237,6 +240,7 @@ function EditSchedule() {
             Cancel
           </Button>,
           <Button
+            key="create"
             type="primary"
             onClick={createSnapshot}
             disabled={!name.trim()}

src/packages/frontend/project_actions.ts

@@ -111,6 +111,7 @@ import {
 import { search } from "@cocalc/frontend/project/search/run";
 import { type CopyOptions } from "@cocalc/conat/files/fs";
 import { getFileTemplate } from "./project/templates";
+import { SNAPSHOTS } from "@cocalc/util/consts/snapshots";
 
 const { defaults, required } = misc;
 
@@ -2510,8 +2511,29 @@ export class ProjectActions extends Actions<ProjectStoreState> {
     this.set_activity({ id, status: `Deleting ${mesg}...` });
 
     try {
-      const fs = this.fs(compute_server_id);
-      await fs.rm(paths, { force: true, recursive: true });
+      // delete any snapshots:
+      const snapshots: string[] = [];
+      const nonSnapshotPaths: string[] = [];
+      for (const path of paths) {
+        if (dirname(path) == SNAPSHOTS) {
+          snapshots.push(basename(path));
+        } else {
+          nonSnapshotPaths.push(path);
+        }
+      }
+      if (snapshots.length > 0) {
+        for (const name of snapshots) {
+          await webapp_client.conat_client.hub.projects.deleteSnapshot({
+            project_id: this.project_id,
+            name,
+          });
+        }
+      }
+      if (nonSnapshotPaths.length > 0) {
+        const fs = this.fs(compute_server_id);
+        await fs.rm(nonSnapshotPaths, { force: true, recursive: true });
+      }
+
       this.log({
         event: "file_action",
         action: "deleted",