review-architecture

Solid

Architecture-focused monorepo review. USE WHEN: user runs /review-architecture or explicitly asks for this review. DO NOT USE WHEN: implementing features or fixing bugs unless the user asked for a review.

Code & Development 19 stars 4 forks Updated today Apache-2.0

Install

View on GitHub

Quality Score: 83/100

Stars 20%
43
Recency 20%
100
Frontmatter 20%
70
Documentation 15%
100
Issue Health 10%
50
License 10%
100
Description 5%
100

Skill Content

# Review architecture Run an **architecture-focused** review of the monorepo: layout, workspace dependencies, Turborepo task graph, app vs package boundaries, dependency direction, and module coupling/cohesion. Your reply must be a **plan of suggested changes**: concise, actionable, and structured-not only prose. ## Invocation Text after the slash command is additional scope/focus - narrow the review accordingly. If none given, use the default scope described below. ## Best practices alignment - **Dependency direction** - Apps depend on packages; packages must not depend on apps. No circular dependencies between workspaces. - **Single responsibility** - Each package has one clear purpose (e.g. `@repo/dtos-common`, `@repo/enums-common`, `@repo/typescript-config`). - **Public API surface** - Packages expose a minimal, stable API via barrel exports; internal modules are not re-exported unless intentional. - **Turborepo** - Task graph is acyclic; cache boundaries and `dependsOn` are correct; no redundant or missing tasks. - **Makefile** - Single entry point for common operations; app-specific Makefiles delegate to root or extend cleanly; port allocation is documented and consistent. Align with root [AGENTS.md](../../../AGENTS.md) and app-level AGENTS.md for stated architecture and naming. ## Deep technical review Conduct an architecture-only review. Inspect the following and call out violations or improvements. ### Monorepo layout and boundaries - **Artifacts:** [pnpm-...

Details

Author
louisbrulenaudet
Repository
louisbrulenaudet/monorepo-template
Created
5 months ago
Last Updated
today
Language
JavaScript
License
Apache-2.0

Integrates with

Similar Skills

Semantically similar based on skill content — not just same category

AI & Automation Listed

review-architecture

Use when: reviewing code for architectural violations — layer violations, dependency direction mistakes, module boundary coupling, SOLID principle breaches, NestJS module/provider structure, React MVVM boundary violations, or MobX store layer misplacement. Triggered by: "review architecture", "check architecture", "architectural review", or dispatched by review-orchestrator with --architecture or --backend. NOT for: style/naming preferences, logic correctness bugs, or security vulnerabilities.

1 Updated yesterday
MrCipherSmith
AI & Automation Listed

architecture-review

Review system-level architecture of a PR, branch, namespace, or whole project — boundary violations, dependency cycles, layer breaches, ADR compliance, cohesion of modules and bounded contexts. Make sure to use this skill whenever the user asks to review the architecture, audit module boundaries, check for circular dependencies, verify domain structure, or says things like "architektur-review", "prüfe die struktur", "check architecture", "review namespace", "audit boundaries", "are our modules clean". Use this skill for HIGH-LEVEL structural review only — SOLID, class design, and method-level issues belong in `component-review`.

0 Updated 1 months ago
lx-wnk
Code & Development Listed

review-architecture

Review system architecture — layering, module boundaries, coupling/cohesion, pattern fit, quality attributes (scalability, resilience, evolvability), and architectural smells. Triggers "review architecture", "architecture review", "system design review", "check architecture".

1 Updated today
nielsmadan