@pmate/chat-sdk 0.12.0-next.77 → 0.12.0-next.79

package/lib/runtime/chatSession.js

@@ -304,12 +304,11 @@ export class ChatSession {
             throw new Error("INVALID_MESSAGE_QUERY");
         // Explicit authoritative changes (for example history assignment) can invalidate
         // even a recently cached empty window. Keep this on the existing read surface.
-        if (options.refresh)
-            await this.messages.read(threadId, options.limit ?? 100, true);
+        const refreshed = options.refresh ? await this.messages.read(threadId, options.limit ?? 100, true) : undefined;
         if (options.all) {
             return this.readAll(threadId);
         }
-        const page = await (options.before ? this.messages.earlier(threadId, options.before, options.limit) : this.messages.read(threadId, options.limit));
+        const page = await (options.before ? this.messages.earlier(threadId, options.before, options.limit) : refreshed ?? this.messages.read(threadId, options.limit));
         this.assertOpen();
         await this.confirmMessages(page.messages);
         if (this.initializing && !this.readingStarted && page.complete)
@@ -339,7 +338,12 @@ export class ChatSession {
         if (running)
             return running;
         const task = (async () => {
-            const recent = await this.getMessages(threadId);
+            const cached = this.messages.getSnapshot(threadId), known = this.historyEdges.get(threadId);
+            // The earlier-page path validates its own durable scope and coverage.
+            // A known boundary does not require rereading/reconfirming the newest
+            // message bodies before loading older messages.
+            const recent = cached?.complete && known?.hasEarlier && known.version === cached.resetVersion
+                ? cached : await this.getMessages(threadId);
             if (!recent.complete)
                 throw new Error("MESSAGE_WINDOW_INCOMPLETE");
             const edge = this.historyEdges.get(threadId);

package/lib/runtime/messageStore.d.ts

@@ -57,6 +57,7 @@ export declare class IndexedDbMessageStore<T extends StoredMessage> {
         hasEarlier: boolean;
         coverage: MessageCoverage | undefined;
     }>;
+    private readRows;
     /** Bounded identifier reads for SDK-owned view handoffs; no transport fallback. */
     readSelected(thread: string, ids: readonly string[]): Promise<{
         messages: T[];

package/lib/runtime/messageStore.js

@@ -42,12 +42,24 @@ export class IndexedDbMessageStore {
             const access = await this.scopeAccess();
             if (access.revoked)
                 return { messages: [], hasEarlier: false, coverage: { complete: false, revoked: true, resetVersion: 0, scopeVersion: access.version } };
-            const range = this.rows.where("[scope+thread+time+id]").between([this.key, thread, Dexie.minKey, Dexie.minKey], [this.key, thread, before?.timestamp ?? Dexie.maxKey, before?.id ?? Dexie.maxKey], true, !before);
-            const records = await range.reverse().limit(limit + 1).toArray();
+            const records = await this.readRows(this.rows, thread, limit, before);
             return this.controlPage({ messages: records.slice(0, limit).reverse().map(row => row.value), hasEarlier: records.length > limit, coverage: this.scopedCoverage(await this.coverage.get([this.key, thread]), access) });
         });
         return this.finishPage(thread, page);
     }
+    readRows(table, thread, limit, before) {
+        const lower = [this.key, thread, Dexie.minKey, Dexie.minKey], upper = [this.key, thread, before?.timestamp ?? Dexie.maxKey, before?.id ?? Dexie.maxKey];
+        const index = Dexie.currentTransaction.idbtrans.objectStore(table.name).index("[scope+thread+time+id]");
+        // Bounded batches avoid a browser round trip per history row. Older engines
+        // retain the same indexed cursor path and pagination boundaries.
+        if (typeof index.getAllRecords === "function")
+            return new Dexie.Promise((resolve, reject) => {
+                const request = index.getAllRecords({ query: IDBKeyRange.bound(lower, upper, false, Boolean(before)), count: limit + 1, direction: "prev" });
+                request.onsuccess = () => resolve(request.result.map(record => record.value));
+                request.onerror = () => reject(request.error);
+            });
+        return table.where("[scope+thread+time+id]").between(lower, upper, true, !before).reverse().limit(limit + 1).toArray();
+    }
     /** Bounded identifier reads for SDK-owned view handoffs; no transport fallback. */
     async readSelected(thread, ids) {
         if (ids.length > 100 || ids.some(id => !id || id.length > 256))
@@ -118,7 +130,7 @@ export class IndexedDbMessageStore {
             const coverage = access.revoked ? { complete: false, revoked: true, resetVersion: 0, scopeVersion: access.version } : this.scopedCoverage(await this.coverage.get([this.key, thread]), access);
             if (coverage?.revoked)
                 return { messages: [], hasEarlier: false, coverage };
-            const read = (table) => table.where("[scope+thread+time+id]").between([this.key, thread, Dexie.minKey, Dexie.minKey], [this.key, thread, before?.timestamp ?? Dexie.maxKey, before?.id ?? Dexie.maxKey], true, !before).reverse().limit(limit + 1).toArray();
+            const read = (table) => this.readRows(table, thread, limit, before);
             const [canonical, legacy] = await Promise.all([read(this.rows), read(this.legacy)]);
             const rows = [...new Map([...legacy, ...canonical].map(row => [row.id, row])).values()].sort((a, b) => a.time - b.time || (a.id < b.id ? -1 : a.id > b.id ? 1 : 0));
             return this.controlPage({ messages: rows.slice(-limit).map(row => row.value), hasEarlier: rows.length > limit, coverage });
@@ -197,7 +209,8 @@ export class IndexedDbMessageStore {
         const thread = page.messages[0]?.threadId;
         const targets = thread && references.length ? (await this.rows.bulkGet(references.map(id => [this.key, thread, id]))).flatMap(row => row ? [row.value] : []) : [];
         const selected = [...page.messages, ...targets];
-        const facts = await this.controls.where("[scope+thread+target]").anyOf(selected.map(message => [this.key, message.threadId, message.id])).toArray();
+        // Exact bounded index reads avoid serial cursor seeks across a message page.
+        const facts = (await Promise.all(selected.map(message => this.controls.where("[scope+thread+target]").equals([this.key, message.threadId, message.id]).toArray()))).flat();
         const latest = new Map();
         for (const event of [...facts, ...events]) {
             const key = JSON.stringify(controlKey(event));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmate/chat-sdk",
-  "version": "0.12.0-next.77",
+  "version": "0.12.0-next.79",
   "description": "Local-first chat SDK, message subscriptions, reading state, and shared chat UI primitives",
   "type": "module",
   "main": "./lib/index.js",