@sk8metal/michi-cli 0.8.1 → 0.8.2

package/docs/design-issue-55.md

@@ -1,240 +0,0 @@
-# Issue #55: バリデーションエラーハンドリング修正 - 詳細設計
-
-## 問題の根本原因
-
-### 現在の実装（src/commands/setup-existing.ts）
-
-```typescript
-// buildConfig関数内（Line 167-172）
-try {
-  projectName = validateProjectName(projectName);
-} catch (error) {
-  throw new Error(`プロジェクト名が不正です: ${error instanceof Error ? error.message : error}`);
-}
-
-// buildConfig関数内（Line 180-185）
-try {
-  jiraKey = validateJiraKey(jiraKey);
-} catch (error) {
-  throw new Error(`JIRAキーが不正です: ${error instanceof Error ? error.message : error}`);
-}
-```
-
-### テストの期待値
-
-```typescript
-// validation.test.ts（Line 214）
-.rejects.toThrow(/プロジェクト名が空です/);
-
-// validation.test.ts（Line 225）
-.rejects.toThrow(/パス区切り文字/);
-```
-
-### 問題
-
-**実際のエラーメッセージ**: `プロジェクト名が不正です: プロジェクト名が空です`  
-**期待されるメッセージ**: `プロジェクト名が空です`
-
-→ **ラッピングによってマッチしない**
-
----
-
-## 修正方針
-
-### Option 1: エラーをラッピングせずそのままスロー（推奨）
-
-**メリット**:
-- テストの期待値と一致する
-- エラーメッセージがシンプル
-- エラーの原因が直接的に伝わる
-
-**実装**:
-```typescript
-// buildConfig関数内（Line 167-172）
-try {
-  projectName = validateProjectName(projectName);
-} catch (error) {
-  // ラッピングせず、そのままスロー
-  throw error;
-}
-
-// buildConfig関数内（Line 180-185）
-try {
-  jiraKey = validateJiraKey(jiraKey);
-} catch (error) {
-  // ラッピングせず、そのままスロー
-  throw error;
-}
-```
-
-### Option 2: テストの期待値を修正（非推奨）
-
-**デメリット**:
-- エラーメッセージが冗長になる
-- ラッピングの必要性が不明瞭
-
-**実装**:
-```typescript
-// テストを修正
-.rejects.toThrow(/プロジェクト名が不正です.*プロジェクト名が空です/);
-```
-
----
-
-## 採用する修正方針
-
-**Option 1を採用**
-
-理由:
-1. エラーメッセージがシンプルで分かりやすい
-2. `validateProjectName`や`validateJiraKey`が既に適切なエラーメッセージを返している
-3. 二重ラッピングの必要性がない
-4. テストの期待値が自然
-
----
-
-## 修正対象ファイル
-
-### 1. src/commands/setup-existing.ts
-
-**修正箇所**: `buildConfig`関数内のエラーハンドリング
-
-```typescript
-// Line 167-172: プロジェクト名バリデーション
-try {
-  projectName = validateProjectName(projectName);
-} catch (error) {
-  throw error; // 修正: ラッピングを削除
-}
-
-// Line 180-185: JIRAキーバリデーション
-try {
-  jiraKey = validateJiraKey(jiraKey);
-} catch (error) {
-  throw error; // 修正: ラッピングを削除
-}
-```
-
-### 2. src/__tests__/integration/setup/validation.test.ts
-
-**修正箇所**: スキップされた5つのテストの`.skip`を削除
-
-```typescript
-// Line 145
-it('should reject unsupported language', async () => { ... });
-
-// Line 207
-it('should reject empty project name', async () => { ... });
-
-// Line 218
-it('should reject project name with path traversal characters', async () => { ... });
-
-// Line 229
-it('should reject project name with backslash', async () => { ... });
-
-// Line 240
-it('should reject project name with control characters', async () => { ... });
-```
-
----
-
-## 影響範囲
-
-### 変更による影響
-
-1. **エラーメッセージの変更**:
-   - 修正前: `プロジェクト名が不正です: <詳細メッセージ>`
-   - 修正後: `<詳細メッセージ>`（例: `プロジェクト名が空です`）
-
-2. **ユーザー体験**:
-   - より直接的でシンプルなエラーメッセージ
-   - エラーの原因が明確になる
-
-3. **他のテストへの影響**:
-   - 既存の成功テスト（プロジェクト名とJIRAキーの正常系）には影響なし
-   - エラーテストのうち、以下は既に成功している（ラッピングされていないため）:
-     - Line 250: `should reject project name that is too long`
-     - Line 328-366: JIRAキーバリデーションのエラーテスト
-
-### 回帰テストの必要性
-
-以下のテストが引き続き成功することを確認:
-- [ ] Line 250: `should reject project name that is too long`
-- [ ] Line 328: `should reject JIRA key with 1 character`
-- [ ] Line 338: `should reject JIRA key with 11 characters`
-- [ ] Line 348: `should reject JIRA key with numbers`
-- [ ] Line 358: `should reject JIRA key with special characters`
-
----
-
-## テスト実行計画
-
-### Step 1: 修正前のテスト実行（失敗確認）
-
-```bash
-cd /Users/arigatatsuya/Work/git/michi
-npm test -- validation.test.ts
-```
-
-**期待結果**: 5つのテストがスキップされていることを確認
-
-### Step 2: 実装修正
-
-1. `src/commands/setup-existing.ts`のエラーハンドリングを修正
-2. まだ`.skip`は残す
-
-### Step 3: テスト実行（成功確認）
-
-```bash
-npm test -- validation.test.ts
-```
-
-**期待結果**: 修正後も既存の成功テストが全て成功
-
-### Step 4: `.skip`削除
-
-`src/__tests__/integration/setup/validation.test.ts`の5つのテストから`.skip`を削除
-
-### Step 5: 最終テスト実行
-
-```bash
-npm test -- validation.test.ts
-```
-
-**期待結果**: 全テスト（5つの新規有効化テストを含む）が成功
-
-### Step 6: 全テストスイート実行
-
-```bash
-npm test
-```
-
-**期待結果**: 全プロジェクトのテストが成功
-
----
-
-## チェックリスト
-
-### 実装
-- [ ] `src/commands/setup-existing.ts` Line 167-172 修正
-- [ ] `src/commands/setup-existing.ts` Line 180-185 修正
-
-### テスト
-- [ ] `validation.test.ts` Line 145 `.skip`削除
-- [ ] `validation.test.ts` Line 207 `.skip`削除
-- [ ] `validation.test.ts` Line 218 `.skip`削除
-- [ ] `validation.test.ts` Line 229 `.skip`削除
-- [ ] `validation.test.ts` Line 240 `.skip`削除
-
-### 検証
-- [ ] 修正前テスト実行（失敗/スキップ確認）
-- [ ] 実装修正
-- [ ] テスト実行（既存テスト成功確認）
-- [ ] `.skip`削除
-- [ ] テスト実行（新規テスト成功確認）
-- [ ] 全テストスイート実行（全テスト成功確認）
-
----
-
-最終更新: 2025-11-17 (月)
-

package/docs/design-issue-56.md

@@ -1,181 +0,0 @@
-# Issue #56: Claude-agentテンプレート構造修正 - 詳細設計
-
-## 問題の根本原因
-
-### 現在の状態
-
-```
-templates/claude-agent/
-└── README.md  # これしかない
-```
-
-### 環境設定（scripts/constants/environments.ts）
-
-```typescript
-'claude-agent': {
-  rulesDir: '.claude/rules',  // ← 問題: テストは .claude/subagents を期待
-  commandsDir: '.claude/commands/kiro',
-  templateSource: 'claude-agent'
-}
-```
-
-### テストの期待値
-
-1. `.claude/subagents/`ディレクトリが作成される
-2. `.claude/rules`ディレクトリは作成されない（Subagents環境では不要）
-3. `.claude/commands/kiro/`ディレクトリが作成される
-
----
-
-## 修正方針
-
-### 1. 環境設定の修正
-
-**ファイル**: `scripts/constants/environments.ts`
-
-```typescript
-'claude-agent': {
-  rulesDir: '.claude/subagents',  // 修正: rules → subagents
-  commandsDir: '.claude/commands/kiro',
-  templateSource: 'claude-agent'
-}
-```
-
-**影響範囲**:
-- `setup-existing.ts`が`rulesDir`を使用してディレクトリを作成
-- `claude-agent`環境では`.claude/subagents`が作成される
-
-### 2. テンプレートディレクトリ構造の作成
-
-**目標構造**:
-
-```
-templates/claude-agent/
-├── README.md
-├── subagents/
-│   ├── .gitkeep  # 空ディレクトリを保持
-│   └── (将来的にサブエージェント定義ファイルを追加)
-└── commands/
-    └── kiro/
-        ├── .gitkeep  # 空ディレクトリを保持
-        └── (将来的にコマンド定義ファイルを追加)
-```
-
-**理由**:
-- Issue #56の主目的は「テストを有効化すること」
-- サブエージェントやコマンドの具体的な内容は別Issueで対応
-- 最低限の構造を作成し、テストを成功させる
-
-### 3. テスト有効化
-
-**対象ファイル**:
-1. `src/__tests__/integration/setup/validation.test.ts` (1個)
-2. `src/__tests__/integration/setup/claude-agent.test.ts` (3個)
-
-**修正箇所**:
-- Line 78 (validation.test.ts): `should accept claude-agent environment flag`
-- Line 43 (claude-agent.test.ts): `should create .claude/subagents directory`
-- Line 99 (claude-agent.test.ts): `should have subagents directory structure`
-- Line 125 (claude-agent.test.ts): `should not create basic Claude rules directory`
-
----
-
-## 実装手順
-
-### Step 1: 環境設定の修正（5分）
-
-```bash
-# ファイル: scripts/constants/environments.ts
-# Line 22を修正
-rulesDir: '.claude/subagents',
-```
-
-### Step 2: テンプレートディレクトリ作成（10分）
-
-```bash
-cd /Users/arigatatsuya/Work/git/michi
-
-# subagentsディレクトリ作成
-mkdir -p templates/claude-agent/subagents
-touch templates/claude-agent/subagents/.gitkeep
-
-# commandsディレクトリ作成
-mkdir -p templates/claude-agent/commands/kiro
-touch templates/claude-agent/commands/kiro/.gitkeep
-```
-
-### Step 3: テスト有効化（5分）
-
-4つのテストから`.skip`を削除
-
-### Step 4: テスト実行（5分）
-
-```bash
-npm test -- claude-agent.test.ts
-npm test -- validation.test.ts
-```
-
----
-
-## テストの期待結果
-
-### validation.test.ts
-
-```
-✓ should accept claude-agent environment flag
-```
-
-### claude-agent.test.ts
-
-```
-✓ should create .claude/subagents directory
-✓ should have subagents directory structure
-✓ should not create basic Claude rules directory
-```
-
----
-
-## チェックリスト
-
-### 実装
-- [ ] `scripts/constants/environments.ts` Line 22 修正
-- [ ] `templates/claude-agent/subagents/.gitkeep` 作成
-- [ ] `templates/claude-agent/commands/kiro/.gitkeep` 作成
-
-### テスト
-- [ ] `validation.test.ts` Line 78 `.skip`削除
-- [ ] `claude-agent.test.ts` Line 43 `.skip`削除
-- [ ] `claude-agent.test.ts` Line 99 `.skip`削除
-- [ ] `claude-agent.test.ts` Line 125 `.skip`削除
-
-### 検証
-- [ ] `claude-agent.test.ts`実行（4つのテスト成功）
-- [ ] `validation.test.ts`実行（1つのテスト成功）
-- [ ] 全テストスイート実行（全テスト成功）
-
----
-
-## 将来的な拡張（別Issue）
-
-### Subagentsファイルの追加
-
-`templates/claude-agent/subagents/`に以下を追加:
-- `manager-pj.md`
-- `manager-agent.md`
-- `developer.md`
-- `test-developer.md`
-- `design-expert.md`
-- `review-cq.md`
-
-### Commandsファイルの追加
-
-`templates/claude-agent/commands/kiro/`に以下を追加:
-- `spec-init.md`
-- `spec-requirements.md`
-- `spec-design.md`
-- `spec-tasks.md`
-
----
-
-最終更新: 2025-11-17 (月)
-