@maximem/synap-js-sdk 0.5.11 → 0.5.12

package/dist/index.d.cts

@@ -1873,7 +1873,7 @@ declare class SynapClient {
  * build when the two disagree, which is the only thing that makes a manual
  * publish safe.
  */
-declare const SDK_VERSION = "0.5.11";
+declare const SDK_VERSION = "0.5.12";
 
 /**
  * `response_shape: "ranked"`: one list in the order the server ranked it.

package/dist/index.d.ts

@@ -1873,7 +1873,7 @@ declare class SynapClient {
  * build when the two disagree, which is the only thing that makes a manual
  * publish safe.
  */
-declare const SDK_VERSION = "0.5.11";
+declare const SDK_VERSION = "0.5.12";
 
 /**
  * `response_shape: "ranked"`: one list in the order the server ranked it.

package/dist/index.js

@@ -639,6 +639,7 @@ function buildFetchRequest(scope, options) {
   const perQuery = opt(options, "max_results_per_query", "maxResultsPerQuery");
   const responseShape = responseShapeOf(options);
   const { mode, precisionLevel } = retrievalOptionsOf(options);
+  const { contextMode, lastN } = scopeOptionsOf(scope, options);
   if (searchQuery !== void 0 && !Array.isArray(searchQuery)) {
     throw new InvalidInputError("search_query must be an array when provided");
   }
@@ -666,21 +667,7 @@ function buildFetchRequest(scope, options) {
   switch (scope) {
     case "user":
       {
-        if (!userId) throw new InvalidInputError("user_id is required");
         body["user_id"] = userId;
-        const contextMode = opt(options, "context_mode", "contextMode") ?? "in-conversation";
-        if (!VALID_CONTEXT_MODES.includes(contextMode)) {
-          throw new InvalidInputError(
-            `Invalid context_mode '${contextMode}'. Must be one of: ${VALID_CONTEXT_MODES.join(", ")}`
-          );
-        }
-        const lastN = opt(options, "last_n_conversations", "lastNConversations") ?? 1;
-        if (!Number.isInteger(lastN)) {
-          throw new InvalidInputError("last_n_conversations must be an integer");
-        }
-        if (lastN < 0 || lastN > 20) {
-          throw new InvalidInputError("last_n_conversations must be between 0 and 20");
-        }
         if (contextMode === "conversation-summary") {
           body["context_mode"] = contextMode;
           body["include_profile"] = opt(options, "include_profile", "includeProfile") ?? true;
@@ -691,11 +678,9 @@ function buildFetchRequest(scope, options) {
       }
       break;
     case "customer":
-      if (!customerId) throw new InvalidInputError("customer_id is required");
       body["customer_id"] = customerId;
       break;
     case "conversation":
-      if (!conversationId) throw new InvalidInputError("conversation_id is required");
       if (userId !== void 0) body["user_id"] = userId;
       if (customerId !== void 0) body["customer_id"] = customerId;
       break;
@@ -714,6 +699,33 @@ function responseShapeOf(options) {
   }
   return shape;
 }
+var REQUIRED_ID = {
+  user: ["user_id", "userId"],
+  customer: ["customer_id", "customerId"],
+  conversation: ["conversation_id", "conversationId"],
+  client: null
+};
+function scopeOptionsOf(scope, options) {
+  const required = REQUIRED_ID[scope];
+  if (required !== null && !opt(options, required[0], required[1])) {
+    throw new InvalidInputError(`${required[0]} is required`);
+  }
+  const contextMode = opt(options, "context_mode", "contextMode") ?? "in-conversation";
+  const lastN = opt(options, "last_n_conversations", "lastNConversations") ?? 1;
+  if (scope !== "user") return { contextMode, lastN };
+  if (!VALID_CONTEXT_MODES.includes(contextMode)) {
+    throw new InvalidInputError(
+      `Invalid context_mode '${contextMode}'. Must be one of: ${VALID_CONTEXT_MODES.join(", ")}`
+    );
+  }
+  if (!Number.isInteger(lastN)) {
+    throw new InvalidInputError("last_n_conversations must be an integer");
+  }
+  if (lastN < 0 || lastN > 20) {
+    throw new InvalidInputError("last_n_conversations must be between 0 and 20");
+  }
+  return { contextMode, lastN };
+}
 var VALID_MODES = ["fast", "accurate"];
 var VALID_PRECISION = ["high", "medium"];
 function retrievalOptionsOf(options) {
@@ -740,12 +752,18 @@ async function fetchContext(transport, scope, options = {}, hooks = {}) {
   );
   const responseShape = responseShapeOf(options);
   retrievalOptionsOf(options);
+  const { contextMode } = scopeOptionsOf(scope, options);
+  const summaryMode = scope === "user" && contextMode === "conversation-summary";
+  const finish = (response2) => {
+    hooks.onResponse?.(scope, options, response2);
+    return response2;
+  };
   const network = async () => {
     const { endpoint, body } = buildFetchRequest(scope, options);
     const result = await transport.request(endpoint, { body });
     return applyResponseShape(result?.["context"] ?? {}, responseShape);
   };
-  const anticipated = await hooks.beforeFetch?.(scope, options) ?? null;
+  const anticipated = summaryMode ? null : await hooks.beforeFetch?.(scope, options) ?? null;
   if (anticipated !== null && anticipated.borderline) {
     const pending = network();
     pending.catch(() => void 0);
@@ -757,17 +775,17 @@ async function fetchContext(transport, scope, options = {}, hooks = {}) {
     if (timer !== void 0) clearTimeout(timer);
     if (won !== null) {
       hooks.onAssembled?.(scope, options, won, startedAt);
-      return won;
+      return finish(won);
     }
   }
   if (anticipated !== null) {
     hooks.onServedFromCache?.(scope, options, anticipated);
     hooks.onAssembled?.(scope, options, anticipated.response, startedAt);
-    return applyResponseShape(anticipated.response, responseShape);
+    return finish(applyResponseShape(anticipated.response, responseShape));
   }
   const response = await network();
   hooks.onAssembled?.(scope, options, response, startedAt);
-  return response;
+  return finish(response);
 }
 
 // src/context/normalize.ts
@@ -1958,7 +1976,7 @@ function createInstanceNamespace(credentials, cache, onTurn, agentModel = () =>
 }
 
 // src/version.ts
-var SDK_VERSION = "0.5.11";
+var SDK_VERSION = "0.5.12";
 
 // src/registry.ts
 var instances = /* @__PURE__ */ new Map();
@@ -3776,6 +3794,21 @@ var SynapClient = class _SynapClient {
       },
       onServedFromCache: (_scope, options2, served) => {
         this.#emitContextUsed(options2, _scope, served);
+      },
+      // The periodic user-summary splice, as Python runs it at the end of the
+      // user, customer and client fetches (the conversation fetch does its own
+      // through `onFetched`). Until 0.5.12 only the conversation fetch had it,
+      // so the same usage gave the model different context in JS.
+      onResponse: (scope, options2, response) => {
+        if (scope === "conversation") return;
+        const o2 = options2;
+        const owner = scope === "user" ? o2["user_id"] ?? o2["userId"] : scope === "customer" ? o2["customer_id"] ?? o2["customerId"] : "_client";
+        this.#injectUserSummary(
+          response,
+          o2["conversation_id"] ?? o2["conversationId"],
+          owner,
+          rungKey(o2["scope_path"] ?? o2["scopePath"])
+        );
       }
     };
     this.user = createUserNamespace(this.#transport, this.#fetchHooks);