ack PR comments

Author: waleedlatif1

apps/sim/app/api/workflows/[id]/route.ts

@@ -228,6 +228,21 @@ export async function DELETE(
       return NextResponse.json({ error: 'Access denied' }, { status: 403 })
     }
 
+    // Check if this is the last workflow in the workspace
+    if (workflowData.workspaceId) {
+      const totalWorkflowsInWorkspace = await db
+        .select({ id: workflow.id })
+        .from(workflow)
+        .where(eq(workflow.workspaceId, workflowData.workspaceId))
+
+      if (totalWorkflowsInWorkspace.length <= 1) {
+        return NextResponse.json(
+          { error: 'Cannot delete the only workflow in the workspace' },
+          { status: 400 }
+        )
+      }
+    }
+
     // Check if workflow has published templates before deletion
     const { searchParams } = new URL(request.url)
     const checkTemplates = searchParams.get('check-templates') === 'true'

apps/sim/app/workspace/[workspaceId]/w/hooks/use-can-delete.ts

@@ -95,28 +95,28 @@ export function useCanDelete({ workspaceId }: UseCanDeleteProps): UseCanDeleteRe
   )
 
   /**
-   * Check if the given workflow IDs can be deleted
+   * Check if the given workflow IDs can be deleted.
+   * Returns false if deleting would remove all workflows from the workspace.
    */
   const canDeleteWorkflows = useCallback(
     (workflowIds: string[]): boolean => {
-      if (totalWorkflows === 0) return true
-
       const workflowsToDelete = workflowIds.filter((id) => workflowIdSet.has(id)).length
 
-      return workflowsToDelete < totalWorkflows
+      // Must have at least one workflow remaining after deletion
+      return totalWorkflows > 0 && workflowsToDelete < totalWorkflows
     },
     [totalWorkflows, workflowIdSet]
   )
 
   /**
-   * Check if the given folder can be deleted
+   * Check if the given folder can be deleted.
+   * Empty folders are always deletable. Folders containing all workspace workflows are not.
    */
   const canDeleteFolder = useCallback(
     (folderId: string): boolean => {
-      if (totalWorkflows === 0) return true
-
       const workflowsInFolder = countWorkflowsInFolder(folderId)
 
+      if (workflowsInFolder === 0) return true
       return workflowsInFolder < totalWorkflows
     },
     [totalWorkflows, countWorkflowsInFolder]