LowFile-scopeKotlin Plug & play

Extract contract processing logic

Move domain-specific processing (validation, normalization, mapping) into cohesive services instead of scattering it across controllers/handlers.

Why this matters

Improves testability and reuse; controllers become thin.

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

Avoid

controller.save() { validate(); map(); calculate(); repo.insert() }

Prefer

service.processAndSave(contract)
// service encapsulates validate/map/calc

Bad

controller does all business logic

Good

service layer encapsulates rules

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