dhh-rails-reviewer

Identify non-idiomatic Rails patterns in code review diffs.

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

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Rails applications often accumulate non-idiomatic code patterns that fight the framework's built-in conventions, leading to unnecessary complexity, higher maintenance costs, and slower development velocity. This Skill provides an opinionated code review perspective aligned with DHH's Rails philosophy to catch these anti-patterns before they are merged into the codebase.

Core Features & Use Cases

  • Opinionated Rails Code Review: Evaluates code diffs from DHH's perspective, focusing exclusively on patterns that deviate from Rails' "omakase" (chef's choice) design principles.
  • Calibrated Confidence Scoring: Assigns clear confidence levels to flagged issues, distinguishing between explicit anti-patterns and debatable architectural choices to reduce review noise.
  • Targeted Anti-Pattern Detection: Flags common high-impact issues including unnecessary JWT auth replacing standard sessions, over-abstraction that hides core Rails functionality, and unnecessary monolith splitting for single-app concerns.
  • Use Case: When reviewing a pull request for a Rails app that introduces a service layer merely wrapping Active Record behavior, or replaces Turbo Hotwire with a custom client-side state machine, this Skill flags these as high-confidence anti-patterns.

Quick Start

Use the dhh-rails-reviewer skill to review the attached Rails pull request diff for patterns that conflict with Rails conventions.

Frequently Asked Questions about dhh-rails-reviewer

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

FAQPage Schema
How do I catch Rails anti-patterns in a pull request?

To catch Rails anti-patterns in a pull request, review the diff to identify non-idiomatic code that fights framework conventions. This process flags unnecessary complexity like over-abstraction or replacing standard sessions with custom JWT auth, ensuring code aligns with Rails' opinionated omakase design.

What are common Ruby on Rails anti-patterns to avoid?

Common Ruby on Rails anti-patterns include replacing Turbo Hotwire with custom client-side state machines, introducing service layers that merely wrap Active Record behavior, and unnecessary monolith splitting for single-app concerns. These patterns introduce unnecessary complexity and deviate from Rails conventions.

How do I review Rails code for unnecessary abstractions?

Reviewing Rails code for unnecessary abstractions involves checking if new architectural choices hide core framework functionality. Evaluate pull request diffs to identify over-abstraction that conflicts with Rails' design, assigning confidence levels to distinguish explicit anti-patterns from debatable architectural choices.

Does this code review approach flag low-confidence philosophical complaints?

This code review approach suppresses low-confidence philosophical complaints by applying calibrated confidence scoring. It evaluates architectural choices and frontend patterns, delivering DHH-aligned review findings that exclusively flag high-confidence anti-patterns deviating from Rails' opinionated design.

When should I not use custom JWT authentication in a Rails monolith?

You should not use custom JWT authentication in a Rails monolith when it replaces standard sessions unnecessarily. Replacing built-in session management with custom JWT auth is flagged as a high-confidence anti-pattern that introduces unnecessary complexity and fights the framework's omakase design.