@jskit-ai/assistant-core 0.2.39 → 0.2.41

package/CHANGELOG.md

@@ -2,6 +2,7 @@
 
 ## Unreleased
 
+- `assistantConversationListItemOutputValidator` (from `@jskit-ai/assistant-core/shared`) is one conversation of `assistantContract`'s `conversationsList`, the record without its `metadata`; the contract's list output is built from it, and `@jskit-ai/assistant-runtime`'s conversations list route serializes its items with it.
 - `createConversationStorage`'s `write` saves nothing when its callback called no write method: the adapter's `writeRecord` does not run, so file storage no longer rewrites and flushes the whole conversation file, or creates an empty one for a missing conversation, when a write only looked (a delivery check, a message already saved), and the saved record stays cached.
 - Checking or recovering one message's delivery (`inspectDelivery`, and a send that finds its message already saved) finds that message's turn by its id instead of reading and cloning every turn of the conversation to find it.
 - With completed envelopes, a streamed turn's saved completed-envelope marker is read once per turn instead of once per delta: the marker is written with the turn's authored message and never changes, and a reply's hundreds of deltas each opened a storage read. A turn not saved yet is read again until it is.

package/examples/conversation/package.json

@@ -8,14 +8,14 @@
     "build": "vite build"
   },
   "dependencies": {
-    "@jskit-ai/assistant-core": "0.2.39",
-    "@jskit-ai/http-runtime": "0.2.18",
-    "@jskit-ai/kernel": "0.2.18",
+    "@jskit-ai/assistant-core": "0.2.41",
+    "@jskit-ai/http-runtime": "0.2.19",
+    "@jskit-ai/kernel": "0.2.19",
     "@jskit-ai/material-web": "0.1.5",
     "vue": "^3.5.13",
     "vuetify": "^4.0.0",
-    "@jskit-ai/connectors-core": "0.2.25",
-    "@jskit-ai/connectors-catalog": "0.2.25"
+    "@jskit-ai/connectors-core": "0.2.27",
+    "@jskit-ai/connectors-catalog": "0.2.27"
   },
   "devDependencies": {
     "@vitejs/plugin-vue": "^6.0.7",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/assistant-core",
-  "version": "0.2.39",
+  "version": "0.2.41",
   "type": "module",
   "scripts": {
     "test": "jskit test"
@@ -52,7 +52,7 @@
     "@ai-sdk/togetherai": "^3.0.49",
     "@ai-sdk/xai": "^4.0.58",
     "@jskit-ai/agent": "0.1.3",
-    "@jskit-ai/resource-core": "0.1.209",
+    "@jskit-ai/resource-core": "0.1.210",
     "@openrouter/ai-sdk-provider": "^3.0.0",
     "ai": "^7.0.152",
     "json-rest-schema": "^1.0.20",
@@ -60,8 +60,8 @@
     "ws": "^8.20.1"
   },
   "peerDependencies": {
-    "@jskit-ai/http-runtime": "0.2.18",
-    "@jskit-ai/kernel": "0.2.18",
+    "@jskit-ai/http-runtime": "0.2.19",
+    "@jskit-ai/kernel": "0.2.19",
     "vue": "^3.5.13",
     "vuetify": "^4.3.0"
   },

package/src/shared/assistantResource.js

@@ -272,6 +272,8 @@ const conversationMessagesListOutputSchema = createSchema({
 });
 
 const assistantConversationOutputValidator = createSchemaDefinition(conversationRecordSchema, "replace");
+// One conversation of a list, as the conversations list route answers each item: the record without its metadata.
+const assistantConversationListItemOutputValidator = createSchemaDefinition(conversationListItemSchema, "replace");
 
 const assistantContract = defineApiContract({
   namespace: "assistant",
@@ -283,7 +285,7 @@ const assistantContract = defineApiContract({
     conversationsList: {
       method: "GET",
       query: conversationsListQuerySchema,
-      output: createCursorListValidator(createSchemaDefinition(conversationListItemSchema, "replace"))
+      output: createCursorListValidator(assistantConversationListItemOutputValidator)
     },
     conversationMessagesList: {
       method: "GET",
@@ -299,5 +301,6 @@ export {
   MAX_MESSAGE_PAGE_SIZE,
   attachmentIdsField,
   assistantContract,
+  assistantConversationListItemOutputValidator,
   assistantConversationOutputValidator
 };

package/src/shared/index.js

@@ -30,6 +30,7 @@ export {
   MAX_MESSAGE_PAGE_SIZE,
   attachmentIdsField,
   assistantContract,
+  assistantConversationListItemOutputValidator,
   assistantConversationOutputValidator
 } from "./assistantResource.js";
 

package/test/serviceToolCatalog.test.js

@@ -3,6 +3,7 @@ import test from "node:test";
 import { createSchema } from "json-rest-schema";
 
 import { createActionCatalogue } from "@jskit-ai/kernel/server/actions";
+import { createValidationError } from "@jskit-ai/kernel/server/runtime/errors";
 import { createServiceToolCatalog } from "../src/server/lib/serviceToolCatalog.js";
 
 function schema(fields = {}, mode = "patch") {
@@ -29,6 +30,7 @@ function action({
   input = schema({}),
   output = schema({ ok: { type: "boolean", required: true } }),
   extensions = {},
+  checkInput = null,
   execute = async () => ({ ok: true })
 } = {}) {
   return {
@@ -41,6 +43,7 @@ function action({
     input,
     output,
     extensions,
+    checkInput,
     execute
   };
 }
@@ -270,6 +273,48 @@ test("assistant tools expose safe field-level input guidance to the model", asyn
   });
 });
 
+test("assistant tools give the model the field and the type to use when an action's own input check refuses a key", async () => {
+  let executed = false;
+  const actions = createActions([action({
+    input: schema({ fields: { type: "object", required: false, additionalProperties: true } }),
+    // A check of what the schema cannot judge, such as a fieldset key only the application's tables can name.
+    checkInput(input) {
+      if (input.fields?.pet) {
+        throw createValidationError({
+          "fields.pet": {
+            code: "demo.fieldsKeys",
+            params: { type: "pets" },
+            message: "fields keys must be resource types. Allowed keys: \"books\", \"pets\"; use \"pets\" instead of \"pet\"."
+          }
+        });
+      }
+    },
+    execute: async () => {
+      executed = true;
+      return { ok: true };
+    }
+  })]);
+  const catalog = createServiceToolCatalog(actions);
+  const context = { actor: { id: "7" }, surface: "admin" };
+  const toolSet = catalog.resolveToolSet(context);
+
+  assert.deepEqual(await catalog.executeToolCall({
+    toolName: toolSet.tools[0].name,
+    argumentsText: JSON.stringify({ fields: { pet: ["name"] } }),
+    context,
+    toolSet
+  }), {
+    ok: false,
+    error: {
+      code: "ACTION_VALIDATION_FAILED",
+      message: "Validation failed. fields.pet: fields keys must be resource types. Allowed keys: \"books\", \"pets\"; " +
+        "use \"pets\" instead of \"pet\".",
+      status: 400
+    }
+  });
+  assert.equal(executed, false);
+});
+
 test("large authorized catalogs use compact paged discovery, exact contracts, and gated execution", async () => {
   const executions = [];
   let clockExecution = null;