dhh-rails-reviewer

Review Rails code for DHH conventions and JavaScript anti-patterns.

49|4|Updated Feb 12, 2026
One-click install
npx skills add https://github.com/gvkhosla/compound-engineering-pi --skill dhh-rails-reviewer-gvkhosla
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: dhh-rails-reviewer
Source: https://github.com/gvkhosla/compound-engineering-pi/tree/main/skills/dhh-rails-reviewer
Command: npx skills add https://github.com/gvkhosla/compound-engineering-pi --skill dhh-rails-reviewer-gvkhosla

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill provides a critical code review for Rails applications, specifically identifying deviations from DHH's core philosophies, such as convention over configuration, the majestic monolith, and avoiding JavaScript framework patterns within Rails.

Core Features & Use Cases

  • Convention Enforcement: Ruthlessly checks for adherence to Rails conventions (e.g., fat models, skinny controllers, RESTful routes).
  • Anti-Pattern Detection: Identifies and critiques the infiltration of JavaScript world patterns (e.g., unnecessary API layers, JWTs, Redux-style state management) into Rails projects.
  • Complexity Analysis: Tears down unnecessary abstractions like service objects that could be model methods, or over-engineered architectural patterns.
  • Use Case: Reviewing a new Rails feature that uses a separate API layer and JWT authentication instead of traditional Rails sessions and server-side rendering.

Quick Start

Review the provided Rails code for adherence to DHH's conventions and identify any JavaScript framework patterns.

Frequently Asked Questions about dhh-rails-reviewer

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review my Ruby on Rails code for convention over configuration adherence?

Review Ruby on Rails code by analyzing it against DHH's philosophies, enforcing fat models, skinny controllers, and RESTful routes while rejecting unnecessary complexity. It identifies deviations from the majestic monolith approach and suggests idiomatic Rails solutions.

What are common Rails anti-patterns when mixing JavaScript framework patterns?

Common Rails anti-patterns include building unnecessary API layers, using JWTs instead of traditional sessions, and importing Redux-style state management. These JavaScript-centric patterns violate the majestic monolith philosophy and overcomplicate Rails applications.

Can I use this to check if my Rails service objects are over-engineered?

Yes, you can check if Rails service objects are over-engineered. The review identifies unnecessary abstractions and critiques service objects that could be replaced by idiomatic model methods, ensuring simplicity and adherence to Rails conventions.

What is the best way to ensure my Rails feature follows the majestic monolith philosophy?

The best way to ensure a Rails feature follows the majestic monolith philosophy is to evaluate it for unnecessary abstractions and JavaScript-centric patterns. It checks for convention adherence, suggesting idiomatic Rails solutions over separate API layers.

When should I not use service objects in Ruby on Rails applications?

You should not use service objects in Ruby on Rails applications when the logic could be a model method. The review tears down unnecessary abstractions and over-engineered patterns that deviate from fat models and convention over configuration.