river-review-architecture

Architecture Review（設計・アーキテクチャレビュー）

設計判断の妥当性、アーキテクチャ境界の整合性、データモデルの一貫性を検証する。

When to Use / いつ使うか

• アーキテクチャに影響する変更を含むPRのレビュー時
• ADR（Architecture Decision Record）のレビュー時
• モジュール構成や依存関係の変更時
• API設計の新規作成・変更時

Routing / ルーティング

入力に応じて、適切な個別スキルへルーティングする。

キーワード	スキルID	説明
ADR, 意思決定	adr-decision-quality	ADR の品質検証
API, エンドポイント	api-design	API 設計レビュー
API互換, バージョニング	api-versioning-compat	API バージョン互換性
境界, モジュール	architecture-boundaries	アーキテクチャ境界
図, ダイアグラム	architecture-diagrams	設計図の整合性
リスク	architecture-risk-register	リスク登録の検証
トレーサビリティ	architecture-traceability	要件追跡性
検証計画	architecture-validation-plan	検証計画レビュー
可用性, 冗長	availability-architecture	可用性設計
ドメイン, コンテキスト	bounded-context-language	境界づけられたコンテキスト
キャッシュ	cache-strategy-consistency	キャッシュ戦略
コスト, キャパシティ	capacity-cost-design	キャパシティ設計
データフロー, 状態	data-flow-state-ownership	データフロー設計
データモデル, DB	data-model-db-design	データモデル設計
DR, マルチリージョン	dr-multiregion	災害復旧設計
イベント駆動	event-driven-semantics	イベント駆動設計
外部依存	external-dependencies	外部依存関係
障害, 可観測性	failure-modes-observability	障害モード分析
結合, コントラクト	integration-contracts	結合コントラクト
マイグレーション	migration-rollout-rollback	マイグレーション計画
マルチテナント	multitenancy-isolation	テナント分離
OpenAPI	openapi-contract	OpenAPI 仕様検証
SLO, 運用性	operability-slo	運用性・SLO
要件, 受入	requirements-acceptance	要件・受入条件

デフォルト動作

• キーワード指定なし → 変更内容から自動判定
• 複数カテゴリに該当 → もっとも関連性の高いスキルを優先

Execution Flow / 実行フロー

1. 変更種別の判定
   ├─ ADR/設計ドキュメント → adr-decision-quality を優先
   ├─ API定義/エンドポイント → api-design, api-versioning-compat を優先
   ├─ モジュール構成変更 → architecture-boundaries を優先
   ├─ データモデル変更 → data-model-db-design を優先
   └─ キーワード指定あり → 該当スキルを直接選択

2. スキルの実行（該当する専門スキルを並列実行可能）
   ├─ 設計品質系: adr-decision-quality, architecture-diagrams, architecture-traceability
   ├─ 構造系: architecture-boundaries, bounded-context-language, data-flow-state-ownership
   ├─ API系: api-design, api-versioning-compat, openapi-contract, integration-contracts
   ├─ 運用系: availability-architecture, capacity-cost-design, dr-multiregion, operability-slo
   └─ リスク系: architecture-risk-register, failure-modes-observability, external-dependencies

3. 統合
   ├─ 重複する指摘の除去
   └─ 複数カテゴリ該当時は関連性の高いスキルを優先

Output Format / 出力形式

<file>:<line>: <message>

• Finding: 何が問題か（1文）
• Impact: 何が困るか（短く）
• Fix: 次の一手（最小の修正案）

他スキルとの関係

スキル	関係	棲み分け
adversarial-review	補完	architecture は「設計の正しさ」、adversarial は「設計の盲点」
river-review-security	補完	architecture は「構造」、security は「脆弱性」
river-review-code	補完	architecture は「マクロ設計」、code は「ミクロ品質」

References

• ROUTING.md: 詳細なルーティングルール