LowFile-scopePython Plug & play

Use dependency injection for resource management

Pass dependencies (clients, sessions, configs) into functions/classes instead of creating them internally.

Why this matters

Improves testability, reuse, and separation of concerns.

Side-by-side examples engineers can pattern-match during review.

Avoid

def fetch_users():
    client = HttpClient()
    return client.get('/users')

Prefer

def fetch_users(client: 'HttpClient'):
    return client.get('/users')

Bad

def repo(): return DB()

Good

def repo(db: DB): return db.query(...)

From the same buckets as this rule.

Critical
Require an ADR for any architectural or cross-team decision
For changes that affect architecture, data models, external APIs, security posture, deployment topology, or cost (>10%), create an ADR in docs/adr/ using the standard template (Context, Decision, Consequences) and link the PR and issue IDs.
docs-adrsmodule-architecture+1
Critical
Use Factory Method Injection in `@Configuration` Classes
Ensure that factory method injection is used in `@Configuration` classes instead of `@Autowired` field injection.
module-architecturestyle-conventions
High
A module should not import itself
Identify cases where a module imports itself. Self-imports are usually unintentional and can cause confusion or unnecessary dependencies. Suggest removing them.
module-architecture