From b2747381ec983e3fafc325b29c2614dbb362c80c Mon Sep 17 00:00:00 2001 From: Lukas Date: Sun, 3 May 2026 22:47:24 +0200 Subject: [PATCH] Remove Issue #51 from Tasks.md (type-enforcement tracked elsewhere) --- Docs/Tasks.md | 26 -------------------------- 1 file changed, 26 deletions(-) diff --git a/Docs/Tasks.md b/Docs/Tasks.md index 61fa313..7d2a352 100644 --- a/Docs/Tasks.md +++ b/Docs/Tasks.md @@ -1,29 +1,3 @@ -### Issue #51: MEDIUM - Repository Boundary Not Type-Enforced - -**Where found**: -- `backend/app/dependencies.py:20-25` – documented constraint that routers must not import `DbDep` directly - -**Why this is needed**: -Nothing prevents a developer from bypassing the service/repository layer and accessing the DB directly from a router, violating the layered architecture and making the codebase harder to test. - -**Goal**: -Make architectural violations detectable automatically. - -**What to do**: -1. Add a custom `flake8` or `ruff` rule (or `import-linter` boundary) that forbids `app.routers.*` from importing `app.dependencies.DbDep`. -2. Add a CI check that runs the import boundary linter. - -**Possible traps and issues**: -- `import-linter` requires defining contracts in `pyproject.toml`; coordinate with the existing linting setup. - -**Docs changes needed**: -- `CONTRIBUTING.md`: document the layer boundary rule and linting check. - -**Doc references**: -- `backend/app/dependencies.py` module docstring - ---- - ### Issue #52: MEDIUM - Error Handling Patterns Not Declared on Services **Where found**: