LowFile-scopeJava Plug & play

Extract business logic into separate methods

Move complex domain calculations out of controllers/handlers into services or private helpers.

Why this matters

Improves cohesion, reuse, and test coverage.

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

Avoid

public Response create(){ /* validate, compute, persist, notify */ }

Prefer

public Response create(){ var order = service.create(cmd); return ok(order); }

Bad

controller does all 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