fix(wrangler): fix r2 data catalog snapshot expiration api call payload (#11764)

Author: terakoya76

.changeset/fix-r2-catalog-snapshot-expiration-field-names.md

@@ -0,0 +1,14 @@
+---
+"wrangler": patch
+---
+
+Fix R2 Data Catalog snapshot-expiration API field names
+
+The `wrangler r2 bucket catalog snapshot-expiration enable` command was sending incorrect field names
+to the Cloudflare API, resulting in a 422 Unprocessable Entity error. This fix updates the API request
+body to use the correct field names:
+
+- `olderThanDays` -> `max_snapshot_age` (as duration string, e.g., "30d")
+- `retainLast` -> `min_snapshots_to_keep`
+
+The CLI options (`--older-than-days` and `--retain-last`) remain unchanged.

packages/wrangler/src/__tests__/r2/bucket.test.ts

@@ -1666,8 +1666,8 @@ describe("r2", () => {
 									expect(body).toEqual({
 										snapshot_expiration: {
 											state: "enabled",
-											olderThanDays: 30,
-											retainLast: 5,
+											max_snapshot_age: "30d",
+											min_snapshots_to_keep: 5,
 										},
 									});
 									return HttpResponse.json(
@@ -1717,8 +1717,8 @@ describe("r2", () => {
 									expect(body).toEqual({
 										snapshot_expiration: {
 											state: "enabled",
-											olderThanDays: 60,
-											retainLast: 10,
+											max_snapshot_age: "60d",
+											min_snapshots_to_keep: 10,
 										},
 									});
 									return HttpResponse.json(

packages/wrangler/src/r2/helpers/catalog.ts

@@ -108,10 +108,10 @@ type R2CatalogSnapshotExpirationConfig = {
 	state: "enabled" | "disabled";
 
 	// If undefined, the service will set the default value
-	olderThanDays?: number;
+	max_snapshot_age?: string;
 
 	// If undefined, the service will set the default value
-	retainLast?: number;
+	min_snapshots_to_keep?: number;
 };
 
 type R2CatalogSnapshotExpirationResponse = {
@@ -127,8 +127,8 @@ export async function enableR2CatalogSnapshotExpiration(
 ): Promise<R2CatalogSnapshotExpirationResponse> {
 	const config: R2CatalogSnapshotExpirationConfig = {
 		state: "enabled",
-		olderThanDays: olderThanDays ?? 30,
-		retainLast: retainLast ?? 5,
+		max_snapshot_age: `${olderThanDays ?? 30}d`,
+		min_snapshots_to_keep: retainLast ?? 5,
 	};
 
 	return await fetchResult(