Repository navigation
feat(mistral): extract PDF images as files and expose table format in Mistral Parser - #9856
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
[Medium impact] The PR appears safe to merge; no actionable issues remain from this review. SummaryAdds image files and separate-table options to Mistral Parser v3.
|
There was a problem hiding this comment.
All reported issues were addressed across 13 files
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
…e format, align image switch with tool param
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 14 files
Reply with feedback, questions, or to request a fix.
Turn on auto-fix | Re-trigger cubic
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
Summary
images(file[]) output, so an agent can pick the ones it wants and pass them onfile_namematching its stored file, andimage_base64is strippedimage_limitis clamped), a 20 MB per-image cap (bigger images are skipped, not failed), and the shared 100 MB buffered-transfer budget across all imagestable_format(markdown/html), sopages[].tablesis populated instead of always empty; default stays inlineparseOptionalNumberInputmistral_parser_v3extracts images; v1/v2 never request them, and the knowledge-base OCR path (executeMistralParse) is unchangedType of Change
Testing
lib/internal/mistral/images.test.tsand 4 new cases inoperations.test.ts: base64 stripped and files named, 100-image cap, oversized image skipped,image_limitclamp (above cap / unset / zero), v3-only gate. Each guard was reverted and its test went redbunx vitest run lib/internal/mistral tools/mistral blocks— 851 passedcheck-block-registry.ts origin/staging,check:canvas-sentences,check:integration-catalog,check:docs; regenerated tool metadata and integration docslint:check+type-checkvia the pre-push hook; fullcheck:auditsandbun run testleft to CIChecklist
test-auditauthoring gate)