improvement(api): default folder deletion to non-recursive

Author: TheodoreSpeaks

apps/docs/openapi-v2-files-audit.json

@@ -1810,7 +1810,7 @@
       "delete": {
         "operationId": "deleteFilesFolder",
         "summary": "Delete Folder",
-        "description": "Delete a folder. With `recursive=false`, the folder must be empty. With `recursive=true`, its descendant folders and resources are deleted too.",
+        "description": "Delete a folder. By default the folder must be empty. With `recursive=true`, its descendant folders and resources are deleted too.",
         "tags": ["Files"],
         "parameters": [
           {
@@ -1828,10 +1828,11 @@
           {
             "name": "recursive",
             "in": "query",
-            "required": true,
+            "required": false,
             "description": "Whether to delete the subtree.",
             "schema": {
-              "type": "boolean"
+              "type": "boolean",
+              "default": false
             }
           }
         ],

apps/docs/openapi-v2-knowledge.json

@@ -1472,7 +1472,7 @@
       "delete": {
         "operationId": "deleteKnowledgeFolder",
         "summary": "Delete Folder",
-        "description": "Delete a folder. With `recursive=false`, the folder must be empty. With `recursive=true`, its descendant folders and resources are deleted too.",
+        "description": "Delete a folder. By default the folder must be empty. With `recursive=true`, its descendant folders and resources are deleted too.",
         "tags": ["Knowledge Bases"],
         "parameters": [
           {
@@ -1490,10 +1490,11 @@
           {
             "name": "recursive",
             "in": "query",
-            "required": true,
+            "required": false,
             "description": "Whether to delete the subtree.",
             "schema": {
-              "type": "boolean"
+              "type": "boolean",
+              "default": false
             }
           }
         ],

apps/docs/openapi-v2-tables.json

@@ -4075,7 +4075,7 @@
       "delete": {
         "operationId": "deleteTablesFolder",
         "summary": "Delete Folder",
-        "description": "Delete a folder. With `recursive=false`, the folder must be empty. With `recursive=true`, its descendant folders and resources are deleted too.",
+        "description": "Delete a folder. By default the folder must be empty. With `recursive=true`, its descendant folders and resources are deleted too.",
         "tags": ["Tables"],
         "parameters": [
           {
@@ -4093,10 +4093,11 @@
           {
             "name": "recursive",
             "in": "query",
-            "required": true,
+            "required": false,
             "description": "Whether to delete the subtree.",
             "schema": {
-              "type": "boolean"
+              "type": "boolean",
+              "default": false
             }
           }
         ],

apps/docs/openapi-v2-workflows.json

@@ -1948,7 +1948,7 @@
       "delete": {
         "operationId": "deleteWorkflowsFolder",
         "summary": "Delete Folder",
-        "description": "Delete a folder. With `recursive=false`, the folder must be empty. With `recursive=true`, its descendant folders and resources are deleted too.",
+        "description": "Delete a folder. By default the folder must be empty. With `recursive=true`, its descendant folders and resources are deleted too.",
         "tags": ["Workflows"],
         "parameters": [
           {
@@ -1966,10 +1966,11 @@
           {
             "name": "recursive",
             "in": "query",
-            "required": true,
+            "required": false,
             "description": "Whether to delete the subtree.",
             "schema": {
-              "type": "boolean"
+              "type": "boolean",
+              "default": false
             }
           }
         ],

apps/sim/lib/api/contracts/v2/__tests__/shared.test.ts

@@ -1,6 +1,7 @@
 import { describe, expect, it } from 'vitest'
 import { v2ListLogsQuerySchema } from '@/lib/api/contracts/v2/logs'
 import {
+  v2DeleteFolderQuerySchema,
   v2FolderPathInputSchema,
   v2FolderPathSchema,
   v2NonRootFolderPathInputSchema,
@@ -43,6 +44,12 @@ describe('v2 folder path contracts', () => {
     ).toBe(false)
   })
 
+  it('defaults folder deletion to non-recursive', () => {
+    expect(v2DeleteFolderQuerySchema.parse({ workspaceId: WORKSPACE_ID, path: 'Reports' })).toEqual(
+      { workspaceId: WORKSPACE_ID, path: '/Reports', recursive: false }
+    )
+  })
+
   it('normalizes every folder path in the logs filter', () => {
     const query = v2ListLogsQuerySchema.parse({
       workspaceId: WORKSPACE_ID,

apps/sim/lib/api/contracts/v2/shared.ts

@@ -179,7 +179,7 @@ export const v2DeleteFolderQuerySchema = z
   .object({
     workspaceId: workspaceIdSchema,
     path: v2NonRootFolderPathInputSchema,
-    recursive: z.stringbool(),
+    recursive: z.stringbool().optional().default(false),
   })
   .strict()