@maximem/synap-js-sdk 0.5.12 → 0.5.13

package/dist/index.cjs

@@ -408,22 +408,40 @@ function isOutcomeUnknown(error) {
   if (error instanceof _chunkUXNVTP4Dcjs.ConflictError) return false;
   if (error instanceof _chunkUXNVTP4Dcjs.InsufficientCreditsError) return false;
   if (error instanceof _chunkUXNVTP4Dcjs.ContextNotFoundError) return false;
-  if (error instanceof _chunkUXNVTP4Dcjs.NetworkTimeoutError) return true;
   if (isConnectionRefusal(error)) return false;
+  if (error instanceof _chunkUXNVTP4Dcjs.NetworkTimeoutError) return true;
   return true;
 }
+var CONNECTION_REFUSAL_CODES = /* @__PURE__ */ new Set([
+  "ECONNREFUSED",
+  "ENOTFOUND",
+  "EAI_AGAIN",
+  "ECONNRESET",
+  "EHOSTUNREACH"
+]);
 function isConnectionRefusal(error) {
-  const code = _nullishCoalesce(_optionalChain([error, 'optionalAccess', _10 => _10.cause, 'optionalAccess', _11 => _11.code]), () => ( _optionalChain([error, 'optionalAccess', _12 => _12.code])));
-  return code === "ECONNREFUSED" || code === "ENOTFOUND" || code === "EAI_AGAIN" || code === "ECONNRESET" || code === "EHOSTUNREACH";
+  let cur = error;
+  for (let depth = 0; depth < 4 && cur !== null && typeof cur === "object"; depth += 1) {
+    const code = cur.code;
+    if (typeof code === "string" && CONNECTION_REFUSAL_CODES.has(code)) return true;
+    cur = cur.cause;
+  }
+  return false;
 }
 function toTransportError(error, correlationId) {
   if (error instanceof _chunkUXNVTP4Dcjs.SynapError) return error;
-  const name = _optionalChain([error, 'optionalAccess', _13 => _13.name]);
+  const name = _optionalChain([error, 'optionalAccess', _10 => _10.name]);
   if (name === "AbortError" || name === "TimeoutError") {
     return new (0, _chunkUXNVTP4Dcjs.NetworkTimeoutError)("Request timed out", { correlationId, cause: error });
   }
+  if (isConnectionRefusal(error)) {
+    return new (0, _chunkUXNVTP4Dcjs.NetworkTimeoutError)(
+      `Connection failed: ${_nullishCoalesce(_optionalChain([error, 'optionalAccess', _11 => _11.message]), () => ( String(error)))}`,
+      { correlationId, cause: error }
+    );
+  }
   return new (0, _chunkUXNVTP4Dcjs.ServiceUnavailableError)(
-    `Network request failed: ${_nullishCoalesce(_optionalChain([error, 'optionalAccess', _14 => _14.message]), () => ( String(error)))}`,
+    `Network request failed: ${_nullishCoalesce(_optionalChain([error, 'optionalAccess', _12 => _12.message]), () => ( String(error)))}`,
     { correlationId, cause: error }
   );
 }
@@ -504,11 +522,11 @@ function stringOrNull(v) {
 }
 function withTimeout(ms, external) {
   const controller = new AbortController();
-  const onExternalAbort = () => controller.abort(_optionalChain([external, 'optionalAccess', _15 => _15.reason]));
+  const onExternalAbort = () => controller.abort(_optionalChain([external, 'optionalAccess', _13 => _13.reason]));
   const timer = setTimeout(() => {
     controller.abort(new (0, _chunkUXNVTP4Dcjs.NetworkTimeoutError)(`Timed out after ${ms}ms`));
   }, ms);
-  _optionalChain([timer, 'access', _16 => _16.unref, 'optionalCall', _17 => _17()]);
+  _optionalChain([timer, 'access', _14 => _14.unref, 'optionalCall', _15 => _15()]);
   if (external) {
     if (external.aborted) onExternalAbort();
     else external.addEventListener("abort", onExternalAbort, { once: true });
@@ -517,7 +535,7 @@ function withTimeout(ms, external) {
     signal: controller.signal,
     dispose() {
       clearTimeout(timer);
-      _optionalChain([external, 'optionalAccess', _18 => _18.removeEventListener, 'call', _19 => _19("abort", onExternalAbort)]);
+      _optionalChain([external, 'optionalAccess', _16 => _16.removeEventListener, 'call', _17 => _17("abort", onExternalAbort)]);
     }
   };
 }
@@ -525,14 +543,14 @@ function sleep(ms, signal) {
   if (ms <= 0) return Promise.resolve();
   return new Promise((resolve, reject) => {
     const timer = setTimeout(() => {
-      _optionalChain([signal, 'optionalAccess', _20 => _20.removeEventListener, 'call', _21 => _21("abort", onAbort)]);
+      _optionalChain([signal, 'optionalAccess', _18 => _18.removeEventListener, 'call', _19 => _19("abort", onAbort)]);
       resolve();
     }, ms);
     function onAbort() {
       clearTimeout(timer);
-      reject(_nullishCoalesce(_optionalChain([signal, 'optionalAccess', _22 => _22.reason]), () => ( new Error("Aborted"))));
+      reject(_nullishCoalesce(_optionalChain([signal, 'optionalAccess', _20 => _20.reason]), () => ( new Error("Aborted"))));
     }
-    _optionalChain([signal, 'optionalAccess', _23 => _23.addEventListener, 'call', _24 => _24("abort", onAbort, { once: true })]);
+    _optionalChain([signal, 'optionalAccess', _21 => _21.addEventListener, 'call', _22 => _22("abort", onAbort, { once: true })]);
   });
 }
 
@@ -745,7 +763,7 @@ function retrievalOptionsOf(options) {
 }
 async function fetchContext(transport, scope, options = {}, hooks = {}) {
   const startedAt = Date.now();
-  await _optionalChain([transport, 'access', _25 => _25.ready, 'optionalCall', _26 => _26()]);
+  await _optionalChain([transport, 'access', _23 => _23.ready, 'optionalCall', _24 => _24()]);
   await transport.checkCustomerId(
     opt(options, "customer_id", "customerId"),
     `${scope}.context.fetch`
@@ -755,11 +773,16 @@ async function fetchContext(transport, scope, options = {}, hooks = {}) {
   const { contextMode } = scopeOptionsOf(scope, options);
   const summaryMode = scope === "user" && contextMode === "conversation-summary";
   const finish = (response2) => {
-    _optionalChain([hooks, 'access', _27 => _27.onResponse, 'optionalCall', _28 => _28(scope, options, response2)]);
+    _optionalChain([hooks, 'access', _25 => _25.onResponse, 'optionalCall', _26 => _26(scope, options, response2)]);
     return response2;
   };
+  const conversationId = opt(options, "conversation_id", "conversationId");
+  const skipServerSt = conversationId ? _optionalChain([hooks, 'access', _27 => _27.skipServerShortTerm, 'optionalCall', _28 => _28(conversationId)]) === true : false;
   const network = async () => {
-    const { endpoint, body } = buildFetchRequest(scope, options);
+    const { endpoint, body } = buildFetchRequest(
+      scope,
+      skipServerSt ? { ...options, include_conversation_context: false } : options
+    );
     const result = await transport.request(endpoint, { body });
     return applyResponseShape(_nullishCoalesce(_optionalChain([result, 'optionalAccess', _29 => _29["context"]]), () => ( {})), responseShape);
   };
@@ -1420,7 +1443,7 @@ function storedBatchRows(bodies, result) {
 function createConversationNamespace(transport, subscribers = new CompactionSubscribers(), onWrite = () => {
 }, onTurn = () => {
 }, onFetched = () => {
-}, localShortTerm = { prompt: () => null, compacted: () => null }, agentModel = () => ({})) {
+}, localShortTerm = { prompt: () => null, compacted: () => null }, agentModel = () => ({}), fetchHooks = {}) {
   const writesInFlight = /* @__PURE__ */ new Map();
   const withTurnModel = (body, options) => {
     const metadata = withAgentModel(
@@ -1527,7 +1550,7 @@ function createConversationNamespace(transport, subscribers = new CompactionSubs
       async fetch(options = {}) {
         const id = _nullishCoalesce(options.conversation_id, () => ( options.conversationId));
         validateConversationId(id);
-        const response = await fetchContext(transport, "conversation", options);
+        const response = await fetchContext(transport, "conversation", options, fetchHooks);
         onFetched(
           response,
           id,
@@ -1976,7 +1999,7 @@ function createInstanceNamespace(credentials, cache, onTurn, agentModel = () =>
 }
 
 // src/version.ts
-var SDK_VERSION = "0.5.12";
+var SDK_VERSION = "0.5.13";
 
 // src/registry.ts
 var instances = /* @__PURE__ */ new Map();
@@ -3799,6 +3822,14 @@ var SynapClient = (_class4 = class _SynapClient {
       // 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.
+      // Python's `_should_skip_server_st`: when this process already holds the
+      // conversation's compaction and every turn after it, the server is told
+      // not to assemble them again.
+      skipServerShortTerm: (conversationId) => {
+        if (!this.#isStAuthoritative()) return false;
+        const entry = this.#shortTerm.get(conversationId);
+        return entry !== null && entry.compactionId !== null && isComplete(entry);
+      },
       onResponse: (scope, options2, response) => {
         if (scope === "conversation") return;
         const o2 = options2;
@@ -3862,7 +3893,12 @@ var SynapClient = (_class4 = class _SynapClient {
         reseed: (conversationId, response) => this.#reseedShortTerm(conversationId, response),
         historyExcerpts: (conversationId, response, query) => this.#withHistoryExcerpts(conversationId, response, query)
       },
-      () => this.#agentModel
+      () => this.#agentModel,
+      // The same hooks as the other three scopes: the anticipation cache first,
+      // as Python's conversation fetch does, and the audit events. Until 0.5.13
+      // the conversation fetch was built without them, so every JS conversation
+      // fetch was a billed round trip, including ones the stream had answered.
+      this.#fetchHooks
     );
     this.memories = createMemoriesInterface(this.#transport, ({ userId, customerId }) => {
       for (const id of /* @__PURE__ */ new Set([userId, customerId])) {
@@ -4476,7 +4512,7 @@ function resolveBaseUrl(options) {
 function resolveCredentials(options) {
   const apiKey = _nullishCoalesce(options.apiKey, () => ( _chunkZKOJDUN4cjs.getEnv.call(void 0, "SYNAP_API_KEY")));
   if (!apiKey) {
-    throw new (0, _chunkUXNVTP4Dcjs.InvalidInputError)(
+    throw new (0, _chunkUXNVTP4Dcjs.AuthenticationError)(
       "No Synap API key found. Set SYNAP_API_KEY in your environment, or pass apiKey to the SynapClient constructor."
     );
   }