Fix CI type checking and lxml build failures

- Add loguru to dev dependencies to ensure consistent type checking
  environment across local and CI (with/without --all-extras)
- Remove now-unused type: ignore comments in logger.py, conftest.py,
  and test_word_loader.py since loguru is always present in dev
- Restore type: ignore[assignment] in logger.py for the logging fallback
- Add libxml2-dev and libxslt-dev system packages for Python 3.14 CI
  to support lxml source builds (no pre-built wheel exists for 3.14)

Fixes ty type check errors that occurred when loguru was installed
(CI with --all-extras) but not locally (dev without extras).

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>

Author: fsecada01

.claude/settings.local.json

@@ -0,0 +1,36 @@
+{
+  "permissions": {
+    "allow": [
+      "Bash(git add:*)",
+      "Bash(git commit:*)",
+      "WebSearch",
+      "Bash(uv sync:*)",
+      "Bash(pre-commit uninstall:*)",
+      "Bash(uv run:*)",
+      "Bash(git push:*)",
+      "Bash(gh run list:*)",
+      "Bash(gh run view:*)",
+      "Bash(git check-ignore:*)",
+      "Bash(rtk git add:*)",
+      "Bash(rtk git:*)",
+      "Bash(gh api:*)",
+      "WebFetch(domain:github.com)",
+      "Bash(git rm:*)",
+      "WebFetch(domain:pdoc.dev)",
+      "WebFetch(domain:raw.githubusercontent.com)",
+      "WebFetch(domain:api.github.com)",
+      "Bash(echo:*)",
+      "Bash(dir:*)",
+      "Bash(git -C \"C:/dev/python/modules/TextSpitter\" status --short)",
+      "Bash(git -C \"C:/dev/python/modules/TextSpitter\" rm tests/txt_test.py)",
+      "Bash(git -C \"C:/dev/python/modules/TextSpitter\" add core_requirements.in core_requirements.txt dev_requirements.in dev_requirements.txt requirements.txt .pre-commit-config.yaml TextSpitter/__init__.py TextSpitter/main.py TextSpitter/core.py tests/test_cli.py tests/test_txt.py CONTRIBUTING.md CHANGELOG.md README.md uv.lock)",
+      "Bash(python -m pytest:*)",
+      "Bash(findstr:*)",
+      "Bash(git -C \"C:/dev/python/modules/TextSpitter\" add TextSpitter/cli.py tests/test_csv.py pyproject.toml .github/workflows/tests.yml uv.lock)",
+      "Bash(git -C:*)",
+      "Bash(uv:*)",
+      "Bash(.venv/Scripts/ty.exe check:*)",
+      "Bash(\".venv/Scripts/ty.exe\" --version)"
+    ]
+  }
+}

.coverage

@@ -23,6 +23,10 @@ jobs:
       - name: Set up Python ${{ matrix.python-version }}
         run: uv python install ${{ matrix.python-version }}
 
+      - name: Install system dependencies (lxml build deps for Python 3.14)
+        if: matrix.python-version == '3.14'
+        run: sudo apt-get install -y libxml2-dev libxslt-dev
+
       - name: Install dependencies
         run: uv sync --all-extras --dev
 

.github/workflows/tests.yml

@@ -6,8 +6,8 @@
 """
 
 try:
-    from loguru import logger  # type: ignore[import]
+    from loguru import logger
 except ImportError:
     import logging
 
-    logger = logging.getLogger("textspitter")
+    logger = logging.getLogger("textspitter")  # type: ignore[assignment]

TextSpitter/logger.py

@@ -109,6 +109,7 @@ dev = [
     "isort",
     "jupyterlab",
     "jupyterlab-code-formatter",
+    "loguru",
     "pdoc",
     "prek",
     "pytest",

pyproject.toml

@@ -31,9 +31,9 @@ def test_something(log_capture):
         def _sink(message):
             messages.append(message.record["message"])
 
-        handler_id = logger.add(_sink, level="DEBUG", format="{message}")  # type: ignore[call-non-callable]
+        handler_id = logger.add(_sink, level="DEBUG", format="{message}")
         yield messages
-        logger.remove(handler_id)  # type: ignore[union-attr]
+        logger.remove(handler_id)
     else:
         # stdlib logging fallback
         class _ListHandler(logging.Handler):

tests/conftest.py

@@ -7,7 +7,7 @@
 try:
     from TextSpitter.logger import logger as word_loader_logger_instance
 except ImportError:
-    from loguru import logger as word_loader_logger_instance  # type: ignore[import]
+    from loguru import logger as word_loader_logger_instance
 
 from TextSpitter.core import FileExtractor
 from TextSpitter.main import WordLoader
@@ -32,13 +32,13 @@ def loguru_test_sink(request):  # request is a pytest fixture
     def sink_function(message):
         log_messages.append(message.record["message"])
 
-    handler_id = word_loader_logger_instance.add(  # type: ignore[call-non-callable]
+    handler_id = word_loader_logger_instance.add(
         sink_function, format="{message}"
     )
     yield log_messages
 
     try:
-        word_loader_logger_instance.remove(handler_id)  # type: ignore[call-non-callable]
+        word_loader_logger_instance.remove(handler_id)
     except ValueError:
         pass  # Handler already removed or never added