@stigmer/runner 3.12.1 → 3.12.3

package/src/shared/__tests__/mcp-manager.test.ts

@@ -66,7 +66,7 @@ describe("toMcpClientConfig", () => {
     });
   });
 
-  it("maps HTTP servers to streamable_http transport", () => {
+  it("maps HTTP servers to streamable_http transport with reconnect enabled", () => {
     const servers = [
       makeServer({
         slug: "github",
@@ -80,10 +80,14 @@ describe("toMcpClientConfig", () => {
       transport: "http",
       url: "https://api.github.com/mcp",
       headers: { Authorization: "Bearer tok" },
+      // oss#316: severed remote connections (e.g. the hosted bridge
+      // restarting mid-execution) re-initialize instead of stranding the
+      // agent on a dead session.
+      reconnect: { enabled: true, maxAttempts: 3, delayMs: 1_000 },
     });
   });
 
-  it("maps SSE servers to streamable_http transport", () => {
+  it("maps SSE servers to streamable_http transport with reconnect enabled", () => {
     const servers = [
       makeServer({
         slug: "sse-server",
@@ -96,6 +100,7 @@ describe("toMcpClientConfig", () => {
       transport: "http",
       url: "https://sse.example.com/mcp",
       headers: undefined,
+      reconnect: { enabled: true, maxAttempts: 3, delayMs: 1_000 },
     });
   });
 

package/src/shared/attachment-vision.ts

@@ -32,6 +32,15 @@
 
 // ---------------------------------------------------------------------------
 // Budget constants (owner decision, 2026-08-09; project T04)
+//
+// ADVERTISED == ENFORCED (stigmer/stigmer#365): the model registry document
+// advertises these values in its `limits.vision` block (rendered by the
+// cloud's ModelRegistryDocumentCodec, mirrored verbatim by the OSS server)
+// so clients can warn BEFORE an upload this budget would degrade. Both sides
+// pin the literal values in tests — attachment-vision.test.ts here, the
+// codec test there — so changing either side alone fails its own suite.
+// Update both together; deploy the runner first if the budget ever shrinks
+// (never advertise more than the runner accepts).
 // ---------------------------------------------------------------------------
 
 /**

package/src/shared/filereview/__tests__/capture.test.ts

@@ -946,6 +946,54 @@ describe("capture orchestration — CAS-only non-git workspace (Slice 2a)", () =
     await expect(readFile(join(ws, "drop.txt"), "utf-8")).rejects.toThrow();
   });
 
+  it("authors a DELETE from {before, after:null} and reconciles it both ways (issue #303)", async () => {
+    // The delete flow end-to-end at this seam: the harness captured the
+    // pre-delete bytes and the file is gone from disk (the tool ran). The
+    // candidate must carry a DELETE with the before blob; an approved DELETE
+    // stays deleted, a rejected DELETE is restored byte-exact from the blob.
+    const status = newStatus();
+    const storage = makeStorage();
+    const readBlob = reconcilerReadBlob(storage);
+    const enc = (s: string) => new TextEncoder().encode(s);
+
+    const baseline = await captureBaselineToLedger({
+      status, gitRoot: ws, executionId: EXEC_ID, changeSetId: CHANGE_SET_ID,
+      harnessId: HARNESS, gitWorkspace: false,
+    });
+    await captureCandidateToLedger({
+      status, gitRoot: ws, executionId: EXEC_ID, changeSetId: CHANGE_SET_ID,
+      baselineTree: baseline, harnessId: HARNESS, gitWorkspace: false, storage,
+      casCaptures: [
+        { path: "gone-approved.txt", before: enc("A"), after: null, captureClass: FileCaptureClass.NON_GIT_CAS },
+        { path: "gone-rejected.txt", before: enc("B"), after: null, captureClass: FileCaptureClass.NON_GIT_CAS },
+      ],
+    });
+
+    const cand = candidateChanges(status);
+    expect(cand.map((c) => c.kind)).toEqual([FileChangeKind.DELETE, FileChangeKind.DELETE]);
+    // A DELETE names its path on the BEFORE side only, with the restorable blob.
+    expect(cand.map((c) => c.pathBefore)).toEqual(["gone-approved.txt", "gone-rejected.txt"]);
+    expect(cand.map((c) => c.pathAfter)).toEqual(["", ""]);
+    expect(cand[0].before?.body.case).toBe("ref");
+    expect(cand[0].after).toBeUndefined();
+
+    const changeSet = decidedChangeSet(status, {
+      "gone-approved.txt": FileDecisionAction.APPROVE,
+      "gone-rejected.txt": FileDecisionAction.REJECT,
+    });
+    const result = await applyCaptureDecisions({
+      status, gitRoot: ws, executionId: EXEC_ID, changeSet, harnessId: HARNESS,
+      gitWorkspace: false, storage, readBlob,
+    });
+
+    expect(result.failed).toBe(false);
+    expect(result.approvedPaths).toEqual(["gone-approved.txt"]);
+    expect(result.rejectedPaths).toEqual(["gone-rejected.txt"]);
+    // Approved DELETE: the file stays gone. Rejected DELETE: restored byte-exact.
+    await expect(readFile(join(ws, "gone-approved.txt"), "utf-8")).rejects.toThrow();
+    expect(await readFile(join(ws, "gone-rejected.txt"), "utf-8")).toBe("B");
+  });
+
   it("is not a capture turn when neither a git ref nor a CAS manifest exists", async () => {
     const status = newStatus();
     const storage = makeStorage();

package/src/shared/mcp-manager.ts

@@ -73,6 +73,14 @@ export function toMcpClientConfig(
         transport: "http",
         url: server.url,
         headers: server.headers,
+        // Re-establish severed connections (oss#316): when a remote MCP
+        // server restarts mid-execution — the hosted bridge's sessions are
+        // in-memory, so a deploy orphans every live one — the adapter's
+        // onclose hook re-runs initialize, minting a fresh session for
+        // subsequent calls instead of stranding the agent on a dead one.
+        // Covers transport-close only; a session lost with no stream open
+        // still surfaces as a JSON-RPC session-not-found on the next call.
+        reconnect: { enabled: true, maxAttempts: 3, delayMs: 1_000 },
       };
     }
   }