@sk8metal/michi-cli 0.14.1 → 0.20.0

package/scripts/markdown-to-confluence.ts

@@ -3,6 +3,7 @@
  */
 
 import MarkdownIt from 'markdown-it';
+import { safeReadFileOrThrow } from './utils/safe-file-reader.js';
 
 const md = new MarkdownIt({
   html: true,
@@ -174,19 +175,18 @@ ${content}
 
 // CLI実行用
 if (import.meta.url === `file://${process.argv[1]}`) {
-  const { readFileSync } = await import('fs');
   const { resolve } = await import('path');
-  
+
   const args = process.argv.slice(2);
   if (args.length === 0) {
     console.error('Usage: tsx markdown-to-confluence.ts <markdown-file>');
     process.exit(1);
   }
-  
+
   const markdownFile = resolve(args[0]);
-  const markdown = readFileSync(markdownFile, 'utf-8');
+  const markdown = safeReadFileOrThrow(markdownFile);
   const confluenceHtml = convertMarkdownToConfluence(markdown);
-  
+
   console.log(confluenceHtml);
 }
 

package/scripts/multi-project-estimate.ts

@@ -6,7 +6,8 @@ import { Octokit } from '@octokit/rest';
 import { loadEnv } from './utils/env-loader.js';
 import ExcelJS from 'exceljs';
 import { resolve, join, dirname } from 'path';
-import { writeFileSync, mkdirSync, unlinkSync, readFileSync } from 'fs';
+import { writeFileSync, mkdirSync, unlinkSync } from 'fs';
+import { safeReadFileOrThrow } from './utils/safe-file-reader.js';
 import { tmpdir } from 'os';
 import { mkdir } from 'fs/promises';
 
@@ -41,7 +42,7 @@ interface RiskEstimate {
  * design.mdから見積もりを抽出（estimate-generator.tsから統合）
  */
 function parseEstimateFromDesign(designPath: string): EstimateData {
-  const content = readFileSync(designPath, 'utf-8');
+  const content = safeReadFileOrThrow(designPath, 'utf-8');
   
   const tasks: TaskEstimate[] = [];
   let totalDays = 0;

package/scripts/phase-runner.ts

@@ -3,7 +3,7 @@
  * 各フェーズを実行し、Confluence/JIRA作成を確実に実行
  */
 
-import { existsSync, writeFileSync, readFileSync } from 'fs';
+import { existsSync, writeFileSync } from 'fs';
 import { join, relative } from 'path';
 import { syncToConfluence } from './confluence-sync.js';
 import { syncTasksToJIRA } from './jira-sync.js';
@@ -12,6 +12,8 @@ import { runPreFlightCheck } from './pre-flight-check.js';
 import { validateFeatureNameOrThrow } from './utils/feature-name-validator.js';
 import { getTestCommands } from './constants/test-commands.js';
 import { loadSpecJson } from './utils/spec-updater.js';
+import { safeReadFileOrThrow, safeReadJsonFile } from './utils/safe-file-reader.js';
+import { type SpecJson } from './utils/spec-updater.js';
 import inquirer from 'inquirer';
 
 type Phase =
@@ -31,6 +33,10 @@ interface PhaseRunResult {
   errors: string[];
 }
 
+interface TestTypeSelection {
+  selectedTypes?: string[];
+}
+
 /**
  * 要件定義フェーズを実行
  */
@@ -91,10 +97,10 @@ async function runRequirementsPhase(feature: string): Promise<PhaseRunResult> {
     `  ${confluenceCreated ? '✅' : '❌'} Confluenceページ: ${confluenceCreated ? '作成済み' : '未作成'}`,
   );
   console.log(
-    `  ${validation.valid ? '✅' : '❌'} バリデーション: ${validation.valid ? '成功' : '失敗'}`,
+    `  ${validation.success ? '✅' : '❌'} バリデーション: ${validation.success ? '成功' : '失敗'}`,
   );
 
-  if (validation.valid && confluenceCreated) {
+  if (validation.success && confluenceCreated) {
     console.log('\n🎉 要件定義フェーズが完了しました！');
     console.log('📢 PMや部長にConfluenceでレビューを依頼してください');
     if (confluenceUrl) {
@@ -110,10 +116,10 @@ async function runRequirementsPhase(feature: string): Promise<PhaseRunResult> {
 
   return {
     phase: 'requirements',
-    success: validation.valid && confluenceCreated,
+    success: validation.success && confluenceCreated,
     confluenceCreated,
     jiraCreated: false,
-    validationPassed: validation.valid,
+    validationPassed: validation.success,
     errors,
   };
 }
@@ -178,10 +184,10 @@ async function runDesignPhase(feature: string): Promise<PhaseRunResult> {
     `  ${confluenceCreated ? '✅' : '❌'} Confluenceページ: ${confluenceCreated ? '作成済み' : '未作成'}`,
   );
   console.log(
-    `  ${validation.valid ? '✅' : '❌'} バリデーション: ${validation.valid ? '成功' : '失敗'}`,
+    `  ${validation.success ? '✅' : '❌'} バリデーション: ${validation.success ? '成功' : '失敗'}`,
   );
 
-  if (validation.valid && confluenceCreated) {
+  if (validation.success && confluenceCreated) {
     console.log('\n🎉 設計フェーズが完了しました！');
     console.log('📢 PMや部長にConfluenceでレビューを依頼してください');
     if (confluenceUrl) {
@@ -197,10 +203,10 @@ async function runDesignPhase(feature: string): Promise<PhaseRunResult> {
 
   return {
     phase: 'design',
-    success: validation.valid && confluenceCreated,
+    success: validation.success && confluenceCreated,
     confluenceCreated,
     jiraCreated: false,
-    validationPassed: validation.valid,
+    validationPassed: validation.success,
     errors,
   };
 }
@@ -249,7 +255,14 @@ async function detectAndConvertAIDLCFormat(
 
   console.log('\n🔍 tasks.mdフォーマット検証中...');
   const { isAIDLCFormat } = await import('./utils/aidlc-parser.js');
-  const tasksContent = readFileSync(tasksPath, 'utf-8');
+
+  let tasksContent: string;
+  try {
+    tasksContent = safeReadFileOrThrow(tasksPath);
+  } catch (error) {
+    errors.push(`tasks.md読み込み失敗: ${error instanceof Error ? error.message : String(error)}`);
+    return { success: false, errors };
+  }
 
   if (!isAIDLCFormat(tasksContent)) {
     return { success: true, errors: [] };
@@ -355,7 +368,7 @@ async function validateTasksFormatAndSyncJIRA(
  */
 function displayTasksPhaseSummary(
   jiraCreated: boolean,
-  validation: { valid: boolean; errors: string[] }
+  validation: { success: boolean; errors: string[] }
 ): void {
   console.log('\n' + '='.repeat(60));
   console.log('📊 タスク分割フェーズ完了チェック:');
@@ -364,10 +377,10 @@ function displayTasksPhaseSummary(
     `  ${jiraCreated ? '✅' : '❌'} JIRA Epic/Story: ${jiraCreated ? '作成済み' : '未作成'}`,
   );
   console.log(
-    `  ${validation.valid ? '✅' : '❌'} バリデーション: ${validation.valid ? '成功' : '失敗'}`,
+    `  ${validation.success ? '✅' : '❌'} バリデーション: ${validation.success ? '成功' : '失敗'}`,
   );
 
-  if (validation.valid && jiraCreated) {
+  if (validation.success && jiraCreated) {
     console.log('\n🎉 タスク分割フェーズが完了しました！');
     console.log('📢 開発チームに実装開始を通知してください');
     console.log('🚀 次のステップ: /kiro:spec-impl <feature>');
@@ -427,10 +440,10 @@ async function runTasksPhase(feature: string): Promise<PhaseRunResult> {
 
   return {
     phase: 'tasks',
-    success: validation.valid && jiraCreated,
+    success: validation.success && jiraCreated,
     confluenceCreated: false,
     jiraCreated,
-    validationPassed: validation.valid,
+    validationPassed: validation.success,
     errors,
   };
 }
@@ -697,8 +710,16 @@ async function updateEnvironmentSpecJson(
     return;
   }
 
+  const readResult = safeReadJsonFile<SpecJson>(specPath);
+  if (!readResult.success) {
+    const error = readResult.errors[0];
+    const message = error.type === 'InvalidJSON' ? error.cause : error.type;
+    errors.push(`spec.json読み込み失敗: ${message}`);
+    return { success: false, errors };
+  }
+
   try {
-    const spec = JSON.parse(readFileSync(specPath, 'utf-8'));
+    const spec = readResult.value!;
     spec.environmentSetup = {
       completed: true,
       language: answers.language,
@@ -840,11 +861,12 @@ function loadPhaseBTestTypes(feature: string): string[] {
 
   let selectedTypes: string[] = [];
   if (existsSync(selectionPath)) {
-    try {
-      const selection = JSON.parse(readFileSync(selectionPath, 'utf-8'));
+    const readResult = safeReadJsonFile<TestTypeSelection>(selectionPath);
+    if (readResult.success) {
+      const selection = readResult.value!;
       selectedTypes = selection.selectedTypes || [];
       console.log(`\n✅ 選択されたテストタイプ: ${selectedTypes.join(', ')}`);
-    } catch {
+    } else {
       console.warn('⚠️  test-type-selection.jsonの読み込みに失敗しました');
     }
   } else {

package/scripts/pre-flight-check.ts

@@ -3,7 +3,8 @@
  * スクリプト実行前に必要な設定が揃っているか確認
  */
 
-import { existsSync, readFileSync } from 'fs';
+import { existsSync } from 'fs';
+import { safeReadJsonFile } from './utils/safe-file-reader.js';
 import { join } from 'path';
 import axios from 'axios';
 import { loadEnv } from './utils/env-loader.js';
@@ -85,14 +86,19 @@ function checkProjectJson(): { errors: string[], warnings: string[] } {
     return { errors, warnings };
   }
   
-  let projectMeta: ProjectMeta;
-  try {
-    projectMeta = JSON.parse(readFileSync(projectJsonPath, 'utf-8'));
-  } catch {
-    errors.push('❌ project.json のパースに失敗しました');
+  const readResult = safeReadJsonFile<ProjectMeta>(projectJsonPath);
+  if (!readResult.success) {
+    const error = readResult.errors[0];
+    if (error.type === 'InvalidJSON') {
+      errors.push(`❌ project.json のパースに失敗しました: ${error.cause}`);
+    } else {
+      errors.push('❌ project.json のパースに失敗しました');
+    }
     return { errors, warnings };
   }
-  
+
+  const projectMeta = readResult.value!;
+
   // 必須フィールドチェック
   const required = ['projectId', 'projectName', 'jiraProjectKey'];
   const missing = required.filter(key => !projectMeta[key]);
@@ -245,10 +251,20 @@ export async function runPreFlightCheck(phase: 'confluence' | 'jira' | 'all'): P
   if (phase === 'jira' || phase === 'all') {
     console.log('\n📋 Step 4: JIRAプロジェクト存在チェック');
     const projectJsonPath = join(process.cwd(), '.kiro', 'project.json');
-    const projectMeta = JSON.parse(readFileSync(projectJsonPath, 'utf-8'));
-    const jiraCheck = await checkJiraProject(projectMeta.jiraProjectKey);
-    allErrors.push(...jiraCheck.errors);
-    allWarnings.push(...jiraCheck.warnings);
+
+    const readResult = safeReadJsonFile<ProjectMeta>(projectJsonPath);
+    if (!readResult.success) {
+      allErrors.push(`❌ project.json読み込み失敗: ${readResult.errors[0].type}`);
+    } else {
+      const projectMeta = readResult.value!;
+      if (!projectMeta.jiraProjectKey) {
+        allErrors.push('❌ project.jsonにjiraProjectKeyが設定されていません');
+      } else {
+        const jiraCheck = await checkJiraProject(projectMeta.jiraProjectKey);
+        allErrors.push(...jiraCheck.errors);
+        allWarnings.push(...jiraCheck.warnings);
+      }
+    }
   }
   
   return {

package/scripts/template/__tests__/multi-repo-renderer.test.ts

@@ -23,6 +23,18 @@ const MICHI_PACKAGE_ROOT = resolve(__dirname, '..', '..', '..');
 
 vi.mock('fs');
 
+// safe-file-reader のモック（実際のreadFileSyncモックを使用するため）
+vi.mock('../../utils/safe-file-reader.js', async () => {
+  const mockFs = await import('fs');
+  return {
+    safeReadFileOrThrow: vi.fn((filePath: string, encoding: BufferEncoding = 'utf-8') => {
+      return (mockFs.readFileSync as ReturnType<typeof vi.fn>)(filePath, encoding);
+    }),
+    safeReadFile: vi.fn(),
+    safeReadJsonFile: vi.fn(),
+  };
+});
+
 describe('createMultiRepoTemplateContext', () => {
   it('必須フィールドを含むコンテキストを作成する', () => {
     const context = createMultiRepoTemplateContext(

package/scripts/template/multi-repo-renderer.ts

@@ -4,7 +4,7 @@
  * Provides template rendering functionality for Multi-Repo projects
  */
 
-import { readFileSync } from 'fs';
+import { safeReadFileOrThrow } from '../utils/safe-file-reader.js';
 import { resolve, relative, isAbsolute, dirname } from 'path';
 import { fileURLToPath } from 'url';
 import { renderTemplate, type TemplateContext } from './renderer.js';
@@ -82,7 +82,7 @@ export const loadMultiRepoTemplate = (
   }
 
   try {
-    return readFileSync(templatePath, 'utf-8');
+    return safeReadFileOrThrow(templatePath, 'utf-8');
   } catch (error) {
     const errorMessage = error instanceof Error ? error.message : String(error);
     throw new Error(

package/scripts/test-execution-generator.ts

@@ -3,8 +3,9 @@
  * Phase Bで選択されたテストタイプに基づいて実行ファイルを生成
  */
 
-import { readFileSync, writeFileSync, existsSync, mkdirSync, chmodSync } from 'fs';
+import { writeFileSync, existsSync, mkdirSync, chmodSync } from 'fs';
 import { join } from 'path';
+import { safeReadFileOrThrow } from './utils/safe-file-reader.js';
 
 /**
  * セキュリティ: URLバリデーション
@@ -104,7 +105,7 @@ function extractEndpointsFromDesign(designPath: string): { endpoint: string; met
     return [{ endpoint: '/api/health', method: 'GET', baseUrl: 'http://localhost:8080' }];
   }
 
-  const content = readFileSync(designPath, 'utf-8');
+  const content = safeReadFileOrThrow(designPath, 'utf-8');
   const endpoints: { endpoint: string; method: string; baseUrl: string }[] = [];
 
   // APIエンドポイントのパターンを検索
@@ -147,7 +148,7 @@ function extractPerformanceRequirements(requirementsPath: string): {
     return defaults;
   }
 
-  const content = readFileSync(requirementsPath, 'utf-8');
+  const content = safeReadFileOrThrow(requirementsPath, 'utf-8');
 
   // レスポンスタイム要件を検索
   const responseTimeMatch = content.match(/(\d+)\s*(?:ms|ミリ秒)以内/);
@@ -771,7 +772,7 @@ export async function generateAllTestExecutions(
     throw new Error('test-type-selection.jsonが存在しません。先にtest-type-selectionフェーズを実行してください');
   }
 
-  const selection = JSON.parse(readFileSync(selectionPath, 'utf-8'));
+  const selection = JSON.parse(safeReadFileOrThrow(selectionPath, 'utf-8'));
   const testTypes: string[] = selection.selectedTypes || [];
 
   const results: GenerationResult[] = [];

package/scripts/utils/__tests__/config-validator.test.ts

@@ -65,7 +65,7 @@ describe('config-validator', () => {
 
       const result = validateProjectConfig(testProjectRoot);
 
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       expect(result.errors).toHaveLength(0);
       expect(result.warnings).toHaveLength(0);
       expect(result.info.length).toBeGreaterThan(0);
@@ -94,7 +94,7 @@ describe('config-validator', () => {
 
       const result = validateProjectConfig(testProjectRoot);
 
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       expect(result.errors).toHaveLength(0);
     });
 
@@ -104,7 +104,7 @@ describe('config-validator', () => {
 
       const result = validateProjectConfig(testProjectRoot);
 
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.length).toBeGreaterThan(0);
       expect(result.errors[0]).toContain('Invalid JSON');
     });
@@ -122,7 +122,7 @@ describe('config-validator', () => {
 
       const result = validateProjectConfig(testProjectRoot);
 
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.length).toBeGreaterThan(0);
       expect(result.errors[0]).toContain('hierarchy');
     });
@@ -140,7 +140,7 @@ describe('config-validator', () => {
 
       const result = validateProjectConfig(testProjectRoot);
 
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.length).toBeGreaterThan(0);
       expect(result.errors[0]).toContain('selectedPhases');
     });
@@ -179,7 +179,7 @@ describe('config-validator', () => {
 
       const result = validateForConfluenceSync('requirements', testProjectRoot);
 
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       // デフォルト設定がある場合、警告が表示されない可能性がある
       // 実際の動作に合わせてテストを調整
       if (result.warnings.length > 0) {
@@ -202,7 +202,7 @@ describe('config-validator', () => {
 
       const result = validateForConfluenceSync('requirements', testProjectRoot);
 
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       expect(result.errors).toHaveLength(0);
     });
 
@@ -227,7 +227,7 @@ describe('config-validator', () => {
 
       // デフォルト設定にhierarchyがある場合、エラーにならない
       // このテストは、デフォルト設定の動作を確認するためのもの
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
     });
 
     it('manualモードでstructure設定がない場合はエラー', () => {
@@ -246,7 +246,7 @@ describe('config-validator', () => {
 
       const result = validateForConfluenceSync('requirements', testProjectRoot);
 
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.length).toBeGreaterThan(0);
       expect(result.errors[0]).toContain('structure');
     });
@@ -266,7 +266,7 @@ describe('config-validator', () => {
 
       const result = validateForConfluenceSync('requirements', testProjectRoot);
 
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       // 環境変数がある場合、infoメッセージが表示される可能性がある
       // 実際の動作に合わせてテストを調整
       if (result.info.length > 0) {
@@ -318,7 +318,7 @@ describe('config-validator', () => {
       const result = validateForJiraSync(testProjectRoot);
 
       // story=nullの場合、環境変数もないのでエラーになる
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.length).toBeGreaterThan(0);
       expect(result.errors[0]).toContain('issueTypes.story');
     });
@@ -338,7 +338,7 @@ describe('config-validator', () => {
 
       const result = validateForJiraSync(testProjectRoot);
 
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       expect(result.errors).toHaveLength(0);
     });
 
@@ -357,7 +357,7 @@ describe('config-validator', () => {
 
       const result = validateForJiraSync(testProjectRoot);
 
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       // 環境変数がある場合、infoメッセージが表示される可能性がある
       // 実際の動作に合わせてテストを調整
       if (result.info.length > 0) {
@@ -382,7 +382,7 @@ describe('config-validator', () => {
       const result = validateForJiraSync(testProjectRoot);
 
       // subtask=nullの場合、環境変数もないので警告になる
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       expect(result.warnings.length).toBeGreaterThan(0);
       expect(result.warnings[0]).toContain('subtask');
     });
@@ -403,7 +403,7 @@ describe('config-validator', () => {
 
       const result = validateForJiraSync(testProjectRoot);
 
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.length).toBeGreaterThan(0);
       expect(result.errors[0]).toContain('selectedPhases');
     });
@@ -490,7 +490,7 @@ describe('config-validator', () => {
 
       const result = await validateForJiraSyncAsync(testProjectRoot);
 
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       expect(result.errors).toHaveLength(0);
     });
 
@@ -536,7 +536,7 @@ describe('config-validator', () => {
 
       const result = await validateForJiraSyncAsync(testProjectRoot);
 
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       expect(result.errors).toHaveLength(0);
     });
 
@@ -588,7 +588,7 @@ describe('config-validator', () => {
 
       const result = await validateForJiraSyncAsync(testProjectRoot);
 
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.length).toBeGreaterThan(0);
       expect(
         result.errors.some(
@@ -635,7 +635,7 @@ describe('config-validator', () => {
 
       const result = await validateForJiraSyncAsync(testProjectRoot);
 
-      expect(result.valid).toBe(true); // エラーにはしない
+      expect(result.success).toBe(true); // エラーにはしない
       expect(result.warnings.length).toBeGreaterThan(0);
       expect(
         result.warnings.some((w) => w.includes('取得できませんでした')),

package/scripts/utils/__tests__/feature-name-validator.test.ts

@@ -9,25 +9,25 @@ describe('validateFeatureName', () => {
   describe('有効なfeature名', () => {
     it('小文字の英数字とハイフンのみ', () => {
       const result = validateFeatureName('user-auth');
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       expect(result.errors).toHaveLength(0);
     });
     
     it('単一単語', () => {
       const result = validateFeatureName('payment');
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       expect(result.errors).toHaveLength(0);
     });
     
     it('複数単語（ハイフン区切り）', () => {
       const result = validateFeatureName('health-check-endpoint');
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       expect(result.errors).toHaveLength(0);
     });
     
     it('数字を含む', () => {
       const result = validateFeatureName('api-v2');
-      expect(result.valid).toBe(true);
+      expect(result.success).toBe(true);
       expect(result.errors).toHaveLength(0);
     });
   });
@@ -35,55 +35,55 @@ describe('validateFeatureName', () => {
   describe('無効なfeature名', () => {
     it('大文字を含む', () => {
       const result = validateFeatureName('UserAuth');
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.some(e => e.includes('大文字'))).toBe(true);
     });
     
     it('日本語を含む', () => {
       const result = validateFeatureName('ユーザー認証');
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.some(e => e.includes('日本語'))).toBe(true);
     });
     
     it('アンダースコアを含む', () => {
       const result = validateFeatureName('user_auth');
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.some(e => e.includes('アンダースコア'))).toBe(true);
     });
     
     it('スペースを含む', () => {
       const result = validateFeatureName('user auth');
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.some(e => e.includes('スペース'))).toBe(true);
     });
     
     it('先頭にハイフン', () => {
       const result = validateFeatureName('-user-auth');
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.some(e => e.includes('先頭'))).toBe(true);
     });
     
     it('末尾にハイフン', () => {
       const result = validateFeatureName('user-auth-');
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.some(e => e.includes('末尾'))).toBe(true);
     });
     
     it('連続したハイフン', () => {
       const result = validateFeatureName('user--auth');
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.some(e => e.includes('連続'))).toBe(true);
     });
     
     it('空文字', () => {
       const result = validateFeatureName('');
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.some(e => e.includes('空'))).toBe(true);
     });
     
     it('特殊文字を含む', () => {
       const result = validateFeatureName('user@auth');
-      expect(result.valid).toBe(false);
+      expect(result.success).toBe(false);
       expect(result.errors.some(e => e.includes('使用できない文字'))).toBe(true);
     });
   });