review

Review code changes against architectural principles and detect semantic violations.

Updated Feb 4, 2026
One-click install
npx skills add https://github.com/edpaget/bot-showalter --skill review-edpaget
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: review
Source: https://github.com/edpaget/bot-showalter/tree/main/.claude/skills/review
Command: npx skills add https://github.com/edpaget/bot-showalter --skill review-edpaget

SYSTEM DOCUMENTATION & REQUIREMENTS

💡 This Skill includes scripts (resource) and references (resource) components.

What problem does it solve?

This Skill ensures that code changes align with architectural principles, catching semantic violations that static analysis might miss.

Core Features & Use Cases

  • Code Review against Principles: Checks code changes against defined architectural principles.
  • Semantic Analysis: Identifies design-intent issues such as complex protocols or business logic in incorrect layers.
  • Detailed Reporting: Provides specific file paths, line numbers, and descriptions of violations, with suggested fixes.

Quick Start

Run the review skill on your last commit with the command: 'review skill --diff-range HEAD~1'.

Frequently Asked Questions about review

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

FAQPage Schema
How do I check code changes against architectural principles?

To check code changes against architectural principles, you can run an architectural review using a principles document. This detects semantic violations, such as business logic in the wrong layer, that static analysis tools often miss.

What is semantic code analysis and how does it detect design-intent issues?

Semantic code analysis evaluates code changes against defined architectural principles to identify design-intent issues. It detects complex protocols or misplaced business logic within layered architectures that standard static analysis cannot catch.

How do I review my last commit for architectural violations?

To review your last commit for architectural violations, run the review skill with the diff range set to HEAD~1. This checks the recent code changes against your specified principles document and reports violations with suggested fixes.

Can I use architectural review for projects without a layered architecture?

Architectural review is specifically suitable for projects with layered architectures. Without a layered structure and a defined principles document to provide context, the semantic analysis cannot accurately detect design-intent violations.

Architectural review vs static analysis: which catches design-intent violations?

Architectural review catches design-intent violations that static analysis misses by checking code changes against a principles document. While static analysis checks syntax and formatting, semantic analysis identifies misplaced business logic or complex protocols.

Do I need a principles document to run a semantic code review?

Yes, you need a principles document to run a semantic code review. The document provides the necessary context for the semantic analysis to evaluate code changes and detect architectural violations within your layered architecture.