fix: update version pinning, ruff and pyright default checks, test runner and clean up Makefile logic

Author: idatsy

.github/workflows/ci.yml

@@ -8,14 +8,11 @@ jobs:
     steps:
     - uses: actions/checkout@v4
 
-    # - uses: webfactory/ssh-agent@v0.9.0
-    #  with:
-    #  ssh-private-key: ${{ secrets.DEPS_SSH_KEY }}
-
-    - uses: astral-sh/setup-uv@v6
-    - uses: actions/setup-python@v5
+    - uses: astral-sh/setup-uv@v7
       with:
-        python-version-file: .python-version
+        version: "0.5.11"
+        enable-cache: true
+        version-file: .python-version
 
     - run: uv sync --locked --all-groups
 

Makefile

@@ -1,9 +1,8 @@
 .PHONY: help install format lint lint-fix typecheck docstring style fix \
-        check sync clean test stats watch live-stream run-client reclaim \
-        monitor monitor-charts
+        check sync clean test stats watch
 .DEFAULT_GOAL := help
 
-TEST_FLAGS := -xvv -s -p no:anchorpy
+TEST_FLAGS := -xvv -s
 
 install: ## Create / update virtual‑env with all dependency groups
 	uv sync --all-groups
@@ -31,8 +30,8 @@ fix: format lint-fix ## Run all formatters and fixers
 check: fix 	typecheck ## Run all checks (format, lint‑fix, typecheck)
 
 sync: ## Re‑lock and install latest versions
-	uv lock --upgrade       # rebuild uv.lock with newer pins
-	uv sync --all-groups    # install everything into .venv
+	uv lock --upgrade
+	uv sync --all-groups --reinstall
 
 clean: ## Remove build artefacts and caches
 	rm -rf .ruff_cache/ .mypy_cache/ .pytest_cache/ dist/ build/
@@ -55,10 +54,10 @@ stats: ## Show code quality statistics
 #   make watch path/to/file   → watch single test file
 watch:
 	@if [ -z "$(filter-out $@,$(MAKECMDGOALS))" ]; then \
-		uv run pytest-watcher . --runner "pytest tests $(TEST_FLAGS) -W ignore::DeprecationWarning"; \
+		uv run pytest-watcher . -- tests $(TEST_FLAGS) -W ignore::DeprecationWarning; \
 	else \
 		path_arg="$(filter-out $@,$(MAKECMDGOALS))"; \
-		uv run pytest-watcher . --runner "pytest src/tests/$$path_arg $(TEST_FLAGS) -W ignore::DeprecationWarning"; \
+		uv run pytest-watcher . -- tests/$$path_arg $(TEST_FLAGS) -W ignore::DeprecationWarning; \
 	fi
 
 # Pattern rule so additional args do not trigger "No rule to make target"

pyproject.toml

@@ -7,7 +7,9 @@ authors = [
     { name = "idatsy", email = "stefan@ranvier.co.uk" }
 ]
 requires-python = ">=3.13"
-dependencies = []
+dependencies = [
+    "loguru",
+]
 
 [project.scripts]
 python-base-uv = "python_base_uv:main"
@@ -16,32 +18,52 @@ python-base-uv = "python_base_uv:main"
 requires = ["hatchling"]
 build-backend = "hatchling.build"
 
+[tool.uv]
+default-groups = ["dev"]
+
+[tool.pytest.ini_options]
+asyncio_mode = "auto"
+asyncio_default_fixture_loop_scope = "function"
+
 [dependency-groups]
 dev = [
-    "pyrefly>=0.25.1",
-    "pyright>=1.1.403",
-    "pytest>=8.4.1",
-    "pytest-asyncio>=1.1.0",
-    "pytest-watcher>=0.4.3",
-    "ruff>=0.12.5",
+    "pyrefly",
+    "pyright",
+    "pytest",
+    "pytest-asyncio",
+    "pytest-watcher",
+    "ruff",
 ]
 
 [tool.ruff]
 line-length = 125
 target-version = "py313"
 
 [tool.ruff.lint]
-extend-select = ["E","W","F","I","B","C4","N","D","UP","S","ANN","ARG","ERA","ICN","PD","NPY","PT","RET","SIM","TCH"]
-ignore = ["ANN204", "C901","S608","D401","E501","D107","W291","RET503","D100","D101","D102","D103","D104","D105","S104","PD901"]
+select = ["ALL"]
+ignore = [
+    "ANN204",  # Missing return type for __init__
+    "C901",    # Too complex
+    "COM812",  # Missing trailing comma (conflicts with formatter)
+    "D100", "D101", "D102", "D103", "D104", "D105", "D107",  # Missing docstrings
+    "D401",    # First line imperative mood
+    "E501",    # Line too long (handled by formatter)
+    "ISC001",  # Single line implicit string concatenation (conflicts with formatter)
+    "RET503",  # Missing explicit return
+    "S104",    # Possible binding to all interfaces
+    "S608",    # SQL injection (too many false positives)
+    "W291",    # Trailing whitespace
+]
 fixable = ["ALL"]
 pydocstyle = { convention = "google" }
 
 [tool.ruff.lint.per-file-ignores]
 "**/wip*.py"   = ["ALL"]
-"**/tests/*"   = ["D","ANN","S","ERA"]
-"*_"           = ["D","ANN","S"]
-"wip*"         = ["D","ANN","S"]
+"**/tests/*"   = ["D", "ANN", "S", "ERA", "INP001"]
+"*_"           = ["D", "ANN", "S"]
+"wip*"         = ["D", "ANN", "S"]
 "__init__.py"  = ["D"]
+"__main__.py"  = ["T201"]  # Allow print in main entry point
 "**/gen/*"     = ["ALL"]
 "**/*.ipynb"   = ["ALL"]
 
@@ -69,7 +91,6 @@ defineConstant = { DEBUG = true }
 typeCheckingMode = "standard"
 pythonVersion = "3.13"
 pythonPlatform = "All"
-stubPath = "typings"
 reportMissingImports = "error"
 reportMissingTypeStubs = false
 reportMissingModuleSource = false
@@ -83,10 +104,3 @@ reportImportCycles = "warning"
 reportUnusedImport = "warning"
 reportUnusedVariable = "warning"
 reportDuplicateImport = "warning"
-
-[[tool.pyright.overrides]]
-pattern = ".venv/**"
-reportUnknownParameterType = "none"
-reportUnknownArgumentType = "none"
-reportUnknownMemberType = "none"
-reportMissingParameterType = "none"

tests/test_something.py

@@ -1,2 +1,12 @@
-def test_something() -> None:
-    return True
+import pytest
+
+
+def test_sync_example() -> None:
+    """Example synchronous test."""
+    assert True
+
+
+@pytest.mark.asyncio
+async def test_async_example() -> None:
+    """Example asynchronous test."""
+    assert True