kodevu 0.1.37 → 0.1.39

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.37",
+  "version": "0.1.39",
   "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/logger.js

@@ -1,5 +1,6 @@
 import fs from "node:fs";
 import path from "node:path";
+import { formatDate } from "./utils.js";
 
 class Logger {
   constructor() {
@@ -18,7 +19,7 @@ class Logger {
         if (!fs.existsSync(config.logsDir)) {
           fs.mkdirSync(config.logsDir, { recursive: true });
         }
-        const date = new Date().toISOString().split("T")[0];
+        const date = formatDate(new Date()).split(" ")[0];
         this.logFile = path.join(config.logsDir, `run-${date}.log`);
         
         // Simple rotation: Clean up logs older than 7 days
@@ -57,7 +58,7 @@ class Logger {
   }
 
   _log(level, message) {
-    const timestamp = new Date().toISOString();
+    const timestamp = formatDate(new Date());
     const logLine = `[${timestamp}] [${level}] ${message}`;
 
     // Write to file

package/src/report-generator.js

@@ -1,3 +1,4 @@
+import { formatDate } from "./utils.js";
 export function getCoreReviewInstruction(lang) {
   const lowLang = (lang || "").toLowerCase();
   if (lowLang.startsWith("zh")) {
@@ -107,7 +108,8 @@ const LOCALIZED_PHRASES = {
     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."
+    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: "只读工作区上下文：",
@@ -123,7 +125,8 @@ const LOCALIZED_PHRASES = {
     commitMessage: "提交信息",
     diffNoteTruncated: "注意：Diff 已截断。原始：{originalLineCount} 行 / {originalCharCount} 字符。包含：{outputLineCount} 行 / {outputCharCount} 字符。",
     diffNoteFull: "注意：包含完整 Diff ({originalLineCount} 行 / {originalCharCount} 字符)。",
-    langRule: "--- 语言规则 ---\n你必须完全使用 {langName} 进行回复。不得使用其他语言进行解释或总结。"
+    langRule: "--- 语言规则 ---\n你必须完全使用 {langName} 进行回复。不得使用其他语言进行解释或总结。",
+    outputDirective: "--- 开始输出审查结果 ---\n请立即输出完整的代码审查结果，必须包含全部四个章节（变更总结、核心缺陷、改进建议、审查结论）。不要提问，不要确认收到指令，不要说准备好了，直接以审查内容开始输出。"
   },
   "zh-tw": {
     workspaceRoot: "唯讀工作區上下文：",
@@ -139,7 +142,8 @@ const LOCALIZED_PHRASES = {
     commitMessage: "提交信息",
     diffNoteTruncated: "注意：Diff 已截斷。原始：{originalLineCount} 行 / {originalCharCount} 字符。包含：{outputLineCount} 行 / {outputCharCount} 字符。",
     diffNoteFull: "注意：包含完整 Diff ({originalLineCount} 行 / {originalCharCount} 字符)。",
-    langRule: "--- 語言規則 ---\n你必須完全使用 {langName} 進行回覆。不得使用其他語言進行解釋或總結。"
+    langRule: "--- 語言規則 ---\n你必須完全使用 {langName} 進行回覆。不得使用其他語言進行解釋或總結。",
+    outputDirective: "--- 開始輸出審查結果 ---\n請立即輸出完整的代碼審查結果，必須包含全部四個章節（變更總結、核心缺陷、改進建議、審查結論）。不要提問，不要確認收到指令，不要說準備好了，直接以審查內容開始輸出。"
   }
 };
 
@@ -176,7 +180,7 @@ export function buildPrompt(config, backend, targetInfo, details, reviewDiffPayl
     `${getPhrase("repoType", lang)}: ${backend.displayName}`,
     `${getPhrase("changeId", lang)}: ${details.displayId}`,
     `${getPhrase("author", lang)}: ${details.author}`,
-    `${getPhrase("date", lang)}: ${details.date || "unknown"}`,
+    `${getPhrase("date", lang)}: ${formatDate(details.date) || "unknown"}`,
     `${getPhrase("changedFiles", lang)}:\n${fileList || "(none)"}`,
     `${getPhrase("commitMessage", lang)}:\n${details.message || "(empty)"}`
   ].join("\n");
@@ -205,7 +209,8 @@ export function buildPrompt(config, backend, targetInfo, details, reviewDiffPayl
       ? `${getPhrase("workspaceRoot", lang)} ${workspaceRoot}\n${getPhrase("besidesDiff", lang)}`
       : getPhrase("noWorkspace", lang),
     getPhrase("fileRefs", lang),
-    getPhrase("langRule", lang, { langName })
+    getPhrase("langRule", lang, { langName }),
+    getPhrase("outputDirective", lang)
   ];
 
   return sections.filter(Boolean).join("\n\n");
@@ -260,8 +265,8 @@ export function buildReport(config, backend, targetInfo, details, diffPayloads,
     `- Target: \`${targetInfo.targetDisplay || config.target}\``,
     `- Change ID: \`${details.displayId}\``,
     `- Author: \`${details.author}\``,
-    `- Commit Date: \`${details.date || "unknown"}\``,
-    `- Generated At: \`${new Date().toISOString()}\``,
+    `- Commit Date: \`${formatDate(details.date)}\``,
+    `- Generated At: \`${formatDate(new Date())}\``,
     `- Reviewer: \`${reviewer.displayName}\``,
     `- Reviewer Exit Code: \`${reviewerResult.code}\``,
     `- Reviewer Timed Out: \`${reviewerResult.timedOut ? "yes" : "no"}\``,
@@ -309,8 +314,8 @@ export function buildJsonReport(config, backend, targetInfo, details, diffPayloa
     target: targetInfo.targetDisplay || config.target,
     changeId: details.displayId,
     author: details.author,
-    commitDate: details.date || "unknown",
-    generatedAt: new Date().toISOString(),
+    commitDate: formatDate(details.date),
+    generatedAt: formatDate(new Date()),
     reviewer: {
       name: reviewer.displayName,
       exitCode: reviewerResult.code,

package/src/review-runner.js

@@ -5,7 +5,8 @@ import { logger } from "./logger.js";
 import {
   ensureDir,
   writeTextFile,
-  writeJsonFile
+  writeJsonFile,
+  formatDate
 } from "./utils.js";
 import {
   shouldWriteFormat,
@@ -42,7 +43,7 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
         repositoryType: backend.displayName,
         target: targetInfo.targetDisplay || config.target,
         changeId: details.displayId,
-        generatedAt: new Date().toISOString(),
+        generatedAt: formatDate(new Date()),
         skipped: true,
         message: "No file changes were captured for this change under the configured target."
       });

package/src/utils.js

@@ -32,3 +32,25 @@ export function countLines(text) {
   }
   return text.split(/\r?\n/).length;
 }
+
+export function formatDate(dateInput) {
+  if (!dateInput || dateInput === "unknown") return "unknown";
+  const d = new Date(dateInput);
+  if (isNaN(d.getTime())) return dateInput;
+
+  const offsetMinutes = -d.getTimezoneOffset();
+  const offsetHours = Math.floor(Math.abs(offsetMinutes) / 60);
+  const offsetMins = Math.abs(offsetMinutes) % 60;
+  const sign = offsetMinutes >= 0 ? "+" : "-";
+
+  const pad = (n) => String(n).padStart(2, "0");
+  const year = d.getFullYear();
+  const month = pad(d.getMonth() + 1);
+  const day = pad(d.getDate());
+  const hours = pad(d.getHours());
+  const minutes = pad(d.getMinutes());
+  const seconds = pad(d.getSeconds());
+  const offset = `${sign}${pad(offsetHours)}:${pad(offsetMins)}`;
+
+  return `${year}-${month}-${day} ${hours}:${minutes}:${seconds} ${offset}`;
+}