Architecture Reviewer Skill
システムアーキテクチャを評価し、設計改善を提案するスキルです。
概要
SOLID原則、デザインパターン、マイクロサービス設計等の観点からアーキテクチャをレビューします。
主な機能
-
SOLID原則評価: 単一責任、開放閉鎖、リスコフ置換等
-
デザインパターン: 適切なパターンの適用状況
-
レイヤー分離: プレゼンテーション、ビジネス、データ層
-
依存性管理: 依存性注入、循環依存の検出
-
スケーラビリティ: 水平・垂直スケーリング
-
マイクロサービス: サービス境界、通信パターン
-
データベース設計: 正規化、インデックス、パーティショニング
使用方法
このアーキテクチャをレビュー： [アーキテクチャ図またはコード]
評価項目:
- SOLID原則
- スケーラビリティ
- 保守性
レビュー観点
- SOLID原則
単一責任原則（SRP）:
// ❌ 複数の責任 class User { saveToDatabase() {} sendEmail() {} generateReport() {} }
// ✅ 単一責任 class User {} class UserRepository { save(user: User) {} } class EmailService { send(to: string) {} } class ReportGenerator { generate(user: User) {} }
依存性逆転（DIP）:
❌ 具象に依存
class UserService: def init(self): self.db = MySQLDatabase() # 具象クラス
✅ 抽象に依存
class UserService: def init(self, database: DatabaseInterface): self.db = database # インターフェース
- レイヤー構造
┌─────────────────────────────┐ │ Presentation Layer │ UI, API Endpoints ├─────────────────────────────┤ │ Application Layer │ Use Cases, Orchestration ├─────────────────────────────┤ │ Domain Layer │ Business Logic, Entities ├─────────────────────────────┤ │ Infrastructure Layer │ Database, External APIs └─────────────────────────────┘
- マイクロサービス設計
推奨パターン:
- API Gateway: 単一エントリーポイント
- Service Discovery: 動的サービス検出
- Circuit Breaker: 障害の連鎖防止
- Event Sourcing: イベント駆動
- CQRS: コマンドとクエリの分離
出力例
アーキテクチャレビュー結果
総合評価: B+
良好な点
✅ クリーンアーキテクチャの採用 ✅ 適切な依存性注入 ✅ レイヤー分離が明確
改善点
[HIGH] 循環依存の存在
場所: OrderService ↔ PaymentService 影響: テスタビリティの低下、デプロイの複雑化 推奨: イベント駆動アーキテクチャに変更
[MEDIUM] 単一責任原則違反
場所: UserController 問題: 認証、認可、ビジネスロジックが混在 推奨: 責務を分離
アーキテクチャ提案
- イベント駆動への移行: サービス間の結合度削減
- CQRS導入: 読み書きの分離で性能向上
- キャッシュ層追加: Redis で頻繁な読み取り最適化
バージョン情報
-
スキルバージョン: 1.0.0
-
最終更新: 2025-01-22