river-review-architecture · v0.1.0 · 2026-07-01 · sha256 74d628fd3275cf3d
river-review-architecture v0.1.0A
Immutable. This exact content is served forever at /api/v1/blob/74d628fd3275cf3d.
---
id: river-review-architecture
name: river-review-architecture
description: |
設計・アーキテクチャ観点のレビューエージェント。
依存関係、境界設計、データモデル、API設計等の個別スキルへルーティングする。
category: upstream
phase: [upstream]
severity: major
applyTo:
- 'src/**/*.{ts,tsx,js,jsx,mjs}'
- 'docs/**/*design*.md'
- 'docs/adr/**/*'
- 'pages/**/*design*.md'
inputContext: [diff, fullFile, adr]
outputKind: [findings, questions, actions]
tags: [architecture, design, entry, routing]
version: 0.1.0
license: MIT
---
# 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 / 実行フロー
```text
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 / 出力形式
```text
<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): 詳細なルーティングルール