kodevu 0.1.45 → 0.1.46

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.45",
+  "version": "0.1.46",
   "license": "MIT",
   "type": "module",
   "description": "Poll SVN revisions or Git commits, send each change diff to a reviewer CLI, and write configurable review reports.",

package/src/config.js

@@ -66,14 +66,13 @@ function detectLanguage() {
     }
   })();
 
-  if (os.platform() === "win32" && intlLocale.startsWith("zh")) return "zh";
-  if (envLang.startsWith("zh")) return "zh";
-  if (envLang.startsWith("en")) return "en";
-  if (envLang) return envLang.split(/[._-]/)[0];
-  if (intlLocale.startsWith("zh")) return "zh";
-  if (intlLocale.startsWith("en")) return "en";
-  if (intlLocale) return intlLocale.split("-")[0];
-  return "en";
+  const locales = [envLang, intlLocale].filter(Boolean);
+  for (const loc of locales) {
+    if (loc.startsWith("zh")) return "zh";
+    if (loc.startsWith("en")) return "en";
+  }
+
+  return locales[0]?.split(/[._-]/)[0] || "en";
 }
 
 async function resolveAutoReviewers(debug) {
@@ -204,20 +203,9 @@ export async function resolveConfig(cliArgs = {}) {
   }
 
   // 2. Merge CLI Arguments
-  const cliMapping = {
-    target: "target",
-    reviewer: "reviewer",
-    prompt: "prompt",
-    lang: "lang",
-    rev: "rev",
-    last: "last",
-    outputDir: "outputDir",
-    outputFormats: "outputFormats"
-  };
-
-  for (const [cliKey, configKey] of Object.entries(cliMapping)) {
-    if (cliArgs[cliKey]) {
-      config[configKey] = cliArgs[cliKey];
+  for (const key of ["target", "reviewer", "prompt", "lang", "rev", "last", "outputDir", "outputFormats"]) {
+    if (cliArgs[key] !== undefined && cliArgs[key] !== "") {
+      config[key] = cliArgs[key];
     }
   }
 

package/src/git-client.js

@@ -61,8 +61,7 @@ export async function getTargetInfo(config) {
     requestedTargetPath,
     targetDisplay: requestedTargetPath,
     targetPathspec: relativeTargetPath ? relativeTargetPath : "",
-    branchName: branchResult.stdout || "HEAD",
-    stateKey: `git:${repoRootPath}:${relativeTargetPath || "."}`
+    branchName: branchResult.stdout || "HEAD"
   };
 }
 
@@ -81,29 +80,6 @@ export async function getLatestCommit(config, targetInfo) {
   return latestCommit;
 }
 
-export async function isValidCheckpoint(config, targetInfo, checkpointCommit, latestCommit) {
-  if (!checkpointCommit) {
-    return true;
-  }
-
-  const commitExists = await runGit(config, ["cat-file", "-e", `${checkpointCommit}^{commit}`], {
-    cwd: targetInfo.repoRootPath,
-    allowFailure: true,
-    trim: true
-  });
-
-  if (commitExists.code !== 0) {
-    return false;
-  }
-
-  const ancestorResult = await runGit(config, ["merge-base", "--is-ancestor", checkpointCommit, latestCommit], {
-    cwd: targetInfo.repoRootPath,
-    allowFailure: true,
-    trim: true
-  });
-
-  return ancestorResult.code === 0;
-}
 
 export async function resolveCommits(config, targetInfo, revSpec) {
   const result = await runGit(
@@ -124,24 +100,6 @@ export async function resolveCommits(config, targetInfo, revSpec) {
   return splitLines(result.stdout);
 }
 
-export async function getPendingCommits(config, targetInfo, startExclusive, endInclusive, limit) {
-  const args = ["rev-list", "--reverse"];
-
-  if (startExclusive) {
-    args.push(endInclusive, `^${startExclusive}`);
-  } else {
-    args.push(endInclusive);
-  }
-
-  args.push(...buildPathArgs(targetInfo));
-
-  const result = await runGit(config, args, {
-    cwd: targetInfo.repoRootPath,
-    trim: true
-  });
-
-  return splitLines(result.stdout).slice(0, limit);
-}
 
 export async function getLatestCommitIds(config, targetInfo, limit) {
   const result = await runGit(

package/src/report-generator.js

@@ -1,29 +1,48 @@
 import { formatDate } from "./utils.js";
-export function getCoreReviewInstruction(lang) {
-  const lowLang = (lang || "").toLowerCase();
-  if (lowLang.startsWith("zh")) {
-    if (lowLang === "zh-tw" || lowLang === "zh-hk") {
-      return `你是一位擁有 10 年以上經驗的高級軟體架構師和代碼審查專家。你的任務是對以下代碼變更進行嚴格且高質量的審查。
-你的目標是：
-1. 發現代碼中的 Bug、邏輯缺陷和潛在的回歸風險。
-2. 識別性能瓶頸、記憶體洩漏或不必要的計算。
-3. 檢查安全漏洞（如注入、越權、敏感信息洩露等）。
-4. 評估代碼的可維護性、可讀性和是否符合最佳實踐。
-5. 檢查是否涵蓋了必要的單元測試和邊界條件。
 
-你的輸出格式必須清晰，請使用以下 Markdown 結構：
-### 1. 變更總結 (Summary)
-簡要描述這次提交的主要目的和影響面。
-### 2. 核心缺陷 (Critical Issues)
-列出 Bug、安全隱患或會導致程序崩潰/邏輯錯誤的問題。請註明檔案名和行號。
-### 3. 改進建議 (Suggestions)
-列出關於代碼風格、性能優化或架構設計的改進點。
-### 4. 審查結論 (Conclusion)
-如果發現明顯缺陷，總結修復建議；如果未發現明顯缺陷，請說明「未發現明顯缺陷」並指出可能的殘留風險。
+const LOCALIZED_DATA = {
+  en: {
+    displayName: "English",
+    coreInstruction: `You are a Senior Software Engineer and Code Review Expert with over 10 years of experience. Your task is to perform a rigorous and high-quality review of the following code changes.
+Your goals are to:
+1. Identify bugs, logical flaws, and potential regression risks.
+2. Spot performance bottlenecks, memory leaks, or unnecessary computations.
+3. Check for security vulnerabilities (e.g., injection, authorization issues, sensitive data leaks).
+4. Evaluate maintainability, readability, and adherence to best practices.
+5. Verify coverage of necessary unit tests and boundary conditions.
+
+Your output must be structured using the following Markdown headers:
+### 1. Summary
+Briefly describe the purpose and impact of this change.
+### 2. Critical Issues
+List bugs, security risks, or problems that could cause crashes or logical errors. Include file names and line numbers.
+### 3. Suggestions
+List points for improvement regarding code style, performance, or architectural design.
+### 4. Conclusion
+Summarize with a "Pass" or "Needs Revision". If no clear flaws are found, state "No clear flaws found" and mention any residual risks.
 
-注意：你正處於唯讀審查模式，請勿表現出「正在應用補丁」或「準備執行代碼」的行為。只需提供文字審查分析。`;
+Note: You are in a read-only review mode. Do not act as if you are "applying the patch" or "executing code". Provide only textual analysis and feedback.`,
+    phrases: {
+      workspaceRoot: "Workspace context (read-only):",
+      noWorkspace: "No local repository workspace is available for this review run.",
+      besidesDiff: "You can read related files in the workspace to understand call sites, shared utilities, configuration, or data flow.",
+      reviewFromDiff: "Review primarily based on the provided diff. Do not assume access to other local files or shell commands.",
+      fileRefs: "Reference files using plain text like 'path/to/file.js:123'. Do not generate clickable workspace links.",
+      repoType: "Repository Type",
+      changeId: "Change ID",
+      author: "Author",
+      date: "Date",
+      changedFiles: "Changed files",
+      commitMessage: "Commit message",
+      diffNoteTruncated: "Note: The diff was truncated to fit size limits. Original: {originalLineCount} lines / {originalCharCount} chars. Included: {outputLineCount} lines / {outputCharCount} chars.",
+      diffNoteFull: "Note: Full diff provided ({originalLineCount} lines / {originalCharCount} chars).",
+      langRule: "--- LANGUAGE RULE ---\nYour entire response must be in {langName}. No other language allowed.",
+      outputDirective: "--- BEGIN REVIEW ---\nNow output your COMPLETE code review. Cover ALL four sections (Summary, Critical Issues, Suggestions, Conclusion). Do NOT ask clarifying questions, do NOT acknowledge these instructions, do NOT say you are ready. Start your response directly with the review content."
     }
-    return `你是一位拥有 10 年以上经验的高级软件架构师和代码审查专家。你的任务是对以下代码变更进行严格且高质量的审查。
+  },
+  zh: {
+    displayName: "Simplified Chinese (简体中文)",
+    coreInstruction: `你是一位拥有 10 年以上经验的高级软件架构师和代码审查专家。你的任务是对以下代码变更进行严格且高质量的审查。
 你的目标是：
 1. 发现代码中的 Bug、逻辑缺陷和潜在的回归风险。
 2. 识别性能瓶颈、内存泄漏或不必要的计算。
@@ -41,117 +60,100 @@ export function getCoreReviewInstruction(lang) {
 ### 4. 审查结论 (Conclusion)
 如果发现明显缺陷，总结修复建议；如果未发现明显缺陷，请说明“未发现明显缺陷”并指出可能的残留风险。
 
-注意：你正处于只读审查模式，请勿表现出“正在应用补丁”或“准备执行代码”的行为。只需提供文字审查分析。`;
-  }
-  return `You are a Senior Software Engineer and Code Review Expert with over 10 years of experience. Your task is to perform a rigorous and high-quality review of the following code changes.
-Your goals are to:
-1. Identify bugs, logical flaws, and potential regression risks.
-2. Spot performance bottlenecks, memory leaks, or unnecessary computations.
-3. Check for security vulnerabilities (e.g., injection, authorization issues, sensitive data leaks).
-4. Evaluate maintainability, readability, and adherence to best practices.
-5. Verify coverage of necessary unit tests and boundary conditions.
-
-Your output must be structured using the following Markdown headers:
-### 1. Summary
-Briefly describe the purpose and impact of this change.
-### 2. Critical Issues
-List bugs, security risks, or problems that could cause crashes or logical errors. Include file names and line numbers.
-### 3. Suggestions
-List points for improvement regarding code style, performance, or architectural design.
-### 4. Conclusion
-Summarize with a "Pass" or "Needs Revision". If no clear flaws are found, state "No clear flaws found" and mention any residual risks.
+注意：你正处于只读审查模式，请勿表现出“正在应用补丁”或“准备执行代码”的行为。只需提供文字审查分析。`,
+    phrases: {
+      workspaceRoot: "只读工作区上下文：",
+      noWorkspace: "此审查运行没有可用的本地仓库工作区。",
+      besidesDiff: "你可以阅读工作区中的其他文件以了解调用点、工具类、配置或数据流。",
+      reviewFromDiff: "主要根据提供的 Diff 进行审查。不要假设可以访问其他文件或执行 Shell 命令。",
+      fileRefs: "使用纯文本引用文件，如 'path/to/file.js:123'。不要生成可点击的链接。",
+      repoType: "仓库类型",
+      changeId: "变更 ID",
+      author: "作者",
+      date: "日期",
+      changedFiles: "已变更文件",
+      commitMessage: "提交信息",
+      diffNoteTruncated: "注意：Diff 已截断。原始：{originalLineCount} 行 / {originalCharCount} 字符。包含：{outputLineCount} 行 / {outputCharCount} 字符。",
+      diffNoteFull: "注意：包含完整 Diff ({originalLineCount} 行 / {originalCharCount} 字符)。",
+      langRule: "--- 语言规则 ---\n你必须完全使用 {langName} 进行回复。不得使用其他语言进行解释或总结。",
+      outputDirective: "--- 开始输出审查结果 ---\n请立即输出完整的代码审查结果，必须包含全部四个章节（变更总结、核心缺陷、改进建议、审查结论）。不要提问，不要确认收到指令，不要说准备好了，直接以审查内容开始输出。"
+    }
+  },
+  "zh-tw": {
+    displayName: "Traditional Chinese (繁體中文)",
+    coreInstruction: `你是一位擁有 10 年以上經驗的高級軟體架構師和代碼審查專家。你的任務是對以下代碼變更進行嚴格且高質量的審查。
+你的目標是：
+1. 發現代碼中的 Bug、邏輯缺陷和潛在的回歸風險。
+2. 識別性能瓶頸、記憶體洩漏或不必要的計算。
+3. 檢查安全漏洞（如注入、越權、敏感信息洩露等）。
+4. 評估代碼的可維護性、可讀性和是否符合最佳實踐。
+5. 檢查是否涵蓋了必要的單元測試和邊界條件。
 
-Note: You are in a read-only review mode. Do not act as if you are "applying the patch" or "executing code". Provide only textual analysis and feedback.`;
-}
+你的輸出格式必須清晰，請使用以下 Markdown 結構：
+### 1. 變更總結 (Summary)
+簡要描述這次提交的主要目的和影響面。
+### 2. 核心缺陷 (Critical Issues)
+列出 Bug、安全隱患或會導致程序崩潰/邏輯錯誤的問題。請註明檔案名和行號。
+### 3. 改進建議 (Suggestions)
+列出關於代碼風格、性能優化或架構設計的改進點。
+### 4. 審查結論 (Conclusion)
+如果發現明顯缺陷，總結修復建議；如果未發現明顯缺陷，請說明「未發現明顯缺陷」並指出可能的殘留風險。
 
-export function getReviewWorkspaceRoot(config, backend, targetInfo) {
-  if (backend.kind === "git" && targetInfo.repoRootPath) {
-    return targetInfo.repoRootPath;
+注意：你正處於唯讀審查模式，請勿表現出「正在應用補丁」或「準備執行代碼」的行為。只需提供文字審查分析。`,
+    phrases: {
+      workspaceRoot: "唯讀工作區上下文：",
+      noWorkspace: "此審查運行沒有可用的本地倉庫工作區。",
+      besidesDiff: "你可以閱讀工作區中的其他文件以了解調用點、工具類、配置或資料流。",
+      reviewFromDiff: "主要根據提供的 Diff 進行審查。不要假設可以訪問其他文件或執行 Shell 命令。",
+      fileRefs: "使用純文本引用文件，如 'path/to/file.js:123'。不要生成可點擊的連結。",
+      repoType: "倉庫類型",
+      changeId: "變更 ID",
+      author: "作者",
+      date: "日期",
+      changedFiles: "已變更文件",
+      commitMessage: "提交信息",
+      diffNoteTruncated: "注意：Diff 已截斷。原始：{originalLineCount} 行 / {originalCharCount} 字符。包含：{outputLineCount} 行 / {outputCharCount} 字符。",
+      diffNoteFull: "注意：包含完整 Diff ({originalLineCount} 行 / {originalCharCount} 字符)。",
+      langRule: "--- 語言規則 ---\n你必須完全使用 {langName} 進行回覆。不得使用其他語言進行解釋或總結。",
+      outputDirective: "--- 開始輸出審查結果 ---\n請立即輸出完整的代碼審查結果，必須包含全部四個章節（變更總結、核心缺陷、改進建議、審查結論）。不要提問，不要確認收到指令，不要說準備好了，直接以審查內容開始輸出。"
+    }
   }
+};
 
-  if (backend.kind === "svn" && targetInfo.workingCopyPath) {
-    return targetInfo.workingCopyPath;
+function getLangKey(lang) {
+  const lowArg = (lang || "en").toLowerCase();
+  if (lowArg.startsWith("zh")) {
+    return (lowArg === "zh-tw" || lowArg === "zh-hk") ? "zh-tw" : "zh";
   }
+  return "en";
+}
+
+export function getCoreReviewInstruction(lang) {
+  return LOCALIZED_DATA[getLangKey(lang)].coreInstruction;
+}
 
-  return config.baseDir;
+export function getReviewWorkspaceRoot(config, backend, targetInfo) {
+  return (backend.kind === "git" ? targetInfo.repoRootPath : targetInfo.workingCopyPath) || config.baseDir;
 }
 
 function getLanguageDisplayName(lang) {
   if (!lang) return "English";
   const low = lang.toLowerCase();
-  if (low.startsWith("zh")) {
-    if (low === "zh-tw" || low === "zh-hk") return "Traditional Chinese (繁體中文)";
-    return "Simplified Chinese (简体中文)";
-  }
-  if (low === "jp" || low.startsWith("ja")) return "Japanese (日本語)";
-  if (low === "kr" || low.startsWith("ko")) return "Korean (한국어)";
-  if (low === "fr") return "French (Français)";
-  if (low === "de") return "German (Deutsch)";
-  if (low === "es") return "Spanish (Español)";
-  if (low === "it") return "Italian (Italiano)";
-  if (low === "ru") return "Russian (Русский)";
-  return lang;
-}
+  const data = LOCALIZED_DATA[getLangKey(lang)];
+  if (data && low.startsWith("zh")) return data.displayName;
 
-const LOCALIZED_PHRASES = {
-  en: {
-    workspaceRoot: "Workspace context (read-only):",
-    noWorkspace: "No local repository workspace is available for this review run.",
-    besidesDiff: "You can read related files in the workspace to understand call sites, shared utilities, configuration, or data flow.",
-    reviewFromDiff: "Review primarily based on the provided diff. Do not assume access to other local files or shell commands.",
-    fileRefs: "Reference files using plain text like 'path/to/file.js:123'. Do not generate clickable workspace links.",
-    repoType: "Repository Type",
-    changeId: "Change ID",
-    author: "Author",
-    date: "Date",
-    changedFiles: "Changed files",
-    commitMessage: "Commit message",
-    diffNoteTruncated: "Note: The diff was truncated to fit size limits. Original: {originalLineCount} lines / {originalCharCount} chars. Included: {outputLineCount} lines / {outputCharCount} chars.",
-    diffNoteFull: "Note: Full diff provided ({originalLineCount} lines / {originalCharCount} chars).",
-    langRule: "--- LANGUAGE RULE ---\nYour entire response must be in {langName}. No other language allowed.",
-    outputDirective: "--- BEGIN REVIEW ---\nNow output your COMPLETE code review. Cover ALL four sections (Summary, Critical Issues, Suggestions, Conclusion). Do NOT ask clarifying questions, do NOT acknowledge these instructions, do NOT say you are ready. Start your response directly with the review content."
-  },
-  zh: {
-    workspaceRoot: "只读工作区上下文：",
-    noWorkspace: "此审查运行没有可用的本地仓库工作区。",
-    besidesDiff: "你可以阅读工作区中的其他文件以了解调用点、工具类、配置或数据流。",
-    reviewFromDiff: "主要根据提供的 Diff 进行审查。不要假设可以访问其他文件或执行 Shell 命令。",
-    fileRefs: "使用纯文本引用文件，如 'path/to/file.js:123'。不要生成可点击的链接。",
-    repoType: "仓库类型",
-    changeId: "变更 ID",
-    author: "作者",
-    date: "日期",
-    changedFiles: "已变更文件",
-    commitMessage: "提交信息",
-    diffNoteTruncated: "注意：Diff 已截断。原始：{originalLineCount} 行 / {originalCharCount} 字符。包含：{outputLineCount} 行 / {outputCharCount} 字符。",
-    diffNoteFull: "注意：包含完整 Diff ({originalLineCount} 行 / {originalCharCount} 字符)。",
-    langRule: "--- 语言规则 ---\n你必须完全使用 {langName} 进行回复。不得使用其他语言进行解释或总结。",
-    outputDirective: "--- 开始输出审查结果 ---\n请立即输出完整的代码审查结果，必须包含全部四个章节（变更总结、核心缺陷、改进建议、审查结论）。不要提问，不要确认收到指令，不要说准备好了，直接以审查内容开始输出。"
-  },
-  "zh-tw": {
-    workspaceRoot: "唯讀工作區上下文：",
-    noWorkspace: "此審查運行沒有可用的本地倉庫工作區。",
-    besidesDiff: "你可以閱讀工作區中的其他文件以了解調用點、工具類、配置或資料流。",
-    reviewFromDiff: "主要根據提供的 Diff 進行審查。不要假設可以訪問其他文件或執行 Shell 命令。",
-    fileRefs: "使用純文本引用文件，如 'path/to/file.js:123'。不要生成可點擊的連結。",
-    repoType: "倉庫類型",
-    changeId: "變更 ID",
-    author: "作者",
-    date: "日期",
-    changedFiles: "已變更文件",
-    commitMessage: "提交信息",
-    diffNoteTruncated: "注意：Diff 已截斷。原始：{originalLineCount} 行 / {originalCharCount} 字符。包含：{outputLineCount} 行 / {outputCharCount} 字符。",
-    diffNoteFull: "注意：包含完整 Diff ({originalLineCount} 行 / {originalCharCount} 字符)。",
-    langRule: "--- 語言規則 ---\n你必須完全使用 {langName} 進行回覆。不得使用其他語言進行解釋或總結。",
-    outputDirective: "--- 開始輸出審查結果 ---\n請立即輸出完整的代碼審查結果，必須包含全部四個章節（變更總結、核心缺陷、改進建議、審查結論）。不要提問，不要確認收到指令，不要說準備好了，直接以審查內容開始輸出。"
-  }
-};
+  const extras = {
+    jp: "Japanese (日本語)", ja: "Japanese (日本語)",
+    kr: "Korean (한국어)", ko: "Korean (한국어)",
+    fr: "French (Français)", de: "German (Deutsch)",
+    es: "Spanish (Español)", it: "Italian (Italiano)",
+    ru: "Russian (Русский)"
+  };
+  return extras[low] || extras[low.split("-")[0]] || lang;
+}
 
 function getPhrase(key, lang, placeholders = {}) {
-  const lowLang = (lang || "en").toLowerCase();
-  const langKey = lowLang.startsWith("zh") ? (lowLang === "zh-tw" || lowLang === "zh-hk" ? "zh-tw" : "zh") : "en";
-  let phrase = LOCALIZED_PHRASES[langKey][key] || LOCALIZED_PHRASES.en[key];
-
+  let phrase = LOCALIZED_DATA[getLangKey(lang)].phrases[key] || LOCALIZED_DATA.en.phrases[key];
   for (const [k, v] of Object.entries(placeholders)) {
     phrase = phrase.replace(`{${k}}`, v);
   }

package/src/svn-client.js

@@ -57,7 +57,6 @@ export async function getTargetInfo(config) {
     targetUrl,
     targetRepoPath,
     targetDisplay: config.target,
-    stateKey: `svn:${targetUrl}`,
     workingCopyPath:
       entry["wc-info"]?.["wcroot-abspath"] || (path.isAbsolute(config.target) ? config.target : null)
   };
@@ -84,33 +83,6 @@ export async function getLatestRevision(config, targetInfo) {
   return revision;
 }
 
-export async function getPendingRevisions(config, targetInfo, startExclusive, endInclusive, limit) {
-  const startRevision = Number.isInteger(startExclusive) ? startExclusive + 1 : 1;
-
-  if (endInclusive < startRevision) {
-    return [];
-  }
-
-  const result = await runCommand(
-    SVN_COMMAND,
-    [
-      "log",
-      "--xml",
-      "--quiet",
-      "-r",
-      `${startRevision}:${endInclusive}`,
-      getRemoteTarget(targetInfo, config)
-    ],
-    { encoding: COMMAND_ENCODING, trim: true, debug: config.debug }
-  );
-  const parsed = xmlParser.parse(result.stdout);
-
-  return asArray(parsed?.log?.logentry)
-    .map((entry) => Number(entry?.revision))
-    .filter((revision) => Number.isInteger(revision))
-    .sort((left, right) => left - right)
-    .slice(0, limit);
-}
 
 export async function getLatestRevisionIds(config, targetInfo, limit) {
   const result = await runCommand(

package/src/token-usage.js

@@ -6,7 +6,7 @@ export function estimateTokenCount(text) {
   return Math.ceil(text.length / 4);
 }
 
-export function parseGeminiTokenUsage(stderr) {
+export function parseTokenUsage(stderr) {
   if (!stderr) {
     return null;
   }
@@ -32,67 +32,8 @@ export function parseGeminiTokenUsage(stderr) {
   return { inputTokens, outputTokens, totalTokens };
 }
 
-export function parseCodexTokenUsage(stderr) {
-  if (!stderr) {
-    return null;
-  }
-
-  const patterns = [
-    /input[_ ]tokens?\s*[:=]\s*(\d+)/i,
-    /output[_ ]tokens?\s*[:=]\s*(\d+)/i,
-    /total[_ ]tokens?\s*[:=]\s*(\d+)/i
-  ];
-
-  const inputMatch = stderr.match(patterns[0]);
-  const outputMatch = stderr.match(patterns[1]);
-  const totalMatch = stderr.match(patterns[2]);
-
-  if (!inputMatch && !outputMatch && !totalMatch) {
-    return null;
-  }
-
-  const inputTokens = inputMatch ? Number(inputMatch[1]) : 0;
-  const outputTokens = outputMatch ? Number(outputMatch[1]) : 0;
-  const totalTokens = totalMatch ? Number(totalMatch[1]) : inputTokens + outputTokens;
-
-  return { inputTokens, outputTokens, totalTokens };
-}
-
-export function parseCopilotTokenUsage(stderr) {
-  if (!stderr) {
-    return null;
-  }
-
-  const patterns = [
-    /input[_ ]tokens?\s*[:=]\s*(\d+)/i,
-    /output[_ ]tokens?\s*[:=]\s*(\d+)/i,
-    /total[_ ]tokens?\s*[:=]\s*(\d+)/i
-  ];
-
-  const inputMatch = stderr.match(patterns[0]);
-  const outputMatch = stderr.match(patterns[1]);
-  const totalMatch = stderr.match(patterns[2]);
-
-  if (!inputMatch && !outputMatch && !totalMatch) {
-    return null;
-  }
-
-  const inputTokens = inputMatch ? Number(inputMatch[1]) : 0;
-  const outputTokens = outputMatch ? Number(outputMatch[1]) : 0;
-  const totalTokens = totalMatch ? Number(totalMatch[1]) : inputTokens + outputTokens;
-
-  return { inputTokens, outputTokens, totalTokens };
-}
-
-export const TOKEN_PARSERS = {
-  gemini: parseGeminiTokenUsage,
-  codex: parseCodexTokenUsage,
-  copilot: parseCopilotTokenUsage
-};
-
 export function resolveTokenUsage(reviewerName, stderr, promptText, diffText, responseText) {
-  const parseFn = TOKEN_PARSERS[reviewerName] || parseCopilotTokenUsage;
-  const parsed = parseFn(stderr);
+  const parsed = parseTokenUsage(stderr);
 
   if (parsed && parsed.totalTokens > 0) {
     return { ...parsed, source: "reviewer" };

package/src/utils.js

@@ -54,3 +54,16 @@ export function formatDate(dateInput) {
 
   return `${year}-${month}-${day} ${hours}:${minutes}:${seconds} ${offset}`;
 }
+export function getTimestampPrefix() {
+  const now = new Date();
+  const pad = (n) => String(n).padStart(2, "0");
+  return (
+    now.getFullYear() +
+    pad(now.getMonth() + 1) +
+    pad(now.getDate()) +
+    "-" +
+    pad(now.getHours()) +
+    pad(now.getMinutes()) +
+    pad(now.getSeconds())
+  );
+}

package/src/vcs-client.js

@@ -1,21 +1,12 @@
-import fs from "node:fs/promises";
 import path from "node:path";
 import * as gitClient from "./git-client.js";
 import * as svnClient from "./svn-client.js";
+import { pathExists, getTimestampPrefix } from "./utils.js";
 
 function isLikelyUrl(value) {
   return /^[a-z][a-z0-9+.-]*:\/\//i.test(value);
 }
 
-async function pathExists(targetPath) {
-  try {
-    await fs.access(targetPath);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
 function createSvnBackend() {
   return {
     kind: "svn",
@@ -25,15 +16,7 @@ function createSvnBackend() {
       return `r${revision}`;
     },
     getReportFileName(revision) {
-      const now = new Date();
-      const datePrefix = now.getFullYear() +
-        String(now.getMonth() + 1).padStart(2, '0') +
-        String(now.getDate()).padStart(2, '0') +
-        '-' +
-        String(now.getHours()).padStart(2, '0') +
-        String(now.getMinutes()).padStart(2, '0') +
-        String(now.getSeconds()).padStart(2, '0');
-      return `${datePrefix}-svn-r${revision}.md`;
+      return `${getTimestampPrefix()}-svn-r${revision}.md`;
     },
 
     async resolveChangeIds(config, targetInfo, revString) {
@@ -76,15 +59,7 @@ function createGitBackend() {
       return commitHash.slice(0, 12);
     },
     getReportFileName(commitHash) {
-      const now = new Date();
-      const datePrefix = now.getFullYear() +
-        String(now.getMonth() + 1).padStart(2, '0') +
-        String(now.getDate()).padStart(2, '0') +
-        '-' +
-        String(now.getHours()).padStart(2, '0') +
-        String(now.getMinutes()).padStart(2, '0') +
-        String(now.getSeconds()).padStart(2, '0');
-      return `${datePrefix}-git-${commitHash.slice(0, 12)}.md`;
+      return `${getTimestampPrefix()}-git-${commitHash.slice(0, 12)}.md`;
     },
 
     async resolveChangeIds(config, targetInfo, revString) {