npm - @staff0rd/assist - Versions diffs - 0.209.1 → 0.210.1 - Mend

@staff0rd/assist 0.209.1 → 0.210.1

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 (4) hide show

package/README.md +1 -0
package/claude/commands/forward-comments.md +59 -0
package/dist/index.js +120 -48
package/package.json +1 -1

package/README.md CHANGED Viewed

@@ -39,6 +39,7 @@ After installation, the `assist` command will be available globally. You can als
 - `/commit` - Commit only relevant files from the session
 - `/devlog` - Generate devlog entry for the next unversioned day
 - `/draft` - Draft a new backlog item with LLM-assisted questioning
+- `/forward-comments` - Split a coarse PR comment (e.g. from Slack) into per-line review comments on the current branch's PR, attributed to the original reviewer
 - `/pr` - Raise a PR with a concise description
 - `/refactor` - Run refactoring checks for code quality
 - `/prompts` - Analyze denied tool calls and suggest settings changes to auto-allow recurring prompts

package/claude/commands/forward-comments.md ADDED Viewed

@@ -0,0 +1,59 @@
+---
+description: Split a coarse PR comment (e.g. from Slack) into per-line review comments on the current branch's PR, attributed to the original reviewer
+allowed_args: "<reviewer handle> — followed by the quoted comment text"
+---
+Take a multi-point comment that was given outside of GitHub's review UI (Slack, chat, in-person notes) and post each concern as a separate line-level review comment on the current branch's PR, attributed to the original reviewer with a `via @handle` prefix.
+## Inputs
+- **Reviewer handle** — the GitHub username of the person whose feedback this is (e.g. `Sebastian-MakerX`)
+- **Comment text** — the verbatim feedback, usually pasted into the user's prompt
+If either is missing or ambiguous, ask before proceeding. The PR is always the current branch's PR.
+## Steps
+1. **Split the comment into atomic concerns.** Each concern becomes one review comment. A concern is a single actionable point about a single location in the code. Framing prose ("the only thing slightly naff is..."), transitions ("Another thing is..."), and meta-comments ("looks pretty good") are NOT concerns — drop them.
+2. **For each concern, locate the target file and line.** Read the PR's changed files (`gh pr diff` or `gh api repos/.../pulls/<n>/files`) and the relevant source. Anchor each comment on the most specific line that the concern is about:
+   - A suggestion about a specific resolver/function → the line of that resolver
+   - A suggestion about a file's name or location → line 1 of the file
+   - A suggestion about a type definition → the line where the type is defined
+   - A suggestion about a return shape → the line that constructs/returns it
+3. **Compose each comment body.** Format:
+   ```
+   via @<handle>
+   > <verbatim quote of just the part relevant to this line>
+   ```
+   Rules for the quote:
+   - **Quote verbatim.** Preserve the reviewer's exact wording, including typos, repeated words, odd spacing, and punctuation. Do not paraphrase, "fix," or tidy. The reviewer's voice matters.
+   - **Drop framing prose.** Only the actionable suggestion that maps to *this specific line* should appear. If the reviewer's sentence opens with broad framing ("the only thing that's slightly naff is...") and then states the suggestion, quote only the suggestion part.
+   - **Add backticks around symbols.** Identifier names, type names, file names, and function names should be wrapped in backticks (e.g. `` `displayValue` ``, `` `SurveyQuestionModel` ``, `` `resolvers.ts` ``) even if the reviewer didn't use them. This is the only formatting change permitted.
+   - **No editorialising.** Do not add your own commentary, explanation, or restatement. The whole body is the attribution line plus the quote.
+4. **Confirm placements before posting.** List each planned comment to the user as `<file>:<line> — <one-line summary>`. Wait for the user to confirm or correct file/line choices.
+5. **Post each comment** via:
+   ```
+   assist prs comment '<path>' <line> '<body>' 2>&1
+   ```
+   - Always single-quote `<path>` and `<body>`. Never double-quote — backticks in the body would break.
+   - For apostrophes inside the body (e.g. `it's`), use the shell escape `'\''`.
+   - Bodies must not contain "claude" or "opus" (the command rejects them).
+   - Run posts in parallel when there are no dependencies between them.
+6. **Report** the resulting `<file>:<line>` line returned by each `assist prs comment` call, plus any failures.
+## Notes
+- `assist prs comment` publishes a new review thread immediately (despite the help text saying "pending review") — the comment appears live and notifies subscribers.
+- `assist prs comment` targets the **current branch's PR**. If the feedback is for a different PR, switch branches first.
+- If the same concern legitimately spans multiple lines/files, post one comment per anchor location, not a single combined comment.
+- If the reviewer's text contains a clear typo of a real symbol name (e.g. `SuveyQuestion` for `SurveyQuestion`), preserve the typo in the quote. Do not silently correct.

package/dist/index.js CHANGED Viewed

@@ -6,7 +6,7 @@ import { Command } from "commander";
 // package.json
 var package_default = {
   name: "@staff0rd/assist",
-  version: "0.209.1",
+  version: "0.210.1",
   type: "module",
   main: "dist/index.js",
   bin: {
@@ -8915,7 +8915,68 @@ import { tmpdir as tmpdir4 } from "os";
 import { join as join29 } from "path";
 // src/commands/prs/shared.ts
+import { execSync as execSync27 } from "child_process";
+// src/commands/prs/getPreferredRemoteRepo.ts
 import { execSync as execSync26 } from "child_process";
+var GITHUB_URL_PATTERN = /(?:git@github\.com:|https:\/\/github\.com\/)([^/]+)\/([^/]+?)(?:\.git)?\/?$/;
+function parseGitHubUrl(url) {
+  const match = url.match(GITHUB_URL_PATTERN);
+  if (!match) return null;
+  return { org: match[1], repo: match[2] };
+}
+function tryGetRemoteUrl(remote) {
+  try {
+    return execSync26(`git remote get-url ${remote}`, {
+      encoding: "utf-8",
+      stdio: ["pipe", "pipe", "pipe"]
+    }).trim();
+  } catch {
+    return null;
+  }
+}
+function getCurrentBranchRemote() {
+  try {
+    const ref = execSync26(
+      "git rev-parse --abbrev-ref --symbolic-full-name @{u}",
+      { encoding: "utf-8", stdio: ["pipe", "pipe", "pipe"] }
+    ).trim();
+    const [remote] = ref.split("/", 1);
+    return remote || null;
+  } catch {
+    return null;
+  }
+}
+function listRemotes() {
+  try {
+    return execSync26("git remote", {
+      encoding: "utf-8",
+      stdio: ["pipe", "pipe", "pipe"]
+    }).trim().split("\n").filter(Boolean);
+  } catch {
+    return [];
+  }
+}
+function getPreferredRemoteRepo() {
+  const candidates = [];
+  const tracked = getCurrentBranchRemote();
+  if (tracked) candidates.push(tracked);
+  if (!candidates.includes("origin")) candidates.push("origin");
+  for (const remote of listRemotes()) {
+    if (remote !== "upstream" && !candidates.includes(remote)) {
+      candidates.push(remote);
+    }
+  }
+  for (const remote of candidates) {
+    const url = tryGetRemoteUrl(remote);
+    if (!url) continue;
+    const parsed = parseGitHubUrl(url);
+    if (parsed) return parsed;
+  }
+  return null;
+}
+// src/commands/prs/shared.ts
 function isGhNotInstalled(error) {
   if (error instanceof Error) {
     const msg = error.message.toLowerCase();
@@ -8930,17 +8991,30 @@ function isNotFound(error) {
   return false;
 }
 function getRepoInfo() {
+  const preferred = getPreferredRemoteRepo();
+  if (preferred) return preferred;
   const repoInfo = JSON.parse(
-    execSync26("gh repo view --json owner,name", { encoding: "utf-8" })
+    execSync27("gh repo view --json owner,name", { encoding: "utf-8" })
   );
   return { org: repoInfo.owner.login, repo: repoInfo.name };
 }
+function getCurrentBranch() {
+  return execSync27("git rev-parse --abbrev-ref HEAD", {
+    encoding: "utf-8"
+  }).trim();
+}
+function viewCurrentPr(fields) {
+  const { org, repo } = getRepoInfo();
+  const branch = getCurrentBranch();
+  return JSON.parse(
+    execSync27(`gh pr view ${branch} --json ${fields} -R ${org}/${repo}`, {
+      encoding: "utf-8"
+    })
+  );
+}
 function getCurrentPrNumber() {
   try {
-    const prInfo = JSON.parse(
-      execSync26("gh pr view --json number", { encoding: "utf-8" })
-    );
-    return prInfo.number;
+    return viewCurrentPr("number").number;
   } catch (error) {
     if (error instanceof Error && error.message.includes("no pull requests")) {
       console.error("Error: No pull request found for the current branch.");
@@ -8951,10 +9025,7 @@ function getCurrentPrNumber() {
 }
 function getCurrentPrNodeId() {
   try {
-    const prInfo = JSON.parse(
-      execSync26("gh pr view --json id", { encoding: "utf-8" })
-    );
-    return prInfo.id;
+    return viewCurrentPr("id").id;
   } catch (error) {
     if (error instanceof Error && error.message.includes("no pull requests")) {
       console.error("Error: No pull request found for the current branch.");
@@ -9023,10 +9094,10 @@ function comment2(path53, line, body) {
 }
 // src/commands/prs/fixed.ts
-import { execSync as execSync28 } from "child_process";
+import { execSync as execSync29 } from "child_process";
 // src/commands/prs/resolveCommentWithReply.ts
-import { execSync as execSync27 } from "child_process";
+import { execSync as execSync28 } from "child_process";
 import { unlinkSync as unlinkSync8, writeFileSync as writeFileSync23 } from "fs";
 import { tmpdir as tmpdir5 } from "os";
 import { join as join31 } from "path";
@@ -9056,7 +9127,7 @@ function deleteCommentsCache(prNumber) {
 // src/commands/prs/resolveCommentWithReply.ts
 function replyToComment(org, repo, prNumber, commentId, message) {
-  execSync27(
+  execSync28(
     `gh api repos/${org}/${repo}/pulls/${prNumber}/comments -f body="${message.replace(/"/g, '\\"')}" -F in_reply_to=${commentId}`,
     { stdio: ["inherit", "pipe", "inherit"] }
   );
@@ -9066,7 +9137,7 @@ function resolveThread(threadId) {
   const queryFile = join31(tmpdir5(), `gh-mutation-${Date.now()}.graphql`);
   writeFileSync23(queryFile, mutation);
   try {
-    execSync27(
+    execSync28(
       `gh api graphql -F query=@${queryFile} -f threadId="${threadId}"`,
       { stdio: ["inherit", "pipe", "inherit"] }
     );
@@ -9118,7 +9189,7 @@ function resolveCommentWithReply(commentId, message) {
 // src/commands/prs/fixed.ts
 function verifySha(sha) {
   try {
-    return execSync28(`git rev-parse --verify ${sha}`, {
+    return execSync29(`git rev-parse --verify ${sha}`, {
       encoding: "utf-8"
     }).trim();
   } catch {
@@ -9132,7 +9203,7 @@ function fixed(commentId, sha) {
     const { org, repo } = getRepoInfo();
     const repoUrl = `https://github.com/${org}/${repo}`;
     const message = `Fixed in [${fullSha}](${repoUrl}/commit/${fullSha})`;
-    execSync28("git push", { stdio: "inherit" });
+    execSync29("git push", { stdio: "inherit" });
     resolveCommentWithReply(commentId, message);
   } catch (error) {
     if (isGhNotInstalled(error)) {
@@ -9150,7 +9221,7 @@ import { join as join33 } from "path";
 import { stringify } from "yaml";
 // src/commands/prs/fetchThreadIds.ts
-import { execSync as execSync29 } from "child_process";
+import { execSync as execSync30 } from "child_process";
 import { unlinkSync as unlinkSync9, writeFileSync as writeFileSync24 } from "fs";
 import { tmpdir as tmpdir6 } from "os";
 import { join as join32 } from "path";
@@ -9159,7 +9230,7 @@ function fetchThreadIds(org, repo, prNumber) {
   const queryFile = join32(tmpdir6(), `gh-query-${Date.now()}.graphql`);
   writeFileSync24(queryFile, THREAD_QUERY);
   try {
-    const result = execSync29(
+    const result = execSync30(
       `gh api graphql -F query=@${queryFile} -F owner="${org}" -F repo="${repo}" -F prNumber=${prNumber}`,
       { encoding: "utf-8" }
     );
@@ -9181,9 +9252,9 @@ function fetchThreadIds(org, repo, prNumber) {
 }
 // src/commands/prs/listComments/fetchReviewComments.ts
-import { execSync as execSync30 } from "child_process";
+import { execSync as execSync31 } from "child_process";
 function fetchJson(endpoint) {
-  const result = execSync30(`gh api --paginate ${endpoint}`, {
+  const result = execSync31(`gh api --paginate ${endpoint}`, {
     encoding: "utf-8"
   });
   if (!result.trim()) return [];
@@ -9322,7 +9393,7 @@ async function listComments() {
 }
 // src/commands/prs/prs/index.ts
-import { execSync as execSync31 } from "child_process";
+import { execSync as execSync32 } from "child_process";
 // src/commands/prs/prs/displayPaginated/index.ts
 import enquirer9 from "enquirer";
@@ -9428,8 +9499,9 @@ async function displayPaginated(pullRequests) {
 async function prs(options2) {
   const state = options2.open ? "open" : options2.closed ? "closed" : "all";
   try {
-    const result = execSync31(
-      `gh pr list --state ${state} --json number,title,url,author,createdAt,mergedAt,closedAt,state,changedFiles --limit 100`,
+    const { org, repo } = getRepoInfo();
+    const result = execSync32(
+      `gh pr list --state ${state} --json number,title,url,author,createdAt,mergedAt,closedAt,state,changedFiles --limit 100 -R ${org}/${repo}`,
       { encoding: "utf-8" }
     );
     const pullRequests = JSON.parse(result);
@@ -9451,7 +9523,7 @@ async function prs(options2) {
 }
 // src/commands/prs/wontfix.ts
-import { execSync as execSync32 } from "child_process";
+import { execSync as execSync33 } from "child_process";
 function validateReason(reason) {
   const lowerReason = reason.toLowerCase();
   if (lowerReason.includes("claude") || lowerReason.includes("opus")) {
@@ -9468,7 +9540,7 @@ function validateShaReferences(reason) {
   const invalidShas = [];
   for (const sha of shas) {
     try {
-      execSync32(`git cat-file -t ${sha}`, { stdio: "pipe" });
+      execSync33(`git cat-file -t ${sha}`, { stdio: "pipe" });
     } catch {
       invalidShas.push(sha);
     }
@@ -9582,10 +9654,10 @@ import chalk107 from "chalk";
 import Enquirer2 from "enquirer";
 // src/commands/ravendb/searchItems.ts
-import { execSync as execSync33 } from "child_process";
+import { execSync as execSync34 } from "child_process";
 import chalk106 from "chalk";
 function opExec(args) {
-  return execSync33(`op ${args}`, {
+  return execSync34(`op ${args}`, {
     encoding: "utf-8",
     stdio: ["pipe", "pipe", "pipe"]
   }).trim();
@@ -9737,7 +9809,7 @@ ${errorText}`
 }
 // src/commands/ravendb/resolveOpSecret.ts
-import { execSync as execSync34 } from "child_process";
+import { execSync as execSync35 } from "child_process";
 import chalk111 from "chalk";
 function resolveOpSecret(reference) {
   if (!reference.startsWith("op://")) {
@@ -9745,7 +9817,7 @@ function resolveOpSecret(reference) {
     process.exit(1);
   }
   try {
-    return execSync34(`op read "${reference}"`, {
+    return execSync35(`op read "${reference}"`, {
       encoding: "utf-8",
       stdio: ["pipe", "pipe", "pipe"]
     }).trim();
@@ -9994,7 +10066,7 @@ Refactor check failed:
 }
 // src/commands/refactor/check/getViolations/index.ts
-import { execSync as execSync35 } from "child_process";
+import { execSync as execSync36 } from "child_process";
 import fs18 from "fs";
 import { minimatch as minimatch4 } from "minimatch";
@@ -10044,7 +10116,7 @@ function getGitFiles(options2) {
   }
   const files = /* @__PURE__ */ new Set();
   if (options2.staged || options2.modified) {
-    const staged = execSync35("git diff --cached --name-only", {
+    const staged = execSync36("git diff --cached --name-only", {
       encoding: "utf-8"
     });
     for (const file of staged.trim().split("\n").filter(Boolean)) {
@@ -10052,7 +10124,7 @@ function getGitFiles(options2) {
     }
   }
   if (options2.unstaged || options2.modified) {
-    const unstaged = execSync35("git diff --name-only", { encoding: "utf-8" });
+    const unstaged = execSync36("git diff --name-only", { encoding: "utf-8" });
     for (const file of unstaged.trim().split("\n").filter(Boolean)) {
       files.add(file);
     }
@@ -12777,7 +12849,7 @@ import { mkdirSync as mkdirSync13 } from "fs";
 import { join as join43 } from "path";
 // src/commands/voice/checkLockFile.ts
-import { execSync as execSync36 } from "child_process";
+import { execSync as execSync37 } from "child_process";
 import { existsSync as existsSync38, mkdirSync as mkdirSync12, readFileSync as readFileSync32, writeFileSync as writeFileSync27 } from "fs";
 import { join as join42 } from "path";
 function isProcessAlive2(pid) {
@@ -12806,7 +12878,7 @@ function bootstrapVenv() {
   if (existsSync38(getVenvPython())) return;
   console.log("Setting up Python environment...");
   const pythonDir = getPythonDir();
-  execSync36(
+  execSync37(
     `uv sync --project "${pythonDir}" --extra runtime --no-install-project`,
     {
       stdio: "inherit",
@@ -12973,11 +13045,11 @@ import { randomBytes } from "crypto";
 import chalk140 from "chalk";
 // src/lib/openBrowser.ts
-import { execSync as execSync37 } from "child_process";
+import { execSync as execSync38 } from "child_process";
 function tryExec(commands) {
   for (const cmd of commands) {
     try {
-      execSync37(cmd);
+      execSync38(cmd);
       return true;
     } catch {
     }
@@ -13319,11 +13391,11 @@ function resolveParams(params, cliArgs) {
 }
 // src/commands/run/runPreCommands.ts
-import { execSync as execSync38 } from "child_process";
+import { execSync as execSync39 } from "child_process";
 function runPreCommands(pre, cwd) {
   for (const cmd of pre) {
     try {
-      execSync38(cmd, { stdio: "inherit", cwd });
+      execSync39(cmd, { stdio: "inherit", cwd });
     } catch (err) {
       const code = err && typeof err === "object" && "status" in err ? err.status : 1;
       process.exit(code);
@@ -13576,7 +13648,7 @@ function registerRun(program2) {
 }
 // src/commands/screenshot/index.ts
-import { execSync as execSync39 } from "child_process";
+import { execSync as execSync40 } from "child_process";
 import { existsSync as existsSync43, mkdirSync as mkdirSync16, unlinkSync as unlinkSync12, writeFileSync as writeFileSync30 } from "fs";
 import { tmpdir as tmpdir7 } from "os";
 import { join as join49, resolve as resolve13 } from "path";
@@ -13719,7 +13791,7 @@ function runPowerShellScript(processName, outputPath) {
   const scriptPath = join49(tmpdir7(), `assist-screenshot-${Date.now()}.ps1`);
   writeFileSync30(scriptPath, captureWindowPs1, "utf-8");
   try {
-    execSync39(
+    execSync40(
       `powershell -NoProfile -ExecutionPolicy Bypass -File "${scriptPath}" -ProcessName "${processName}" -OutputPath "${outputPath}"`,
       { stdio: ["ignore", "pipe", "pipe"], encoding: "utf-8" }
     );
@@ -14120,7 +14192,7 @@ function syncCommands(claudeDir, targetBase) {
 }
 // src/commands/update.ts
-import { execSync as execSync40 } from "child_process";
+import { execSync as execSync41 } from "child_process";
 import * as path52 from "path";
 function isGlobalNpmInstall(dir) {
   try {
@@ -14128,7 +14200,7 @@ function isGlobalNpmInstall(dir) {
     if (resolved.split(path52.sep).includes("node_modules")) {
       return true;
     }
-    const globalPrefix = execSync40("npm prefix -g", { stdio: "pipe" }).toString().trim();
+    const globalPrefix = execSync41("npm prefix -g", { stdio: "pipe" }).toString().trim();
     return resolved.toLowerCase().startsWith(path52.resolve(globalPrefix).toLowerCase());
   } catch {
     return false;
@@ -14139,18 +14211,18 @@ async function update2() {
   console.log(`Assist is installed at: ${installDir}`);
   if (isGitRepo(installDir)) {
     console.log("Detected git repo installation, pulling latest...");
-    execSync40("git pull", { cwd: installDir, stdio: "inherit" });
+    execSync41("git pull", { cwd: installDir, stdio: "inherit" });
     console.log("Installing dependencies...");
-    execSync40("npm i", { cwd: installDir, stdio: "inherit" });
+    execSync41("npm i", { cwd: installDir, stdio: "inherit" });
     console.log("Building...");
-    execSync40("npm run build", { cwd: installDir, stdio: "inherit" });
+    execSync41("npm run build", { cwd: installDir, stdio: "inherit" });
     console.log("Syncing commands...");
-    execSync40("assist sync", { stdio: "inherit" });
+    execSync41("assist sync", { stdio: "inherit" });
   } else if (isGlobalNpmInstall(installDir)) {
     console.log("Detected global npm installation, updating...");
-    execSync40("npm i -g @staff0rd/assist@latest", { stdio: "inherit" });
+    execSync41("npm i -g @staff0rd/assist@latest", { stdio: "inherit" });
     console.log("Syncing commands...");
-    execSync40("assist sync", { stdio: "inherit" });
+    execSync41("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.209.1",
+	"version": "0.210.1",
 	"type": "module",
 	"main": "dist/index.js",
 	"bin": {