Skip to content

fix(docling): strip page images from structured_document; emit as page_images sibling#69

Open
pinglin wants to merge 1 commit into
mainfrom
fix-docling-strip-page-images
Open

fix(docling): strip page images from structured_document; emit as page_images sibling#69
pinglin wants to merge 1 commit into
mainfrom
fix-docling-strip-page-images

Conversation

@pinglin

@pinglin pinglin commented Jun 24, 2026

Copy link
Copy Markdown
Member

Page images (~60 KB base64 PNG each) were embedded in structured_document, bloating it to ~65 KB. Pipeline proto field limits caused it to be dropped → evidenceTreeBytes stayed 0. Strips them from the tree (keeps it ~3 KB), emits as a sibling page_images list.

🤖 Generated with Claude Code

…e_images sibling

Page images (~60 KB base64 PNG each) were embedded inside structured_document,
bloating it to ~65 KB. Pipeline proto field limits caused it to be dropped →
evidenceTreeBytes stayed 0 for all Docling-processed files. Fix:
- strip pages[*].image from the tree before json.dumps (keeps it ~3 KB)
- emit page_images: [{page_no, uri, width, height}] as a sibling field
  (available to the trigger caller; not passed through the pipeline)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant