Architecture Review（設計・アーキテクチャレビュー）
設計判断の妥当性、アーキテクチャ境界の整合性、データモデルの一貫性を検証する。
When to Use / いつ使うか
- アーキテクチャに影響する変更を含むPRのレビュー時
- ADR（Architecture Decision Record）のレビュー時
- モジュール構成や依存関係の変更時
- API設計の新規作成・変更時
Routing / ルーティング
入力に応じて、適切な個別スキルへルーティングする。
| キーワード |
スキルID |
説明 |
| ADR, 意思決定 |
rr-upstream-adr-decision-quality-001 |
ADR の品質検証 |
| API, エンドポイント |
rr-upstream-api-design-001 |
API 設計レビュー |
| API互換, バージョニング |
rr-upstream-api-versioning-compat-001 |
API バージョン互換性 |
| 境界, モジュール |
rr-upstream-architecture-boundaries-001 |
アーキテクチャ境界 |
| 図, ダイアグラム |
rr-upstream-architecture-diagrams-001 |
設計図の整合性 |
| リスク |
rr-upstream-architecture-risk-register-001 |
リスク登録の検証 |
| トレーサビリティ |
rr-upstream-architecture-traceability-001 |
要件追跡性 |
| 検証計画 |
rr-upstream-architecture-validation-plan-001 |
検証計画レビュー |
| 可用性, 冗長 |
rr-upstream-availability-architecture-001 |
可用性設計 |
| ドメイン, コンテキスト |
rr-upstream-bounded-context-language-001 |
境界づけられたコンテキスト |
| キャッシュ |
rr-upstream-cache-strategy-consistency-001 |
キャッシュ戦略 |
| コスト, キャパシティ |
rr-upstream-capacity-cost-design-001 |
キャパシティ設計 |
| データフロー, 状態 |
rr-upstream-data-flow-state-ownership-001 |
データフロー設計 |
| データモデル, DB |
rr-upstream-data-model-db-design-001 |
データモデル設計 |
| DR, マルチリージョン |
rr-upstream-dr-multiregion-001 |
災害復旧設計 |
| イベント駆動 |
rr-upstream-event-driven-semantics-001 |
イベント駆動設計 |
| 外部依存 |
rr-upstream-external-dependencies-001 |
外部依存関係 |
| 障害, 可観測性 |
rr-upstream-failure-modes-observability-001 |
障害モード分析 |
| 結合, コントラクト |
rr-upstream-integration-contracts-001 |
結合コントラクト |
| マイグレーション |
rr-upstream-migration-rollout-rollback-001 |
マイグレーション計画 |
| マルチテナント |
rr-upstream-multitenancy-isolation-001 |
テナント分離 |
| OpenAPI |
rr-upstream-openapi-contract-001 |
OpenAPI 仕様検証 |
| SLO, 運用性 |
rr-upstream-operability-slo-001 |
運用性・SLO |
| 要件, 受入 |
rr-upstream-requirements-acceptance-001 |
要件・受入条件 |
デフォルト動作
- キーワード指定なし → 変更内容から自動判定
- 複数カテゴリに該当 → もっとも関連性の高いスキルを優先
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
1---2name: river-review-architecture3description: 設計・アーキテクチャ観点のレビューエージェント。 依存関係、境界設計、データモデル、API設計等の個別スキルへルーティングする。4license: MIT5---67# Architecture Review（設計・アーキテクチャレビュー）89設計判断の妥当性、アーキテクチャ境界の整合性、データモデルの一貫性を検証する。1011## When to Use / いつ使うか1213- アーキテクチャに影響する変更を含むPRのレビュー時14- ADR（Architecture Decision Record）のレビュー時15- モジュール構成や依存関係の変更時16- API設計の新規作成・変更時1718## Routing / ルーティング1920入力に応じて、適切な個別スキルへルーティングする。2122| キーワード | スキルID | 説明 |23| ----------------------- | ---------------------------------------------- | -------------------------- |24| ADR, 意思決定 | `rr-upstream-adr-decision-quality-001` | ADR の品質検証 |25| API, エンドポイント | `rr-upstream-api-design-001` | API 設計レビュー |26| API互換, バージョニング | `rr-upstream-api-versioning-compat-001` | API バージョン互換性 |27| 境界, モジュール | `rr-upstream-architecture-boundaries-001` | アーキテクチャ境界 |28| 図, ダイアグラム | `rr-upstream-architecture-diagrams-001` | 設計図の整合性 |29| リスク | `rr-upstream-architecture-risk-register-001` | リスク登録の検証 |30| トレーサビリティ | `rr-upstream-architecture-traceability-001` | 要件追跡性 |31| 検証計画 | `rr-upstream-architecture-validation-plan-001` | 検証計画レビュー |32| 可用性, 冗長 | `rr-upstream-availability-architecture-001` | 可用性設計 |33| ドメイン, コンテキスト | `rr-upstream-bounded-context-language-001` | 境界づけられたコンテキスト |34| キャッシュ | `rr-upstream-cache-strategy-consistency-001` | キャッシュ戦略 |35| コスト, キャパシティ | `rr-upstream-capacity-cost-design-001` | キャパシティ設計 |36| データフロー, 状態 | `rr-upstream-data-flow-state-ownership-001` | データフロー設計 |37| データモデル, DB | `rr-upstream-data-model-db-design-001` | データモデル設計 |38| DR, マルチリージョン | `rr-upstream-dr-multiregion-001` | 災害復旧設計 |39| イベント駆動 | `rr-upstream-event-driven-semantics-001` | イベント駆動設計 |40| 外部依存 | `rr-upstream-external-dependencies-001` | 外部依存関係 |41| 障害, 可観測性 | `rr-upstream-failure-modes-observability-001` | 障害モード分析 |42| 結合, コントラクト | `rr-upstream-integration-contracts-001` | 結合コントラクト |43| マイグレーション | `rr-upstream-migration-rollout-rollback-001` | マイグレーション計画 |44| マルチテナント | `rr-upstream-multitenancy-isolation-001` | テナント分離 |45| OpenAPI | `rr-upstream-openapi-contract-001` | OpenAPI 仕様検証 |46| SLO, 運用性 | `rr-upstream-operability-slo-001` | 運用性・SLO |47| 要件, 受入 | `rr-upstream-requirements-acceptance-001` | 要件・受入条件 |4849### デフォルト動作5051- キーワード指定なし → 変更内容から自動判定52- 複数カテゴリに該当 → もっとも関連性の高いスキルを優先5354## Execution Flow / 実行フロー5556```text571. 変更種別の判定58 ├─ ADR/設計ドキュメント → adr-decision-quality を優先59 ├─ API定義/エンドポイント → api-design, api-versioning-compat を優先60 ├─ モジュール構成変更 → architecture-boundaries を優先61 ├─ データモデル変更 → data-model-db-design を優先62 └─ キーワード指定あり → 該当スキルを直接選択63642. スキルの実行（該当する専門スキルを並列実行可能）65 ├─ 設計品質系: adr-decision-quality, architecture-diagrams, architecture-traceability66 ├─ 構造系: architecture-boundaries, bounded-context-language, data-flow-state-ownership67 ├─ API系: api-design, api-versioning-compat, openapi-contract, integration-contracts68 ├─ 運用系: availability-architecture, capacity-cost-design, dr-multiregion, operability-slo69 └─ リスク系: architecture-risk-register, failure-modes-observability, external-dependencies70713. 統合72 ├─ 重複する指摘の除去73 └─ 複数カテゴリ該当時は関連性の高いスキルを優先74```7576## Output Format / 出力形式7778```text79<file>:<line>: <message>80```8182- **Finding**: 何が問題か（1文）83- **Impact**: 何が困るか（短く）84- **Fix**: 次の一手（最小の修正案）8586## 他スキルとの関係8788| スキル | 関係 | 棲み分け |89| ----------------------- | ---- | ------------------------------------------------------------- |90| `adversarial-review` | 補完 | architecture は「設計の正しさ」、adversarial は「設計の盲点」 |91| `river-review-security` | 補完 | architecture は「構造」、security は「脆弱性」 |92| `river-review-code` | 補完 | architecture は「マクロ設計」、code は「ミクロ品質」 |9394## References9596- [ROUTING.md](./references/ROUTING.md): 詳細なルーティングルール
ConcertoNotes