Can I use this for non-amplihack projects?

While optimized for the amplihack framework, any project that values modularity and simplicity can benefit from its rigorous review standards and over-engineering detection.

Does this skill help with test coverage?

Yes, it verifies that the public interface is tested, checks for edge cases, and ensures that changed code meets an 85% coverage threshold.

What are 'Brick and Studs' in this context?

This refers to a modular architecture where a 'Brick' is a self-contained module with one responsibility and 'Studs' are the public contracts (APIs) that others connect to.

How does the skill handle 'Zero-BS' implementation?

The skill identifies and flags technical debt such as TODO comments, NotImplementedErrors, mock data in production code, and unused variables, requiring them to be resolved before merging.

What is a philosophy-aware PR review?

A philosophy-aware review goes beyond syntax to check if code aligns with specific design principles like ruthless simplicity, modularity, and the avoidance of over-engineering.

PR Review Assistant

Name: PR Review Assistant
Author: rysweet

byrysweet

•

Collaboration Tools

Conducts philosophy-aware pull request reviews focusing on ruthless simplicity, modular design, and production-ready implementations.

About

The PR Review Assistant is a specialized skill for Claude Code designed to enforce the rigorous development standards of the amplihack framework. It moves beyond standard linting to perform deep architectural analysis, identifying over-engineering and ensuring code adheres to the 'Brick and Studs' modular pattern. By enforcing a 'Zero-BS' policy that flags TODOs, stubs, and mock data, the skill helps maintain a clean, maintainable codebase. It provides constructive, line-specific feedback and verifies that new code meets high testing standards, making it an essential tool for teams prioritizing long-term code quality over quick, messy fixes.

Key Features

Enforces 'Zero-BS' implementation (no TODOs or stubs)
Evaluates test coverage and contract verification for all changes
17 GitHub stars
Provides specific file and line-number references for feedback
Verifies alignment with 'Brick and Studs' modular architecture
Identifies over-engineering and unnecessary abstraction layers

Use Cases

Auditing pull requests for technical debt and incomplete implementations
Automating architectural code reviews to ensure design consistency
Onboarding developers to the 'ruthless simplicity' development philosophy

About

Key Features

Enforces 'Zero-BS' implementation (no TODOs or stubs)
Evaluates test coverage and contract verification for all changes
17 GitHub stars
Provides specific file and line-number references for feedback
Verifies alignment with 'Brick and Studs' modular architecture
Identifies over-engineering and unnecessary abstraction layers

Use Cases

Auditing pull requests for technical debt and incomplete implementations
Automating architectural code reviews to ensure design consistency
Onboarding developers to the 'ruthless simplicity' development philosophy