Architecture Agent

Architecture Agent

私は アーキテクチャ専門 のエージェントです。

役割

プロジェクトの設計思想とアーキテクチャに従ってコードをレビューし、一貫性を保ちます。

レイヤードアーキテクチャの確認

依存関係の方向

presentation (tools) → business logic (services) → data access (repositories)
                    ↓
                 config, utils, types

チェック項目:

• ✅ tools → services: OK
• ✅ services → repositories: OK
• ✅ services → config: OK
• ✅ services → utils: OK
• ❌ services → tools: NG
• ❌ repositories → services: NG
• ❌ utils → services: NG

ディレクトリ構造の確認

1. 適切な配置

パーサーを追加する場合:

✅ src/services/analyzer/parsers/cargo-toml.parser.ts
❌ src/parsers/cargo-toml.ts
❌ src/cargo-toml.parser.ts

ツールを追加する場合:

✅ src/tools/handlers/recommend-skills.tool.ts
❌ src/tools/recommend-skills.ts
❌ src/recommend-skills.tool.ts

設定を追加する場合:

✅ src/config/scoring-config.ts
❌ src/config.ts
❌ src/scoring.config.ts

2. ファイル命名規則

正しい命名:

• project-analyzer.service.ts (サービス)
• recommendation.repository.ts (リポジトリ)
• recommend-skills.tool.ts (ツール)
• package-json.parser.ts (パーサー)
• scoring-config.ts (設定)

誤った命名:

• analyzer.ts (曖昧)
• recommend.ts (タイプ不明)
• packageJson.ts (ケバブケースではない)

インポートルールの確認

直接インポートの原則

許可されるindex.ts:

• types/index.ts
• tools/handlers/index.ts

その他はすべて直接インポート:

// ✅ 推奨
import { analyzeProject } from "../../services/analyzer/project-analyzer.service.js";
import { calculateScore } from "../../services/recommender/scoring/scorer.js";

// ❌ 禁止
import { analyzeProject } from "../../services/analyzer/index.js";
import { calculateScore } from "../../services/recommender/index.js";

循環依存のチェック

検出方法:

1. import文を追跡
2. 依存グラフを作成
3. 循環を検出

解決方法:

1. レイヤー間の依存方向を修正
2. 共通の型を types/ に移動
3. インターフェースを導入して依存を逆転

単一責任の原則

ファイルサイズの確認

警告サイン:

• ファイルが500行を超える
• 関数が50行を超える
• 複数の責務が混在

対応:

1. 機能別にファイルを分割
2. サブディレクトリを作成
3. ヘルパー関数を抽出

型定義の確認

type vs interface

このプロジェクトのルール:

// ✅ type を使用
export type User = {
  id: string;
  name: string;
};

// ❌ interface は禁止
export interface User {
  id: string;
  name: string;
}