@jskit-ai/assistant-core 0.1.195 → 0.1.197

package/examples/conversation/package.json

@@ -8,14 +8,14 @@
     "build": "vite build"
   },
   "dependencies": {
-    "@jskit-ai/assistant-core": "0.1.195",
-    "@jskit-ai/http-runtime": "0.1.215",
-    "@jskit-ai/kernel": "0.1.217",
+    "@jskit-ai/assistant-core": "0.1.197",
+    "@jskit-ai/http-runtime": "0.1.217",
+    "@jskit-ai/kernel": "0.1.219",
     "@mdi/js": "^7.4.47",
     "vue": "^3.5.13",
     "vuetify": "^4.0.0",
-    "@jskit-ai/connectors-core": "0.1.34",
-    "@jskit-ai/connectors-catalog": "0.1.34"
+    "@jskit-ai/connectors-core": "0.1.36",
+    "@jskit-ai/connectors-catalog": "0.1.36"
   },
   "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.195",
+  "version": "0.1.197",
   "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.159",
-    "@jskit-ai/resource-crud-core": "0.1.159",
+    "@jskit-ai/resource-core": "0.1.161",
+    "@jskit-ai/resource-crud-core": "0.1.161",
     "@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.215",
-    "@jskit-ai/kernel": "0.1.217",
+    "@jskit-ai/http-runtime": "0.1.217",
+    "@jskit-ai/kernel": "0.1.219",
     "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/src/server/conversation/codexEvents.js

@@ -566,10 +566,17 @@ function codexAppServerProviderThreadTurn(value = {}, turnId = "") {
 
 function codexAppServerProviderTurnAssistantSegments(turn = {}) {
   const seenItemIds = new Set();
-  return codexAppServerProviderTurnItems(turn)
-    .filter((item) => {
+  const items = codexAppServerProviderTurnItems(turn);
+  // Providers without phases emit progress as ordinary assistant messages.
+  // Only the trailing response can be a final; reasoning/tools separate it
+  // from earlier updates. Explicit finals retain their native attribution.
+  let responseStart = items.length;
+  while (responseStart > 0 && codexAppServerAssistantItemText(items[responseStart - 1])) responseStart -= 1;
+  const active = ["inProgress", "starting"].includes(codexAppServerStatusFromValue(turn.status));
+  return items
+    .filter((item, index) => {
       const phase = normalizeText(item.phase);
-      return !phase || phase === "final_answer";
+      return phase === "final_answer" || (!phase && !active && index >= responseStart);
     })
     .map((item) => {
       const itemId = normalizeText(item.id);

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 () => {

package/test/codexEvents.test.js

@@ -0,0 +1,40 @@
+import assert from "node:assert/strict";
+import test from "node:test";
+import { codexAppServerProviderThreadAssistantSegments } from "../src/server/conversation/codexEvents.js";
+
+test("phase-less Codex progress before tools does not become a final answer", () => {
+  const items = [
+    { type: "reasoning", id: "reasoning-1", content: ["Details"] },
+    { type: "agentMessage", id: "progress-1", text: "Checking sources." },
+    { type: "commandExecution", id: "tool-1" },
+    { type: "agentMessage", id: "progress-2", text: "Comparing prices." },
+    { type: "commandExecution", id: "tool-2" },
+    { type: "reasoning", id: "reasoning-2", content: ["More details"] },
+    { type: "agentMessage", id: "answer-1", text: "The answer." },
+    { type: "agentMessage", id: "answer-2", text: "Supporting sources." }
+  ];
+  const segments = (status, count = items.length) => codexAppServerProviderThreadAssistantSegments({
+    turns: [{ id: "turn", status, items: items.slice(0, count) }]
+  }, "turn");
+  assert.deepEqual(segments("inProgress"), []);
+  assert.deepEqual(segments("failed", 5), [], "a failed tool cannot turn earlier progress into an answer");
+  assert.deepEqual(segments("completed"), [
+    { itemId: "answer-1", text: "The answer." },
+    { itemId: "answer-2", text: "Supporting sources." }
+  ]);
+  assert.deepEqual(segments(undefined), segments("completed"), "saved history without status retains its trailing response");
+});
+
+test("explicit Codex finals remain independent across goal continuation and hooks", () => {
+  assert.deepEqual(codexAppServerProviderThreadAssistantSegments({ turns: [{
+    id: "turn", status: "inProgress", items: [
+      { type: "agentMessage", id: "answer-1", phase: "final_answer", text: "First result." },
+      { type: "hookPrompt", id: "hook" },
+      { type: "agentMessage", id: "progress", phase: "commentary", text: "Continuing." },
+      { type: "agentMessage", id: "answer-2", phase: "final_answer", text: "Second result." }
+    ]
+  }] }, "turn"), [
+    { itemId: "answer-1", text: "First result." },
+    { itemId: "answer-2", text: "Second result." }
+  ]);
+});