Analyzer Architecture Review

Analyzer Architecture Review

Overview

このスキルは、analyzerアプリケーションのPort&Adapterアーキテクチャ（ヘキサゴナルアーキテクチャ）への準拠をレビューします。

アーキテクチャの詳細: references/analyzer-architecture.mdを参照してください。

Review Workflow

1. 変更ファイルの特定

git diff --name-only

以下のディレクトリの変更を確認：

• app/analyzer/internal/port/
• app/analyzer/internal/adapter/
• app/analyzer/internal/usecase/
• app/analyzer/internal/model/
• app/analyzer/internal/registory/

2. レイヤー別レビュー

変更されたファイルのレイヤーに応じて、該当するチェック項目を確認します。

Port層の変更

• ✅ 関数型(type Name = func(...))で定義されているか
• ✅ 第一引数がcontext.Contextか（必要な場合）
• ✅ 実装コードが含まれていないか（型定義のみ）
• ✅ 必要に応じて入力/出力用の構造体が定義されているか
• ❌ Port層に具体的な実装が含まれていないか

詳細: analyzer-architecture.md#Port層

Adapter層の変更

• ✅ New*という命名規則のコンストラクタがあるか
• ✅ Portインターフェースを実装しているか
• ✅ 外部依存(HTTP、DB、ファイルなど)がこの層に閉じ込められているか
• ✅ 実装関数は非公開(小文字始まり)か
• ❌ ビジネスロジックが含まれていないか

詳細: analyzer-architecture.md#Adapter層

Usecase層の変更

• ✅ New*コンストラクタで全ての依存性を注入しているか
• ✅ Portインターフェースのみに依存しているか
• ✅ 実装関数は非公開(小文字始まり)か
• ❌ Adapter層の具体的な実装を直接参照していないか
• ❌ 外部システムに直接アクセスしていないか

importチェック: Usecaseはinternal/port/*とinternal/modelのみをimportすべき

詳細: analyzer-architecture.md#Usecase層

Model層の変更

• ✅ ドメイン知識を表現する型を定義しているか
• ✅ ドメインロジックがここに集約されているか
• ❌ 外部依存(DB、APIなど)を持っていないか
• ❌ Port/Adapter/Usecaseに依存していないか

詳細: analyzer-architecture.md#Model層

Registry層の変更

• ✅ 全ての依存性の組み立てがここで行われているか
• ✅ 具体的な実装の選択がここで行われているか
• ✅ Entry point(cmd/)から呼び出されているか
• ❌ ビジネスロジックが含まれていないか

詳細: analyzer-architecture.md#Registry層

3. 依存関係の検証

許可される依存:

• Usecase → Port
• Usecase → Model
• Adapter → Port
• Adapter → Model
• Port → Model（必要に応じて）

禁止される依存:

• ❌ Port → Usecase
• ❌ Port → Adapter
• ❌ Usecase → Adapter
• ❌ Model → Port/Usecase/Adapter

検証方法: ファイルのimport文を確認

// ✅ 正しい Usecase の import
import (
    "github.com/ss49919201/keeput/app/analyzer/internal/port/fetcher"
    "github.com/ss49919201/keeput/app/analyzer/internal/model"
)

// ❌ 間違った Usecase の import
import (
    "github.com/ss49919201/keeput/app/analyzer/internal/adapter/fetcher/hatena"  // NG!
)

詳細: analyzer-architecture.md#依存関係のルール

4. 命名規則の確認

• ✅ New*関数がPort型を返しているか
• ✅ 実装関数は非公開（小文字始まり）か
• ✅ Port型の命名が適切か

詳細: analyzer-architecture.md#命名規則

Quick Reference

アーキテクチャ違反の典型例

1. UsecaseがAdapterに依存

   // ❌ NG
   import "github.com/.../adapter/fetcher/hatena"
   

2. Port層に実装を含める

   // ❌ NG: Port層に実装があってはいけない
   func DefaultFetchLatestEntry() FetchLatestEntry { /* 実装 */ }
   

3. Modelが外部依存を持つ

   // ❌ NG
   import "github.com/aws/aws-sdk-go-v2/service/s3"
   

詳細な例: analyzer-architecture.md#アーキテクチャ違反の例

Output Format

レビュー結果は以下の形式で出力：

# Analyzer Architecture Review

## Summary
- 変更ファイル数: X
- アーキテクチャ違反: Y件
- 警告: Z件

## Details

### ✅ Good: Port層の定義（port/fetcher/entry.go）
- 関数型で正しく定義されている
- 実装コードが含まれていない

### ❌ Issue: Usecase層の依存関係（usecase/analyze.go）
- Adapter層を直接importしている
- 修正方法: Registry層で依存性を注入する

### ⚠️ Warning: 命名規則（adapter/fetcher/hatena.go）
- 実装関数が公開されている（小文字始まりにすべき）

Resources

このスキルには以下のリファレンスが含まれています：

• references/analyzer-architecture.md: analyzerアプリケーションの包括的なアーキテクチャガイド
  • ディレクトリ構造
  • 各層の役割と責務
  • 依存関係のルール
  • 命名規則
  • アーキテクチャ違反の例
  • レビュー時の確認事項