@jskit-ai/assistant-core 0.1.215 → 0.1.216

package/examples/conversation/package.json

@@ -8,14 +8,14 @@
     "build": "vite build"
   },
   "dependencies": {
-    "@jskit-ai/assistant-core": "0.1.215",
-    "@jskit-ai/http-runtime": "0.1.235",
-    "@jskit-ai/kernel": "0.1.237",
+    "@jskit-ai/assistant-core": "0.1.216",
+    "@jskit-ai/http-runtime": "0.1.236",
+    "@jskit-ai/kernel": "0.1.238",
     "@mdi/js": "^7.4.47",
     "vue": "^3.5.13",
     "vuetify": "^4.0.0",
-    "@jskit-ai/connectors-core": "0.1.54",
-    "@jskit-ai/connectors-catalog": "0.1.54"
+    "@jskit-ai/connectors-core": "0.1.55",
+    "@jskit-ai/connectors-catalog": "0.1.55"
   },
   "devDependencies": {
     "@vitejs/plugin-vue": "^6.0.7",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/assistant-core",
-  "version": "0.1.215",
+  "version": "0.1.216",
   "type": "module",
   "scripts": {
     "test": "node --test --test-concurrency=1"
@@ -47,8 +47,8 @@
     "@ai-sdk/perplexity": "^4.0.43",
     "@ai-sdk/togetherai": "^3.0.49",
     "@ai-sdk/xai": "^4.0.58",
-    "@jskit-ai/resource-core": "0.1.179",
-    "@jskit-ai/resource-crud-core": "0.1.179",
+    "@jskit-ai/resource-core": "0.1.180",
+    "@jskit-ai/resource-crud-core": "0.1.180",
     "@mdi/js": "^7.4.47",
     "@openrouter/ai-sdk-provider": "^3.0.0",
     "ai": "^7.0.101",
@@ -57,8 +57,8 @@
     "ws": "^8.20.1"
   },
   "peerDependencies": {
-    "@jskit-ai/http-runtime": "0.1.235",
-    "@jskit-ai/kernel": "0.1.237",
+    "@jskit-ai/http-runtime": "0.1.236",
+    "@jskit-ai/kernel": "0.1.238",
     "vue": "^3.5.13",
     "vuetify": "^4.0.0"
   },

package/src/client/conversation/AssistantTranscript.vue

@@ -139,7 +139,8 @@
             class="assistant-transcript__message assistant-transcript__message--user"
             :class="{
               'assistant-transcript__message--pending': entry.turn.optimistic?.status === 'pending',
-              'assistant-transcript__message--failed': entry.turn.optimistic?.status === 'failed'
+              'assistant-transcript__message--failed': entry.turn.optimistic?.status === 'failed',
+              'assistant-transcript__message--unconfirmed': Boolean(entry.delivery)
             }"
           >
             <div
@@ -175,6 +176,12 @@
               <time v-if="entry.message.displayAt">{{ entry.message.displayAt }}</time>
               <span v-if="entry.turn.optimistic?.status === 'pending'" role="status">Pending</span>
             </div>
+            <div v-if="entry.delivery" class="assistant-transcript__unconfirmed-delivery">
+              <AssistantConversationStatus
+                :message="{ ...entry.delivery, text: 'Delivery unconfirmed' }"
+                @check-delivery="emit('check-delivery', $event)"
+              />
+            </div>
             <slot name="user-message-actions" :message="entry.message" :turn="entry.turn" />
             <div
               v-if="entry.turn.optimistic?.status === 'failed'"
@@ -584,8 +591,11 @@ const displayEntries = computed(() => {
   const entries = [];
   for (const [index, turn] of props.turns.entries()) {
     const turnId = String(turn.turnId || index + 1);
+    // A delivery notice belongs to its exact user message, not the previous reply.
+    const userDelivery = turn.user && turn.system?.delivery && turn.system.delivery.messageId === turn.user.messageId
+      ? turn.system : null;
     const messages = [
-      turn.system && { ...turn.system, role: "system" },
+      turn.system && !userDelivery && { ...turn.system, role: "system" },
       turn.user && { ...turn.user, role: "user" },
       ...conversationAgentMessages(turn)
     ].filter(Boolean);
@@ -604,6 +614,7 @@ const displayEntries = computed(() => {
       } else {
         entries.push({
           key, role, turn: { ...turn, turnId }, message: displayed,
+          delivery: role === "user" ? userDelivery : null,
           ...(role === "thinking" ? { messages: [{ ...displayed, key }] } : {})
         });
       }
@@ -1431,6 +1442,26 @@ watch(() => props.visible, (visible) => {
   margin-top: -0.15rem;
 }
 
+.assistant-transcript__message--user.assistant-transcript__message--unconfirmed {
+  background: rgba(var(--v-theme-warning), 0.08);
+  border: 1px solid rgba(var(--v-theme-warning), 0.6);
+}
+
+.assistant-transcript__unconfirmed-delivery {
+  border-top: 1px solid rgba(var(--v-theme-warning), 0.4);
+  font-size: 0.875rem;
+  padding-top: 0.65rem;
+}
+
+.assistant-transcript__unconfirmed-delivery :deep(p:first-child) {
+  font-weight: 700;
+  margin-bottom: 0.4rem;
+}
+
+.assistant-transcript__unconfirmed-delivery :deep(p) {
+  overflow-wrap: anywhere;
+}
+
 .assistant-transcript__optimistic-failure {
   align-items: center;
   color: rgba(var(--v-theme-error), 0.92);

package/src/client/conversation/messageDelivery.js

@@ -111,6 +111,7 @@ function createAssistantMessageDelivery({ deliver: defaultDeliver } = {}) {
     const ownerPendingSends = pendingSends;
     const current = () => pendingSends === ownerPendingSends && isCurrent();
     const predecessor = deliveryTail;
+    const knownUncertain = new Set(state.messages.filter(entry => entry.status === "uncertain").map(entry => entry.id));
     const settled = Promise.withResolvers();
     deliveryTail = settled.promise;
     ownerPendingSends.set(messageId, settled);
@@ -132,7 +133,7 @@ function createAssistantMessageDelivery({ deliver: defaultDeliver } = {}) {
     try {
       if (predecessor) await predecessor;
       if (!current()) return false;
-      if (uncertainOnError && state.messages.some(entry => entry.id !== messageId && entry.status === "uncertain")) {
+      if (uncertainOnError && state.messages.some(entry => entry.id !== messageId && entry.status === "uncertain" && !knownUncertain.has(entry.id))) {
         fail(messageId, "Check the previous message's delivery before sending again.");
         return false;
       }

package/src/client/conversation/useAssistantSuggestions.js

@@ -3,7 +3,7 @@ import { computed, onScopeDispose, ref, toValue, unref, watch } from "vue";
 /** Suggestion requests use an application-supplied agent; credentials stay on its server. */
 export function useAssistantSuggestions({
   active = true, requestKey = "", draft = "", configuration = {},
-  generate, onSelect = () => false, debounceMs = 750
+  generate, onSelect = () => false, debounceMs = 750, minIntervalMs = 0
 } = {}) {
   const items = ref([]);
   const loading = ref(false);
@@ -18,6 +18,7 @@ export function useAssistantSuggestions({
   const visible = computed(() => eligible.value && (loading.value || items.value.length > 0));
   let timer;
   let request;
+  let lastStartedAt = -Infinity;
 
   function cancel() {
     clearTimeout(timer);
@@ -43,6 +44,7 @@ export function useAssistantSuggestions({
     const run = unref(generate);
     loading.value = true;
     timer = setTimeout(async () => {
+      lastStartedAt = Date.now();
       try {
         const result = await run(input);
         if (request !== controller) return;
@@ -58,7 +60,8 @@ export function useAssistantSuggestions({
           loading.value = false;
         }
       }
-    }, Math.max(0, Number(toValue(debounceMs)) || 0));
+    }, Math.max(0, Number(toValue(debounceMs)) || 0,
+      lastStartedAt + Math.max(0, Number(toValue(minIntervalMs)) || 0) - Date.now()));
   }, { immediate: true, flush: "sync" });
 
   function currentSuggestion(value) {
@@ -80,7 +83,6 @@ export function useAssistantSuggestions({
   }
   function focus() {
     if (toValue(active) === false || composerFocused.value) return;
-    dismissedKey.value = null;
     composerFocused.value = true;
   }
   function blur() {

package/test/messageDelivery.test.js

@@ -397,3 +397,18 @@ test("restored application requests retain their origin and attachment-only user
   assert.equal(delivery.turns()[0].user.text, "");
   assert.deepEqual(delivery.turns()[0].user.attachments, [{ attachmentId: "image" }]);
 });
+
+test("a deliberate new message can follow known uncertainty without resending the old request", async () => {
+  const sent = [];
+  const delivery = createAssistantMessageDelivery({ deliver: async payload => {
+    sent.push(payload);
+    if (payload.messageId === "old") throw new Error("Lost receipt");
+    return { ok: true };
+  } });
+  await assert.rejects(delivery.send({ message: "Old request" }, { messageId: "old", uncertainOnError: true }));
+  assert.equal(await delivery.send({ message: "Old request" }, { messageId: "old", uncertainOnError: true }), false);
+  assert.deepEqual(await delivery.send({ message: "A new request" }, { messageId: "new", uncertainOnError: true }), { ok: true });
+  assert.deepEqual(sent.map(message => message.messageId), ["old", "new"]);
+  assert.equal(delivery.find("old").status, "uncertain");
+  assert.equal(delivery.find("new").status, "accepted");
+});

package/test/suggestions.test.js

@@ -87,7 +87,7 @@ test("suggestion previews preserve a draft and stale selections cannot submit",
   state.focus();
   t.mock.timers.tick(750);
   await flush();
-  assert.equal(state.visible.value, true);
+  assert.equal(state.visible.value, false, "Refocusing does not regenerate dismissed suggestions");
   draft.value = "Keep my text";
   assert.equal(state.preview.value, "");
   assert.equal(state.select(suggestion("Next step")), false);
@@ -112,3 +112,36 @@ test("a failing suggestion agent leaves the composer independent and stops retry
   t.mock.timers.tick(60_000);
   assert.equal(requests, 1);
 });
+
+
+test("suggestions wait five seconds and coalesce edited drafts through a thirty-second cooldown", async t => {
+  t.mock.timers.enable({ apis: ["setTimeout", "Date"], now: 1_000 });
+  const scope = effectScope();
+  t.after(() => scope.stop());
+  const draft = ref("First");
+  const active = ref(true);
+  const calls = [];
+  const state = scope.run(() => useAssistantSuggestions({ draft, active, debounceMs: 5_000, minIntervalMs: 30_000,
+    generate: async input => { calls.push({ draft: input.draft, at: Date.now() }); return [suggestion(input.draft)]; }
+  }));
+  t.mock.timers.tick(4_999);
+  state.focus(); state.blur(); state.focus();
+  assert.equal(calls.length, 0);
+  t.mock.timers.tick(1);
+  await flush();
+  draft.value = "Second";
+  t.mock.timers.tick(15_000);
+  draft.value = "Newest";
+  state.focus();
+  t.mock.timers.tick(14_999);
+  assert.equal(calls.length, 1);
+  t.mock.timers.tick(1);
+  await flush();
+  assert.deepEqual(calls, [{ draft: "First", at: 6_000 }, { draft: "Newest", at: 36_000 }]);
+  t.mock.timers.tick(90_000);
+  assert.equal(calls.length, 2, "Cooldown does not poll when the input is unchanged");
+  draft.value = "Cancelled before dispatch";
+  active.value = false;
+  t.mock.timers.tick(30_000);
+  assert.equal(calls.length, 2);
+});