Replace mypy with ty, modernize ruff configuration

- Remove mypy config, use ty for type checking
- Configure ruff with comprehensive lint rules (E, F, W, B, I, UP, C4, SIM, RUF)
- Configure isort via ruff with plone-style settings
- Fix lint issues: unused imports, mutable defaults, dict/list literals
- Add ClassVar annotations for mutable class attributes

Author: jensens

CHANGES.md

@@ -7,7 +7,9 @@
 - Feature: Add `qa.ty` domain for Astral's ty type checker.
   ty is an extremely fast Python type checker (10-100x faster than mypy).
   Registers with both CHECK_TARGETS and TYPECHECK_TARGETS for fast feedback.
-  [jensens]
+  [jensens, 2026-01-28]
+- Use ty and extended ruff for check, drop isort and mypy from `pyproject.toml`.
+  [jensens, 2026-01-28]
 
 ## 2.1.0
 

pyproject.toml

@@ -88,20 +88,34 @@ testpaths = [
     "src",
 ]
 
-[tool.isort]
-profile = "black"
-force_alphabetical_sort = true
-force_single_line = true
-lines_after_imports = 2
-
-[tool.mypy]
-ignore_missing_imports = true
-python_version = "3.10"
-
 [tool.ruff]
 target-version = "py310"
 # Exclude a variety of commonly ignored directories.
 exclude = ["src/mxmake/_version.py"]
 
 [tool.ruff.lint]
-select = ["UP"]  # Enable pyupgrade rules for Python 3.10+ modernization
+select = [
+    "E",    # pycodestyle errors
+    "F",    # pyflakes
+    "W",    # pycodestyle warnings
+    "B",    # flake8-bugbear
+    "I",    # isort
+    "UP",   # pyupgrade
+    "C4",   # flake8-comprehensions
+    "SIM",  # flake8-simplify
+    "RUF",  # ruff-specific
+]
+
+[tool.ruff.lint.isort]
+force-single-line = true
+from-first = true
+lines-after-imports = 2
+lines-between-types = 1
+no-sections = true
+order-by-type = false
+
+[tool.ruff.lint.pycodestyle]
+max-line-length = 120  # E501 triggers above this, allows flexibility
+
+[tool.ruff.lint.per-file-ignores]
+"src/mxmake/tests/*" = ["E101", "RUF012"]

src/mxmake/hook.py

@@ -38,7 +38,7 @@ def generate_templates(self, state: mxdev.State):
             try:
                 factory = template.lookup(name, bound=True)
             except RuntimeError as e:
-                msg = f"mxmake: {str(e)}"
+                msg = f"mxmake: {e!s}"
                 logger.warning(msg)
                 continue
             factory(config, environment).write()

src/mxmake/parser.py

@@ -1,8 +1,6 @@
 from mxmake.topics import get_domain
 from pathlib import Path
 
-import typing
-
 
 class SettingMissing(Exception):
     """Exception used in parser to indicate a missing setting in an existing

src/mxmake/templates.py

@@ -25,7 +25,7 @@ def get_template_environment() -> Environment:
 class template:
     """Template decorator and registry."""
 
-    _registry: dict = dict()
+    _registry: typing.ClassVar[dict] = {}
 
     def __init__(self, name: str) -> None:
         self.name = name
@@ -160,15 +160,15 @@ def template_name(self) -> str:
 
     @property
     def template_variables(self) -> dict[str, typing.Any]:
-        return dict(
-            description=self.description,
-            env=self.env,
-            testpaths=self.package_paths(ns_name("test-path")),
-            testargs=self.config.settings.get("mxmake-test-runner-args", ""),
-        )
+        return {
+            "description": self.description,
+            "env": self.env,
+            "testpaths": self.package_paths(ns_name("test-path")),
+            "testargs": self.config.settings.get("mxmake-test-runner-args", ""),
+        }
 
     def package_paths(self, attr: str) -> list[str]:
-        paths = list()
+        paths = []
         for name, package in self.config.packages.items():
             if attr not in package:
                 continue
@@ -220,13 +220,13 @@ def target_folder(self) -> Path:
 
     @property
     def template_variables(self) -> dict[str, typing.Any]:
-        return dict(
-            find_links=[
+        return {
+            "find_links": [
                 link.strip()
                 for link in self.settings.get("find-links", "").split("\n")
                 if link.strip()
             ]
-        )
+        }
 
 
 ##############################################################################
@@ -260,17 +260,17 @@ def template_variables(self) -> dict[str, typing.Any]:
         for domain in self.domains:
             if not domain.settings:
                 continue
-            domain_setting = dict(fqn=domain.fqn, settings=[])
+            domain_setting = {"fqn": domain.fqn, "settings": []}
             settings.append(domain_setting)
             for setting in domain.settings:
                 sfqn = f"{domain.fqn}.{setting.name}"
                 domain_setting["settings"].append(
-                    dict(
-                        name=setting.name,
-                        description=setting.description.split("\n"),
-                        default=setting.default,
-                        value=self.domain_settings.get(sfqn, setting.default),
-                    )
+                    {
+                        "name": setting.name,
+                        "description": setting.description.split("\n"),
+                        "default": setting.default,
+                        "value": self.domain_settings.get(sfqn, setting.default),
+                    }
                 )
         # render domain sections
         sections = io.StringIO()
@@ -284,12 +284,12 @@ def template_variables(self) -> dict[str, typing.Any]:
         topics = {domain.topic for domain in self.domains}
         additional_targets["qa"] = "qa" in topics
         # return template variables
-        return dict(
-            settings=settings,
-            sections=sections,
-            fqns=fqns,
-            additional_targets=additional_targets,
-        )
+        return {
+            "settings": settings,
+            "sections": sections,
+            "fqns": fqns,
+            "additional_targets": additional_targets,
+        }
 
 
 ##############################################################################
@@ -317,7 +317,7 @@ def target_folder(self) -> Path:
 
     @property
     def template_variables(self) -> dict[str, typing.Any]:
-        return dict(additional_sources_targets=self.additional_sources_targets)
+        return {"additional_sources_targets": self.additional_sources_targets}
 
 
 ##############################################################################
@@ -351,24 +351,24 @@ def template_variables(self) -> dict[str, typing.Any]:
         for domain in self.domains:
             if domain.fqn == "qa.test":
                 mxmake_env = True
-                template = dict(name="run-tests", settings=dict(environment="env"))
+                template = {"name": "run-tests", "settings": {"environment": "env"}}
                 mxmake_templates.append(template)
             if domain.fqn == "qa.coverage":
                 mxmake_env = True
-                template = dict(name="run-coverage", settings=dict(environment="env"))
+                template = {"name": "run-coverage", "settings": {"environment": "env"}}
                 mxmake_templates.append(template)
             if domain.fqn == "applications.plone":
-                template = dict(
-                    name="plone-site",
-                    settings=dict(
-                        distribution="volto", extension_ids="plone.volto:default"
-                    ),
-                )
+                template = {
+                    "name": "plone-site",
+                    "settings": {
+                        "distribution": "volto", "extension_ids": "plone.volto:default"
+                    },
+                }
                 mxmake_templates.append(template)
-        return dict(
-            mxmake_templates=mxmake_templates,
-            mxmake_env=mxmake_env,
-        )
+        return {
+            "mxmake_templates": mxmake_templates,
+            "mxmake_env": mxmake_env,
+        }
 
 
 ##############################################################################
@@ -439,7 +439,7 @@ def write(self) -> None:
 
 
 class ci_template:
-    templates: list = list()
+    templates: typing.ClassVar[list] = []
 
     def __init__(self, name: str) -> None:
         self.name = name
@@ -461,9 +461,9 @@ def __init__(
 
     @property
     def template_variables(self) -> dict[str, typing.Any]:
-        return dict(
-            project_path_python=self.settings.get("PROJECT_PATH_PYTHON", ""),
-        )
+        return {
+            "project_path_python": self.settings.get("PROJECT_PATH_PYTHON", ""),
+        }
 
     @property
     def target_folder(self) -> Path:
@@ -547,7 +547,7 @@ def template_variables(self):
             vars["distribution"] = site.pop("distribution")
 
         # handle extension ids
-        if site.get("extension_ids", None) is not None:
+        if site.get("extension_ids") is not None:
             site["extension_ids"] = [
                 eid.strip() for eid in site["extension_ids"].split("\n") if eid.strip()
             ]
@@ -588,5 +588,5 @@ def template_variables(self):
                         domains.append(topic.domain(domain_name.strip()))
                 for domain in domains:
                     for target in domain.targets:
-                        targets.append(dict(name=target.name, folder=folder))
-        return dict(targets=targets)
+                        targets.append({"name": target.name, "folder": folder})
+        return {"targets": targets}

src/mxmake/testing/__init__.py

@@ -26,7 +26,7 @@ def wrapper(self):
 @contextmanager
 def reset_template_registry():
     registry_orgin = templates.template._registry
-    templates.template._registry = dict()
+    templates.template._registry = {}
     try:
         yield
     finally:
@@ -59,17 +59,17 @@ def wrapper(*a):
 class TestConfiguration(mxdev.Configuration):
     def __init__(
         self,
-        settings: dict[str, str] = {},
-        overrides: dict[str, str] = {},
-        ignore_keys: list[str] = [],
-        packages: dict[str, dict[str, str]] = {},
-        hooks: dict[str, dict[str, str]] = {},
+        settings: dict[str, str] | None = None,
+        overrides: dict[str, str] | None = None,
+        ignore_keys: list[str] | None = None,
+        packages: dict[str, dict[str, str]] | None = None,
+        hooks: dict[str, dict[str, str]] | None = None,
     ):
-        self.settings = settings
-        self.overrides = overrides
-        self.ignore_keys = ignore_keys
-        self.packages = packages
-        self.hooks = hooks
+        self.settings = settings if settings is not None else {}
+        self.overrides = overrides if overrides is not None else {}
+        self.ignore_keys = ignore_keys if ignore_keys is not None else []
+        self.packages = packages if packages is not None else {}
+        self.hooks = hooks if hooks is not None else {}
 
 
 class RenderTestCase(unittest.TestCase):

src/mxmake/tests/test_templates.py

@@ -9,7 +9,6 @@
 
 import doctest
 import mxdev
-import typing
 
 
 EXPECTED_DIRECTORY = Path(__file__).parent / "expected"
@@ -23,7 +22,7 @@ def test_template(self):
             class Template(templates.Template):
                 pass
 
-            self.assertEqual(templates.template._registry, dict(template=Template))
+            self.assertEqual(templates.template._registry, {"template": Template})
             self.assertEqual(templates.template.lookup("template"), Template)
             with self.assertRaises(RuntimeError):
                 templates.template.lookup("inexistent")
@@ -60,7 +59,7 @@ class Template(templates.Template):
             target_folder = tempdir
             target_name = "target.out"
             template_name = "target.in"
-            template_variables = dict(param="value")
+            template_variables = {"param": "value"}
 
         # cannot write template without template environment
         template = Template()
@@ -99,9 +98,9 @@ class Template(templates.MxIniBoundTemplate):
         # template settings
         hooks: dict[str, dict] = {}
         template = Template(testing.TestConfiguration(hooks=hooks))
-        self.assertEqual(template.settings, dict())
-        hooks["mxmake-template"] = dict(key="val")
-        self.assertEqual(template.settings, dict(key="val"))
+        self.assertEqual(template.settings, {})
+        hooks["mxmake-template"] = {"key": "val"}
+        self.assertEqual(template.settings, {"key": "val"})
 
     def test_ShellScriptTemplate(self):
         self.assertEqual(templates.ShellScriptTemplate.description, "")
@@ -579,11 +578,13 @@ def test_Makefile(self, tempdir):
 
         template.write()
         print("Makefile written to", tempdir / "Makefile")
-        with (tempdir / "Makefile").open() as result:
-            with open(EXPECTED_DIRECTORY / "Makefile") as expected:
-                self.checkOutput(
-                    expected.read(), result.read(), optionflags=doctest.REPORT_UDIFF
-                )
+        with (
+            (tempdir / "Makefile").open() as result,
+            (EXPECTED_DIRECTORY / "Makefile").open() as expected,
+        ):
+            self.checkOutput(
+                expected.read(), result.read(), optionflags=doctest.REPORT_UDIFF
+            )
 
     @testing.temp_directory
     def test_MxIni(self, tempdir):

src/mxmake/tests/test_topics.py

@@ -6,7 +6,6 @@
 from pathlib import Path
 
 import configparser
-import typing
 import unittest
 
 

src/mxmake/topics.py

@@ -186,7 +186,7 @@ def get_domain(fqn: str) -> Domain:
 
 class DomainConflictError(Exception):
     def __init__(self, counter: Counter):
-        conflicting = list()
+        conflicting = []
         for name, count in counter.items():
             if count > 1:
                 conflicting.append(name)

src/mxmake/utils.py

@@ -31,5 +31,5 @@ def list_value(value: str | None) -> list[str]:
     space. Supports newline.
     """
     if not value:
-        return list()
+        return []
     return [v.strip() for v in value.replace("\n", " ").strip().split(" ")]