Agent Skills: Architecture Review

Use when auditing project structure, planning refactors, improving code organization, analyzing dependencies and module boundaries, or identifying structural issues. TypeScript/JavaScript-primary with language-agnostic patterns.

UncategorizedID: srstomp/pokayokay/architecture-review

Install this agent skill to your local

pnpm dlx add-skill https://github.com/srstomp/pokayokay/tree/HEAD/plugins/pokayokay/skills/architecture-review

Skill Files

Browse the full folder contents for architecture-review.

Download Skill

Loading file tree…

plugins/pokayokay/skills/architecture-review/SKILL.md

Skill Metadata

Name
architecture-review
Description
Use when auditing project structure, planning refactors, improving code organization, analyzing dependencies and module boundaries, or identifying structural issues. TypeScript/JavaScript-primary with language-agnostic patterns.

Architecture Review

Analyze, audit, and improve project structure.

Key Principles

  • Measure before changing — Map structure and identify concrete issues before proposing changes
  • Clear boundaries — Layers (UI, logic, data) separated with consistent dependency direction
  • Colocation — Related code together; easy to find, change, and delete features
  • Incremental migration — Refactor in phases, validate each step with tests

Quick Assessment

  • No circular dependencies
  • Consistent directory naming and grouping (by feature or by type)
  • Single responsibility per file/module, reasonable file sizes (<500 lines)
  • External dependencies isolated, shared code properly extracted
  • New developers can navigate and find code for any feature easily

Quick Start Checklist

  1. Map current structure: directory tree, dependency graph, module boundaries
  2. Identify issues: circular deps, god modules, leaky abstractions, deep nesting
  3. Classify severity: critical (blocks dev), high (maintenance burden), medium/low (friction)
  4. Propose target structure with migration plan
  5. Execute incrementally, validating with tests after each move

References

| Reference | Description | |-----------|-------------| | analysis-techniques.md | Dependency graphs, complexity metrics, code analysis | | refactoring-patterns.md | Safe refactoring techniques, migration strategies | | structural-patterns.md | Directory structures for different project types | | dependency-management.md | Circular deps, coupling, module boundaries | | cleanup-strategies.md | Dead code removal, consolidation, naming conventions | | anti-rationalization.md | Iron Law, common rationalizations, red flag STOP list for architecture discipline |