deepcode-ai 1.1.23 → 1.1.24

package/dist/index.js

@@ -10983,6 +10983,130 @@ async function projectsCommand(options) {
   );
   await waitUntilExit();
 }
+var DIFF_MAX_CHARS = 2e4;
+async function runGit2(cwd, args) {
+  const result = await execFileAsync("git", args, { cwd, timeoutMs: 3e4 });
+  if (result.exitCode !== 0) {
+    throw new Error(result.stderr || `git ${args.join(" ")} failed`);
+  }
+  return result.stdout;
+}
+async function isGitRepo(cwd) {
+  const result = await execFileAsync(
+    "git",
+    ["rev-parse", "--is-inside-work-tree"],
+    { cwd, timeoutMs: 5e3 }
+  );
+  return result.exitCode === 0;
+}
+function buildDiffArgs(options) {
+  if (options.staged) {
+    const args2 = ["diff", "--cached"];
+    if (options.file) args2.push("--", options.file);
+    return { args: args2, label: "staged changes" };
+  }
+  if (options.ref) {
+    const args2 = ["diff", options.ref];
+    if (options.file) args2.push("--", options.file);
+    return { args: args2, label: `diff vs ${options.ref}` };
+  }
+  const args = ["diff", "HEAD"];
+  if (options.file) args.push("--", options.file);
+  return { args, label: options.file ? `local changes in ${options.file}` : "local changes vs HEAD" };
+}
+function buildPrompt(diff, label, focus, truncated) {
+  const focusLine = focus.length > 0 ? `
+Focus areas: ${focus.join(", ")}.` : "";
+  const truncationNote = truncated ? `
+(Diff truncated at ${DIFF_MAX_CHARS} characters; some changes are not shown.)
+` : "";
+  return [
+    `Review the following local git diff (${label}).`,
+    "Do not modify any files. Output the review only.",
+    focusLine,
+    "",
+    `\`\`\`diff`,
+    diff,
+    `\`\`\``,
+    truncationNote,
+    "Produce a structured code review:",
+    "1. **Summary** \u2014 what changed (inferred from the diff)",
+    "2. **Issues** \u2014 bugs, security concerns, logic errors, missing error handling; quote the relevant lines",
+    "3. **Suggestions** \u2014 improvements and nitpicks",
+    "4. **Verdict** \u2014 Looks good / Has issues, with a one-line rationale"
+  ].filter((l) => l !== void 0).join("\n");
+}
+async function reviewCommand(options) {
+  if (!await isGitRepo(options.cwd)) {
+    await writeStderrLine("error: not inside a git repository");
+    process.exit(1);
+  }
+  const { args, label } = buildDiffArgs(options);
+  let rawDiff;
+  try {
+    rawDiff = await runGit2(options.cwd, args);
+  } catch (err) {
+    const msg = err instanceof Error ? err.message : String(err);
+    await writeStderrLine(`error: ${msg}`);
+    process.exit(1);
+  }
+  const trimmed = rawDiff.trim();
+  if (!trimmed) {
+    await writeStdoutLine(`No changes to review (${label}).`);
+    return;
+  }
+  let diff = trimmed;
+  let truncated = false;
+  if (diff.length > DIFF_MAX_CHARS) {
+    diff = diff.slice(0, DIFF_MAX_CHARS);
+    truncated = true;
+  }
+  const runtime = await createRuntime({
+    cwd: options.cwd,
+    configPath: options.config,
+    interactive: Boolean(options.yes)
+  });
+  if (options.yes) {
+    runtime.events.on("approval:request", (request) => {
+      runtime.events.emit("approval:decision", {
+        requestId: request.id,
+        decision: { allowed: true }
+      });
+    });
+  }
+  const target = resolveSessionTarget(runtime.config, {
+    provider: options.provider,
+    model: options.model
+  });
+  const session = runtime.sessions.create({
+    provider: target.provider,
+    model: target.model
+  });
+  const prompt = buildPrompt(diff, label, options.focus ?? [], truncated);
+  const secretValues = collectSecretValues(runtime.config);
+  await writeStdoutLine(`Reviewing ${label}\u2026
+`);
+  let streamed = false;
+  try {
+    const output = await runtime.agent.run({
+      session,
+      input: prompt,
+      mode: "build",
+      provider: target.provider,
+      onChunk: (text) => {
+        streamed = true;
+        process.stdout.write(redactText(text, secretValues));
+      }
+    });
+    if (!streamed && output) {
+      process.stdout.write(redactText(output, secretValues));
+    }
+    if (!streamed || !output) process.stdout.write("\n");
+  } finally {
+    await runtime.sessions.persist(session.id).catch(() => {
+    });
+  }
+}
 function sessionLabel(session) {
   const name = typeof session.metadata["name"] === "string" ? session.metadata["name"] : void 0;
   const firstUser = session.messages.find((m) => m.role === "user");
@@ -11260,7 +11384,7 @@ var FileSearchFactory = class {
     return new ProjectFileSearch(options);
   }
 };
-var execFileAsync3 = promisify(execFile3);
+var execFileAsync4 = promisify(execFile3);
 var GIT_TIMEOUT_MS = 5e3;
 var MAX_FILES2 = 50;
 var MAX_FILES_FOR_DETAILS = 500;
@@ -11285,7 +11409,7 @@ async function fetchGitDiff(cwd) {
   if (!gitRoot) return null;
   if (await isInTransientGitState(gitRoot)) return null;
   const [shortstatOut, untrackedOut] = await Promise.all([
-    runGit2(
+    runGit3(
       [
         "--no-optional-locks",
         "diff",
@@ -11296,7 +11420,7 @@ async function fetchGitDiff(cwd) {
       ],
       gitRoot
     ),
-    runGit2(
+    runGit3(
       [
         "--no-optional-locks",
         "ls-files",
@@ -11319,7 +11443,7 @@ async function fetchGitDiff(cwd) {
     };
   }
   const [numstatOut, nameStatusOut] = await Promise.all([
-    runGit2(
+    runGit3(
       [
         "--no-optional-locks",
         "diff",
@@ -11331,7 +11455,7 @@ async function fetchGitDiff(cwd) {
       ],
       gitRoot
     ),
-    runGit2(
+    runGit3(
       [
         "--no-optional-locks",
         "diff",
@@ -11491,7 +11615,7 @@ function splitNulDelimited(stdout) {
   return stdout.split("\0").filter(Boolean);
 }
 async function resolveGitRoot(cwd) {
-  const output = await runGit2(["rev-parse", "--show-toplevel"], cwd);
+  const output = await runGit3(["rev-parse", "--show-toplevel"], cwd);
   const root = output?.trim();
   return root ? root : null;
 }
@@ -11601,10 +11725,10 @@ async function mapWithConcurrency(items, limit, mapper) {
   }
   return results;
 }
-async function runGit2(args, cwd) {
+async function runGit3(args, cwd) {
   const fullArgs = ["-c", "core.quotepath=false", ...args];
   try {
-    const { stdout } = await execFileAsync3("git", fullArgs, {
+    const { stdout } = await execFileAsync4("git", fullArgs, {
       cwd,
       timeout: GIT_TIMEOUT_MS,
       maxBuffer: 64 * 1024 * 1024,
@@ -31218,6 +31342,27 @@ function createProgram() {
       model: options.model
     });
   });
+  program.command("review").description("AI code review of local git changes").argument("[ref]", "git ref to diff against (e.g. HEAD~3, main); defaults to HEAD").option("--staged", "review only staged changes (git diff --cached)").option("--file <path>", "limit review to a specific file").option(
+    "--focus <area>",
+    "focus area: security, performance, correctness, style; repeat for multiple",
+    (val, acc) => {
+      acc.push(val);
+      return acc;
+    },
+    []
+  ).option("--provider <provider>", "provider override").option("--model <model>", "model override").option("-y, --yes", "approve permission requests").action(async (ref, options) => {
+    await reviewCommand({
+      cwd: program.opts().cwd,
+      config: program.opts().config,
+      ref,
+      staged: options.staged,
+      file: options.file,
+      focus: options.focus,
+      provider: options.provider,
+      model: options.model,
+      yes: options.yes
+    });
+  });
   program.command("projects").description('interactive project browser \u2014 Enter/c prints selected path (add shell fn: dc() { cd "$(deepcode projects)"; })').option("--path <path>", "root path to scan for git repos (default: $HOME)").action(async (options) => {
     await projectsCommand({
       cwd: options.path ?? process.env["HOME"] ?? program.opts().cwd