@staff0rd/assist 0.518.0 → 0.520.0

package/dist/index.js

@@ -6,7 +6,7 @@ import { Command } from "commander";
 // package.json
 var package_default = {
   name: "@staff0rd/assist",
-  version: "0.518.0",
+  version: "0.520.0",
   type: "module",
   main: "dist/index.js",
   bin: {
@@ -10322,7 +10322,10 @@ async function fileContent(req, res) {
       respondJson(res, 413, { error: "File too large" });
       return;
     }
-    respondJson(res, 200, { content: await readFile(resolved.target, "utf8") });
+    respondJson(res, 200, {
+      content: await readFile(resolved.target, "utf8"),
+      mtimeMs: info.mtimeMs
+    });
   } catch {
     respondJson(res, 404, { error: "File not found" });
   }
@@ -12078,7 +12081,7 @@ async function uploadPrImage(req, res) {
 }
 
 // src/commands/sessions/web/writeFileContent.ts
-import { readFile as readFile2, writeFile as writeFile3 } from "fs/promises";
+import { readFile as readFile2, stat as stat3, writeFile as writeFile3 } from "fs/promises";
 
 // src/commands/sessions/web/formatWithOxfmt.ts
 import { execFile as execFile8 } from "child_process";
@@ -12108,26 +12111,50 @@ async function formatWithOxfmt(target, root) {
   });
 }
 
-// src/commands/sessions/web/writeFileContent.ts
-async function writeFileContent(req, res) {
-  const resolved = getFileTarget(req, res);
-  if (!resolved) return;
+// src/commands/sessions/web/getFileWriteBody.ts
+async function getFileWriteBody(req, res) {
   let body;
   try {
     body = await readJsonBody(req);
   } catch {
     respondJson(res, 400, { error: "Invalid JSON body" });
-    return;
+    return null;
   }
   if (typeof body.content !== "string") {
     respondJson(res, 400, { error: "Missing content" });
+    return null;
+  }
+  if (typeof body.mtimeMs !== "number") {
+    respondJson(res, 400, { error: "Missing mtimeMs" });
+    return null;
+  }
+  return { content: body.content, mtimeMs: body.mtimeMs };
+}
+
+// src/commands/sessions/web/writeFileContent.ts
+var STALE_MESSAGE = "The file changed on disk since it was opened";
+async function currentMtimeMs(target) {
+  try {
+    return (await stat3(target)).mtimeMs;
+  } catch {
+    return null;
+  }
+}
+async function writeFileContent(req, res) {
+  const resolved = getFileTarget(req, res);
+  if (!resolved) return;
+  const body = await getFileWriteBody(req, res);
+  if (!body) return;
+  if (await currentMtimeMs(resolved.target) !== body.mtimeMs) {
+    respondJson(res, 409, { error: STALE_MESSAGE });
     return;
   }
   try {
     await writeFile3(resolved.target, body.content, "utf8");
     await formatWithOxfmt(resolved.target, repoRoot(resolved.cwd));
     respondJson(res, 200, {
-      content: await readFile2(resolved.target, "utf8")
+      content: await readFile2(resolved.target, "utf8"),
+      mtimeMs: await currentMtimeMs(resolved.target)
     });
   } catch (error) {
     respondJson(res, 500, {
@@ -18499,10 +18526,10 @@ function findSlnFiles(dir, maxDepth, depth = 0) {
       continue;
     const full = path32.join(dir, entry);
     try {
-      const stat3 = statSync6(full);
-      if (stat3.isFile() && entry.endsWith(".sln")) {
+      const stat4 = statSync6(full);
+      if (stat4.isFile() && entry.endsWith(".sln")) {
         results.push(full);
-      } else if (stat3.isDirectory()) {
+      } else if (stat4.isDirectory()) {
         results.push(...findSlnFiles(full, maxDepth, depth + 1));
       }
     } catch {
@@ -22603,6 +22630,65 @@ function wontfix(commentId, reason4) {
   }
 }
 
+// src/commands/prs/prConcisenessGuidance.ts
+var prConcisenessGuidance = `Brevity budget \u2014 one paragraph per section, and these are ceilings, not targets:
+
+  ## What  2\u20133 sentences.
+  ## Why   1\u20132 sentences.
+  ## How   omit by default; a sentence or two, only for non-obvious decisions.
+
+The budget is a total, not a per-paragraph allowance. Splitting a section across
+two paragraphs buys no extra room. The whole body should read in well under a
+minute \u2014 roughly 6 sentences and 900 characters of prose for a typical change,
+counting every section together. A body that is long-winded overall is wrong even
+when each paragraph is individually short.
+
+Terse technical register. Write like a commit message, not a report. Short
+declarative sentences, one clause each: state the fact and stop. Cut
+scene-setting, restatements of the heading, "this change \u2026", "in order to", "it
+is worth noting", hedges ("essentially", "fairly", "somewhat"), and adverbs that
+carry no information. Present tense, active voice. If a sentence can lose half
+its words and still say the same thing, it is too long.
+
+  Wordy: This change introduces a new validation step so that the command is
+         able to reject pull request bodies which exceed the configured limit.
+  Terse: Rejects bodies over the limit.
+
+Sentence count is a floor on brevity, not the whole of it \u2014 a body inside the
+budget can still be too verbose, and then it needs cutting rather than reflowing.
+Prefer deleting a sentence to shortening it.
+
+Write prose. A short paragraph is the natural form for ## What and ## Why \u2014
+bullets in ## What are a smell. Use bullets only when there are genuinely several
+independent, parallel items (3+); a single bullet is never right. A single
+non-list paragraph over ~600 characters or ~4 sentences is a wall of text and
+will be rejected.`;
+
+// src/commands/prs/editHelpText.ts
+function editHelpText() {
+  return `
+Update individual sections of the current branch's pull request. Only the sections
+you pass are replaced; every other section of the existing body is preserved.
+
+Do not reference Claude or any AI assistance in the title or body. Wrap symbols,
+file paths, function names, class names, variable names, config keys, CLI
+commands, and flag names in backticks.
+
+A replacement section is held to the same standard as one written by
+'assist prs raise', and the budget applies to the resulting body as a whole \u2014 an
+edit must not push the body over it. Read the sections you are not touching first:
+if the section you are replacing was the concise one, trimming it further will not
+rescue a body that is already long-winded.
+
+  ## What  what is observably different for someone using or calling this.
+  ## Why   the problem or motivation that made the change worth doing.
+  ## How   only non-obvious decisions the diff alone won't explain; omit by
+           default.
+
+${prConcisenessGuidance}
+`;
+}
+
 // src/commands/registerPrsEdit.ts
 function collect3(value, previous) {
   return previous.concat([value]);
@@ -22615,7 +22701,7 @@ function registerPrsEdit(prsCommand) {
     "Jira issue key resolved by this PR, appended to ## Why (repeatable)",
     collect3,
     []
-  ).action(edit);
+  ).addHelpText("after", () => editHelpText()).action(edit);
 }
 
 // src/commands/prs/raiseHelpText.ts
@@ -22667,17 +22753,7 @@ a genuine non-obvious decision. Litmus \u2014 a sentence is almost certainly mec
 X filters/needs/uses", or names an internal component, property, function, or
 file.
 
-Brevity budget \u2014 keep each section within these limits:
-
-  ## What  a few sentences (2\u20133).
-  ## Why   1\u20132 sentences.
-  ## How   omit by default; a sentence or two, only for non-obvious decisions.
-
-Write prose. A short paragraph is the natural form for ## What and ## Why \u2014
-bullets in ## What are a smell. Use bullets only when there are genuinely several
-independent, parallel items (3+); a single bullet is never right. A single
-non-list paragraph over ~600 characters or ~4 sentences is a wall of text and
-will be rejected.
+${prConcisenessGuidance}
 
 If a pull request already exists for the branch, this command errors \u2014 pass
 --force to fully overwrite its title and body, or use 'assist prs edit' to update
@@ -34246,13 +34322,13 @@ import { readFileSync as readFileSync54, statSync as statSync14 } from "fs";
 import { isAbsolute as isAbsolute4, join as join78, resolve as resolve21 } from "path";
 function resolveGitDir(cwd) {
   const dotGit = join78(cwd, ".git");
-  let stat3;
+  let stat4;
   try {
-    stat3 = statSync14(dotGit);
+    stat4 = statSync14(dotGit);
   } catch {
     return null;
   }
-  if (stat3.isDirectory()) {
+  if (stat4.isDirectory()) {
     return dotGit;
   }
   let contents;

package/package.json

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