@dedot/api 0.9.4 → 0.9.5-next.20891389.0

package/cjs/json-rpc/group/ChainHead/ChainHead.js

@@ -167,28 +167,18 @@ class ChainHead extends JsonRpcGroup_js_1.JsonRpcGroup {
                         return;
                     this.#finalizedQueue.push(hash);
                 });
-                this.emit('finalizedBlock', this.findBlock(this.#finalizedHash));
-                // TODO should we find all descendants of the pruned blocks and unpin them as well?
-                //      that's probably a premature optimization
-                const finalizedBlockHeights = finalizedBlockHashes.map((hash) => this.findBlock(hash).number);
-                const pinnedHashes = Object.keys(this.#pinnedBlocks);
-                const hashesToUnpin = new Set([
-                    ...prunedBlockHashes.filter((hash) => pinnedHashes.includes(hash)),
-                    // Since we have the current finalized blocks,
-                    // we can mark all the other blocks at the same height as pruned and unpin all together with the reported pruned blocks
-                    ...Object.values(this.#pinnedBlocks)
-                        .filter((b) => finalizedBlockHeights.includes(b.number))
-                        .filter((b) => !finalizedBlockHashes.includes(b.hash))
-                        .map((b) => b.hash),
-                ]);
+                const currentFinalizedBlock = this.findBlock(this.#finalizedHash);
+                this.emit('finalizedBlock', currentFinalizedBlock);
                 // TODO account for operations that haven't received its operationId yet
                 Object.values(this.#handlers).forEach(({ defer, hash, operationId }) => {
-                    if (hashesToUnpin.has(hash)) {
+                    if (prunedBlockHashes.includes(hash)) {
                         defer.reject(new error_js_1.ChainHeadBlockPrunedError());
                         this.stopOperation(operationId).catch(utils_1.noop);
                         delete this.#handlers[operationId];
                     }
                 });
+                const pinnedHashes = Object.keys(this.#pinnedBlocks);
+                const hashesToUnpin = new Set(prunedBlockHashes.filter((hash) => pinnedHashes.includes(hash)));
                 // Unpin the oldest finalized pinned blocks to maintain the queue size
                 if (this.#finalizedQueue.length > exports.MIN_FINALIZED_QUEUE_SIZE) {
                     const finalizedQueue = this.#finalizedQueue.slice();
@@ -206,6 +196,17 @@ class ChainHead extends JsonRpcGroup_js_1.JsonRpcGroup {
                     });
                     this.#finalizedQueue = finalizedQueue;
                 }
+                // Unpin all obsolete blocks with blockNumber < the latest finalized block number
+                // & not a finalized block & is not in use
+                pinnedHashes.forEach((hash) => {
+                    if (this.#blockUsage.usage(hash) > 0)
+                        return;
+                    if (this.#finalizedQueue.includes(hash))
+                        return;
+                    if (this.findBlock(hash).number > currentFinalizedBlock.number)
+                        return;
+                    hashesToUnpin.add(hash);
+                });
                 hashesToUnpin.forEach((hash) => {
                     if (!this.isPinned(hash))
                         return;

package/json-rpc/group/ChainHead/ChainHead.js

@@ -164,28 +164,18 @@ export class ChainHead extends JsonRpcGroup {
                         return;
                     this.#finalizedQueue.push(hash);
                 });
-                this.emit('finalizedBlock', this.findBlock(this.#finalizedHash));
-                // TODO should we find all descendants of the pruned blocks and unpin them as well?
-                //      that's probably a premature optimization
-                const finalizedBlockHeights = finalizedBlockHashes.map((hash) => this.findBlock(hash).number);
-                const pinnedHashes = Object.keys(this.#pinnedBlocks);
-                const hashesToUnpin = new Set([
-                    ...prunedBlockHashes.filter((hash) => pinnedHashes.includes(hash)),
-                    // Since we have the current finalized blocks,
-                    // we can mark all the other blocks at the same height as pruned and unpin all together with the reported pruned blocks
-                    ...Object.values(this.#pinnedBlocks)
-                        .filter((b) => finalizedBlockHeights.includes(b.number))
-                        .filter((b) => !finalizedBlockHashes.includes(b.hash))
-                        .map((b) => b.hash),
-                ]);
+                const currentFinalizedBlock = this.findBlock(this.#finalizedHash);
+                this.emit('finalizedBlock', currentFinalizedBlock);
                 // TODO account for operations that haven't received its operationId yet
                 Object.values(this.#handlers).forEach(({ defer, hash, operationId }) => {
-                    if (hashesToUnpin.has(hash)) {
+                    if (prunedBlockHashes.includes(hash)) {
                         defer.reject(new ChainHeadBlockPrunedError());
                         this.stopOperation(operationId).catch(noop);
                         delete this.#handlers[operationId];
                     }
                 });
+                const pinnedHashes = Object.keys(this.#pinnedBlocks);
+                const hashesToUnpin = new Set(prunedBlockHashes.filter((hash) => pinnedHashes.includes(hash)));
                 // Unpin the oldest finalized pinned blocks to maintain the queue size
                 if (this.#finalizedQueue.length > MIN_FINALIZED_QUEUE_SIZE) {
                     const finalizedQueue = this.#finalizedQueue.slice();
@@ -203,6 +193,17 @@ export class ChainHead extends JsonRpcGroup {
                     });
                     this.#finalizedQueue = finalizedQueue;
                 }
+                // Unpin all obsolete blocks with blockNumber < the latest finalized block number
+                // & not a finalized block & is not in use
+                pinnedHashes.forEach((hash) => {
+                    if (this.#blockUsage.usage(hash) > 0)
+                        return;
+                    if (this.#finalizedQueue.includes(hash))
+                        return;
+                    if (this.findBlock(hash).number > currentFinalizedBlock.number)
+                        return;
+                    hashesToUnpin.add(hash);
+                });
                 hashesToUnpin.forEach((hash) => {
                     if (!this.isPinned(hash))
                         return;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dedot/api",
-  "version": "0.9.4",
+  "version": "0.9.5-next.20891389.0+2089138",
   "description": "A delightful JavaScript/TypeScript client for Polkadot & Substrate",
   "author": "Thang X. Vu <thang@dedot.dev>",
   "homepage": "https://dedot.dev",
@@ -13,13 +13,13 @@
   "type": "module",
   "sideEffects": false,
   "dependencies": {
-    "@dedot/codecs": "0.9.4",
-    "@dedot/providers": "0.9.4",
-    "@dedot/runtime-specs": "0.9.4",
-    "@dedot/shape": "0.9.4",
-    "@dedot/storage": "0.9.4",
-    "@dedot/types": "0.9.4",
-    "@dedot/utils": "0.9.4"
+    "@dedot/codecs": "0.9.5-next.20891389.0+2089138",
+    "@dedot/providers": "0.9.5-next.20891389.0+2089138",
+    "@dedot/runtime-specs": "0.9.5-next.20891389.0+2089138",
+    "@dedot/shape": "0.9.5-next.20891389.0+2089138",
+    "@dedot/storage": "0.9.5-next.20891389.0+2089138",
+    "@dedot/types": "0.9.5-next.20891389.0+2089138",
+    "@dedot/utils": "0.9.5-next.20891389.0+2089138"
   },
   "scripts": {
     "build": "tsc --project tsconfig.build.json && tsc --project tsconfig.build.cjs.json",
@@ -48,7 +48,7 @@
     "node": ">=18"
   },
   "license": "Apache-2.0",
-  "gitHead": "1deabbf35425a688e76f1612210bcda3f7bedba3",
+  "gitHead": "208913895242cc9c16cb8bb9bcb880393cb61e6b",
   "module": "./index.js",
   "types": "./index.d.ts"
 }