sales-frontend-gemini-cli 0.4.1 → 0.4.2

package/dist/pr-review/review-one-by-one.js

@@ -1,12 +1,13 @@
 #!/usr/bin/env node
 import { exec, execSync } from 'child_process';
-import fs5 from 'fs';
-import util from 'util';
+import fs from 'fs';
+import util, { inspect } from 'util';
 import path from 'path';
 import readline from 'readline';
 import { fileURLToPath } from 'url';
 
 var __dirname = path.dirname(fileURLToPath(import.meta.url));
+var traceMessages = [];
 var rulesPath = path.resolve(__dirname, "../../src/common/rules/review-rules.md");
 var namingRulesPath = path.resolve(__dirname, "../../src/common/rules/naming-rule.md");
 var codingConventionRulesPath = path.resolve(__dirname, "../../src/common/rules/coding-convention.md");
@@ -29,66 +30,226 @@ var ignoreList = [
   ".review-report/"
   // 생성되는 리포트 폴더도 제외
 ];
-function parseServiceFromArgs(args4 = process.argv.slice(2)) {
-  const serviceIndex = args4.indexOf("--service");
-  const rawService = serviceIndex !== -1 ? args4[serviceIndex + 1] : "";
-  if (!rawService) {
-    return "";
-  }
-  const normalizedService = rawService.toLowerCase();
-  if (AIServices.includes(normalizedService)) {
-    return normalizedService;
-  }
-  console.error(
-    `\u274C \uC9C0\uC6D0\uD558\uC9C0 \uC54A\uB294 \uC11C\uBE44\uC2A4\uC785\uB2C8\uB2E4: ${rawService}. \uC0AC\uC6A9 \uAC00\uB2A5 \uAC12: ${AIServices.join(", ")} (\uC608: --service codex)`
-  );
-  process.exit(1);
-}
 function isTestMode(args4 = process.argv.slice(2)) {
   return args4.includes("--test");
 }
+function clearTraceMessages() {
+  traceMessages.length = 0;
+}
+function getTraceMessages() {
+  return [...traceMessages];
+}
 function createTraceLogger(scope, args4 = process.argv.slice(2)) {
   const enabled = isTestMode(args4);
   return (step, detail) => {
+    const timestamp = (/* @__PURE__ */ new Date()).toISOString();
+    const message = `[${timestamp}][TRACE][${scope}] ${step}${detail ? ` | ${detail}` : ""}`;
+    traceMessages.push(message);
     if (!enabled) {
       return;
     }
-    console.log(`[TRACE][${scope}] ${step}${detail ? ` | ${detail}` : ""}`);
+    console.log(message);
+  };
+}
+var helperTrace = createTraceLogger("helper");
+function getTimestampParts(now = /* @__PURE__ */ new Date()) {
+  return {
+    YYYY: now.getFullYear(),
+    MM: String(now.getMonth() + 1).padStart(2, "0"),
+    DD: String(now.getDate()).padStart(2, "0"),
+    HH: String(now.getHours()).padStart(2, "0"),
+    mm: String(now.getMinutes()).padStart(2, "0"),
+    ss: String(now.getSeconds()).padStart(2, "0")
   };
 }
+function getHumanReadableNowString(now = /* @__PURE__ */ new Date()) {
+  const { YYYY, MM, DD, HH, mm, ss } = getTimestampParts(now);
+  return `${YYYY}-${MM}-${DD} ${HH}:${mm}:${ss}`;
+}
+function stringifyUnknown(value) {
+  if (value === void 0 || value === null) {
+    return "";
+  }
+  if (typeof value === "string") {
+    return value;
+  }
+  if (Buffer.isBuffer(value)) {
+    return value.toString();
+  }
+  if (value instanceof Error) {
+    return value.stack || value.message;
+  }
+  return inspect(value, { depth: 5, breakLength: 120 });
+}
+function getErrorSummary(error) {
+  if (error instanceof Error) {
+    return `${error.name}: ${error.message}`;
+  }
+  return stringifyUnknown(error) || "Unknown error";
+}
+function serializeError(error) {
+  const serialized = {
+    summary: getErrorSummary(error)
+  };
+  if (error instanceof Error) {
+    serialized.name = error.name;
+    serialized.message = error.message;
+    serialized.stack = error.stack;
+  } else {
+    serialized.value = stringifyUnknown(error);
+  }
+  if (error && typeof error === "object") {
+    const errorLike = error;
+    const extraKeys = ["code", "errno", "syscall", "path", "cmd", "status", "signal", "spawnargs"];
+    extraKeys.forEach((key) => {
+      if (errorLike[key] !== void 0) {
+        serialized[key] = errorLike[key];
+      }
+    });
+    const stdout = stringifyUnknown(errorLike.stdout);
+    if (stdout) {
+      serialized.stdout = stdout;
+    }
+    const stderr = stringifyUnknown(errorLike.stderr);
+    if (stderr) {
+      serialized.stderr = stderr;
+    }
+    const cause = stringifyUnknown(errorLike.cause);
+    if (cause) {
+      serialized.cause = cause;
+    }
+  }
+  return serialized;
+}
 function getNextFilePath(dir, baseName, extension) {
   let counter = 1;
   while (true) {
     const filePath = path.join(dir, `${baseName}-${counter}${extension}`);
-    if (!fs5.existsSync(filePath)) {
+    if (!fs.existsSync(filePath)) {
       return filePath;
     }
     counter++;
   }
 }
+function getAvailableFilePath(dir, baseName, extension) {
+  const firstFilePath = path.join(dir, `${baseName}${extension}`);
+  if (!fs.existsSync(firstFilePath)) {
+    return firstFilePath;
+  }
+  return getNextFilePath(dir, baseName, extension);
+}
 function deleteFile(filePath) {
-  if (fs5.existsSync(filePath)) {
-    fs5.unlinkSync(filePath);
+  if (fs.existsSync(filePath)) {
+    fs.unlinkSync(filePath);
   }
 }
 function deleteTempDiff() {
   deleteFile(tempDiffPath);
 }
 function createReportDirectory() {
-  if (!fs5.existsSync(REPORT_DIR)) {
-    fs5.mkdirSync(REPORT_DIR, { recursive: true });
-  }
-}
-function getNowString() {
-  const now = /* @__PURE__ */ new Date();
-  const YYYY = now.getFullYear();
-  const MM = String(now.getMonth() + 1).padStart(2, "0");
-  const DD = String(now.getDate()).padStart(2, "0");
-  const HH = String(now.getHours()).padStart(2, "0");
-  const mm = String(now.getMinutes()).padStart(2, "0");
-  const ss = String(now.getSeconds()).padStart(2, "0");
+  if (!fs.existsSync(REPORT_DIR)) {
+    fs.mkdirSync(REPORT_DIR, { recursive: true });
+  }
+}
+function getNowString(now = /* @__PURE__ */ new Date()) {
+  const { YYYY, MM, DD, HH, mm, ss } = getTimestampParts(now);
   return `${YYYY}-${MM}-${DD}_${HH}-${mm}-${ss}`;
 }
+function getErrorLogTimestamp(now = /* @__PURE__ */ new Date()) {
+  const { YYYY, MM, DD, HH, mm, ss } = getTimestampParts(now);
+  return `${YYYY}-${MM}-${DD}-${HH}\uC2DC-${mm}\uBD84-${ss}\uCD08`;
+}
+function writeErrorReport(error, options = {}) {
+  try {
+    const now = /* @__PURE__ */ new Date();
+    helperTrace("error-report:write:start", options.scope || "unknown");
+    createReportDirectory();
+    const reportPath = getAvailableFilePath(REPORT_DIR, `error-log-${getErrorLogTimestamp(now)}`, ".md");
+    const serializedError = serializeError(error);
+    const traceSnapshot = options.traceMessages ?? getTraceMessages();
+    const extraSections = options.extraSections || [];
+    const report = `# Error Log
+
+- \uBC1C\uC0DD \uC2DC\uAC01: ${getHumanReadableNowString(now)}
+- Scope: \`${options.scope || "unknown"}\`
+- \uC791\uC5C5 \uACBD\uB85C: \`${process.cwd()}\`
+- \uC2E4\uD589 \uC778\uC790: \`${JSON.stringify(options.args ?? process.argv.slice(2))}\`
+- \uC2E4\uD589 \uD658\uACBD: \`${process.platform} ${process.arch} / Node ${process.version}\`
+
+## Summary
+
+${options.title || serializedError.summary || "Unknown error"}
+
+## Error
+
+\`\`\`json
+${JSON.stringify(serializedError, null, 2)}
+\`\`\`
+
+## Trace
+
+\`\`\`json
+${JSON.stringify(traceSnapshot, null, 2)}
+\`\`\`${extraSections.length ? `
+${extraSections.map((section) => `
+## ${section.heading}
+
+${section.markdown}`).join("\n")}
+` : "\n"}
+`;
+    fs.writeFileSync(reportPath, report);
+    helperTrace("error-report:write:done", reportPath);
+    return reportPath;
+  } catch (writeError) {
+    console.error("\u26A0\uFE0F \uC5D0\uB7EC \uB85C\uADF8 \uD30C\uC77C \uC0DD\uC131\uC5D0 \uC2E4\uD328\uD588\uC2B5\uB2C8\uB2E4.");
+    console.error(writeError);
+    return "";
+  }
+}
+function exitWithError(message, options = {}) {
+  const reportPath = writeErrorReport(options.error || new Error(message), {
+    ...options,
+    title: message
+  });
+  console.error(message);
+  if (options.error) {
+    console.error(options.error);
+  }
+  if (reportPath) {
+    console.error(`\u{1F4C4} \uC5D0\uB7EC \uB85C\uADF8 \uC800\uC7A5 \uC704\uCE58: ${reportPath}`);
+  }
+  process.exit(1);
+}
+function parseServiceFromArgs(args4 = process.argv.slice(2)) {
+  helperTrace("parse-service:start", `args=${JSON.stringify(args4)}`);
+  const serviceIndex = args4.indexOf("--service");
+  const rawService = serviceIndex !== -1 ? args4[serviceIndex + 1] : "";
+  if (!rawService) {
+    helperTrace("parse-service:empty");
+    return "";
+  }
+  const normalizedService = rawService.toLowerCase();
+  if (AIServices.includes(normalizedService)) {
+    helperTrace("parse-service:resolved", normalizedService);
+    return normalizedService;
+  }
+  helperTrace("parse-service:invalid", rawService);
+  exitWithError(
+    `\u274C \uC9C0\uC6D0\uD558\uC9C0 \uC54A\uB294 \uC11C\uBE44\uC2A4\uC785\uB2C8\uB2E4: ${rawService}. \uC0AC\uC6A9 \uAC00\uB2A5 \uAC12: ${AIServices.join(", ")} (\uC608: --service codex)`,
+    {
+      scope: "helper:parseServiceFromArgs",
+      args: args4,
+      extraSections: [
+        {
+          heading: "Allowed Services",
+          markdown: `\`\`\`json
+${JSON.stringify(AIServices, null, 2)}
+\`\`\``
+        }
+      ]
+    }
+  );
+}
 function getGitDiffFilter() {
   const includeExtensions = ["*.ts", "*.tsx", "*.js", "*.jsx"];
   const excludePatterns = ignoreList.map((item) => `:(exclude)${item}`);
@@ -100,6 +261,7 @@ function getGitDiffFilter() {
 function openReport(reportPath) {
   const resolvedPath = path.resolve(reportPath);
   const { platform } = process;
+  helperTrace("open-report:start", resolvedPath);
   const openWithChrome = () => {
     if (platform === "darwin") {
       execSync(`open -a "Google Chrome" "${resolvedPath}"`, { stdio: "ignore" });
@@ -124,32 +286,41 @@ function openReport(reportPath) {
   };
   try {
     if (openWithChrome()) {
+      helperTrace("open-report:chrome:success", platform);
       console.log("\u{1F680} Google Chrome\uC5D0\uC11C \uB9AC\uD3EC\uD2B8\uB97C \uC5F4\uC5C8\uC2B5\uB2C8\uB2E4.");
       return;
     }
-  } catch {
+  } catch (error) {
+    helperTrace("open-report:chrome:failed", getErrorSummary(error));
   }
   try {
     if (openWithDefaultBrowser()) {
+      helperTrace("open-report:default-browser:success", platform);
       console.log("\u{1F680} \uAE30\uBCF8 \uBE0C\uB77C\uC6B0\uC800\uC5D0\uC11C \uB9AC\uD3EC\uD2B8\uB97C \uC5F4\uC5C8\uC2B5\uB2C8\uB2E4.");
       return;
     }
-  } catch (e) {
-    console.error("\u26A0\uFE0F \uBE0C\uB77C\uC6B0\uC800 \uC5F4\uAE30 \uC2E4\uD328:", e);
+  } catch (error) {
+    helperTrace("open-report:default-browser:failed", getErrorSummary(error));
+    console.error("\u26A0\uFE0F \uBE0C\uB77C\uC6B0\uC800 \uC5F4\uAE30 \uC2E4\uD328:", error);
     return;
   }
+  helperTrace("open-report:unsupported-platform", platform);
   console.error(`\u26A0\uFE0F \uC9C0\uC6D0\uD558\uC9C0 \uC54A\uB294 \uD50C\uB7AB\uD3FC\uC785\uB2C8\uB2E4: ${platform}`);
 }
 function getDiffArgs() {
   const args4 = process.argv.slice(2);
   const commitIndex = args4.indexOf("--commit");
   const { includeParams, excludeParams } = getGitDiffFilter();
+  helperTrace("diff-args:resolve:start", `args=${JSON.stringify(args4)}`);
   let diffArgs = "";
   if (commitIndex !== -1) {
     const commitHash = args4[commitIndex + 1];
     if (!commitHash) {
-      console.error("\u274C \uCEE4\uBC0B \uD574\uC2DC\uAC00 \uC81C\uACF5\uB418\uC9C0 \uC54A\uC558\uC2B5\uB2C8\uB2E4.");
-      process.exit(1);
+      helperTrace("diff-args:commit-hash-missing");
+      exitWithError("\u274C \uCEE4\uBC0B \uD574\uC2DC\uAC00 \uC81C\uACF5\uB418\uC9C0 \uC54A\uC558\uC2B5\uB2C8\uB2E4.", {
+        scope: "helper:getDiffArgs",
+        args: args4
+      });
     }
     const nextArg = args4[commitIndex + 2];
     let n = 0;
@@ -159,28 +330,37 @@ function getDiffArgs() {
         n = 0;
       }
     }
+    helperTrace("diff-args:commit-mode", `${commitHash}~${n + 1} ${commitHash}`);
     console.log(`\u2139\uFE0F \uCEE4\uBC0B '${commitHash}' ${n > 0 ? ` \uD3EC\uD568 \uCD1D ${n + 1}\uAC1C\uC758 \uCEE4\uBC0B` : ""}\uC744 \uB9AC\uBDF0\uD569\uB2C8\uB2E4...`);
     diffArgs = `${commitHash}~${n + 1} ${commitHash}`;
   } else {
     try {
+      helperTrace("diff-args:unstaged-check:start");
       const check = execSync(`git diff --name-only -- ${includeParams} ${excludeParams}`).toString();
       if (!check.trim()) {
+        helperTrace("diff-args:unstaged-check:empty", "use HEAD~1 HEAD");
         console.log("\u2139\uFE0F Unstaged \uBCC0\uACBD\uC0AC\uD56D\uC774 \uC5C6\uC2B5\uB2C8\uB2E4. \uB9C8\uC9C0\uB9C9 \uCEE4\uBC0B(HEAD)\uC744 \uB9AC\uBDF0\uD569\uB2C8\uB2E4...");
         diffArgs = "HEAD~1 HEAD";
+      } else {
+        helperTrace("diff-args:unstaged-check:has-changes", `length=${check.length}`);
       }
-    } catch {
+    } catch (error) {
+      helperTrace("diff-args:unstaged-check:failed", getErrorSummary(error));
     }
   }
+  helperTrace("diff-args:resolve:done", diffArgs || "(default)");
   return diffArgs;
 }
 async function showSelectionAIService() {
   const selectedServiceFromArgs = parseServiceFromArgs();
   if (selectedServiceFromArgs) {
+    helperTrace("show-selection:from-args", selectedServiceFromArgs);
     console.log(`
 \u2705 \x1B[32m${selectedServiceFromArgs}\x1B[0m \uC11C\uBE44\uC2A4\uAC00 \uC120\uD0DD\uB418\uC5C8\uC2B5\uB2C8\uB2E4. (--service)
 `);
     return selectedServiceFromArgs;
   }
+  helperTrace("show-selection:interactive:start");
   let selectedIndex = 0;
   const rl = readline.createInterface({
     input: process.stdin,
@@ -194,6 +374,7 @@ async function showSelectionAIService() {
       readline.moveCursor(process.stdout, 0, -(AIServices.length + 1));
     }
     firstRender = false;
+    helperTrace("show-selection:interactive:render", AIServices[selectedIndex] || "unknown");
     readline.clearScreenDown(process.stdout);
     process.stdout.write(
       "\u{1F916} AI \uC11C\uBE44\uC2A4\uB97C \uC120\uD0DD\uD574\uC8FC\uC138\uC694 (\x1B[33m\u2191\u2193 \uBC29\uD5A5\uD0A4\x1B[0m \uC774\uB3D9, \x1B[33mEnter\x1B[0m \uC120\uD0DD):\n"
@@ -213,6 +394,7 @@ async function showSelectionAIService() {
     const onData = (data) => {
       const key = data.toString();
       if (key === "") {
+        helperTrace("show-selection:interactive:ctrl-c");
         process.stdout.write("\x1B[?25h");
         process.exit(0);
       }
@@ -233,6 +415,7 @@ async function showSelectionAIService() {
 `);
         const result = AIServices[selectedIndex];
         if (result) {
+          helperTrace("show-selection:interactive:confirmed", result);
           resolve(result);
         }
       }
@@ -346,14 +529,17 @@ var createClaudeCommand = (tempDiffPath2, reviewFormPath2) => {
     { path: namingRulesPath, display: "\uB124\uC774\uBC0D \uADDC\uCE59" },
     { path: codingConventionRulesPath, display: "\uCF54\uB529 \uCEE8\uBCA4\uC158" }
   ];
-  const existingRuleFiles = rules.filter((rule) => fs5.existsSync(rule.path)).map((rule) => rule.path);
+  const existingRuleFiles = rules.filter((rule) => fs.existsSync(rule.path)).map((rule) => rule.path);
   trace("rules:loaded", `count=${existingRuleFiles.length}`);
-  const reviewFormExists = fs5.existsSync(reviewFormPath2);
+  const reviewFormExists = fs.existsSync(reviewFormPath2);
   trace("reviewForm:status", reviewFormExists ? "exists" : "missing");
   const systemPromptFiles = reviewFormExists ? [...existingRuleFiles, reviewFormPath2] : existingRuleFiles;
   const prompt = "\uC704 \uADDC\uCE59\uB4E4\uC744 \uCC38\uACE0\uD558\uC5EC \uC774 diff\uB97C \uCF54\uB4DC\uB9AC\uBDF0\uD574\uC918. \uB9AC\uBDF0\uC591\uC2DD\uC5D0 \uB9DE\uCDB0\uC11C \uC791\uC131\uD574\uC918.";
+  trace("prompt:prepared", `length=${prompt.length}`);
+  trace("system-prompt-files", `count=${systemPromptFiles.length}`);
   const modelCandidates = toUnique(customModel ? [customModel, ...aliasFallbacks] : aliasFallbacks);
   trace("model:candidates", modelCandidates.join(", "));
+  trace("command:candidates:count", String(modelCandidates.length + 1));
   if (customModel) {
     console.warn(
       `\u26A0\uFE0F \uCEE4\uC2A4\uD140 \uBAA8\uB378(${customModel})\uC744 \uC6B0\uC120 \uC2DC\uB3C4\uD558\uACE0 \uC2E4\uD328\uD558\uBA74 alias(${aliasFallbacks.join(
@@ -394,6 +580,7 @@ var createClaudeCommand = (tempDiffPath2, reviewFormPath2) => {
 \uC0DD\uC131\uB420 \uBA85\uB839\uC5B4 \uBBF8\uB9AC\uBCF4\uAE30:
 ${safeCommand}"`;
   }
+  trace("command:mode", "execute");
   trace("createClaudeCommand:end");
   return command;
 };
@@ -404,23 +591,25 @@ function checkClaudeCliInstalled() {
     trace2("version-check:run", "claude --version");
     execSync("claude --version", { stdio: "ignore" });
     trace2("version-check:ok");
-  } catch {
-    trace2("version-check:failed", "install-start");
+  } catch (error) {
+    trace2("version-check:failed", getErrorSummary(error));
+    trace2("install:start", "@anthropic-ai/claude-code");
     console.log(
       "\u2139\uFE0F claude-cli\uAC00 \uC124\uCE58\uB418\uC5B4 \uC788\uC9C0 \uC54A\uC2B5\uB2C8\uB2E4. \uC124\uCE58\uB97C \uC9C4\uD589\uD569\uB2C8\uB2E4... npm install -g @anthropic-ai/claude-code"
     );
     try {
       execSync("npm install -g @anthropic-ai/claude-code", { stdio: "inherit" });
-      trace2("install:ok", "exit(1) for login");
+      trace2("install:ok", "login-required");
       console.log("\u2705 claude-cli \uC124\uCE58\uAC00 \uC644\uB8CC\uB418\uC5C8\uC2B5\uB2C8\uB2E4.");
       console.log("\u26A0\uFE0F  claude-cli \uC0AC\uC6A9\uC744 \uC704\uD574 \uC778\uC99D\uC774 \uD544\uC694\uD569\uB2C8\uB2E4.");
       console.log('   \uD130\uBBF8\uB110\uC5D0\uC11C "claude" \uB97C \uC785\uB825\uD558\uC5EC \uBE0C\uB77C\uC6B0\uC800 \uB85C\uADF8\uC778\uC744 \uC644\uB8CC\uD55C \uD6C4, \uB2E4\uC2DC \uC2DC\uB3C4\uD574\uC8FC\uC138\uC694.');
       process.exit(1);
     } catch (installError) {
-      trace2("install:failed");
-      console.error("\u274C claude-cli \uC124\uCE58 \uC911 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4. \uAD8C\uD55C \uBB38\uC81C\uC77C \uC218 \uC788\uC2B5\uB2C8\uB2E4 (sudo \uD544\uC694).");
-      console.error(installError);
-      process.exit(1);
+      trace2("install:failed", getErrorSummary(installError));
+      exitWithError("\u274C claude-cli \uC124\uCE58 \uC911 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4. \uAD8C\uD55C \uBB38\uC81C\uC77C \uC218 \uC788\uC2B5\uB2C8\uB2E4 (sudo \uD544\uC694).", {
+        scope: "installation-claude",
+        error: installError
+      });
     }
   }
   trace2("checkClaudeCliInstalled:end");
@@ -471,10 +660,10 @@ var createCodexCommand = (tempDiffPath2, reviewFormPath2) => {
   trace3("createCodexCommand:start", `tempDiffPath=${tempDiffPath2}, reviewFormPath=${reviewFormPath2}`);
   const customModel = getArgValue2("--model");
   const reasoningEffort = resolveReasoningEffort2();
-  const rules = [rulesPath, namingRulesPath, codingConventionRulesPath].filter((filePath) => fs5.existsSync(filePath)).map((filePath) => `- ${filePath}`).join("\n");
+  const rules = [rulesPath, namingRulesPath, codingConventionRulesPath].filter((filePath) => fs.existsSync(filePath)).map((filePath) => `- ${filePath}`).join("\n");
   const rulesCount = rules ? rules.split("\n").length : 0;
   trace3("rules:loaded", `count=${rulesCount}`);
-  const hasReviewForm = fs5.existsSync(reviewFormPath2);
+  const hasReviewForm = fs.existsSync(reviewFormPath2);
   const reviewFormLine = hasReviewForm ? `- ${reviewFormPath2}` : "";
   trace3("reviewForm:status", reviewFormLine ? "exists" : "missing");
   const prompt = `\uC544\uB798 \uD30C\uC77C\uB4E4\uC744 \uCC38\uACE0\uD574\uC11C \uCF54\uB4DC \uB9AC\uBDF0\uB97C \uC9C4\uD589\uD574\uC918.
@@ -486,6 +675,7 @@ ${reviewFormLine || "- (\uC5C6\uC74C)"}
 - ${tempDiffPath2}
 
 \uBC18\uB4DC\uC2DC \uB9AC\uBDF0 \uC591\uC2DD\uC5D0 \uB9DE\uCDB0 \uC791\uC131\uD574\uC918.`;
+  trace3("prompt:prepared", `length=${prompt.length}`);
   let command = "";
   if (customModel) {
     console.warn("\u26A0\uFE0F \uC9C0\uC815\uD55C \uBAA8\uB378\uC774 \uC5C6\uB294 \uACBD\uC6B0, \uC5D0\uB7EC\uAC00 \uBC1C\uC0DD\uD558\uB2C8 \uC8FC\uC758\uD558\uC138\uC694.");
@@ -511,6 +701,7 @@ ${reviewFormLine || "- (\uC5C6\uC74C)"}
 \uC0DD\uC131\uB420 \uBA85\uB839\uC5B4 \uBBF8\uB9AC\uBCF4\uAE30:
 ${safeCommand}"`;
   }
+  trace3("command:mode", "execute");
   trace3("createCodexCommand:end");
   return command;
 };
@@ -521,21 +712,23 @@ function checkCodexCliInstalled() {
     trace4("version-check:run", "codex --version");
     execSync("codex --version", { stdio: "ignore" });
     trace4("version-check:ok");
-  } catch {
-    trace4("version-check:failed", "install-start");
+  } catch (error) {
+    trace4("version-check:failed", getErrorSummary(error));
+    trace4("install:start", "@openai/codex");
     console.log("\u2139\uFE0F codex-cli\uAC00 \uC124\uCE58\uB418\uC5B4 \uC788\uC9C0 \uC54A\uC2B5\uB2C8\uB2E4. \uC124\uCE58\uB97C \uC9C4\uD589\uD569\uB2C8\uB2E4... npm install -g @openai/codex");
     try {
       execSync("npm install -g @openai/codex", { stdio: "inherit" });
-      trace4("install:ok", "exit(1) for login");
+      trace4("install:ok", "login-required");
       console.log("\u2705 codex-cli \uC124\uCE58\uAC00 \uC644\uB8CC\uB418\uC5C8\uC2B5\uB2C8\uB2E4.");
       console.log("\u26A0\uFE0F codex-cli \uC0AC\uC6A9\uC744 \uC704\uD574 \uC778\uC99D\uC774 \uD544\uC694\uD569\uB2C8\uB2E4.");
       console.log('   \uD130\uBBF8\uB110\uC5D0\uC11C "codex login" \uC744 \uC785\uB825\uD558\uC5EC \uB85C\uADF8\uC778\uC744 \uC644\uB8CC\uD55C \uD6C4, \uB2E4\uC2DC \uC2DC\uB3C4\uD574\uC8FC\uC138\uC694.');
       process.exit(1);
     } catch (installError) {
-      trace4("install:failed");
-      console.error("\u274C codex-cli \uC124\uCE58 \uC911 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4. \uAD8C\uD55C \uBB38\uC81C\uC77C \uC218 \uC788\uC2B5\uB2C8\uB2E4 (sudo \uD544\uC694).");
-      console.error(installError);
-      process.exit(1);
+      trace4("install:failed", getErrorSummary(installError));
+      exitWithError("\u274C codex-cli \uC124\uCE58 \uC911 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4. \uAD8C\uD55C \uBB38\uC81C\uC77C \uC218 \uC788\uC2B5\uB2C8\uB2E4 (sudo \uD544\uC694).", {
+        scope: "installation-codex",
+        error: installError
+      });
     }
   }
   trace4("checkCodexCliInstalled:end");
@@ -643,17 +836,19 @@ var createGeminiCommand = (tempDiffPath2, reviewFormPath2) => {
     { path: namingRulesPath, display: "\uB124\uC774\uBC0D \uADDC\uCE59" },
     { path: codingConventionRulesPath, display: "\uCF54\uB529 \uCEE8\uBCA4\uC158" }
   ];
-  const validRules = rules.filter((rule) => fs5.existsSync(rule.path)).map((rule) => `@${rule.path}`).join(", ");
+  const validRules = rules.filter((rule) => fs.existsSync(rule.path)).map((rule) => `@${rule.path}`).join(", ");
   const rulesCount = validRules ? validRules.split(",").length : 0;
   trace5("rules:loaded", `count=${rulesCount}`);
-  const reviewFormRef = fs5.existsSync(reviewFormPath2) ? `@${reviewFormPath2}` : "(\uC5C6\uC74C)";
+  const reviewFormRef = fs.existsSync(reviewFormPath2) ? `@${reviewFormPath2}` : "(\uC5C6\uC74C)";
   trace5("reviewForm:status", reviewFormRef === "(\uC5C6\uC74C)" ? "missing" : "exists");
   const reasoningInstruction = getReasoningInstruction(reasoningEffort);
   const prompt = `\uB2E4\uC74C \uADDC\uCE59\uB4E4\uC744 \uCC38\uACE0\uD574\uC11C(${validRules || "(\uC5C6\uC74C)"}) \uC774 diff(@${tempDiffPath2})\uB97C \uB9AC\uBDF0\uD574\uC918.
 \uB9AC\uBDF0 \uC591\uC2DD\uC740 ${reviewFormRef} \uC5D0 \uB9DE\uCDB0\uC11C \uC791\uC131\uD574\uC918.
 \uCD94\uB860 \uAC15\uB3C4 \uC9C0\uCE68: ${reasoningInstruction}`;
+  trace5("prompt:prepared", `length=${prompt.length}`);
   const modelCandidates = toUnique2(customModel ? [customModel, ...aliasFallbacks] : aliasFallbacks);
   trace5("model:candidates", modelCandidates.join(", "));
+  trace5("command:candidates:count", String(modelCandidates.length + 1));
   if (customModel) {
     console.warn(
       `\u26A0\uFE0F \uCEE4\uC2A4\uD140 \uBAA8\uB378(${customModel})\uC744 \uC6B0\uC120 \uC2DC\uB3C4\uD558\uACE0 \uC2E4\uD328\uD558\uBA74 alias(${aliasFallbacks.join(
@@ -676,6 +871,7 @@ var createGeminiCommand = (tempDiffPath2, reviewFormPath2) => {
 \uC0DD\uC131\uB420 \uBA85\uB839\uC5B4 \uBBF8\uB9AC\uBCF4\uAE30:
 ${safeCommand}"`;
   }
+  trace5("command:mode", "execute");
   trace5("createGeminiCommand:end");
   return command;
 };
@@ -686,21 +882,23 @@ function checkGeminiCliInstalled() {
     trace6("version-check:run", "gemini --version");
     execSync("gemini --version", { stdio: "ignore" });
     trace6("version-check:ok");
-  } catch {
-    trace6("version-check:failed", "install-start");
+  } catch (error) {
+    trace6("version-check:failed", getErrorSummary(error));
+    trace6("install:start", "@google/gemini-cli");
     console.log("\u2139\uFE0F gemini-cli\uAC00 \uC124\uCE58\uB418\uC5B4 \uC788\uC9C0 \uC54A\uC2B5\uB2C8\uB2E4. \uC124\uCE58\uB97C \uC9C4\uD589\uD569\uB2C8\uB2E4... npm install -g @google/gemini-cli");
     try {
       execSync("npm install -g @google/gemini-cli", { stdio: "inherit" });
-      trace6("install:ok", "exit(1) for login");
+      trace6("install:ok", "login-required");
       console.log("\u2705 gemini-cli \uC124\uCE58\uAC00 \uC644\uB8CC\uB418\uC5C8\uC2B5\uB2C8\uB2E4.");
       console.log("\u26A0\uFE0F  Gemini API \uC0AC\uC6A9\uC744 \uC704\uD574 \uC778\uC99D\uC774 \uD544\uC694\uD569\uB2C8\uB2E4.");
       console.log('   \uD130\uBBF8\uB110\uC5D0\uC11C "gemini" \uB97C \uC785\uB825\uD558\uC5EC \uBE0C\uB77C\uC6B0\uC800 \uB85C\uADF8\uC778\uC744 \uC644\uB8CC\uD55C \uD6C4, \uB2E4\uC2DC \uC2DC\uB3C4\uD574\uC8FC\uC138\uC694.');
       process.exit(1);
     } catch (installError) {
-      trace6("install:failed");
-      console.error("\u274C gemini-cli \uC124\uCE58 \uC911 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4. \uAD8C\uD55C \uBB38\uC81C\uC77C \uC218 \uC788\uC2B5\uB2C8\uB2E4 (sudo \uD544\uC694).");
-      console.error(installError);
-      process.exit(1);
+      trace6("install:failed", getErrorSummary(installError));
+      exitWithError("\u274C gemini-cli \uC124\uCE58 \uC911 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4. \uAD8C\uD55C \uBB38\uC81C\uC77C \uC218 \uC788\uC2B5\uB2C8\uB2E4 (sudo \uD544\uC694).", {
+        scope: "installation-gemini",
+        error: installError
+      });
     }
   }
   trace6("checkGeminiCliInstalled:end");
@@ -710,30 +908,34 @@ function checkGeminiCliInstalled() {
 var execAsync = util.promisify(exec);
 async function main() {
   const args4 = process.argv.slice(2);
+  clearTraceMessages();
   const isTest = isTestMode(args4);
   const trace7 = createTraceLogger("review-one-by-one", args4);
   trace7("main:start", `args=${JSON.stringify(args4)}`);
-  trace7("service-selection:start");
-  const service = await showSelectionAIService();
-  trace7("service-selection:done", `service=${service}`);
-  switch (service) {
-    case "gemini":
-      trace7("install-check:start", "service=gemini");
-      checkGeminiCliInstalled();
-      trace7("install-check:done", "service=gemini");
-      break;
-    case "claude":
-      trace7("install-check:start", "service=claude");
-      checkClaudeCliInstalled();
-      trace7("install-check:done", "service=claude");
-      break;
-    case "codex":
-      trace7("install-check:start", "service=codex");
-      checkCodexCliInstalled();
-      trace7("install-check:done", "service=codex");
-      break;
-  }
+  let service = "";
+  let savedDiffPath = "";
+  let savedReportPath = "";
   try {
+    trace7("service-selection:start");
+    service = await showSelectionAIService();
+    trace7("service-selection:done", `service=${service}`);
+    switch (service) {
+      case "gemini":
+        trace7("install-check:start", "service=gemini");
+        checkGeminiCliInstalled();
+        trace7("install-check:done", "service=gemini");
+        break;
+      case "claude":
+        trace7("install-check:start", "service=claude");
+        checkClaudeCliInstalled();
+        trace7("install-check:done", "service=claude");
+        break;
+      case "codex":
+        trace7("install-check:start", "service=codex");
+        checkCodexCliInstalled();
+        trace7("install-check:done", "service=codex");
+        break;
+    }
     trace7("review-flow:start");
     console.log("\u{1F680} AI Code Review\uB97C \uC2DC\uC791\uD569\uB2C8\uB2E4...");
     trace7("report-dir:create:start");
@@ -741,6 +943,7 @@ async function main() {
     trace7("report-dir:create:done");
     const { includeParams, excludeParams } = getGitDiffFilter();
     trace7("diff-filter:loaded", `include=${includeParams} | exclude=${excludeParams}`);
+    trace7("diff-args:build:start");
     const diffArgs = getDiffArgs();
     trace7("diff-args:build:done", `diffArgs=${diffArgs || "(default)"}`);
     const filesCommand = `git diff --name-only ${diffArgs} -- ${includeParams} ${excludeParams}`;
@@ -760,26 +963,26 @@ async function main() {
     trace7("full-diff:done", `length=${fullDiff.length}`);
     trace7("timestamp:created", nowStr);
     trace7("temp-diff:write:start", tempDiffPath);
-    fs5.writeFileSync(tempDiffPath, fullDiff);
+    fs.writeFileSync(tempDiffPath, fullDiff);
     trace7("temp-diff:write:done");
     trace7("saved-diff:copy:start");
-    const savedDiffPath = getNextFilePath(REPORT_DIR, `${nowStr}-diff`, ".txt");
-    fs5.copyFileSync(tempDiffPath, savedDiffPath);
+    savedDiffPath = getNextFilePath(REPORT_DIR, `${nowStr}-diff`, ".txt");
+    fs.copyFileSync(tempDiffPath, savedDiffPath);
     trace7("saved-diff:copy:done", savedDiffPath);
-    const savedReportPath = getNextFilePath(REPORT_DIR, nowStr, ".md");
+    savedReportPath = getNextFilePath(REPORT_DIR, nowStr, ".md");
     trace7("saved-report:path", savedReportPath);
     const promises = fileList.map(async (file) => {
+      const tempOneFileDiffPath = `temp_diff_${file.replace(/\//g, "_")}.txt`;
+      let command = "";
       try {
         trace7("file-review:start", file);
         console.log(`\u{1F50D} Reviewing: ${file}...`);
         trace7("file-diff:run", file);
         const fileDiff = execSync(`git diff ${diffArgs} -- "${file}"`).toString();
         trace7("file-diff:done", `${file} | length=${fileDiff.length}`);
-        const tempOneFileDiffPath = `temp_diff_${file.replace(/\//g, "_")}.txt`;
         trace7("file-temp-diff:write:start", tempOneFileDiffPath);
-        fs5.writeFileSync(tempOneFileDiffPath, fileDiff);
+        fs.writeFileSync(tempOneFileDiffPath, fileDiff);
         trace7("file-temp-diff:write:done", tempOneFileDiffPath);
-        let command = "";
         trace7("file-command:create:start", file);
         switch (service) {
           case "gemini":
@@ -792,7 +995,7 @@ async function main() {
             command = createCodexCommand(tempOneFileDiffPath, reviewFormOneByOnePath);
             break;
         }
-        trace7("file-command:create:done", file);
+        trace7("file-command:create:done", `${file} | commandLength=${command.length}`);
         trace7("file-command:exec:start", file);
         const { stdout } = await execAsync(command, { maxBuffer: 1024 * 1024 * 20 });
         const result = stdout.toString();
@@ -806,11 +1009,11 @@ ${result}
 `;
         console.log(tempReport);
         trace7("file-report:append:start", file);
-        fs5.appendFileSync(savedReportPath, tempReport);
+        fs.appendFileSync(savedReportPath, tempReport);
         trace7("file-report:append:done", file);
         if (isTest) {
           trace7("file-test-command:append:start", file);
-          fs5.appendFileSync(savedReportPath, `
+          fs.appendFileSync(savedReportPath, `
 
 ## \uC0AC\uC6A9\uB41C \uBA85\uB839\uC5B4
 
@@ -819,26 +1022,51 @@ ${command}`);
         }
         trace7("file-review:end", file);
       } catch (err) {
-        trace7("file-review:catch", file);
+        trace7("file-review:catch", `${file} | ${getErrorSummary(err)}`);
+        const errorLogPath = writeErrorReport(err, {
+          scope: "review-one-by-one:file",
+          args: args4,
+          extraSections: [
+            {
+              heading: "Execution Context",
+              markdown: `\`\`\`json
+${JSON.stringify(
+                {
+                  service,
+                  file,
+                  command: command || null,
+                  tempOneFileDiffPath,
+                  savedDiffPath: savedDiffPath || null,
+                  savedReportPath: savedReportPath || null
+                },
+                null,
+                2
+              )}
+\`\`\``
+            }
+          ]
+        });
         console.error(`\u274C Error reviewing file ${file}:`, err);
+        if (errorLogPath) {
+          console.error(`\u{1F4C4} \uC5D0\uB7EC \uB85C\uADF8 \uC800\uC7A5 \uC704\uCE58: ${errorLogPath}`);
+        }
         const errorReport = `### File: ${file}
 \u274C Review Failed
 \`\`\`
-${err}
+${getErrorSummary(err)}
 \`\`\`
 
 `;
-        fs5.appendFileSync(savedReportPath, errorReport);
+        fs.appendFileSync(savedReportPath, errorReport);
         try {
-          const tempOneFileDiffPath = `temp_diff_${file.replace(/\//g, "_")}.txt`;
-          if (fs5.existsSync(tempOneFileDiffPath)) {
+          if (fs.existsSync(tempOneFileDiffPath)) {
             trace7("file-temp-diff:delete:start(catch)", tempOneFileDiffPath);
             deleteFile(tempOneFileDiffPath);
             trace7("file-temp-diff:delete:done(catch)", tempOneFileDiffPath);
           }
-        } catch (e) {
-          trace7("file-temp-diff:delete:failed(catch)", file);
-          console.error(`\u274C Error deleting temp file for ${file}:`, e);
+        } catch (cleanupError) {
+          trace7("file-temp-diff:delete:failed(catch)", `${file} | ${getErrorSummary(cleanupError)}`);
+          console.error(`\u274C Error deleting temp file for ${file}:`, cleanupError);
         }
       }
     });
@@ -857,12 +1085,44 @@ ${err}
     trace7("cleanup-temp-diff:done");
     trace7("review-flow:end");
   } catch (error) {
-    trace7("review-flow:catch");
+    trace7("review-flow:catch", getErrorSummary(error));
+    let errorReportPath = "";
+    trace7("cleanup-temp-diff:start(catch)");
+    try {
+      deleteTempDiff();
+      trace7("cleanup-temp-diff:done(catch)");
+    } catch (cleanupError) {
+      trace7("cleanup-temp-diff:failed(catch)", getErrorSummary(cleanupError));
+      console.error("\u26A0\uFE0F \uC784\uC2DC diff \uC815\uB9AC \uC911 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4.");
+      console.error(cleanupError);
+    }
+    trace7("error-report:prepare", `service=${service || "unknown"}`);
+    errorReportPath = writeErrorReport(error, {
+      scope: "review-one-by-one",
+      args: args4,
+      extraSections: [
+        {
+          heading: "Execution Context",
+          markdown: `\`\`\`json
+${JSON.stringify(
+            {
+              service: service || null,
+              tempDiffPath,
+              savedDiffPath: savedDiffPath || null,
+              savedReportPath: savedReportPath || null
+            },
+            null,
+            2
+          )}
+\`\`\``
+        }
+      ]
+    });
     console.error("\u274C \uB9AC\uBDF0 \uB3C4\uC911 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4.");
     console.error(error);
-    trace7("cleanup-temp-diff:start(catch)");
-    deleteTempDiff();
-    trace7("cleanup-temp-diff:done(catch)");
+    if (errorReportPath) {
+      console.error(`\u{1F4C4} \uC5D0\uB7EC \uB85C\uADF8 \uC800\uC7A5 \uC704\uCE58: ${errorReportPath}`);
+    }
     process.exit(1);
   }
 }