@ocap/tx-protocols 1.23.0 → 1.23.1

package/lib/protocols/token-factory/burn.js

@@ -3,6 +3,7 @@ const { Joi, schemas } = require('@arcblock/validator');
 const { BN, fromUnitToToken } = require('@ocap/util');
 const { Runner, pipes } = require('@ocap/tx-pipeline');
 const { account, tokenFactory, token } = require('@ocap/state');
+const pick = require('lodash/pick');
 
 const EnsureTxGas = require('../../pipes/ensure-gas');
 const EnsureTxCost = require('../../pipes/ensure-cost');
@@ -252,7 +253,7 @@ runner.use(
     }
     // Ensure the sender exists in accountUpdates, because even if there is no balance change, we need to update his nonce
     if (!accountUpdates[tx.from]) {
-      accountUpdates[tx.from] = {};
+      accountUpdates[tx.from] = senderState || {};
     }
 
     // update owner
@@ -281,12 +282,14 @@ runner.use(
           // We can use updateOrCreate here because the owner and signer have already been validated to exist earlier,
           // the sender and receiver are allowed to be created in the transaction.
           const state = [senderState, receiverState, ownerState, ...signerStates].find((x) => x?.address === address);
-          const senderUpdated = address === tx.from ? { nonce: tx.nonce, pk: tx.pk } : {};
-          return statedb.account.updateOrCreate(
-            state,
-            account.updateOrCreate(state, Object.assign({ address }, senderUpdated, updates), context),
-            context
-          );
+
+          // Should only update nonce and pk for sender and new accounts
+          const actualUpdates =
+            address === tx.from || !state
+              ? { ...updates, address, nonce: tx.nonce, pk: tx.pk }
+              : pick(updates, ['tokens']);
+
+          return statedb.account.updateOrCreate(state, account.updateOrCreate(state, actualUpdates, context), context);
         })
       ),
 

package/lib/protocols/token-factory/mint.js

@@ -3,6 +3,7 @@ const { Joi, schemas } = require('@arcblock/validator');
 const { BN, fromUnitToToken } = require('@ocap/util');
 const { Runner, pipes } = require('@ocap/tx-pipeline');
 const { account, tokenFactory, token } = require('@ocap/state');
+const pick = require('lodash/pick');
 
 const EnsureTxGas = require('../../pipes/ensure-gas');
 const EnsureTxCost = require('../../pipes/ensure-cost');
@@ -237,7 +238,7 @@ runner.use(
     }
     // Ensure the sender exists in accountUpdates, because even if there is no balance change, we need to update his nonce
     if (!accountUpdates[tx.from]) {
-      accountUpdates[tx.from] = {};
+      accountUpdates[tx.from] = senderState || {};
     }
 
     // update owner
@@ -266,12 +267,14 @@ runner.use(
           // We can use updateOrCreate here because the owner and signer have already been validated to exist earlier,
           // the sender and receiver are allowed to be created in the transaction.
           const state = [senderState, receiverState, ownerState, ...signerStates].find((x) => x?.address === address);
-          const senderUpdated = address === tx.from ? { nonce: tx.nonce, pk: tx.pk } : {};
-          return statedb.account.updateOrCreate(
-            state,
-            account.updateOrCreate(state, Object.assign({ address }, senderUpdated, updates), context),
-            context
-          );
+
+          // Should only update nonce and pk for sender and new accounts
+          const actualUpdates =
+            address === tx.from || !state
+              ? { ...updates, address, nonce: tx.nonce, pk: tx.pk }
+              : pick(updates, ['tokens']);
+
+          return statedb.account.updateOrCreate(state, account.updateOrCreate(state, actualUpdates, context), context);
         })
       ),
 

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "1.23.0",
+  "version": "1.23.1",
   "description": "Predefined tx pipeline sets to execute certain type of transactions",
   "main": "lib/index.js",
   "files": [
@@ -18,19 +18,19 @@
     "empty-value": "^1.0.1",
     "lodash": "^4.17.21",
     "url-join": "^4.0.1",
-    "@arcblock/did": "1.23.0",
-    "@arcblock/did-util": "1.23.0",
-    "@arcblock/jwt": "1.23.0",
-    "@arcblock/validator": "1.23.0",
-    "@ocap/asset": "1.23.0",
-    "@ocap/client": "1.23.0",
-    "@ocap/mcrypto": "1.23.0",
-    "@ocap/merkle-tree": "1.23.0",
-    "@ocap/message": "1.23.0",
-    "@ocap/state": "1.23.0",
-    "@ocap/tx-pipeline": "1.23.0",
-    "@ocap/util": "1.23.0",
-    "@ocap/wallet": "1.23.0"
+    "@arcblock/did": "1.23.1",
+    "@arcblock/did-util": "1.23.1",
+    "@arcblock/jwt": "1.23.1",
+    "@arcblock/validator": "1.23.1",
+    "@ocap/asset": "1.23.1",
+    "@ocap/client": "1.23.1",
+    "@ocap/mcrypto": "1.23.1",
+    "@ocap/merkle-tree": "1.23.1",
+    "@ocap/message": "1.23.1",
+    "@ocap/state": "1.23.1",
+    "@ocap/tx-pipeline": "1.23.1",
+    "@ocap/util": "1.23.1",
+    "@ocap/wallet": "1.23.1"
   },
   "resolutions": {
     "bn.js": "5.2.1",
@@ -39,8 +39,8 @@
   "devDependencies": {
     "jest": "^29.7.0",
     "start-server-and-test": "^1.14.0",
-    "@ocap/e2e-test": "1.23.0",
-    "@ocap/statedb-memory": "1.23.0"
+    "@ocap/e2e-test": "1.23.1",
+    "@ocap/statedb-memory": "1.23.1"
   },
   "scripts": {
     "lint": "eslint tests lib",