sales-frontend-gemini-cli 0.4.3 → 0.4.4

package/dist/pr-review/review.js

@@ -71,6 +71,9 @@ var ignoreList = [
 function isTestMode(args4 = process.argv.slice(2)) {
   return args4.includes("--test");
 }
+function shouldStreamAIOutput(args4 = process.argv.slice(2)) {
+  return args4.includes("--stream-output");
+}
 function clearTraceMessages() {
   traceMessages.length = 0;
 }
@@ -240,11 +243,41 @@ async function executeShellCommandWithProgress(command, options = {}) {
         return;
       }
       const exitSummary = signal ? `signal=${signal}` : `code=${String(code ?? "unknown")}`;
-      reject(new Error(`\uC258 \uBA85\uB839 \uC2E4\uD589 \uC2E4\uD328 (${exitSummary})${stderr.trim() ? `
-${stderr.trim()}` : ""}`));
+      const failureDetails = {
+        code,
+        command,
+        signal,
+        stderr,
+        stdout
+      };
+      reject(createShellCommandExecutionError(failureDetails, exitSummary));
     });
   });
 }
+function getShellCommandFailurePreview(failureDetails) {
+  const stderrText = failureDetails.stderr.trim();
+  const stdoutText = failureDetails.stdout.trim();
+  const combinedOutput = stderrText || stdoutText;
+  if (!combinedOutput) {
+    return "";
+  }
+  const MAX_PREVIEW_LENGTH = 4e3;
+  if (combinedOutput.length <= MAX_PREVIEW_LENGTH) {
+    return combinedOutput;
+  }
+  return combinedOutput.slice(-4e3);
+}
+function createShellCommandExecutionError(failureDetails, exitSummary) {
+  const failurePreview = getShellCommandFailurePreview(failureDetails);
+  const error = new Error(`\uC258 \uBA85\uB839 \uC2E4\uD589 \uC2E4\uD328 (${exitSummary})${failurePreview ? `
+${failurePreview}` : ""}`);
+  error.code = failureDetails.code;
+  error.signal = failureDetails.signal;
+  error.stdout = failureDetails.stdout;
+  error.stderr = failureDetails.stderr;
+  error.command = failureDetails.command;
+  return error;
+}
 function formatReviewTargetFiles(files, visibleCount = 5) {
   if (files.length === 0) {
     return "(\uC5C6\uC74C)";
@@ -317,7 +350,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];
@@ -423,6 +456,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,
@@ -810,6 +924,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]) {
@@ -889,9 +1010,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,
@@ -1016,12 +1137,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(
@@ -1030,8 +1161,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");
@@ -1226,7 +1357,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) => {
@@ -1245,19 +1376,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(", "));
@@ -1320,8 +1451,10 @@ function checkGeminiCliInstalled() {
 // src/pr-review/review.ts
 async function main() {
   const args4 = process.argv.slice(2);
+  const startedAt = /* @__PURE__ */ new Date();
   clearTraceMessages();
   const isTest = isTestMode(args4);
+  const shouldStreamOutput = shouldStreamAIOutput(args4);
   const trace7 = createTraceLogger("review", args4);
   trace7("main:start", `args=${JSON.stringify(args4)}`);
   let command = "";
@@ -1330,6 +1463,12 @@ async function main() {
   let service = "";
   let selectedCommitSummary = "";
   let reviewTargetFiles = [];
+  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 resultLength = 0;
+  let exitCode = 0;
   try {
     trace7("service-selection:start");
     service = await showSelectionAIService();
@@ -1353,14 +1492,18 @@ async function main() {
     }
     trace7("review-flow:start");
     console.log("\u{1F680} AI Code Review\uB97C \uC2DC\uC791\uD569\uB2C8\uB2E4...");
+    if (shouldStreamOutput) {
+      console.log("\u2139\uFE0F AI \uC2E4\uC2DC\uAC04 \uC751\uB2F5 \uD45C\uC2DC \uBAA8\uB4DC\uAC00 \uD65C\uC131\uD654\uB418\uC5C8\uC2B5\uB2C8\uB2E4.");
+    }
     trace7("commit-selection:start");
     const selectedCommits = await selectReviewCommits();
     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);
@@ -1372,9 +1515,10 @@ async function main() {
     trace7("git-diff:build:done", `length=${diff.length}`);
     if (!diff.trim() && !isTest) {
       trace7("empty-diff: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;
     }
     const nowStr = getNowString();
     trace7("timestamp:created", nowStr);
@@ -1404,8 +1548,9 @@ async function main() {
     trace7("command:exec:start");
     const result = (await executeShellCommandWithProgress(command, {
       progressMessage: `\u23F3 [\uB9AC\uBDF0 \uC9C4\uD589] ${service} | \uB300\uC0C1 ${reviewTargetFiles.length}\uAC1C \uD30C\uC77C`,
-      streamOutput: isTest
+      streamOutput: isTest || shouldStreamOutput
     })).stdout;
+    resultLength = result.length;
     trace7("command:exec:done", `resultLength=${result.length}`);
     trace7("report:write:start");
     savedReportPath = getNextFilePath(REPORT_DIR, nowStr, ".md");
@@ -1431,9 +1576,15 @@ ${command}`);
     deleteTempDiff();
     trace7("cleanup-temp-diff:done");
     trace7("review-flow:end");
+    executionStatus = "success";
+    executionTitle = "\uB9AC\uBDF0\uAC00 \uC131\uACF5\uC801\uC73C\uB85C \uC644\uB8CC\uB418\uC5C8\uC2B5\uB2C8\uB2E4.";
   } catch (error) {
     trace7("review-flow:catch", getErrorSummary(error));
     let errorReportPath = "";
+    executionStatus = "failed";
+    executionTitle = "\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();
@@ -1473,7 +1624,50 @@ ${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",
+      status: executionStatus,
+      title: executionTitle,
+      args: args4,
+      startedAt,
+      error: executionError,
+      extraSections: [
+        {
+          heading: "Execution Context",
+          markdown: `\`\`\`json
+${JSON.stringify(
+            {
+              service: service || null,
+              selectedCommitSummary: selectedCommitSummary || null,
+              reviewTargetFiles,
+              command: command || null,
+              tempDiffPath,
+              savedDiffPath: savedDiffPath || null,
+              savedReportPath: savedReportPath || null,
+              shouldStreamOutput,
+              resultLength
+            },
+            null,
+            2
+          )}
+\`\`\``
+        },
+        {
+          heading: "Generated Command",
+          markdown: command ? `\`\`\`sh
+${command}
+\`\`\`` : "(\uC5C6\uC74C)"
+        }
+      ]
+    });
+    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();