firecrawl / firecrawl/pdf-inspector
Expose per-page pages_with_images signal
- Dominant language
- Rust
- Stars
- 19.1k
- Forks
- 1.3k
- Avg merge
- 9h 21m
- Merged PRs (30d)
- 51
Description
PdfClassification and PdfResult already expose per-page structural signals (pages_needing_ocr, layout.pages_with_tables, layout.pages_with_columns) so callers can make routing decisions without a second pass. One signal is conspicuously missing from that set: which pages contain embedded images. The internal detector already computes this (PageAnalysis.has_images) during its existing classification pass, but aggregates it into pdf_type and discards the per-page list before it reaches the public API.
This signal is useful for my very own pdf processing pipeline. Would love to make a contribution to add this.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start at PageAnalysis.has_images and trace how the existing classification pass aggregates it into pdf_type before reaching PdfClassification and PdfResult. Confirm the per-page image list is preserved and exposed alongside the existing per-page signals, then validate the public result for PDFs with and without embedded images.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- api, backend-api-design
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 68/100