@jskit-ai/assistant-core 0.1.34 → 0.1.36

package/package.descriptor.mjs

@@ -1,7 +1,7 @@
 export default Object.freeze({
   packageVersion: 1,
   packageId: "@jskit-ai/assistant-core",
-  version: "0.1.34",
+  version: "0.1.36",
   kind: "runtime",
   description: "Reusable assistant client/server/shared primitives without surface-specific routes or settings ownership.",
   dependsOn: [
@@ -47,11 +47,11 @@ export default Object.freeze({
   mutations: {
     dependencies: {
       runtime: {
-        "@jskit-ai/http-runtime": "0.1.57",
-        "@jskit-ai/kernel": "0.1.58",
-        "@jskit-ai/resource-core": "0.1.3",
-        "@jskit-ai/resource-crud-core": "0.1.3",
-        "@jskit-ai/users-core": "0.1.68",
+        "@jskit-ai/http-runtime": "0.1.59",
+        "@jskit-ai/kernel": "0.1.60",
+        "@jskit-ai/resource-core": "0.1.5",
+        "@jskit-ai/resource-crud-core": "0.1.5",
+        "@jskit-ai/users-core": "0.1.70",
         "@tanstack/vue-query": "^5.90.5",
         "dompurify": "^3.3.3",
         "json-rest-schema": "1.x.x",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/assistant-core",
-  "version": "0.1.34",
+  "version": "0.1.36",
   "type": "module",
   "scripts": {
     "test": "node --test"
@@ -11,12 +11,12 @@
     "./shared": "./src/shared/index.js"
   },
   "dependencies": {
-    "@jskit-ai/database-runtime": "0.1.58",
-    "@jskit-ai/http-runtime": "0.1.57",
-    "@jskit-ai/kernel": "0.1.58",
-    "@jskit-ai/resource-crud-core": "0.1.3",
-    "@jskit-ai/resource-core": "0.1.3",
-    "@jskit-ai/users-core": "0.1.68",
+    "@jskit-ai/database-runtime": "0.1.60",
+    "@jskit-ai/http-runtime": "0.1.59",
+    "@jskit-ai/kernel": "0.1.60",
+    "@jskit-ai/resource-crud-core": "0.1.5",
+    "@jskit-ai/resource-core": "0.1.5",
+    "@jskit-ai/users-core": "0.1.70",
     "@tanstack/vue-query": "^5.90.5",
     "dompurify": "^3.3.3",
     "json-rest-schema": "1.x.x",

package/src/client/lib/assistantApi.js

@@ -1,5 +1,9 @@
 import { appendQueryString } from "@jskit-ai/kernel/shared/support";
 import {
+  ASSISTANT_CONVERSATIONS_TRANSPORT,
+  ASSISTANT_CONVERSATION_MESSAGES_TRANSPORT,
+  ASSISTANT_SETTINGS_TRANSPORT,
+  ASSISTANT_SETTINGS_UPDATE_TRANSPORT,
   ASSISTANT_STREAM_EVENT_TYPES,
   normalizeAssistantStreamEventType
 } from "../../shared/index.js";
@@ -114,7 +118,10 @@ function createAssistantApi({ request, requestStream, resolveBasePath, resolveSu
 
       return request(
         appendQueryString(`${basePath}/conversations`, params.toString()),
-        requestHeaders ? { headers: requestHeaders } : {}
+        {
+          ...(requestHeaders ? { headers: requestHeaders } : {}),
+          transport: ASSISTANT_CONVERSATIONS_TRANSPORT
+        }
       );
     },
 
@@ -128,7 +135,10 @@ function createAssistantApi({ request, requestStream, resolveBasePath, resolveSu
 
       return request(
         appendQueryString(`${basePath}/conversations/${encodedConversationId}/messages`, params.toString()),
-        requestHeaders ? { headers: requestHeaders } : {}
+        {
+          ...(requestHeaders ? { headers: requestHeaders } : {}),
+          transport: ASSISTANT_CONVERSATION_MESSAGES_TRANSPORT
+        }
       );
     },
 
@@ -138,7 +148,10 @@ function createAssistantApi({ request, requestStream, resolveBasePath, resolveSu
 
       return request(
         `${basePath}/settings`,
-        requestHeaders ? { headers: requestHeaders } : {}
+        {
+          ...(requestHeaders ? { headers: requestHeaders } : {}),
+          transport: ASSISTANT_SETTINGS_TRANSPORT
+        }
       );
     },
 
@@ -151,7 +164,8 @@ function createAssistantApi({ request, requestStream, resolveBasePath, resolveSu
         {
           method: "PATCH",
           ...(requestHeaders ? { headers: requestHeaders } : {}),
-          body: payload
+          body: payload,
+          transport: ASSISTANT_SETTINGS_UPDATE_TRANSPORT
         }
       );
     }

package/src/shared/assistantResource.js

@@ -279,6 +279,8 @@ const conversationMessagesListOutputSchema = createSchema({
   }
 });
 
+const assistantConversationOutputValidator = createSchemaDefinition(conversationRecordSchema, "replace");
+
 const assistantResource = defineResource({
   namespace: "assistant",
   operations: {
@@ -303,5 +305,6 @@ const assistantResource = defineResource({
 export {
   MAX_INPUT_CHARS,
   MAX_HISTORY_MESSAGES,
-  assistantResource
+  assistantResource,
+  assistantConversationOutputValidator
 };

package/src/shared/index.js

@@ -28,7 +28,8 @@ export {
 export {
   MAX_INPUT_CHARS,
   MAX_HISTORY_MESSAGES,
-  assistantResource
+  assistantResource,
+  assistantConversationOutputValidator
 } from "./assistantResource.js";
 
 export {
@@ -36,6 +37,13 @@ export {
   assistantConfigResource
 } from "./assistantSettingsResource.js";
 
+export {
+  ASSISTANT_SETTINGS_TRANSPORT,
+  ASSISTANT_SETTINGS_UPDATE_TRANSPORT,
+  ASSISTANT_CONVERSATIONS_TRANSPORT,
+  ASSISTANT_CONVERSATION_MESSAGES_TRANSPORT
+} from "./jsonApiTransports.js";
+
 export { assistantSettingsEvents } from "./settingsEvents.js";
 
 export {

package/src/shared/jsonApiTransports.js

@@ -0,0 +1,31 @@
+import { deepFreeze } from "@jskit-ai/kernel/shared/support/deepFreeze";
+
+const ASSISTANT_SETTINGS_TRANSPORT = deepFreeze({
+  kind: "jsonapi-resource",
+  responseType: "assistant-settings",
+  responseKind: "record"
+});
+
+const ASSISTANT_SETTINGS_UPDATE_TRANSPORT = deepFreeze({
+  ...ASSISTANT_SETTINGS_TRANSPORT,
+  requestType: "assistant-settings"
+});
+
+const ASSISTANT_CONVERSATIONS_TRANSPORT = deepFreeze({
+  kind: "jsonapi-resource",
+  responseType: "assistant-conversations",
+  responseKind: "collection"
+});
+
+const ASSISTANT_CONVERSATION_MESSAGES_TRANSPORT = deepFreeze({
+  kind: "jsonapi-resource",
+  responseType: "assistant-conversation-messages",
+  responseKind: "record"
+});
+
+export {
+  ASSISTANT_SETTINGS_TRANSPORT,
+  ASSISTANT_SETTINGS_UPDATE_TRANSPORT,
+  ASSISTANT_CONVERSATIONS_TRANSPORT,
+  ASSISTANT_CONVERSATION_MESSAGES_TRANSPORT
+};

package/test/assistantApiSurfaceHeader.test.js

@@ -1,19 +1,31 @@
 import test from "node:test";
 import assert from "node:assert/strict";
 import { createAssistantApi } from "../src/client/lib/assistantApi.js";
+import {
+  ASSISTANT_CONVERSATIONS_TRANSPORT,
+  ASSISTANT_CONVERSATION_MESSAGES_TRANSPORT,
+  ASSISTANT_SETTINGS_TRANSPORT,
+  ASSISTANT_SETTINGS_UPDATE_TRANSPORT
+} from "../src/shared/index.js";
 
 test("assistant api forwards normalized surface header on requests", async () => {
   const observed = {
     stream: null,
     list: null,
-    messages: null
+    messages: null,
+    settingsRead: null,
+    settingsUpdate: null
   };
 
   const api = createAssistantApi({
     resolveBasePath: () => "/api/assistant",
     resolveSurfaceId: () => "AdMiN",
     async request(url, options = {}) {
-      if (url.includes("/messages")) {
+      if (url.endsWith("/settings") && options?.method === "PATCH") {
+        observed.settingsUpdate = { url, options };
+      } else if (url.endsWith("/settings")) {
+        observed.settingsRead = { url, options };
+      } else if (url.includes("/messages")) {
         observed.messages = { url, options };
       } else {
         observed.list = { url, options };
@@ -37,10 +49,20 @@ test("assistant api forwards normalized surface header on requests", async () =>
     page: 1,
     pageSize: 5
   });
+  await api.getSettings();
+  await api.updateSettings({
+    systemPrompt: "Be concise."
+  });
 
   assert.equal(observed.stream?.options?.headers?.["x-jskit-surface"], "admin");
   assert.equal(observed.list?.options?.headers?.["x-jskit-surface"], "admin");
   assert.equal(observed.messages?.options?.headers?.["x-jskit-surface"], "admin");
+  assert.equal(observed.settingsRead?.options?.headers?.["x-jskit-surface"], "admin");
+  assert.equal(observed.settingsUpdate?.options?.headers?.["x-jskit-surface"], "admin");
+  assert.deepEqual(observed.list?.options?.transport, ASSISTANT_CONVERSATIONS_TRANSPORT);
+  assert.deepEqual(observed.messages?.options?.transport, ASSISTANT_CONVERSATION_MESSAGES_TRANSPORT);
+  assert.deepEqual(observed.settingsRead?.options?.transport, ASSISTANT_SETTINGS_TRANSPORT);
+  assert.deepEqual(observed.settingsUpdate?.options?.transport, ASSISTANT_SETTINGS_UPDATE_TRANSPORT);
 });
 
 test("assistant api omits surface header when surface id is empty", async () => {