fix(ci+lint): unblock main + dependabot (#18)

Two preexisting issues kept CI red on main:

1) src/pitchcritic/extractor.py:47 had a 108-char line that ruff
   E501 (max 100) was failing on. Hoisted the size_mb / max_mb
   computations to locals so the f-string fits in 100 cols.
   Message content is identical.

2) .pre-commit-config.yaml pinned `types-all` as a mypy
   additional_dependency. `types-all` is yanked on PyPI and pulls
   the broken `types-pkg-resources` placeholder, so pre-commit
   install always fails. The repo's actual imports (anthropic,
   fastapi, pdfplumber, pydantic, rich, typer) all ship inline
   type info — no external stubs needed.

Net effect: main goes green; dependabot PR #17 (idna 3.11 to 3.15)
should auto-unblock once CI rebuilds.

Author: Techknowmadlabs

.pre-commit-config.yaml

@@ -10,6 +10,5 @@ repos:
     rev: v1.15.0
     hooks:
       - id: mypy
-        additional_dependencies: [types-all]
         args: [--ignore-missing-imports]
         files: ^src/

src/pitchcritic/extractor.py

@@ -43,9 +43,9 @@ def extract_pdf(path: Path | str) -> PitchContent:
     if file_size == 0:
         raise ExtractionError("PDF file is empty (0 bytes)")
     if file_size > MAX_PDF_SIZE_BYTES:
-        raise ExtractionError(
-            f"PDF too large: {file_size / 1024 / 1024:.1f} MB (max {MAX_PDF_SIZE_BYTES // 1024 // 1024} MB)"
-        )
+        size_mb = file_size / 1024 / 1024
+        max_mb = MAX_PDF_SIZE_BYTES // 1024 // 1024
+        raise ExtractionError(f"PDF too large: {size_mb:.1f} MB (max {max_mb} MB)")
 
     content_hash = _file_hash(path)
     if content_hash in _extraction_cache: