@jskit-ai/assistant-core 0.1.194 → 0.1.196

package/examples/conversation/package.json

@@ -8,14 +8,14 @@
     "build": "vite build"
   },
   "dependencies": {
-    "@jskit-ai/assistant-core": "0.1.194",
-    "@jskit-ai/http-runtime": "0.1.214",
-    "@jskit-ai/kernel": "0.1.216",
+    "@jskit-ai/assistant-core": "0.1.196",
+    "@jskit-ai/http-runtime": "0.1.216",
+    "@jskit-ai/kernel": "0.1.218",
     "@mdi/js": "^7.4.47",
     "vue": "^3.5.13",
     "vuetify": "^4.0.0",
-    "@jskit-ai/connectors-core": "0.1.33",
-    "@jskit-ai/connectors-catalog": "0.1.33"
+    "@jskit-ai/connectors-core": "0.1.35",
+    "@jskit-ai/connectors-catalog": "0.1.35"
   },
   "devDependencies": {
     "@vitejs/plugin-vue": "^6.0.7",

package/fixtures/responsive-assistant/ProgressFixture.vue

@@ -10,6 +10,9 @@ const combined = ref(false);
 const messages = ref([1, 2, 3].map(index => ({
   messageId: `progress-${index}`, role: "thinking", text: `Reasoning ${index}`
 })));
+if (new URLSearchParams(location.search).has("detail")) {
+  messages.value = [{ messageId: "raw-first", role: "thinking", text: "Full reasoning before the update", preview: false }];
+}
 let sequence = 3;
 function append(role) {
   sequence += 1;
@@ -47,6 +50,8 @@ const adapter = computed(() => ({
           <button @click="combined = !combined">Change storage rows</button>
           <button @click="scope = scope === 'first' ? 'second' : 'first'">Change conversation</button>
           <button v-for="role in ['thinking', 'commentary', 'assistant', 'user', 'system']" :key="role" @click="append(role)">Append {{ role }}</button>
+          <button @click="messages.push({ messageId: `summary-${sequence++}`, role: 'thinking', text: 'Checking the sources.' })">Add preview</button>
+          <button @click="messages.push({ messageId: `detail-${sequence++}`, role: 'thinking', text: 'Full reasoning after the update', preview: false })">Add detail</button>
         </nav>
         <AssistantConversationElement :adapter="adapter" />
       </main>

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/assistant-core",
-  "version": "0.1.194",
+  "version": "0.1.196",
   "type": "module",
   "scripts": {
     "test": "node --test"
@@ -34,8 +34,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.158",
-    "@jskit-ai/resource-crud-core": "0.1.158",
+    "@jskit-ai/resource-core": "0.1.160",
+    "@jskit-ai/resource-crud-core": "0.1.160",
     "@mdi/js": "^7.4.47",
     "@openrouter/ai-sdk-provider": "^3.0.0",
     "ai": "^7.0.101",
@@ -44,8 +44,8 @@
     "ws": "^8.20.1"
   },
   "peerDependencies": {
-    "@jskit-ai/http-runtime": "0.1.214",
-    "@jskit-ai/kernel": "0.1.216",
+    "@jskit-ai/http-runtime": "0.1.216",
+    "@jskit-ai/kernel": "0.1.218",
     "vue": "^3.5.13",
     "vuetify": "^4.0.0"
   },

package/src/client/conversation/AssistantProgress.vue

@@ -1,7 +1,7 @@
 <template>
   <div class="assistant-progress">
     <button
-      v-if="messages.length > previewCount"
+      v-if="expanded || messages.length > previewMessages.length"
       :aria-expanded="expanded"
       class="assistant-progress__toggle"
       type="button"
@@ -29,16 +29,20 @@ const props = defineProps({
 });
 const expanded = ref(false);
 const previewCount = computed(() => props.pending ? props.previewLimit : 0);
+// Applications may retain detailed progress for expansion without previewing it.
+const previewMessages = computed(() => previewCount.value > 0
+  ? props.messages.filter(message => message.preview !== false).slice(-previewCount.value)
+  : []);
 const visibleMessages = computed(() => {
   if (expanded.value) {
     return props.messages;
   }
-  return previewCount.value > 0 ? props.messages.slice(-previewCount.value) : [];
+  return previewMessages.value;
 });
 const toggleLabel = computed(() => {
   if (expanded.value) {
-    return previewCount.value > 0
-      ? `Show latest ${previewCount.value} progress updates`
+    return previewMessages.value.length > 0
+      ? `Show latest ${previewMessages.value.length} progress ${previewMessages.value.length === 1 ? "update" : "updates"}`
       : "Hide progress updates";
   }
   return `Show all ${props.messages.length} progress ${props.messages.length === 1 ? "update" : "updates"}`;

package/test/assistantScroll.browser.test.js

@@ -22,6 +22,40 @@ const VIEWPORTS = Object.freeze([
   Object.freeze({ name: "drawer", width: 360, height: 600 })
 ]);
 
+test("progress previews omit expandable details without hiding the current update", {
+  skip: !RUN_BROWSER_TEST, timeout: 60_000
+}, async () => {
+  const vite = await startViteFixture({ fixtureRoot: FIXTURE_ROOT });
+  const browser = await chromium.launch(createChromiumLaunchOptions());
+  try {
+    for (const width of [390, 800, 1365]) {
+      const page = await browser.newPage({ viewport: { width, height: 900 } });
+      await page.goto(`${vite.baseURL}/?progress=1&detail=1`);
+      const messages = page.locator(".assistant-progress__message");
+      await expect(messages).toHaveCount(0);
+      const toggle = page.locator(".assistant-progress__toggle");
+      await expect(toggle).toHaveText("Show all 1 progress update");
+      await page.getByRole("button", { name: "Add preview", exact: true }).click();
+      await page.getByRole("button", { name: "Add detail", exact: true }).click();
+      await expect(messages).toHaveText(["Checking the sources."]);
+      await toggle.click();
+      await expect(messages).toHaveText(["Full reasoning before the update", "Checking the sources.", "Full reasoning after the update"]);
+      await expect(toggle).toHaveAttribute("aria-expanded", "true");
+      await toggle.click();
+      await expect(messages).toHaveText(["Checking the sources."]);
+      await page.getByRole("button", { name: "Toggle working", exact: true }).click();
+      await expect(messages).toHaveCount(0);
+      await toggle.click();
+      await expect(messages).toHaveCount(3);
+      assert.ok(await page.evaluate(() => document.documentElement.scrollWidth <= innerWidth));
+      await page.close();
+    }
+  } finally {
+    await browser.close();
+    await stopProcess(vite);
+  }
+});
+
 test("saved reply attribution remains when the current assistant changes", {
   skip: !RUN_BROWSER_TEST, timeout: 60_000
 }, async () => {
@@ -211,14 +245,17 @@ test("conversation scrolling preserves history anchors and resets expansion on s
     await page.goto(`${vite.baseURL}/?controls=1`);
     const body = page.locator(".assistant-transcript__body");
     await expect(page.getByText("Conversation line 70:", { exact: false })).toBeVisible();
-    // Keep the explicit button path covered independently of automatic loading.
-    await body.evaluate(element => { element.scrollTop = 0; });
-    await expect.poll(() => body.evaluate((element) => element.scrollTop)).toBe(0);
+    // User input cancels startup following; a scrollTop assignment does not.
+    // Automatic history loading may preserve an anchor above scrollTop zero.
+    await body.hover();
+    await page.mouse.wheel(0, -100_000);
+    await expect(page.getByRole("button", { name: "Read more", exact: true })).toBeVisible();
     await page.getByRole("button", { name: "Read more", exact: true }).click();
     await expect(page.getByRole("button", { name: "Show less", exact: true })).toBeVisible();
     const original = page.getByText("Conversation line 1:", { exact: false }).first();
     const originalTop = (await original.boundingBox()).y;
-    await page.getByRole("button", { name: "Load older messages", exact: true }).click();
+    // Measure history anchoring without Playwright first scrolling to the button.
+    await page.getByRole("button", { name: "Load older messages", exact: true }).evaluate(element => element.click());
     await expect.poll(async () => Math.abs((await original.boundingBox()).y - originalTop)).toBeLessThan(2);
     const before = await body.evaluate((element) => element.scrollTop);
     await page.getByRole("button", { name: "Append reply", exact: true }).click();