@pmate/chat-sdk 0.12.0-next.47 → 0.12.0-next.49

package/lib/runtime/durableSender.js

@@ -424,19 +424,32 @@ export class DurableChatSender {
         if (!hashes.length)
             return;
         await this.ready;
-        await this.withEpoch("rw", async () => {
-            const previous = await this.outbox.bulkGet(hashes.map(hash => [this.scope, hash]));
-            const confirmed = previous.filter((record) => Boolean(record && !record.confirmed)).map(record => ({ ...record, voiceBlob: undefined, voiceData: undefined, status: "sent", confirmed: true }));
-            if (confirmed.length)
-                await this.outbox.bulkPut(confirmed);
-        });
         let changed = false;
-        for (const hash of hashes) {
-            changed = this.values.delete(hash) || changed;
-            this.releaseUrl(hash);
+        try {
+            for (let offset = 0; offset < hashes.length; offset += 128) {
+                const batch = hashes.slice(offset, offset + 128);
+                await this.withEpoch("rw", async () => {
+                    const previous = await this.outbox.bulkGet(batch.map(hash => [this.scope, hash]));
+                    const confirmed = previous.filter((record) => Boolean(record && !record.confirmed)).map(record => ({ ...record, voiceBlob: undefined, voiceData: undefined, status: "sent", confirmed: true }));
+                    if (confirmed.length)
+                        await this.outbox.bulkPut(confirmed);
+                });
+                // Retire only committed rows. A later failed transaction leaves the
+                // remaining sends retryable, including after a browser restart.
+                for (const hash of batch) {
+                    changed = this.values.delete(hash) || changed;
+                    this.releaseUrl(hash);
+                }
+                if (offset + batch.length < hashes.length)
+                    await new Promise(resolve => setTimeout(resolve, 0));
+            }
+        }
+        finally {
+            // One confirmation operation publishes at most once, also on partial
+            // failure so consumers see exactly the progress that was committed.
+            if (changed)
+                this.emit();
         }
-        if (changed)
-            this.emit();
     }
     async resume() {
         await this.ready;

package/lib/runtime/readIndex.d.ts

@@ -71,6 +71,7 @@ export declare function installReadSnapshot(current: ReadIndexState, snapshot: R
 export declare function applyReadChange(current: ReadIndexState, change: ReadChange): ReadIndexState;
 /** Persist observations even if the message arrived before its Sequence metadata. */
 export declare function recordReadIntent(current: ReadIndexState, intent: ObservedReadIntent): ReadIndexState;
+export declare function resolveObservedReads(current: ReadIndexState): ReadIndexState;
 /** Called only for a message the visible viewport actually reached. */
 export declare function recordVisibleRead(current: ReadIndexState, threadId: string, messageId: string, operationId: string): ReadIndexState;
 export declare function recordExplicitRead(current: ReadIndexState, threadId: string, messageId: string, operationId: string): ReadIndexState;

package/lib/runtime/readIndex.js

@@ -105,12 +105,12 @@ export function recordReadIntent(current, intent) {
         throw new Error("INVALID_READ_INTENT");
     const observed = current.observedReadIntents ?? [];
     if (observed.some(item => item.threadId === intent.threadId && item.messageId === intent.messageId && item.readPolicy === intent.readPolicy))
-        return current;
+        return resolveObservedReads(current);
     if (observed.some(item => item.operationId === intent.operationId) || current.pending.some(item => item.operationId === intent.operationId))
         throw new Error("INVALID_READ_OPERATION_ID");
     return resolveObservedReads({ ...current, observedReadIntents: [...observed, structuredClone(intent)] });
 }
-function resolveObservedReads(current) {
+export function resolveObservedReads(current) {
     let next = current;
     const remaining = [];
     for (const intent of current.observedReadIntents ?? []) {
@@ -127,6 +127,9 @@ function resolveObservedReads(current) {
         next = intent.readPolicy === "explicit" ? recordExplicitRead(next, intent.threadId, intent.messageId, intent.operationId)
             : recordVisibleRead(next, intent.threadId, intent.messageId, intent.operationId);
     }
+    const previous = current.observedReadIntents;
+    if (next === current && previous && previous.length === remaining.length && previous.every((intent, i) => intent === remaining[i]))
+        return current;
     return { ...next, observedReadIntents: remaining };
 }
 /** Called only for a message the visible viewport actually reached. */
@@ -231,6 +234,9 @@ export class IndexedDbReadIndex {
         // Native indexes track scalar source identity even when an older writer
         // changes rows. Reading these keys does not deserialize messageIds arrays.
         this.database.version(4).stores({ readThreads: "[owner+id],owner,[owner+id+value.kind+value.teamId],[owner+value.conversationId+value.kind+value.teamId+id],[owner+id+value.conversationId]" });
+        // Keep v4 indexes for already-open older SDKs, and make source discovery
+        // dense for legitimate legacy rows whose optional team field is absent.
+        this.database.version(5).stores({ readThreads: "[owner+id],owner,[owner+id+value.kind+value.teamId],[owner+value.conversationId+value.kind+value.teamId+id],[owner+id+value.conversationId],[owner+id+value.kind],[owner+value.conversationId+value.kind+id]" });
         this.positionRows = this.database.table("readPositions");
         this.threadRows = this.database.table("readThreads");
         this.messageRows = this.database.table("readMessages");
@@ -451,16 +457,23 @@ export class IndexedDbReadIndex {
             return undefined;
         const sources = new Map();
         for (const threadId of new Set(scope.targets.map(target => target.threadId))) {
-            const direct = await this.threadRows.where("[owner+id+value.kind+value.teamId]").between([owner, threadId, Dexie.minKey], [owner, threadId, Dexie.maxKey], true, true).limit(129).keys();
-            const assigned = await this.threadRows.where("[owner+value.conversationId+value.kind+value.teamId+id]").between([owner, threadId, Dexie.minKey], [owner, threadId, Dexie.maxKey], true, true).limit(129).keys();
+            const direct = await this.threadRows.where("[owner+id+value.kind]").between([owner, threadId, Dexie.minKey], [owner, threadId, Dexie.maxKey], true, true).limit(129).keys();
+            const assigned = await this.threadRows.where("[owner+value.conversationId+value.kind+id]").between([owner, threadId, Dexie.minKey], [owner, threadId, Dexie.maxKey], true, true).limit(129).keys();
             if (direct.length + assigned.length > 128)
                 return undefined;
-            const identities = [...direct.map(value => ({ id: value[1], kind: value[2], teamId: value[3] })), ...assigned.map(value => ({ id: value[4], kind: value[2], teamId: value[3] }))];
-            if (!identities.length && base.threads.some(thread => thread.id === threadId || thread.conversationId === threadId))
+            const identities = [...new Map([...direct.map(value => ({ id: value[1], kind: value[2] })), ...assigned.map(value => ({ id: value[3], kind: value[2] }))].map(value => [value.id, value])).values()];
+            const indexedIds = new Set(identities.map(identity => identity.id));
+            const expected = base.threads.filter(thread => thread.id === threadId || thread.conversationId === threadId);
+            // Sparse indexes omit legacy sources without team metadata. A partial
+            // family cannot safely resolve an observation for the logical thread.
+            if (indexedIds.size !== expected.length || expected.some(thread => !indexedIds.has(thread.id)))
                 return undefined;
             for (const identity of identities) {
                 const cached = base.threads.find(thread => thread.id === identity.id);
-                if (!cached || cached.kind !== identity.kind || cached.teamId !== identity.teamId)
+                if (!cached || cached.kind !== identity.kind)
+                    return undefined;
+                const team = await this.threadRows.where("[owner+id+value.kind+value.teamId]").between([owner, identity.id, Dexie.minKey], [owner, identity.id, Dexie.maxKey], true, true).limit(2).keys();
+                if (team.length > 1 || cached.teamId !== team[0]?.[3])
                     return undefined;
                 const mapping = await this.threadRows.where("[owner+id+value.conversationId]").between([owner, identity.id, Dexie.minKey], [owner, identity.id, Dexie.maxKey], true, true).limit(2).keys();
                 if (mapping.length > 1 || cached.conversationId !== mapping[0]?.[2])

package/lib/runtime/readSync.js

@@ -1,4 +1,4 @@
-import { readingMetadata, assertReadIndexVersions, applyReadChange, ReadSyncGap, createReadIndex, installReadSnapshot } from "./readIndex.js";
+import { readingMetadata, resolveObservedReads, assertReadIndexVersions, applyReadChange, ReadSyncGap, createReadIndex, installReadSnapshot } from "./readIndex.js";
 /** Keeps local operations until an authoritative projection covers them. */
 export class ReadSyncClient {
     options;
@@ -11,9 +11,13 @@ export class ReadSyncClient {
     async loadLocal() {
         const { appId, profileId, persistence, metadataOnly } = this.options;
         const state = await persistence.load(appId, profileId) ?? createReadIndex(appId, profileId);
-        if (!metadataOnly || readingMetadata(state) === state)
+        const normalize = (current) => {
+            const next = metadataOnly ? readingMetadata(current) : current;
+            return next.observedReadIntents?.length ? resolveObservedReads(next) : next;
+        };
+        if (normalize(state) === state)
             return state;
-        return persistence.update(appId, profileId, readingMetadata);
+        return persistence.update(appId, profileId, normalize);
     }
     stop() { this.controller.abort(); }
     synchronize() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pmate/chat-sdk",
-  "version": "0.12.0-next.47",
+  "version": "0.12.0-next.49",
   "description": "Cross-platform PMate chat input components, voice input primitives, and ASR helpers",
   "type": "module",
   "main": "./lib/index.js",