deepcode-ai 1.1.24 → 1.1.26

package/dist/index.js

@@ -10360,6 +10360,163 @@ 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(", ")}`);
+}
+var DIFF_MAX_CHARS = 2e4;
+function truncateDiff(raw, maxChars = DIFF_MAX_CHARS) {
+  const fileChunks = raw.split(/(?=^diff --git )/m).filter(Boolean);
+  const totalFiles = fileChunks.length;
+  let result = "";
+  let included = 0;
+  for (const chunk of fileChunks) {
+    if (result.length + chunk.length > maxChars) break;
+    result += chunk;
+    included++;
+  }
+  if (!result && fileChunks.length > 0) {
+    result = fileChunks[0].slice(0, maxChars);
+    included = 1;
+  }
+  return { diff: result.trimEnd(), omittedFiles: totalFiles - included, totalFiles };
+}
+async function runGit(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, truncation) {
+  const focusLine = focus.length > 0 ? `
+Focus areas: ${focus.join(", ")}.` : "";
+  const truncationNote = truncation.omittedFiles > 0 ? `
+(Showing ${truncation.totalFiles - truncation.omittedFiles} of ${truncation.totalFiles} changed files; ${truncation.omittedFiles} file(s) omitted due to size.)
+` : "";
+  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 runGit(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;
+  }
+  const truncation = truncateDiff(trimmed);
+  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(truncation.diff, label, options.focus ?? [], truncation);
+  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(() => {
+    });
+  }
+}
 async function githubLoginCommand(options) {
   const loader = new ConfigLoader();
   const loadOptions = { cwd: options.cwd, configPath: options.config };
@@ -10539,8 +10696,8 @@ async function solveIssueCommand(issueNumber, options) {
   const issue = await client.getIssue({ ...repo, number: issueNumber });
   const base = options.base ?? "main";
   const branch = `deepcode/issue-${issueNumber}-${slugify(issue.title)}`.slice(0, 80);
-  await runGit(options.cwd, ["fetch", "origin", base]);
-  await runGit(options.cwd, ["checkout", "-B", branch, `origin/${base}`]);
+  await runGit2(options.cwd, ["fetch", "origin", base]);
+  await runGit2(options.cwd, ["checkout", "-B", branch, `origin/${base}`]);
   const target = resolveUsableProviderTarget(runtime.config, [runtime.config.defaultProvider]);
   const session = runtime.sessions.create({
     provider: target.provider,
@@ -10565,16 +10722,16 @@ async function solveIssueCommand(issueNumber, options) {
     onChunk: (text) => void writeStdout(redactText(text, secretValues))
   });
   await writeStdout("\n");
-  const status = await runGit(options.cwd, ["status", "--porcelain"]);
-  const aheadLog = await runGit(options.cwd, ["log", `origin/${base}..HEAD`, "--oneline"]);
+  const status = await runGit2(options.cwd, ["status", "--porcelain"]);
+  const aheadLog = await runGit2(options.cwd, ["log", `origin/${base}..HEAD`, "--oneline"]);
   const hasUncommitted = Boolean(status.stdout.trim());
   const hasCommits = Boolean(aheadLog.stdout.trim());
   if (!hasUncommitted && !hasCommits) {
     throw new Error("Agent completed without file changes; no PR was created.");
   }
   if (hasUncommitted) {
-    await runGit(options.cwd, ["add", "."]);
-    await runGit(options.cwd, [
+    await runGit2(options.cwd, ["add", "."]);
+    await runGit2(options.cwd, [
       "commit",
       "-m",
       `fix: resolve issue #${issue.number}`,
@@ -10584,7 +10741,7 @@ async function solveIssueCommand(issueNumber, options) {
 Closes #${issue.number}`
     ]);
   }
-  await runGit(options.cwd, ["push", "-u", "origin", branch]);
+  await runGit2(options.cwd, ["push", "-u", "origin", branch]);
   const pr = await client.createPullRequest({
     ...repo,
     title: `Fix: ${issue.title}`,
@@ -10623,17 +10780,23 @@ async function reviewPrCommand(prNumber, options) {
   ]);
   const focusLine = options.focus && options.focus.length > 0 ? `
 Focus areas: ${options.focus.join(", ")}.` : "";
+  const truncation = truncateDiff(diff.trim());
+  const truncationNote = truncation.omittedFiles > 0 ? `
+(Showing ${truncation.totalFiles - truncation.omittedFiles} of ${truncation.totalFiles} changed files; ${truncation.omittedFiles} file(s) omitted due to size.)
+` : "";
   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.",
     "",
     `Diff:
 \`\`\`diff
-${diff}
+${truncation.diff}
 \`\`\``,
+    truncationNote,
     "",
     `Produce a structured code review with:${focusLine}`,
     "1. **Summary** \u2014 what the PR does",
@@ -10641,21 +10804,30 @@ ${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, {
+    provider: options.provider,
+    model: options.model
+  });
   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) {
+async function runGit2(cwd, args) {
   const result = await execFileAsync("git", args, { cwd, timeoutMs: 18e4 });
   if (result.exitCode !== 0) {
     throw new Error(result.stderr || result.stdout || `git ${args.join(" ")} failed`);
@@ -10665,28 +10837,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,130 +11133,6 @@ 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");
@@ -31477,7 +31503,7 @@ function createProgram() {
     "focus area: security, performance, correctness, style; repeat for multiple",
     collectOption,
     []
-  ).action(async (number, options) => {
+  ).option("--provider <provider>", "provider override").option("--model <model>", "model override").action(async (number, options) => {
     const prNumber = Number.parseInt(number, 10);
     if (!Number.isInteger(prNumber) || prNumber <= 0) {
       throw new Error(`Invalid PR number: ${number}`);
@@ -31485,7 +31511,9 @@ function createProgram() {
     await reviewPrCommand(prNumber, {
       cwd: program.opts().cwd,
       config: program.opts().config,
-      focus: options.focus
+      focus: options.focus,
+      provider: options.provider,
+      model: options.model
     });
   });
   github.command("solve").description("solve a GitHub issue end-to-end with branch, commit, push, and PR").argument("<number>", "issue number").option("--base <base>", "base branch", "main").option("-y, --yes", "approve commit/push/PR workflow").action(async (number, options) => {