"The best code is the code you don't write. The second best is the code that's obviously correct."
Vanilla Rails is plenty:
- Rich domain models over service objects
- CRUD controllers over custom actions
- Concerns for horizontal code sharing
- Records as state instead of boolean columns
- Database-backed everything (no Redis)
- Build solutions before reaching for gems
What they deliberately avoid:
- devise (custom ~150-line auth instead)
- pundit/cancancan (simple role checks in models)
- sidekiq (Solid Queue uses database)
- redis (database for everything)
- view_component (partials work fine)
- GraphQL (REST with Turbo sufficient)
- Controllers - REST mapping, concerns, Turbo responses
- Models - Concerns, state records, callbacks, scopes
- Views & Frontend - Turbo, Stimulus, CSS, partials
- Architecture - Routing, multi-tenancy, authentication, jobs
- Code Review - Review code against DHH style
- General Guidance - Philosophy and conventions
Specify a number or describe your task.
After reading relevant references, apply patterns to the user's code.
Verbs: card.close, card.gild, board.publish (not set_style methods)
Predicates: card.closed?, card.golden? (derived from presence of related record)
Concerns: Adjectives describing capability (Closeable, Publishable, Watchable)
Controllers: Nouns matching resources (Cards::ClosuresController)
Scopes:
chronologically, reverse_chronologically, alphabetically, latest
preloaded (standard eager loading name)
indexed_by, sorted_by (parameterized)
REST Mapping
Instead of custom actions, create new resources:
POST /cards/:id/close → POST /cards/:id/closure
DELETE /cards/:id/close → DELETE /cards/:id/closure
POST /cards/:id/archive → POST /cards/:id/archival
All detailed patterns in references/:
| File |
Topics |
| controllers.md |
REST mapping, concerns, Turbo responses, API patterns |
| models.md |
Concerns, state records, callbacks, scopes, POROs |
| frontend.md |
Turbo, Stimulus, CSS architecture, view patterns |
| architecture.md |
Routing, auth, jobs, caching, multi-tenancy, config |
| gems.md |
What they use vs avoid, and why |
|
|
1---2name: dhh-rails-style3description: This skill should be used when writing Ruby and Rails code in DHH's distinctive 37signals style. It applies when writing Ruby code, Rails applications, creating models, controllers, or any Ruby file. Triggers on Ruby/Rails code generation, refactoring requests, code review, or when the user mentions DHH, 37signals, Basecamp, HEY, or Campfire style. Embodies REST purity, fat models, thin controllers, Current attributes, Hotwire patterns, and the "clarity over cleverness" philosophy.4---56<objective>7Apply 37signals/DHH Rails conventions to Ruby and Rails code. This skill provides domain expertise extracted from analyzing production 37signals codebases (Fizzy/Campfire).8</objective>910<essential_principles>11## Core Philosophy1213"The best code is the code you don't write. The second best is the code that's obviously correct."1415**Vanilla Rails is plenty:**16- Rich domain models over service objects17- CRUD controllers over custom actions18- Concerns for horizontal code sharing19- Records as state instead of boolean columns20- Database-backed everything (no Redis)21- Build solutions before reaching for gems2223**What they deliberately avoid:**24- devise (custom ~150-line auth instead)25- pundit/cancancan (simple role checks in models)26- sidekiq (Solid Queue uses database)27- redis (database for everything)28- view_component (partials work fine)29- GraphQL (REST with Turbo sufficient)30</essential_principles>3132<intake>33What are you working on?34351. **Controllers** - REST mapping, concerns, Turbo responses362. **Models** - Concerns, state records, callbacks, scopes373. **Views & Frontend** - Turbo, Stimulus, CSS, partials384. **Architecture** - Routing, multi-tenancy, authentication, jobs395. **Code Review** - Review code against DHH style406. **General Guidance** - Philosophy and conventions4142**Specify a number or describe your task.**43</intake>4445<routing>46| Response | Reference to Read |47|----------|-------------------|48| 1, "controller" | [controllers.md](./references/controllers.md) |49| 2, "model" | [models.md](./references/models.md) |50| 3, "view", "frontend", "turbo", "stimulus", "css" | [frontend.md](./references/frontend.md) |51| 4, "architecture", "routing", "auth", "job" | [architecture.md](./references/architecture.md) |52| 5, "review" | Read all references, then review code |53| 6, general task | Read relevant references based on context |5455**After reading relevant references, apply patterns to the user's code.**56</routing>5758<quick_reference>59## Naming Conventions6061**Verbs:** `card.close`, `card.gild`, `board.publish` (not `set_style` methods)6263**Predicates:** `card.closed?`, `card.golden?` (derived from presence of related record)6465**Concerns:** Adjectives describing capability (`Closeable`, `Publishable`, `Watchable`)6667**Controllers:** Nouns matching resources (`Cards::ClosuresController`)6869**Scopes:**70- `chronologically`, `reverse_chronologically`, `alphabetically`, `latest`71- `preloaded` (standard eager loading name)72- `indexed_by`, `sorted_by` (parameterized)7374## REST Mapping7576Instead of custom actions, create new resources:7778```79POST /cards/:id/close → POST /cards/:id/closure80DELETE /cards/:id/close → DELETE /cards/:id/closure81POST /cards/:id/archive → POST /cards/:id/archival82```83</quick_reference>8485<reference_index>86## Domain Knowledge8788All detailed patterns in `references/`:8990| File | Topics |91|------|--------|92| [controllers.md](./references/controllers.md) | REST mapping, concerns, Turbo responses, API patterns |93| [models.md](./references/models.md) | Concerns, state records, callbacks, scopes, POROs |94| [frontend.md](./references/frontend.md) | Turbo, Stimulus, CSS architecture, view patterns |95| [architecture.md](./references/architecture.md) | Routing, auth, jobs, caching, multi-tenancy, config |96| [gems.md](./references/gems.md) | What they use vs avoid, and why |97</reference_index>9899<success_criteria>100Code follows DHH style when:101- Controllers map to CRUD verbs on resources102- Models use concerns for horizontal behavior103- State is tracked via records, not booleans104- No unnecessary service objects or abstractions105- Database-backed solutions preferred over external services106- Tests use Minitest with fixtures107- Turbo/Stimulus for interactivity (no heavy JS frameworks)108</success_criteria>109110<credits>111Based on [The Unofficial 37signals/DHH Rails Style Guide](https://gist.github.com/marckohlbrugge/d363fb90c89f71bd0c816d24d7642aca) by [Marc Köhlbrugge](https://x.com/marckohlbrugge), generated through deep analysis of the Fizzy codebase.112</credits>