Skip to content
arch-review logo

Architecture Review

arch-review

アーキテクチャレビュー。SOLID/YAGNI/DRY/KISS原則に基づき、オーバーエンジニアリングを避けた実践的な設計改善を提案する。

SKILL.md

Full skill instructions

Architecture Review

オーバーエンジニアリングを避けた、実践的なアーキテクチャレビューを行います。

基本姿勢

「動くコードは正義」 - 完璧な設計より、適切な設計を目指す。

  • クリーンアーキテクチャは理想だが、大抵やり過ぎ
  • 抽象化は「3回同じパターンが出たら」検討
  • 将来の要件を予測しすぎない

レビュー原則

採用する原則

原則適用基準やり過ぎライン
SRP1クラス1責務責務を細かく分けすぎてクラス爆発
OCP拡張ポイントが明確な箇所のみStrategy全部入り
DIP外部依存(DB, API)の境界全てにInterface
DRY3回以上の重複似てるだけで共通化
YAGNI今必要なものだけ将来のための抽象化
KISS最もシンプルな解法「エレガント」な解法

軽視してよい原則

  • LSP: 継承より合成を使えば問題にならない
  • ISP: 巨大なInterfaceがない限り不要

アーキテクチャパターン評価

推奨度: シンプルMVC > レイヤード > ヘキサゴナル > クリーン
        ←─────── プロジェクト規模 ───────→
パターン適用条件避けるべき状況
シンプルMVC小〜中規模、CRUD中心複雑なドメインロジック
レイヤード中規模、明確な層分離が必要層間の依存が単純な場合
ヘキサゴナル外部依存が多い、テスト重視外部依存が少ない
クリーン大規模、長期保守、複雑なドメインほとんどのプロジェクト

チェックポイント

🔴 即座に修正が必要

  • 循環参照
  • 神クラス(500行超、10メソッド超)
  • 外部依存の直接呼び出しがビジネスロジックに混在
  • シークレットのハードコード

🟡 改善を推奨

  • 1ファイル300行超
  • 3箇所以上の重複コード
  • ネストが4段以上
  • 曖昧な命名(data, info, manager, handler)

🟢 現状維持でOK

  • 2箇所だけの類似コード → 共通化しない
  • 将来使うかもしれない拡張ポイント → 作らない
  • 「念のため」のInterface → 不要

出力形式

## アーキテクチャレビュー結果

### 現状評価
- パターン: [検出されたパターン]
- 規模感: [小/中/大]
- 適合度: [適切 / やや過剰 / 不足]

### 🔴 Critical(要修正)
[問題] → [修正案]

### 🟡 Warning(改善推奨)
[問題] → [修正案]

### 🟢 Good(このままでOK)
- [あえて抽象化しなくてよい箇所]
- [シンプルなままで正解な箇所]

### オーバーエンジニアリング警告
[不要な抽象化、過剰な設計パターンの指摘]

アンチパターン検出

やり過ぎサイン:

  • interface が impl と1:1対応
  • Factory が1種類しか生成しない
  • Strategy が2パターンしかない
  • Repository が findAll と findById だけ
  • 3層以上の継承階層
  • DTO↔Entity変換が機械的コピー

これらは「将来のため」ではなく「今の複雑さ」

More Other skills

Delegate independent tasks to specialized agents working concurrently with isolated context.

295.4K 206.3K
View

Applies Anthropic's official brand colors and typography to any sort of artifact that may benefit from having Anthropic's look-and-feel. Use it when brand colors or style guidelines, visual formatting, or company design standards apply.

179.7K 100.1K
View

Toolkit for styling artifacts with a theme. These artifacts can be slides, docs, reportings, HTML landing pages, etc. There are 10 pre-set themes with colors/fonts that you can apply to any artifact that has been creating, or can generate a new theme on-the-fly.

179.7K 89.5K
View

A set of resources to help me write all kinds of internal communications, using the formats that my company likes to use. Claude should use this skill whenever asked to write some sort of internal communications (status reports, leadership updates, 3P updates, company newsletters, FAQs, incident reports, project updates, etc.).

179.7K 76.1K
View

Fundamental design principles for writing maintainable, testable Python code.

40.2K 21.9K
View

Modern responsive layouts using container queries, fluid typography, CSS Grid, and mobile-first strategies.

40.2K 20K
View

Generate Playwright tests from scenarios using interactive browser exploration and validation.

39.7K 18.2K
View

Modern React state management with Redux Toolkit, Zustand, Jotai, and React Query for every state category.

40.2K 14.7K
View

GDPR-compliant data handling with consent management, data subject rights, and privacy controls.

40.2K 14.3K
View

Python error handling patterns including input validation, exception hierarchies, and partial failure handling. Use when implementing validation logic, designing exception strategies, handling batch processing failures, or building robust APIs.

40.2K 13.5K
View

Detailed multi-file refactor planning with safety checks and rollback guidance.

39.7K 13K
View