ai-git-tools 2.0.71 → 2.0.73

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ai-git-tools",
-  "version": "2.0.71",
+  "version": "2.0.73",
   "description": "AI-powered Git automation tools for commit messages and PR generation",
   "main": "src/index.js",
   "type": "module",

package/src/core/ai-client.js

@@ -9,7 +9,7 @@ export class AIClient {
   /**
    * 發送 prompt 並等待回應（帶重試機制和超時保護）
    */
-  static async sendAndWait(prompt, model = 'gpt-4.1', maxRetries = 3, timeout = 60000) {
+  static async sendAndWait(prompt, model = 'gpt-4.1', maxRetries = 3, timeout = 150000) {
     let lastError = null;
 
     for (let attempt = 1; attempt <= maxRetries; attempt++) {

package/src/pr-modules/ai/code-analyzer.js

@@ -2,24 +2,52 @@ import { CopilotClient, approveAll } from '@github/copilot-sdk';
 import { CONSTANTS, PROJECT_SKILLS_CONTEXT } from '../utils/constants.js';
 import { getSkillsSummaryForPrompt, log } from '../utils/helpers.js';
 
+// CopilotClient 子程序啟動 + AI 模型回應可能共需 60-120s，設為 150s 保留足夠緩衝
+const AI_TIMEOUT_MS = 150000;
+
 /**
  * AI 分析器 - 負責程式碼分析和 PR 內容生成
  */
 export class AIAnalyzer {
   constructor(config = {}) {
     this.model = config.model || 'gpt-4.1';
+    this._client = null; // 複用同一個 CopilotClient，避免重複啟動子程序
+  }
+
+  /**
+   * 取得（或建立）共用的 CopilotClient
+   */
+  async _getOrCreateClient() {
+    if (!this._client) {
+      this._client = new CopilotClient();
+    }
+    return this._client;
   }
 
   /**
-   * 建立 AI 客戶端
+   * 建立 AI Session（複用已有的 client）
    */
-  async createClient() {
-    const client = new CopilotClient();
-    const session = await client.createSession({ 
+  async _createSession() {
+    const client = await this._getOrCreateClient();
+    return client.createSession({
       model: this.model,
-      onPermissionRequest: approveAll 
+      onPermissionRequest: approveAll,
     });
-    return { client, session };
+  }
+
+  /**
+   * 釋放 CopilotClient 子程序資源
+   */
+  async close() {
+    if (this._client) {
+      try {
+        await this._client.stop();
+      } catch (e) {
+        // 忽略關閉錯誤
+      } finally {
+        this._client = null;
+      }
+    }
   }
 
   /**
@@ -29,31 +57,23 @@ export class AIAnalyzer {
     const skillsSummary = getSkillsSummaryForPrompt(PROJECT_SKILLS_CONTEXT);
     const prompt = this.buildPRPrompt(commits, diff, skillsSummary);
 
-    const { client, session } = await this.createClient();
+    const session = await this._createSession();
 
-    try {
-      // 使用超時保護 (60 秒)
-      const responsePromise = session.sendAndWait({ prompt });
-      const timeoutPromise = new Promise((_, reject) => {
-        setTimeout(() => reject(new Error('AI 請求超時 (60 秒)')), 60000);
-      });
+    // 使用超時保護（150 秒，含子程序啟動 + AI 回應）
+    const responsePromise = session.sendAndWait({ prompt });
+    const timeoutPromise = new Promise((_, reject) => {
+      setTimeout(() => reject(new Error(`AI 請求超時 (${AI_TIMEOUT_MS / 1000} 秒)`)), AI_TIMEOUT_MS);
+    });
 
-      const response = await Promise.race([responsePromise, timeoutPromise]);
-      const prContent = response?.data.content?.trim() || '';
+    const response = await Promise.race([responsePromise, timeoutPromise]);
+    const prContent = response?.data.content?.trim() || '';
 
-      if (!prContent) {
-        throw new Error('AI 未能生成 PR 內容');
-      }
-
-      return this.parsePRContent(prContent);
-    } finally {
-      // 確保 client 一定會被關閉
-      try {
-        await client.stop();
-      } catch (e) {
-        // 忽略關閉錯誤
-      }
+    if (!prContent) {
+      throw new Error('AI 未能生成 PR 內容');
     }
+
+    return this.parsePRContent(prContent);
+    // 注意：不在此 stop() client，改由 close() 統一清理以便複用
   }
 
   /**
@@ -63,15 +83,15 @@ export class AIAnalyzer {
     const skillsSummary = getSkillsSummaryForPrompt(PROJECT_SKILLS_CONTEXT);
     const prompt = this.buildAnalysisPrompt(changedFiles, diff, commits, skillsSummary);
 
-    const { client, session } = await this.createClient();
+    const session = await this._createSession();
 
     try {
       log.info('  正在使用 AI 深度分析程式碼變更...');
-      
-      // 使用超時保護 (60 秒)
+
+      // 使用超時保護（150 秒）
       const responsePromise = session.sendAndWait({ prompt });
       const timeoutPromise = new Promise((_, reject) => {
-        setTimeout(() => reject(new Error('AI 請求超時 (60 秒)')), 60000);
+        setTimeout(() => reject(new Error(`AI 請求超時 (${AI_TIMEOUT_MS / 1000} 秒)`)), AI_TIMEOUT_MS);
       });
 
       const response = await Promise.race([responsePromise, timeoutPromise]);
@@ -103,14 +123,8 @@ export class AIAnalyzer {
     } catch (error) {
       log.warning(`  AI 分析失敗 (${error.message})，使用基礎分析...\n`);
       return this.getFallbackAnalysis(changedFiles);
-    } finally {
-      // 確保 client 一定會被關閉
-      try {
-        await client.stop();
-      } catch (e) {
-        // 忽略關閉錯誤
-      }
     }
+    // 注意：不在此 stop() client，改由 close() 統一清理以便複用
   }
 
   /**
@@ -129,13 +143,25 @@ ${skillsSummary}
 > type 必須是以下之一：feat / fix / refactor / style / docs / test / chore / perf
 > **重要**：如果有新增任何功能、新增檔案、新增 API、新增組件，優先使用 **feat**
 
-## 📝 變更摘要
-[簡述這個 PR 的主要目的和影響範圍，2-3 句話]
+## � 摘要
+[1-2 句話說明這個 PR 做了什麼、解決了什麼問題]
 
-## 🎯 主要變更
-- [變更項目 1]
-- [變更項目 2]
-- [變更項目 3]
+---
+
+## ✏️ 做了什麼 (What)
+- [具體變更項目 1]
+- [具體變更項目 2]
+- [具體變更項目 3]
+
+## ❓ 為什麼 (Why)
+- [說明這個改動的背景或原因]
+- [使用者痛點或產品需求]
+
+## ⚙️ 怎麼做到的 (How)
+- [技術實作重點說明 1]
+- [技術實作重點說明 2]
+
+---
 
 ## 🔀 變更類型
 - [ ] ✨ 新功能 (feat)
@@ -159,53 +185,51 @@ ${skillsSummary}
 > 
 > **特別注意**: 如果 diff 中有「新增檔案」或「新增功能」，**務必勾選** ✨ 新功能 (feat)
 
-## 🧪 測試方法
-1. [具體的測試步驟 1]
-2. [具體的測試步驟 2]
-3. [具體的測試步驟 3]
-
-## 💥 Breaking Changes
-[如果有破壞性變更請詳細說明，沒有則填寫「無」]
-
-## 📌 注意事項
-[需要特別注意的事項]
-
-## 📸 截圖
-[如果是 UI 變更，提醒需要截圖]
+## 🎯 影響範圍
+- 影響模組：[列出受影響的頁面 / 模組 / API]
+- 無影響其他模組（或說明有影響的範圍）
 
 ---
 
-## ✅ 已套用規則總結
+## 🧪 測試項目
+- [ ] [測試項目 1]
+- [ ] [測試項目 2]
+- [ ] [測試項目 3]
 
-> 請根據上方提供的專案規範（React Best Practices 和 Frontend Guidelines），分析本次程式碼變更
+## 📸 截圖（UI 變更必填）
+[放截圖，非 UI 變更可填 N/A]
 
-### React Best Practices 規則
+---
 
-**✅ 已正確套用的規則**：
-[列出本次變更中有正確使用的規則]
+## ⚠️ 風險與注意事項
+**Risk Level**: \`LOW\` / \`MEDIUM\` / \`HIGH\`
 
-**❌ 需要改善的項目**：
-[列出本次變更中違反的規則]
+[說明潛在風險、破壞性變更（breaking changes）、需要特別小心的地方；沒有則填「無」]
 
-### Frontend Guidelines
+## 👀 Reviewer 重點
+- [請 reviewer 特別關注的邏輯或設計決策 1]
+- [請 reviewer 特別關注的邏輯或設計決策 2]
 
-[根據程式碼實際內容檢查相關項目]
+## 🔗 相關 Issue
+- Closes #[issue 編號]（沒有則填「無」）
 
 ---
 
 **規則**：
 1. PR 標題格式：type: 簡短描述（不超過 50 字）
 2. type 必須符合 Conventional Commits
-3. 變更摘要用 2-3 句話概括整體影響
-4. 全部使用繁體中文（台灣正體）
-5. 不要在開頭加引導語句
-6. 直接開始輸出 # [type]: [標題]
+3. 摘要用 1-2 句話說明目的，不要流水帳
+4. Why 說明背景原因，How 說明技術手段，兩者不可混淆
+5. 全部使用繁體中文（台灣正體）
+6. 不要在開頭加引導語句，直接輸出 # [type]: [標題]
 7. **變更類型判斷必須準確**：
    - 檢查 diff 中是否有 "new file mode" 或大量 "+++" 行（表示新增檔案）
    - 檢查 commit 訊息是否包含「新增」、「add」、「feat」等關鍵字
-   - 檢查主要變更列表，如果提到「新增 xxx」就必須勾選 ✨ 新功能 (feat)
    - 新增配置檔、新增組件、新增 API、新增功能都算 feat
    - 一個 PR 可以同時是多種類型（如：feat + refactor + chore）
+8. **Risk Level 判斷**：HIGH = 影響付款/登入/資料寫入核心流程；MEDIUM = 影響現有功能但有降級保護；LOW = 新增功能或純重構
+9. **Reviewer 重點**：列出最值得仔細看的 1-3 個地方（核心演算法、架構決策、潛在邊界條件）
+10. **相關 Issue**：若 commit 訊息含 #issue 編號或 closes/fixes/resolves，自動帶入；否則填「無」
 
 ---
 

package/src/pr-modules/core/workflow.js

@@ -32,70 +32,75 @@ export class PRWorkflow {
    * 執行完整工作流程
    */
   async execute() {
-    // 0. 確認 gh CLI 已登入（預覽模式可跳過）
-    if (!this.config.preview) {
-      const auth = this.github.checkAuth();
-      if (!auth.authenticated) {
-        log.error('GitHub CLI 未登入，請先執行: gh auth login');
-        throw new Error('GitHub CLI 未登入');
+    try {
+      // 0. 確認 gh CLI 已登入（預覽模式可跳過）
+      if (!this.config.preview) {
+        const auth = this.github.checkAuth();
+        if (!auth.authenticated) {
+          log.error('GitHub CLI 未登入，請先執行: gh auth login');
+          throw new Error('GitHub CLI 未登入');
+        }
       }
-    }
 
-    // 1. 驗證環境和分支
-    const { baseBranch, headBranch } = await this.detectAndValidateBranches();
+      // 1. 驗證環境和分支
+      const { baseBranch, headBranch } = await this.detectAndValidateBranches();
 
-    // 2. 檢查是否有變更
-    await this.validateChanges(baseBranch, headBranch);
+      // 2. 檢查是否有變更
+      await this.validateChanges(baseBranch, headBranch);
 
-    // 3. 推送到遠端（預覽模式跳過）
-    if (!this.config.preview) {
-      await this.pushToRemote(headBranch);
-    }
+      // 3. 推送到遠端（預覽模式跳過）
+      if (!this.config.preview) {
+        await this.pushToRemote(headBranch);
+      }
 
-    // 4. 收集變更資訊
-    const changeData = this.collectChangeData(baseBranch, headBranch);
+      // 4. 收集變更資訊
+      const changeData = this.collectChangeData(baseBranch, headBranch);
 
-    // 5. AI 分析和生成 PR 內容
-    const prContent = await this.generatePRContent(changeData);
+      // 5. AI 分析和生成 PR 內容
+      const prContent = await this.generatePRContent(changeData);
 
-    // 6. 顯示預覽
-    this.displayPreview(prContent, changeData.stats);
+      // 6. 顯示預覽
+      this.displayPreview(prContent, changeData.stats);
 
-    // 預覽模式：僅顯示不創建
-    if (this.config.preview) {
-      log.info('預覽模式：未創建 PR');
-      return;
-    }
+      // 預覽模式：僅顯示不創建
+      if (this.config.preview) {
+        log.info('預覽模式：未創建 PR');
+        return;
+      }
 
-    // 7. 選擇 Reviewers
-    const reviewers = await this.selectReviewers(changeData);
+      // 7. 選擇 Reviewers
+      const reviewers = await this.selectReviewers(changeData);
 
-    // 8. 確認創建
-    if (!this.config.noConfirm) {
-      const confirmed = await this.askConfirmation('是否創建此 Pull Request?');
-      if (!confirmed) {
-        log.info('已取消創建 PR');
-        return;
+      // 8. 確認創建
+      if (!this.config.noConfirm) {
+        const confirmed = await this.askConfirmation('是否創建此 Pull Request?');
+        if (!confirmed) {
+          log.info('已取消創建 PR');
+          return;
+        }
       }
-    }
 
-    // 9. 創建 PR
-    const prUrl = await this.createPR(prContent, baseBranch, headBranch, reviewers);
+      // 9. 創建 PR
+      const prUrl = await this.createPR(prContent, baseBranch, headBranch, reviewers);
 
-    // 10. 添加 Labels（如果啟用）
-    if (this.config.github.autoLabels === true && prUrl) {
-      try {
-        const prNumber = prUrl.split('/').pop();
-        await this.addLabels(prNumber, {
-          ...prContent,
-          stats: changeData.stats,
-        });
-      } catch (error) {
-        log.warning('無法自動添加 Labels: ' + error.message);
+      // 10. 添加 Labels（如果啟用）
+      if (this.config.github.autoLabels === true && prUrl) {
+        try {
+          const prNumber = prUrl.split('/').pop();
+          await this.addLabels(prNumber, {
+            ...prContent,
+            stats: changeData.stats,
+          });
+        } catch (error) {
+          log.warning('無法自動添加 Labels: ' + error.message);
+        }
       }
-    }
 
-    this.logger.success('完成！');
+      this.logger.success('完成！');
+    } finally {
+      // 確保 AI 子程序一定被釋放
+      await this.ai.close();
+    }
   }
 
   /**