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