@jskit-ai/assistant-core 0.1.215 → 0.1.217

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.217",
+    "@jskit-ai/http-runtime": "0.1.237",
+    "@jskit-ai/kernel": "0.1.239",
     "@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.56",
+    "@jskit-ai/connectors-catalog": "0.1.56"
   },
   "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.217",
   "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.181",
+    "@jskit-ai/resource-crud-core": "0.1.181",
     "@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.237",
+    "@jskit-ai/kernel": "0.1.239",
     "vue": "^3.5.13",
     "vuetify": "^4.0.0"
   },

package/src/client/conversation/AssistantConversationStatus.vue

@@ -1,14 +1,15 @@
 <template>
-  <div v-if="message.delivery" role="status">
-    <p>{{ message.text }}</p>
-    <p v-if="message.delivery.error" class="text-body-small">{{ message.delivery.error }}</p>
+  <div v-if="message.delivery" class="d-flex flex-wrap align-center ga-1 text-caption text-error" role="status">
+    <span>{{ message.delivery.error || message.text }}</span>
     <v-btn
-      variant="tonal"
-      min-height="48"
+      variant="text"
+      color="error"
+      size="x-small"
+      aria-label="Retry delivery check"
       :disabled="message.delivery.checking"
       @click="$emit('check-delivery', message.delivery.messageId)"
     >
-      {{ message.delivery.checking ? 'Checking delivery…' : 'Check delivery' }}
+      {{ message.delivery.checking ? 'Checking…' : 'Retry' }}
     </v-btn>
   </div>
   <slot v-else>

package/src/client/conversation/AssistantTranscript.vue

@@ -177,22 +177,26 @@
             </div>
             <slot name="user-message-actions" :message="entry.message" :turn="entry.turn" />
             <div
-              v-if="entry.turn.optimistic?.status === 'failed'"
+              v-if="entry.delivery || entry.turn.optimistic?.status === 'failed'"
               class="assistant-transcript__optimistic-failure"
             >
-              <span role="status">Failed: {{ entry.turn.optimistic.error || "Message could not be sent." }}</span>
+              <span role="status">{{ entry.delivery ? entry.delivery.delivery.error || 'Delivery unconfirmed.' : `Failed: ${entry.turn.optimistic.error || 'Message could not be sent.'}` }}</span>
               <div v-if="!entry.turn.preview" class="assistant-transcript__optimistic-actions">
                 <v-btn
                   class="assistant-transcript__delivery-action"
-                  color="primary"
+                  color="error"
                   size="x-small"
                   type="button"
-                  variant="tonal"
-                  @click="emit('resend-turn', entry.turn.optimistic.id)"
+                  variant="text"
+                  :disabled="entry.delivery?.delivery.checking"
+                  :aria-label="entry.delivery ? 'Retry delivery check' : undefined"
+                  :title="entry.delivery ? entry.delivery.delivery.error || 'Check delivery before resending.' : undefined"
+                  @click="entry.delivery ? emit('check-delivery', entry.delivery.delivery.messageId) : emit('resend-turn', entry.turn.optimistic.id)"
                 >
-                  Retry
+                  {{ entry.delivery?.delivery.checking ? 'Checking…' : 'Retry' }}
                 </v-btn>
                 <v-btn
+                  v-if="!entry.delivery"
                   class="assistant-transcript__delivery-action"
                   size="x-small"
                   type="button"
@@ -202,6 +206,7 @@
                   Cancel
                 </v-btn>
                 <v-btn
+                  v-if="!entry.delivery"
                   class="assistant-transcript__delivery-action"
                   size="x-small"
                   type="button"
@@ -584,8 +589,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 +612,7 @@ const displayEntries = computed(() => {
       } else {
         entries.push({
           key, role, turn: { ...turn, turnId }, message: displayed,
+          delivery: role === "user" ? userDelivery : null,
           ...(role === "thinking" ? { messages: [{ ...displayed, key }] } : {})
         });
       }

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);
+});