architectural-review

Validate PRs for module boundaries, dependency direction, and API exports.

Updated Apr 16, 2026
One-click install
npx skills add https://github.com/Leep-GH/Lee-s-squad --skill architectural-review-leep-gh
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: architectural-review
Source: https://github.com/Leep-GH/Lee-s-squad/tree/main/.copilot/skills/architectural-review
Command: npx skills add https://github.com/Leep-GH/Lee-s-squad --skill architectural-review-leep-gh

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Validates PRs for architectural integrity by verifying module boundaries, dependency direction, and API export surfaces to prevent architectural erosion.

Core Features & Use Cases

  • Enforces module boundary rules and the correct dependency direction (CLI → SDK; SDK must NEVER depend on CLI)
  • Validates API surface changes in barrel exports and detects cross-package imports
  • Flags sweeping architectural changes or refactors for structured review and guardrails

Quick Start

Review a PR and apply the architectural-review checklist to verify architecture invariants.

Frequently Asked Questions about architectural-review

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

FAQPage Schema
How do I enforce module boundaries and dependency direction in a pull request?

To enforce module boundaries and dependency direction in a pull request, validate architectural integrity by checking that dependencies follow repository rules like CLI to SDK, ensuring the SDK never depends on the CLI.

What is architectural erosion and how do I prevent it during refactoring?

Architectural erosion is the gradual degradation of a system's structure during refactoring. You prevent it by applying an architectural review checklist to verify invariants like module boundaries and API export surfaces before merging changes.

How do I validate API surface changes and detect unsafe cross-package imports?

You validate API surface changes and detect unsafe cross-package imports by reviewing barrel exports and checking import statements against repository dependency rules during the pull request review process.

When should I perform an architectural review on my codebase?

You should perform an architectural review when a pull request introduces new modules, modifies dependencies, changes API export surfaces, or executes sweeping refactors that affect core architecture invariants.

Can I use automated checks to guard bootstrap file constraints and module dependencies?

Yes, you can apply automated architectural review checks to guard bootstrap file constraints, validate module dependencies, and flag sweeping architectural changes for structured review and guardrails.