npm - @staff0rd/assist - Versions diffs - 0.239.1 → 0.240.0 - Mend

@staff0rd/assist 0.239.1 → 0.240.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

@@ -91,13 +91,14 @@ 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 [sha] [options]` - 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.
-  - `[sha]` - Review that commit's diff (`sha^..sha`) instead of the open PR. Files land under `.assist/reviews/<shortSha>/`, no GitHub lookup or posting happens, and `--refine` / `--apply` / `--submit` are rejected
+- `assist review [number] [options]` - 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.
+  - `[number]` - Run `gh pr checkout <number>` first, then review that PR's branch. If the checkout fails (dirty working tree, unknown PR number), the review aborts
   - `--no-prompt` - Skip all confirmations
   - `--submit` - Default the submit prompt to yes (or auto-submit when combined with `--no-prompt`)
   - `--force` - Clear all cached files and re-run every phase
   - `--refine` - Skip posting; launch an interactive Claude session that walks through `synthesis.md` and edits it in place. A subsequent `assist review` reuses the refined file and posts only the surviving findings
   - `--apply` - Skip posting; launch an interactive Claude session that walks through each finding asking apply/skip. Applied findings are fixed in the working tree (unstaged) and removed from `synthesis.md`; skipped findings stay so a subsequent `assist review` posts them. Cannot be combined with `--refine`
+  - `--backlog` - Skip posting; launch an interactive Claude session running `/bug` that files all findings (including `already-raised`) as a single bug backlog item with one phase per finding. `synthesis.md` is left untouched; `--submit` is ignored. Cannot be combined with `--refine` or `--apply`
   - `--verbose` - Disable the stacked-spinner UI and fall back to per-line log output. Non-TTY environments (CI) automatically use this mode
 - `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

package/dist/index.js CHANGED Viewed

@@ -6,7 +6,7 @@ import { Command } from "commander";
 // package.json
 var package_default = {
   name: "@staff0rd/assist",
-  version: "0.239.1",
+  version: "0.240.0",
   type: "module",
   main: "dist/index.js",
   bin: {
@@ -12863,6 +12863,9 @@ function registerRefactor(program2) {
   registerRestructure(refactorCommand);
 }
+// src/commands/review/review.ts
+import { execFileSync as execFileSync6 } from "child_process";
 // src/commands/review/formatPriorComments.ts
 function threadKey(c, byId) {
   if (c.threadId) return c.threadId;
@@ -12936,23 +12939,6 @@ ${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()}
 \`\`\`
@@ -13465,6 +13451,29 @@ async function runApplySession(synthesisPath) {
   await done2;
 }
+// src/commands/review/runBacklogSession.ts
+function buildBacklogPrompt(synthesisPath) {
+  return `/bug add each finding in ${synthesisPath} as a phase
+Read ${synthesisPath}. It contains a list of findings under the \`## Findings\` heading, each in a \`### Finding:\` block with Severity, Source, Location, Impact, and Recommendation fields.
+File ONE bug backlog item covering the review findings:
+- Include EVERY finding in the file as a phase on that single item \u2014 including findings whose Source is \`already-raised\`.
+- Each phase should be named after its finding's title and its tasks should capture the finding's Location, Impact, and Recommendation.
+- Use \`assist backlog add\` to create the item, then \`assist backlog add-phase\` for each finding.
+Important constraints:
+- Do not edit ${synthesisPath} \u2014 leave it untouched.
+- Do not post anything to a PR.
+- Do not stage, commit, or push any changes.`;
+}
+async function runBacklogSession(synthesisPath) {
+  const { done: done2 } = spawnClaude(buildBacklogPrompt(synthesisPath), {
+    allowEdits: true
+  });
+  await done2;
+}
 // src/commands/review/cachedReviewerResult.ts
 import { statSync as statSync2 } from "fs";
 function cachedReviewerResult(name, outputPath) {
@@ -14381,6 +14390,10 @@ function setupReviewDir(repoRoot, context, force) {
   return paths;
 }
 async function runPostSynthesis(synthesisPath, options2) {
+  if (options2.backlog) {
+    await runBacklogSession(synthesisPath);
+    return;
+  }
   if (options2.apply) {
     await runApplySession(synthesisPath);
     return;
@@ -14401,58 +14414,6 @@ async function reviewPr(repoRoot, options2) {
   console.log(`Done. Review folder: ${paths.reviewDir}`);
 }
-// src/commands/review/gatherShaContext.ts
-import { execSync as execSync44 } from "child_process";
-function resolveSha(ref, format2) {
-  const flag = format2 === "short" ? "--short=7 " : "";
-  try {
-    return execSync44(`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 = execSync44(`git diff --name-only ${range}`, {
-    encoding: "utf-8",
-    maxBuffer: 64 * 1024 * 1024
-  }).trim().split("\n").filter(Boolean);
-  const diff2 = execSync44(`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());
@@ -14460,41 +14421,40 @@ function resolveRepoRoot() {
   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");
+  if (options2.backlog && (options2.refine || options2.apply)) {
+    console.error(
+      "Error: --backlog cannot be combined with --refine or --apply."
+    );
+    process.exit(1);
+  }
+}
+function checkoutPr(number) {
+  try {
+    execFileSync6("gh", ["pr", "checkout", number], { stdio: "inherit" });
+  } catch {
+    console.error(`gh pr checkout ${number} failed; aborting.`);
+    process.exit(1);
+  }
 }
 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;
-  }
+  if (options2.number) checkoutPr(options2.number);
   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, or a single commit when a SHA is given"
+    "Run Claude and Codex in parallel to review the current branch's PR, or check out a PR by number first when given"
   ).argument(
-    "[sha]",
-    "Optional commit SHA to review (sha^..sha); when provided, no PR lookup or GitHub posting happens"
+    "[number]",
+    "Optional PR number; when provided, runs `gh pr checkout <number>` before reviewing"
   ).option(
     "--no-prompt",
     "Skip confirmation prompts; use flag defaults non-interactively"
@@ -14510,11 +14470,14 @@ function registerReview(program2) {
   ).option(
     "--apply",
     "After synthesis, launch an interactive Claude session to apply fixes for each finding; applied findings are removed from synthesis, skipped ones remain for a later post"
+  ).option(
+    "--backlog",
+    "After synthesis, launch an interactive Claude session running /bug to file all findings as a single backlog item with one phase per finding, instead of posting to the PR"
   ).option(
     "--verbose",
     "Disable spinner UI and use per-line log output (per-tool lines, starting/done lines)"
   ).action(
-    (sha, options2) => review({ ...options2, sha })
+    (number, options2) => review({ ...options2, number })
   );
 }
@@ -15859,7 +15822,7 @@ import { mkdirSync as mkdirSync14 } from "fs";
 import { join as join45 } from "path";
 // src/commands/voice/checkLockFile.ts
-import { execSync as execSync45 } from "child_process";
+import { execSync as execSync44 } from "child_process";
 import { existsSync as existsSync44, mkdirSync as mkdirSync13, readFileSync as readFileSync34, writeFileSync as writeFileSync27 } from "fs";
 import { join as join44 } from "path";
 function isProcessAlive2(pid) {
@@ -15888,7 +15851,7 @@ function bootstrapVenv() {
   if (existsSync44(getVenvPython())) return;
   console.log("Setting up Python environment...");
   const pythonDir = getPythonDir();
-  execSync45(
+  execSync44(
     `uv sync --project "${pythonDir}" --extra runtime --no-install-project`,
     {
       stdio: "inherit",
@@ -16055,11 +16018,11 @@ import { randomBytes } from "crypto";
 import chalk153 from "chalk";
 // src/lib/openBrowser.ts
-import { execSync as execSync46 } from "child_process";
+import { execSync as execSync45 } from "child_process";
 function tryExec(commands) {
   for (const cmd of commands) {
     try {
-      execSync46(cmd);
+      execSync45(cmd);
       return true;
     } catch {
     }
@@ -16254,7 +16217,7 @@ async function auth() {
 }
 // src/commands/roam/postRoamActivity.ts
-import { execFileSync as execFileSync6 } from "child_process";
+import { execFileSync as execFileSync7 } from "child_process";
 import { readdirSync as readdirSync7, readFileSync as readFileSync37, statSync as statSync4 } from "fs";
 import { join as join47 } from "path";
 function findPortFile(roamDir) {
@@ -16287,7 +16250,7 @@ function postRoamActivity(app, event) {
   }
   const url = `http://127.0.0.1:${port}/api/v1/activity/${app}/${event}?pid=${app === "codex" ? 99998 : 99999}`;
   try {
-    execFileSync6("curl", ["-sf", "--max-time", "0.2", "-X", "POST", url], {
+    execFileSync7("curl", ["-sf", "--max-time", "0.2", "-X", "POST", url], {
       stdio: "ignore"
     });
   } catch {
@@ -16401,11 +16364,11 @@ function resolveParams(params, cliArgs) {
 }
 // src/commands/run/runPreCommands.ts
-import { execSync as execSync47 } from "child_process";
+import { execSync as execSync46 } from "child_process";
 function runPreCommands(pre, cwd) {
   for (const cmd of pre) {
     try {
-      execSync47(cmd, { stdio: "inherit", cwd });
+      execSync46(cmd, { stdio: "inherit", cwd });
     } catch (err) {
       const code = err && typeof err === "object" && "status" in err ? err.status : 1;
       process.exit(code);
@@ -16414,13 +16377,13 @@ function runPreCommands(pre, cwd) {
 }
 // src/commands/run/spawnRunCommand.ts
-import { execFileSync as execFileSync7, spawn as spawn8 } from "child_process";
+import { execFileSync as execFileSync8, spawn as spawn8 } from "child_process";
 import { existsSync as existsSync47 } from "fs";
 import { dirname as dirname25, join as join48, resolve as resolve11 } from "path";
 function resolveCommand2(command) {
   if (process.platform !== "win32" || command !== "bash") return command;
   try {
-    const gitPath = execFileSync7("where", ["git"], { encoding: "utf8" }).trim().split("\r\n")[0];
+    const gitPath = execFileSync8("where", ["git"], { encoding: "utf8" }).trim().split("\r\n")[0];
     const gitRoot = resolve11(dirname25(gitPath), "..");
     const gitBash = join48(gitRoot, "bin", "bash.exe");
     if (existsSync47(gitBash)) return gitBash;
@@ -16668,7 +16631,7 @@ function registerRun(program2) {
 }
 // src/commands/screenshot/index.ts
-import { execSync as execSync48 } from "child_process";
+import { execSync as execSync47 } from "child_process";
 import { existsSync as existsSync49, mkdirSync as mkdirSync17, unlinkSync as unlinkSync15, writeFileSync as writeFileSync30 } from "fs";
 import { tmpdir as tmpdir7 } from "os";
 import { join as join51, resolve as resolve13 } from "path";
@@ -16811,7 +16774,7 @@ function runPowerShellScript(processName, outputPath) {
   const scriptPath = join51(tmpdir7(), `assist-screenshot-${Date.now()}.ps1`);
   writeFileSync30(scriptPath, captureWindowPs1, "utf-8");
   try {
-    execSync48(
+    execSync47(
       `powershell -NoProfile -ExecutionPolicy Bypass -File "${scriptPath}" -ProcessName "${processName}" -OutputPath "${outputPath}"`,
       { stdio: ["ignore", "pipe", "pipe"], encoding: "utf-8" }
     );
@@ -16852,7 +16815,7 @@ function summaryPathFor(jsonlPath2) {
 }
 // src/commands/sessions/summarise/summariseSession.ts
-import { execFileSync as execFileSync8 } from "child_process";
+import { execFileSync as execFileSync9 } from "child_process";
 // src/commands/sessions/summarise/iterateUserMessages.ts
 import * as fs27 from "fs";
@@ -16929,7 +16892,7 @@ function summariseSession(jsonlPath2) {
   }
   const prompt = buildPrompt2(firstMessage, backlogIds);
   try {
-    const output = execFileSync8("claude", ["-p", "--model", "haiku", prompt], {
+    const output = execFileSync9("claude", ["-p", "--model", "haiku", prompt], {
       encoding: "utf8",
       timeout: 3e4,
       stdio: ["ignore", "pipe", "ignore"]
@@ -17199,7 +17162,7 @@ function syncCommands(claudeDir, targetBase) {
 }
 // src/commands/update.ts
-import { execSync as execSync49 } from "child_process";
+import { execSync as execSync48 } from "child_process";
 import * as path51 from "path";
 function isGlobalNpmInstall(dir) {
   try {
@@ -17207,7 +17170,7 @@ function isGlobalNpmInstall(dir) {
     if (resolved.split(path51.sep).includes("node_modules")) {
       return true;
     }
-    const globalPrefix = execSync49("npm prefix -g", { stdio: "pipe" }).toString().trim();
+    const globalPrefix = execSync48("npm prefix -g", { stdio: "pipe" }).toString().trim();
     return resolved.toLowerCase().startsWith(path51.resolve(globalPrefix).toLowerCase());
   } catch {
     return false;
@@ -17218,18 +17181,18 @@ async function update2() {
   console.log(`Assist is installed at: ${installDir}`);
   if (isGitRepo(installDir)) {
     console.log("Detected git repo installation, pulling latest...");
-    execSync49("git pull", { cwd: installDir, stdio: "inherit" });
+    execSync48("git pull", { cwd: installDir, stdio: "inherit" });
     console.log("Installing dependencies...");
-    execSync49("npm i", { cwd: installDir, stdio: "inherit" });
+    execSync48("npm i", { cwd: installDir, stdio: "inherit" });
     console.log("Building...");
-    execSync49("npm run build", { cwd: installDir, stdio: "inherit" });
+    execSync48("npm run build", { cwd: installDir, stdio: "inherit" });
     console.log("Syncing commands...");
-    execSync49("assist sync", { stdio: "inherit" });
+    execSync48("assist sync", { stdio: "inherit" });
   } else if (isGlobalNpmInstall(installDir)) {
     console.log("Detected global npm installation, updating...");
-    execSync49("npm i -g @staff0rd/assist@latest", { stdio: "inherit" });
+    execSync48("npm i -g @staff0rd/assist@latest", { stdio: "inherit" });
     console.log("Syncing commands...");
-    execSync49("assist sync", { stdio: "inherit" });
+    execSync48("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.239.1",
+	"version": "0.240.0",
 	"type": "module",
 	"main": "dist/index.js",
 	"bin": {