fix: silence the three remaining pyright errors

cast() the single-task lookup in the test (tasks() returns a dict for a uuid),

add a targeted pyright ignore for the py2app options stub limitation,

and run pyright via pipenv so it resolves project imports.

Author: AlexanderWillner

Makefile

@@ -95,8 +95,8 @@ code-lint: code-style ## Lint the code
 
 code-check-types: ## Check the code types
 	@echo Pyright...
-	@if type pyright >/dev/null 2>&1 ; then PYRIGHT_PYTHON_FORCE_VERSION=latest pyright *.py $(SRC_CORE) $(SRC_TEST) ; \
-	 else echo "SKIPPED. Run 'npm install -f pyright' first." >&2 ; fi
+	@if type $(PIPENV) >/dev/null 2>&1 ; then PYRIGHT_PYTHON_FORCE_VERSION=latest $(PIPENV) run pyright *.py $(SRC_CORE) $(SRC_TEST) ; \
+	 else echo "SKIPPED. Run '$(PIP) install pyright' or 'npm install -f pyright' first." >&2 ; fi
 	@echo MyPy...
 	@if type mypy >/dev/null 2>&1 ; then mypy --check-untyped-defs --no-error-summary --ignore-missing-imports *.py $(SRC_CORE) $(SRC_TEST) ; \
 	 else echo "SKIPPED. Run '$(PIP) install mypy' first." >&2 ; fi

setup.py

@@ -23,6 +23,6 @@ def package_files(directory):
     app=[""],
     packages=find_packages(),
     data_files=package_files(""),
-    options={"py2app": {"argv_emulation": False}},
+    options={"py2app": {"argv_emulation": False}},  # pyright: ignore[reportArgumentType]
     setup_requires=["py2app"],
 )

tests/test_things.py

@@ -8,6 +8,7 @@
 import tracemalloc
 import unittest
 import unittest.mock
+from typing import cast
 
 import things
 import things.database
@@ -453,7 +454,7 @@ def test_thingsdate(self):
         self.assertEqual("AND stopDate IS NULL", sqlfilter)
 
     def test_thingstime(self):
-        test_task = things.tasks("7F4vqUNiTvGKaCUfv5pqYG")
+        test_task = cast(dict, things.tasks("7F4vqUNiTvGKaCUfv5pqYG"))
         self.assertEqual(test_task.get("reminder_time"), "12:34")