sales-frontend-gemini-cli 0.4.3 → 0.4.4

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

@@ -269,7 +269,7 @@ function serializeError(error) {
   }
   if (error && typeof error === "object") {
     const errorLike = error;
-    const extraKeys = ["code", "errno", "syscall", "path", "cmd", "status", "signal", "spawnargs"];
+    const extraKeys = ["code", "errno", "syscall", "path", "cmd", "status", "signal", "spawnargs", "command"];
     extraKeys.forEach((key) => {
       if (errorLike[key] !== void 0) {
         serialized[key] = errorLike[key];
@@ -375,6 +375,87 @@ ${section.markdown}`).join("\n")}
     return "";
   }
 }
+function getExecutionLogSummary(status, title) {
+  if (title) {
+    return title;
+  }
+  switch (status) {
+    case "success":
+      return "\uB9AC\uBDF0 \uC2E4\uD589\uC774 \uC131\uACF5\uC801\uC73C\uB85C \uC644\uB8CC\uB418\uC5C8\uC2B5\uB2C8\uB2E4.";
+    case "failed":
+      return "\uB9AC\uBDF0 \uC2E4\uD589 \uC911 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4.";
+    case "partial_failure":
+      return "\uB9AC\uBDF0 \uC2E4\uD589\uC740 \uC644\uB8CC\uB418\uC5C8\uC9C0\uB9CC \uC77C\uBD80 \uB2E8\uACC4\uC5D0\uC11C \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4.";
+    default:
+      return "\uB9AC\uBDF0 \uC2E4\uD589\uC774 \uCDE8\uC18C\uB418\uC5C8\uAC70\uB098 \uB9AC\uBDF0 \uB300\uC0C1\uC774 \uC5C6\uC5B4 \uC885\uB8CC\uB418\uC5C8\uC2B5\uB2C8\uB2E4.";
+  }
+}
+function formatExecutionDuration(startedAt, finishedAt) {
+  const durationMs = Math.max(0, finishedAt.getTime() - startedAt.getTime());
+  if (durationMs < 1e3) {
+    return `${durationMs}ms`;
+  }
+  const durationSeconds = durationMs / 1e3;
+  if (durationSeconds < 60) {
+    return `${durationSeconds.toFixed(1)}s`;
+  }
+  const minutes = Math.floor(durationSeconds / 60);
+  const seconds = Math.round(durationSeconds % 60);
+  return `${minutes}m ${seconds}s`;
+}
+function writeExecutionLog(options = {}) {
+  try {
+    const startedAt = options.startedAt ?? /* @__PURE__ */ new Date();
+    const finishedAt = options.finishedAt ?? /* @__PURE__ */ new Date();
+    const status = options.status ?? "success";
+    helperTrace("execution-log:write:start", options.scope || "unknown");
+    createReportDirectory();
+    const reportPath = getAvailableFilePath(REPORT_DIR, `${getNowString(finishedAt)}-execution-log`, ".md");
+    const traceSnapshot = options.traceMessages ?? getTraceMessages();
+    const extraSections = options.extraSections || [];
+    const serializedError = options.error ? serializeError(options.error) : null;
+    const report = `# Execution Log
+
+- \uC2DC\uC791 \uC2DC\uAC01: ${getHumanReadableNowString(startedAt)}
+- \uC885\uB8CC \uC2DC\uAC01: ${getHumanReadableNowString(finishedAt)}
+- \uC2E4\uD589 \uC2DC\uAC04: ${formatExecutionDuration(startedAt, finishedAt)}
+- \uC0C1\uD0DC: \`${status}\`
+- 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
+
+${getExecutionLogSummary(status, options.title)}
+${serializedError ? `
+
+## 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("execution-log:write:done", reportPath);
+    return reportPath;
+  } catch (writeError) {
+    console.error("\u26A0\uFE0F \uC2E4\uD589 \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,
@@ -778,6 +859,13 @@ var ALLOWED_REASONING_EFFORTS = ["minimal", "low", "medium", "high"];
 function shellQuote(value) {
   return `'${value.replace(/'/g, `'\\''`)}'`;
 }
+function toShellOptionToken(value) {
+  const SIMPLE_SHELL_TOKEN_PATTERN = /^[A-Za-z0-9._:/=-]+$/;
+  if (SIMPLE_SHELL_TOKEN_PATTERN.test(value)) {
+    return value;
+  }
+  return shellQuote(value);
+}
 function getArgValue(flag) {
   const index = args.indexOf(flag);
   if (index === -1 || !args[index + 1]) {
@@ -857,9 +945,9 @@ function getAliasFallbacks(primaryAlias) {
 }
 function buildClaudeExecCommand(options) {
   const { tempDiffPath: tempDiffPath2, prompt, systemPromptFiles, effort, model, fallbackModel } = options;
-  const modelOption = model ? `--model ${shellQuote(model)}` : "";
-  const fallbackOption = model && fallbackModel ? `--fallback-model ${shellQuote(fallbackModel)}` : "";
-  const effortOption = `--effort ${shellQuote(effort)}`;
+  const modelOption = model ? `--model ${toShellOptionToken(model)}` : "";
+  const fallbackOption = model && fallbackModel ? `--fallback-model ${toShellOptionToken(fallbackModel)}` : "";
+  const effortOption = `--effort ${toShellOptionToken(effort)}`;
   const appendedPromptFiles = systemPromptFiles.map((path3) => `--append-system-prompt-file ${shellQuote(path3)}`).join(" ");
   return `cat ${shellQuote(tempDiffPath2)} | claude ${[
     modelOption,
@@ -984,12 +1072,22 @@ function printNotice2(message) {
     console.warn(message);
   }
 }
+function normalizeEffort2(level) {
+  if (level === "minimal") {
+    return "low";
+  }
+  return level;
+}
 function resolveReasoningEffort2() {
   const customReasoningEffort = getArgValue2("--reasoning-effort");
   if (customReasoningEffort) {
     if (ALLOWED_REASONING_EFFORTS2.includes(customReasoningEffort)) {
-      trace3("reasoning:custom", customReasoningEffort);
-      return customReasoningEffort;
+      const normalized = normalizeEffort2(customReasoningEffort);
+      trace3("reasoning:custom", `${customReasoningEffort} -> ${normalized}`);
+      if (customReasoningEffort === "minimal") {
+        printNotice2("\u26A0\uFE0F Codex\uB294 minimal\uC774 web_search \uB3C4\uAD6C\uC640 \uCDA9\uB3CC\uD560 \uC218 \uC788\uC5B4 low\uB85C \uB9E4\uD551\uD569\uB2C8\uB2E4.");
+      }
+      return normalized;
     }
     printNotice2(
       `\u26A0\uFE0F \uC9C0\uC6D0\uB418\uC9C0 \uC54A\uB294 reasoning effort(${customReasoningEffort})\uC785\uB2C8\uB2E4. allowed: ${ALLOWED_REASONING_EFFORTS2.join(
@@ -998,8 +1096,8 @@ function resolveReasoningEffort2() {
     );
   }
   if (args2.includes("--flash")) {
-    trace3("reasoning:flash-default", "minimal");
-    return "minimal";
+    trace3("reasoning:flash-default", "low");
+    return "low";
   }
   if (args2.includes("--review")) {
     trace3("reasoning:review-default", "high");
@@ -1194,7 +1292,7 @@ function buildGeminiFileReferenceSection(files) {
   const existingFiles = files.filter((file) => fs.existsSync(file.path));
   return {
     count: existingFiles.length,
-    lines: existingFiles.map((file) => `- ${file.display}: ${toGeminiFileReference(file.path)}`)
+    items: existingFiles.map((file) => `${file.display} ${toGeminiFileReference(file.path)}`)
   };
 }
 var createGeminiCommand = (tempDiffPath2, reviewFormPath2) => {
@@ -1213,19 +1311,19 @@ var createGeminiCommand = (tempDiffPath2, reviewFormPath2) => {
   const ruleSection = buildGeminiFileReferenceSection(rules);
   trace5("rules:loaded", `count=${ruleSection.count}`);
   const reviewFormExists = fs.existsSync(resolvedReviewFormPath);
-  const reviewFormLine = reviewFormExists ? `- \uB9AC\uBDF0 \uC591\uC2DD: ${toGeminiFileReference(resolvedReviewFormPath)}` : "- \uB9AC\uBDF0 \uC591\uC2DD: (\uC5C6\uC74C)";
+  const reviewFormText = reviewFormExists ? `\uB9AC\uBDF0 \uC591\uC2DD ${toGeminiFileReference(resolvedReviewFormPath)}` : "\uB9AC\uBDF0 \uC591\uC2DD (\uC5C6\uC74C)";
   trace5("reviewForm:status", reviewFormExists ? "exists" : "missing");
   const reasoningInstruction = getReasoningInstruction(reasoningEffort);
-  const prompt = `\uC544\uB798 \uD30C\uC77C\uB4E4\uC744 \uC21C\uC11C\uB300\uB85C \uC77D\uACE0 \uCF54\uB4DC \uB9AC\uBDF0\uB97C \uC9C4\uD589\uD574\uC918.
-\uADDC\uCE59 \uD30C\uC77C:
-${ruleSection.lines.join("\n") || "- (\uC5C6\uC74C)"}
-\uB9AC\uBDF0 \uC591\uC2DD \uD30C\uC77C:
-${reviewFormLine}
-\uB9AC\uBDF0 \uB300\uC0C1 diff \uD30C\uC77C:
-- \uB9AC\uBDF0 \uB300\uC0C1 diff: ${toGeminiFileReference(resolvedTempDiffPath)}
-
-\uBC18\uB4DC\uC2DC \uB9AC\uBDF0 \uC591\uC2DD\uC5D0 \uB9DE\uCDB0 \uC791\uC131\uD574\uC918.
-\uCD94\uB860 \uAC15\uB3C4 \uC9C0\uCE68: ${reasoningInstruction}`;
+  const ruleText = ruleSection.items.join(" ") || "(\uC5C6\uC74C)";
+  const diffText = `\uB9AC\uBDF0 \uB300\uC0C1 diff ${toGeminiFileReference(resolvedTempDiffPath)}`;
+  const prompt = [
+    "\uC544\uB798 \uD30C\uC77C\uB4E4\uC744 \uC21C\uC11C\uB300\uB85C \uC77D\uACE0 \uCF54\uB4DC \uB9AC\uBDF0\uB97C \uC9C4\uD589\uD574\uC918.",
+    `\uADDC\uCE59 \uD30C\uC77C: ${ruleText}`,
+    `\uB9AC\uBDF0 \uC591\uC2DD \uD30C\uC77C: ${reviewFormText}`,
+    `\uB9AC\uBDF0 \uB300\uC0C1 diff \uD30C\uC77C: ${diffText}`,
+    "\uBC18\uB4DC\uC2DC \uB9AC\uBDF0 \uC591\uC2DD\uC5D0 \uB9DE\uCDB0 \uC791\uC131\uD574\uC918.",
+    `\uCD94\uB860 \uAC15\uB3C4 \uC9C0\uCE68: ${reasoningInstruction}`
+  ].join(" ");
   trace5("prompt:prepared", `length=${prompt.length}`);
   const modelCandidates = toUnique2(customModel ? [customModel, ...aliasFallbacks] : aliasFallbacks);
   trace5("model:candidates", modelCandidates.join(", "));
@@ -1289,6 +1387,7 @@ function checkGeminiCliInstalled() {
 var execAsync = util.promisify(exec);
 async function main() {
   const args4 = process.argv.slice(2);
+  const startedAt = /* @__PURE__ */ new Date();
   clearTraceMessages();
   const isTest = isTestMode(args4);
   const trace7 = createTraceLogger("review-one-by-one", args4);
@@ -1297,6 +1396,13 @@ async function main() {
   let savedDiffPath = "";
   let savedReportPath = "";
   let selectedCommitSummary = "";
+  let fileList = [];
+  let executionLogPath = "";
+  let executionStatus = "cancelled";
+  let executionTitle = "\uB9AC\uBDF0 \uC2E4\uD589\uC774 \uCDE8\uC18C\uB418\uC5C8\uAC70\uB098 \uB9AC\uBDF0 \uB300\uC0C1\uC774 \uC5C6\uC5B4 \uC885\uB8CC\uB418\uC5C8\uC2B5\uB2C8\uB2E4.";
+  let executionError = null;
+  let exitCode = 0;
+  const fileExecutionLogs = [];
   try {
     trace7("service-selection:start");
     service = await showSelectionAIService();
@@ -1325,22 +1431,24 @@ async function main() {
     trace7("commit-selection:done", `count=${selectedCommits.length}`);
     if (selectedCommits.length === 0) {
       trace7("commit-selection:empty");
+      executionTitle = "\uC120\uD0DD\uB41C \uCEE4\uBC0B\uC774 \uC5C6\uC2B5\uB2C8\uB2E4.";
       console.log("\u2139\uFE0F \uC120\uD0DD\uB41C \uCEE4\uBC0B\uC774 \uC5C6\uC2B5\uB2C8\uB2E4.");
       deleteTempDiff();
-      process.exit(0);
+      return;
     }
     selectedCommitSummary = buildSelectedCommitSummary(selectedCommits);
     trace7("commit-summary:prepared", selectedCommitSummary);
     console.log("\u23F3 \uC120\uD0DD\uD55C \uCEE4\uBC0B\uC758 \uD30C\uC77C \uBAA9\uB85D\uACFC diff\uB97C \uC815\uB9AC\uD558\uB294 \uC911\uC785\uB2C8\uB2E4...");
     trace7("files-command:run");
-    const fileList = getSelectedCommitFiles(selectedCommits);
+    fileList = getSelectedCommitFiles(selectedCommits);
     trace7("files-command:done", `fileCount=${fileList.length}`);
     console.log(`\u{1F4C2} \uB9AC\uBDF0 \uB300\uC0C1 \uD30C\uC77C(${fileList.length}\uAC1C): ${formatReviewTargetFiles(fileList)}`);
     if (fileList.length === 0) {
       trace7("empty-file-list:exit");
+      executionTitle = "\uC120\uD0DD\uD55C \uCEE4\uBC0B\uC5D0\uC11C \uB9AC\uBDF0 \uB300\uC0C1 \uD30C\uC77C \uBCC0\uACBD\uC744 \uCC3E\uC9C0 \uBABB\uD588\uC2B5\uB2C8\uB2E4.";
       console.log("\u2139\uFE0F \uC120\uD0DD\uD55C \uCEE4\uBC0B\uC5D0\uC11C \uB9AC\uBDF0 \uB300\uC0C1 \uD30C\uC77C \uBCC0\uACBD\uC744 \uCC3E\uC9C0 \uBABB\uD588\uC2B5\uB2C8\uB2E4.");
       deleteTempDiff();
-      process.exit(0);
+      return;
     }
     trace7("report-dir:create:start");
     createReportDirectory();
@@ -1370,6 +1478,15 @@ async function main() {
         trace7("file-diff:build:done", `${file} | length=${fileDiff.length}`);
         if (!fileDiff.trim()) {
           trace7("file-diff:empty", file);
+          fileExecutionLogs.push({
+            command: null,
+            errorSummary: null,
+            file,
+            index,
+            resultLength: 0,
+            status: "skipped",
+            tempOneFileDiffPath
+          });
           return;
         }
         trace7("file-temp-diff:write:start", tempOneFileDiffPath);
@@ -1412,9 +1529,27 @@ ${result}
 ${command}`);
           trace7("file-test-command:append:done", file);
         }
+        fileExecutionLogs.push({
+          command,
+          errorSummary: null,
+          file,
+          index,
+          resultLength: result.length,
+          status: "success",
+          tempOneFileDiffPath
+        });
         trace7("file-review:end", file);
       } catch (err) {
         trace7("file-review:catch", `${file} | ${getErrorSummary(err)}`);
+        fileExecutionLogs.push({
+          command: command || null,
+          errorSummary: getErrorSummary(err),
+          file,
+          index,
+          resultLength: 0,
+          status: "failed",
+          tempOneFileDiffPath
+        });
         const errorLogPath = writeErrorReport(err, {
           scope: "review-one-by-one:file",
           args: args4,
@@ -1466,6 +1601,15 @@ ${getErrorSummary(err)}
     trace7("parallel-review:await-start");
     await Promise.all(promises);
     trace7("parallel-review:await-done");
+    const failedFileCount = fileExecutionLogs.filter((log) => log.status === "failed").length;
+    const successfulFileCount = fileExecutionLogs.filter((log) => log.status === "success").length;
+    if (failedFileCount > 0) {
+      executionStatus = "partial_failure";
+      executionTitle = `\uD30C\uC77C\uBCC4 \uB9AC\uBDF0 \uC911 \uC77C\uBD80\uAC00 \uC2E4\uD328\uD588\uC2B5\uB2C8\uB2E4. (\uC131\uACF5 ${successfulFileCount}\uAC74 / \uC2E4\uD328 ${failedFileCount}\uAC74)`;
+    } else {
+      executionStatus = "success";
+      executionTitle = "\uD30C\uC77C\uBCC4 \uB9AC\uBDF0\uAC00 \uC131\uACF5\uC801\uC73C\uB85C \uC644\uB8CC\uB418\uC5C8\uC2B5\uB2C8\uB2E4.";
+    }
     console.log(`
 \u2705 \uB9AC\uBDF0\uAC00 \uC644\uB8CC\uB418\uC5C8\uC2B5\uB2C8\uB2E4.`);
     console.log(`\u{1F4C4} \uB9AC\uD3EC\uD2B8 \uC800\uC7A5 \uC704\uCE58: ${savedReportPath}`);
@@ -1480,6 +1624,10 @@ ${getErrorSummary(err)}
   } catch (error) {
     trace7("review-flow:catch", getErrorSummary(error));
     let errorReportPath = "";
+    executionStatus = "failed";
+    executionTitle = "\uD30C\uC77C\uBCC4 \uB9AC\uBDF0 \uC2E4\uD589 \uC911 \uC624\uB958\uAC00 \uBC1C\uC0DD\uD588\uC2B5\uB2C8\uB2E4.";
+    executionError = error;
+    exitCode = 1;
     trace7("cleanup-temp-diff:start(catch)");
     try {
       deleteTempDiff();
@@ -1517,7 +1665,58 @@ ${JSON.stringify(
     if (errorReportPath) {
       console.error(`\u{1F4C4} \uC5D0\uB7EC \uB85C\uADF8 \uC800\uC7A5 \uC704\uCE58: ${errorReportPath}`);
     }
-    process.exit(1);
+  } finally {
+    executionLogPath = writeExecutionLog({
+      scope: "review-one-by-one",
+      status: executionStatus,
+      title: executionTitle,
+      args: args4,
+      startedAt,
+      error: executionError,
+      extraSections: [
+        {
+          heading: "Execution Context",
+          markdown: `\`\`\`json
+${JSON.stringify(
+            {
+              service: service || null,
+              selectedCommitSummary: selectedCommitSummary || null,
+              fileList,
+              tempDiffPath,
+              savedDiffPath: savedDiffPath || null,
+              savedReportPath: savedReportPath || null
+            },
+            null,
+            2
+          )}
+\`\`\``
+        },
+        {
+          heading: "Generated Commands",
+          markdown: `\`\`\`json
+${JSON.stringify(
+            fileExecutionLogs.slice().sort((left, right) => left.index - right.index).map((log) => ({
+              file: log.file,
+              status: log.status,
+              resultLength: log.resultLength,
+              errorSummary: log.errorSummary,
+              tempOneFileDiffPath: log.tempOneFileDiffPath,
+              command: log.command
+            })),
+            null,
+            2
+          )}
+\`\`\``
+        }
+      ]
+    });
+    if (executionLogPath) {
+      const writeLog = executionStatus === "failed" ? console.error : console.log;
+      writeLog(`\u{1F4DD} \uC2E4\uD589 \uB85C\uADF8 \uC800\uC7A5 \uC704\uCE58: ${executionLogPath}`);
+    }
+  }
+  if (exitCode !== 0) {
+    process.exit(exitCode);
   }
 }
 main();