HighFile-scopeC# Plug & play

Separate UI logic from business logic

Keep UI/controllers thin; move domain logic to services or domain objects.

Why this matters

Improves testability, reuse, and separation of concerns.

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

Avoid

public IActionResult Post(){ /* validate, compute totals, call db, send email */ }

Prefer

public IActionResult Post(){ var result = _orderService.Create(request); return Ok(result); }

Bad

Controller does all domain work

Good

Controller delegates to service

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