Add Cython compilation and update project infrastructure

alganet · alganet · commit ec3ee739874b · 2025-11-19T06:59:23.000Z
- Introduce Cython compilation for performance optimization
 - Update SPDX headers across codebase for REUSE compliance
 - Simplify type annotations and modernize build system
 - Enhance CI/CD workflows and development tooling
 - Support for Python &gt;=3.10
diff --git a/.github/CODE_OF_CONDUCT.md b/.github/CODE_OF_CONDUCT.md
@@ -1,6 +1,8 @@
-<!-- SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com> -->
-<!-- SPDX-License-Identifier: ISC -->
+<!--
+SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
 
+SPDX-License-Identifier: ISC
+-->
 # Code of Conduct
 
 ## Our Pledge
diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md
@@ -1,6 +1,8 @@
-<!-- SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com> -->
-<!-- SPDX-License-Identifier: ISC -->
+<!--
+SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
 
+SPDX-License-Identifier: ISC
+-->
 # Contributing
 
 ## Development Setup
diff --git a/.github/ISSUE_TEMPLATES/bug_report.md b/.github/ISSUE_TEMPLATES/bug_report.md
@@ -1,6 +1,8 @@
-<!-- SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com> -->
-<!-- SPDX-License-Identifier: ISC -->
+<!--
+SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
 
+SPDX-License-Identifier: ISC
+-->
 ---
 name: Bug Report
 about: Report a bug
diff --git a/.github/ISSUE_TEMPLATES/feature_request.md b/.github/ISSUE_TEMPLATES/feature_request.md
@@ -1,6 +1,8 @@
-<!-- SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com> -->
-<!-- SPDX-License-Identifier: ISC -->
+<!--
+SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
 
+SPDX-License-Identifier: ISC
+-->
 ---
 name: Feature Request
 about: Suggest a new feature
diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md
@@ -1,6 +1,8 @@
-<!-- SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com> -->
-<!-- SPDX-License-Identifier: ISC -->
+<!--
+SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
 
+SPDX-License-Identifier: ISC
+-->
 ## Description
 
 Brief description of changes.
diff --git a/.github/SECURITY.md b/.github/SECURITY.md
@@ -1,6 +1,8 @@
-<!-- SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com> -->
-<!-- SPDX-License-Identifier: ISC -->
+<!--
+SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
 
+SPDX-License-Identifier: ISC
+-->
 # Security Policy
 
 ## Reporting Vulnerabilities
diff --git a/.github/copilot-instructions.md b/.github/copilot-instructions.md
@@ -1,6 +1,8 @@
-<!-- SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com> -->
-<!-- SPDX-License-Identifier: ISC -->
+<!--
+SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
 
+SPDX-License-Identifier: ISC
+-->
 # AI Coding Guidelines for apywire
 
 ## Overview
@@ -15,8 +17,8 @@ apywire is a Python library for object wiring with lazy loading. It provides dep
 
 ## Developer Workflows
 - **Full Check**: `make all` (format, lint, test coverage)
-- **Build & Publish**: `make build` (creates dist/), `make publish` (uploads to PyPI)
-- **Clean**: `make clean` (removes __pycache__, .coverage, etc.)
+- **Build & Publish**: `make dist` (ensures dist/), `make publish` (uploads to PyPI)
+- **Clean**: `make clean` (removes dist/, __pycache__, .coverage, etc.)
 - **Debug**: Use `pytest -xvs` for verbose test runs
 
 ## Conventions
@@ -29,7 +31,6 @@ apywire is a Python library for object wiring with lazy loading. It provides dep
 ## Examples
 - **Lazy Access**: `wired = wire(spec); obj = wired.someName` (instantiates on first access)
 - **Spec Format**: `{"module.Class name": {"param": "value"}}` (YAML/JSON-compatible structure)
-- **Generic Usage**: Functions use `[T]` for type safety without runtime generics
 - **Equivalence**: `compile(spec)` generates code that behaves identically to `wire(spec)`
 
 ## Key Files
diff --git a/.github/dependabot.yml b/.github/dependabot.yml
@@ -1,6 +1,6 @@
 # SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
+#
 # SPDX-License-Identifier: ISC
-
 version: 2
 updates:
   - package-ecosystem: "pip"
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -1,6 +1,8 @@
 # SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
+#
 # SPDX-License-Identifier: ISC
 
+
 name: CI
 
 on:
@@ -14,7 +16,7 @@ jobs:
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        python-version: ["3.12", "3.13"]
+        python-version: ["3.10", "3.11", "3.12", "3.13"]
 
     steps:
     - uses: actions/checkout@v5
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -1,6 +1,8 @@
 # SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
+#
 # SPDX-License-Identifier: ISC
 
+
 name: Release
 
 on:
@@ -18,7 +20,7 @@ jobs:
       with:
         python-version: '3.13'
     - run: make pip
-    - run: make build
+    - run: make dist
     - run: make publish
       env:
         TWINE_USERNAME: __token__
diff --git a/.gitignore b/.gitignore
@@ -1,12 +1,19 @@
 # SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
+#
 # SPDX-License-Identifier: ISC
 
+# Project Specific
+apywire/**.c
+
+# General
 __pycache__/
 *.pyc
 *.pyo
 *.pyd
+*.so
 .mypy_cache/
 .pytest_cache/
 .coverage
 *.egg-info/
-dist/
+dist/
+build/
diff --git a/Makefile b/Makefile
@@ -1,15 +1,23 @@
 # SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
+#
 # SPDX-License-Identifier: ISC
 
-.PHONY: all format lint test coverage clean build publish pip
+.PHONY: all format lint test coverage clean publish pip reuse
 
-all: format lint coverage
+all: format lint coverage build
 
 pip:
 	python -m pip install --upgrade pip
 	python -m pip install -e ".[dev]"
 
-format:
+reuse:
+	reuse annotate \
+		--copyright "Alexandre Gomes Gaigalas <alganet@gmail.com>" \
+		--license ISC \
+		--recursive .
+
+# if it gets too slow, separate reuse from format
+format: reuse
 	python -m black .
 	python -m isort .
 
@@ -22,14 +30,19 @@ test:
 	python -m pytest -q
 
 coverage:
-	python -m pytest --cov=apywire --cov-report=term-missing --cov-fail-under=95
+	rm -f apywire/*.so apywire/wiring.c
+	python -m pytest --cov
 
 clean:
 	find . -name __pycache__ -type d -exec rm -rf {} +
 	rm -f *.pyc *.pyo *.pyd .coverage
-	rm -rf .mypy_cache .pytest_cache *.egg-info dist
+	rm -f apywire/*.so apywire/wiring.c
+	rm -rf .mypy_cache .pytest_cache *.egg-info dist build
 
 build:
+	python setup.py build_ext --inplace
+
+dist:
 	python -m build
 
 publish:
diff --git a/README.md b/README.md
@@ -1,6 +1,8 @@
-<!-- SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com> -->
-<!-- SPDX-License-Identifier: ISC -->
+<!--
+SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
 
+SPDX-License-Identifier: ISC
+-->
 # apywire
 
 A Python package to wire up objects.
diff --git a/REUSE.toml b/REUSE.toml
@@ -9,3 +9,4 @@ path = "**/__pycache__/**"
 precedence = "aggregate"
 SPDX-FileCopyrightText = "2025 Alexandre Gomes Gaigalas <alganet@gmail.com>"
 SPDX-License-Identifier = "ISC"
+
diff --git a/apywire/__init__.py b/apywire/__init__.py
@@ -1,4 +1,5 @@
 # SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
+#
 # SPDX-License-Identifier: ISC
 
 """A package to wire up objects."""
diff --git a/apywire/wiring.py b/apywire/wiring.py
@@ -1,20 +1,26 @@
 # SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
+#
 # SPDX-License-Identifier: ISC
 
 """Core wiring functionality."""
 
 import importlib
-from typing import Dict, List, Tuple, TypeAlias, TypeVar, cast
-
-T = TypeVar("T")
+from typing import (
+    Callable,
+    Dict,
+    List,
+    Tuple,
+    TypeAlias,
+    cast,
+)
 
 InstanceData: TypeAlias = Dict[str, object]
 Blueprint: TypeAlias = Tuple[str, str, str, InstanceData]
 
 Spec: TypeAlias = Dict[str, InstanceData]
 
 
-def _parse_spec[T](spec: Spec) -> List[Blueprint]:
+def _parse_spec(spec: Spec) -> List[Blueprint]:
     """Parses a Spec into Blueprints for instantiation."""
     items = []
     for key, value in spec.items():
@@ -26,7 +32,7 @@ def _parse_spec[T](spec: Spec) -> List[Blueprint]:
     return items
 
 
-class Wired[T]:
+class Wired:
     """Lazy-loaded container for wired objects."""
 
     def __init__(self, parsed: List[Blueprint]) -> None:
@@ -43,32 +49,36 @@ def __getattr__(self, name: str) -> object:
         if name not in self._instances:
             module_name, class_name, value = self._parsed[name]
             module = importlib.import_module(module_name)
-            cls = cast(type[T], getattr(module, class_name))
+            cls = cast(  # type: ignore[explicit-any,misc]
+                # Less type-safe than generics but necessary for Cython
+                Callable[..., object],
+                getattr(module, class_name),
+            )
             self._instances[name] = cls(**value)
         return self._instances[name]
 
 
-def wire[T](spec: Spec) -> Wired[T]:
+def wire(spec: Spec) -> Wired:
     """Creates a lazy-loaded Wired object from a Spec."""
     return Wired(_parse_spec(spec))
 
 
-def compile[T](spec: Spec) -> str:
+def compile(spec: Spec) -> str:
     """Compiles a Spec into a string containing Python code."""
     parsed = _parse_spec(spec)
     modules = set()
     properties = []
     for module_name, class_name, name, value in parsed:
         modules.add(module_name)
         type_str = f"{module_name}.{class_name}"
-        if isinstance(value, dict):
-            kwargs = ", ".join(f"{k}={repr(v)}" for k, v in value.items())
-            prop = (
-                "    @property\n"
-                f"    def {name}(self):\n"
-                f"        return {type_str}({kwargs})"
-            )
-            properties.append(prop)
+        # value is always dict per type
+        kwargs = ", ".join(f"{k}={repr(v)}" for k, v in value.items())
+        prop = (
+            "    @property\n"
+            f"    def {name}(self):\n"
+            f"        return {type_str}({kwargs})"
+        )
+        properties.append(prop)
     imports = [f"import {module}" for module in sorted(modules)]
     class_def = ["class Compiled:"] + properties + ["compiled = Compiled()"]
     return "\n".join(imports + class_def)
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,30 +1,32 @@
 # SPDX-FileCopyrightText: 2025 Alexandre Gomes Gaigalas <alganet@gmail.com>
+#
 # SPDX-License-Identifier: ISC
 
 [build-system]
-requires = ["setuptools", "wheel"]
+requires = ["setuptools", "wheel", "Cython"]
 build-backend = "setuptools.build_meta"
 
 [project]
 name = "apywire"
 version = "0.1.0"
 description = "A package to wire up objects"
 readme = "README.md"
-license = {text = "ISC"}
+license = "ISC"
 authors = [{name = "Alexandre Gomes Gaigalas", email = "alganet@gmail.com"}]
 maintainers = [{name = "Alexandre Gomes Gaigalas", email = "alganet@gmail.com"}]
 keywords = ["object", "wiring", "dependency", "injection"]
 classifiers = [
     "Development Status :: 3 - Alpha",
     "Intended Audience :: Developers",
-    "License :: OSI Approved :: ISC License (ISCL)",
     "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3.10",
+    "Programming Language :: Python :: 3.11",
     "Programming Language :: Python :: 3.12",
     "Programming Language :: Python :: 3.13",
     "Topic :: Software Development :: Libraries :: Python Modules",
 ]
 dependencies = []
-requires-python = ">=3.12"
+requires-python = ">=3.10"
 
 [project.urls]
 Homepage = "https://github.com/alganet/apywire"
@@ -36,13 +38,15 @@ dev = [
     "black",
     "build",
     "coverage",
+    "Cython",
     "flake8",
     "isort",
     "mypy",
     "pytest-cov",
     "pytest",
     "reuse",
     "twine",
+    "types-setuptools",
 ]
 
 [tool.setuptools.packages.find]
@@ -71,3 +75,11 @@ warn_unused_configs = true
 
 [tool.pytest.ini_options]
 testpaths = ["tests"]
+
+[tool.coverage.run]
+branch = true
+source = ["apywire"]
+
+[tool.coverage.report]
+show_missing = true
+fail_under = 95
diff --git a/setup.py b/setup.py
diff --git a/tests/__init__.py b/tests/__init__.py
diff --git a/tests/test_simple.py b/tests/test_simple.py
