Improve type hinting

* Address some typing errors noticed by mypy
* Fix "Incomplete" API type hinting to satisfy stubgen

Author: ctrueden

src/appose/builder/__init__.py

@@ -77,8 +77,8 @@ def __init__(
             verb = "interrupted" if isinstance(cause, KeyboardInterrupt) else "failed"
             message = f"{noun} {verb}"
         super().__init__(message)
-        self.builder = builder
-        self.__cause__ = cause
+        self.builder: Builder | None = builder
+        self.__cause__: Exception | None = cause
 
 
 class Builder(Protocol):
@@ -584,13 +584,13 @@ def __init__(
                 bin_path_list: list[str],
                 launch_arg_list: list[str],
                 env_var_dict: dict[str, str],
-                parent_builder: BaseBuilder,
+                parent_builder: Builder,
             ):
                 super().__init__(base_path)
-                self._bin_paths = bin_path_list
-                self._launch_args = launch_arg_list
-                self._env_vars = env_var_dict
-                self._builder = parent_builder
+                self._bin_paths: list[str] = bin_path_list
+                self._launch_args: list[str] = launch_arg_list
+                self._env_vars: dict[str, str] = env_var_dict
+                self._builder: Builder = parent_builder
 
             def bin_paths(self) -> list[str]:
                 return self._bin_paths

src/appose/builder/mamba.py

@@ -49,6 +49,10 @@ class MambaBuilder(BaseBuilder):
     def __init__(self):
         super().__init__()
 
+        # Note: Already assigned in BaseBuilder, but stubgen wants these here, too.
+        self.source_content: str | None = None
+        self.scheme: str | None = None
+
     def name(self) -> str:
         return "mamba"
 

src/appose/builder/pixi.py

@@ -50,6 +50,10 @@ def __init__(self):
         self.conda_packages: list[str] = []
         self.pypi_packages: list[str] = []
 
+        # Note: Already assigned in BaseBuilder, but stubgen wants these here, too.
+        self.source_content: str | None = None
+        self.scheme: str | None = None
+
     def name(self) -> str:
         return "pixi"
 

src/appose/builder/uv.py

@@ -51,6 +51,10 @@ def __init__(self):
         self.python_version: str | None = None
         self.packages: list[str] = []
 
+        # Note: Already assigned in BaseBuilder, but stubgen wants these here, too.
+        self.source_content: str | None = None
+        self.scheme: str | None = None
+
     def name(self) -> str:
         return "uv"
 

src/appose/service.py

@@ -55,7 +55,7 @@ class TaskException(Exception):
 
     def __init__(self, message: str, task: "Task") -> None:
         super().__init__(message)
-        self.task = task
+        self.task: Task = task
 
     @property
     def status(self) -> "TaskStatus":

src/appose/shm.py

@@ -49,7 +49,7 @@ class which can be used with a `with` statement. When the program flow
     `unlink_on_dispose` flag.
     """
 
-    def __init__(self, name: str = None, create: bool = False, rsize: int = 0):
+    def __init__(self, name: str | None = None, create: bool = False, rsize: int = 0):
         """
         Create a new shared memory block, or attach to an existing one.
 
@@ -146,7 +146,7 @@ class NDArray:
     a particular shape, and flattened into SharedMemory.
     """
 
-    def __init__(self, dtype: str, shape: list[int], shm: SharedMemory = None):
+    def __init__(self, dtype: str, shape: list[int], shm: SharedMemory | None = None):
         """
         Create an NDArray.
         :param dtype: The type of the data elements; e.g. int8, uint8, float32, float64.

src/appose/syntax.py

@@ -198,7 +198,7 @@ def invoke_method(
 ]
 
 
-def get(name: str) -> ScriptSyntax:
+def get(name: str) -> ScriptSyntax | None:
     """
     Detects and returns the script syntax with the given name.
 

src/appose/tool/__init__.py

@@ -58,10 +58,10 @@ def __init__(self, name: str, url: str, command: str, rootdir: str):
             command: Path to the tool's executable command.
             rootdir: Root directory where the tool is installed.
         """
-        self.name = name
-        self.url = url
-        self.command = command
-        self.rootdir = rootdir
+        self.name: str = name
+        self.url: str = url
+        self.command: str = command
+        self.rootdir: str = rootdir
 
         # Consumer callbacks
         self._output_consumer: Callable[[str], None] | None = None

src/appose/tool/mamba.py

@@ -99,15 +99,15 @@ class Mamba(Tool):
     """
 
     # Path where Appose installs Micromamba by default
-    BASE_PATH = str(Path(filepath.appose_envs_dir()) / ".mamba")
+    BASE_PATH: str = str(Path(filepath.appose_envs_dir()) / ".mamba")
 
     # The platform string for micromamba download
-    MICROMAMBA_PLATFORM = _micromamba_platform()
+    PLATFORM: str | None = _micromamba_platform()
 
     # URL from where Micromamba is downloaded to be installed
-    MICROMAMBA_URL = (
-        f"https://micro.mamba.pm/api/micromamba/{MICROMAMBA_PLATFORM}/latest"
-        if MICROMAMBA_PLATFORM
+    DOWNLOAD_URL: str | None = (
+        f"https://micro.mamba.pm/api/micromamba/{PLATFORM}/latest"
+        if PLATFORM
         else None
     )
 
@@ -128,7 +128,7 @@ def __init__(self, rootdir: str | None = None):
 
         command_path = str(Path(root) / mamba_relative_path)
 
-        super().__init__("micromamba", self.MICROMAMBA_URL, command_path, root)
+        super().__init__("micromamba", self.DOWNLOAD_URL, command_path, root)
 
     def _decompress(self, archive: Path) -> None:
         """

src/appose/tool/pixi.py

@@ -70,16 +70,16 @@ class Pixi(Tool):
     """
 
     # Pixi version to download
-    PIXI_VERSION = "v0.58.0"
+    PIXI_VERSION: str = "v0.58.0"
 
     # Path where Appose installs Pixi by default (.pixi subdirectory thereof)
-    BASE_PATH = filepath.appose_envs_dir()
+    BASE_PATH: str = filepath.appose_envs_dir()
 
     # The filename to download for the current platform
-    PIXI_BINARY = _pixi_binary()
+    PIXI_BINARY: str | None = _pixi_binary()
 
     # URL from where Pixi is downloaded to be installed
-    PIXI_URL = (
+    DOWNLOAD_URL: str | None = (
         f"https://github.com/prefix-dev/pixi/releases/download/{PIXI_VERSION}/{PIXI_BINARY}"
         if PIXI_BINARY
         else None
@@ -102,7 +102,7 @@ def __init__(self, rootdir: str | None = None):
 
         command_path = str(Path(root) / pixi_relative_path)
 
-        super().__init__("pixi", self.PIXI_URL, command_path, root)
+        super().__init__("pixi", self.DOWNLOAD_URL, command_path, root)
 
     def _decompress(self, archive: Path) -> None:
         """