Address feedback

Author: stevepolitodesign

rails/ai-rules/rules/controllers.md

@@ -1,6 +1,7 @@
 # Controllers
 
 - Controllers handle HTTP only: receive request, delegate to model, return response.
+- Avoid long actions, since they often signal business logic that belongs in a model or PORO.
 - Maximum one instance variable per action.
 - No business logic, calculations, email sending, or multi-object operations in controllers.
 - Return `status: :unprocessable_entity` on failed form renders (required by Turbo).

rails/ai-rules/rules/models.md

@@ -4,6 +4,7 @@
 - Name classes after domain nouns, not actions. No `*Service`, `*Manager`, `*Handler` suffixes.
 - Use `ActiveModel::Model` for POROs that need validation or form integration.
 - Replace `.call` / `.perform` with domain verbs: `#save`, `#complete`, `#submit`, `#deliver`.
+- Look to identify domain models that can be extracted when an existing model is large.
 - Callbacks only for data integrity (normalise fields, set defaults). Never for emails, payments, or external systems.
 - Prefer composition over inheritance. Extract behaviour into small, focused objects.
-- Avoid feature envy, case statements on type, and mixin abuse.
+- Avoid feature envy, long parameter lists, case statements on type, and mixin abuse.