@novasamatech/host-substrate-chain-connection 0.7.6 → 0.7.7

package/dist/connectionPool.js

@@ -106,15 +106,31 @@ export const createChainConnection = ({ resolve, clientOptions, createProvider,
         },
         getProvider(chain) {
             return getSyncProvider(onResult => {
+                let teardownCalled = false;
+                let pendingUnlock = null;
+                // Idempotent: subsequent calls (e.g. teardown after disconnect) are no-ops.
+                const releaseUnlock = () => {
+                    const unlock = pendingUnlock;
+                    pendingUnlock = null;
+                    unlock?.();
+                };
                 rawAcquire(chain)
                     .then(({ pooled, unlock }) => {
-                    onResult((onMessage, _onHalt) => pooled.provider.branch(unlock)(onMessage));
+                    if (teardownCalled) {
+                        unlock();
+                        onResult(null);
+                        return;
+                    }
+                    pendingUnlock = unlock;
+                    onResult((onMessage, _onHalt) => pooled.provider.branch(releaseUnlock)(onMessage));
                 })
                     .catch(() => {
                     onResult(null);
                 });
+                // Covers teardown without disconnect, and teardown before acquire resolves.
                 return () => {
-                    /* empty */
+                    teardownCalled = true;
+                    releaseUnlock();
                 };
             });
         },

package/dist/metadataCache.js

@@ -39,7 +39,10 @@ export const createMetadataCache = (options) => {
                 setMetadata(key, metadata) {
                     const k = cacheKey(chainId, key);
                     memory.set(k, metadata);
-                    storage?.write(k, bytesToBase64(metadata));
+                    // setMetadata is fire-and-forget by contract; log persist failures.
+                    storage?.write(k, bytesToBase64(metadata)).orTee(error => {
+                        console.error(`[metadataCache] failed to persist metadata for ${k}:`, error);
+                    });
                 },
             };
         },

package/dist/metadataCache.spec.js

@@ -3,8 +3,8 @@ import { describe, expect, it, vi } from 'vitest';
 import { createMetadataCache } from './metadataCache.js';
 const createMockStorage = () => ({
     read: vi.fn(),
-    write: vi.fn(),
-    clear: vi.fn(),
+    write: vi.fn(() => okAsync(undefined)),
+    clear: vi.fn(() => okAsync(undefined)),
     // eslint-disable-next-line @typescript-eslint/no-empty-function
     subscribe: vi.fn(() => () => { }),
 });

package/dist/subscriptionReplayProvider.js

@@ -69,6 +69,15 @@ export const withSubscriptionReplay = (provider, onReconnect) => onMessage => {
         onMessage(message);
     });
     const unsubReconnect = onReconnect(() => {
+        // Replay unconfirmed subscribes first: their `reconnectFor === null`, so
+        // they're disjoint from the entries inserted by the active-replay loop
+        // below (which all carry a non-null `reconnectFor`).
+        for (const [, pending] of pendingSubscriptions) {
+            if (pending.reconnectFor === null)
+                conn.send(pending.payload);
+        }
+        // Replay confirmed subscriptions: the server returns a fresh subId, which
+        // we map back to the consumer's stable subId.
         for (const [consumerSubId, sub] of activeSubscriptions) {
             pendingSubscriptions.set(sub.id, { payload: sub.payload, reconnectFor: consumerSubId });
             conn.send(sub.payload);

package/dist/subscriptionReplayProvider.spec.js

@@ -61,15 +61,16 @@ describe('withSubscriptionReplay', () => {
         control.triggerReconnect();
         expect(mock.send).not.toHaveBeenCalled();
     });
-    it('does not replay pending subscriptions (no server response yet) on reconnect', () => {
+    it('replays pending subscriptions (no server response yet) on reconnect', () => {
         const mock = createMockProvider();
         const control = createReconnectControl();
         const conn = withSubscriptionReplay(mock.provider, control.onReconnect)(vi.fn());
-        conn.send(req(1, 'statement_subscribeStatement'));
+        const subscribeMsg = req(1, 'statement_subscribeStatement');
+        conn.send(subscribeMsg);
         // intentionally no simulateMessage — subscription not confirmed by server
         mock.send.mockClear();
         control.triggerReconnect();
-        expect(mock.send).not.toHaveBeenCalled();
+        expect(mock.send).toHaveBeenCalledWith(subscribeMsg);
     });
     it('replays active subscriptions on reconnect', () => {
         const mock = createMockProvider();

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@novasamatech/host-substrate-chain-connection",
   "type": "module",
-  "version": "0.7.6",
+  "version": "0.7.7",
   "description": "Chain connection pool with ref counting and provider branching for Polkadot API",
   "license": "Apache-2.0",
   "repository": {
@@ -25,7 +25,7 @@
     "README.md"
   ],
   "dependencies": {
-    "@novasamatech/storage-adapter": "0.7.6",
+    "@novasamatech/storage-adapter": "0.7.7",
     "@polkadot-api/ws-provider": "^0.9.0",
     "@polkadot-api/json-rpc-provider": "^0.2.0",
     "@polkadot-api/json-rpc-provider-proxy": "^0.4.0",