claude code — brooks-lint
›
⠋ Reviewing `UserService.update_profile` against R1–R6…
28 / 100 Health Score— 4 responsibilities, a silent bug, SQL injection

#### 🔴 R2 — Change Propagation

Symptom: one method does updates, email, loyalty & cache.
Remedy: extract Notification / Loyalty / CacheInvalidator.

#### 🔴 R6 — Domain Model Distortion

Symptom: email overwritten before the `!=` check — dead branch.
Source: McConnell — Code Complete — Ch. 17.

#### 🔴 R5 — Dependency Disorder + SQL injection ×2

Remedy: parameterise queries; invert the db dependency.
+ 5 more findings · every one cited to a book
Symptom → Source → Consequence → Remedy · grounded in 12 classic engineering books