@staff0rd/assist 0.496.2 → 0.498.0

package/dist/index.js

@@ -6,7 +6,7 @@ import { Command } from "commander";
 // package.json
 var package_default = {
   name: "@staff0rd/assist",
-  version: "0.496.2",
+  version: "0.498.0",
   type: "module",
   main: "dist/index.js",
   bin: {
@@ -19738,7 +19738,18 @@ async function checkoutPr(number) {
 }
 
 // src/commands/reviewPrComments.ts
-async function reviewPrComments(number) {
+function buildPrompt2(number, announce) {
+  if (!announce) return "/review-pr-comments";
+  return `/review-pr-comments --announce ${number}`;
+}
+function validateAnnounce(number, announce) {
+  if (!announce || number) return;
+  console.error("Error: --announce requires a PR number.");
+  process.exit(1);
+}
+async function reviewPrComments(number, options2 = {}) {
+  const announce = options2.announce === true;
+  validateAnnounce(number, announce);
   if (number) await checkoutPr(number);
   const claudeSessionId = randomUUID6();
   emitActivity({
@@ -19746,7 +19757,7 @@ async function reviewPrComments(number) {
     name: "review-pr-comments",
     claudeSessionId
   });
-  const { done: done2 } = spawnClaude("/review-pr-comments", {
+  const { done: done2 } = spawnClaude(buildPrompt2(number, announce), {
     permissionMode: "acceptEdits",
     sessionId: claudeSessionId
   });
@@ -19774,7 +19785,12 @@ function registerDescriptionLaunch(program2, name, slashCommand, description, al
   if (alias) command.alias(alias);
 }
 function registerReviewPrComments(program2) {
-  program2.command("review-pr-comments").argument("[number]", "PR number to check out first").description("Launch Claude in /review-pr-comments mode (single session)").action((number) => reviewPrComments(number));
+  program2.command("review-pr-comments").argument("[number]", "PR number to check out first").description("Launch Claude in /review-pr-comments mode (single session)").option(
+    "--announce",
+    "After every comment thread is processed, announce the PR in Slack via /prs-slack <number> --no-confirm (requires a PR number)"
+  ).action(
+    (number, opts) => reviewPrComments(number, opts)
+  );
 }
 function registerLaunch(program2) {
   registerNext(program2);
@@ -24323,22 +24339,19 @@ function gatherContext() {
   };
 }
 
-// src/commands/review/postReviewToPr.ts
-import { readFileSync as readFileSync41 } from "fs";
-
-// src/commands/sessions/shared/requestAssistSession.ts
-function parseIncoming(line) {
+// src/commands/sessions/shared/requestSession.ts
+function parseIncoming(line, type) {
   try {
     const msg = JSON.parse(line);
-    if (msg.type === "error" && (msg.message ?? "").includes("create-assist"))
-      return { kind: "error", message: msg.message ?? "create-assist failed" };
+    if (msg.type === "error" && (msg.message ?? "").includes(type))
+      return { kind: "error", message: msg.message ?? `${type} failed` };
     if (msg.type !== "created" || !msg.sessionId) return null;
     return { kind: "created", sessionId: msg.sessionId };
   } catch {
     return null;
   }
 }
-function requestAssistSession(assistArgs, cwd) {
+function requestSession(message3) {
   return new Promise((resolve22, reject) => {
     connectToDaemon().then((socket) => {
       let settled = false;
@@ -24350,7 +24363,7 @@ function requestAssistSession(assistArgs, cwd) {
         else resolve22(sessionId);
       };
       readSocketLines(socket, (line) => {
-        const incoming = parseIncoming(line);
+        const incoming = parseIncoming(line, message3.type);
         if (!incoming) return;
         if (incoming.kind === "error") finish(new Error(incoming.message));
         else finish(void 0, incoming.sessionId);
@@ -24360,30 +24373,69 @@ function requestAssistSession(assistArgs, cwd) {
         "close",
         () => finish(new Error("daemon closed before the session was created"))
       );
-      socket.write(
-        `${JSON.stringify({ type: "create-assist", assistArgs, cwd })}
-`
-      );
+      socket.write(`${JSON.stringify(message3)}
+`);
     }, reject);
   });
 }
 
-// src/commands/review/chainAddressComments.ts
-async function chainAddressComments(prNumber) {
+// src/commands/sessions/shared/requestClaudeSession.ts
+function requestClaudeSession(prompt, cwd) {
+  return requestSession({ type: "create", prompt, cwd });
+}
+
+// src/commands/review/startChainedSession.ts
+async function startChainedSession(label2, start3) {
   if (process.env.ASSIST_SESSION !== "1") return;
   try {
-    await requestAssistSession(
-      ["review-pr-comments", String(prNumber)],
-      process.cwd()
-    );
-    console.log(`Started an Address Comments session for PR #${prNumber}.`);
+    await start3();
+    console.log(`Started ${label2}.`);
   } catch (error) {
     console.error(
-      `Warning: could not start an Address Comments session: ${error instanceof Error ? error.message : String(error)}`
+      `Warning: could not start ${label2}: ${error instanceof Error ? error.message : String(error)}`
     );
   }
 }
 
+// src/commands/review/announcePr.ts
+function announcePr(prNumber) {
+  return startChainedSession(
+    `a Slack announce session for PR #${prNumber}`,
+    () => requestClaudeSession(
+      `/prs-slack ${prNumber} --no-confirm`,
+      process.cwd()
+    )
+  );
+}
+
+// src/commands/review/postReviewToPr.ts
+import { readFileSync as readFileSync41 } from "fs";
+
+// src/commands/sessions/shared/requestAssistSession.ts
+function requestAssistSession(assistArgs, cwd) {
+  return requestSession({ type: "create-assist", assistArgs, cwd });
+}
+
+// src/commands/review/chainAddressComments.ts
+function chainAddressComments(prNumber, announce) {
+  const args = ["review-pr-comments", String(prNumber)];
+  if (announce) args.push("--announce");
+  return startChainedSession(
+    `an Address Comments session for PR #${prNumber}`,
+    () => requestAssistSession(args, process.cwd())
+  );
+}
+
+// src/commands/review/chainAfterReview.ts
+async function chainAfterReview(prNumber, outcome, options2) {
+  const raisedComments = outcome.posted > 0 && outcome.submitted;
+  if (options2.addressComments && raisedComments) {
+    await chainAddressComments(prNumber, options2.announce);
+    return;
+  }
+  if (options2.announce) await announcePr(prNumber);
+}
+
 // src/commands/review/parseFindings.ts
 var SEVERITIES = ["blocker", "major", "minor", "nit"];
 var SOURCES = [
@@ -24715,89 +24767,29 @@ function selectPostableFindings(markdown, prInfo) {
 }
 
 // src/commands/review/postReviewToPr.ts
+var NOTHING_POSTED = { posted: 0, submitted: false };
 async function confirmPost(prNumber, count8, options2) {
   if (!options2.prompt) return true;
   return promptConfirm(`Post ${count8} comment(s) to PR #${prNumber}?`, false);
 }
-async function postReviewToPr(synthesisPath, options2) {
-  const prInfo = fetchPrDiffInfo();
-  const prNumber = prInfo.prNumber;
+async function postFindingsToPr(prInfo, synthesisPath, options2) {
   const markdown = readFileSync41(synthesisPath, "utf8");
   const inDiff = selectPostableFindings(markdown, prInfo);
-  if (inDiff.length === 0) return;
+  if (inDiff.length === 0) return NOTHING_POSTED;
   console.log(
-    `Found PR #${prNumber} with ${inDiff.length} line-bound finding(s) in the diff.`
+    `Found PR #${prInfo.prNumber} with ${inDiff.length} line-bound finding(s) in the diff.`
   );
-  const confirmed = await confirmPost(prNumber, inDiff.length, options2);
+  const confirmed = await confirmPost(prInfo.prNumber, inDiff.length, options2);
   if (!confirmed) {
     console.log("Skipped posting.");
-    return;
-  }
-  const outcome = await postAndMaybeSubmit(inDiff, markdown, options2);
-  if (options2.addressComments && outcome.posted > 0 && outcome.submitted)
-    await chainAddressComments(prNumber);
-}
-
-// src/commands/review/runRefineSession.ts
-function buildRefinePrompt(synthesisPath) {
-  return `You are refining a code review synthesis with the user.
-
-Read ${synthesisPath} and walk the user through each finding one at a time. Before asking the user what to do with a finding, do some investigation so you can give an informed opinion:
-- Read the referenced file/lines (use the Location field) to confirm the issue is real and current.
-- Check nearby code, callers, or related files when the impact depends on context.
-- Form a view on whether the finding is applicable, whether the stated severity is right, and whether the recommendation actually fixes the problem.
-
-Then present your assessment briefly (one or two sentences) and ask the user whether to keep it as-is, edit it (e.g. adjust severity, sharpen the recommendation), or drop it. After all existing findings are reviewed, ask whether to add any new findings.
-
-Edit ${synthesisPath} in place so the resulting file is the user's final list:
-- To drop a finding, remove its entire \`### Finding:\` block.
-- To edit a finding, modify only the fields the user wants changed.
-- To add a new finding, append a block in the exact format below under the \`## Findings\` heading.
-
-Each finding block must use this exact format so it parses correctly:
-
-### Finding: <short title>
-- Severity: blocker | major | minor | nit
-- Source: confirmed | disputed | claude-only | codex-only | already-raised
-- Location: \`path/to/file.ext:LINE\` or \`n/a\` when not tied to a specific line
-- Impact: one sentence on what could go wrong
-- Recommendation: one or two sentences with a concrete change
-
-User-supplied findings should use Source: \`confirmed\`. Use Location \`n/a\` only when the finding is not tied to a specific line.
-
-Do not post anything to a PR; only edit ${synthesisPath}. When the user is done, exit.`;
-}
-async function runRefineSession(synthesisPath) {
-  const { done: done2 } = spawnClaude(buildRefinePrompt(synthesisPath), {
-    allowEdits: true
-  });
-  await done2;
-}
-
-// src/commands/review/handlePostSynthesis.ts
-async function handlePostSynthesis(synthesisPath, options2) {
-  if (options2.refine) {
-    await runRefineSession(synthesisPath);
-    return;
-  }
-  await postReviewToPr(synthesisPath, {
-    prompt: options2.prompt,
-    submit: options2.submit,
-    addressComments: options2.addressComments
-  });
-}
-
-// src/commands/review/prepareReviewDir.ts
-import { existsSync as existsSync49, mkdirSync as mkdirSync19, unlinkSync as unlinkSync17, writeFileSync as writeFileSync36 } from "fs";
-function clearReviewFiles(paths) {
-  for (const path71 of [paths.claudePath, paths.codexPath, paths.synthesisPath]) {
-    if (existsSync49(path71)) unlinkSync17(path71);
+    return NOTHING_POSTED;
   }
+  return postAndMaybeSubmit(inDiff, markdown, options2);
 }
-function prepareReviewDir(paths, requestBody, force) {
-  mkdirSync19(paths.reviewDir, { recursive: true });
-  if (force) clearReviewFiles(paths);
-  writeFileSync36(paths.requestPath, requestBody);
+async function postReviewToPr(synthesisPath, options2) {
+  const prInfo = fetchPrDiffInfo();
+  const outcome = await postFindingsToPr(prInfo, synthesisPath, options2);
+  await chainAfterReview(prInfo.prNumber, outcome, options2);
 }
 
 // src/commands/review/runApplySession.ts
@@ -24858,6 +24850,77 @@ async function runBacklogSession(synthesisPath) {
   await done2;
 }
 
+// src/commands/review/runRefineSession.ts
+function buildRefinePrompt(synthesisPath) {
+  return `You are refining a code review synthesis with the user.
+
+Read ${synthesisPath} and walk the user through each finding one at a time. Before asking the user what to do with a finding, do some investigation so you can give an informed opinion:
+- Read the referenced file/lines (use the Location field) to confirm the issue is real and current.
+- Check nearby code, callers, or related files when the impact depends on context.
+- Form a view on whether the finding is applicable, whether the stated severity is right, and whether the recommendation actually fixes the problem.
+
+Then present your assessment briefly (one or two sentences) and ask the user whether to keep it as-is, edit it (e.g. adjust severity, sharpen the recommendation), or drop it. After all existing findings are reviewed, ask whether to add any new findings.
+
+Edit ${synthesisPath} in place so the resulting file is the user's final list:
+- To drop a finding, remove its entire \`### Finding:\` block.
+- To edit a finding, modify only the fields the user wants changed.
+- To add a new finding, append a block in the exact format below under the \`## Findings\` heading.
+
+Each finding block must use this exact format so it parses correctly:
+
+### Finding: <short title>
+- Severity: blocker | major | minor | nit
+- Source: confirmed | disputed | claude-only | codex-only | already-raised
+- Location: \`path/to/file.ext:LINE\` or \`n/a\` when not tied to a specific line
+- Impact: one sentence on what could go wrong
+- Recommendation: one or two sentences with a concrete change
+
+User-supplied findings should use Source: \`confirmed\`. Use Location \`n/a\` only when the finding is not tied to a specific line.
+
+Do not post anything to a PR; only edit ${synthesisPath}. When the user is done, exit.`;
+}
+async function runRefineSession(synthesisPath) {
+  const { done: done2 } = spawnClaude(buildRefinePrompt(synthesisPath), {
+    allowEdits: true
+  });
+  await done2;
+}
+
+// src/commands/review/handlePostSynthesis.ts
+function nonPostingSession(options2) {
+  if (options2.backlog) return runBacklogSession;
+  if (options2.apply) return runApplySession;
+  if (options2.refine) return runRefineSession;
+  return null;
+}
+async function handlePostSynthesis(synthesisPath, prNumber, options2) {
+  const session = nonPostingSession(options2);
+  if (!session) {
+    await postReviewToPr(synthesisPath, {
+      prompt: options2.prompt,
+      submit: options2.submit,
+      addressComments: options2.addressComments,
+      announce: options2.announce
+    });
+    return;
+  }
+  await session(synthesisPath);
+  if (options2.announce) await announcePr(prNumber);
+}
+
+// src/commands/review/prepareReviewDir.ts
+import { existsSync as existsSync49, mkdirSync as mkdirSync19, unlinkSync as unlinkSync17, writeFileSync as writeFileSync36 } from "fs";
+function clearReviewFiles(paths) {
+  for (const path71 of [paths.claudePath, paths.codexPath, paths.synthesisPath]) {
+    if (existsSync49(path71)) unlinkSync17(path71);
+  }
+}
+function prepareReviewDir(paths, requestBody, force) {
+  mkdirSync19(paths.reviewDir, { recursive: true });
+  if (force) clearReviewFiles(paths);
+  writeFileSync36(paths.requestPath, requestBody);
+}
+
 // src/commands/review/cachedReviewerResult.ts
 import { statSync as statSync8 } from "fs";
 function cachedReviewerResult(name, outputPath) {
@@ -25771,20 +25834,15 @@ function setupReviewDir(repoRoot, context, force) {
   console.log(`Review folder: ${paths.reviewDir}`);
   return paths;
 }
-async function runPostSynthesis(synthesisPath, options2) {
-  if (options2.backlog) {
-    await runBacklogSession(synthesisPath);
-    return;
-  }
-  if (options2.apply) {
-    await runApplySession(synthesisPath);
-    return;
-  }
-  await handlePostSynthesis(synthesisPath, {
+function runPostSynthesis(synthesisPath, prNumber, options2) {
+  return handlePostSynthesis(synthesisPath, prNumber, {
     refine: options2.refine ?? false,
+    apply: options2.apply ?? false,
+    backlog: options2.backlog ?? false,
     prompt: options2.prompt ?? true,
     submit: options2.submit ?? false,
-    addressComments: options2.addressComments ?? false
+    addressComments: options2.addressComments ?? false,
+    announce: options2.announce ?? false
   });
 }
 async function reviewPr(repoRoot, options2) {
@@ -25793,7 +25851,8 @@ async function reviewPr(repoRoot, options2) {
   const synthesisOk = await runReviewPipeline(paths, {
     verbose: options2.verbose ?? false
   });
-  if (synthesisOk) await runPostSynthesis(paths.synthesisPath, options2);
+  if (synthesisOk)
+    await runPostSynthesis(paths.synthesisPath, context.prNumber, options2);
   console.log(`Done. Review folder: ${paths.reviewDir}`);
 }
 
@@ -25853,6 +25912,9 @@ function registerReview(program2) {
   ).option(
     "--address-comments",
     "After posting and submitting the review, start an Address Comments session (assist review-pr-comments <n>) for the PR; no-op when nothing was posted or the review was not submitted, and only inside an assist session"
+  ).option(
+    "--announce",
+    "Announce the PR in Slack (/prs-slack <n> --no-confirm) once the chain finishes: from the Address Comments session when one was started, otherwise from a session started directly; only inside an assist session"
   ).option(
     "--verbose",
     "Disable spinner UI and use per-line log output (per-tool lines, starting/done lines)"
@@ -26517,7 +26579,7 @@ function askQuestion(rl, question) {
 }
 
 // src/commands/transcript/configure.ts
-function buildPrompt2(label2, current) {
+function buildPrompt3(label2, current) {
   return current ? `${label2} [${current}]: ` : `${label2}: `;
 }
 function printExisting(existing) {
@@ -26530,15 +26592,15 @@ function printExisting(existing) {
 async function promptDirectories(rl, existing) {
   const vttDir = await askQuestion(
     rl,
-    buildPrompt2("VTT directory", existing?.vttDir)
+    buildPrompt3("VTT directory", existing?.vttDir)
   );
   const transcriptsDir = await askQuestion(
     rl,
-    buildPrompt2("Transcripts directory", existing?.transcriptsDir)
+    buildPrompt3("Transcripts directory", existing?.transcriptsDir)
   );
   const summaryDir = await askQuestion(
     rl,
-    buildPrompt2("Summary directory", existing?.summaryDir)
+    buildPrompt3("Summary directory", existing?.summaryDir)
   );
   return {
     vttDir: vttDir || existing?.vttDir || "",
@@ -29846,7 +29908,7 @@ async function generateSessionTitle(prompt) {
   try {
     const { stdout } = await execFileAsync7(
       "claude",
-      ["-p", "--model", "haiku", buildPrompt3(prompt)],
+      ["-p", "--model", "haiku", buildPrompt4(prompt)],
       { encoding: "utf8", timeout: 3e4 }
     );
     return normaliseTitle(stdout);
@@ -29858,7 +29920,7 @@ function normaliseTitle(stdout) {
   const title = (stdout.trim().split(/\r?\n/)[0] ?? "").trim().replace(/^["'`]+/, "").replace(/["'`]+$/, "").replace(/[.,:;!?]+$/, "").trim().slice(0, SESSION_TITLE_MAX_LENGTH).trim();
   return title || void 0;
 }
-function buildPrompt3(prompt) {
+function buildPrompt4(prompt) {
   return [
     "Summarise the task below as a short title for a session card.",
     "Rules:",
@@ -33402,7 +33464,7 @@ function summariseSession(jsonlPath2) {
   if (!firstMessage && backlogIds.length === 0) {
     return void 0;
   }
-  const prompt = buildPrompt4(firstMessage, backlogIds);
+  const prompt = buildPrompt5(firstMessage, backlogIds);
   try {
     const output = execFileSync15("claude", ["-p", "--model", "haiku", prompt], {
       encoding: "utf8",
@@ -33416,7 +33478,7 @@ function summariseSession(jsonlPath2) {
     return void 0;
   }
 }
-function buildPrompt4(firstMessage, backlogIds) {
+function buildPrompt5(firstMessage, backlogIds) {
   const parts = [
     "Summarise this Claude Code session in ONE short sentence (under 100 chars).",
     "Return ONLY the summary, no quotes or explanation."

package/package.json

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