@frockbot/plugin-computer 0.3.26 → 0.3.28

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@frockbot/plugin-computer",
-  "version": "0.3.26",
+  "version": "0.3.28",
   "private": false,
   "type": "module",
   "exports": {
@@ -29,21 +29,21 @@
   },
   "dependencies": {
     "@cordisjs/client": "0.8.2",
-    "@frockbot/client-core": "0.3.26",
-    "@frockbot/client-ui": "0.3.26",
-    "@frockbot/computer-core": "0.3.26",
-    "@frockbot/computer-host-runtime": "0.3.26",
-    "@frockbot/kernel-agent-loop": "0.3.26",
-    "@frockbot/kernel-contracts": "0.3.26",
-    "@frockbot/plugin-prompt": "0.3.26",
-    "@frockbot/plugin-shell": "0.3.26",
-    "@frockbot/plugin-tools": "0.3.26",
+    "@frockbot/client-core": "0.3.28",
+    "@frockbot/client-ui": "0.3.28",
+    "@frockbot/computer-core": "0.3.28",
+    "@frockbot/computer-host-runtime": "0.3.28",
+    "@frockbot/kernel-agent-loop": "0.3.28",
+    "@frockbot/kernel-contracts": "0.3.28",
+    "@frockbot/plugin-prompt": "0.3.28",
+    "@frockbot/plugin-shell": "0.3.28",
+    "@frockbot/plugin-tools": "0.3.28",
     "cordis": "4.0.0-rc.8",
     "vue": "3.5.41"
   },
   "devDependencies": {
-    "@frockbot/plugin-models": "0.3.26",
-    "@frockbot/plugin-testkit": "0.3.26",
+    "@frockbot/plugin-models": "0.3.28",
+    "@frockbot/plugin-testkit": "0.3.28",
     "@types/bun": "1.4.0",
     "@types/node": "26.2.0",
     "@vitejs/plugin-vue": "6.0.8",

package/src/agent.test.ts

@@ -492,3 +492,48 @@ describe("computer agent contribution", () => {
     });
   });
 });
+
+describe("the computer/sync event", () => {
+  test("records a publish sync without the per-path answers the caller asked for", async () => {
+    const { recordComputerSyncV1 } = await import("./agent.js");
+    const { decodeSessionEvent } = await import("@frockbot/kernel-contracts");
+    const appended: unknown[] = [];
+    const sessions = {
+      get: () => ({
+        disposed: false,
+        append: (event: unknown) => appended.push(event),
+        flush: () => Promise.resolve(),
+      }),
+    } as unknown as SessionStore;
+    await recordComputerSyncV1(sessions, "session-1", 3, "publish", {
+      status: "ok",
+      detail: "",
+      pulled: 0,
+      pushed: 1,
+      restored: 0,
+      removed: 0,
+      adopted: 0,
+      ignored: 5,
+      omitted: 0,
+      conflicts: 0,
+      failures: 0,
+      required: [
+        {
+          path: "dist/manifest.json",
+          contentHash: "a".repeat(64),
+          durable: true,
+        },
+      ],
+    } as never);
+    expect(appended).toHaveLength(1);
+    expect(appended[0]).not.toHaveProperty("required");
+    // The durable log's decoder has an exact field set; the event must pass it.
+    expect(() =>
+      decodeSessionEvent({
+        ...(appended[0] as object),
+        seq: 1,
+        timestamp: "2026-09-05T00:00:00.000Z",
+      }),
+    ).not.toThrow();
+  });
+});

package/src/agent.ts

@@ -488,7 +488,7 @@ function text(bytes: Uint8Array): string {
  * cannot record the same fact in two shapes. A Session that is gone or
  * disposed records nothing: a sync is never a reason to fail anything.
  */
-async function recordComputerSyncV1(
+export async function recordComputerSyncV1(
   sessions: SessionStore,
   sessionId: string,
   turn: number,
@@ -497,11 +497,16 @@ async function recordComputerSyncV1(
 ): Promise<void> {
   const session = sessions.get(sessionId);
   if (!session || session.disposed) return;
+  // The per-path answers a publish asked for travel back to that caller, not
+  // into the durable log: the `computer/sync` event has an exact field set,
+  // and a summary spread with `required` on it made every publish Turn fail
+  // with "session event has invalid fields" (v0.3.27, 2026-09-05).
+  const { required: _required, ...recorded } = summary;
   session.append({
     type: "computer/sync",
     turn: Math.max(1, turn),
     reason,
-    ...summary,
+    ...recorded,
   });
   // The record is durable before anything reports the sync happened.
   await session.flush();