How does this skill handle over-engineering?

It identifies unnecessary abstractions, generic frameworks for hypothetical needs, and premature optimizations, providing specific suggestions to simplify the implementation.

What is the 'Brick & Stud' architecture reviewed by this skill?

It refers to a modular design philosophy where 'Bricks' are self-contained modules with a single responsibility and 'Studs' are the public contracts (APIs/models) that connect them.

What does 'Zero-BS Implementation' mean?

It means the skill flags TODO comments, NotImplementedErrors, mock data in production, and dead code to ensure that every merged PR is fully functional and production-ready.

Does this skill check for test coverage?

Yes, it assesses whether changed functionality includes adequate tests for public interfaces, edge cases, and error paths, typically looking for 85%+ coverage.

PR Review Assistant

Name: PR Review Assistant
Author: rysweet

byrysweet

•

Herramientas de Colaboración

Conducts philosophy-aware pull request reviews to ensure ruthless simplicity, modular design, and production-ready implementation.

Acerca de

The PR Review Assistant is an advanced code review tool for Claude Code that aligns your codebase with the amplihack development philosophy. It goes beyond basic linting to evaluate architectural integrity, identifying over-engineering and verifying that modules follow a 'Brick & Studs' structure. By enforcing 'Zero-BS' implementation—rejecting TODOs, stubs, and dead code—it helps teams maintain a clean, efficient, and highly maintainable repository with detailed, line-specific feedback and constructive refactoring suggestions.

Características Principales

Identifies over-engineering and premature optimizations
17 GitHub stars
Evaluates test coverage for new and modified functionality
Enforces ruthless simplicity and minimal abstraction layers
Flags 'Zero-BS' violations like TODOs, stubs, and dead code
Verifies modular 'Brick & Studs' architectural patterns

Casos de Uso

Pre-screening code for technical debt and future-proofing bloat
Coaching developers on modular design and code simplicity
Automating architectural alignment during pull request reviews

Acerca de

Características Principales

Identifies over-engineering and premature optimizations
17 GitHub stars
Evaluates test coverage for new and modified functionality
Enforces ruthless simplicity and minimal abstraction layers
Flags 'Zero-BS' violations like TODOs, stubs, and dead code
Verifies modular 'Brick & Studs' architectural patterns

Casos de Uso

Pre-screening code for technical debt and future-proofing bloat
Coaching developers on modular design and code simplicity
Automating architectural alignment during pull request reviews