River review architecture

Skill s977043/river-review/skills/agent-skills/river-review-architecture

設計・アーキテクチャ観点のレビューエージェント。依存関係、境界設計、データモデル、API設計等の個別スキルへルーティングする。From its SKILL.md

Install

npx -y skills add s977043/river-review --skill river-review-architecture

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

2 stars2 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

What its file declares

Copied from the file, not written here

The file declares its own license as MIT. That is the author’s claim about this one file, and it is not the same thing as the license GitHub reports for the repository, which is listed with the other numbers below.

SKILL.md

5.9 KB, as published. Nobody here has run it

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: 詳細なルーティングルール