fix(files): bound decoded text for encoded responses

Author: waleedlatif1

apps/sim/lib/uploads/client/text-content.test.ts

@@ -71,10 +71,20 @@ describe('bounded file text decoding', () => {
     }
   )
 
-  it('accepts the exact limit and empty responses', async () => {
-    await expect(
-      readFileText(new Response('a'.repeat(MAX_TEXT_PREVIEW_BYTES)))
-    ).resolves.toHaveLength(MAX_TEXT_PREVIEW_BYTES)
+  it.each([undefined, 'gzip'])(
+    'accepts the exact decoded limit with encoding %s',
+    async (encoding) => {
+      const headers = new Headers({
+        'content-length': String(MAX_TEXT_PREVIEW_BYTES + (encoding ? 100 : 0)),
+      })
+      if (encoding) headers.set('content-encoding', encoding)
+      await expect(
+        readFileText(new Response('a'.repeat(MAX_TEXT_PREVIEW_BYTES), { headers }))
+      ).resolves.toHaveLength(MAX_TEXT_PREVIEW_BYTES)
+    }
+  )
+
+  it('accepts empty responses', async () => {
     await expect(readFileText(new Response(null))).resolves.toBe('')
   })
 })

apps/sim/lib/uploads/client/text-content.ts

@@ -13,9 +13,12 @@ export class FileTextSizeError extends Error {
   }
 }
 
-/** Bounds text previews even when storage omits or underreports Content-Length. */
+/** Bounds decoded text; Content-Length measures wire bytes for encoded responses. */
 export async function readFileText(response: Response): Promise<string> {
-  if (Number(response.headers.get('content-length')) > MAX_TEXT_PREVIEW_BYTES) {
+  if (
+    !response.headers.has('content-encoding') &&
+    Number(response.headers.get('content-length')) > MAX_TEXT_PREVIEW_BYTES
+  ) {
     await response.body?.cancel().catch(() => {})
     throw new FileTextSizeError()
   }