@sk8metal/michi-cli 0.8.6 → 0.10.1

package/docs/user-guide/hands-on/workflow-walkthrough.md

@@ -817,11 +817,180 @@ michi jira:comment DEMO-103 "PRを作成しました: https://github.com/..."
 - [ ] Confluenceページが作成された（設計書）
 - [ ] `spec.json` に `confluence.designPageId` が記録された
 
-### Phase 0.3-0.4: テスト計画（このガイドではスキップ）
+### Phase 0.3-0.4: テスト計画
 
-- このハンズオンでは省略していますが、実際のプロジェクトでは：
-- [ ] テストタイプを選択（Phase 0.3）
-- [ ] テスト仕様書を作成（Phase 0.4）
+#### Phase 0.3: テストタイプの選択
+
+health-check-endpointの場合、以下のテストタイプを選択します：
+
+**判断基準**:
+
+| テストタイプ | 必要性 | 判断理由 |
+|-------------|--------|----------|
+| **単体テスト** | ✅ 必須 | HealthControllerとHealthServiceのロジックを検証 |
+| **統合テスト** | ✅ 推奨 | API全体の動作とレスポンス形式を検証 |
+| E2Eテスト | ❌ 不要 | 単一エンドポイントのみで複雑なフローなし |
+| パフォーマンステスト | ⚠️ 任意 | ヘルスチェックは高頻度で呼ばれるため検討可能 |
+| セキュリティテスト | ❌ 不要 | 認証不要の公開エンドポイント |
+
+**結論**: health-check-endpointでは**単体テスト**と**統合テスト**を実施します。
+
+#### Phase 0.4: テスト仕様書の作成
+
+テストタイプごとに仕様書を作成します。
+
+##### 単体テスト仕様書
+
+**作成場所**: `tests/specs/unit-test-spec.md`
+
+**期待される内容（例）**:
+
+```markdown
+# health-check-endpoint 単体テスト仕様
+
+## テスト対象
+
+- HealthController
+- HealthService
+
+## HealthControllerTest
+
+### TC-U-001: 正常系 - ステータス取得
+
+**目的**: HealthControllerが正常にヘルスチェック情報を返す
+
+**前提条件**:
+- HealthServiceがモック化されている
+- HealthService.getStatus()が`{status: "ok", timestamp: "2025-01-15T10:00:00Z"}`を返す
+
+**実行手順**:
+1. GET /health エンドポイントを呼び出す
+
+**期待結果**:
+- HTTPステータス: 200
+- レスポンスボディ: `{"status": "ok", "timestamp": "2025-01-15T10:00:00Z"}`
+
+### TC-U-002: 異常系 - サービスエラー
+
+**目的**: サービス層でエラーが発生した場合の処理
+
+**前提条件**:
+- HealthService.getStatus()が例外をスローする
+
+**実行手順**:
+1. GET /health エンドポイントを呼び出す
+
+**期待結果**:
+- HTTPステータス: 503
+- レスポンスボディ: `{"status": "error", "message": "..."}`
+
+## HealthServiceTest
+
+### TC-U-101: システムステータス正常
+
+**目的**: HealthServiceがシステムの正常状態を正しく判定
+
+**前提条件**:
+- すべての依存サービスが正常
+
+**実行手順**:
+1. HealthService.getStatus()を呼び出す
+
+**期待結果**:
+- status: "ok"
+- timestamp: 現在時刻のISO 8601形式
+```
+
+##### 統合テスト仕様書
+
+**作成場所**: `tests/specs/integration-test-spec.md`
+
+**期待される内容（例）**:
+
+```markdown
+# health-check-endpoint 統合テスト仕様
+
+## テスト対象
+
+- Health API全体（Controller → Service → Repository）
+
+## TC-I-001: 正常系 - エンドツーエンド
+
+**目的**: Health APIが期待通りに動作する
+
+**前提条件**:
+- テストサーバーが起動している
+- データベース接続が有効
+
+**実行手順**:
+1. GET http://localhost:8080/health をHTTPクライアントで呼び出す
+
+**期待結果**:
+- HTTPステータス: 200
+- Content-Type: application/json
+- レスポンスボディ:
+  ```json
+  {
+    "status": "ok",
+    "timestamp": "<ISO 8601形式>",
+    "version": "1.0.0"
+  }
+  ```
+
+## TC-I-002: 異常系 - データベース切断
+
+**目的**: DB接続エラー時の503レスポンス
+
+**前提条件**:
+- テストサーバーが起動している
+- データベースが停止している
+
+**実行手順**:
+1. GET http://localhost:8080/health を呼び出す
+
+**期待結果**:
+- HTTPステータス: 503
+- Content-Type: application/json
+- レスポンスボディ:
+  ```json
+  {
+    "status": "error",
+    "message": "Database connection failed"
+  }
+  ```
+```
+
+#### ディレクトリ構造の準備
+
+テスト仕様書を配置するディレクトリを作成します：
+
+```bash
+mkdir -p tests/specs
+```
+
+**確認**:
+
+```bash
+ls -la tests/specs/
+```
+
+**期待されるファイル**:
+
+```
+tests/specs/
+├── unit-test-spec.md
+└── integration-test-spec.md
+```
+
+#### 検証チェックリスト
+
+テスト計画が完了したら、以下を確認してください：
+
+- [ ] Phase 0.3: 必要なテストタイプを選択した（単体テスト + 統合テスト）
+- [ ] Phase 0.4: 単体テスト仕様書を作成した（`tests/specs/unit-test-spec.md`）
+- [ ] Phase 0.4: 統合テスト仕様書を作成した（`tests/specs/integration-test-spec.md`）
+- [ ] テストケースにID（TC-U-XXX、TC-I-XXX）を付与した
+- [ ] 各テストケースに目的・前提条件・実行手順・期待結果を記載した
 
 ### Phase 0.5-0.6: タスク分割とJIRA同期
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sk8metal/michi-cli",
-  "version": "0.8.6",
+  "version": "0.10.1",
   "description": "Managed Intelligent Comprehensive Hub for Integration - AI-driven development workflow automation",
   "license": "MIT",
   "type": "module",

package/scripts/__tests__/multi-repo-config-schema.test.ts

@@ -37,6 +37,50 @@ describe('RepositorySchema', () => {
         expect(result.data.branch).toBe('main');
       }
     });
+
+    it('localPathが未指定の場合も受け入れる（オプショナル）', () => {
+      const repoWithoutLocalPath = {
+        name: 'my-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+      };
+
+      const result = RepositorySchema.safeParse(repoWithoutLocalPath);
+      expect(result.success).toBe(true);
+      if (result.success) {
+        expect(result.data.localPath).toBeUndefined();
+      }
+    });
+
+    it('Unix絶対パス（/path/to/repo）を受け入れる', () => {
+      const repoWithUnixPath = {
+        name: 'my-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: '/Users/user/repos/my-repo',
+      };
+
+      const result = RepositorySchema.safeParse(repoWithUnixPath);
+      expect(result.success).toBe(true);
+      if (result.success) {
+        expect(result.data.localPath).toBe('/Users/user/repos/my-repo');
+      }
+    });
+
+    it('Windows絶対パス（C:\\path\\to\\repo）を受け入れる', () => {
+      const repoWithWindowsPath = {
+        name: 'my-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: 'C:\\Users\\user\\repos\\my-repo',
+      };
+
+      const result = RepositorySchema.safeParse(repoWithWindowsPath);
+      expect(result.success).toBe(true);
+      if (result.success) {
+        expect(result.data.localPath).toBe('C:\\Users\\user\\repos\\my-repo');
+      }
+    });
   });
 
   describe('異常ケース', () => {
@@ -73,6 +117,68 @@ describe('RepositorySchema', () => {
       expect(result.success).toBe(false);
     });
   });
+
+  describe('localPathバリデーション', () => {
+    it('相対パス（./repo）を含む場合はエラー', () => {
+      const invalidRepo = {
+        name: 'my-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: './repos/my-repo',
+      };
+
+      const result = RepositorySchema.safeParse(invalidRepo);
+      expect(result.success).toBe(false);
+    });
+
+    it('相対パス（../repo）を含む場合はエラー', () => {
+      const invalidRepo = {
+        name: 'my-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: '../repos/my-repo',
+      };
+
+      const result = RepositorySchema.safeParse(invalidRepo);
+      expect(result.success).toBe(false);
+    });
+
+    it('Windowsの相対パス（.\\repo）を含む場合はエラー', () => {
+      const invalidRepo = {
+        name: 'my-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: '.\\repos\\my-repo',
+      };
+
+      const result = RepositorySchema.safeParse(invalidRepo);
+      expect(result.success).toBe(false);
+    });
+
+    it('空文字列の場合はエラー', () => {
+      const invalidRepo = {
+        name: 'my-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: '',
+      };
+
+      const result = RepositorySchema.safeParse(invalidRepo);
+      expect(result.success).toBe(false);
+    });
+
+    it('相対パス（repo/subdir）を含む場合はエラー', () => {
+      const invalidRepo = {
+        name: 'my-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: 'repos/my-repo',
+      };
+
+      const result = RepositorySchema.safeParse(invalidRepo);
+      expect(result.success).toBe(false);
+    });
+  });
 });
 
 describe('MultiRepoProjectSchema', () => {

package/scripts/__tests__/multi-repo-validator.test.ts

@@ -3,12 +3,17 @@
  * Task 11.1: バリデーション関数の単体テスト
  */
 
-import { describe, it, expect } from 'vitest';
+import { describe, it, expect, beforeEach, afterEach } from 'vitest';
+import * as fs from 'fs';
+import * as path from 'path';
+import { execSync } from 'child_process';
 import {
   validateProjectName,
   validateJiraKey,
   validateRepositoryUrl,
+  validateLocalPath,
 } from '../utils/multi-repo-validator.js';
+import type { Repository } from '../config/config-schema.js';
 
 describe('validateProjectName', () => {
   describe('正常ケース', () => {
@@ -522,3 +527,226 @@ describe('validateRepositoryUrl', () => {
     });
   });
 });
+
+describe('validateLocalPath', () => {
+  let tempDir: string;
+
+  beforeEach(() => {
+    // テスト用の一時ディレクトリを作成
+    tempDir = fs.mkdtempSync(path.join(process.cwd(), 'test-temp-'));
+  });
+
+  afterEach(() => {
+    // テスト用の一時ディレクトリを削除
+    if (fs.existsSync(tempDir)) {
+      fs.rmSync(tempDir, { recursive: true, force: true });
+    }
+  });
+
+  describe('localPath未設定ケース', () => {
+    it('localPathが未設定の場合、警告を返す', () => {
+      const repository: Repository = {
+        name: 'test-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        // localPath: undefined (省略)
+      };
+
+      const result = validateLocalPath(repository);
+
+      expect(result.isValid).toBe(false);
+      expect(result.errors).toHaveLength(0);
+      expect(result.warnings).toHaveLength(1);
+      expect(result.warnings[0]).toContain(
+        "Repository 'test-repo' does not have localPath configured",
+      );
+      expect(result.exists).toBe(false);
+      expect(result.isGitRepository).toBe(false);
+      expect(result.currentBranch).toBeNull();
+      expect(result.branchMatches).toBe(false);
+      expect(result.hasUncommittedChanges).toBe(false);
+    });
+  });
+
+  describe('ディレクトリ存在確認', () => {
+    it('localPathが存在しない場合、エラーを返す', () => {
+      const nonExistentPath = path.join(tempDir, 'non-existent');
+      const repository: Repository = {
+        name: 'test-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: nonExistentPath,
+      };
+
+      const result = validateLocalPath(repository);
+
+      expect(result.isValid).toBe(false);
+      expect(result.errors).toHaveLength(1);
+      expect(result.errors[0]).toContain('does not exist');
+      expect(result.exists).toBe(false);
+    });
+
+    it('localPathがファイルの場合、エラーを返す', () => {
+      const filePath = path.join(tempDir, 'test-file.txt');
+      fs.writeFileSync(filePath, 'test content');
+
+      const repository: Repository = {
+        name: 'test-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: filePath,
+      };
+
+      const result = validateLocalPath(repository);
+
+      expect(result.isValid).toBe(false);
+      expect(result.errors).toHaveLength(1);
+      expect(result.errors[0]).toContain('is not a directory');
+      expect(result.exists).toBe(true);
+      expect(result.isGitRepository).toBe(false);
+    });
+  });
+
+  describe('Gitリポジトリ確認', () => {
+    it('Gitリポジトリではないディレクトリの場合、エラーを返す', () => {
+      const nonGitDir = path.join(tempDir, 'non-git-repo');
+      fs.mkdirSync(nonGitDir);
+
+      const repository: Repository = {
+        name: 'test-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: nonGitDir,
+      };
+
+      const result = validateLocalPath(repository);
+
+      expect(result.isValid).toBe(false);
+      expect(result.errors).toHaveLength(1);
+      expect(result.errors[0]).toContain('is not a Git repository');
+      expect(result.exists).toBe(true);
+      expect(result.isGitRepository).toBe(false);
+    });
+
+    it('Gitリポジトリの場合、基本的なチェックを通過する', () => {
+      const gitDir = path.join(tempDir, 'git-repo');
+      fs.mkdirSync(gitDir);
+
+      // Gitリポジトリを初期化
+      execSync('git init', { cwd: gitDir });
+      execSync('git config user.email "test@example.com"', { cwd: gitDir });
+      execSync('git config user.name "Test User"', { cwd: gitDir });
+
+      // 初回コミットを作成（ブランチが作成されるため）
+      fs.writeFileSync(path.join(gitDir, 'README.md'), '# Test');
+      execSync('git add .', { cwd: gitDir });
+      execSync('git commit -m "Initial commit"', { cwd: gitDir });
+
+      // mainブランチに切り替え（git initでデフォルトブランチ名が異なる可能性があるため）
+      try {
+        execSync('git branch -M main', { cwd: gitDir });
+      } catch {
+        // すでにmainの場合は無視
+      }
+
+      const repository: Repository = {
+        name: 'test-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: gitDir,
+      };
+
+      const result = validateLocalPath(repository);
+
+      expect(result.isValid).toBe(true);
+      expect(result.errors).toHaveLength(0);
+      expect(result.exists).toBe(true);
+      expect(result.isGitRepository).toBe(true);
+      expect(result.currentBranch).toBe('main');
+      expect(result.branchMatches).toBe(true);
+      expect(result.hasUncommittedChanges).toBe(false);
+    });
+
+    it('ブランチが一致しない場合、警告を返す', () => {
+      const gitDir = path.join(tempDir, 'git-repo-branch-mismatch');
+      fs.mkdirSync(gitDir);
+
+      // Gitリポジトリを初期化
+      execSync('git init', { cwd: gitDir });
+      execSync('git config user.email "test@example.com"', { cwd: gitDir });
+      execSync('git config user.name "Test User"', { cwd: gitDir });
+
+      // 初回コミットを作成
+      fs.writeFileSync(path.join(gitDir, 'README.md'), '# Test');
+      execSync('git add .', { cwd: gitDir });
+      execSync('git commit -m "Initial commit"', { cwd: gitDir });
+
+      // developブランチに切り替え
+      execSync('git checkout -b develop', { cwd: gitDir });
+
+      const repository: Repository = {
+        name: 'test-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main', // 実際はdevelopブランチにいる
+        localPath: gitDir,
+      };
+
+      const result = validateLocalPath(repository);
+
+      expect(result.isValid).toBe(true); // 警告のみでエラーではない
+      expect(result.errors).toHaveLength(0);
+      expect(result.warnings.length).toBeGreaterThan(0);
+      expect(result.warnings.some((w) => w.includes('does not match'))).toBe(
+        true,
+      );
+      expect(result.exists).toBe(true);
+      expect(result.isGitRepository).toBe(true);
+      expect(result.currentBranch).toBe('develop');
+      expect(result.branchMatches).toBe(false);
+    });
+
+    it('未コミット変更がある場合、警告を返す', () => {
+      const gitDir = path.join(tempDir, 'git-repo-uncommitted');
+      fs.mkdirSync(gitDir);
+
+      // Gitリポジトリを初期化
+      execSync('git init', { cwd: gitDir });
+      execSync('git config user.email "test@example.com"', { cwd: gitDir });
+      execSync('git config user.name "Test User"', { cwd: gitDir });
+
+      // 初回コミットを作成
+      fs.writeFileSync(path.join(gitDir, 'README.md'), '# Test');
+      execSync('git add .', { cwd: gitDir });
+      execSync('git commit -m "Initial commit"', { cwd: gitDir });
+
+      // mainブランチに切り替え
+      try {
+        execSync('git branch -M main', { cwd: gitDir });
+      } catch {
+        // すでにmainの場合は無視
+      }
+
+      // 未コミットの変更を追加
+      fs.writeFileSync(path.join(gitDir, 'new-file.txt'), 'uncommitted');
+
+      const repository: Repository = {
+        name: 'test-repo',
+        url: 'https://github.com/owner/repo',
+        branch: 'main',
+        localPath: gitDir,
+      };
+
+      const result = validateLocalPath(repository);
+
+      expect(result.isValid).toBe(true); // 警告のみでエラーではない
+      expect(result.errors).toHaveLength(0);
+      expect(result.warnings.length).toBeGreaterThan(0);
+      expect(
+        result.warnings.some((w) => w.includes('uncommitted changes')),
+      ).toBe(true);
+      expect(result.exists).toBe(true);
+      expect(result.isGitRepository).toBe(true);
+      expect(result.hasUncommittedChanges).toBe(true);
+    });
+  });
+});

package/scripts/config/config-schema.ts

@@ -183,6 +183,26 @@ export const RepositorySchema = z.object({
       message: 'GitHub URL must be in format: https://github.com/{owner}/{repo}',
     }),
   branch: z.string().default('main'),
+  localPath: z
+    .string()
+    .optional()
+    .refine(
+      (path) => {
+        // localPath が指定されていない場合は検証スキップ
+        if (path === undefined) return true;
+        // 空文字列はエラー
+        if (path === '') return false;
+        // 絶対パスであることを検証（セキュリティ考慮）
+        // Unix系: '/' で始まる、Windows系: 'C:\' 等で始まる
+        const isUnixAbsolutePath = path.startsWith('/');
+        const isWindowsAbsolutePath = /^[A-Za-z]:\\/.test(path);
+        return isUnixAbsolutePath || isWindowsAbsolutePath;
+      },
+      {
+        message:
+          'localPath must be an absolute path (Unix: /path/to/repo, Windows: C:\\path\\to\\repo)',
+      },
+    ),
 });
 
 /**

package/scripts/constants/__tests__/environments.test.ts

@@ -22,21 +22,21 @@ describe('environments', () => {
     it('should have correct structure for claude', () => {
       const config = ENV_CONFIG.claude;
       expect(config.rulesDir).toBe('.claude/rules');
-      expect(config.commandsDir).toBe('.claude/commands/kiro');
+      expect(config.commandsDir).toBe('.claude/commands');
       expect(config.templateSource).toBe('claude');
     });
 
     it('should have correct structure for claude-agent', () => {
       const config = ENV_CONFIG['claude-agent'];
       expect(config.rulesDir).toBe('.claude/agents');
-      expect(config.commandsDir).toBe('.claude/commands/kiro');
+      expect(config.commandsDir).toBe('.claude/commands');
       expect(config.templateSource).toBe('claude-agent');
     });
 
     it('should have correct structure for cursor', () => {
       const config = ENV_CONFIG.cursor;
       expect(config.rulesDir).toBe('.cursor/rules');
-      expect(config.commandsDir).toBe('.cursor/commands/kiro');
+      expect(config.commandsDir).toBe('.cursor/commands');
       expect(config.templateSource).toBe('cursor');
     });
 

package/scripts/constants/environments.ts

@@ -21,17 +21,17 @@ export type Environment =
 export const ENV_CONFIG: Record<Environment, EnvironmentConfig> = {
   claude: {
     rulesDir: '.claude/rules',
-    commandsDir: '.claude/commands/kiro',
+    commandsDir: '.claude/commands',
     templateSource: 'claude',
   },
   'claude-agent': {
     rulesDir: '.claude/agents',
-    commandsDir: '.claude/commands/kiro',
+    commandsDir: '.claude/commands',
     templateSource: 'claude-agent',
   },
   cursor: {
     rulesDir: '.cursor/rules',
-    commandsDir: '.cursor/commands/kiro',
+    commandsDir: '.cursor/commands',
     templateSource: 'cursor',
   },
   gemini: {