@sk8metal/michi-cli 0.3.0 → 0.4.0

package/scripts/utils/interactive-helpers.ts

@@ -0,0 +1,135 @@
+/**
+ * 対話的設定の共通ヘルパー関数
+ */
+
+import * as readline from 'readline';
+
+/**
+ * readlineインターフェースを作成
+ */
+export function createInterface(): readline.Interface {
+  return readline.createInterface({
+    input: process.stdin,
+    output: process.stdout,
+  });
+}
+
+/**
+ * 質問を表示して回答を取得
+ */
+export function question(rl: readline.Interface, query: string): Promise<string> {
+  return new Promise((resolve) => {
+    rl.question(query, (answer) => {
+      resolve(answer.trim());
+    });
+  });
+}
+
+/**
+ * 選択肢から選択
+ */
+export async function select(
+  rl: readline.Interface,
+  prompt: string,
+  choices: Array<{ value: string; label: string; description?: string }>,
+  defaultValue?: string,
+  maxRetries: number = 3,
+): Promise<string> {
+  if (choices.length === 0) {
+    throw new Error('select: choices must contain at least one option');
+  }
+
+  console.log(`\n${prompt}`);
+  choices.forEach((choice, index) => {
+    const defaultMark = defaultValue === choice.value ? ' (デフォルト)' : '';
+    const desc = choice.description ? ` - ${choice.description}` : '';
+    console.log(`  ${index + 1}. ${choice.label}${desc}${defaultMark}`);
+  });
+
+  const answer = await question(
+    rl,
+    `\n選択してください [1-${choices.length}]: `,
+  );
+
+  if (!answer && defaultValue) {
+    return defaultValue;
+  }
+
+  const index = parseInt(answer, 10) - 1;
+  if (index >= 0 && index < choices.length) {
+    return choices[index].value;
+  }
+
+  if (defaultValue) {
+    return defaultValue;
+  }
+
+  // 無効な入力の場合は再試行（最大試行回数まで）
+  if (maxRetries > 0) {
+    console.log(
+      `⚠️  無効な選択です。もう一度入力してください（残り試行回数: ${maxRetries}）。`,
+    );
+    return select(rl, prompt, choices, defaultValue, maxRetries - 1);
+  }
+
+  // 最大試行回数に達した場合はデフォルト値または最初の選択肢を返す
+  // 注: choices.length > 0 は関数冒頭で保証されている
+  const fallbackValue = defaultValue || choices[0].value;
+  console.log(
+    `⚠️  最大試行回数に達しました。デフォルト値を使用します: ${fallbackValue}`,
+  );
+  return fallbackValue;
+}
+
+/**
+ * Yes/No質問
+ */
+export async function confirm(
+  rl: readline.Interface,
+  prompt: string,
+  defaultValue: boolean = true,
+): Promise<boolean> {
+  const defaultText = defaultValue ? '[Y/n]' : '[y/N]';
+  const answer = await question(rl, `${prompt} ${defaultText}: `);
+
+  if (!answer) {
+    return defaultValue;
+  }
+
+  return answer.toLowerCase() === 'y' || answer.toLowerCase() === 'yes';
+}
+
+/**
+ * 複数選択
+ */
+export async function multiSelect(
+  rl: readline.Interface,
+  prompt: string,
+  choices: Array<{ value: string; label: string }>,
+  defaults: string[] = [],
+): Promise<string[]> {
+  console.log(`\n${prompt}`);
+  choices.forEach((choice, index) => {
+    const checked = defaults.includes(choice.value) ? '[x]' : '[ ]';
+    console.log(`  ${checked} ${index + 1}. ${choice.label}`);
+  });
+
+  const answer = await question(
+    rl,
+    '\n選択してください（カンマ区切り、例: 1,2,3）: ',
+  );
+
+  if (!answer && defaults.length > 0) {
+    return defaults;
+  }
+
+  if (!answer) {
+    return [];
+  }
+
+  const indices = answer
+    .split(',')
+    .map((s) => parseInt(s.trim(), 10) - 1)
+    .filter((i) => i >= 0 && i < choices.length);
+  return indices.map((i) => choices[i].value);
+}

package/scripts/utils/jira-issue-type-fetcher.ts

@@ -100,38 +100,46 @@ export async function getProjectIssueTypes(
     });
 
     const issueTypes = response.data.issueTypes || [];
-    
-    return issueTypes.map((it: any) => ({
+
+    return issueTypes.map((it: Partial<IssueTypeInfo> & { id: string; name: string }) => ({
       id: it.id,
       name: it.name,
       description: it.description || undefined,
       iconUrl: it.iconUrl || undefined,
       subtask: it.subtask || false
     }));
-  } catch (error: any) {
+  } catch (error: unknown) {
     // エラーをログに記録（デバッグ用）
-    if (error.response) {
-      // HTTPエラー（4xx, 5xx）
-      const status = error.response.status;
-      const statusText = error.response.statusText;
-      
-      if (status === 401) {
-        console.error('❌ JIRA認証に失敗しました。認証情報を確認してください。');
-      } else if (status === 403) {
-        console.error('❌ JIRAへのアクセス権限がありません。');
-      } else if (status === 404) {
-        console.error(`❌ JIRAプロジェクト "${projectKey}" が見つかりません。`);
+    // axios.isAxiosError()またはerror.response/requestの存在で判定
+    if (axios.isAxiosError(error) || (typeof error === 'object' && error !== null && ('response' in error || 'request' in error))) {
+      const axiosError = error as { response?: { status: number; statusText: string }; request?: unknown; message?: string };
+      if (axiosError.response) {
+        // HTTPエラー（4xx, 5xx）
+        const status = axiosError.response.status;
+        const statusText = axiosError.response.statusText;
+
+        if (status === 401) {
+          console.error('❌ JIRA認証に失敗しました。認証情報を確認してください。');
+        } else if (status === 403) {
+          console.error('❌ JIRAへのアクセス権限がありません。');
+        } else if (status === 404) {
+          console.error(`❌ JIRAプロジェクト "${projectKey}" が見つかりません。`);
+        } else {
+          console.error(`❌ JIRA APIエラー: ${status} ${statusText}`);
+        }
+      } else if (axiosError.request) {
+        // ネットワークエラー
+        console.error('❌ JIRA APIへの接続に失敗しました。ネットワークを確認してください。');
       } else {
-        console.error(`❌ JIRA APIエラー: ${status} ${statusText}`);
+        // その他のエラー
+        const message = axiosError.message || String(error);
+        console.error(`❌ エラー: ${message}`);
       }
-    } else if (error.request) {
-      // ネットワークエラー
-      console.error('❌ JIRA APIへの接続に失敗しました。ネットワークを確認してください。');
     } else {
-      // その他のエラー
-      console.error(`❌ エラー: ${error.message}`);
+      const message = error instanceof Error ? error.message : String(error);
+      console.error(`❌ エラー: ${message}`);
     }
-    
+
     return null;
   }
 }

package/scripts/utils/release-notes-generator.ts

@@ -101,8 +101,9 @@ export async function getCommits(
     }
 
     return commits;
-  } catch (error: any) {
-    console.error('Failed to get commits:', error.message);
+  } catch (error: unknown) {
+    const message = error instanceof Error ? error.message : String(error);
+    console.error('Failed to get commits:', message);
     return [];
   }
 }

package/scripts/utils/spec-updater.ts

@@ -29,12 +29,24 @@ export interface SpecJson {
       url?: string;
       title?: string;
     };
+    // 旧形式フィールド（後方互換性のため）
+    requirementsPageId?: string;
+    requirementsUrl?: string;
+    designPageId?: string;
+    designUrl?: string;
+    tasksPageId?: string;
+    tasksUrl?: string;
   };
   jira?: {
     projectKey?: string;
     epicKey?: string;
     epicUrl?: string;
     storyKeys?: string[];
+    // 旧形式フィールド（後方互換性のため）
+    stories?: {
+      created?: number;
+      total?: number;
+    };
   };
   environmentSetup?: {
     completed?: boolean;
@@ -48,6 +60,16 @@ export interface SpecJson {
     designCompleted?: boolean;
     tasksCompleted?: boolean;
     jiraSyncCompleted?: boolean;
+    // 旧形式フィールド（後方互換性のため）
+    requirements?: {
+      completed?: boolean;
+    };
+    design?: {
+      completed?: boolean;
+    };
+    tasks?: {
+      completed?: boolean;
+    };
   };
   lastUpdated?: string;
 }
@@ -144,14 +166,14 @@ export function updateSpecJsonAfterConfluenceSync(
 
   // 旧形式（後方互換性のため併記）
   if (docType === 'requirements') {
-    (spec.confluence as any).requirementsPageId = pageInfo.pageId;
-    (spec.confluence as any).requirementsUrl = pageInfo.url;
+    spec.confluence.requirementsPageId = pageInfo.pageId;
+    spec.confluence.requirementsUrl = pageInfo.url;
   } else if (docType === 'design') {
-    (spec.confluence as any).designPageId = pageInfo.pageId;
-    (spec.confluence as any).designUrl = pageInfo.url;
+    spec.confluence.designPageId = pageInfo.pageId;
+    spec.confluence.designUrl = pageInfo.url;
   } else if (docType === 'tasks') {
-    (spec.confluence as any).tasksPageId = pageInfo.pageId;
-    (spec.confluence as any).tasksUrl = pageInfo.url;
+    spec.confluence.tasksPageId = pageInfo.pageId;
+    spec.confluence.tasksUrl = pageInfo.url;
   }
 
   // マイルストーンを更新
@@ -163,24 +185,24 @@ export function updateSpecJsonAfterConfluenceSync(
   if (docType === 'requirements') {
     spec.milestones.requirementsCompleted = true;
     // 旧形式（後方互換性のため併記）
-    if (!(spec.milestones as any).requirements) {
-      (spec.milestones as any).requirements = {};
+    if (!spec.milestones.requirements) {
+      spec.milestones.requirements = {};
     }
-    (spec.milestones as any).requirements.completed = true;
+    spec.milestones.requirements.completed = true;
   } else if (docType === 'design') {
     spec.milestones.designCompleted = true;
     // 旧形式（後方互換性のため併記）
-    if (!(spec.milestones as any).design) {
-      (spec.milestones as any).design = {};
+    if (!spec.milestones.design) {
+      spec.milestones.design = {};
     }
-    (spec.milestones as any).design.completed = true;
+    spec.milestones.design.completed = true;
   } else if (docType === 'tasks') {
     spec.milestones.tasksCompleted = true;
     // 旧形式（後方互換性のため併記）
-    if (!(spec.milestones as any).tasks) {
-      (spec.milestones as any).tasks = {};
+    if (!spec.milestones.tasks) {
+      spec.milestones.tasks = {};
     }
-    (spec.milestones as any).tasks.completed = true;
+    spec.milestones.tasks.completed = true;
   }
 
   saveSpecJson(featureName, spec, projectRoot);

package/scripts/utils/tasks-converter.ts

@@ -4,8 +4,7 @@
  * AI-DLC形式のtasks.mdをMichiが期待するPhase構造に変換
  */
 
-import { writeFileSync, copyFileSync, existsSync } from 'fs';
-import { join, dirname } from 'path';
+import { writeFileSync, copyFileSync } from 'fs';
 import type {
   AIDLCDocument,
   AIDLCCategory,
@@ -20,7 +19,6 @@ import { readFileSync } from 'fs';
 import {
   getWeekdayNotation,
   getWeekdayRangeNotation,
-  getWeekNumber,
   ensureBusinessDay,
 } from './business-days.js';
 
@@ -77,8 +75,8 @@ interface PhaseMapping {
  */
 function mapCategoryToPhase(
   category: AIDLCCategory,
-  index: number,
-  totalCategories: number,
+  _index: number,
+  _totalCategories: number,
 ): PhaseMapping {
   const title = category.title.toLowerCase();
 
@@ -364,7 +362,7 @@ export function convertToMichiFormat(
     const group = phaseGroups.get(mapping.phaseId)!;
 
     // タスクをStoryに変換
-    category.tasks.forEach((task, taskIndex) => {
+    category.tasks.forEach(task => {
       const storyContent = convertTaskToStory(
         task,
         mapping.phaseId,

package/scripts/utils/tasks-format-validator.ts

@@ -41,19 +41,6 @@ export function validateTasksFormat(tasksPath: string): void {
 
   // 2. フェーズ構造の検証（新旧両方をサポート）
 
-  // 新ワークフロー構造（推奨）
-  const newWorkflowPhases = [
-    'Phase 0.1:', // 要件定義
-    'Phase 0.2:', // 設計
-    'Phase 1:',   // 環境構築（任意）
-    'Phase 2:',   // TDD実装
-    'Phase A:',   // PR前自動テスト（任意）
-    'Phase 3:',   // 追加QA（任意）
-    'Phase B:',   // リリース準備テスト（任意）
-    'Phase 4:',   // リリース準備
-    'Phase 5:',   // リリース
-  ];
-
   // 旧6-Phase構造（互換性のためサポート）
   const legacyPhases = [
     'Phase 0: 要件定義（Requirements）',

package/scripts/utils/test-runner.ts

@@ -53,15 +53,16 @@ export async function executeTests(
       duration,
       timestamp
     };
-  } catch (error: any) {
+  } catch (error: unknown) {
     const duration = (Date.now() - startTime) / 1000;
+    const errorObj = error as { stdout?: string; message?: string };
 
     return {
       success: false,
       language,
       command,
-      output: error.stdout || '',
-      error: error.message || String(error),
+      output: errorObj.stdout || '',
+      error: errorObj.message || String(error),
       duration,
       timestamp
     };

package/scripts/validate-phase.ts

@@ -7,12 +7,11 @@ import { existsSync, readFileSync } from 'fs';
 import { join } from 'path';
 import { validateFeatureName } from './utils/feature-name-validator.js';
 import { loadConfig } from './utils/config-loader.js';
+import { type SpecJson } from './utils/spec-updater.js';
 
 type Phase =
   | 'requirements'
   | 'design'
-  | 'test-type-selection'
-  | 'test-spec'
   | 'tasks'
   | 'environment-setup'
   | 'phase-a'
@@ -28,7 +27,7 @@ interface ValidationResult {
 /**
  * spec.jsonを読み込み
  */
-function loadSpecJson(feature: string): any {
+function loadSpecJson(feature: string): SpecJson {
   const specPath = join(process.cwd(), '.kiro', 'specs', feature, 'spec.json');
   
   if (!existsSync(specPath)) {
@@ -58,11 +57,12 @@ function validateRequirements(feature: string): ValidationResult {
   }
   
   // 2. spec.json読み込み
-  let spec: any;
+  let spec: SpecJson;
   try {
     spec = loadSpecJson(feature);
-  } catch (error: any) {
-    errors.push(`❌ spec.json読み込みエラー: ${error.message}`);
+  } catch (error: unknown) {
+    const message = error instanceof Error ? error.message : String(error);
+    errors.push(`❌ spec.json読み込みエラー: ${message}`);
     return { phase: 'requirements', valid: false, errors, warnings };
   }
   
@@ -110,11 +110,12 @@ function validateDesign(feature: string): ValidationResult {
   }
   
   // 2. spec.json読み込み
-  let spec: any;
+  let spec: SpecJson;
   try {
     spec = loadSpecJson(feature);
-  } catch (error: any) {
-    errors.push(`❌ spec.json読み込みエラー: ${error.message}`);
+  } catch (error: unknown) {
+    const message = error instanceof Error ? error.message : String(error);
+    errors.push(`❌ spec.json読み込みエラー: ${message}`);
     return { phase: 'design', valid: false, errors, warnings };
   }
   
@@ -191,11 +192,12 @@ function validateTasks(feature: string): ValidationResult {
   }
   
   // 2. spec.json読み込み
-  let spec: any;
+  let spec: SpecJson;
   try {
     spec = loadSpecJson(feature);
-  } catch (error: any) {
-    errors.push(`❌ spec.json読み込みエラー: ${error.message}`);
+  } catch (error: unknown) {
+    const message = error instanceof Error ? error.message : String(error);
+    errors.push(`❌ spec.json読み込みエラー: ${message}`);
     return { phase: 'tasks', valid: false, errors, warnings };
   }
   
@@ -214,7 +216,7 @@ function validateTasks(feature: string): ValidationResult {
   // spec.jira.storyKeys 配列をチェック（新フォーマット）
   // または spec.jira.stories.created（旧フォーマット）をチェック
   const hasStories = spec.jira?.storyKeys && Array.isArray(spec.jira.storyKeys) && spec.jira.storyKeys.length > 0;
-  const hasLegacyStories = spec.jira?.stories && spec.jira.stories.created > 0;
+  const hasLegacyStories = spec.jira?.stories && spec.jira.stories.created !== undefined && spec.jira.stories.created > 0;
   
   if (!hasStories && !hasLegacyStories) {
     errors.push('❌ JIRA Storyが1つも作成されていません');
@@ -222,7 +224,7 @@ function validateTasks(feature: string): ValidationResult {
   } else if (spec.jira?.storyKeys && spec.jira.storyKeys.length > 0) {
     // 新フォーマット: storyKeys配列が存在する場合
     // 成功として扱う（警告なし）
-  } else if (spec.jira?.stories && spec.jira.stories.created < spec.jira.stories.total) {
+  } else if (spec.jira?.stories && spec.jira.stories.created !== undefined && spec.jira.stories.total !== undefined && spec.jira.stories.created < spec.jira.stories.total) {
     // 旧フォーマット: 一部未作成の場合のみ警告
     warnings.push(`⚠️  JIRA Storyが一部未作成: ${spec.jira.stories.created}/${spec.jira.stories.total}`);
   }
@@ -240,59 +242,6 @@ function validateTasks(feature: string): ValidationResult {
   };
 }
 
-/**
- * テストタイプ選択フェーズのバリデーション（Phase 0.3）
- * マニュアル対応フェーズ - バリデーション不要（常に成功）
- */
-function validateTestTypeSelection(feature: string): ValidationResult {
-  const errors: string[] = [];
-  const warnings: string[] = [];
-
-  // feature名のバリデーション
-  const nameValidation = validateFeatureName(feature);
-  if (!nameValidation.valid) {
-    errors.push(...nameValidation.errors);
-  }
-
-  warnings.push('⚠️  このフェーズはマニュアル対応です。ガイダンスに従ってテストタイプを選択してください');
-
-  return {
-    phase: 'test-type-selection',
-    valid: errors.length === 0,
-    errors,
-    warnings
-  };
-}
-
-/**
- * テスト仕様書作成フェーズのバリデーション（Phase 0.4）
- * マニュアル対応フェーズ - バリデーション不要（常に成功）
- */
-function validateTestSpec(feature: string): ValidationResult {
-  const errors: string[] = [];
-  const warnings: string[] = [];
-
-  // feature名のバリデーション
-  const nameValidation = validateFeatureName(feature);
-  if (!nameValidation.valid) {
-    errors.push(...nameValidation.errors);
-  }
-
-  // テスト仕様書ディレクトリの存在チェック（任意）
-  const testSpecDir = join(process.cwd(), 'docs', 'testing', 'specs', feature);
-  if (!existsSync(testSpecDir)) {
-    warnings.push('⚠️  テスト仕様書ディレクトリがありません: docs/testing/specs/' + feature);
-  }
-
-  warnings.push('⚠️  このフェーズはマニュアル対応です。テンプレートを使用してテスト仕様書を作成してください');
-
-  return {
-    phase: 'test-spec',
-    valid: errors.length === 0,
-    errors,
-    warnings
-  };
-}
 
 /**
  * 環境構築フェーズのバリデーション（Phase 1）
@@ -381,12 +330,6 @@ export function validatePhase(feature: string, phase: Phase): ValidationResult {
   case 'design':
     result = validateDesign(feature);
     break;
-  case 'test-type-selection':
-    result = validateTestTypeSelection(feature);
-    break;
-  case 'test-spec':
-    result = validateTestSpec(feature);
-    break;
   case 'tasks':
     result = validateTasks(feature);
     break;
@@ -435,12 +378,11 @@ if (import.meta.url === `file://${process.argv[1]}`) {
     console.error('\nAvailable Phases:');
     console.error('  requirements       - Phase 0.1: 要件定義');
     console.error('  design             - Phase 0.2: 設計');
-    console.error('  test-type-selection- Phase 0.3: テストタイプ選択（任意）');
-    console.error('  test-spec          - Phase 0.4: テスト仕様書作成（任意）');
     console.error('  tasks              - Phase 0.5-0.6: タスク分割・JIRA同期');
     console.error('  environment-setup  - Phase 1: 環境構築（任意）');
     console.error('  phase-a            - Phase A: PR前自動テスト（任意）');
     console.error('  phase-b            - Phase B: リリース準備テスト（任意）');
+    console.error('\nNote: For test planning (Phase 0.3-0.4), use /michi:test-planning AI command');
     process.exit(1);
   }
 
@@ -449,8 +391,6 @@ if (import.meta.url === `file://${process.argv[1]}`) {
   const validPhases = [
     'requirements',
     'design',
-    'test-type-selection',
-    'test-spec',
     'tasks',
     'environment-setup',
     'phase-a',
@@ -459,15 +399,16 @@ if (import.meta.url === `file://${process.argv[1]}`) {
 
   if (!validPhases.includes(phase)) {
     console.error(`Invalid phase: ${phase}`);
-    console.error('Must be one of: requirements, design, test-type-selection, test-spec, tasks, environment-setup, phase-a, phase-b');
+    console.error('Must be one of: requirements, design, tasks, environment-setup, phase-a, phase-b');
     process.exit(1);
   }
   
   try {
     const result = validatePhase(feature, phase as Phase);
     process.exit(result.valid ? 0 : 1);
-  } catch (error: any) {
-    console.error(`\n❌ Validation error: ${error.message}`);
+  } catch (error: unknown) {
+    const message = error instanceof Error ? error.message : String(error);
+    console.error(`\n❌ Validation error: ${message}`);
     process.exit(1);
   }
 }