chore(textract): drop redundant inline comments

Author: waleedlatif1

apps/sim/app/api/tools/textract/shared.test.ts

@@ -139,7 +139,6 @@ describe('pollTextractJob', () => {
             NextToken: 'next',
           }
         }
-        // Follow-up pages from AWS often omit DocumentMetadata/model-version fields.
         return { JobStatus: 'SUCCEEDED', Blocks: [{ Id: '2' }] }
       },
       (accumulated, page) => ({

apps/sim/app/api/tools/textract/shared.ts

@@ -75,8 +75,6 @@ export function mapTextractSdkError(
     return new TextractRouteError(`This document format is not supported.${hint}`, 400)
   }
 
-  // No status at all (e.g. a network failure before AWS responded) is a server-side problem, not a
-  // bad request — default to 500 so tool-execution retry logic still treats it as retryable.
   const status = err.$metadata?.httpStatusCode ?? 500
   return new TextractRouteError(err.message || 'Textract API error', status)
 }
@@ -91,6 +89,7 @@ export type ResolveDocumentResult =
   | { ok: true; document: ResolvedDocument }
   | { ok: false; response: NextResponse }
 
+/** Passes through the document host's real HTTP status on failure, so tool-execution retry logic can still treat a transient 5xx as retryable. */
 async function fetchDocumentBytes(url: string): Promise<{ bytes: Buffer; contentType: string }> {
   const urlValidation = await validateUrlWithDNS(url, 'Document URL')
   if (!urlValidation.isValid) {
@@ -102,8 +101,6 @@ async function fetchDocumentBytes(url: string): Promise<{ bytes: Buffer; content
   })
   if (!response.ok) {
     await response.text().catch(() => {})
-    // Pass through the document host's real status (e.g. a transient 503) instead of a
-    // hardcoded 400, so tool-execution retry logic can still treat 5xx as retryable.
     throw new TextractRouteError(
       `Failed to fetch document: ${response.statusText}`,
       response.status

apps/sim/blocks/blocks/textract.ts

@@ -407,12 +407,10 @@ export const TextractV2Block: BlockConfig<TextractParserOutput> = {
           }
           parameters.s3Uri = params.s3Uri.trim()
         } else if (operation === 'analyze_expense') {
-          // textract_analyze_expense accepts filePath as a fallback for URL-based documents.
           const resolved = resolveDocumentParam(params.document)
           if (!resolved.file && !resolved.filePath) throw new Error('Document file is required')
           Object.assign(parameters, resolved)
         } else {
-          // textract_parser_v2 (analyze_document) has no filePath param — file only.
           const file = normalizeFileInput(params.document, { single: true })
           if (!file) throw new Error('Document file is required')
           parameters.file = file