deepcode-ai 1.1.23 → 1.1.25

package/dist/index.js

@@ -10360,6 +10360,28 @@ async function initCommand(cwd) {
   const filePath = await new ConfigLoader().init(cwd);
   await writeStdoutLine(`DeepCode config created at ${filePath}`);
 }
+function resolveSessionTarget(config, overrides = {}) {
+  const requestedProvider = parseProviderId(overrides.provider);
+  const fallback = resolveUsableProviderTarget(config, [
+    requestedProvider,
+    config.defaultProvider
+  ]);
+  const parsedSelection = overrides.model ? parseModelSelection(overrides.model, requestedProvider ?? fallback.provider) : null;
+  if (overrides.model && !parsedSelection) {
+    throw new Error(
+      `Invalid model selection: ${overrides.model}. Use "<model>" or "<provider>/<model>".`
+    );
+  }
+  const provider = parsedSelection?.provider ?? requestedProvider ?? fallback.provider;
+  const model = parsedSelection?.model ?? resolveConfiguredModelForProvider(config, provider) ?? (provider === fallback.provider ? fallback.model : void 0);
+  return { provider, model };
+}
+function parseProviderId(value) {
+  if (!value) return void 0;
+  const parsed = ProviderIdSchema.safeParse(value);
+  if (parsed.success) return parsed.data;
+  throw new Error(`Invalid provider: ${value}. Expected one of: ${PROVIDER_IDS.join(", ")}`);
+}
 async function githubLoginCommand(options) {
   const loader = new ConfigLoader();
   const loadOptions = { cwd: options.cwd, configPath: options.config };
@@ -10626,6 +10648,7 @@ Focus areas: ${options.focus.join(", ")}.` : "";
   const prompt = [
     `Review PR #${pr.number}: ${pr.title}`,
     `Branch: ${pr.head ?? "?"} \u2192 ${pr.base ?? "?"}`,
+    "Do not modify any files. Output the review only.",
     "",
     pr.body ? `Description:
 ${pr.body}` : "No description provided.",
@@ -10641,18 +10664,24 @@ ${diff}
     "3. **Suggestions** \u2014 improvements and nitpicks",
     "4. **Verdict** \u2014 Approve / Request Changes / Neutral with a one-line rationale"
   ].join("\n");
-  const target = resolveUsableProviderTarget(runtime.config, [runtime.config.defaultProvider]);
+  const target = resolveSessionTarget(runtime.config, {});
   const session = runtime.sessions.create({
     provider: target.provider,
     model: target.model
   });
   const secretValues = collectSecretValues(runtime.config);
   await writeStdoutLine(`Reviewing PR #${pr.number}: ${pr.title}`);
-  await runtime.agent.run({
-    session,
-    input: prompt,
-    onChunk: (text) => void writeStdout(redactText(text, secretValues))
-  });
+  try {
+    await runtime.agent.run({
+      session,
+      input: prompt,
+      mode: "plan",
+      onChunk: (text) => void writeStdout(redactText(text, secretValues))
+    });
+  } finally {
+    await runtime.sessions.persist(session.id).catch(() => {
+    });
+  }
   await writeStdout("\n");
 }
 async function runGit(cwd, args) {
@@ -10665,28 +10694,6 @@ async function runGit(cwd, args) {
 function slugify(input) {
   return input.toLowerCase().normalize("NFKD").replace(/[^\w\s-]/g, "").trim().replace(/\s+/g, "-").replace(/-+/g, "-").slice(0, 48);
 }
-function resolveSessionTarget(config, overrides = {}) {
-  const requestedProvider = parseProviderId(overrides.provider);
-  const fallback = resolveUsableProviderTarget(config, [
-    requestedProvider,
-    config.defaultProvider
-  ]);
-  const parsedSelection = overrides.model ? parseModelSelection(overrides.model, requestedProvider ?? fallback.provider) : null;
-  if (overrides.model && !parsedSelection) {
-    throw new Error(
-      `Invalid model selection: ${overrides.model}. Use "<model>" or "<provider>/<model>".`
-    );
-  }
-  const provider = parsedSelection?.provider ?? requestedProvider ?? fallback.provider;
-  const model = parsedSelection?.model ?? resolveConfiguredModelForProvider(config, provider) ?? (provider === fallback.provider ? fallback.model : void 0);
-  return { provider, model };
-}
-function parseProviderId(value) {
-  if (!value) return void 0;
-  const parsed = ProviderIdSchema.safeParse(value);
-  if (parsed.success) return parsed.data;
-  throw new Error(`Invalid provider: ${value}. Expected one of: ${PROVIDER_IDS.join(", ")}`);
-}
 async function runCommand(input, options) {
   if (options.mode && options.mode !== "plan" && options.mode !== "build") {
     throw new Error(`Invalid mode: ${options.mode}. Expected plan or build.`);
@@ -10983,6 +10990,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: "plan",
+      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 +11391,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 +11416,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 +11427,7 @@ async function fetchGitDiff(cwd) {
       ],
       gitRoot
     ),
-    runGit2(
+    runGit3(
       [
         "--no-optional-locks",
         "ls-files",
@@ -11319,7 +11450,7 @@ async function fetchGitDiff(cwd) {
     };
   }
   const [numstatOut, nameStatusOut] = await Promise.all([
-    runGit2(
+    runGit3(
       [
         "--no-optional-locks",
         "diff",
@@ -11331,7 +11462,7 @@ async function fetchGitDiff(cwd) {
       ],
       gitRoot
     ),
-    runGit2(
+    runGit3(
       [
         "--no-optional-locks",
         "diff",
@@ -11491,7 +11622,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 +11732,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 +31349,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