@jskit-ai/assistant-core 0.1.235 → 0.1.236

package/examples/conversation/package.json

@@ -8,7 +8,7 @@
     "build": "vite build"
   },
   "dependencies": {
-    "@jskit-ai/assistant-core": "0.1.235",
+    "@jskit-ai/assistant-core": "0.1.236",
     "@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.235",
+  "version": "0.1.236",
   "type": "module",
   "scripts": {
     "test": "node --test --test-concurrency=1"

package/src/server/conversation/runtime.js

@@ -442,13 +442,19 @@ export function createConversationRuntime({ engine: defaultEngine = "api", defau
         : Math.max(0, initialWindow - (request?.input.origin === "user" ? 1 : 0)),
       maximumInitialMessageCharacters: limits.maxInitialNativeHistoryMessageCharacters,
       ...(budget ? { maximumHistoryBytes: async options => {
-        await access(context, entry.id, request.operation, entry);
-        signal.throwIfAborted();
-        // A host may free native context first; show that as compacting.
-        const bytes = await budget({ ...options, context, signal,
-          setPhase: phase => setPhase(entry, phase === "compacting" ? "compacting" : "preparing") });
-        signal.throwIfAborted();
-        return bytes;
+        try {
+          await access(context, entry.id, request.operation, entry);
+          signal.throwIfAborted();
+          // A host may free native context first; show that as compacting.
+          const bytes = await budget({ ...options, context, signal,
+            setPhase: phase => setPhase(entry, phase === "compacting" ? "compacting" : "preparing") });
+          signal.throwIfAborted();
+          return bytes;
+        } catch (error) {
+          // Sizing missed history precedes native dispatch, so this message
+          // was definitely not sent. Hosts and clients report it as such.
+          throw Object.assign(error, { delivery: "not-sent" });
+        }
       } } : {}),
       presentation: entry.conversation?.presentation || { unconfirmedCode: "conversation_delivery_uncertain",
         unconfirmedMessage: "The previous submission has uncertain delivery. Retry this message to check its native receipt; it will not be resent automatically." },

package/test/claudeTurn.test.js

@@ -2361,7 +2361,8 @@ test("bound Main cannot dispatch or retain a pending prompt when its history bud
   f.binding.transcript.history = async () => [{ id: "earlier", version: "v1", role: "user", text: "Earlier work." }];
   await f.binding.state.write({ lastEngine: "codex", engines: { codex: { seen: {} } } });
   f.binding.maximumHistoryBytes = async () => { throw new Error("Current native context usage unavailable"); };
-  await assert.rejects(f.conversation.send({ messageId: "no-budget", text: "Continue" }), /Current native context usage unavailable/u);
+  await assert.rejects(f.conversation.send({ messageId: "no-budget", text: "Continue" }), error =>
+    /Current native context usage unavailable/u.test(error.message) && error.delivery === "not-sent");
   assert.equal(f.processes.length, 0);
   assert.equal(f.messages.length, 0);
   assert.equal((await f.binding.state.read())?.engines?.claude?.pending, undefined);