@coreplane/switchboard 1.261.0 → 1.261.1

package/dist/assets/deploy/cloudflare-memory/worker.ts

@@ -2963,7 +2963,7 @@ export class RunHistoryDO extends DurableObject<Env> {
             req.card ? JSON.stringify(req.card) : null,
             req.system,
             JSON.stringify(req.tools),
-            JSON.stringify(req.state ?? {}),
+            JSON.stringify({ ...existing!.state, ...(req.state ?? {}) }),
             req.runId,
           );
           this.registerSession(req);

package/dist/assets/package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "switchboard",
-  "version": "1.261.0",
+  "version": "1.261.1",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "switchboard",
-      "version": "1.261.0",
+      "version": "1.261.1",
       "license": "Apache-2.0",
       "workspaces": [
         "web",
@@ -21060,7 +21060,7 @@
     },
     "packages/switchboard": {
       "name": "@coreplane/switchboard",
-      "version": "1.261.0",
+      "version": "1.261.1",
       "license": "Apache-2.0",
       "dependencies": {
         "@earendil-works/pi-ai": "0.85.1",

package/dist/assets/package.json

@@ -1,6 +1,6 @@
 {
   "name": "switchboard",
-  "version": "1.261.0",
+  "version": "1.261.1",
   "private": true,
   "description": "Mention it in Slack and an agent reviews the PR, ships the fix, or answers the question — on the model you choose, with its tools running where you decide.",
   "license": "Apache-2.0",

package/dist/assets/source.json

@@ -1,5 +1,5 @@
 {
-  "version": "1.261.0",
-  "commit": "c60b26943f774d0ba0e71af0df8db7d1d90613c5",
-  "builtAt": "2026-09-23T08:28:35.014Z"
+  "version": "1.261.1",
+  "commit": "3746a1c5fa2d30bebd66baec1fb3534a1f927e3a",
+  "builtAt": "2026-09-23T21:48:10.555Z"
 }

package/dist/assets/src/core/coordinator/driver.ts

@@ -510,8 +510,9 @@ function prCheckReturn(step: string, a: BotAnswer): Extract<StepReturn, { type:
       },
       at,
     };
-  // A merged pull request is read whole or not at all: the merge commit and
-  // the time are what the unit's ending and its report carry.
+  // A merged pull request is read whole or not at all: its source head is
+  // retained for findings reconciliation, while the merge commit and time
+  // are what the unit's ending and report carry.
   if (
     ok === true &&
     state === "merged" &&
@@ -527,6 +528,7 @@ function prCheckReturn(step: string, a: BotAnswer): Extract<StepReturn, { type:
         state: "merged",
         prNumber,
         url,
+        ...(typeof headSha === "string" ? { headSha } : {}),
         sha,
         mergedAt,
         ...(typeof mergedBy === "string" ? { mergedBy } : {}),
@@ -923,9 +925,19 @@ async function tellStepThrew(
       : at.round.kind === "review" && /\/review\/(read|checks)\b/.test(at.step)
         ? `after round ${at.round.index}'s review verdict (\`${at.step}\`)`
         : `in round ${at.round.index} (\`${at.step}\`)`;
-  const report =
-    `⚠️ The runner failed ${where}: ${line}\n\n` +
-    "The unit remains bound to this thread; the next reply continues it, and a pull request already approved with green checks resumes at the checks step, never at a fresh coding round.";
+  const findingsReconcile = at.round?.kind === "findings" && /\/findings(?:\/a\d+)?\/pr-check$/.test(at.step);
+  const report = findingsReconcile
+    ? [
+        `⚠️ Review did not restart because Switchboard could not read the pull request after the completed findings work: ${line}`,
+        pr !== undefined
+          ? `Saved-work fact: the existing pull request is ${pr.url}, but its current branch and exact head were not verified.`
+          : "Saved-work fact: no open pull request or exact remote head was verified.",
+        "No review was started. Next action: retry ship when the pull request is readable so Switchboard can verify the exact head first.",
+      ].join("\n\n")
+    : [
+        `⚠️ Switchboard failed ${where}: ${line}`,
+        "The unit remains resumable from its recorded branch and pull request facts. No continuation action was scheduled. Next action: start ship again after the failing operation is available.",
+      ].join("\n\n");
   const body = {
     ...tag,
     ending: {

package/dist/assets/src/core/reviewVerdict.ts

@@ -150,6 +150,10 @@ export interface ReviewVerdict {
 export const LGTM_TOKEN = "LGTM:";
 export const CHANGES_TOKEN = "Changes requested:";
 export const NO_VERDICT_LINE = "No verdict submitted — not approving.";
+/** Conservative ceiling below GitHub's 65,536-character review-body limit.
+ * Counted as Unicode code points so clipping never splits a surrogate pair. */
+export const MAX_REVIEW_POST_CODE_POINTS = 65_000;
+const REVIEW_TRUNCATION_NOTE = "_(review truncated to fit GitHub's review size limit)_";
 
 /** Parse an arbitrary tool input into a verdict, or null when it is not one.
  *  `addressSeverity` is the level in force for the run (default `minor`): an
@@ -383,7 +387,11 @@ function verdictMarker(verdict: ReviewVerdict | undefined, target: ReviewBodyTar
  * `approve`), a GitHub alert callout with the verdict word, the pinned head
  * and the finding counts, the findings as a table (severity, id + title, the
  * file linked at the head), the model's text folded under `Full review`, and
- * the machine-readable marker last. The prose decides nothing above the fold.
+ * the machine-readable marker last. Only the prose may be clipped to fit the
+ * platform: every fixed verdict section and the marker remain byte-for-byte.
+ * If those sections leave no room for a visible bounded review, the oversized
+ * body is returned so the publication boundary can refuse it instead of
+ * silently dropping the write-up. The prose decides nothing above the fold.
  */
 export function buildReviewPostBody(
   answer: string,
@@ -395,10 +403,10 @@ export function buildReviewPostBody(
     ...(target ? [`head \`${target.head.slice(0, 7)}\``] : []),
     verdict ? findingCounts(verdict.findings) : "the run ended without a submit_verdict call",
   ];
-  const parts: string[] = [verdictLine(verdict), `> [!${calloutKind(verdict)}]\n> ${facts.join(" · ")}`];
+  const fixedParts: string[] = [verdictLine(verdict), `> [!${calloutKind(verdict)}]\n> ${facts.join(" · ")}`];
   const findings = verdict?.findings ?? [];
   if (findings.length > 0) {
-    parts.push(
+    fixedParts.push(
       [
         "| Severity | Finding | Where |",
         "| --- | --- | --- |",
@@ -409,10 +417,29 @@ export function buildReviewPostBody(
       ].join("\n"),
     );
   }
+
+  const marker = verdictMarker(verdict, target);
+  const fixedBody = [...fixedParts, marker].join("\n\n");
   const prose = answer.trim();
-  if (prose) parts.push(`<details>\n<summary>Full review</summary>\n\n${prose}\n\n</details>`);
-  parts.push(verdictMarker(verdict, target));
-  return parts.join("\n\n");
+  if (!prose) return fixedBody;
+
+  const details = (text: string, truncated: boolean): string =>
+    `<details>\n<summary>Full review</summary>\n\n${text}${truncated ? `${text ? "\n\n" : ""}${REVIEW_TRUNCATION_NOTE}` : ""}\n\n</details>`;
+  const render = (text: string, truncated: boolean): string =>
+    [...fixedParts, details(text, truncated), marker].join("\n\n");
+  const full = render(prose, false);
+  if ([...full].length <= MAX_REVIEW_POST_CODE_POINTS) return full;
+
+  const clippedShell = render("", true);
+  const proseSeparatorCodePoints = 2; // the blank line before the note when clipped prose is present
+  const proseBudget = MAX_REVIEW_POST_CODE_POINTS - [...clippedShell].length - proseSeparatorCodePoints;
+  if (proseBudget > 0) return render([...prose].slice(0, proseBudget).join(""), true);
+  if ([...clippedShell].length <= MAX_REVIEW_POST_CODE_POINTS) return clippedShell;
+
+  // A successful post must retain a visible write-up. Return the full body
+  // oversized when even the bounded shell cannot fit, so postReviewComment
+  // rejects before fetch and the review remains on the Slack-only path.
+  return full;
 }
 
 /**