architecture-review

# Architecture Review (orchestrator) Comprehensive multi-perspective architecture review. This skill is an **orchestrator** — it dispatches one subagent invocation per active team member in parallel, then aggregates the returned reviews into a consolidated document. The deep-perspective work happens in the subagents (`agents/<id>.md`), not in this skill's main thread. See [ADR-013](../../.architecture/decisions/adrs/ADR-013-skill-orchestrator-subagent-delegation.md) for the rationale. ## Process ### 1. Determine scope Identify the review target from the user's request: - **Version** — "version 2.0.0" → filename `2-0-0.md` - **Feature** — "authentication feature" → filename `feature-authentication.md` - **Component** — "payments module" → filename `component-payments-module.md` Apply input sanitization from [`../_patterns.md`](../_patterns.md). If the scope is ambiguous, ask one clarifying question and stop. ### 2. Load team and config Read `.architecture/config.yml` and `.architecture/members.yml`: - `config.yml.pragmatic_mode.enabled` decides whether to include the `pragmatic_enforcer` subagent. - `members.yml.members[]` is the list of subagents to dispatch. Each member's `id` (e.g., `security_specialist`) maps to `agents/<kebab-id>.md` (e.g., `agents/security-specialist.md`). The drift check in CI guarantees these files exist and are in sync. **Active set**: every member in `members.yml`, **excluding** `pragmatic_enforcer` if `config.yml.pragmatic_mode.enabled ==...