fix(cloudflare): require explicit R2 lifecycle replacement rules

Author: waleedlatif1

apps/docs/content/docs/integrations/cloudflare.mdx

@@ -7744,7 +7744,7 @@ Replaces the object lifecycle rules for an R2 bucket. Rules match object-key pre
 | `accountId` | string | Yes | Account id |
 | `bucketName` | string | Yes | Bucket name |
 | `apiKey` | string | Yes | Cloudflare API token |
-| `rules` | json | No | JSON array. rules Fields: abortMultipartUploadsTransition, conditions, deleteObjectsTransition, enabled, id, storageClassTransitions. |
+| `rules` | json | Yes | Complete JSON array of lifecycle rules. Read the current rules first and include every rule to retain. Use \[\] only to remove all lifecycle rules. Fields: abortMultipartUploadsTransition, conditions, deleteObjectsTransition, enabled, id, storageClassTransitions. |
 | `jurisdiction` | string | No | Jurisdiction where objects in this bucket are guaranteed to be stored. |
 
 #### Output

apps/sim/blocks/blocks/cloudflare.ts

@@ -4300,14 +4300,13 @@ export const CloudflareBlock: BlockConfig = {
       id: 'r2BucketLifecycleRules',
       title: 'Rules',
       type: 'long-input',
-      placeholder: 'Enter rules as JSON',
+      placeholder: 'Enter all lifecycle rules as JSON; [] removes all rules',
       condition: { field: 'operation', value: ['update_r2_bucket_lifecycle'] },
-      required: false,
-      mode: 'advanced',
+      required: true,
       wandConfig: {
         enabled: true,
         prompt:
-          'Generate the Cloudflare rules value. JSON array. rules Fields: abortMultipartUploadsTransition, conditions, deleteObjectsTransition, enabled, id, storageClassTransitions. Return ONLY the value.',
+          'Generate the complete Cloudflare lifecycle rules JSON array. Include every existing rule the user wants to retain. Use [] only when the user wants to remove all lifecycle rules. Fields: abortMultipartUploadsTransition, conditions, deleteObjectsTransition, enabled, id, storageClassTransitions. Return ONLY the value.',
         placeholder: 'Describe the rules',
       },
     },
@@ -5302,7 +5301,7 @@ export const CloudflareBlock: BlockConfig = {
     r2BucketLifecycleRules: {
       type: 'json',
       description:
-        'JSON array. rules Fields: abortMultipartUploadsTransition, conditions, deleteObjectsTransition, enabled, id, storageClassTransitions.',
+        'Complete JSON array of lifecycle rules. Read the current rules first and include every rule to retain. Use [] only to remove all lifecycle rules. Fields: abortMultipartUploadsTransition, conditions, deleteObjectsTransition, enabled, id, storageClassTransitions.',
     },
     rulesetRulePosition: {
       type: 'json',

apps/sim/tools/cloudflare/operation-types.ts

@@ -327,7 +327,7 @@ export interface CloudflareUpdateR2BucketLifecycleParams {
   accountId: string
   bucketName: string
   apiKey: string
-  rules?: string | Record<string, unknown> | unknown[]
+  rules: string | unknown[]
   jurisdiction?: string
 }
 

apps/sim/tools/cloudflare/update_r2_bucket_lifecycle.ts

@@ -41,10 +41,10 @@ export const updateR2BucketLifecycleTool: ToolConfig<
     },
     rules: {
       type: 'json',
-      required: false,
+      required: true,
       visibility: 'user-or-llm',
       description:
-        'JSON array. rules Fields: abortMultipartUploadsTransition, conditions, deleteObjectsTransition, enabled, id, storageClassTransitions.',
+        'Complete JSON array of lifecycle rules. Read the current rules first and include every rule to retain. Use [] only to remove all lifecycle rules. Fields: abortMultipartUploadsTransition, conditions, deleteObjectsTransition, enabled, id, storageClassTransitions.',
     },
     jurisdiction: {
       type: 'string',
@@ -61,7 +61,15 @@ export const updateR2BucketLifecycleTool: ToolConfig<
       ...cloudflareHeaders(params.apiKey),
       ...(params.jurisdiction ? { 'cf-r2-jurisdiction': params.jurisdiction } : {}),
     }),
-    body: (params) => ({ rules: parseJsonArrayParam(params.rules, 'rules') }),
+    body: (params) => {
+      const rules = parseJsonArrayParam(params.rules, 'rules')
+      if (rules === undefined) {
+        throw new Error(
+          'rules is required; supply a JSON array or [] to remove all lifecycle rules'
+        )
+      }
+      return { rules }
+    },
   },
   transformResponse: async (response) => {
     const data = await readCloudflareResponse(response)