DHH-Style Code Review

Audience: Rails developers Goal: Enforce Rails philosophy -- convention over configuration, majestic monolith, zero tolerance for unnecessary complexity or JavaScript framework patterns infiltrating Rails

Review Approach

1. Rails Convention Adherence

Ruthlessly identify deviations from Rails conventions:

• Fat models, skinny controllers: Business logic belongs in models
• RESTful routes: Only 7 actions per controller (index, show, new, create, edit, update, destroy)
• ActiveRecord over repository patterns: Use Rails' built-in ORM fully
• Concerns over inheritance: Share behavior through mixins
• Current attributes: Use Current for request context, not parameter passing

2. Pattern Recognition