@sk8metal/michi-cli 0.14.1 → 0.19.0

package/docs/onion-architecture-phase0-complete.md

@@ -0,0 +1,91 @@
+# オニオンアーキテクチャ Phase 0 完了レポート
+
+**日付**: 2025-12-28
+**タスク**: Task 1.6 - コード削減効果の検証
+**ステータス**: ✅ 完了
+
+---
+
+## 📊 Phase 0 成果サマリー
+
+### コード削減
+- **純削減行数**: 1,132行
+- **目標**: 約1,600行
+- **達成率**: 71%
+
+### 変更統計
+- 変更ファイル数: 67
+- 追加行数: 2,410
+- 削除行数: 3,542
+
+---
+
+## ✅ 完了タスク一覧
+
+### Task 1.1: 非推奨コマンドと完全未使用ファイルの削除
+- setup-existing.ts (794行) 削除
+- 未使用ファイル5件削除 (~400行)
+- コミット: 419347b
+
+### Task 1.2: 未使用エクスポートの削除
+- 5つの未使用関数を削除
+- コミット: 419347b
+
+### Task 1.3: ValidationResult インターフェースの統合
+- Result<T, E>型に統一
+- PR #148 マージ完了
+
+### Task 1.4: project検出系ユーティリティの統合
+- ProjectAnalyzer クラスに統合
+- PR #149 マージ完了
+
+### Task 1.5: ファイル読み込み処理の統合
+- 31ファイルをsafeReadFileOrThrowに移行
+- PR #150 マージ完了
+
+### Task 1.6: コード削減効果の検証
+- ✅ 削減行数測定: 1,132行
+- ✅ 全テスト合格: 816 passed, 5 skipped
+- ✅ Multi-Repo機能確認: 正常動作
+- ✅ Phase 0 検証完了
+
+---
+
+## 🎯 品質指標
+
+### テスト結果
+```
+Test Files: 48 passed | 1 skipped (49)
+Tests: 816 passed | 5 skipped (821)
+Duration: 4.63s
+```
+
+### ビルド結果
+- TypeScript compilation: ✅ Success
+- Static assets copy: ✅ Success
+- Permissions: ✅ Success
+
+### Multi-Repo機能
+- Multi-Repo テスト: ✅ 全合格
+- Multi-Repo コマンド: ✅ ビルド成功
+- セキュリティテスト: ✅ 全合格
+
+---
+
+## 📋 次のステップ
+
+**Phase 1: 基盤整備 + scripts/分類（Week 2-3）**
+
+準備完了。以下のタスクに進めます：
+- Task 2.1: 新ディレクトリ構造の作成
+- Task 2.2: パスエイリアスの追加
+- Task 2.3: ts-archのインストールと基本設定
+
+---
+
+## 📝 備考
+
+Phase 0の全タスクが完了し、コードベース簡素化の基盤が整いました。
+次のフェーズでオニオンアーキテクチャの層構造を構築します。
+
+詳細な検証レポートは `.kiro/specs/onion-architecture/phase0-verification.md` を参照してください。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sk8metal/michi-cli",
-  "version": "0.14.1",
+  "version": "0.19.0",
   "description": "Managed Intelligent Comprehensive Hub for Integration - AI-driven development workflow automation",
   "license": "MIT",
   "type": "module",
@@ -42,11 +42,9 @@
     "mcp.json.example"
   ],
   "scripts": {
-    "build": "tsc && node scripts/copy-static-assets.js",
-    "postbuild": "node scripts/set-permissions.js",
+    "build": "tsc && node scripts/build/copy-static-assets.js",
+    "postbuild": "node scripts/build/set-permissions.js",
     "prepare": "husky",
-    "michi:setup:claude": "tsx src/cli.ts setup-existing --claude",
-    "michi:setup:claude-agent": "tsx src/cli.ts setup-existing --claude-agent",
     "confluence:sync": "tsx scripts/confluence-sync.ts",
     "jira:sync": "tsx scripts/jira-sync.ts",
     "github:create-pr": "tsx scripts/pr-automation.ts",
@@ -61,11 +59,12 @@
     "test:package": "bash scripts/test-npm-package.sh",
     "config:global": "tsx scripts/config-global.ts",
     "config:validate": "tsx scripts/utils/config-validator.ts",
-    "test:interactive": "tsx scripts/test-interactive.ts",
+    "test:interactive": "tsx scripts/dev-tools/test-interactive.ts",
     "test": "vitest",
     "test:run": "vitest --run",
     "test:ui": "vitest --ui",
     "test:coverage": "vitest --run --coverage",
+    "test:arch": "vitest --run src/__tests__/arch/",
     "test:integration:setup": "vitest --run src/__tests__/integration/setup",
     "test:coverage:setup": "vitest --run --coverage --coverage.thresholds.lines=0 --coverage.thresholds.functions=0 --coverage.thresholds.branches=0 --coverage.thresholds.statements=0 src/__tests__/integration/setup",
     "lint": "eslint .",
@@ -97,6 +96,7 @@
     "husky": "^9.1.7",
     "lint-staged": "^16.2.6",
     "prettier": "^3.1.1",
+    "ts-arch-kit": "^2.4.0",
     "tsx": "^4.7.0",
     "typescript": "^5.3.3",
     "typescript-eslint": "^8.46.4",

package/scripts/README.md

@@ -0,0 +1,148 @@
+# scripts/ - Build and Development Tools
+
+このディレクトリはビルドツールおよび開発ツール専用です。プロダクションコードは含めないでください。
+
+## ディレクトリ構造
+
+```
+scripts/
+├── build/              # ビルド関連スクリプト
+│   ├── copy-static-assets.js    # 静的ファイルコピー
+│   └── set-permissions.js       # 実行権限設定
+├── dev-tools/          # 開発支援ツール
+│   ├── test-interactive.ts      # 対話型テスト
+│   └── mermaid-converter.ts     # Mermaid図変換ユーティリティ
+├── utils/              # 共通ユーティリティ
+│   ├── env-loader.js            # 環境変数読み込み
+│   ├── config-loader.ts         # 設定ファイル読み込み
+│   ├── safe-file-reader.ts      # 安全なファイル読み込み
+│   └── ...
+├── config/             # 設定ファイル・スキーマ
+└── constants/          # 定数定義
+```
+
+## スクリプト分類
+
+### Entry Points（エントリーポイント）
+
+プロダクションコード（`src/`）へのCLIエントリーポイント。実装本体は`src/`に配置されています。
+
+- `confluence-sync.ts` → `src/presentation/commands/confluence/`
+- `jira-sync.ts` → `src/presentation/commands/jira/`
+- `workflow-orchestrator.ts` → `src/presentation/commands/workflow/`
+- `github-actions-client.ts` → `src/infrastructure/external-apis/github/`
+
+**使用例:**
+```bash
+npm run confluence:sync <feature-name> [docType]
+npm run jira:sync <feature-name>
+npm run workflow:run -- --feature <feature-name>
+```
+
+### Development Tools（開発ツール）
+
+開発・テスト支援のためのスタンドアロンツール。
+
+- `config-global.ts` - グローバル設定ファイル作成・更新
+- `pre-flight-check.ts` - 実行前環境チェック
+- `validate-phase.ts` - フェーズ完了バリデーション
+- `test-workflow-stages.ts` - ワークフロー新規ステージテスト
+- `health-check-service.ts` - 依存サービス状態確認
+- `test-execution-generator.ts` - テスト実行ファイル生成
+- `test-script-runner.ts` - Multi-Repoテストスクリプト実行
+
+**使用例:**
+```bash
+npm run config:global
+npm run preflight
+npm run validate:phase <feature-name> <phase>
+npm run test:interactive
+```
+
+### Workflow Automation（ワークフロー自動化）
+
+開発ワークフロー自動化のためのツール。
+
+- `phase-runner.ts` - フェーズ実行オーケストレーション
+- `spec-impl-workflow.ts` - spec-impl統合ワークフロー管理
+- `pr-automation.ts` - GitHub PR自動作成
+- `multi-project-estimate.ts` - マルチプロジェクト見積もり集計
+
+**使用例:**
+```bash
+npm run phase:run -- --feature <feature-name> --phase <phase>
+npm run github:create-pr
+npm run multi-estimate
+```
+
+### Utilities（ユーティリティ）
+
+変換・フォーマット処理。
+
+- `markdown-to-confluence.ts` - Markdown → Confluence Storage Format変換
+
+**使用例:**
+```bash
+npm run markdown:convert <input.md> <output.html>
+```
+
+## 重要な原則
+
+### ❌ scripts/に配置してはいけないもの
+
+- **プロダクションコード**: ビジネスロジック、ドメインモデル、アプリケーションサービス
+  - → `src/domain/`、`src/application/`、`src/infrastructure/`、`src/presentation/` に配置
+- **再利用可能なライブラリコード**: 複数の機能で使われる共通コード
+  - → `src/` の適切な層に配置
+
+### ✅ scripts/に配置すべきもの
+
+- **ビルドツール**: コンパイル、アセットコピー、パーミッション設定など
+- **開発ツール**: テストヘルパー、対話型ツール、デバッグユーティリティ
+- **自動化スクリプト**: CI/CD、ワークフロー、デプロイメント
+- **Entry Points**: 本体が`src/`にある機能へのCLIラッパー
+
+## 実装ガイドライン
+
+### 新しいスクリプトを追加する場合
+
+1. **配置場所を決定**:
+   - ビルドツール → `scripts/build/`
+   - 開発ツール → `scripts/dev-tools/`
+   - 共通ユーティリティ → `scripts/utils/`
+
+2. **ファイルヘッダーを記載**:
+   ```typescript
+   /**
+    * スクリプト名
+    * 簡潔な説明（1-2行）
+    */
+   ```
+
+3. **Entry Pointの場合**:
+   ```typescript
+   /**
+    * 機能名 - Entry Point
+    * The actual logic has been moved to src/path/to/implementation/
+    */
+   ```
+
+4. **package.jsonにスクリプトを追加**:
+   ```json
+   "scripts": {
+     "your-script": "tsx scripts/path/to/script.ts"
+   }
+   ```
+
+### コーディング規約
+
+- **環境変数**: `loadEnv()`を使用
+- **ファイル読み込み**: `safeReadFileOrThrow()`を使用
+- **設定読み込み**: `loadConfig()`を使用
+- **エラーハンドリング**: 適切なエラーメッセージとexit code
+
+## 関連ドキュメント
+
+- [Onion Architecture設計書](../.kiro/specs/onion-architecture/design.md)
+- [プロジェクト構成](../.kiro/steering/structure.md)
+- [開発ワークフロー](../.kiro/steering/workflow.md)

package/scripts/__tests__/jira-transitions.test.ts

@@ -6,18 +6,48 @@ import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import axios from 'axios';
 
 // axiosをモック
-vi.mock('axios');
+vi.mock('axios', async () => {
+  const actual = await vi.importActual<typeof import('axios')>('axios');
+  return {
+    ...actual,
+    default: {
+      ...actual.default,
+      create: vi.fn(() => ({
+        get: vi.fn(),
+        post: vi.fn(),
+        put: vi.fn(),
+        delete: vi.fn(),
+      })),
+      get: vi.fn(),
+      post: vi.fn(),
+      put: vi.fn(),
+      isAxiosError: vi.fn(),
+    },
+  };
+});
 
 // dotenvをモック
 vi.mock('dotenv', () => ({ config: vi.fn() }));
 
 describe('JIRAClient transitions', () => {
+  let mockAxiosInstance: any;
   const mockAxiosGet = vi.mocked(axios.get);
   const mockAxiosPost = vi.mocked(axios.post);
   const mockAxiosIsAxiosError = vi.mocked(axios.isAxiosError);
+  const mockAxiosCreate = vi.mocked(axios.create);
 
   beforeEach(() => {
     vi.clearAllMocks();
+
+    // axios.createから返されるモックインスタンス
+    mockAxiosInstance = {
+      get: vi.fn(),
+      post: vi.fn(),
+      put: vi.fn(),
+      delete: vi.fn(),
+    };
+    mockAxiosCreate.mockReturnValue(mockAxiosInstance);
+
     // isAxiosError をデフォルトでtrueに
     mockAxiosIsAxiosError.mockReturnValue(true);
   });
@@ -29,7 +59,7 @@ describe('JIRAClient transitions', () => {
   describe('transitionIssue', () => {
     it('利用可能なトランジションを取得してステータスを変更できる', async () => {
       // トランジション一覧のモック
-      mockAxiosGet.mockResolvedValueOnce({
+      mockAxiosInstance.get.mockResolvedValueOnce({
         data: {
           transitions: [
             { id: '21', name: 'In Progress' },
@@ -40,7 +70,7 @@ describe('JIRAClient transitions', () => {
       });
 
       // トランジション実行のモック
-      mockAxiosPost.mockResolvedValueOnce({
+      mockAxiosInstance.post.mockResolvedValueOnce({
         data: {},
       });
 
@@ -55,29 +85,19 @@ describe('JIRAClient transitions', () => {
       await client.transitionIssue('PROJ-123', 'In Progress');
 
       // GETリクエストの検証
-      expect(mockAxiosGet).toHaveBeenCalledWith(
+      expect(mockAxiosInstance.get).toHaveBeenCalledWith(
         'https://test.atlassian.net/rest/api/3/issue/PROJ-123/transitions',
-        expect.objectContaining({
-          headers: expect.objectContaining({
-            'Content-Type': 'application/json',
-          }),
-        }),
       );
 
       // POSTリクエストの検証
-      expect(mockAxiosPost).toHaveBeenCalledWith(
+      expect(mockAxiosInstance.post).toHaveBeenCalledWith(
         'https://test.atlassian.net/rest/api/3/issue/PROJ-123/transitions',
         { transition: { id: '21' } },
-        expect.objectContaining({
-          headers: expect.objectContaining({
-            'Content-Type': 'application/json',
-          }),
-        }),
       );
     });
 
     it('部分一致でトランジションを検索できる', async () => {
-      mockAxiosGet.mockResolvedValueOnce({
+      mockAxiosInstance.get.mockResolvedValueOnce({
         data: {
           transitions: [
             { id: '21', name: 'Start Progress' },
@@ -86,7 +106,7 @@ describe('JIRAClient transitions', () => {
         },
       });
 
-      mockAxiosPost.mockResolvedValueOnce({
+      mockAxiosInstance.post.mockResolvedValueOnce({
         data: {},
       });
 
@@ -100,15 +120,14 @@ describe('JIRAClient transitions', () => {
       // "Progress" で部分一致
       await client.transitionIssue('PROJ-123', 'Progress');
 
-      expect(mockAxiosPost).toHaveBeenCalledWith(
-        expect.any(String),
+      expect(mockAxiosInstance.post).toHaveBeenCalledWith(
+        'https://test.atlassian.net/rest/api/3/issue/PROJ-123/transitions',
         { transition: { id: '21' } },
-        expect.any(Object),
       );
     });
 
     it('トランジションが見つからない場合はエラーをスロー', async () => {
-      mockAxiosGet.mockResolvedValueOnce({
+      mockAxiosInstance.get.mockResolvedValueOnce({
         data: {
           transitions: [
             { id: '21', name: 'In Progress' },
@@ -130,7 +149,7 @@ describe('JIRAClient transitions', () => {
     });
 
     it('利用可能なトランジションがない場合もエラーをスロー', async () => {
-      mockAxiosGet.mockResolvedValueOnce({
+      mockAxiosInstance.get.mockResolvedValueOnce({
         data: {
           transitions: [],
         },
@@ -151,7 +170,7 @@ describe('JIRAClient transitions', () => {
 
   describe('addComment', () => {
     it('コメントを追加できる', async () => {
-      mockAxiosPost.mockResolvedValueOnce({
+      mockAxiosInstance.post.mockResolvedValueOnce({
         data: {
           id: 'comment-123',
         },
@@ -169,7 +188,7 @@ describe('JIRAClient transitions', () => {
         'PRを作成しました: https://github.com/...',
       );
 
-      expect(mockAxiosPost).toHaveBeenCalledWith(
+      expect(mockAxiosInstance.post).toHaveBeenCalledWith(
         'https://test.atlassian.net/rest/api/3/issue/PROJ-123/comment',
         {
           body: {
@@ -188,16 +207,11 @@ describe('JIRAClient transitions', () => {
             ],
           },
         },
-        expect.objectContaining({
-          headers: expect.objectContaining({
-            'Content-Type': 'application/json',
-          }),
-        }),
       );
     });
 
     it('APIエラー時は適切なエラーメッセージを表示', async () => {
-      mockAxiosPost.mockRejectedValueOnce({
+      mockAxiosInstance.post.mockRejectedValueOnce({
         response: {
           status: 404,
           data: {

package/scripts/__tests__/mermaid-converter.test.ts

@@ -2,8 +2,8 @@
  * Tests for MermaidConverter
  */
 
-import { describe, it, expect } from 'vitest';
-import { MermaidConverter } from '../mermaid-converter.js';
+import { describe, it, expect, beforeEach } from 'vitest';
+import { MermaidConverter } from '../dev-tools/mermaid-converter.js';
 
 describe('MermaidConverter', () => {
   let converter: MermaidConverter;