phasegate 0.15.0 → 0.31.0

package/skills/it-test-logic-designer/SKILL.md

@@ -168,385 +168,21 @@ TDD実装フェーズ
 
 ### it_test_logic.md の構成
 
-```markdown
-# ITテストロジック設計: {Unit名}
-
-## 1. テストファイル構成
-
-| ファイルパス | 対象コンポーネント | ケース数 |
-|------------|------------------|---------|
-| `backend/test/integration/{context}/{usecase}.test.ts` | {UseCase名} | X |
-| `backend/test/integration/{context}/{repository}.test.ts` | {Repository名} | X |
-| `backend/test/integration/{context}/{controller}.test.ts` | {Controller名} | X |
-
-## 2. テストヘルパー・シードデータ
-
-### テストヘルパーのインポート
-
-```typescript
-import { target, context } from '../../helpers/common-helper.js';
-import { getTestClient, cleanupTestData } from '../../helpers/supabase-test-helper.js';
-```
-
-### シードデータ設計
+出力する `it_test_logic.md` は以下のセクションで構成する:
 
-```typescript
-// backend/test/seeds/{context}/{seed-name}.ts
+1. **テストファイル構成** — ファイルパス・対象コンポーネント・ケース数の一覧表
+2. **テストヘルパー・シードデータ** — インポートとシードデータ定義
+3. **Repositoryテスト詳細ロジック** — CRUD操作の疑似コード
+4. **UseCaseテスト詳細ロジック** — ビジネスロジック統合の疑似コード
+5. **Controllerテスト詳細ロジック** — 認証・認可・バリデーションの疑似コード
+6. **トランザクション・クリーンアップ戦略** — データ独立性の担保方法
+7. **テスト実行コマンド** — 実行方法
 
-export const {SEED_NAME}_SEED = {
-  // テストデータ定義
-  organizations: [
-    { id: 'org-001', name: 'テスト組織' },
-  ],
-  users: [
-    { id: 'user-001', organizationId: 'org-001', name: 'テストユーザー' },
-  ],
-};
+各セクションのテンプレート・疑似コードは以下の参照ファイルに従う:
 
-export async function seed{SeedName}(client: SupabaseClient): Promise<void> {
-  // INSERT処理
-}
-
-export async function cleanup{SeedName}(client: SupabaseClient): Promise<void> {
-  // DELETE処理
-}
-```
-
-## 3. Repositoryテスト詳細ロジック
-
-### {Repository名} テスト
-
-#### ファイル: `{repository}.test.ts`
-
-```typescript
-import { describe, it, expect, beforeAll, afterEach } from 'vitest';
-import { target, context } from '../../helpers/common-helper.js';
-import { getTestClient, cleanupTestData } from '../../helpers/supabase-test-helper.js';
-import { {Repository} } from '../../../src/{context}/infrastructure/repositories/{repository}.js';
-
-target('{Repository}', () => {
-  let repository: {Repository};
-  const idsToCleanup: string[] = [];
-
-  beforeAll(() => {
-    repository = new {Repository}(getTestClient());
-  });
-
-  afterEach(async () => {
-    for (const id of idsToCleanup) {
-      await cleanupTestData(id);
-    }
-    idsToCleanup.length = 0;
-  });
-
-  // IT-REPO-{Name}-001: 保存テスト
-  describe('save', () => {
-    context('新規エンティティを保存する場合', () => {
-      it('DBに永続化される', async () => {
-        // Arrange
-        const entity = create{Entity}({
-          // テストデータ
-        });
-        idsToCleanup.push(entity.id);
-
-        // Act
-        await repository.save(entity);
-
-        // Assert — DBから直接確認
-        const client = getTestClient();
-        const { data: row } = await client
-          .from('{table_name}')
-          .select('*')
-          .eq('id', entity.id)
-          .single();
-
-        expect(row).not.toBeNull();
-        expect(row!.{column}).toBe({expectedValue});
-      });
-    });
-  });
-
-  // IT-REPO-{Name}-002: 取得テスト
-  describe('findById', () => {
-    context('存在するIDで取得する場合', () => {
-      it('エンティティが再構築される', async () => {
-        // Arrange
-        const entity = create{Entity}({ /* ... */ });
-        await repository.save(entity);
-        idsToCleanup.push(entity.id);
-
-        // Act
-        const actual = await repository.findById(entity.id);
-
-        // Assert
-        expect(actual).not.toBeNull();
-        expect(actual!.id).toBe(entity.id);
-        expect(actual!.{property}).toBe({expectedValue});
-      });
-    });
-
-    context('存在しないIDで取得する場合', () => {
-      it('nullが返される', async () => {
-        // Act
-        const actual = await repository.findById(to{Entity}Id('non-existent-id'));
-
-        // Assert
-        expect(actual).toBeNull();
-      });
-    });
-  });
-
-  // IT-REPO-{Name}-003: 更新テスト
-  describe('update', () => {
-    it('既存エンティティが更新される', async () => {
-      // Arrange
-      const entity = create{Entity}({ /* 初期値 */ });
-      await repository.save(entity);
-      idsToCleanup.push(entity.id);
-
-      // Act
-      entity.updateXxx({newValue});
-      await repository.save(entity);
-
-      // Assert
-      const actual = await repository.findById(entity.id);
-      expect(actual!.{property}).toBe({newValue});
-    });
-  });
-});
-```
-
-## 4. UseCaseテスト詳細ロジック
-
-### {UseCase名} テスト
-
-#### ファイル: `{usecase}.test.ts`
-
-```typescript
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { target, context } from '../../helpers/common-helper.js';
-import { {UseCase} } from '../../../src/{context}/application/usecases/{usecase}.js';
-import { {Repository}Port } from '../../../src/{context}/application/ports/{repository}-port.js';
-
-target('{UseCase}', () => {
-  let useCase: {UseCase};
-  let mockRepository: {Repository}Port;
-
-  beforeEach(() => {
-    // モックRepository作成
-    mockRepository = {
-      save: vi.fn(),
-      findById: vi.fn(),
-      // ...
-    };
-    useCase = new {UseCase}(mockRepository);
-  });
-
-  // IT-UC-{Name}-001: 正常系
-  describe('execute', () => {
-    context('有効な入力の場合', () => {
-      it('処理が成功し結果が返される', async () => {
-        // Arrange
-        const input = {
-          // UseCaseへの入力
-        };
-        mockRepository.findById.mockResolvedValue(create{Entity}({}));
-
-        // Act
-        const actual = await useCase.execute(input);
-
-        // Assert
-        expect(actual.isSuccess()).toBe(true);
-        expect(actual.value.{property}).toBe({expectedValue});
-        expect(mockRepository.save).toHaveBeenCalledWith(
-          expect.objectContaining({ /* 期待されるエンティティ */ })
-        );
-      });
-    });
-
-    // IT-UC-{Name}-002: 異常系（バリデーションエラー）
-    context('無効な入力の場合', () => {
-      it('バリデーションエラーが返される', async () => {
-        // Arrange
-        const invalidInput = {
-          // 無効な入力
-        };
-
-        // Act
-        const actual = await useCase.execute(invalidInput);
-
-        // Assert
-        expect(actual.isFailure()).toBe(true);
-        expect(actual.error.code).toBe('{ERROR_CODE}');
-      });
-    });
-
-    // IT-UC-{Name}-003: 異常系（リソース不存在）
-    context('対象リソースが存在しない場合', () => {
-      it('NotFoundエラーが返される', async () => {
-        // Arrange
-        mockRepository.findById.mockResolvedValue(null);
-        const input = { id: 'non-existent-id' };
-
-        // Act
-        const actual = await useCase.execute(input);
-
-        // Assert
-        expect(actual.isFailure()).toBe(true);
-        expect(actual.error.code).toBe('NOT_FOUND');
-      });
-    });
-  });
-});
-```
-
-## 5. Controllerテスト詳細ロジック
-
-### {Controller名} テスト
-
-#### ファイル: `{controller}.test.ts`
-
-```typescript
-import { describe, it, expect, beforeEach, vi } from 'vitest';
-import { target, context } from '../../helpers/common-helper.js';
-import { {Controller} } from '../../../src/{context}/interfaces/controllers/{controller}.js';
-import { createMockCallableContext } from '../../helpers/firebase-test-helper.js';
-
-target('{Controller}', () => {
-  let controller: {Controller};
-  let mockUseCase: {UseCase};
-
-  beforeEach(() => {
-    mockUseCase = {
-      execute: vi.fn(),
-    };
-    controller = new {Controller}(mockUseCase);
-  });
-
-  // IT-API-{Name}-001: 認証テスト
-  describe('認証', () => {
-    context('未認証ユーザーの場合', () => {
-      it('UNAUTHENTICATEDエラーが返される', async () => {
-        // Arrange
-        const ctx = createMockCallableContext({ auth: null });
-        const data = { /* リクエストデータ */ };
-
-        // Act & Assert
-        await expect(controller.handle(data, ctx))
-          .rejects.toThrow('UNAUTHENTICATED');
-      });
-    });
-  });
-
-  // IT-API-{Name}-002: 認可テスト
-  describe('認可', () => {
-    context('権限のないユーザーの場合', () => {
-      it('PERMISSION_DENIEDエラーが返される', async () => {
-        // Arrange
-        const ctx = createMockCallableContext({
-          auth: { uid: 'user-001', role: 'VIEWER' },
-        });
-        const data = { /* リクエストデータ */ };
-
-        // Act & Assert
-        await expect(controller.handle(data, ctx))
-          .rejects.toThrow('PERMISSION_DENIED');
-      });
-    });
-  });
-
-  // IT-API-{Name}-003: バリデーションテスト
-  describe('バリデーション', () => {
-    context('必須フィールドが欠落している場合', () => {
-      it('INVALID_ARGUMENTエラーが返される', async () => {
-        // Arrange
-        const ctx = createMockCallableContext({
-          auth: { uid: 'user-001', role: 'ADMIN' },
-        });
-        const data = { /* 不完全なデータ */ };
-
-        // Act & Assert
-        await expect(controller.handle(data, ctx))
-          .rejects.toThrow('INVALID_ARGUMENT');
-      });
-    });
-  });
-
-  // IT-API-{Name}-004: 正常系
-  describe('正常系', () => {
-    it('UseCaseが実行され結果が返される', async () => {
-      // Arrange
-      const ctx = createMockCallableContext({
-        auth: { uid: 'user-001', role: 'ADMIN' },
-      });
-      const data = { /* 有効なリクエストデータ */ };
-      mockUseCase.execute.mockResolvedValue(
-        Result.ok({ /* UseCase結果 */ })
-      );
-
-      // Act
-      const actual = await controller.handle(data, ctx);
-
-      // Assert
-      expect(mockUseCase.execute).toHaveBeenCalledWith(
-        expect.objectContaining({ /* 期待される入力 */ })
-      );
-      expect(actual).toEqual({ /* 期待されるレスポンス */ });
-    });
-  });
-});
-```
-
-## 6. トランザクション・クリーンアップ戦略
-
-### クリーンアップパターン
-
-```typescript
-// テスト毎にクリーンアップ
-afterEach(async () => {
-  for (const id of idsToCleanup) {
-    await cleanupTestData(id);
-  }
-  idsToCleanup.length = 0;
-});
-```
-
-### トランザクション分離
-
-```typescript
-// トランザクション内でテスト実行（ロールバック）
-describe('トランザクションテスト', () => {
-  it('エラー時にロールバックされる', async () => {
-    // Arrange
-    const client = getTestClient();
-
-    // Act
-    try {
-      await client.rpc('test_transaction', { /* ... */ });
-    } catch (e) {
-      // Expected error
-    }
-
-    // Assert — ロールバックされていることを確認
-    const { data } = await client.from('{table}').select('*');
-    expect(data).toHaveLength(0);
-  });
-});
-```
-
-## 7. テスト実行コマンド
-
-```bash
-# 全ITテスト実行
-pnpm --filter backend test:integration
-
-# 特定ファイルのみ
-pnpm --filter backend test:integration -- {repository}.test.ts
-
-# watchモード
-pnpm --filter backend test:integration -- --watch
-```
-```
+- **Repository + シード + クリーンアップ**: [references/repository-test-patterns.md](references/repository-test-patterns.md)
+- **UseCase**: [references/usecase-test-patterns.md](references/usecase-test-patterns.md)
+- **Controller**: [references/controller-test-patterns.md](references/controller-test-patterns.md)
 
 ---
 
@@ -558,16 +194,7 @@ pnpm --filter backend test:integration -- --watch
 - **Assert**: 結果の検証（DB状態確認含む）
 
 ### 2. DB直接確認パターン
-```typescript
-// Assert — DBから直接確認
-const client = getTestClient();
-const { data: row } = await client
-  .from('{table}')
-  .select('*')
-  .eq('id', entity.id)
-  .single();
-expect(row).not.toBeNull();
-```
+Repositoryテストでは、`getTestClient()` でDBに直接クエリし永続化を検証する。テンプレートは [references/repository-test-patterns.md](references/repository-test-patterns.md) を参照。
 
 ### 3. モック戦略
 - **Repository**: 実DB使用（統合テストの本質）
@@ -581,8 +208,6 @@ expect(row).not.toBeNull();
 
 ---
 
----
-
 ## Phase 3: レビュー（Opus review）
 
 ### 実行主体

package/skills/it-test-logic-designer/references/controller-test-patterns.md

@@ -0,0 +1,100 @@
+# Controller テストパターン・テンプレート
+
+本ファイルは `it-test-logic-designer` スキルで使用するController/APIレイヤーのテストテンプレート集。
+
+---
+
+## Controller テストテンプレート
+
+### ファイル: `{controller}.test.ts`
+
+```typescript
+import { describe, it, expect, beforeEach, vi } from 'vitest';
+import { target, context } from '../../helpers/common-helper.js';
+import { {Controller} } from '../../../src/{context}/interfaces/controllers/{controller}.js';
+import { createMockCallableContext } from '../../helpers/firebase-test-helper.js';
+
+target('{Controller}', () => {
+  let controller: {Controller};
+  let mockUseCase: {UseCase};
+
+  beforeEach(() => {
+    mockUseCase = {
+      execute: vi.fn(),
+    };
+    controller = new {Controller}(mockUseCase);
+  });
+
+  // IT-API-{Name}-001: 認証テスト
+  describe('認証', () => {
+    context('未認証ユーザーの場合', () => {
+      it('UNAUTHENTICATEDエラーが返される', async () => {
+        // Arrange
+        const ctx = createMockCallableContext({ auth: null });
+        const data = { /* リクエストデータ */ };
+
+        // Act & Assert
+        await expect(controller.handle(data, ctx))
+          .rejects.toThrow('UNAUTHENTICATED');
+      });
+    });
+  });
+
+  // IT-API-{Name}-002: 認可テスト
+  describe('認可', () => {
+    context('権限のないユーザーの場合', () => {
+      it('PERMISSION_DENIEDエラーが返される', async () => {
+        // Arrange
+        const ctx = createMockCallableContext({
+          auth: { uid: 'user-001', role: 'VIEWER' },
+        });
+        const data = { /* リクエストデータ */ };
+
+        // Act & Assert
+        await expect(controller.handle(data, ctx))
+          .rejects.toThrow('PERMISSION_DENIED');
+      });
+    });
+  });
+
+  // IT-API-{Name}-003: バリデーションテスト
+  describe('バリデーション', () => {
+    context('必須フィールドが欠落している場合', () => {
+      it('INVALID_ARGUMENTエラーが返される', async () => {
+        // Arrange
+        const ctx = createMockCallableContext({
+          auth: { uid: 'user-001', role: 'ADMIN' },
+        });
+        const data = { /* 不完全なデータ */ };
+
+        // Act & Assert
+        await expect(controller.handle(data, ctx))
+          .rejects.toThrow('INVALID_ARGUMENT');
+      });
+    });
+  });
+
+  // IT-API-{Name}-004: 正常系
+  describe('正常系', () => {
+    it('UseCaseが実行され結果が返される', async () => {
+      // Arrange
+      const ctx = createMockCallableContext({
+        auth: { uid: 'user-001', role: 'ADMIN' },
+      });
+      const data = { /* 有効なリクエストデータ */ };
+      mockUseCase.execute.mockResolvedValue(
+        Result.ok({ /* UseCase結果 */ })
+      );
+
+      // Act
+      const actual = await controller.handle(data, ctx);
+
+      // Assert
+      expect(mockUseCase.execute).toHaveBeenCalledWith(
+        expect.objectContaining({ /* 期待される入力 */ })
+      );
+      expect(actual).toEqual({ /* 期待されるレスポンス */ });
+    });
+  });
+});
+```