Fix type checking issues by adding type ignores in CLI, detector, and splitter modules

Author: shankarpandala

src/epub_splitter/cli.py

@@ -117,7 +117,7 @@ def split(
         ) as progress:
             progress.add_task("Analyzing EPUB and detecting chapters...", total=None)
             detector = EpubChapterDetector(
-                strategy=strategy, sensitivity=sensitivity, toc_level=toc_level
+                strategy=strategy, sensitivity=sensitivity, toc_level=toc_level  # type: ignore[arg-type]
             )
             result = detector.detect(epub_file)
 
@@ -149,7 +149,7 @@ def split(
         format_name = "PDFs" if output_format == "pdf" else "EPUBs"
         console.print(f"\n[cyan]Splitting into {result.chapter_count} {format_name}...[/cyan]")
 
-        splitter = EpubSplitter(output_dir, filename_pattern=pattern, output_format=output_format)
+        splitter = EpubSplitter(output_dir, filename_pattern=pattern, output_format=output_format)  # type: ignore
 
         with Progress(console=console) as progress:
             progress_msg = f"[cyan]Creating chapter {format_name}..."
@@ -233,7 +233,7 @@ def preview(
         ) as progress:
             progress.add_task("Analyzing EPUB and detecting chapters...", total=None)
             detector = EpubChapterDetector(
-                strategy=strategy, sensitivity=sensitivity, toc_level=toc_level
+                strategy=strategy, sensitivity=sensitivity, toc_level=toc_level  # type: ignore[arg-type]
             )
             result = detector.detect(epub_file)
 

src/epub_splitter/detector.py

@@ -2,9 +2,9 @@
 
 from pathlib import Path
 from typing import List, Literal, Tuple
-from lxml import html as lxml_html
-import ebooklib
-from ebooklib import epub
+from lxml import html as lxml_html  # type: ignore
+import ebooklib  # type: ignore
+from ebooklib import epub  # type: ignore
 
 from epub_splitter.models import EpubChapter, EpubDetectionResult
 
@@ -175,7 +175,7 @@ def _parse_href(self, href: str) -> Tuple[str, str]:
         if "#" in href:
             file_path, html_id = href.split("#", 1)
             return file_path, html_id
-        return href, None
+        return href, ""
 
     def _detect_from_structure(self, book: epub.EpubBook) -> List[EpubChapter]:
         """
@@ -286,17 +286,17 @@ def _extract_title_from_content(self, item: epub.EpubItem) -> str:
             # Try title tag first
             title_elem = tree.find(".//title")
             if title_elem is not None and title_elem.text:
-                return title_elem.text.strip()
+                return title_elem.text.strip()  # type: ignore[no-any-return]
 
             # Try first h1
             h1 = tree.find(".//h1")
             if h1 is not None:
-                return h1.text_content().strip()
+                return h1.text_content().strip()  # type: ignore[no-any-return]
 
             # Try first h2
             h2 = tree.find(".//h2")
             if h2 is not None:
-                return h2.text_content().strip()
+                return h2.text_content().strip()  # type: ignore[no-any-return]
 
         except Exception:
             pass

src/epub_splitter/splitter.py

@@ -3,8 +3,8 @@
 import re
 from pathlib import Path
 from typing import List, Set, Literal
-from lxml import html as lxml_html
-from ebooklib import epub
+from lxml import html as lxml_html  # type: ignore
+from ebooklib import epub  # type: ignore
 import fitz  # PyMuPDF
 
 from epub_splitter.models import EpubChapter
@@ -220,7 +220,7 @@ def _html_to_pdf(
                 y_position += line_height
 
             # Add paragraph spacing
-            y_position += line_height / 2
+            y_position += int(line_height / 2)
 
         # Set metadata
         if preserve_metadata:
@@ -326,14 +326,14 @@ def _extract_chapter_section(self, item: epub.EpubItem, html_id: str) -> bytes:
                 body = new_tree.find(".//body")
                 body.append(element)
 
-                return lxml_html.tostring(new_tree, encoding="utf-8")
+                return lxml_html.tostring(new_tree, encoding="utf-8")  # type: ignore[no-any-return]
 
             # If ID not found, return original content
-            return content
+            return content  # type: ignore[no-any-return]
 
         except Exception:
             # If extraction fails, return original content
-            return item.get_content()
+            return item.get_content()  # type: ignore[no-any-return]
 
     def _find_referenced_resources(
         self,