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

•

Collaboration Tools

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

About

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.

Key Features

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

Use Cases

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

About

Key Features

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

Use Cases

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