feat(pptx): implement error handling and logging in PPTX parser

- Added custom exceptions for PPTX parsing errors, including PptxExtractionError, PptxExtractorError, PptxParserError, PptxPresentationError, and PptxSlideProcessingError.
- Enhanced the PptxDocumentParser to raise appropriate exceptions during parsing failures and log detailed error messages.
- Improved logging throughout the extraction process to track successful and failed extractions, including shape processing and metadata extraction.
- Updated extractor classes to handle errors gracefully and provide informative logs for debugging.

Author: maxpill

packages/ragbits-document-search/src/ragbits/document_search/ingestion/parsers/pptx/__init__.py

@@ -1,5 +1,17 @@
+from .exceptions import (
+    PptxExtractionError,
+    PptxExtractorError,
+    PptxParserError,
+    PptxPresentationError,
+    PptxSlideProcessingError,
+)
 from .parser import PptxDocumentParser
 
 __all__ = [
     "PptxDocumentParser",
+    "PptxExtractionError",
+    "PptxExtractorError",
+    "PptxParserError",
+    "PptxPresentationError",
+    "PptxSlideProcessingError",
 ]

packages/ragbits-document-search/src/ragbits/document_search/ingestion/parsers/pptx/exceptions.py

@@ -0,0 +1,94 @@
+from ragbits.document_search.ingestion.parsers.exceptions import ParserError
+
+
+class PptxParserError(ParserError):
+    """
+    Base class for all PPTX parser related exceptions.
+    """
+
+
+class PptxExtractionError(PptxParserError):
+    """
+    Raised when an extractor fails to extract content from a shape or slide.
+    """
+
+    def __init__(self, extractor_name: str, slide_idx: int, shape_info: str, original_error: Exception) -> None:
+        """
+        Initialize the PptxExtractionError.
+
+        Args:
+            extractor_name: Name of the extractor that failed.
+            slide_idx: Index of the slide where extraction failed.
+            shape_info: Information about the shape that caused the failure.
+            original_error: The original exception that caused the failure.
+        """
+        message = (
+            f"Extractor '{extractor_name}' failed to extract content from slide {slide_idx}. "
+            f"Shape info: {shape_info}. Original error: {original_error}"
+        )
+        super().__init__(message)
+        self.extractor_name = extractor_name
+        self.slide_idx = slide_idx
+        self.shape_info = shape_info
+        self.original_error = original_error
+
+
+class PptxSlideProcessingError(PptxParserError):
+    """
+    Raised when processing of an entire slide fails.
+    """
+
+    def __init__(self, extractor_name: str, slide_idx: int, original_error: Exception) -> None:
+        """
+        Initialize the PptxSlideProcessingError.
+
+        Args:
+            extractor_name: Name of the extractor that failed.
+            slide_idx: Index of the slide that failed to process.
+            original_error: The original exception that caused the failure.
+        """
+        message = (
+            f"Extractor '{extractor_name}' failed to process slide {slide_idx}. " f"Original error: {original_error}"
+        )
+        super().__init__(message)
+        self.extractor_name = extractor_name
+        self.slide_idx = slide_idx
+        self.original_error = original_error
+
+
+class PptxPresentationError(PptxParserError):
+    """
+    Raised when the PPTX presentation cannot be loaded or processed.
+    """
+
+    def __init__(self, file_path: str, original_error: Exception) -> None:
+        """
+        Initialize the PptxPresentationError.
+
+        Args:
+            file_path: Path to the PPTX file that failed to load.
+            original_error: The original exception that caused the failure.
+        """
+        message = f"Failed to load or process PPTX presentation from '{file_path}'. Original error: {original_error}"
+        super().__init__(message)
+        self.file_path = file_path
+        self.original_error = original_error
+
+
+class PptxExtractorError(PptxParserError):
+    """
+    Raised when an extractor fails completely.
+    """
+
+    def __init__(self, extractor_name: str, original_error: Exception) -> None:
+        """
+        Initialize the PptxExtractorError.
+
+        Args:
+            extractor_name: Name of the extractor that failed.
+            original_error: The original exception that caused the failure.
+        """
+        message = f"Extractor '{extractor_name}' failed completely. Original error: {original_error}"
+        super().__init__(message)
+        self.extractor_name = extractor_name
+        self.original_error = original_error