ai-git-tools 2.1.12 → 2.1.13

package/package.json

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

package/src/core/ai-client.js

@@ -6,6 +6,49 @@
 import { CopilotClient, approveAll } from '@github/copilot-sdk';
 import { toTraditionalChinese } from '../utils/traditional-chinese.js';
 
+function extractJsonCandidates(content) {
+  const candidates = [];
+
+  for (let start = 0; start < content.length; start += 1) {
+    if (content[start] !== '{' && content[start] !== '[') continue;
+
+    const expectedClosers = [];
+    let inString = false;
+    let escaped = false;
+
+    for (let index = start; index < content.length; index += 1) {
+      const character = content[index];
+
+      if (inString) {
+        if (escaped) {
+          escaped = false;
+        } else if (character === '\\') {
+          escaped = true;
+        } else if (character === '"') {
+          inString = false;
+        }
+        continue;
+      }
+
+      if (character === '"') {
+        inString = true;
+      } else if (character === '{') {
+        expectedClosers.push('}');
+      } else if (character === '[') {
+        expectedClosers.push(']');
+      } else if (character === '}' || character === ']') {
+        if (expectedClosers.pop() !== character) break;
+        if (expectedClosers.length === 0) {
+          candidates.push(content.slice(start, index + 1));
+          break;
+        }
+      }
+    }
+  }
+
+  return candidates;
+}
+
 export class AIClient {
   /**
    * 判斷 Node.js 是否符合 Copilot SDK 的版本需求
@@ -105,7 +148,7 @@ export class AIClient {
    */
   static parseJSON(content) {
     // 移除可能的 markdown code block 標記
-    const jsonContent = content
+    const jsonContent = String(content || '')
       .replace(/```json\n?/g, '')
       .replace(/```\n?/g, '')
       .trim();
@@ -113,6 +156,14 @@ export class AIClient {
     try {
       return JSON.parse(jsonContent);
     } catch (error) {
+      for (const candidate of extractJsonCandidates(jsonContent)) {
+        try {
+          return JSON.parse(candidate);
+        } catch {
+          // 繼續嘗試下一個完整 JSON 區塊
+        }
+      }
+
       throw new Error(`無法解析 AI 回應為 JSON: ${error.message}`);
     }
   }

package/src/redmine/issue-analyzer.js

@@ -25,7 +25,7 @@ function normalizeFlowcharts(value) {
         : [];
 
   const usedIds = new Set();
-  return flowcharts
+  return flowcharts.slice(0, 1)
     .filter(flowchart => flowchart && typeof flowchart === 'object')
     .map((flowchart, index) => {
       const title = typeof flowchart.title === 'string' ? flowchart.title.trim() : '';
@@ -95,7 +95,9 @@ export function buildIssueAnalysisPrompt({ issue, evidence }) {
 - behaviorRules 必須使用 scenario/behavior 物件描述不同情境的行為。
 - impactScope 必須描述受到影響的功能、API、頁面或模組。
 - 只要 Issue 或 Git evidence 能推導出頁面、API、驗證或狀態之間的實際步驟，flowcharts 必須至少產生一張 Mermaid 流程圖，不得因為已有文字摘要而省略。
-- 只有在證據不足以描述任何流程時，flowcharts 才能回傳空陣列；每張流程圖必須有穩定 id、title、source 與 evidence。
+- 每個 Issue 只能產生一張流程圖；flowcharts 陣列最多只能有一個元素。若有多個候選流程，請合併成一張涵蓋完整流程的圖。
+- API path 不得使用大括號參數，例如 /items/{id} 必須改成 /items/:id；每張流程圖必須有穩定 id、title、source 與 evidence。
+- 只有在證據不足以描述任何流程時，flowcharts 才能回傳空陣列。
 - 沒有明確測試或驗證證據時，verification 必須是空陣列。
 - 不要產生 suggestedStatus、coverage 或 confidence，也不要決定 Redmine status。
 

package/src/redmine/redmine-formatters.js

@@ -41,7 +41,7 @@ function formatBehaviorRules(rules = []) {
 function formatFlowcharts(flowcharts = []) {
   if (!Array.isArray(flowcharts) || flowcharts.length === 0) return '';
 
-  return flowcharts
+  return flowcharts.slice(0, 1)
     .map((flowchart, index) => {
       const mermaid = formatMermaid(flowchart?.source);
       if (!mermaid) return '';
@@ -77,6 +77,18 @@ function normalizeFlowchartInput(flowcharts) {
   return [];
 }
 
+function normalizeApiPathPlaceholders(source) {
+  return source
+    .split('\n')
+    .map(line => {
+      const containsApiPath = /(?:\b(?:GET|POST|PUT|PATCH|DELETE|OPTIONS|HEAD)\b|https?:\/\/|\/[^\s/]+\/)/i.test(line);
+      return containsApiPath
+        ? line.replace(/\{([A-Za-z][A-Za-z0-9_-]*)\}/g, ':$1')
+        : line;
+    })
+    .join('\n');
+}
+
 function hasSameMermaid(description, mermaid) {
   const existing = description.match(/\{\{\s*mermaid\b[\s\S]*?\}\}/gi) || [];
   return existing.some(block => formatMermaid(block) === mermaid);
@@ -122,6 +134,7 @@ export function formatMermaid(source) {
   }
 
   cleanSource = cleanSource.replace(/^flowchart\s+TD\b/i, 'graph TD');
+  cleanSource = normalizeApiPathPlaceholders(cleanSource);
 
   if (
     cleanSource.includes('{{mermaid') ||