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](./references/ROUTING.md): 詳細なルーティングルール