VS Code + Cline で ”Devin 風”開発を行う方法② - 実践編

[Environment]::SetEnvironmentVariable("OLLAMA_MAX_LOADED_MODELS","1","User")
[Environment]::SetEnvironmentVariable("OLLAMA_KEEP_ALIVE","30m","User")

YelpCampCleanArch/
└─ memory-bank/
   ├─ projectbrief.md
   ├─ productContext.md
   ├─ systemPatterns.md
   ├─ techContext.md
   ├─ activeContext.md
   └─ progress.md

# Cline のメモリーバンク

私は Cline です。優秀なソフトウェアエンジニアですが、1つ特徴があります。それは「セッションが切り替わると記憶が完全にリセットされる」という点です。
これは制限ではなく、完璧なドキュメントを維持するための仕組みです。
リセット後、私はプロジェクトを理解し作業を継続するために、メモリーバンクに完全に依存します。すべてのタスクの開始時に、必ずすべてのメモリーバンクファイルを読む必要があります。これは必須であり、省略することはできません。

---

## メモリーバンクの構成

メモリーバンクは、memory-bank/配下にMarkdown形式で記述されたコアファイルと追加のコンテキストファイルで構成されます。これらのファイルは明確な階層構造を持ち、互いに補完し合います。

---

### ■ コアファイル（必須）

1. `projectbrief.md`
- すべてのファイルの基礎となるドキュメント
- 存在しない場合はプロジェクト開始時に作成する
- プロジェクトの基本要件と目標を定義する
- プロジェクト範囲の基準となる

---

2. `productContext.md`
- このプロジェクトの存在理由
- 解決する問題
- どのように動作するべきか
- ユーザー体験の目標

---

3. `activeContext.md`
- 現在の作業の焦点
- 最近の変更内容
- 次に行う作業
- 現在の意思決定や検討事項
- 重要なパターンや方針
- 学びやプロジェクトの知見

---

4. `systemPatterns.md`
- システムのアーキテクチャ
- 重要な技術的意思決定
- 使用している設計パターン
- コンポーネント間の関係
- 重要な実装経路

---

5. `techContext.md`
- 使用している技術
- 開発環境
- 技術的制約
- 依存関係
- ツールの使用パターン

---

6. `progress.md`
- 現在うまくいっていること
- まだ実装されていないこと
- 現在の状況
- 既知の問題
- 意思決定の変遷

---

### ■ 追加コンテキスト

必要に応じて memory-bank/ 配下に追加ファイルやフォルダを作成して構いません。

例：
- 複雑な機能のドキュメント
- 外部連携仕様
- API仕様書
- テスト戦略
- デプロイ手順

---

## ドキュメントの更新

メモリーバンクは以下のタイミングで更新します：

1. 新しいパターンを発見したとき
2. 大きな変更を実装した後
3. ユーザーから「update memory bank」と指示されたとき（必ず全ファイルを確認する）
4. 文脈の明確化が必要なとき

---

## 重要

記憶がリセットされるたびに、私は完全に初期状態からスタートします。
メモリーバンクは、過去の作業とつながる唯一の手段です。
そのため、正確かつ明確に維持されなければなりません。私の能力は、このメモリーバンクの質に完全に依存しています。

# 言語ルール

## 出力言語
- すべての出力は必ず日本語で行うこと  
- 英語への切り替えは禁止する（明示的な指示がある場合を除く）  

## ファイル生成
- 生成するすべてのドキュメントは日本語で記述する  

## 補足
- コードおよび技術用語は英語のままで問題ない  

YelpCampCleanArch/
└─ .clinerules/
   └─ rules.md

# プロジェクトルール

## 基本ルール
- 既存の挙動を変更しない
- 小さく安全な変更を優先する
- 無関係なファイルを変更しない
- 一度に大規模な変更を行わない

## 出力ルール
- すべての説明は日本語で行う
- 変更前に方針を説明する

## リファクタリング方針
- 明示的に許可されていない限り、既存の挙動を変更しない
- 全体を書き換えるのではなく、責務の分離・抽出を優先する
- ファイル名・クラス名は既存の命名規則に合わせる（必要な場合のみ変更）
- アーキテクチャが不明確な場合は、編集前に2〜3案を提示する

## Strangler Fig パターンに関する制約
- 既存の MVC アプリは Strangler Fig パターンで段階的に置き換える
- 既存実装を一括で書き換えない
- 旧実装は参照専用とし、明示的な指示がない限り変更しない

## 変更禁止領域
以下の既存 MVC 領域は、明示的な指示がない限り変更しないこと
- `app/controllers/`
- `app/db/`
- `app/models/`
- `app/routes/`
- `app/utils/`
- `app/views/`

## 実装先の制約
- 新しい実装は `app/src/` 配下に追加する
- Controller は `Adapter/Controllers/` に配置する
- UseCase は `Application/UseCases/` 配下に配置する
- Entity は `Domain/Entities/` に配置する
- Repository interface は `Domain/Repositories/` に配置する
- Repository の実装は `Infrastructure/Db/` 配下に配置する
- 既存 MVC 領域には新しいロジックを追加しない

## 補足
- 既存コードの責務は分析してよいが、変更は行わない
- 必要な接続ポイントがある場合は、変更前に必ず方針を説明する

## テストコマンド
- `cd app 2>$null; vendor/bin/pest`
- `cd app -ErrorAction SilentlyContinue; vendor/bin/pest`

YelpCampCleanArch/
└─ .agents/
   └─ skills/
      ├─ analyze-project/
      │  └─ SKILL.md
      └─ refactor-usecase/
         └─ SKILL.md

---
name: analyze-project
description: プロジェクトの構造解析を行います
---

# スキル：プロジェクト構造の分析

## 目的
リファクタリングを行う前に、現在のプロジェクト構造を正しく把握する。

## 実行内容
このスキルを使用する場合、以下を実施すること。

1. ディレクトリ構成および主要なエントリーポイントを確認する  
2. 以下の要素を特定する  
   - コントローラ（controllers）  
   - サービス / ユースケース（services / use cases）  
   - インフラ層 / データベースアクセス（infrastructure / database）  
   - ビュー / テンプレート（views / templates）  
   - 設定ファイル（configuration）  
3. 現在のアーキテクチャを簡潔に要約する  
4. 責務分離の観点で問題がありそうな箇所を指摘する  
5. 安全に実施可能なリファクタリング候補を最大3つ提案する（小さい順）  
6. 明示的な指示がない限り、ファイルの変更は行わない  

## 出力形式
以下の形式で出力すること。

- 現在の構造の要約  
- 想定されるアーキテクチャ上の問題点  
- リファクタリング候補（最大3つ）

---
name: refactor-usecase
description: ユースケースへのリファクタリングを行います
---

# スキル：単一ユースケースのリファクタリング

## 目的
大規模な書き換えを行わず、1つのユースケースを安全に改善する。

## 実行内容
このスキルを使用する場合、以下を実施すること。

1. 関連するファイルを事前に確認する  
2. 改善対象となる「単一のユースケース」または責務を特定する  
3. 以下を説明する  
   - 現在の実装内容  
   - 改善したいポイント  
   - 変更対象となる最小限のファイル  
4. 対象のユースケースのみに限定してリファクタリングを行う  
5. 明示的な指示がない限り、既存の挙動は維持する  
6. コードの結合度が高すぎる場合は作業を中断し、より小さな単位での分離案（抽出）を提案する  
7. 実装後、以下をまとめる  
   - 何を変更したか  
   - 依然として結合が残っている箇所  
   - 次に行うべき改善案  

## 推奨リファクタリングパターン
- メソッドの抽出（extract method）  
- サービス / ユースケースの抽出（extract service / use case）  
- データベースアクセスの分離（isolate database access）  
- 可能な範囲で、コントローラからバリデーションロジックを分離  

## 禁止・回避事項
- プロジェクト全体に及ぶ書き換え  
- 多数のファイルを一度にリネームすること  
- 必要性が明確でないルーティングやブートストラップの変更  

## 目的
既存の MVC コントローラ（users.php の login 処理）から、
クリーンアーキテクチャの UseCase を切り出すための設計ドキュメントを作成する。

## ソース
- `app\controllers\users.php`
- `login` 関数

## ユースケース
- ベースパス: `app\src\application\UseCases\User\`
- 入力ファイル:`UserLoginInput.php`
- 出力ファイル:`UserLoginOutput.php`
- ユースケースインターフェイス: `UserLoginUseCaseInterface.php`
- ユースケース:`UserLoginUseCase.php`

## 参照ファイル
- ベースパス: `app\src\application\UseCases\User\`
- 入力ファイル:`UserRegisterInput.php`
- 出力ファイル:`UserRegisterOutput.php`
- ユースケース:`UserRegisterUseCase.php`
- ユースケースインターフェイス: `UserRegisterUseCaseInterface.php`
- 例外ファイル:`app\src\application\Exception\ApplicationException.php`
- ユーザー情報ファイル:`app\src\domain\Entities\User.php`

## 名前空間
- `UserRepositoryInterface`: `Domain\Repositories\UserRepositoryInterface`
- `ApplicationException`: `Application\Exception\ApplicationException`

## コーディング規約
- `final class` を使用する
- コンストラクタプロモーション形式で `private` プロパティを定義する
- プロパティは公開しない
- setter は作成しない
- 値の参照は getter メソッドで行う

## 生成予定ファイル
- app\src\application\UseCases\User\UserLoginInput.php
- app\src\application\UseCases\User\UserLoginOutput.php
- app\src\application\UseCases\User\UserLoginUseCaseInterface.php
- app\src\application\UseCases\User\UserLoginUseCase.php
- それ以外の一切のファイルは変更や生成は不可

<?php

namespace Application\UseCases\User;

class UserLoginInput
{
    private string $username;
    private string $password;

    public function __construct(string $username, string $password)
    {
        $this->username = $username;
        $this->password = $password;
    }

    public function getUsername(): string
    {
        return $this->username;
    }

    public function getPassword(): string
    {
        return $this->password;
    }
}

<?php

declare(strict_types=1);

namespace Application\UseCases\User;

/**
 * ユーザー登録ユースケースの入力データ
 * （Controller や CLI などの外側の層から渡される）
 */
final class UserRegisterInput
{
    public function __construct(
        private string $username,
        private string $email,
        private string $plainPassword
    ) {}

    public function username(): string
    {
        return $this->username;
    }

    public function email(): string
    {
        return $this->email;
    }

    public function plainPassword(): string
    {
        return $this->plainPassword;
    }
}

<?php

namespace Application\UseCases\User;

class UserLoginUseCase implements UserLoginUseCaseInterface
{
    public function execute(UserLoginInput $input): UserLoginOutput
    {
        // Placeholder implementation - replace with actual login logic
        $token = bin2hex(random_bytes(16)); // Generate a random token for now
        return new UserLoginOutput($token);
    }
}

不明な点はいくつかあります。

1.  `app\controllers\users.php` の `login` 関数はどのような処理を行っていますか？ 具体的な処理内容を教えていただけますでしょうか。
2.  `UserLoginInput.php` と `UserLoginOutput.php` の具体的なフィールド構成はどのようなものですか？
3.  `Domain\Repositories\UserRepositoryInterface` で定義されているメソッドの内容を教えていただけますでしょうか。

以下のソースからクリーンアーキテクチャのユースケースを作成するプランを日本語で考えてください。

## ソース
- `app\controllers\users.php`
- `login` 関数

## ユースケース
- ベースパス: `app\src\application\UseCases\User\`
- 入力ファイル:`UserLoginInput.php`
  - `username`, `password` getter 関数, setter なし
- 出力ファイル:`UserLoginOutput.php`
  - `username` getter 関数, setter なし
- ユースケースインターフェイス: `UserLoginUseCaseInterface.php`
  - `public function execute(UserLoginInput $input): UserLoginOutput`
- ユースケース:`UserLoginUseCase.php`
  - `__construct(private UserRepositoryInterface $userRepository)`
  - `execute(UserLoginInput $input): UserLoginOutput`
    - `$user = $this->userRepository->findByUsername($input->username())`
    - `password_verify($input->password(), $user->passwordHash())`
    - 上記の処理がエラーの場合: `throw new ApplicationException` メッセージは日本語で

## 参照ファイル
- ベースパス: `app\src\application\UseCases\User\`
- 入力ファイル:`UserRegisterInput.php`
- 出力ファイル:`UserRegisterOutput.php`
- ユースケース:`UserRegisterUseCase.php`
- ユースケースインターフェイス: `UserRegisterUseCaseInterface.php`
- 例外ファイル:`app\src\application\Exception\ApplicationException.php`
- ユーザー情報ファイル:`app\src\domain\Entities\User.php`

## 名前空間
- `UserRepositoryInterface`: `Domain\Repositories\UserRepositoryInterface`
- `ApplicationException`: `Application\Exception\ApplicationException`

## コーディング規約
- `final class` を使用する
- コンストラクタプロモーション形式で `private` プロパティを定義する
  - 実装方法は次のファイルを参照: `app\src\application\UseCases\User\UserRegisterOutput.php`
- プロパティは公開しない
- setter は作成しない
- 値の参照は getter メソッドで行う

## コンストラクタプロモーション形式
```php
final class UserRegisterOutput
{
    public function __construct(
        private string $username
    ) {}

    public function username(): string
    {
        return $this->username;
    }
}
```

## 生成可能ファイル
- app\src\application\UseCases\User\UserLoginInput.php
- app\src\application\UseCases\User\UserLoginOutput.php
- app\src\application\UseCases\User\UserLoginUseCaseInterface.php
- app\src\application\UseCases\User\UserLoginUseCase.php
- それ以外の一切のファイルは変更や生成は不可

activeContext.md ファイルの内容を実装してください。

<?php

declare(strict_types=1);

namespace Application\UseCases\User;

use Application\Exception\ApplicationException;
use db\UserQuery;
use Domain\Entities\User;
use Domain\Repositories\UserRepositoryInterface;

/**
 * ログインユースケースの実装
 * 
 * 既存の MVC アーキテクチャの影響を受けず、
 * クリーンアーキテクチャに従って実装されます。
 */
final class UserLoginUseCase implements UserLoginUseCaseInterface
{
    public function __construct(
        private readonly UserRepositoryInterface $userRepository
    ) {}

    /**
     * ログインを実行する
     * 
     * @param UserLoginInput $input ログイン入力データ
     * @return UserLoginOutput ログイン結果
     * @throws ApplicationException ユーザー名またはパスワードが間違っている場合
     */
    public function execute(UserLoginInput $input): UserLoginOutput
    {
        // ユーザー名で検索
        $user = $this->userRepository->findByUsername($input->username());

        if ($user === null) {
            // ユーザーが存在しない場合
            throw new ApplicationException('ユーザー名またはパスワードが間違っています。');
        }

        // パスワード認証
        if (!password_verify($input->password(), $user->passwordHash())) {
            throw new ApplicationException('ユーザー名またはパスワードが間違っています。');
        }

        // ログイン成功
        return new UserLoginOutput($user->username());
    }
}

UserLoginUseCase.php のテストケースを作成して検証してください。
app\tests\Pest\Unit\Application\UseCases\User\UserRegisterUseCaseTest.php ファイルを参照して
app\tests\Pest\Unit\Application\UseCases\User フォルダ以下に作成してください。
テストコマンドは `cd app -ErrorAction SilentlyContinue; vendor/bin/pest` です。

<?php

use Domain\Entities\User;
use Domain\Repositories\UserRepositoryInterface;
use Application\UseCases\User\UserLoginInput;
use Application\UseCases\User\UserLoginOutput;
use Application\UseCases\User\UserLoginUseCase;
use Application\Exception\ApplicationException;

it('logs in a user successfully', function () {

    // -------- Mock Repository --------
    $repo = $this->createMock(UserRepositoryInterface::class);

    // bcrypt ハッシュを作成（パスワード検証用）
    $hashedPassword = password_hash('secret123', PASSWORD_DEFAULT);

    // findByUsername → ユーザーを返す
    $repo->method('findByUsername')
        ->willReturn(
            new User(
                username: 'alice',
                email: 'alice@example.com',
                passwordHash: $hashedPassword
            )
        );

    // -------- UseCase --------
    $useCase = new UserLoginUseCase($repo);

    $input = new UserLoginInput(
        username: 'alice',
        password: 'secret123'
    );

    $output = $useCase->execute($input);

    // -------- Assertions --------
    expect($output)->toBeInstanceOf(UserLoginOutput::class);
    expect($output->username())->toBe('alice');
});

it('throws when user not found', function () {

    // -------- Mock Repository --------
    $repo = $this->createMock(UserRepositoryInterface::class);

    // findByUsername → null（ユーザーが存在しない）
    $repo->method('findByUsername')
        ->willReturn(null);

    $useCase = new UserLoginUseCase($repo);

    $input = new UserLoginInput(
        username: 'alice',
        password: 'secret123'
    );

    // -------- Exception Expectation --------
    $this->expectException(ApplicationException::class);

    $useCase->execute($input);
});

it('throws when password is invalid', function () {

    // -------- Mock Repository --------
    $repo = $this->createMock(UserRepositoryInterface::class);

    // bcrypt ハッシュを作成（正しいパスワード用）
    $hashedPassword = password_hash('secret123', PASSWORD_DEFAULT);

    // findByUsername → ユーザーを返すが、パスワードは異なる
    $repo->method('findByUsername')
        ->willReturn(
            new User(
                username: 'alice',
                email: 'alice@example.com',
                passwordHash: $hashedPassword
            )
        );

    $useCase = new UserLoginUseCase($repo);

    $input = new UserLoginInput(
        username: 'alice',
        password: 'wrong_password'
    );

    // -------- Exception Expectation --------
    $this->expectException(ApplicationException::class);

    $useCase->execute($input);
});