address comments

Author: icecrasher321

apps/sim/lib/execution/event-buffer.ts

@@ -309,20 +309,30 @@ async function compactEventForBuffer(
 
   // SSE/replay events are size-bounded by LARGE_VALUE_THRESHOLD_BYTES. When a
   // payload that preserved UserFile base64 (e.g., for chat/streaming) exceeds
-  // the cap, drop the inline base64 so consumers can lazily re-hydrate via
-  // sim.files.readBase64. This preserves the file metadata and keeps the
-  // event flowing instead of stalling the stream.
+  // the cap, recompact the already-compacted result with base64 stripped so
+  // consumers can lazily re-hydrate via sim.files.readBase64. Recompacting the
+  // *compacted* value (not the raw event.data) lets existing LargeValueRefs
+  // pass through unchanged and avoids minting fresh storage objects for the
+  // same large fields.
   if (
     context.preserveUserFileBase64 &&
     eventDataSize !== null &&
     eventDataSize > LARGE_VALUE_THRESHOLD_BYTES
   ) {
-    compactedData = await compactExecutionPayload(event.data, {
+    const oversizedBytes = eventDataSize
+    compactedData = await compactExecutionPayload(compactedData, {
       ...baseOptions,
       preserveUserFileBase64: false,
     })
     eventData = trimFinalBlockLogsForEventData(compactedData)
     eventDataSize = getJsonSize(eventData)
+    logger.warn('Stripped inline UserFile base64 from execution event to fit size limit', {
+      executionId: baseOptions.executionId,
+      eventType: 'type' in event ? event.type : undefined,
+      thresholdBytes: LARGE_VALUE_THRESHOLD_BYTES,
+      originalBytes: oversizedBytes,
+      strippedBytes: eventDataSize,
+    })
   }
 
   if (eventDataSize !== null && eventDataSize > LARGE_VALUE_THRESHOLD_BYTES) {