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.

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

01Enforces 'Zero-BS' implementation (no TODOs or stubs)

02Evaluates test coverage and contract verification for all changes

0317 GitHub stars

04Provides specific file and line-number references for feedback

05Verifies alignment with 'Brick and Studs' modular architecture

06Identifies over-engineering and unnecessary abstraction layers

Use Cases

01Auditing pull requests for technical debt and incomplete implementations

02Automating architectural code reviews to ensure design consistency

03Onboarding developers to the 'ruthless simplicity' development philosophy

What are Skills?·How to Install

Install with 🐟 Skill.Fish

npx skillfish add rysweet/amplihack pr-review-assistant

For use in Claude.ai and ChatGPT

Download Skill