Remove completed task from documentation
Task #2 about hidden cross-service coupling has been resolved. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This commit is contained in:
@@ -1,26 +1,3 @@
|
||||
## 2) Hidden cross-service coupling (service imports service)
|
||||
- Where found:
|
||||
- [backend/app/services/jail_service.py](backend/app/services/jail_service.py)
|
||||
- [backend/app/services/jail_config_service.py](backend/app/services/jail_config_service.py)
|
||||
- [backend/app/services/history_service.py](backend/app/services/history_service.py)
|
||||
- Why this is needed:
|
||||
- Direct service imports create hidden dependency graphs and make testing harder.
|
||||
- Goal:
|
||||
- Make service dependencies explicit and injectable.
|
||||
- What to do:
|
||||
- Define clear service interfaces.
|
||||
- Pass dependencies via constructor/function parameters.
|
||||
- Remove direct cross-service module imports.
|
||||
- Possible traps and issues:
|
||||
- Circular references may surface during refactor.
|
||||
- Docs changes needed:
|
||||
- Document service dependency rules and injection style.
|
||||
- Doc references:
|
||||
- [Docs/Architekture.md](Docs/Architekture.md)
|
||||
- [Docs/Backend-Development.md](Docs/Backend-Development.md)
|
||||
|
||||
---
|
||||
|
||||
## 3) Blocklist import flow mixes too many responsibilities
|
||||
- Where found:
|
||||
- [backend/app/services/blocklist_service.py](backend/app/services/blocklist_service.py)
|
||||
|
||||
Reference in New Issue
Block a user