fix(parser): set etag optional for delete object notifications (#2429)

* set etag optional, for delete object notifications

* remove etag and size from the test event

* add comment why etag and size is optional

---------

Co-authored-by: Andrea Amorosi <[email protected]>

Author: am29d

Diff for: packages/parser/src/schemas/s3.ts

@@ -72,8 +72,8 @@ const S3EventNotificationEventBridgeDetailSchema = z.object({
   }),
   object: z.object({
     key: z.string(),
-    size: z.number().nonnegative().optional(),
-    etag: z.string(),
+    size: z.number().nonnegative().optional(), // not present in DeleteObject events
+    etag: z.string().optional(), // not present in DeleteObject events
     'version-id': z.string().optional(),
     sequencer: z.string().optional(),
   }),

Diff for: packages/parser/tests/events/s3EventBridgeNotificationObjectDeletedEvent.json

@@ -16,8 +16,6 @@
     },
     "object": {
         "key": "IMG_m7fzo3.jpg",
-        "size": 184662,
-        "etag": "4e68adba0abe2dc8653dc3354e14c01d",
         "sequencer": "006408CAD69598B05E"
     },
     "request-id": "0BH729840619AG5K",