wontopos 2.2.20 → 2.2.22

package/dist/wontopos.d.ts

@@ -76,8 +76,6 @@ export declare class RateLimitError extends WosError {
 /** 5xx — the service failed. Safe to retry. */
 export declare class ServerError extends WosError {
 }
-/** Build the most specific WosError subclass for an HTTP status. */
-export declare function errorFor(status: number, message: string, requestId?: string): WosError;
 export interface Usage {
     input_tokens?: number;
     output_tokens?: number;

package/dist/wontopos.js

@@ -37,7 +37,7 @@
  * runtime's (never disabled by this SDK; Node 18+ floors TLS at 1.2). Prefer
  * `Client.fromEnv()` over keys in source code.
  */
-const VERSION = "2.2.20";
+const VERSION = "2.2.22";
 /** Runtime info helps support debug a report ("node 18 on Windows...") —
  * platform only, never anything identifying. Browsers have no `process` (and
  * silently drop the UA header anyway). */
@@ -129,8 +129,35 @@ function parseRateLimit(h) {
 // that isn't actually an array/object to the empty value.
 const asList = (x) => (Array.isArray(x) ? x : []);
 const asObj = (x) => (x != null && typeof x === "object" && !Array.isArray(x) ? x : {});
+// A list of API records (memories, turns, models...) — every element is an object
+// by contract. `asList` only fixes the CONTAINER: a hostile/broken server sending
+// `{"memories": [null, 1, "x", {...}]}` still handed the caller those non-objects,
+// typed as Memory[], and the first `m.content` threw a raw TypeError from inside
+// user code. Drop elements that aren't objects and keep the valid records, so one
+// bad element never poisons the batch (and never destroys it).
+const asRecords = (x) => asList(x).filter((m) => m != null && typeof m === "object" && !Array.isArray(m));
 // Attach a delivery-form override (memoir/archive) + tz to a body, the way search
 // passes them; on Scroll 1.2+ this renders each returned memory's time in that form.
+/** Every memory a search returned — main lane and self lane together.
+ *
+ * On a self-lane model the service splits the result: the assistant's OWN words
+ * come back in `self_memories` and are REMOVED from `memories`, so the two never
+ * double-count. `search()` read only `memories`, so on Scroll 1.2 an assistant
+ * turn stored with `addTurn` was silently missing from search while the same
+ * query on tablet-1 returned it — upgrading the model made search return LESS,
+ * and the dropped memories were already on the wire and already paid for.
+ *
+ * Both lanes, de-duplicated by id, each memory keeping its `speaker` so the
+ * caller can still tell who said what. Callers who want the lanes kept apart use
+ * `searchSelf()`, which exists for exactly that. */
+function mergeLanes(r) {
+    const main = asRecords(r?.memories);
+    const mine = asRecords(r?.self_memories);
+    if (mine.length === 0)
+        return main;
+    const seen = new Set(main.map((m) => m.id).filter((id) => typeof id === "string"));
+    return main.concat(mine.filter((m) => !(typeof m.id === "string" && seen.has(m.id))));
+}
 function withForm(body, form, tz) {
     if (form !== undefined)
         body.form = form;
@@ -188,7 +215,8 @@ const STATUS_ERRORS = {
     429: RateLimitError,
 };
 /** Build the most specific WosError subclass for an HTTP status. */
-export function errorFor(status, message, requestId) {
+// Internal factory — the typed error CLASSES are the public surface.
+function errorFor(status, message, requestId) {
     const Cls = status === 0
         ? APIConnectionError
         : STATUS_ERRORS[status] ?? (status >= 500 && status < 600 ? ServerError : WosError);
@@ -343,7 +371,7 @@ export class Client {
         // Reserved fields win over ...opts: an app that forwards untrusted input as
         // opts must not be able to override the store (user_id), query, or limit.
         const r = await this.post("/api/v1/memory/search", { ...opts, user_id: this.uid(userId), query, max_results: limit });
-        return asList(r.memories);
+        return mergeLanes(r);
     }
     /** Search a self-memory model (Scroll 1.2+): both lanes from ONE call. Returns
      * `{ memories, self_memories }` — `memories` is the main lane (what others said /
@@ -352,7 +380,7 @@ export class Client {
      * without a self lane, `self_memories` is `[]`. `userId` optional (default store). */
     async searchSelf(query, userId, limit = 10, opts = {}) {
         const r = await this.post("/api/v1/memory/search", { ...opts, user_id: this.uid(userId), query, max_results: limit });
-        return { memories: asList(r.memories), self_memories: asList(r.self_memories) };
+        return { memories: asRecords(r.memories), self_memories: asRecords(r.self_memories) };
     }
     /** One-call LLM context: short-term turns + long-term matches + surrounding context.
      * `form` ("memoir"/"archive", Scroll 1.2+) renders each long-term memory's time in
@@ -368,7 +396,7 @@ export class Client {
     /** Recent conversation turns (short-term memory). */
     async history(userId) {
         const r = await this.post("/api/v1/memory/history", { user_id: this.uid(userId) });
-        return asList(r.turns);
+        return asRecords(r.turns);
     }
     /** Memory counts for a store: { total_memories, short_term_turns }. */
     stats(userId) {
@@ -416,7 +444,7 @@ export class Client {
         // server that mints a FRESH cursor every page forever — bound the walk.
         for (let page = 0; page < MAX_PAGES; page++) {
             const p = await this.listMemories(userId, { limit: opts.pageSize ?? 100, cursor });
-            for (const m of asList(p.memories))
+            for (const m of asRecords(p.memories))
                 yield m;
             const next = p.next_cursor ?? undefined;
             if (!next || seen.has(next))
@@ -443,7 +471,7 @@ export class Client {
     /** Available models: `[{ id, name, available, memory }, ...]`. Needs no API key. */
     async listModels() {
         const data = await this.request("GET", "/api/v1/models");
-        return asList(data.models);
+        return asRecords(data.models);
     }
     // ----- stores -----
     /** Create a store — the `userId` you read and write under. Stores are explicit:
@@ -456,7 +484,7 @@ export class Client {
     /** List your stores: `[{ user_id, created_at }, ...]` (`default` first). */
     async listStores() {
         const r = await this.request("GET", "/api/v1/memory/collections");
-        return asList(r.collections);
+        return asRecords(r.collections);
     }
     /** Delete a store and ALL its memories. Returns `{ user_id, status }`. */
     deleteStore(userId) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wontopos",
-  "version": "2.2.20",
+  "version": "2.2.22",
   "description": "Wontopos — long-term memory for AI agents. Pure semantic retrieval, identical recall in every language, ~100× lower LLM bill.",
   "keywords": [
     "memory",