chores: updated pyproject, ruff, mypy and AGENTS.md (#32)

Author: lackhove

AGENTS.md

@@ -21,9 +21,7 @@ uv sync --locked --all-extras --dev
 
 | Task | Command |
 |---|---|
-| Format check | `uv run ruff format --check .` |
 | Auto-format | `uv run ruff format .` |
-| Lint | `uv run ruff check .` |
 | Lint + auto-fix | `uv run ruff check --fix --unsafe-fixes .` |
 | Type check | `uv run mypy .` |
 | Run all tests | `uv run pytest` |
@@ -32,7 +30,7 @@ uv sync --locked --all-extras --dev
 
 **Before marking any task as complete, always run:**
 ```bash
-uv run ruff check . && uv run mypy . && uv run pytest
+uv run ruff check --fix --unsafe-fixes . && uv run mypy . && uv run pytest
 ```
 
 ---
@@ -70,60 +68,7 @@ PEP8 + Black. Run `uv run ruff check --fix --unsafe-fixes .` then `uv run ruff f
 
 ---
 
-## Import Conventions
 
-Every source file begins with:
-
-```python
-from __future__ import annotations
-```
-
-Import ordering within a file:
-
-1. `from __future__ import annotations` — always first
-2. Standard library bare imports (`import os`, `import sys`, …)
-3. Standard library `from … import …`
-4. Third-party bare imports
-5. Third-party `from … import …`
-6. `if TYPE_CHECKING:` block — types needed only for annotations
-
-```python
-from __future__ import annotations
-
-import contextlib
-import os
-from pathlib import Path
-from typing import TYPE_CHECKING, Any
-
-import coverage
-
-if TYPE_CHECKING:
-    from collections.abc import Iterable
-    from coverage.types import TLineNo
-    from tree_sitter import Node
-```
-
-- Within the package: use **relative** imports (`from .plugin import ShellPlugin`)
-- In tests and externally: use **absolute** imports (`from coverage_sh.plugin import …`)
-- Put `TYPE_CHECKING`-only imports in the `if TYPE_CHECKING:` block to avoid runtime cost
-
----
-
-## Naming Conventions
-
-| Construct | Convention | Example |
-|---|---|---|
-| Modules / files | `snake_case` | `plugin.py`, `test_plugin.py` |
-| Classes | `PascalCase` | `ShellFileReporter`, `CovLineParser` |
-| Test classes | `Test` prefix | `TestShellFileReporter` |
-| Functions / methods | `snake_case` | `find_executable_files()` |
-| Private methods | leading `_` | `_parse_ast()`, `_iterdir()` |
-| Test methods | `test_<what>_should_<expected_outcome>` | `test_source_should_be_cached` |
-| Module constants | `UPPER_SNAKE_CASE` | `TRACEFILE_PREFIX`, `TMP_PATH` |
-| Local variables | `snake_case` | `fifo_path`, `line_data` |
-| Type aliases | `PascalCase` | `LineData = dict[str, set[int]]` |
-
----
 
 ## Error Handling
 

coverage_sh/plugin.py

@@ -155,7 +155,7 @@ def write(self, line_data: LineData) -> None:
         )
 
         coverage_data.add_file_tracers(
-            {f: "coverage_sh.ShellPlugin" for f in line_data}
+            dict.fromkeys(line_data, "coverage_sh.ShellPlugin")
         )
         coverage_data.add_lines(line_data)
         coverage_data.write()

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "coverage-sh"
-version = "0.4.5    "
+version = "0.4.5"
 description = "A  Coverage.py plugin to measure code coverage of shell scripts from python."
 authors = [
     { name = "Kilian Lackhove", email = "kilian@lackhove.de" }
@@ -13,11 +13,10 @@ dependencies = [
 ]
 readme = "README.md"
 requires-python = ">= 3.9"
-license = { text = "MIT" }
+license = "MIT"
 keywords = ["coverage", "plugin", "shell", "bash", "sh"]
 classifiers = [
     "Development Status :: 2 - Pre-Alpha",
-    "License :: OSI Approved :: MIT License",
     "Programming Language :: Python :: 3.9",
     "Programming Language :: Python :: 3.10",
     "Programming Language :: Python :: 3.11",
@@ -34,15 +33,12 @@ build-backend = "hatchling.build"
 
 [dependency-groups]
 dev = [
-    "ruff==0.3.2",
-    "mypy==1.9.0",
+    "ruff==0.15.*",
+    "mypy>=1.19",
     "pytest>=7.4.3",
     "setuptools>=69.0.2",
 ]
 
-[tool.hatch.metadata]
-allow-direct-references = true
-
 [tool.hatch.build.targets.wheel]
 packages = ["coverage_sh"]
 
@@ -60,10 +56,10 @@ select = ["ALL"]
 ignore = [
     "E501", # Line too long
     "D", # missing docstring
-    "ANN101", # Missing type annotation for `self` in method
     "ANN204", # Missing return type annotation for special method `__init__`
     "ANN002", # Missing type annotation for `*args`
     "ANN003", # Missing type annotation for `**kwargs`
+    "S311", # Standard pseudo-random generators are not suitable for cryptographic purposes (we use os.urandom)
     "S603", # subprocess-without-shell-equals-true
     "TD", # flake8-todos
     "FIX", #flake8-fixme
@@ -74,7 +70,7 @@ ignore = [
     "EM102", # Exception must not use an f-string literal, assign to variable first
 ]
 
-[tool.ruff.per-file-ignores]
+[tool.ruff.lint.per-file-ignores]
 "tests/**/*.py" = ["ANN", "D", "S101", "PLR2004", "N802", "PT015", "B011", "INP001", "FBT001", "T201"]
 "tests/resources/**/*.py" = ["ALL"]
 

tests/conftest.py

@@ -7,12 +7,12 @@
 import pytest
 
 
-@pytest.fixture()
+@pytest.fixture
 def resources_dir():
     return Path(__file__).parent / "resources"
 
 
-@pytest.fixture()
+@pytest.fixture
 def dummy_project_dir(resources_dir: Path, tmp_path: Path) -> Path:
     """Fixture for a temporary copy of `testproject`"""
     source = resources_dir / "testproject"

tests/test_plugin.py

@@ -137,12 +137,12 @@
 INNER_PY_EXECUTED_LINES = [2]
 
 
-@pytest.fixture()
+@pytest.fixture
 def examples_dir(resources_dir):
     return resources_dir / "examples"
 
 
-@pytest.fixture()
+@pytest.fixture
 def syntax_example_path(resources_dir, tmp_path):
     original_path = resources_dir / "syntax_example.sh"
     working_copy_path = tmp_path / "syntax_example.sh"
@@ -199,7 +199,7 @@ def test_end2end(
 
 
 @pytest.fixture(scope="session")
-def covpy_installs_pth_at_install_time() -> None:  # noqa: PT004
+def covpy_installs_pth_at_install_time() -> None:
     """Skip if coveragepy does not install a .pth file into site-packages at install time.
 
     - >= 7.13.0: writes the .pth file during pip install — reliable in all environments.
@@ -255,7 +255,7 @@ def test_end2end_python_subprocess_via_shell(
 
 
 class TestShellFileReporter:
-    @pytest.fixture()
+    @pytest.fixture
     def reporter(self, syntax_example_path):
         return ShellFileReporter(str(syntax_example_path))