1002-python-style-guide

description: Maintain consistent Python code style with proper docstrings, typing, and indentation globs: src/**/*.py

This class provides functionality to read files, detect their MIME types,
and extract content for further processing.
"""

def __init__(
    self,
    max_workers: int = 2,
    preview_length: int = 100,
    cache_manager: CacheManager | None = None,
) -> None:
    """Initialize FileReader.

    Args:
        max_workers: Maximum number of worker threads for
            concurrent operations
        preview_length: Maximum length of file previews
        cache_manager: Cache manager for caching extraction results
    """
    self.max_workers = max_workers
    self.preview_length = preview_length
    self.logger = logging.getLogger(__name__)

    # Initialize the magic library for file type detection
    self._magic_instance = None

    self.executor = ThreadPoolExecutor(max_workers=max_workers)
    self.cache_manager = cache_manager

    # Initialize the metadata manager
    self.metadata_manager = MetadataManager(cache_manager)

def get_mime_type(self, file_path: str | Path) -> str:
    """Get the MIME type of a file.

    Args:
        file_path: Path to the file

    Returns:
        MIME type of the file

    Raises:
        FileNotFoundError: If the file does not exist
    """
    path = Path(file_path)
    if not path.exists():
        raise FileNotFoundError(f"{path} does not exist.")

    if magic is None:
        self.logger.warning("Magic library not available, using default MIME type")
        return "application/octet-stream"

    # Implementation details...
    return self._fallback_mime_type(path)
  </code>
</good-example>

<bad-example>
  <title>Poor Python Formatting Style</title>
  <code>

class file_reader: # Incorrect class naming (should be PascalCase) '''Class that reads files''' # Incomplete docstring

def __init__(self, max_workers=2, preview_length=100, cache_manager=None):  # Missing type hints
    # No docstring for method
    self.max_workers=max_workers  # Missing space around operator
    self.preview_length = preview_length
    self.logger = logging.getLogger(__name__)

    self._magic_instance = None

    self.executor = ThreadPoolExecutor(max_workers=max_workers)
    self.cache_manager=cache_manager  # Missing space around operator
    self.metadata_manager = MetadataManager(cache_manager)

def get_mime_type(self, file_path):  # Missing type hints
    # Docstring should be more descriptive
    """Get MIME type."""
    path = Path(file_path)
    if not path.exists():
        raise FileNotFoundError("Path %s does not exist." % path)  # Using %-formatting instead of f-string