@jskit-ai/assistant-core 0.1.196 → 0.1.197

package/examples/conversation/package.json

@@ -8,14 +8,14 @@
     "build": "vite build"
   },
   "dependencies": {
-    "@jskit-ai/assistant-core": "0.1.196",
-    "@jskit-ai/http-runtime": "0.1.216",
-    "@jskit-ai/kernel": "0.1.218",
+    "@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.35",
-    "@jskit-ai/connectors-catalog": "0.1.35"
+    "@jskit-ai/connectors-core": "0.1.36",
+    "@jskit-ai/connectors-catalog": "0.1.36"
   },
   "devDependencies": {
     "@vitejs/plugin-vue": "^6.0.7",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/assistant-core",
-  "version": "0.1.196",
+  "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.160",
-    "@jskit-ai/resource-crud-core": "0.1.160",
+    "@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.216",
-    "@jskit-ai/kernel": "0.1.218",
+    "@jskit-ai/http-runtime": "0.1.217",
+    "@jskit-ai/kernel": "0.1.219",
     "vue": "^3.5.13",
     "vuetify": "^4.0.0"
   },

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/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." }
+  ]);
+});