kodevu 0.1.37 → 0.1.38

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.37",
+  "version": "0.1.38",
   "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")) {
@@ -176,7 +177,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");
@@ -260,8 +261,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 +310,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}`;
+}