libp2r2p 0.11.2 → 0.11.3

package/README.md

@@ -912,4 +912,5 @@ Private-message consumers can now substitute recovery persistence through
 [`private-messenger/event-store`](private-messenger/event-store/README.md), or use
 local defaults, including a bounded streaming file cache. The optional
 [`private-messenger/session`](private-messenger/session/README.md) coordinator
-provides reusable DM inbox/outbox and attachment lifecycle without UI coupling.
+provides reusable DM inbox/outbox and attachment lifecycle without UI coupling,
+including an optional `onSendError` callback identifying the failed outbox message.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "libp2r2p",
-  "version": "0.11.2",
+  "version": "0.11.3",
   "description": "Peer-to-relay-to-peer",
   "keywords": [
     "p2r2p",

package/private-messenger/session/README.md

@@ -14,8 +14,16 @@ persistence; absent file storage uses the library cache. `mode` defaults to
 
 Methods: `setPeers`, `setAvailable(boolean)`, `enqueue`, `retry`, `cancel`,
 `download`, `cancelDownload`, and `close`. Callbacks are `onOutbox`, `onMedia`,
-and `onError`. The caller maps its account permissions/connectivity/lock state
-to availability and owns view subscriptions. There are no DOM or UI imports.
+`onError`, and optional `onSendError(error, { id, peer })`. The caller maps its
+account permissions/connectivity/lock state to availability and owns view subscriptions. There are no DOM or UI imports.
+
+`onSendError` identifies the main outbox message even when its quote, metadata,
+file chunks, or local persistence failed. The original error is also reported to
+`onError` unchanged. It fires once per failed send attempt (including retries),
+not when persisted failures are read, nor for inbox/download/deletion operations
+or cancelled/closed sends. Apps own route visibility and user-intent filtering;
+automatic retries may also fail. Callbacks must not throw. Errors are not copied
+into persisted outbox records.
 
 `enqueue({ peer, event, context, requiredFiles, deletion })` preserves prepared
 identities. Local message commits and remote stages have independent checkpoints.

package/private-messenger/session/index.js

@@ -16,7 +16,7 @@ export function wireEvent (value, owner) {
 // Relay rejection text must not be interpreted as a local signer denial.
 const retryable = error => error?.code === 'MESSAGE_NOT_PUBLISHED' || !/DENIED|PERMISSION|REVOKED|READ_ONLY|INVALID|BLOCKED|EXPIRED|NOT_IN_PERSONA/i.test(`${error?.code || ''} ${error?.message || ''}`)
 
-export function createPrivateMessageSession ({ owner, signer, eventStore, messageStorage, chunkStorage, recoveryStorage, mode = 'seeder', seedersForPeer = peer => [peer], allowedKinds = [5, 9, 1063, 34601], onMedia = () => {}, onOutbox = () => {}, onError = () => {}, Messenger = createPrivateMessenger, openOutbox = createChatOutbox, FileTransfer = createPrivateFileTransfer, openDownloads = options => createChatOutbox({ ...options, namespace: 'downloads' }) }) {
+export function createPrivateMessageSession ({ owner, signer, eventStore, messageStorage, chunkStorage, recoveryStorage, mode = 'seeder', seedersForPeer = peer => [peer], allowedKinds = [5, 9, 1063, 34601], onMedia = () => {}, onOutbox = () => {}, onError = () => {}, onSendError = () => {}, Messenger = createPrivateMessenger, openOutbox = createChatOutbox, FileTransfer = createPrivateFileTransfer, openDownloads = options => createChatOutbox({ ...options, namespace: 'downloads' }) }) {
   const userSigner = messengerSigner(signer)
   messageStorage ||= createEventStoreMessageStorage({ eventStore })
   chunkStorage ||= eventStore ? createEventStoreChunkStorage({ eventStore }) : undefined
@@ -207,7 +207,14 @@ export function createPrivateMessageSession ({ owner, signer, eventStore, messag
       entries.delete(entry.id)
       emit()
     } catch (error) {
-      if (cancelled.has(entry.id)) { await storage.remove(entry.id); entries.delete(entry.id) } else { entry.status = 'error'; entry.failed = true; entry.retryable = retryable(error); await storage.put(entry, { existing: true }).catch(onError); onError(error) }
+      if (cancelled.has(entry.id)) { await storage.remove(entry.id); entries.delete(entry.id) } else {
+        entry.status = 'error'; entry.failed = true; entry.retryable = retryable(error)
+        await storage.put(entry, { existing: true }).catch(onError)
+        onError(error)
+        // The failed wire event may be a quote or file chunk. Report the
+        // owning outbox item separately, without mutating the native error.
+        if (!closed && !entry.deletion && !cancelled.has(entry.id)) onSendError(error, { id: entry.id, peer: entry.peer })
+      }
       emit()
     }
   }