kreuzberg 4.0.0.pre.rc.11 → 4.0.0.pre.rc.13

data/vendor/kreuzberg/src/extractors/pdf.rs

@@ -1,722 +1,749 @@
-//! PDF document extractor.
-
-use crate::Result;
-use crate::core::config::ExtractionConfig;
-use crate::plugins::{DocumentExtractor, Plugin};
-use crate::types::{ExtractionResult, Metadata, PageContent};
-use async_trait::async_trait;
-use std::path::Path;
-
-#[cfg(feature = "pdf")]
-use crate::pdf::error::PdfError;
-#[cfg(feature = "ocr")]
-use crate::pdf::rendering::{PageRenderOptions, PdfRenderer};
-#[cfg(all(feature = "pdf", feature = "ocr"))]
-use crate::types::Table;
-#[cfg(feature = "pdf")]
-use pdfium_render::prelude::*;
-
-#[cfg(feature = "ocr")]
-const MIN_TOTAL_NON_WHITESPACE: usize = 64;
-#[cfg(feature = "ocr")]
-const MIN_NON_WHITESPACE_PER_PAGE: f64 = 32.0;
-#[cfg(feature = "ocr")]
-const MIN_MEANINGFUL_WORD_LEN: usize = 4;
-#[cfg(feature = "ocr")]
-const MIN_MEANINGFUL_WORDS: usize = 3;
-#[cfg(feature = "ocr")]
-const MIN_ALNUM_RATIO: f64 = 0.3;
-
-#[cfg(feature = "ocr")]
-struct NativeTextStats {
-    non_whitespace: usize,
-    alnum: usize,
-    meaningful_words: usize,
-    alnum_ratio: f64,
-}
-
-#[cfg(feature = "ocr")]
-struct OcrFallbackDecision {
-    stats: NativeTextStats,
-    avg_non_whitespace: f64,
-    avg_alnum: f64,
-    fallback: bool,
-}
-
-#[cfg(feature = "ocr")]
-impl NativeTextStats {
-    fn from(text: &str) -> Self {
-        let mut non_whitespace = 0usize;
-        let mut alnum = 0usize;
-
-        for ch in text.chars() {
-            if !ch.is_whitespace() {
-                non_whitespace += 1;
-                if ch.is_alphanumeric() {
-                    alnum += 1;
-                }
-            }
-        }
-
-        let meaningful_words = text
-            .split_whitespace()
-            .filter(|word| {
-                word.chars()
-                    .filter(|c| c.is_alphanumeric())
-                    .take(MIN_MEANINGFUL_WORD_LEN)
-                    .count()
-                    >= MIN_MEANINGFUL_WORD_LEN
-            })
-            .take(MIN_MEANINGFUL_WORDS)
-            .count();
-
-        let alnum_ratio = if non_whitespace == 0 {
-            0.0
-        } else {
-            alnum as f64 / non_whitespace as f64
-        };
-
-        Self {
-            non_whitespace,
-            alnum,
-            meaningful_words,
-            alnum_ratio,
-        }
-    }
-}
-
-#[cfg(feature = "ocr")]
-fn evaluate_native_text_for_ocr(native_text: &str, page_count: Option<usize>) -> OcrFallbackDecision {
-    let trimmed = native_text.trim();
-
-    if trimmed.is_empty() {
-        let empty_stats = NativeTextStats {
-            non_whitespace: 0,
-            alnum: 0,
-            meaningful_words: 0,
-            alnum_ratio: 0.0,
-        };
-        return OcrFallbackDecision {
-            stats: empty_stats,
-            avg_non_whitespace: 0.0,
-            avg_alnum: 0.0,
-            fallback: true,
-        };
-    }
-
-    let stats = NativeTextStats::from(trimmed);
-    let pages = page_count.unwrap_or(1).max(1) as f64;
-    let avg_non_whitespace = stats.non_whitespace as f64 / pages;
-    let avg_alnum = stats.alnum as f64 / pages;
-
-    let has_substantial_text = stats.non_whitespace >= MIN_TOTAL_NON_WHITESPACE
-        && avg_non_whitespace >= MIN_NON_WHITESPACE_PER_PAGE
-        && stats.meaningful_words >= MIN_MEANINGFUL_WORDS;
-
-    let fallback = if stats.non_whitespace == 0 || stats.alnum == 0 {
-        true
-    } else if has_substantial_text {
-        false
-    } else if (stats.alnum_ratio < MIN_ALNUM_RATIO && avg_alnum < MIN_NON_WHITESPACE_PER_PAGE)
-        || (stats.non_whitespace < MIN_TOTAL_NON_WHITESPACE && avg_non_whitespace < MIN_NON_WHITESPACE_PER_PAGE)
-    {
-        true
-    } else {
-        stats.meaningful_words == 0 && avg_non_whitespace < MIN_NON_WHITESPACE_PER_PAGE
-    };
-
-    OcrFallbackDecision {
-        stats,
-        avg_non_whitespace,
-        avg_alnum,
-        fallback,
-    }
-}
-
-/// Extract tables from PDF document using native text positions.
-///
-/// This function converts PDF character positions to HocrWord format,
-/// then uses the existing table reconstruction logic to detect tables.
-#[cfg(all(feature = "pdf", feature = "ocr"))]
-fn extract_tables_from_document(
-    document: &PdfDocument,
-    _metadata: &crate::pdf::metadata::PdfExtractionMetadata,
-) -> Result<Vec<Table>> {
-    use crate::ocr::table::{reconstruct_table, table_to_markdown};
-    use crate::pdf::table::extract_words_from_page;
-
-    let mut all_tables = Vec::new();
-
-    for (page_index, page) in document.pages().iter().enumerate() {
-        let words = extract_words_from_page(&page, 0.0)?;
-
-        if words.is_empty() {
-            continue;
-        }
-
-        let column_threshold = 50;
-        let row_threshold_ratio = 0.5;
-
-        let table_cells = reconstruct_table(&words, column_threshold, row_threshold_ratio, true);
-
-        if !table_cells.is_empty() {
-            let markdown = table_to_markdown(&table_cells);
-
-            all_tables.push(Table {
-                cells: table_cells,
-                markdown,
-                page_number: page_index + 1,
-            });
-        }
-    }
-
-    Ok(all_tables)
-}
-
-/// Fallback for when OCR feature is not enabled - returns empty tables.
-#[cfg(all(feature = "pdf", not(feature = "ocr")))]
-fn extract_tables_from_document(
-    _document: &PdfDocument,
-    _metadata: &crate::pdf::metadata::PdfExtractionMetadata,
-) -> Result<Vec<crate::types::Table>> {
-    Ok(vec![])
-}
-
-/// Helper function to assign tables and images to pages.
-///
-/// If page_contents is None, returns None (no per-page tracking enabled).
-/// Otherwise, iterates through tables and images, assigning them to pages based on page_number.
-fn assign_tables_and_images_to_pages(
-    mut page_contents: Option<Vec<PageContent>>,
-    tables: &[crate::types::Table],
-    images: &[crate::types::ExtractedImage],
-) -> Option<Vec<PageContent>> {
-    let pages = page_contents.take()?;
-
-    let mut updated_pages = pages;
-
-    for table in tables {
-        if let Some(page) = updated_pages.iter_mut().find(|p| p.page_number == table.page_number) {
-            page.tables.push(table.clone());
-        }
-    }
-
-    for image in images {
-        if let Some(page_num) = image.page_number
-            && let Some(page) = updated_pages.iter_mut().find(|p| p.page_number == page_num)
-        {
-            page.images.push(image.clone());
-        }
-    }
-
-    Some(updated_pages)
-}
-
-/// PDF document extractor using pypdfium2 and playa-pdf.
-pub struct PdfExtractor;
-
-impl Default for PdfExtractor {
-    fn default() -> Self {
-        Self::new()
-    }
-}
-
-impl PdfExtractor {
-    pub fn new() -> Self {
-        Self
-    }
-
-    /// Extract text from PDF using OCR.
-    ///
-    /// Renders all pages to images and processes them with OCR.
-    #[cfg(feature = "ocr")]
-    async fn extract_with_ocr(&self, content: &[u8], config: &ExtractionConfig) -> Result<String> {
-        use crate::plugins::registry::get_ocr_backend_registry;
-        use image::ImageEncoder;
-        use image::codecs::png::PngEncoder;
-        use std::io::Cursor;
-
-        let ocr_config = config.ocr.as_ref().ok_or_else(|| crate::KreuzbergError::Parsing {
-            message: "OCR config required for force_ocr".to_string(),
-            source: None,
-        })?;
-
-        let backend = {
-            let registry = get_ocr_backend_registry();
-            let registry = registry.read().map_err(|e| crate::KreuzbergError::Plugin {
-                message: format!("Failed to acquire read lock on OCR backend registry: {}", e),
-                plugin_name: "ocr-registry".to_string(),
-            })?;
-            registry.get(&ocr_config.backend)?
-        };
-
-        let images = {
-            let render_options = PageRenderOptions::default();
-            let renderer = PdfRenderer::new().map_err(|e| crate::KreuzbergError::Parsing {
-                message: format!("Failed to initialize PDF renderer: {}", e),
-                source: None,
-            })?;
-
-            renderer
-                .render_all_pages(content, &render_options)
-                .map_err(|e| crate::KreuzbergError::Parsing {
-                    message: format!("Failed to render PDF pages: {}", e),
-                    source: None,
-                })?
-        };
-
-        let mut page_texts = Vec::with_capacity(images.len());
-
-        for image in images {
-            let rgb_image = image.to_rgb8();
-            let (width, height) = rgb_image.dimensions();
-
-            let mut image_bytes = Cursor::new(Vec::new());
-            let encoder = PngEncoder::new(&mut image_bytes);
-            encoder
-                .write_image(&rgb_image, width, height, image::ColorType::Rgb8.into())
-                .map_err(|e| crate::KreuzbergError::Parsing {
-                    message: format!("Failed to encode image: {}", e),
-                    source: None,
-                })?;
-
-            let image_data = image_bytes.into_inner();
-
-            let ocr_result = backend.process_image(&image_data, ocr_config).await?;
-
-            page_texts.push(ocr_result.content);
-        }
-
-        Ok(page_texts.join("\n\n"))
-    }
-}
-
-impl Plugin for PdfExtractor {
-    fn name(&self) -> &str {
-        "pdf-extractor"
-    }
-
-    fn version(&self) -> String {
-        env!("CARGO_PKG_VERSION").to_string()
-    }
-
-    fn initialize(&self) -> Result<()> {
-        Ok(())
-    }
-
-    fn shutdown(&self) -> Result<()> {
-        Ok(())
-    }
-}
-
-#[async_trait]
-impl DocumentExtractor for PdfExtractor {
-    #[cfg_attr(feature = "otel", tracing::instrument(
-        skip(self, content, config),
-        fields(
-            extractor.name = self.name(),
-            content.size_bytes = content.len(),
-        )
-    ))]
-    async fn extract_bytes(
-        &self,
-        content: &[u8],
-        mime_type: &str,
-        config: &ExtractionConfig,
-    ) -> Result<ExtractionResult> {
-        #[cfg(feature = "pdf")]
-        let (pdf_metadata, native_text, tables, page_contents) = {
-            // WASM target: always synchronous (no tokio::task::spawn_blocking)
-            // Other targets: use spawn_blocking in batch mode for better parallelism
-            #[cfg(target_arch = "wasm32")]
-            {
-                // SAFETY: For WASM targets, this code path should only be reached if the
-                // WASM environment has properly initialized PDFium. The error message
-                // will direct users to the documentation for setup requirements.
-                let bindings =
-                    crate::pdf::bindings::bind_pdfium(PdfError::MetadataExtractionFailed, "initialize Pdfium")
-                        .map_err(|pdf_err| {
-                            // Provide context-specific error for WASM PDF failures
-                            if pdf_err.to_string().contains("WASM") || pdf_err.to_string().contains("Module") {
-                                crate::error::KreuzbergError::Parsing {
-                                    message: "PDF extraction requires proper WASM module initialization. \
-                                         Ensure your WASM environment is set up with PDFium support. \
-                                         See: https://docs.kreuzberg.dev/wasm/pdf"
-                                        .to_string(),
-                                    source: None,
-                                }
-                            } else {
-                                pdf_err.into()
-                            }
-                        })?;
-                let pdfium = Pdfium::new(bindings);
-
-                let document = pdfium.load_pdf_from_byte_slice(content, None).map_err(|e| {
-                    let err_msg = e.to_string();
-                    if err_msg.contains("password") || err_msg.contains("Password") {
-                        PdfError::PasswordRequired
-                    } else {
-                        PdfError::InvalidPdf(err_msg)
-                    }
-                })?;
-
-                let (native_text, boundaries, page_contents) =
-                    crate::pdf::text::extract_text_from_pdf_document(&document, config.pages.as_ref())?;
-
-                let pdf_metadata =
-                    crate::pdf::metadata::extract_metadata_from_document(&document, boundaries.as_deref())?;
-
-                let tables = extract_tables_from_document(&document, &pdf_metadata)?;
-
-                (pdf_metadata, native_text, tables, page_contents)
-            }
-            #[cfg(not(target_arch = "wasm32"))]
-            {
-                if crate::core::batch_mode::is_batch_mode() {
-                    let content_owned = content.to_vec();
-                    let span = tracing::Span::current();
-                    let pages_config = config.pages.clone();
-                    tokio::task::spawn_blocking(move || {
-                        let _guard = span.entered();
-                        let bindings =
-                            crate::pdf::bindings::bind_pdfium(PdfError::MetadataExtractionFailed, "initialize Pdfium")?;
-
-                        let pdfium = Pdfium::new(bindings);
-
-                        let document = pdfium.load_pdf_from_byte_slice(&content_owned, None).map_err(|e| {
-                            let err_msg = e.to_string();
-                            if err_msg.contains("password") || err_msg.contains("Password") {
-                                PdfError::PasswordRequired
-                            } else {
-                                PdfError::InvalidPdf(err_msg)
-                            }
-                        })?;
-
-                        let (native_text, boundaries, page_contents) =
-                            crate::pdf::text::extract_text_from_pdf_document(&document, pages_config.as_ref())?;
-
-                        let pdf_metadata =
-                            crate::pdf::metadata::extract_metadata_from_document(&document, boundaries.as_deref())?;
-
-                        let tables = extract_tables_from_document(&document, &pdf_metadata)?;
-
-                        if let Some(ref page_cfg) = pages_config
-                            && page_cfg.extract_pages
-                            && page_contents.is_none()
-                        {
-                            return Err(PdfError::ExtractionFailed(
-                                "Page extraction was configured but no page data was extracted in batch mode"
-                                    .to_string(),
-                            )
-                            .into());
-                        }
-
-                        Ok::<_, crate::error::KreuzbergError>((pdf_metadata, native_text, tables, page_contents))
-                    })
-                    .await
-                    .map_err(|e| crate::error::KreuzbergError::Other(format!("PDF extraction task failed: {}", e)))??
-                } else {
-                    let bindings =
-                        crate::pdf::bindings::bind_pdfium(PdfError::MetadataExtractionFailed, "initialize Pdfium")?;
-
-                    let pdfium = Pdfium::new(bindings);
-
-                    let document = pdfium.load_pdf_from_byte_slice(content, None).map_err(|e| {
-                        let err_msg = e.to_string();
-                        if err_msg.contains("password") || err_msg.contains("Password") {
-                            PdfError::PasswordRequired
-                        } else {
-                            PdfError::InvalidPdf(err_msg)
-                        }
-                    })?;
-
-                    let (native_text, boundaries, page_contents) =
-                        crate::pdf::text::extract_text_from_pdf_document(&document, config.pages.as_ref())?;
-
-                    let pdf_metadata =
-                        crate::pdf::metadata::extract_metadata_from_document(&document, boundaries.as_deref())?;
-
-                    let tables = extract_tables_from_document(&document, &pdf_metadata)?;
-
-                    (pdf_metadata, native_text, tables, page_contents)
-                }
-            }
-        };
-
-        #[cfg(feature = "ocr")]
-        let text = if config.force_ocr {
-            if config.ocr.is_some() {
-                self.extract_with_ocr(content, config).await?
-            } else {
-                native_text
-            }
-        } else if config.ocr.is_some() {
-            let decision = evaluate_native_text_for_ocr(&native_text, None);
-
-            if std::env::var("KREUZBERG_DEBUG_OCR").is_ok() {
-                eprintln!(
-                    "[kreuzberg::pdf::ocr] fallback={} non_whitespace={} alnum={} meaningful_words={} \
-                     avg_non_whitespace={:.2} avg_alnum={:.2} alnum_ratio={:.3}",
-                    decision.fallback,
-                    decision.stats.non_whitespace,
-                    decision.stats.alnum,
-                    decision.stats.meaningful_words,
-                    decision.avg_non_whitespace,
-                    decision.avg_alnum,
-                    decision.stats.alnum_ratio
-                );
-            }
-
-            if decision.fallback {
-                self.extract_with_ocr(content, config).await?
-            } else {
-                native_text
-            }
-        } else {
-            native_text
-        };
-
-        #[cfg(not(feature = "ocr"))]
-        let text = native_text;
-
-        #[cfg(feature = "pdf")]
-        if let Some(ref page_cfg) = config.pages
-            && page_cfg.insert_page_markers
-        {
-            let marker_placeholder = page_cfg.marker_format.replace("{page_num}", "");
-            if !marker_placeholder.is_empty() && !text.contains(&marker_placeholder) {
-                #[cfg(feature = "otel")]
-                tracing::warn!(
-                    "Page markers were configured but none found in extracted content. \
-                     This may indicate very short documents or incomplete extraction."
-                );
-            }
-        }
-
-        let images = if config.images.is_some() {
-            match crate::pdf::images::extract_images_from_pdf(content) {
-                Ok(pdf_images) => Some(
-                    pdf_images
-                        .into_iter()
-                        .enumerate()
-                        .map(|(idx, img)| {
-                            let format = img.filters.first().cloned().unwrap_or_else(|| "unknown".to_string());
-                            crate::types::ExtractedImage {
-                                data: img.data,
-                                format,
-                                image_index: idx,
-                                page_number: Some(img.page_number),
-                                width: Some(img.width as u32),
-                                height: Some(img.height as u32),
-                                colorspace: img.color_space,
-                                bits_per_component: img.bits_per_component.map(|b| b as u32),
-                                is_mask: false,
-                                description: None,
-                                ocr_result: None,
-                            }
-                        })
-                        .collect(),
-                ),
-                Err(_) => None,
-            }
-        } else {
-            None
-        };
-
-        let final_pages = assign_tables_and_images_to_pages(page_contents, &tables, images.as_deref().unwrap_or(&[]));
-
-        Ok(ExtractionResult {
-            content: text,
-            mime_type: mime_type.to_string(),
-            metadata: Metadata {
-                #[cfg(feature = "pdf")]
-                title: pdf_metadata.title.clone(),
-                #[cfg(feature = "pdf")]
-                subject: pdf_metadata.subject.clone(),
-                #[cfg(feature = "pdf")]
-                authors: pdf_metadata.authors.clone(),
-                #[cfg(feature = "pdf")]
-                keywords: pdf_metadata.keywords.clone(),
-                #[cfg(feature = "pdf")]
-                created_at: pdf_metadata.created_at.clone(),
-                #[cfg(feature = "pdf")]
-                modified_at: pdf_metadata.modified_at.clone(),
-                #[cfg(feature = "pdf")]
-                created_by: pdf_metadata.created_by.clone(),
-                #[cfg(feature = "pdf")]
-                pages: pdf_metadata.page_structure.clone(),
-                #[cfg(feature = "pdf")]
-                format: Some(crate::types::FormatMetadata::Pdf(pdf_metadata.pdf_specific)),
-                ..Default::default()
-            },
-            pages: final_pages,
-            tables,
-            detected_languages: None,
-            chunks: None,
-            images,
-        })
-    }
-
-    #[cfg(feature = "tokio-runtime")]
-    async fn extract_file(&self, path: &Path, mime_type: &str, config: &ExtractionConfig) -> Result<ExtractionResult> {
-        let bytes = tokio::fs::read(path).await?;
-        self.extract_bytes(&bytes, mime_type, config).await
-    }
-
-    fn supported_mime_types(&self) -> &[&str] {
-        &["application/pdf"]
-    }
-
-    fn priority(&self) -> i32 {
-        50
-    }
-}
-
-#[cfg(test)]
-mod tests {
-    use super::*;
-
-    #[test]
-    fn test_pdf_extractor_plugin_interface() {
-        let extractor = PdfExtractor::new();
-        assert_eq!(extractor.name(), "pdf-extractor");
-        assert!(extractor.initialize().is_ok());
-        assert!(extractor.shutdown().is_ok());
-    }
-
-    #[test]
-    fn test_pdf_extractor_supported_mime_types() {
-        let extractor = PdfExtractor::new();
-        let mime_types = extractor.supported_mime_types();
-        assert_eq!(mime_types.len(), 1);
-        assert!(mime_types.contains(&"application/pdf"));
-    }
-
-    #[test]
-    fn test_pdf_extractor_priority() {
-        let extractor = PdfExtractor::new();
-        assert_eq!(extractor.priority(), 50);
-    }
-
-    #[cfg(feature = "ocr")]
-    #[test]
-    fn test_should_fallback_to_ocr_for_empty_text() {
-        assert!(evaluate_native_text_for_ocr("", Some(1)).fallback);
-    }
-
-    #[cfg(feature = "ocr")]
-    #[test]
-    fn test_should_not_fallback_for_meaningful_text() {
-        let sample = "This page has searchable vector text and should avoid OCR.";
-        assert!(!evaluate_native_text_for_ocr(sample, Some(1)).fallback);
-    }
-
-    #[cfg(feature = "ocr")]
-    #[test]
-    fn test_should_fallback_for_punctuation_only_text() {
-        let sample = " . , ; : -- -- ";
-        assert!(evaluate_native_text_for_ocr(sample, Some(2)).fallback);
-    }
-
-    #[tokio::test]
-    #[cfg(feature = "pdf")]
-    async fn test_pdf_batch_mode_validates_page_config_enabled() {
-        use crate::core::config::PageConfig;
-
-        let extractor = PdfExtractor::new();
-
-        let config = ExtractionConfig {
-            pages: Some(PageConfig {
-                extract_pages: true,
-                insert_page_markers: false,
-                marker_format: "<!-- PAGE {page_num} -->".to_string(),
-            }),
-            ..Default::default()
-        };
-
-        let pdf_path =
-            std::path::Path::new(env!("CARGO_MANIFEST_DIR")).join("../../test_documents/pdfs/google_doc_document.pdf");
-        if let Ok(content) = std::fs::read(pdf_path) {
-            let result = extractor.extract_bytes(&content, "application/pdf", &config).await;
-            assert!(
-                result.is_ok(),
-                "Failed to extract PDF with page config: {:?}",
-                result.err()
-            );
-
-            let extraction_result = result.unwrap();
-            assert!(
-                extraction_result.pages.is_some(),
-                "Pages should be extracted when extract_pages is true"
-            );
-        }
-    }
-
-    #[tokio::test]
-    #[cfg(feature = "pdf")]
-    async fn test_pdf_batch_mode_validates_page_config_disabled() {
-        let extractor = PdfExtractor::new();
-        let config = ExtractionConfig::default();
-
-        let pdf_path =
-            std::path::Path::new(env!("CARGO_MANIFEST_DIR")).join("../../test_documents/pdfs/google_doc_document.pdf");
-        if let Ok(content) = std::fs::read(pdf_path) {
-            let result = extractor.extract_bytes(&content, "application/pdf", &config).await;
-            assert!(
-                result.is_ok(),
-                "Failed to extract PDF without page config: {:?}",
-                result.err()
-            );
-
-            let extraction_result = result.unwrap();
-            assert!(
-                extraction_result.pages.is_none(),
-                "Pages should not be extracted when pages config is None"
-            );
-        }
-    }
-
-    #[tokio::test]
-    #[cfg(feature = "pdf")]
-    async fn test_pdf_page_marker_validation() {
-        use crate::core::config::PageConfig;
-
-        let extractor = PdfExtractor::new();
-
-        let config = ExtractionConfig {
-            pages: Some(PageConfig {
-                extract_pages: true,
-                insert_page_markers: true,
-                marker_format: "\n\n<!-- PAGE {page_num} -->\n\n".to_string(),
-            }),
-            ..Default::default()
-        };
-
-        let pdf_path =
-            std::path::Path::new(env!("CARGO_MANIFEST_DIR")).join("../../test_documents/pdfs/multi_page.pdf");
-        if let Ok(content) = std::fs::read(pdf_path) {
-            let result = extractor.extract_bytes(&content, "application/pdf", &config).await;
-            assert!(
-                result.is_ok(),
-                "Failed to extract PDF with page markers: {:?}",
-                result.err()
-            );
-
-            let extraction_result = result.unwrap();
-            let marker_placeholder = "<!-- PAGE ";
-            if extraction_result.content.len() > 100 {
-                assert!(
-                    extraction_result.content.contains(marker_placeholder),
-                    "Page markers should be inserted when configured and document has multiple pages"
-                );
-            }
-        }
-    }
-
-    #[test]
-    #[cfg(feature = "pdf")]
-    fn test_pdf_extractor_without_feature_pdf() {
-        let extractor = PdfExtractor::new();
-        assert_eq!(extractor.name(), "pdf-extractor");
-    }
-}
+//! PDF document extractor.
+
+use crate::Result;
+use crate::core::config::ExtractionConfig;
+use crate::plugins::{DocumentExtractor, Plugin};
+use crate::types::{ExtractionResult, Metadata, PageContent};
+use async_trait::async_trait;
+#[cfg(feature = "tokio-runtime")]
+use std::path::Path;
+
+#[cfg(feature = "pdf")]
+use crate::pdf::error::PdfError;
+#[cfg(feature = "ocr")]
+use crate::pdf::rendering::{PageRenderOptions, PdfRenderer};
+#[cfg(all(feature = "pdf", feature = "ocr"))]
+use crate::types::Table;
+#[cfg(feature = "pdf")]
+use pdfium_render::prelude::*;
+
+#[cfg(feature = "ocr")]
+const MIN_TOTAL_NON_WHITESPACE: usize = 64;
+#[cfg(feature = "ocr")]
+const MIN_NON_WHITESPACE_PER_PAGE: f64 = 32.0;
+#[cfg(feature = "ocr")]
+const MIN_MEANINGFUL_WORD_LEN: usize = 4;
+#[cfg(feature = "ocr")]
+const MIN_MEANINGFUL_WORDS: usize = 3;
+#[cfg(feature = "ocr")]
+const MIN_ALNUM_RATIO: f64 = 0.3;
+
+#[cfg(feature = "ocr")]
+struct NativeTextStats {
+    non_whitespace: usize,
+    alnum: usize,
+    meaningful_words: usize,
+    alnum_ratio: f64,
+}
+
+#[cfg(feature = "ocr")]
+struct OcrFallbackDecision {
+    stats: NativeTextStats,
+    avg_non_whitespace: f64,
+    avg_alnum: f64,
+    fallback: bool,
+}
+
+#[cfg(feature = "ocr")]
+impl NativeTextStats {
+    fn from(text: &str) -> Self {
+        let mut non_whitespace = 0usize;
+        let mut alnum = 0usize;
+
+        for ch in text.chars() {
+            if !ch.is_whitespace() {
+                non_whitespace += 1;
+                if ch.is_alphanumeric() {
+                    alnum += 1;
+                }
+            }
+        }
+
+        let meaningful_words = text
+            .split_whitespace()
+            .filter(|word| {
+                word.chars()
+                    .filter(|c| c.is_alphanumeric())
+                    .take(MIN_MEANINGFUL_WORD_LEN)
+                    .count()
+                    >= MIN_MEANINGFUL_WORD_LEN
+            })
+            .take(MIN_MEANINGFUL_WORDS)
+            .count();
+
+        let alnum_ratio = if non_whitespace == 0 {
+            0.0
+        } else {
+            alnum as f64 / non_whitespace as f64
+        };
+
+        Self {
+            non_whitespace,
+            alnum,
+            meaningful_words,
+            alnum_ratio,
+        }
+    }
+}
+
+#[cfg(feature = "ocr")]
+fn evaluate_native_text_for_ocr(native_text: &str, page_count: Option<usize>) -> OcrFallbackDecision {
+    let trimmed = native_text.trim();
+
+    if trimmed.is_empty() {
+        let empty_stats = NativeTextStats {
+            non_whitespace: 0,
+            alnum: 0,
+            meaningful_words: 0,
+            alnum_ratio: 0.0,
+        };
+        return OcrFallbackDecision {
+            stats: empty_stats,
+            avg_non_whitespace: 0.0,
+            avg_alnum: 0.0,
+            fallback: true,
+        };
+    }
+
+    let stats = NativeTextStats::from(trimmed);
+    let pages = page_count.unwrap_or(1).max(1) as f64;
+    let avg_non_whitespace = stats.non_whitespace as f64 / pages;
+    let avg_alnum = stats.alnum as f64 / pages;
+
+    let has_substantial_text = stats.non_whitespace >= MIN_TOTAL_NON_WHITESPACE
+        && avg_non_whitespace >= MIN_NON_WHITESPACE_PER_PAGE
+        && stats.meaningful_words >= MIN_MEANINGFUL_WORDS;
+
+    let fallback = if stats.non_whitespace == 0 || stats.alnum == 0 {
+        true
+    } else if has_substantial_text {
+        false
+    } else if (stats.alnum_ratio < MIN_ALNUM_RATIO && avg_alnum < MIN_NON_WHITESPACE_PER_PAGE)
+        || (stats.non_whitespace < MIN_TOTAL_NON_WHITESPACE && avg_non_whitespace < MIN_NON_WHITESPACE_PER_PAGE)
+    {
+        true
+    } else {
+        stats.meaningful_words == 0 && avg_non_whitespace < MIN_NON_WHITESPACE_PER_PAGE
+    };
+
+    OcrFallbackDecision {
+        stats,
+        avg_non_whitespace,
+        avg_alnum,
+        fallback,
+    }
+}
+
+/// Extract tables from PDF document using native text positions.
+///
+/// This function converts PDF character positions to HocrWord format,
+/// then uses the existing table reconstruction logic to detect tables.
+#[cfg(all(feature = "pdf", feature = "ocr"))]
+fn extract_tables_from_document(
+    document: &PdfDocument,
+    _metadata: &crate::pdf::metadata::PdfExtractionMetadata,
+) -> Result<Vec<Table>> {
+    use crate::ocr::table::{reconstruct_table, table_to_markdown};
+    use crate::pdf::table::extract_words_from_page;
+
+    let mut all_tables = Vec::new();
+
+    for (page_index, page) in document.pages().iter().enumerate() {
+        let words = extract_words_from_page(&page, 0.0)?;
+
+        if words.is_empty() {
+            continue;
+        }
+
+        let column_threshold = 50;
+        let row_threshold_ratio = 0.5;
+
+        let table_cells = reconstruct_table(&words, column_threshold, row_threshold_ratio, true);
+
+        if !table_cells.is_empty() {
+            let markdown = table_to_markdown(&table_cells);
+
+            all_tables.push(Table {
+                cells: table_cells,
+                markdown,
+                page_number: page_index + 1,
+            });
+        }
+    }
+
+    Ok(all_tables)
+}
+
+/// Fallback for when OCR feature is not enabled - returns empty tables.
+#[cfg(all(feature = "pdf", not(feature = "ocr")))]
+fn extract_tables_from_document(
+    _document: &PdfDocument,
+    _metadata: &crate::pdf::metadata::PdfExtractionMetadata,
+) -> Result<Vec<crate::types::Table>> {
+    Ok(vec![])
+}
+
+/// Helper function to assign tables and images to pages.
+///
+/// If page_contents is None, returns None (no per-page tracking enabled).
+/// Otherwise, iterates through tables and images, assigning them to pages based on page_number.
+fn assign_tables_and_images_to_pages(
+    mut page_contents: Option<Vec<PageContent>>,
+    tables: &[crate::types::Table],
+    images: &[crate::types::ExtractedImage],
+) -> Option<Vec<PageContent>> {
+    let pages = page_contents.take()?;
+
+    let mut updated_pages = pages;
+
+    for table in tables {
+        if let Some(page) = updated_pages.iter_mut().find(|p| p.page_number == table.page_number) {
+            page.tables.push(table.clone());
+        }
+    }
+
+    for image in images {
+        if let Some(page_num) = image.page_number
+            && let Some(page) = updated_pages.iter_mut().find(|p| p.page_number == page_num)
+        {
+            page.images.push(image.clone());
+        }
+    }
+
+    Some(updated_pages)
+}
+
+/// PDF document extractor using pypdfium2 and playa-pdf.
+pub struct PdfExtractor;
+
+impl Default for PdfExtractor {
+    fn default() -> Self {
+        Self::new()
+    }
+}
+
+impl PdfExtractor {
+    pub fn new() -> Self {
+        Self
+    }
+
+    /// Extract text from PDF using OCR.
+    ///
+    /// Renders all pages to images and processes them with OCR.
+    #[cfg(feature = "ocr")]
+    async fn extract_with_ocr(&self, content: &[u8], config: &ExtractionConfig) -> Result<String> {
+        use crate::plugins::registry::get_ocr_backend_registry;
+        use image::ImageEncoder;
+        use image::codecs::png::PngEncoder;
+        use std::io::Cursor;
+
+        let ocr_config = config.ocr.as_ref().ok_or_else(|| crate::KreuzbergError::Parsing {
+            message: "OCR config required for force_ocr".to_string(),
+            source: None,
+        })?;
+
+        let backend = {
+            let registry = get_ocr_backend_registry();
+            let registry = registry.read().map_err(|e| crate::KreuzbergError::Plugin {
+                message: format!("Failed to acquire read lock on OCR backend registry: {}", e),
+                plugin_name: "ocr-registry".to_string(),
+            })?;
+            registry.get(&ocr_config.backend)?
+        };
+
+        let images = {
+            let render_options = PageRenderOptions::default();
+            let renderer = PdfRenderer::new().map_err(|e| crate::KreuzbergError::Parsing {
+                message: format!("Failed to initialize PDF renderer: {}", e),
+                source: None,
+            })?;
+
+            renderer
+                .render_all_pages(content, &render_options)
+                .map_err(|e| crate::KreuzbergError::Parsing {
+                    message: format!("Failed to render PDF pages: {}", e),
+                    source: None,
+                })?
+        };
+
+        let mut page_texts = Vec::with_capacity(images.len());
+
+        for image in images {
+            let rgb_image = image.to_rgb8();
+            let (width, height) = rgb_image.dimensions();
+
+            let mut image_bytes = Cursor::new(Vec::new());
+            let encoder = PngEncoder::new(&mut image_bytes);
+            encoder
+                .write_image(&rgb_image, width, height, image::ColorType::Rgb8.into())
+                .map_err(|e| crate::KreuzbergError::Parsing {
+                    message: format!("Failed to encode image: {}", e),
+                    source: None,
+                })?;
+
+            let image_data = image_bytes.into_inner();
+
+            let ocr_result = backend.process_image(&image_data, ocr_config).await?;
+
+            page_texts.push(ocr_result.content);
+        }
+
+        Ok(page_texts.join("\n\n"))
+    }
+}
+
+impl Plugin for PdfExtractor {
+    fn name(&self) -> &str {
+        "pdf-extractor"
+    }
+
+    fn version(&self) -> String {
+        env!("CARGO_PKG_VERSION").to_string()
+    }
+
+    fn initialize(&self) -> Result<()> {
+        Ok(())
+    }
+
+    fn shutdown(&self) -> Result<()> {
+        Ok(())
+    }
+}
+
+#[async_trait]
+impl DocumentExtractor for PdfExtractor {
+    #[cfg_attr(feature = "otel", tracing::instrument(
+        skip(self, content, config),
+        fields(
+            extractor.name = self.name(),
+            content.size_bytes = content.len(),
+        )
+    ))]
+    async fn extract_bytes(
+        &self,
+        content: &[u8],
+        mime_type: &str,
+        config: &ExtractionConfig,
+    ) -> Result<ExtractionResult> {
+        #[cfg(feature = "pdf")]
+        let (pdf_metadata, native_text, tables, page_contents) = {
+            // WASM target: always synchronous (no tokio::task::spawn_blocking)
+            // Other targets: use spawn_blocking in batch mode for better parallelism
+            #[cfg(target_arch = "wasm32")]
+            {
+                // SAFETY: For WASM targets, this code path should only be reached if the
+                // WASM environment has properly initialized PDFium. The error message
+                // will direct users to the documentation for setup requirements.
+                let bindings =
+                    crate::pdf::bindings::bind_pdfium(PdfError::MetadataExtractionFailed, "initialize Pdfium")
+                        .map_err(|pdf_err| {
+                            // Provide context-specific error for WASM PDF failures
+                            if pdf_err.to_string().contains("WASM") || pdf_err.to_string().contains("Module") {
+                                crate::error::KreuzbergError::Parsing {
+                                    message: "PDF extraction requires proper WASM module initialization. \
+                                         Ensure your WASM environment is set up with PDFium support. \
+                                         See: https://docs.kreuzberg.dev/wasm/pdf"
+                                        .to_string(),
+                                    source: None,
+                                }
+                            } else {
+                                pdf_err.into()
+                            }
+                        })?;
+                let pdfium = Pdfium::new(bindings);
+
+                let document = pdfium.load_pdf_from_byte_slice(content, None).map_err(|e| {
+                    let err_msg = e.to_string();
+                    if err_msg.contains("password") || err_msg.contains("Password") {
+                        PdfError::PasswordRequired
+                    } else {
+                        PdfError::InvalidPdf(err_msg)
+                    }
+                })?;
+
+                let (native_text, boundaries, page_contents) =
+                    crate::pdf::text::extract_text_from_pdf_document(&document, config.pages.as_ref())?;
+
+                let pdf_metadata =
+                    crate::pdf::metadata::extract_metadata_from_document(&document, boundaries.as_deref())?;
+
+                let tables = extract_tables_from_document(&document, &pdf_metadata)?;
+
+                (pdf_metadata, native_text, tables, page_contents)
+            }
+            #[cfg(all(not(target_arch = "wasm32"), feature = "tokio-runtime"))]
+            {
+                if crate::core::batch_mode::is_batch_mode() {
+                    let content_owned = content.to_vec();
+                    let span = tracing::Span::current();
+                    let pages_config = config.pages.clone();
+                    tokio::task::spawn_blocking(move || {
+                        let _guard = span.entered();
+                        let bindings =
+                            crate::pdf::bindings::bind_pdfium(PdfError::MetadataExtractionFailed, "initialize Pdfium")?;
+
+                        let pdfium = Pdfium::new(bindings);
+
+                        let document = pdfium.load_pdf_from_byte_slice(&content_owned, None).map_err(|e| {
+                            let err_msg = e.to_string();
+                            if err_msg.contains("password") || err_msg.contains("Password") {
+                                PdfError::PasswordRequired
+                            } else {
+                                PdfError::InvalidPdf(err_msg)
+                            }
+                        })?;
+
+                        let (native_text, boundaries, page_contents) =
+                            crate::pdf::text::extract_text_from_pdf_document(&document, pages_config.as_ref())?;
+
+                        let pdf_metadata =
+                            crate::pdf::metadata::extract_metadata_from_document(&document, boundaries.as_deref())?;
+
+                        let tables = extract_tables_from_document(&document, &pdf_metadata)?;
+
+                        if let Some(ref page_cfg) = pages_config
+                            && page_cfg.extract_pages
+                            && page_contents.is_none()
+                        {
+                            return Err(PdfError::ExtractionFailed(
+                                "Page extraction was configured but no page data was extracted in batch mode"
+                                    .to_string(),
+                            )
+                            .into());
+                        }
+
+                        Ok::<_, crate::error::KreuzbergError>((pdf_metadata, native_text, tables, page_contents))
+                    })
+                    .await
+                    .map_err(|e| crate::error::KreuzbergError::Other(format!("PDF extraction task failed: {}", e)))??
+                } else {
+                    let bindings =
+                        crate::pdf::bindings::bind_pdfium(PdfError::MetadataExtractionFailed, "initialize Pdfium")?;
+
+                    let pdfium = Pdfium::new(bindings);
+
+                    let document = pdfium.load_pdf_from_byte_slice(content, None).map_err(|e| {
+                        let err_msg = e.to_string();
+                        if err_msg.contains("password") || err_msg.contains("Password") {
+                            PdfError::PasswordRequired
+                        } else {
+                            PdfError::InvalidPdf(err_msg)
+                        }
+                    })?;
+
+                    let (native_text, boundaries, page_contents) =
+                        crate::pdf::text::extract_text_from_pdf_document(&document, config.pages.as_ref())?;
+
+                    let pdf_metadata =
+                        crate::pdf::metadata::extract_metadata_from_document(&document, boundaries.as_deref())?;
+
+                    let tables = extract_tables_from_document(&document, &pdf_metadata)?;
+
+                    (pdf_metadata, native_text, tables, page_contents)
+                }
+            }
+            #[cfg(all(not(target_arch = "wasm32"), not(feature = "tokio-runtime")))]
+            {
+                let bindings =
+                    crate::pdf::bindings::bind_pdfium(PdfError::MetadataExtractionFailed, "initialize Pdfium")?;
+
+                let pdfium = Pdfium::new(bindings);
+
+                let document = pdfium.load_pdf_from_byte_slice(content, None).map_err(|e| {
+                    let err_msg = e.to_string();
+                    if err_msg.contains("password") || err_msg.contains("Password") {
+                        PdfError::PasswordRequired
+                    } else {
+                        PdfError::InvalidPdf(err_msg)
+                    }
+                })?;
+
+                let (native_text, boundaries, page_contents) =
+                    crate::pdf::text::extract_text_from_pdf_document(&document, config.pages.as_ref())?;
+
+                let pdf_metadata =
+                    crate::pdf::metadata::extract_metadata_from_document(&document, boundaries.as_deref())?;
+
+                let tables = extract_tables_from_document(&document, &pdf_metadata)?;
+
+                (pdf_metadata, native_text, tables, page_contents)
+            }
+        };
+
+        #[cfg(feature = "ocr")]
+        let text = if config.force_ocr {
+            if config.ocr.is_some() {
+                self.extract_with_ocr(content, config).await?
+            } else {
+                native_text
+            }
+        } else if config.ocr.is_some() {
+            let decision = evaluate_native_text_for_ocr(&native_text, None);
+
+            if std::env::var("KREUZBERG_DEBUG_OCR").is_ok() {
+                eprintln!(
+                    "[kreuzberg::pdf::ocr] fallback={} non_whitespace={} alnum={} meaningful_words={} \
+                     avg_non_whitespace={:.2} avg_alnum={:.2} alnum_ratio={:.3}",
+                    decision.fallback,
+                    decision.stats.non_whitespace,
+                    decision.stats.alnum,
+                    decision.stats.meaningful_words,
+                    decision.avg_non_whitespace,
+                    decision.avg_alnum,
+                    decision.stats.alnum_ratio
+                );
+            }
+
+            if decision.fallback {
+                self.extract_with_ocr(content, config).await?
+            } else {
+                native_text
+            }
+        } else {
+            native_text
+        };
+
+        #[cfg(not(feature = "ocr"))]
+        let text = native_text;
+
+        #[cfg(feature = "pdf")]
+        if let Some(ref page_cfg) = config.pages
+            && page_cfg.insert_page_markers
+        {
+            let marker_placeholder = page_cfg.marker_format.replace("{page_num}", "");
+            if !marker_placeholder.is_empty() && !text.contains(&marker_placeholder) {
+                #[cfg(feature = "otel")]
+                tracing::warn!(
+                    "Page markers were configured but none found in extracted content. \
+                     This may indicate very short documents or incomplete extraction."
+                );
+            }
+        }
+
+        let images = if config.images.is_some() {
+            match crate::pdf::images::extract_images_from_pdf(content) {
+                Ok(pdf_images) => Some(
+                    pdf_images
+                        .into_iter()
+                        .enumerate()
+                        .map(|(idx, img)| {
+                            let format = img.filters.first().cloned().unwrap_or_else(|| "unknown".to_string());
+                            crate::types::ExtractedImage {
+                                data: img.data,
+                                format,
+                                image_index: idx,
+                                page_number: Some(img.page_number),
+                                width: Some(img.width as u32),
+                                height: Some(img.height as u32),
+                                colorspace: img.color_space,
+                                bits_per_component: img.bits_per_component.map(|b| b as u32),
+                                is_mask: false,
+                                description: None,
+                                ocr_result: None,
+                            }
+                        })
+                        .collect(),
+                ),
+                Err(_) => None,
+            }
+        } else {
+            None
+        };
+
+        let final_pages = assign_tables_and_images_to_pages(page_contents, &tables, images.as_deref().unwrap_or(&[]));
+
+        Ok(ExtractionResult {
+            content: text,
+            mime_type: mime_type.to_string(),
+            metadata: Metadata {
+                #[cfg(feature = "pdf")]
+                title: pdf_metadata.title.clone(),
+                #[cfg(feature = "pdf")]
+                subject: pdf_metadata.subject.clone(),
+                #[cfg(feature = "pdf")]
+                authors: pdf_metadata.authors.clone(),
+                #[cfg(feature = "pdf")]
+                keywords: pdf_metadata.keywords.clone(),
+                #[cfg(feature = "pdf")]
+                created_at: pdf_metadata.created_at.clone(),
+                #[cfg(feature = "pdf")]
+                modified_at: pdf_metadata.modified_at.clone(),
+                #[cfg(feature = "pdf")]
+                created_by: pdf_metadata.created_by.clone(),
+                #[cfg(feature = "pdf")]
+                pages: pdf_metadata.page_structure.clone(),
+                #[cfg(feature = "pdf")]
+                format: Some(crate::types::FormatMetadata::Pdf(pdf_metadata.pdf_specific)),
+                ..Default::default()
+            },
+            pages: final_pages,
+            tables,
+            detected_languages: None,
+            chunks: None,
+            images,
+        })
+    }
+
+    #[cfg(feature = "tokio-runtime")]
+    async fn extract_file(&self, path: &Path, mime_type: &str, config: &ExtractionConfig) -> Result<ExtractionResult> {
+        let bytes = tokio::fs::read(path).await?;
+        self.extract_bytes(&bytes, mime_type, config).await
+    }
+
+    fn supported_mime_types(&self) -> &[&str] {
+        &["application/pdf"]
+    }
+
+    fn priority(&self) -> i32 {
+        50
+    }
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test]
+    fn test_pdf_extractor_plugin_interface() {
+        let extractor = PdfExtractor::new();
+        assert_eq!(extractor.name(), "pdf-extractor");
+        assert!(extractor.initialize().is_ok());
+        assert!(extractor.shutdown().is_ok());
+    }
+
+    #[test]
+    fn test_pdf_extractor_supported_mime_types() {
+        let extractor = PdfExtractor::new();
+        let mime_types = extractor.supported_mime_types();
+        assert_eq!(mime_types.len(), 1);
+        assert!(mime_types.contains(&"application/pdf"));
+    }
+
+    #[test]
+    fn test_pdf_extractor_priority() {
+        let extractor = PdfExtractor::new();
+        assert_eq!(extractor.priority(), 50);
+    }
+
+    #[cfg(feature = "ocr")]
+    #[test]
+    fn test_should_fallback_to_ocr_for_empty_text() {
+        assert!(evaluate_native_text_for_ocr("", Some(1)).fallback);
+    }
+
+    #[cfg(feature = "ocr")]
+    #[test]
+    fn test_should_not_fallback_for_meaningful_text() {
+        let sample = "This page has searchable vector text and should avoid OCR.";
+        assert!(!evaluate_native_text_for_ocr(sample, Some(1)).fallback);
+    }
+
+    #[cfg(feature = "ocr")]
+    #[test]
+    fn test_should_fallback_for_punctuation_only_text() {
+        let sample = " . , ; : -- -- ";
+        assert!(evaluate_native_text_for_ocr(sample, Some(2)).fallback);
+    }
+
+    #[tokio::test]
+    #[cfg(feature = "pdf")]
+    async fn test_pdf_batch_mode_validates_page_config_enabled() {
+        use crate::core::config::PageConfig;
+
+        let extractor = PdfExtractor::new();
+
+        let config = ExtractionConfig {
+            pages: Some(PageConfig {
+                extract_pages: true,
+                insert_page_markers: false,
+                marker_format: "<!-- PAGE {page_num} -->".to_string(),
+            }),
+            ..Default::default()
+        };
+
+        let pdf_path =
+            std::path::Path::new(env!("CARGO_MANIFEST_DIR")).join("../../test_documents/pdfs/google_doc_document.pdf");
+        if let Ok(content) = std::fs::read(pdf_path) {
+            let result = extractor.extract_bytes(&content, "application/pdf", &config).await;
+            assert!(
+                result.is_ok(),
+                "Failed to extract PDF with page config: {:?}",
+                result.err()
+            );
+
+            let extraction_result = result.unwrap();
+            assert!(
+                extraction_result.pages.is_some(),
+                "Pages should be extracted when extract_pages is true"
+            );
+        }
+    }
+
+    #[tokio::test]
+    #[cfg(feature = "pdf")]
+    async fn test_pdf_batch_mode_validates_page_config_disabled() {
+        let extractor = PdfExtractor::new();
+        let config = ExtractionConfig::default();
+
+        let pdf_path =
+            std::path::Path::new(env!("CARGO_MANIFEST_DIR")).join("../../test_documents/pdfs/google_doc_document.pdf");
+        if let Ok(content) = std::fs::read(pdf_path) {
+            let result = extractor.extract_bytes(&content, "application/pdf", &config).await;
+            assert!(
+                result.is_ok(),
+                "Failed to extract PDF without page config: {:?}",
+                result.err()
+            );
+
+            let extraction_result = result.unwrap();
+            assert!(
+                extraction_result.pages.is_none(),
+                "Pages should not be extracted when pages config is None"
+            );
+        }
+    }
+
+    #[tokio::test]
+    #[cfg(feature = "pdf")]
+    async fn test_pdf_page_marker_validation() {
+        use crate::core::config::PageConfig;
+
+        let extractor = PdfExtractor::new();
+
+        let config = ExtractionConfig {
+            pages: Some(PageConfig {
+                extract_pages: true,
+                insert_page_markers: true,
+                marker_format: "\n\n<!-- PAGE {page_num} -->\n\n".to_string(),
+            }),
+            ..Default::default()
+        };
+
+        let pdf_path =
+            std::path::Path::new(env!("CARGO_MANIFEST_DIR")).join("../../test_documents/pdfs/multi_page.pdf");
+        if let Ok(content) = std::fs::read(pdf_path) {
+            let result = extractor.extract_bytes(&content, "application/pdf", &config).await;
+            assert!(
+                result.is_ok(),
+                "Failed to extract PDF with page markers: {:?}",
+                result.err()
+            );
+
+            let extraction_result = result.unwrap();
+            let marker_placeholder = "<!-- PAGE ";
+            if extraction_result.content.len() > 100 {
+                assert!(
+                    extraction_result.content.contains(marker_placeholder),
+                    "Page markers should be inserted when configured and document has multiple pages"
+                );
+            }
+        }
+    }
+
+    #[test]
+    #[cfg(feature = "pdf")]
+    fn test_pdf_extractor_without_feature_pdf() {
+        let extractor = PdfExtractor::new();
+        assert_eq!(extractor.name(), "pdf-extractor");
+    }
+}