firecrawl / firecrawl/pdf-inspector

Expose per-page pages_with_images signal

Open
#36 3 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.