sales-frontend-gemini-cli 0.4.4 → 0.5.0

package/dist/pr-review/review.js

@@ -200,15 +200,51 @@ function runGitCommand(args4, options = {}) {
     throw error;
   }
 }
+function resolveShellLaunchConfig() {
+  const candidates = [];
+  const seen = /* @__PURE__ */ new Set();
+  const appendCandidate = (executable, shellArgs) => {
+    const normalizedExecutable = executable?.trim();
+    if (!normalizedExecutable) {
+      return;
+    }
+    if (path.isAbsolute(normalizedExecutable) && !fs.existsSync(normalizedExecutable)) {
+      return;
+    }
+    const key = `${normalizedExecutable}::${shellArgs.join("\0")}`;
+    if (seen.has(key)) {
+      return;
+    }
+    seen.add(key);
+    candidates.push({
+      executable: normalizedExecutable,
+      shellArgs
+    });
+  };
+  const shellFromEnv = process.env.SHELL?.trim();
+  const envShellName = shellFromEnv ? path.basename(shellFromEnv) : "";
+  if (["bash", "zsh", "ksh"].includes(envShellName)) {
+    appendCandidate(shellFromEnv, ["-lc"]);
+  }
+  ["/bin/bash", "/usr/bin/bash", "bash", "/bin/zsh", "/usr/bin/zsh", "zsh"].forEach((shellPath) => {
+    appendCandidate(shellPath, ["-lc"]);
+  });
+  ["/bin/sh", "/usr/bin/sh", "sh"].forEach((shellPath) => {
+    appendCandidate(shellPath, ["-c"]);
+  });
+  return candidates[0] || { executable: "sh", shellArgs: ["-c"] };
+}
 async function executeShellCommandWithProgress(command, options = {}) {
   const { progressIntervalMs = 1e4, progressMessage = "\u23F3 \uBA85\uB839\uC744 \uC2E4\uD589\uD558\uB294 \uC911\uC785\uB2C8\uB2E4...", streamOutput = false } = options;
   const { spawn } = await import('child_process');
+  const shellLaunchConfig = resolveShellLaunchConfig();
   return new Promise((resolve, reject) => {
     let stdout = "";
     let stderr = "";
     const startedAt = Date.now();
     console.log(progressMessage);
-    const child = spawn("/bin/zsh", ["-lc", command], {
+    helperTrace("shell-command:launcher", `${shellLaunchConfig.executable} ${shellLaunchConfig.shellArgs.join(" ")}`);
+    const child = spawn(shellLaunchConfig.executable, [...shellLaunchConfig.shellArgs, command], {
       stdio: ["ignore", "pipe", "pipe"]
     });
     const progressTimer = setInterval(() => {
@@ -587,6 +623,62 @@ function truncateCommitSubject(subject) {
   }
   return `${subject.slice(0, 69)}...`;
 }
+function formatReviewTargetFileCount(fileCount) {
+  return fileCount === 0 ? "\uBCC0\uACBD \uC5C6\uC74C" : `${fileCount}\uAC1C \uD30C\uC77C`;
+}
+function buildReviewTargetSectionTitle(target) {
+  return target.kind === "commit" ? `${target.hash} ${target.subject}` : `${target.hash} | ${target.subject}`;
+}
+function buildReviewTargetSummaryLine(target) {
+  if (target.kind === "commit") {
+    return `- ${target.hash} | ${target.subject} | ${target.author} | ${target.relativeDate}`;
+  }
+  return `- ${target.hash} | ${target.subject}`;
+}
+function buildReviewTargetDiffArgs(target, filePath) {
+  const reviewPathspecArgs = getReviewPathspecArgs();
+  if (target.kind === "commit") {
+    return ["show", "--stat", "--patch", "--format=", target.hash, "--", ...reviewPathspecArgs];
+  }
+  const diffArgs = ["diff"];
+  if (target.kind === "staged") {
+    diffArgs.push("--cached");
+  }
+  diffArgs.push("--stat", "--patch", "--");
+  return [...diffArgs, ...reviewPathspecArgs];
+}
+function buildReviewTargetFileArgs(target) {
+  const reviewPathspecArgs = getReviewPathspecArgs();
+  if (target.kind === "commit") {
+    return ["show", "--pretty=format:", "--name-only", target.hash, "--", ...reviewPathspecArgs];
+  }
+  const diffArgs = ["diff"];
+  if (target.kind === "staged") {
+    diffArgs.push("--cached");
+  }
+  return [...diffArgs, "--name-only", "--", ...reviewPathspecArgs];
+}
+function getReviewTargetFiles(target) {
+  const output = runGitCommand(buildReviewTargetFileArgs(target), {
+    allowFailure: true
+  });
+  return output.split("\n").map((line) => line.trim()).filter(Boolean);
+}
+function createWorkingTreeReviewOption(kind) {
+  const files = getReviewTargetFiles({
+    hash: kind,
+    kind});
+  const subject = kind === "unstaged" ? "\uC544\uC9C1 git add \uD558\uC9C0 \uC54A\uC740 \uBCC0\uACBD\uC0AC\uD56D" : "git add \uB41C \uBCC0\uACBD\uC0AC\uD56D";
+  return {
+    author: "",
+    description: `${subject} | ${formatReviewTargetFileCount(files.length)}`,
+    hash: kind,
+    kind,
+    label: kind,
+    relativeDate: "",
+    subject
+  };
+}
 function getRecentCommitOptions() {
   const output = runGitCommand(
     ["log", `-${COMMIT_FETCH_LIMIT}`, "--date=relative", "--pretty=format:%h%x09%an%x09%ar%x09%s"],
@@ -602,44 +694,43 @@ function getRecentCommitOptions() {
       author,
       description: `${author} | ${relativeDate}`,
       hash,
+      kind: "commit",
       label: `${hash} | ${truncateCommitSubject(subject)}`,
       relativeDate,
       subject
     };
   });
 }
+function getReviewTargetOptions() {
+  return [createWorkingTreeReviewOption("unstaged"), createWorkingTreeReviewOption("staged"), ...getRecentCommitOptions()];
+}
 function buildSelectedCommitSummary(commits) {
-  return commits.map((commit) => `- ${commit.hash} | ${commit.subject} | ${commit.author} | ${commit.relativeDate}`).join("\n");
+  return commits.map((commit) => buildReviewTargetSummaryLine(commit)).join("\n");
 }
 function getReviewPathspecArgs() {
   const { includePatterns, excludePatterns } = getGitDiffPathspecs();
   return [...includePatterns, ...excludePatterns];
 }
 function buildSelectedCommitDiff(commits) {
-  const reviewPathspecArgs = getReviewPathspecArgs();
   const sections = commits.map((commit) => {
-    const diff = runGitCommand(["show", "--stat", "--patch", "--format=", commit.hash, "--", ...reviewPathspecArgs], {
+    const diff = runGitCommand(buildReviewTargetDiffArgs(commit), {
       allowFailure: true,
       trimOutput: false
     }).trim();
     if (!diff) {
       return "";
     }
-    return [`## ${commit.hash} ${commit.subject}`, diff].join("\n\n");
+    return [`## ${buildReviewTargetSectionTitle(commit)}`, diff].join("\n\n");
   }).filter(Boolean).join("\n\n");
   if (!sections) {
     return "";
   }
-  return ["# \uC120\uD0DD\uD55C \uCEE4\uBC0B", buildSelectedCommitSummary(commits), "", "# \uB9AC\uBDF0 \uB300\uC0C1 diff", sections].join("\n");
+  return ["# \uC120\uD0DD\uD55C \uB9AC\uBDF0 \uB300\uC0C1", buildSelectedCommitSummary(commits), "", "# \uB9AC\uBDF0 \uB300\uC0C1 diff", sections].join("\n");
 }
 function getSelectedCommitFiles(commits) {
-  const reviewPathspecArgs = getReviewPathspecArgs();
   const files = /* @__PURE__ */ new Set();
   commits.forEach((commit) => {
-    const output = runGitCommand(["show", "--pretty=format:", "--name-only", commit.hash, "--", ...reviewPathspecArgs], {
-      allowFailure: true
-    });
-    output.split("\n").map((line) => line.trim()).filter(Boolean).forEach((filePath) => files.add(filePath));
+    getReviewTargetFiles(commit).forEach((filePath) => files.add(filePath));
   });
   return [...files];
 }
@@ -827,13 +918,13 @@ async function showMultiSelect(question, options, windowSize = COMMIT_SELECTION_
   });
 }
 async function selectReviewCommits() {
-  const commits = getRecentCommitOptions();
+  const commits = getReviewTargetOptions();
   if (commits.length === 0) {
-    console.log("\u2139\uFE0F \uB9AC\uBDF0\uD560 \uCD5C\uADFC \uCEE4\uBC0B\uC774 \uC5C6\uC2B5\uB2C8\uB2E4.");
+    console.log("\u2139\uFE0F \uB9AC\uBDF0\uD560 \uB300\uC0C1\uC774 \uC5C6\uC2B5\uB2C8\uB2E4.");
     return [];
   }
   return showMultiSelect(
-    "\uB9AC\uBDF0\uD560 \uCEE4\uBC0B\uC744 \uC120\uD0DD\uD574\uC8FC\uC138\uC694.",
+    "\uB9AC\uBDF0\uD560 \uB300\uC0C1\uC744 \uC120\uD0DD\uD574\uC8FC\uC138\uC694.",
     commits.map((commit) => ({
       description: commit.description,
       label: commit.label,
@@ -1500,8 +1591,8 @@ 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.");
+      executionTitle = "\uC120\uD0DD\uB41C \uB9AC\uBDF0 \uB300\uC0C1\uC774 \uC5C6\uC2B5\uB2C8\uB2E4.";
+      console.log("\u2139\uFE0F \uC120\uD0DD\uB41C \uB9AC\uBDF0 \uB300\uC0C1\uC774 \uC5C6\uC2B5\uB2C8\uB2E4.");
       deleteTempDiff();
       return;
     }
@@ -1515,8 +1606,8 @@ 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.");
+      executionTitle = "\uC120\uD0DD\uD55C \uB9AC\uBDF0 \uB300\uC0C1\uC5D0\uC11C \uB9AC\uBDF0\uD560 \uBCC0\uACBD\uC744 \uCC3E\uC9C0 \uBABB\uD588\uC2B5\uB2C8\uB2E4.";
+      console.log("\u2139\uFE0F \uC120\uD0DD\uD55C \uB9AC\uBDF0 \uB300\uC0C1\uC5D0\uC11C \uB9AC\uBDF0\uD560 \uBCC0\uACBD\uC744 \uCC3E\uC9C0 \uBABB\uD588\uC2B5\uB2C8\uB2E4.");
       deleteTempDiff();
       return;
     }