@jskit-ai/assistant-runtime 0.1.152 → 0.1.153

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/assistant-runtime",
-  "version": "0.1.152",
+  "version": "0.1.153",
   "type": "module",
   "scripts": {
     "test": "node --test"
@@ -11,15 +11,15 @@
     "./server/actionIds": "./src/server/actionIds.js"
   },
   "dependencies": {
-    "@jskit-ai/assistant-core": "0.1.157",
-    "@jskit-ai/database-runtime": "0.1.181",
+    "@jskit-ai/assistant-core": "0.1.158",
+    "@jskit-ai/database-runtime": "0.1.182",
     "json-rest-schema": "^1.0.17"
   },
   "peerDependencies": {
-    "@jskit-ai/http-runtime": "0.1.179",
-    "@jskit-ai/http-web": "0.1.26",
-    "@jskit-ai/kernel": "0.1.181",
-    "@jskit-ai/shell-web": "0.1.185",
+    "@jskit-ai/http-runtime": "0.1.180",
+    "@jskit-ai/http-web": "0.1.27",
+    "@jskit-ai/kernel": "0.1.182",
+    "@jskit-ai/shell-web": "0.1.186",
     "@tanstack/vue-query": "^5.90.5",
     "vue": "^3.5.13",
     "vuetify": "^4.0.0"

package/src/server/services/chatService.js

@@ -193,6 +193,20 @@ function buildSystemPrompt({ targetSurfaceId = "", toolDescriptors = [], workspa
     toolSummary,
     toolContracts
   ];
+  if (toolDescriptors.length > 0) {
+    promptSegments.push(
+      "For counts or summaries, first look for a suitable count, aggregate, or report action before reading individual records. Use collection reads when no such action can answer the request. For requests to read several records, compare records, or examine all records, look for a collection action such as list, search, or query.",
+      "Match the user's current scope. When the user broadens a request, find the underlying collection and remove earlier date or other filters that no longer apply; a previous result subset is not the full collection.",
+      "Inspect the action's input and output contracts, use supported filters and field selection, and follow returned pagination until the requested number or scope is covered. For all records or a comparison across the whole collection, cover every relevant page unless a suitable aggregate action answers the request. Never present a partial page or sample as complete; state any incomplete coverage."
+    );
+  }
+  if (toolDescriptors.some((tool) => tool.name === "assistant_action_search")) {
+    promptSegments.push(
+      "The directly exposed tools are entry points into a larger authorized action catalog. Additional application capabilities are available through assistant_action_search, assistant_action_contract, and assistant_action_execute.",
+      "Before claiming an operation or dataset is unavailable or asking the user for access, discover relevant actions with assistant_action_search. Search for the resource and the requested operation: list/search/query for reading records, count/aggregate/report for totals or summaries. Search matches literal words, not meanings: use short resource or operation terms, try broader terms such as list or query when wording differs, or omit query to browse. Continue through catalog nextCursor pages when needed to find a relevant action; one empty search or unrelated page does not establish that a capability is absent.",
+      "Choose an action by its returned description and exact contract, then load assistant_action_contract before assistant_action_execute. Use discovered action IDs and contract fields; do not invent them. Catalog pagination discovers actions; a collection action's pagination retrieves records."
+    );
+  }
   if (normalizedCustomSystemPrompt) {
     promptSegments.push(`Additional instructions for this surface: ${normalizedCustomSystemPrompt}`);
   }

package/test/chatServiceLifecycle.test.js

@@ -207,6 +207,54 @@ function assistantMessages(events) {
     .map((event) => event.payload.text);
 }
 
+test("the provider receives collection guidance in direct and discovery modes", async (t) => {
+  for (const discovery of [false, true]) {
+    await t.test(discovery ? "discovery" : "direct", async () => {
+      const toolNames = discovery
+        ? ["assistant_action_search", "assistant_action_contract", "assistant_action_execute"]
+        : ["inventory_assets_list"];
+      const harness = createHarness([textCompletion("The requested comparison is complete.")], {
+        tools: toolNames.map((name) => ({
+          name,
+          parameters: { type: "object" },
+          outputSchema: { type: "object" }
+        }))
+      });
+
+      await harness.run("Compare all records, including those outside last week's results.");
+
+      const prompt = harness.completionRequests[0].messages[0].content;
+      assert.match(prompt, /collection action such as list, search, or query/u);
+      assert.match(prompt, /first look for a suitable count, aggregate, or report action before reading individual records/u);
+      assert.match(prompt, /remove earlier date or other filters that no longer apply/u);
+      assert.match(prompt, /follow returned pagination until the requested number or scope is covered/u);
+      assert.match(prompt, /Never present a partial page or sample as complete/u);
+      if (discovery) {
+        assert.match(prompt, /Before claiming an operation or dataset is unavailable.*assistant_action_search/u);
+        assert.match(prompt, /count\/aggregate\/report for totals or summaries/u);
+        assert.match(prompt, /broader terms such as list or query.*omit query to browse/u);
+        assert.match(prompt, /one empty search or unrelated page does not establish that a capability is absent/u);
+        assert.match(prompt, /load assistant_action_contract before assistant_action_execute/u);
+      } else {
+        assert.doesNotMatch(prompt, /assistant_action_search|assistant_action_contract|assistant_action_execute/u);
+      }
+    });
+  }
+});
+
+test("an empty authorized tool set does not advertise discovery or collection actions", async () => {
+  const harness = createHarness([textCompletion("No application actions are available in this session.")], {
+    tools: []
+  });
+
+  await harness.run("List the available records.");
+
+  const request = harness.completionRequests[0];
+  assert.deepEqual(request.tools, []);
+  assert.match(request.messages[0].content, /No tools are currently available for this user\/session/u);
+  assert.doesNotMatch(request.messages[0].content, /assistant_action_search|collection action/u);
+});
+
 test("progress-only output is retried silently and current-time prompts require a workspace clock", async () => {
   const harness = createHarness(
     [