clean-architecture-review

Identify and report clean-architecture boundary violations in code changes.

1|1|Updated Nov 14, 2025
One-click install
npx skills add https://github.com/erikalira/python-tts --skill clean-architecture-review-erikalira
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: clean-architecture-review
Source: https://github.com/erikalira/python-tts/tree/main/.codex/skills/clean-architecture-review
Command: npx skills add https://github.com/erikalira/python-tts --skill clean-architecture-review-erikalira

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Review changes for clean-architecture boundary violations, layer leakage, and misplaced business logic.

Core Features & Use Cases

  • Identify changed files and map each file to a layer
  • Inspect imports and dependencies between those files
  • Mark boundary violations, layer leakage, or duplicated logic
  • Note weak contracts, lingering mapping-style payloads, or compatibility layers
  • Separate critical issues from minor improvements
  • Preserve execution contexts for both bot and desktop app flows

Quick Start

Run the review on a focused PR to surface clean-architecture boundary violations.

Frequently Asked Questions about clean-architecture-review

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

FAQPage Schema
How do I detect clean architecture boundary violations in a pull request?

Detect clean architecture boundary violations by mapping changed PR files to layers and inspecting imports to flag coupling, layer leakage, or misplaced business logic. The review outputs documented audit results for your architecture constraints.

What is layer leakage in code review and how does it relate to clean architecture?

Layer leakage in clean architecture occurs when inner layers depend on outer layers or carry mapping-style payloads. A code review identifies these boundary violations by inspecting file imports and dependencies to prevent duplicated logic and weak contracts.

Can I audit dependency analysis for refactor plans across both bot and desktop app paths?

Yes, you can audit dependency analysis for refactor plans across both bot and desktop app paths. The review maps files to layers while preserving execution contexts for each flow to identify coupling and boundary violations accurately.

How do I separate critical boundary violations from minor improvements during a refactor review?

Separate critical boundary violations from minor improvements during a refactor review by categorizing issues found in dependency analysis. The audit distinguishes severe layer leakage and coupling from weak contracts or lingering compatibility layers.

What is the best way to map files to layers for clean architecture compliance?

The best way to map files to layers for clean architecture compliance is through a structured review process with clear ownership mapping. This approach inspects imports and dependencies to document audit outputs satisfying architecture constraints.

Why does my code review still show boundary violations after a clean architecture refactor?

Boundary violations after a refactor often stem from lingering mapping-style payloads or compatibility layers creating hidden coupling. Reviewing file imports and dependencies across bot and desktop paths reveals these remaining layer leakage issues.