npm - @staff0rd/assist - Versions diffs - 0.219.0 → 0.220.0 - Mend

@staff0rd/assist 0.219.0 → 0.220.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (3) hide show

package/README.md CHANGED Viewed

@@ -88,7 +88,7 @@ After installation, the `assist` command will be available globally. You can als
 - `assist prs fixed <comment-id> <sha>` - Reply with commit link and resolve thread
 - `assist prs wontfix <comment-id> <reason>` - Reply with reason and resolve thread
 - `assist prs comment <path> <line> <body>` - Add a line comment to the pending review
-- `assist review [--no-prompt] [--submit] [--force] [--refine] [--apply] [--verbose]` - Run Claude and Codex in parallel to review the open PR for the current branch. The diff is fetched from GitHub (base SHA → head SHA via `gh pr diff`), so stale local base branches don't pollute the review; fails fast if no PR is open. By default, prompts before posting line-bound comments and then prompts again to submit the pending review (defaulting to no). Cached `claude.md` / `codex.md` / `synthesis.md` are reused when present; if any reviewer is re-run, the synthesis is invalidated. `--no-prompt` skips all confirmations. `--submit` defaults the submit prompt to yes (or auto-submits when combined with `--no-prompt`). `--force` clears all cached files and re-runs every phase. `--refine` skips posting and instead launches an interactive Claude session that walks through `synthesis.md` and edits it in place; a subsequent `assist review` (no flag) reuses the refined file and posts only the surviving findings. `--apply` skips posting and instead launches an interactive Claude session that walks through each remaining finding asking apply/skip; applied findings have their code fix made in the working tree (unstaged) and are removed from `synthesis.md`, while skipped findings stay so a subsequent `assist review` posts them. `--apply` cannot be combined with `--refine`. `--verbose` disables the stacked-spinner UI and falls back to per-line log output (per-tool lines, starting/done lines); non-TTY environments (CI) automatically fall back to verbose-style output
+- `assist review [sha] [--no-prompt] [--submit] [--force] [--refine] [--apply] [--verbose]` - Run Claude and Codex in parallel to review the open PR for the current branch. The diff is fetched from GitHub (base SHA → head SHA via `gh pr diff`), so stale local base branches don't pollute the review; fails fast if no PR is open. By default, prompts before posting line-bound comments and then prompts again to submit the pending review (defaulting to no). Cached `claude.md` / `codex.md` / `synthesis.md` are reused when present; if any reviewer is re-run, the synthesis is invalidated. `--no-prompt` skips all confirmations. `--submit` defaults the submit prompt to yes (or auto-submits when combined with `--no-prompt`). `--force` clears all cached files and re-runs every phase. `--refine` skips posting and instead launches an interactive Claude session that walks through `synthesis.md` and edits it in place; a subsequent `assist review` (no flag) reuses the refined file and posts only the surviving findings. `--apply` skips posting and instead launches an interactive Claude session that walks through each remaining finding asking apply/skip; applied findings have their code fix made in the working tree (unstaged) and are removed from `synthesis.md`, while skipped findings stay so a subsequent `assist review` posts them. `--apply` cannot be combined with `--refine`. `--verbose` disables the stacked-spinner UI and falls back to per-line log output (per-tool lines, starting/done lines); non-TTY environments (CI) automatically fall back to verbose-style output. Pass a commit `<sha>` to review that commit's diff (`sha^..sha`) instead; the review files land under `.assist/reviews/<shortSha>/`, no GitHub lookup or posting happens, and `--refine` / `--apply` / `--submit` are rejected.
 - `assist news` - Start the news web UI showing latest RSS feed items (same as `news web`)
 - `assist news add [url]` - Add an RSS feed URL to the config
 - `assist news web [-p, --port <number>]` - Start a web view of the news feeds (default port 3001)

package/dist/index.js CHANGED Viewed

@@ -6,7 +6,7 @@ import { Command } from "commander";
 // package.json
 var package_default = {
   name: "@staff0rd/assist",
-  version: "0.219.0",
+  version: "0.220.0",
   type: "module",
   main: "dist/index.js",
   bin: {
@@ -11639,60 +11639,6 @@ function registerRefactor(program2) {
   registerRestructure(refactorCommand);
 }
-// src/commands/review/buildRequest.ts
-import { execSync as execSync38 } from "child_process";
-// src/commands/review/fetchPrDiffInfo.ts
-import { execSync as execSync37 } from "child_process";
-function getCurrentBranch2() {
-  return execSync37("git rev-parse --abbrev-ref HEAD", {
-    encoding: "utf-8"
-  }).trim();
-}
-function fetchPrDiffInfo() {
-  const { org, repo } = getRepoInfo();
-  const branch = getCurrentBranch2();
-  const fields = "number,baseRefName,baseRefOid,headRefName,headRefOid";
-  let raw;
-  try {
-    raw = execSync37(`gh pr view ${branch} --json ${fields} -R ${org}/${repo}`, {
-      encoding: "utf-8",
-      stdio: ["ignore", "pipe", "pipe"]
-    });
-  } catch (error) {
-    if (error instanceof Error && error.message.includes("no pull requests")) {
-      console.error(
-        `Error: No open pull request found for branch \`${branch}\`. Open a PR for this branch before running \`assist review\`.`
-      );
-      process.exit(1);
-    }
-    throw error;
-  }
-  const parsed = JSON.parse(raw);
-  return {
-    prNumber: parsed.number,
-    baseRef: parsed.baseRefName,
-    baseSha: parsed.baseRefOid,
-    headRef: parsed.headRefName,
-    headSha: parsed.headRefOid
-  };
-}
-function fetchPrChangedFiles(prNumber) {
-  const { org, repo } = getRepoInfo();
-  const out = execSync37(`gh pr diff ${prNumber} --name-only -R ${org}/${repo}`, {
-    encoding: "utf-8",
-    maxBuffer: 64 * 1024 * 1024
-  });
-  return out.trim().split("\n").filter(Boolean);
-}
-function fetchPrDiff(prNumber) {
-  const { org, repo } = getRepoInfo();
-  return execSync37(`gh pr diff ${prNumber} -R ${org}/${repo}`, {
-    encoding: "utf-8",
-    maxBuffer: 256 * 1024 * 1024
-  });
-}
 // src/commands/review/formatPriorComments.ts
 function threadKey(c, byId) {
   if (c.threadId) return c.threadId;
@@ -11743,30 +11689,6 @@ ${blocks.join("\n\n")}`;
 }
 // src/commands/review/buildRequest.ts
-function gatherContext() {
-  const branch = execSync38("git rev-parse --abbrev-ref HEAD", {
-    encoding: "utf-8"
-  }).trim();
-  const sha = execSync38("git rev-parse HEAD", { encoding: "utf-8" }).trim();
-  const shortSha = execSync38("git rev-parse --short=7 HEAD", {
-    encoding: "utf-8"
-  }).trim();
-  const prInfo = fetchPrDiffInfo();
-  const changedFiles = fetchPrChangedFiles(prInfo.prNumber);
-  const diff2 = fetchPrDiff(prInfo.prNumber);
-  return {
-    branch,
-    sha,
-    shortSha,
-    prNumber: prInfo.prNumber,
-    baseRef: prInfo.baseRef,
-    baseSha: prInfo.baseSha,
-    headRef: prInfo.headRef,
-    headSha: prInfo.headSha,
-    changedFiles,
-    diff: diff2
-  };
-}
 function formatFiles(files) {
   if (files.length === 0) return "(none)";
   return files.map((file) => `- ${file}`).join("\n");
@@ -11790,6 +11712,23 @@ ${formatFiles(context.changedFiles)}
 ${priorBlock}
 ## Diff (PR #${context.prNumber}: ${context.baseSha}..${context.headSha})
+\`\`\`diff
+${context.diff.trimEnd()}
+\`\`\`
+`;
+}
+function buildShaRequest(context) {
+  return `# Code review request
+- Commit: \`${context.sha}\`
+- Parent: \`${context.parentSha}\`
+## Changed files
+${formatFiles(context.changedFiles)}
+## Diff (commit ${context.sha}: ${context.parentSha}..${context.sha})
 \`\`\`diff
 ${context.diff.trimEnd()}
 \`\`\`
@@ -11798,13 +11737,8 @@ ${context.diff.trimEnd()}
 // src/commands/review/buildReviewPaths.ts
 import { join as join35 } from "path";
-function buildReviewPaths(repoRoot, branch, shortSha) {
-  const reviewDir = join35(
-    repoRoot,
-    ".assist",
-    "reviews",
-    `${branch}-${shortSha}`
-  );
+function buildReviewPaths(repoRoot, key) {
+  const reviewDir = join35(repoRoot, ".assist", "reviews", key);
   return {
     reviewDir,
     requestPath: join35(reviewDir, "request.md"),
@@ -11815,9 +11749,9 @@ function buildReviewPaths(repoRoot, branch, shortSha) {
 }
 // src/commands/review/fetchExistingComments.ts
-import { execSync as execSync39 } from "child_process";
+import { execSync as execSync37 } from "child_process";
 function fetchRawComments(org, repo, prNumber) {
-  const out = execSync39(
+  const out = execSync37(
     `gh api --paginate repos/${org}/${repo}/pulls/${prNumber}/comments`,
     { encoding: "utf-8", maxBuffer: 64 * 1024 * 1024 }
   );
@@ -11847,6 +11781,86 @@ function fetchExistingComments() {
   });
 }
+// src/commands/review/gatherContext.ts
+import { execSync as execSync39 } from "child_process";
+// src/commands/review/fetchPrDiffInfo.ts
+import { execSync as execSync38 } from "child_process";
+function getCurrentBranch2() {
+  return execSync38("git rev-parse --abbrev-ref HEAD", {
+    encoding: "utf-8"
+  }).trim();
+}
+function fetchPrDiffInfo() {
+  const { org, repo } = getRepoInfo();
+  const branch = getCurrentBranch2();
+  const fields = "number,baseRefName,baseRefOid,headRefName,headRefOid";
+  let raw;
+  try {
+    raw = execSync38(`gh pr view ${branch} --json ${fields} -R ${org}/${repo}`, {
+      encoding: "utf-8",
+      stdio: ["ignore", "pipe", "pipe"]
+    });
+  } catch (error) {
+    if (error instanceof Error && error.message.includes("no pull requests")) {
+      console.error(
+        `Error: No open pull request found for branch \`${branch}\`. Open a PR for this branch before running \`assist review\`.`
+      );
+      process.exit(1);
+    }
+    throw error;
+  }
+  const parsed = JSON.parse(raw);
+  return {
+    prNumber: parsed.number,
+    baseRef: parsed.baseRefName,
+    baseSha: parsed.baseRefOid,
+    headRef: parsed.headRefName,
+    headSha: parsed.headRefOid
+  };
+}
+function fetchPrChangedFiles(prNumber) {
+  const { org, repo } = getRepoInfo();
+  const out = execSync38(`gh pr diff ${prNumber} --name-only -R ${org}/${repo}`, {
+    encoding: "utf-8",
+    maxBuffer: 64 * 1024 * 1024
+  });
+  return out.trim().split("\n").filter(Boolean);
+}
+function fetchPrDiff(prNumber) {
+  const { org, repo } = getRepoInfo();
+  return execSync38(`gh pr diff ${prNumber} -R ${org}/${repo}`, {
+    encoding: "utf-8",
+    maxBuffer: 256 * 1024 * 1024
+  });
+}
+// src/commands/review/gatherContext.ts
+function gatherContext() {
+  const branch = execSync39("git rev-parse --abbrev-ref HEAD", {
+    encoding: "utf-8"
+  }).trim();
+  const sha = execSync39("git rev-parse HEAD", { encoding: "utf-8" }).trim();
+  const shortSha = execSync39("git rev-parse --short=7 HEAD", {
+    encoding: "utf-8"
+  }).trim();
+  const prInfo = fetchPrDiffInfo();
+  const changedFiles = fetchPrChangedFiles(prInfo.prNumber);
+  const diff2 = fetchPrDiff(prInfo.prNumber);
+  return {
+    branch,
+    sha,
+    shortSha,
+    prNumber: prInfo.prNumber,
+    baseRef: prInfo.baseRef,
+    baseSha: prInfo.baseSha,
+    headRef: prInfo.headRef,
+    headSha: prInfo.headSha,
+    changedFiles,
+    diff: diff2
+  };
+}
 // src/commands/review/postReviewToPr.ts
 import { readFileSync as readFileSync29 } from "fs";
@@ -12866,19 +12880,7 @@ async function runReviewPipeline(paths, options2) {
   }
 }
-// src/commands/review/review.ts
-function resolveRepoRoot() {
-  const repoRoot = findRepoRoot(process.cwd());
-  if (repoRoot) return repoRoot;
-  console.error("Error: not inside a git repository.");
-  process.exit(1);
-}
-function validateOptions(options2) {
-  if (options2.apply && options2.refine) {
-    console.error("Error: --apply cannot be combined with --refine.");
-    process.exit(1);
-  }
-}
+// src/commands/review/reviewPr.ts
 function logPriorComments(count) {
   if (count === 0) return;
   console.log(`Including ${count} prior review comment(s) in request.md.`);
@@ -12892,7 +12894,10 @@ function gatherChangedContext() {
   process.exit(1);
 }
 function setupReviewDir(repoRoot, context, force) {
-  const paths = buildReviewPaths(repoRoot, context.branch, context.shortSha);
+  const paths = buildReviewPaths(
+    repoRoot,
+    `${context.branch}-${context.shortSha}`
+  );
   const priorComments = fetchExistingComments();
   logPriorComments(priorComments?.length ?? 0);
   prepareReviewDir(paths, buildRequest(context, priorComments), force);
@@ -12910,9 +12915,7 @@ async function runPostSynthesis(synthesisPath, options2) {
     submit: options2.submit ?? false
   });
 }
-async function review(options2 = {}) {
-  validateOptions(options2);
-  const repoRoot = resolveRepoRoot();
+async function reviewPr(repoRoot, options2) {
   const context = gatherChangedContext();
   const paths = setupReviewDir(repoRoot, context, options2.force ?? false);
   const synthesisOk = await runReviewPipeline(paths, {
@@ -12922,10 +12925,100 @@ async function review(options2 = {}) {
   console.log(`Done. Review folder: ${paths.reviewDir}`);
 }
+// src/commands/review/gatherShaContext.ts
+import { execSync as execSync40 } from "child_process";
+function resolveSha(ref, format2) {
+  const flag = format2 === "short" ? "--short=7 " : "";
+  try {
+    return execSync40(`git rev-parse --verify ${flag}${ref}^{commit}`, {
+      encoding: "utf-8",
+      stdio: ["ignore", "pipe", "pipe"]
+    }).trim();
+  } catch {
+    console.error(`Error: could not resolve commit \`${ref}\`.`);
+    process.exit(1);
+  }
+}
+function gatherShaContext(ref) {
+  const sha = resolveSha(ref, "long");
+  const shortSha = resolveSha(sha, "short");
+  const parentSha = resolveSha(`${sha}^`, "long");
+  const range = `${parentSha}..${sha}`;
+  const changedFiles = execSync40(`git diff --name-only ${range}`, {
+    encoding: "utf-8",
+    maxBuffer: 64 * 1024 * 1024
+  }).trim().split("\n").filter(Boolean);
+  const diff2 = execSync40(`git diff ${range}`, {
+    encoding: "utf-8",
+    maxBuffer: 256 * 1024 * 1024
+  });
+  return { sha, shortSha, parentSha, changedFiles, diff: diff2 };
+}
+// src/commands/review/reviewSha.ts
+function gatherShaChangedContext(ref) {
+  const context = gatherShaContext(ref);
+  if (context.changedFiles.length > 0) return context;
+  console.error(
+    `Error: commit ${context.sha} has no changed files \u2014 nothing to review.`
+  );
+  process.exit(1);
+}
+function setupShaReviewDir(repoRoot, context, force) {
+  const paths = buildReviewPaths(repoRoot, context.shortSha);
+  prepareReviewDir(paths, buildShaRequest(context), force);
+  console.log(`Review folder: ${paths.reviewDir}`);
+  return paths;
+}
+async function reviewSha(repoRoot, options2) {
+  const context = gatherShaChangedContext(options2.sha);
+  const paths = setupShaReviewDir(repoRoot, context, options2.force ?? false);
+  await runReviewPipeline(paths, { verbose: options2.verbose ?? false });
+  console.log(`Done. Review folder: ${paths.reviewDir}`);
+}
+// src/commands/review/review.ts
+function resolveRepoRoot() {
+  const repoRoot = findRepoRoot(process.cwd());
+  if (repoRoot) return repoRoot;
+  console.error("Error: not inside a git repository.");
+  process.exit(1);
+}
+function rejectShaFlag(flag) {
+  console.error(`Error: ${flag} cannot be combined with a SHA argument.`);
+  process.exit(1);
+}
+function validateOptions(options2) {
+  if (options2.apply && options2.refine) {
+    console.error("Error: --apply cannot be combined with --refine.");
+    process.exit(1);
+  }
+  if (!options2.sha) return;
+  if (options2.refine) rejectShaFlag("--refine");
+  if (options2.apply) rejectShaFlag("--apply");
+  if (options2.submit) rejectShaFlag("--submit");
+}
+async function review(options2 = {}) {
+  validateOptions(options2);
+  const repoRoot = resolveRepoRoot();
+  if (options2.sha) {
+    await reviewSha(repoRoot, {
+      sha: options2.sha,
+      force: options2.force,
+      verbose: options2.verbose
+    });
+    return;
+  }
+  await reviewPr(repoRoot, options2);
+}
 // src/commands/registerReview.ts
 function registerReview(program2) {
   program2.command("review").description(
-    "Run Claude and Codex in parallel to review the current branch"
+    "Run Claude and Codex in parallel to review the current branch, or a single commit when a SHA is given"
+  ).argument(
+    "[sha]",
+    "Optional commit SHA to review (sha^..sha); when provided, no PR lookup or GitHub posting happens"
   ).option(
     "--no-prompt",
     "Skip confirmation prompts; use flag defaults non-interactively"
@@ -12944,7 +13037,9 @@ function registerReview(program2) {
   ).option(
     "--verbose",
     "Disable spinner UI and use per-line log output (per-tool lines, starting/done lines)"
-  ).action((options2) => review(options2));
+  ).action(
+    (sha, options2) => review({ ...options2, sha })
+  );
 }
 // src/commands/seq/seqAuth.ts
@@ -14288,7 +14383,7 @@ import { mkdirSync as mkdirSync14 } from "fs";
 import { join as join45 } from "path";
 // src/commands/voice/checkLockFile.ts
-import { execSync as execSync40 } from "child_process";
+import { execSync as execSync41 } from "child_process";
 import { existsSync as existsSync42, mkdirSync as mkdirSync13, readFileSync as readFileSync34, writeFileSync as writeFileSync29 } from "fs";
 import { join as join44 } from "path";
 function isProcessAlive2(pid) {
@@ -14317,7 +14412,7 @@ function bootstrapVenv() {
   if (existsSync42(getVenvPython())) return;
   console.log("Setting up Python environment...");
   const pythonDir = getPythonDir();
-  execSync40(
+  execSync41(
     `uv sync --project "${pythonDir}" --extra runtime --no-install-project`,
     {
       stdio: "inherit",
@@ -14484,11 +14579,11 @@ import { randomBytes } from "crypto";
 import chalk143 from "chalk";
 // src/lib/openBrowser.ts
-import { execSync as execSync41 } from "child_process";
+import { execSync as execSync42 } from "child_process";
 function tryExec(commands) {
   for (const cmd of commands) {
     try {
-      execSync41(cmd);
+      execSync42(cmd);
       return true;
     } catch {
     }
@@ -14830,11 +14925,11 @@ function resolveParams(params, cliArgs) {
 }
 // src/commands/run/runPreCommands.ts
-import { execSync as execSync42 } from "child_process";
+import { execSync as execSync43 } from "child_process";
 function runPreCommands(pre, cwd) {
   for (const cmd of pre) {
     try {
-      execSync42(cmd, { stdio: "inherit", cwd });
+      execSync43(cmd, { stdio: "inherit", cwd });
     } catch (err) {
       const code = err && typeof err === "object" && "status" in err ? err.status : 1;
       process.exit(code);
@@ -15097,7 +15192,7 @@ function registerRun(program2) {
 }
 // src/commands/screenshot/index.ts
-import { execSync as execSync43 } from "child_process";
+import { execSync as execSync44 } from "child_process";
 import { existsSync as existsSync47, mkdirSync as mkdirSync17, unlinkSync as unlinkSync15, writeFileSync as writeFileSync32 } from "fs";
 import { tmpdir as tmpdir7 } from "os";
 import { join as join51, resolve as resolve13 } from "path";
@@ -15240,7 +15335,7 @@ function runPowerShellScript(processName, outputPath) {
   const scriptPath = join51(tmpdir7(), `assist-screenshot-${Date.now()}.ps1`);
   writeFileSync32(scriptPath, captureWindowPs1, "utf-8");
   try {
-    execSync43(
+    execSync44(
       `powershell -NoProfile -ExecutionPolicy Bypass -File "${scriptPath}" -ProcessName "${processName}" -OutputPath "${outputPath}"`,
       { stdio: ["ignore", "pipe", "pipe"], encoding: "utf-8" }
     );
@@ -15641,7 +15736,7 @@ function syncCommands(claudeDir, targetBase) {
 }
 // src/commands/update.ts
-import { execSync as execSync44 } from "child_process";
+import { execSync as execSync45 } from "child_process";
 import * as path51 from "path";
 function isGlobalNpmInstall(dir) {
   try {
@@ -15649,7 +15744,7 @@ function isGlobalNpmInstall(dir) {
     if (resolved.split(path51.sep).includes("node_modules")) {
       return true;
     }
-    const globalPrefix = execSync44("npm prefix -g", { stdio: "pipe" }).toString().trim();
+    const globalPrefix = execSync45("npm prefix -g", { stdio: "pipe" }).toString().trim();
     return resolved.toLowerCase().startsWith(path51.resolve(globalPrefix).toLowerCase());
   } catch {
     return false;
@@ -15660,18 +15755,18 @@ async function update2() {
   console.log(`Assist is installed at: ${installDir}`);
   if (isGitRepo(installDir)) {
     console.log("Detected git repo installation, pulling latest...");
-    execSync44("git pull", { cwd: installDir, stdio: "inherit" });
+    execSync45("git pull", { cwd: installDir, stdio: "inherit" });
     console.log("Installing dependencies...");
-    execSync44("npm i", { cwd: installDir, stdio: "inherit" });
+    execSync45("npm i", { cwd: installDir, stdio: "inherit" });
     console.log("Building...");
-    execSync44("npm run build", { cwd: installDir, stdio: "inherit" });
+    execSync45("npm run build", { cwd: installDir, stdio: "inherit" });
     console.log("Syncing commands...");
-    execSync44("assist sync", { stdio: "inherit" });
+    execSync45("assist sync", { stdio: "inherit" });
   } else if (isGlobalNpmInstall(installDir)) {
     console.log("Detected global npm installation, updating...");
-    execSync44("npm i -g @staff0rd/assist@latest", { stdio: "inherit" });
+    execSync45("npm i -g @staff0rd/assist@latest", { stdio: "inherit" });
     console.log("Syncing commands...");
-    execSync44("assist sync", { stdio: "inherit" });
+    execSync45("assist sync", { stdio: "inherit" });
   } else {
     console.error(
       "Could not determine installation method. Expected a git repo or global npm install."

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
 	"name": "@staff0rd/assist",
-	"version": "0.219.0",
+	"version": "0.220.0",
 	"type": "module",
 	"main": "dist/index.js",
 	"bin": {