test(cloudflare): cover explicit lifecycle replacement intent

Author: waleedlatif1

apps/sim/tools/cloudflare/request-boundary.test.ts

@@ -1,9 +1,57 @@
 import { describe, expect, it } from 'vitest'
 import { createDnsRecordTool } from '@/tools/cloudflare/create_dns_record'
 import { deleteDnsRecordTool } from '@/tools/cloudflare/delete_dns_record'
+import { updateR2BucketLifecycleTool } from '@/tools/cloudflare/update_r2_bucket_lifecycle'
 import { updateRulesetRuleTool } from '@/tools/cloudflare/update_ruleset_rule'
 
 describe('Cloudflare provider request boundaries', () => {
+  it.each([undefined, null, '', '   '])(
+    'rejects an R2 lifecycle replacement without explicit rules (%j)',
+    (rules) => {
+      const body = updateR2BucketLifecycleTool.request.body
+      if (!body) throw new Error('Expected a request body')
+      const wireInput = { apiKey: 'token', accountId: 'account', bucketName: 'bucket', rules }
+      expect(() => Reflect.apply(body, undefined, [wireInput])).toThrow(/rules/)
+    }
+  )
+
+  it.each([[], '[]'])('preserves an explicit R2 lifecycle clear (%j)', (rules) => {
+    const body = updateR2BucketLifecycleTool.request.body
+    if (!body) throw new Error('Expected a request body')
+    expect(body({ apiKey: 'token', accountId: 'account', bucketName: 'bucket', rules })).toEqual({
+      rules: [],
+    })
+  })
+
+  it('preserves every explicitly supplied R2 lifecycle rule', () => {
+    const body = updateR2BucketLifecycleTool.request.body
+    if (!body) throw new Error('Expected a request body')
+    const rules = [
+      {
+        id: 'expire-logs',
+        enabled: true,
+        conditions: { prefix: 'logs/' },
+        deleteObjectsTransition: { condition: { type: 'Age', maxAge: 86400 } },
+      },
+      {
+        id: 'retain-archive-transition',
+        enabled: false,
+        conditions: { prefix: 'archive/' },
+        storageClassTransitions: [
+          { condition: { type: 'Age', maxAge: 2592000 }, storageClass: 'InfrequentAccess' },
+        ],
+      },
+    ]
+    expect(
+      body({
+        apiKey: 'token',
+        accountId: 'account',
+        bucketName: 'bucket',
+        rules: JSON.stringify(rules),
+      })
+    ).toEqual({ rules })
+  })
+
   it('rejects a DNS deletion identifier that would address the collection instead', () => {
     const url = deleteDnsRecordTool.request.url
     if (typeof url !== 'function') throw new Error('Expected a parameterized URL')