rails-code-review

Identify code-review gaps in Rails pull requests using a structured framework.

22|6|Updated Mar 11, 2026
One-click install
npx skills add https://github.com/igmarin/rails-agent-skills --skill rails-code-review-igmarin
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: rails-code-review
Source: https://github.com/igmarin/rails-agent-skills/tree/main/rails-code-review
Command: npx skills add https://github.com/igmarin/rails-agent-skills --skill rails-code-review-igmarin

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Reviews of Rails code changes can be inconsistent and time-consuming, leading to missed conventions, performance issues, and security gaps.

Core Features & Use Cases

  • Structured review framework for controllers, models, migrations, queries, and security checks.
  • Enforces Rails conventions and Rails Way compliance during PR reviews.
  • Scalable guidance for teams handling multiple PRs, audits, and migrations.

Quick Start

Invoke rails-code-review to assess a Rails PR against the defined review order and return a prioritized list of issues.

Frequently Asked Questions about rails-code-review

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

FAQPage Schema
How do I review Rails pull requests for security and performance issues?

Rails code reviews should follow a structured framework evaluating controllers, models, migrations, queries, security, caching, and performance. This enforces Rails Way compliance and a hard-gate self-review before merge to ensure quality.

What is the best way to enforce Rails conventions during a PR review?

The best way to enforce Rails conventions during a PR review is to apply a structured rubric covering controllers, models, migrations, queries, and caching. This ensures scalable, consistent compliance with the Rails Way across team workflows.

Does this Rails code review framework cover database migrations and queries?

Yes, this Rails code review framework explicitly covers database migrations and queries. It assesses these components alongside controllers, models, security, and caching to identify performance issues and convention gaps before merging.

How do I structure a hard-gate self-review before merging Rails code?

To structure a hard-gate self-review before merging Rails code, apply a defined rubric to assess controllers, models, migrations, queries, security, and performance. Return a prioritized list of issues to resolve before merging.

Can I use this structured rubric for Rails audits across multiple PRs?

Yes, you can use this structured rubric for Rails audits across multiple PRs. It provides scalable guidance for teams, ensuring consistent evaluation of security, performance, and conventions across high-volume pull requests.