@jskit-ai/assistant-core 0.1.229 → 0.1.230

package/examples/conversation/package.json

@@ -8,7 +8,7 @@
     "build": "vite build"
   },
   "dependencies": {
-    "@jskit-ai/assistant-core": "0.1.229",
+    "@jskit-ai/assistant-core": "0.1.230",
     "@jskit-ai/http-runtime": "0.1.242",
     "@jskit-ai/kernel": "0.1.244",
     "@mdi/js": "^7.4.47",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/assistant-core",
-  "version": "0.1.229",
+  "version": "0.1.230",
   "type": "module",
   "scripts": {
     "test": "node --test --test-concurrency=1"

package/src/server/conversation/providers/claudeDriver.js

@@ -2,6 +2,7 @@ import { createHash, randomUUID } from "node:crypto";
 import { homedir } from "node:os";
 import path from "node:path";
 import { realpath } from "node:fs/promises";
+import { isDeepStrictEqual } from "node:util";
 import { bindClaudeConversationAccount, claudeConnectionIdentity, claudeFlagSettings, claudeModelConfiguration, readClaudeCodeAuthStatus } from "../claudeProcess.js";
 import { nativeAiModel, nativeAiProvider } from "../../../shared/nativeProviders.js";
 import { requireClaudeSessionId, listClaudeConversationStorage } from "../claudeHistory.js";
@@ -107,7 +108,13 @@ export function createClaudeConversationDriver({ connections, host = {}, limits
     const store = {
       read(context) { return JSON.stringify({ executionId: context.binding.executionId, sent: context.binding.sent, pausedGoal: context.binding.pausedGoal, blockedGoal: context.binding.blockedGoal }); },
       async save(entry, { execution = false } = {}) {
-        await entry.context.updateBinding({ pausedGoal: entry.pausedGoal, blockedGoal: entry.blockedGoal, ...(execution ? { executionId: entry.executionId } : {}) });
+        // Only execution and retained-goal changes belong to the binding. An
+        // unchanged save (such as opening a replacement successor before its
+        // owner commits) must not write through the provider's binding fence.
+        const { binding } = entry.context;
+        const goals = { pausedGoal: entry.pausedGoal ?? null, blockedGoal: entry.blockedGoal ?? null };
+        if (!execution && isDeepStrictEqual(goals, { pausedGoal: binding.pausedGoal ?? null, blockedGoal: binding.blockedGoal ?? null })) return;
+        await entry.context.updateBinding({ ...goals, ...(execution ? { executionId: entry.executionId } : {}) });
       },
       async releaseExecution(entry) {
         await entry.context.updateBinding({ executionId: "" });

package/test/claudeConversationSession.test.js

@@ -1550,6 +1550,18 @@ for (const saveFails of [false, true]) test(`Claude retains failed-start custody
   }
 });
 
+test("opening an inert Claude successor binding does not write through the provider fence", async t => {
+  const f = await fixture(t);
+  await f.first.close();
+  const driver = createClaudeConversationDriver(f.driverOptions);
+  // A replacement opens its successor before the runtime commits the new owner.
+  const writeBinding = () => { throw new Error("A retired provider cannot change a native binding."); };
+  for (const binding of [await driver.createBinding(), { ...await driver.createBinding(), pausedGoal: null, blockedGoal: null }]) {
+    const provider = await driver.open({ binding, writeBinding, onFailure() {} });
+    await provider.dispose();
+  }
+});
+
 test("original scoped Claude history resumes from the authorized credential home while retaining its private scope", async t => {
   const f = await fixture(t);
   await f.first.close();