@sk8metal/michi-cli 0.0.8 → 0.1.0

package/scripts/__tests__/setup-existing-project.test.ts

@@ -1,6 +1,7 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { join } from 'path';
-import { mkdirSync, cpSync } from 'fs';
+import { mkdirSync } from 'fs';
+import { parseEnvFile, generateEnvContent } from '../utils/env-config.js';
 
 // モジュールのモック
 vi.mock('fs', () => ({
@@ -8,19 +9,137 @@ vi.mock('fs', () => ({
   mkdirSync: vi.fn(),
   cpSync: vi.fn(),
   writeFileSync: vi.fn(),
-  readFileSync: vi.fn(() => '{}')
+  readFileSync: vi.fn(() => '{}'),
+  readdirSync: vi.fn(() => []),
+  statSync: vi.fn()
 }));
 vi.mock('child_process', () => ({
   execSync: vi.fn()
 }));
-vi.mock('./utils/project-finder.js', () => ({
+vi.mock('../utils/project-finder.js', () => ({
   findRepositoryRoot: vi.fn(() => '/test/repo')
 }));
+vi.mock('../constants/environments.js', () => ({
+  getEnvironmentConfig: vi.fn(() => ({
+    rulesDir: '.cursor/rules',
+    commandsDir: '.cursor/commands/kiro',
+    templateSource: 'cursor'
+  })),
+  isSupportedEnvironment: vi.fn(() => true)
+}));
+vi.mock('../constants/languages.js', () => ({
+  isSupportedLanguage: vi.fn(() => true)
+}));
+vi.mock('../template/renderer.js', () => ({
+  createTemplateContext: vi.fn(() => ({
+    LANG_CODE: 'ja',
+    DEV_GUIDELINES: '- Think in English, but generate responses in Japanese',
+    KIRO_DIR: '.kiro',
+    AGENT_DIR: '.cursor'
+  })),
+  renderTemplate: vi.fn((content: string) => content)
+}));
 
 describe('setup-existing-project.ts 修正内容のテスト', () => {
   beforeEach(() => {
     vi.clearAllMocks();
   });
+  
+  describe('.env 対話的設定機能', () => {
+    it('.env ファイルが存在しない場合、対話的設定を提供する', () => {
+      // 新規作成の場合の動作を確認
+      // parseEnvFile が空のMapを返すことを確認
+      const result = parseEnvFile('nonexistent.env');
+
+      expect(result).toBeInstanceOf(Map);
+      expect(result.size).toBe(0);
+    });
+    
+    it('.env ファイルが存在する場合、既存値を読み込む', async () => {
+      const fs = await import('fs');
+
+      // モックの設定
+      vi.mocked(fs.existsSync).mockReturnValue(true);
+      vi.mocked(fs.readFileSync).mockReturnValue(`ATLASSIAN_URL=https://existing.atlassian.net
+ATLASSIAN_EMAIL=existing@example.com
+ATLASSIAN_API_TOKEN=existing_token
+`);
+
+      const result = parseEnvFile('.env');
+
+      expect(result.size).toBeGreaterThan(0);
+      expect(result.get('ATLASSIAN_URL')).toBe('https://existing.atlassian.net');
+    });
+    
+    it('generateEnvContent が正しい形式の .env を生成する', () => {
+      const values = new Map([
+        ['ATLASSIAN_URL', 'https://test.atlassian.net'],
+        ['ATLASSIAN_EMAIL', 'test@example.com'],
+        ['ATLASSIAN_API_TOKEN', 'test_token'],
+        ['JIRA_PROJECT_KEYS', 'TEST'],
+        ['JIRA_ISSUE_TYPE_STORY', '10036'],
+        ['JIRA_ISSUE_TYPE_SUBTASK', '10037']
+      ]);
+
+      const content = generateEnvContent(values);
+
+      expect(content).toContain('# Atlassian設定');
+      expect(content).toContain('ATLASSIAN_URL=https://test.atlassian.net');
+      expect(content).toContain('ATLASSIAN_EMAIL=test@example.com');
+      expect(content).toContain('ATLASSIAN_API_TOKEN=test_token');
+    });
+    
+    it('.gitignore に .env エントリが追加される', () => {
+      // .gitignore 更新のロジック
+      let gitignoreContent = '# Existing content\nnode_modules/\n';
+      
+      const entriesToAdd = [
+        '# Environment variables',
+        '.env',
+        '.env.local',
+        '.env.*.local'
+      ];
+      
+      const lines = gitignoreContent.split('\n').map(l => l.trim());
+      let modified = false;
+      
+      for (const entry of entriesToAdd) {
+        if (!lines.includes(entry.trim())) {
+          if (!modified) {
+            gitignoreContent += '\n\n# Added by michi setup\n';
+            modified = true;
+          }
+          gitignoreContent += entry + '\n';
+        }
+      }
+      
+      expect(modified).toBe(true);
+      expect(gitignoreContent).toContain('# Added by michi setup');
+      expect(gitignoreContent).toContain('.env');
+      expect(gitignoreContent).toContain('.env.local');
+      expect(gitignoreContent).toContain('.env.*.local');
+    });
+    
+    it('.gitignore に既に .env がある場合、重複しない', () => {
+      const gitignoreContent = `# Environment
+.env
+.env.local
+.env.*.local
+`;
+      
+      const lines = gitignoreContent.split('\n').map(l => l.trim());
+      const entriesToAdd = ['.env', '.env.local', '.env.*.local'];
+      
+      let modified = false;
+      for (const entry of entriesToAdd) {
+        if (!lines.includes(entry.trim())) {
+          modified = true;
+        }
+      }
+      
+      expect(modified).toBe(false);
+    });
+  });
 
   it('ディレクトリがコピー前に作成される', () => {
     const projectDir = '/test/repo/projects/test-project';
@@ -58,12 +177,14 @@ describe('setup-existing-project.ts 修正内容のテスト', () => {
   it('完了メッセージに scripts/ ディレクトリが含まれていない', () => {
     const projectDir = '/test/repo/projects/test-project';
     const repoRoot = '/test/repo';
+    const envConfigRulesDir = '.cursor/rules';
+    const envConfigCommandsDir = '.cursor/commands/kiro';
     
-    // 実際に作成されるファイルのリスト
+    // 実際に作成されるファイルのリスト（環境別）
     const createdFiles = [
       `${projectDir}/.kiro/project.json`,
-      `${projectDir}/.cursor/rules/ (3ファイル)`,
-      `${projectDir}/.cursor/commands/kiro/ (2ファイル)`,
+      `${projectDir}/${envConfigRulesDir}/`,
+      `${projectDir}/${envConfigCommandsDir}/`,
       `${projectDir}/.kiro/steering/ (3ファイル)`,
       `${projectDir}/.kiro/settings/templates/ (3ファイル)`,
       `${repoRoot}/package.json (新規の場合)`,
@@ -75,5 +196,49 @@ describe('setup-existing-project.ts 修正内容のテスト', () => {
     const hasScriptsDir = createdFiles.some(file => file.includes('scripts/'));
     expect(hasScriptsDir).toBe(false);
   });
+
+  it('project.jsonにlanguageフィールドが含まれる', () => {
+    const projectJson = {
+      projectId: 'test-project',
+      projectName: 'Test Project',
+      language: 'ja',
+      jiraProjectKey: 'TEST',
+      confluenceLabels: ['project:test-project'],
+      status: 'active',
+      team: [],
+      stakeholders: ['@企画', '@部長'],
+      repository: 'https://github.com/org/test',
+      description: 'Test Projectの開発'
+    };
+
+    expect(projectJson).toHaveProperty('language');
+    expect(projectJson.language).toBe('ja');
+  });
+
+  it('環境別のディレクトリマッピングが正しい', () => {
+    const envConfig = {
+      rulesDir: '.cursor/rules',
+      commandsDir: '.cursor/commands/kiro',
+      templateSource: 'cursor'
+    };
+
+    expect(envConfig.rulesDir).toBe('.cursor/rules');
+    expect(envConfig.commandsDir).toBe('.cursor/commands/kiro');
+    expect(envConfig.templateSource).toBe('cursor');
+  });
+
+  it('テンプレートコンテキストが正しく作成される', () => {
+    const context = {
+      LANG_CODE: 'ja',
+      DEV_GUIDELINES: '- Think in English, but generate responses in Japanese',
+      KIRO_DIR: '.kiro',
+      AGENT_DIR: '.cursor'
+    };
+
+    expect(context).toHaveProperty('LANG_CODE');
+    expect(context).toHaveProperty('DEV_GUIDELINES');
+    expect(context).toHaveProperty('KIRO_DIR');
+    expect(context).toHaveProperty('AGENT_DIR');
+  });
 });
 

package/scripts/__tests__/setup-interactive.test.ts

@@ -1,5 +1,4 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { basename } from 'path';
 
 // validateProjectId関数のロジックを再現
 function validateProjectId(projectId: string): boolean {
@@ -8,7 +7,11 @@ function validateProjectId(projectId: string): boolean {
     return false;
   }
   // パストラバーサル文字を拒否
-  if (projectId.includes('..') || projectId.includes('/') || projectId.includes('\\')) {
+  if (
+    projectId.includes('..') ||
+    projectId.includes('/') ||
+    projectId.includes('\\')
+  ) {
     return false;
   }
   // 許可する文字のみ（英数字、ハイフン、アンダースコア）
@@ -24,36 +27,48 @@ describe('setup-interactive.ts 修正内容のテスト', () => {
     it('getProjectMetadata 内で validateProjectId が呼び出される', () => {
       const projectIdDefault = 'test-project';
       const projectId = projectIdDefault;
-      
+
       // バリデーション（getProjectMetadata内のロジックを再現）
       if (!validateProjectId(projectId)) {
-        throw new Error('無効なプロジェクトIDです。英数字、ハイフン、アンダースコアのみ使用できます。');
+        throw new Error(
+          '無効なプロジェクトIDです。英数字、ハイフン、アンダースコアのみ使用できます。',
+        );
       }
-      
+
       // 有効なプロジェクトIDはエラーを投げない
       expect(projectId).toBe('test-project');
     });
 
     it('無効なプロジェクトID（パストラバーサル）は拒否される', () => {
       const invalidIds = ['../test', 'test/../', 'test\\..', '../../'];
-      
-      invalidIds.forEach(id => {
+
+      invalidIds.forEach((id) => {
         expect(validateProjectId(id)).toBe(false);
       });
     });
 
     it('無効なプロジェクトID（特殊文字）は拒否される', () => {
-      const invalidIds = ['test@project', 'test#project', 'test$project', 'test project'];
-      
-      invalidIds.forEach(id => {
+      const invalidIds = [
+        'test@project',
+        'test#project',
+        'test$project',
+        'test project',
+      ];
+
+      invalidIds.forEach((id) => {
         expect(validateProjectId(id)).toBe(false);
       });
     });
 
     it('有効なプロジェクトIDは受け入れられる', () => {
-      const validIds = ['test-project', 'test_project', 'test123', 'Test-Project_123'];
-      
-      validIds.forEach(id => {
+      const validIds = [
+        'test-project',
+        'test_project',
+        'test123',
+        'Test-Project_123',
+      ];
+
+      validIds.forEach((id) => {
         expect(validateProjectId(id)).toBe(true);
       });
     });
@@ -66,7 +81,7 @@ describe('setup-interactive.ts 修正内容のテスト', () => {
         // 正常終了
         return 0;
       }
-      
+
       const result = await main();
       expect(typeof result).toBe('number');
       expect(result).toBe(0);
@@ -77,11 +92,11 @@ describe('setup-interactive.ts 修正内容のテスト', () => {
       async function main(): Promise<number> {
         try {
           throw new Error('テストエラー');
-        } catch (error) {
+        } catch {
           return 1;
         }
       }
-      
+
       const result = await main();
       expect(result).toBe(1);
     });
@@ -92,14 +107,14 @@ describe('setup-interactive.ts 修正内容のテスト', () => {
         // 設定する項目がない場合
         const configureProject = false;
         const configureEnv = false;
-        
+
         if (!configureProject && !configureEnv) {
           return 0;
         }
-        
+
         return 0;
       }
-      
+
       const result = await main();
       expect(result).toBe(0);
     });
@@ -109,15 +124,11 @@ describe('setup-interactive.ts 修正内容のテスト', () => {
     it('process.exit の代わりに return を使用する', () => {
       // 修正前: process.exit(0)
       // 修正後: return 0
-      
-      function oldStyle(): void {
-        // process.exit(0); // 削除
-      }
-      
+
       function newStyle(): number {
         return 0; // 新しいスタイル
       }
-      
+
       const result = newStyle();
       expect(result).toBe(0);
       expect(typeof result).toBe('number');
@@ -126,15 +137,11 @@ describe('setup-interactive.ts 修正内容のテスト', () => {
     it('エラー時も process.exit の代わりに return を使用する', () => {
       // 修正前: process.exit(1)
       // 修正後: return 1
-      
-      function oldStyle(): void {
-        // process.exit(1); // 削除
-      }
-      
+
       function newStyle(): number {
         return 1; // 新しいスタイル
       }
-      
+
       const result = newStyle();
       expect(result).toBe(1);
       expect(typeof result).toBe('number');
@@ -145,55 +152,54 @@ describe('setup-interactive.ts 修正内容のテスト', () => {
     it('main() の戻り値を受け取って process.exit を呼び出す', async () => {
       // CLIエントリーポイントのロジックを再現
       let exitCode: number | null = null;
-      
+
       const mockProcessExit = (code: number) => {
         exitCode = code;
       };
-      
+
       async function main(): Promise<number> {
         return 0;
       }
-      
+
       // CLIエントリーポイント
       main()
-        .then(code => {
+        .then((code) => {
           mockProcessExit(code);
         })
         .catch(() => {
           mockProcessExit(1);
         });
-      
+
       // 非同期処理を待つ
-      await new Promise(resolve => setTimeout(resolve, 10));
-      
+      await new Promise((resolve) => setTimeout(resolve, 10));
+
       expect(exitCode).toBe(0);
     });
 
     it('エラー時は 1 で終了する', async () => {
       let exitCode: number | null = null;
-      
+
       const mockProcessExit = (code: number) => {
         exitCode = code;
       };
-      
+
       async function main(): Promise<number> {
         throw new Error('テストエラー');
       }
-      
+
       // CLIエントリーポイント
       main()
-        .then(code => {
+        .then((code) => {
           mockProcessExit(code);
         })
         .catch(() => {
           mockProcessExit(1);
         });
-      
+
       // 非同期処理を待つ
-      await new Promise(resolve => setTimeout(resolve, 10));
-      
+      await new Promise((resolve) => setTimeout(resolve, 10));
+
       expect(exitCode).toBe(1);
     });
   });
 });
-