@stellar-expert/tx-meta-effects-parser 5.5.4 → 5.6.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stellar-expert/tx-meta-effects-parser",
-  "version": "5.5.4",
+  "version": "5.6.1",
   "description": "Low-level effects parser for Stellar transaction results and meta XDR",
   "main": "src/index.js",
   "scripts": {
@@ -9,7 +9,7 @@
   "author": "team@stellar.expert",
   "license": "MIT",
   "peerDependencies": {
-    "@stellar/stellar-base": "^12.0.1"
+    "@stellar/stellar-base": "^12.1.0"
   },
   "devDependencies": {
     "@babel/core": "^7.22.9",

package/src/aggregation/events-analyzer.js

@@ -155,30 +155,27 @@ class EventsAnalyzer {
                         classicAsset = null  //not an SAC event
                     }
                 }
-                if (classicAsset && (classicAsset.includes(from) || classicAsset.includes(to))) { //SAC transfer by asset issuer
-                    if (classicAsset.includes(from)) {
-                        this.effectsAnalyzer.mint(contract, amount)
-                        this.effectsAnalyzer.credit(amount, isContractAddress(to) ? contract : classicAsset, to)
-                    }
-                    if (classicAsset.includes(to)) {
-                        this.effectsAnalyzer.debit(amount, isContractAddress(from) ? contract : classicAsset, from)
-                        this.effectsAnalyzer.burn(contract, amount)
+                if (classicAsset) {
+                    if (classicAsset.includes(from)) {  //SAC transfer by asset issuer
+                        this.effectsAnalyzer.mint(classicAsset, amount)
+                        this.effectsAnalyzer.credit(amount, classicAsset, to)
+                        return
                     }
-                } else { //other cases
-                    if (classicAsset && !isContractAddress(from)) { //classic asset bridged to Soroban
+                    if (classicAsset.includes(to)) {  //SAC transfer by asset issuer
+                        this.effectsAnalyzer.debit(amount, classicAsset, from)
                         this.effectsAnalyzer.burn(classicAsset, amount)
-                        this.effectsAnalyzer.mint(contract, amount)
-                    } else {
-                        this.effectsAnalyzer.debit(amount, contract, from)
+                        return
                     }
-                    if (classicAsset && !isContractAddress(to)) { //classic asset bridged from Soroban
-                        this.effectsAnalyzer.burn(contract, amount)
-                        this.effectsAnalyzer.mint(classicAsset, amount)
-                    } else {
-                        this.effectsAnalyzer.credit(amount, contract, to)
+                    if (isContractAddress(from)) {
+                        this.effectsAnalyzer.debit(amount, classicAsset, from)
                     }
+                    if (isContractAddress(to)) {
+                        this.effectsAnalyzer.credit(amount, classicAsset, to)
+                    }
+                } else { //other cases
+                    this.effectsAnalyzer.debit(amount, this.effectsAnalyzer.resolveAsset(contract), from)
+                    this.effectsAnalyzer.credit(amount, this.effectsAnalyzer.resolveAsset(contract), to)
                 }
-
             }
                 break
             case 'mint': {
@@ -188,15 +185,14 @@ class EventsAnalyzer {
                 const amount = processEventBodyValue(body.data())
                 if (!this.matchInvocationEffect(e => e.function === 'mint' && matchArrays([to, amount], e.args)))
                     return
-                this.effectsAnalyzer.addEffect({
-                    type: effectTypes.assetMinted,
-                    asset: contract,
-                    amount
-                })
-                this.effectsAnalyzer.credit(amount, contract, to)
                 if (topics.length > 3) {
                     mapSacContract(this.effectsAnalyzer, contract, xdrParseAsset(xdrParseScVal(topics[3])))
                 }
+                const asset = this.effectsAnalyzer.resolveAsset(contract)
+                this.effectsAnalyzer.mint(asset, amount)
+                if (isContractAddress(asset) || isContractAddress(to)) {
+                    this.effectsAnalyzer.credit(amount, asset, to)
+                }
             }
                 break
             case 'burn': {
@@ -209,12 +205,14 @@ class EventsAnalyzer {
                     (e.function === 'burn_from' && matchArrays([undefined, from, amount], e.args))
                 ))
                     return
-
-                this.effectsAnalyzer.debit(amount, contract, from)
-                this.effectsAnalyzer.burn(contract, amount)
                 if (topics.length > 2) {
                     mapSacContract(this.effectsAnalyzer, contract, xdrParseAsset(xdrParseScVal(topics[2])))
                 }
+                const asset = this.effectsAnalyzer.resolveAsset(contract)
+                if (isContractAddress(asset) || isContractAddress(from)) {
+                    this.effectsAnalyzer.debit(amount, asset, from)
+                }
+                this.effectsAnalyzer.burn(asset, amount)
             }
                 break
             case 'clawback': {
@@ -224,11 +222,14 @@ class EventsAnalyzer {
                 const amount = processEventBodyValue(body.data())
                 if (!this.matchInvocationEffect(e => e.function === 'clawback' && matchArrays([from, amount], e.args)))
                     return
-                this.effectsAnalyzer.debit(amount, contract, from)
-                this.effectsAnalyzer.burn(contract, amount)
                 if (topics.length > 3) {
                     mapSacContract(this.effectsAnalyzer, contract, xdrParseAsset(xdrParseScVal(topics[3])))
                 }
+                const asset = this.effectsAnalyzer.resolveAsset(contract)
+                if (isContractAddress(asset)) {
+                    this.effectsAnalyzer.debit(amount, asset, from)
+                }
+                this.effectsAnalyzer.burn(asset, amount)
             }
                 break
             case 'set_admin': {
@@ -238,10 +239,10 @@ class EventsAnalyzer {
                 const newAdmin = processEventBodyValue(body.data())
                 if (!this.matchInvocationEffect(e => e.function === 'set_admin' && matchArrays([currentAdmin, newAdmin], [this.effectsAnalyzer.source, e.args])))
                     return
-                this.effectsAnalyzer.setAdmin(contract, newAdmin)
                 if (topics.length > 2) {
                     mapSacContract(this.effectsAnalyzer, contract, xdrParseAsset(xdrParseScVal(topics[2])))
                 }
+                this.effectsAnalyzer.setAdmin(contract, newAdmin)
             }
                 break
             /*case 'approve': { //TODO: think about processing this effect

package/src/aggregation/sac-contract-mapper.js

@@ -15,14 +15,22 @@ function mapSacContract(effectsAnalyzer, contractAddress, classicAsset) {
     if (!classicAsset)
         return false
     const {network, sacMap} = effectsAnalyzer
+    if (!sacMap)
+        return false
+    const prevMapping = sacMap[contractAddress]
+    if (prevMapping) {
+        return prevMapping === classicAsset
+    }
     //try to load from cache first
-    const fromCache = sacCache.get(contractAddress + network)
+    const fromCache = sacCache.get(classicAsset + network)
     if (!fromCache) {
         const encodedContract = contractIdFromAsset(toStellarAsset(classicAsset), network)
-        sacCache.set(encodedContract + network, classicAsset)
-        if (encodedContract !== contractAddress)
+        sacCache.set(classicAsset + network, contractAddress)
+        if (contractAddress === undefined) {
+            contractAddress = encodedContract
+        } else if (encodedContract !== contractAddress)
             return false
-    } else if (classicAsset !== fromCache)
+    } else if (contractAddress !== fromCache)
         return false //check whether validated contract from cache matches the asset
     if (sacMap) {
         sacMap[contractAddress] = classicAsset

package/src/effects-analyzer.js

@@ -7,6 +7,7 @@ const {contractIdFromPreimage} = require('./parser/contract-preimage-encoder')
 const EventsAnalyzer = require('./aggregation/events-analyzer')
 const AssetSupplyAnalyzer = require('./aggregation/asset-supply-analyzer')
 const {UnexpectedTxMetaChangeError, TxMetaEffectParserError} = require('./errors')
+const {mapSacContract} = require('./aggregation/sac-contract-mapper')
 
 class EffectsAnalyzer {
     constructor({operation, meta, result, network, events, diagnosticEvents, mapSac, processSystemEvents}) {
@@ -571,6 +572,9 @@ class EffectsAnalyzer {
     }
 
     processBalanceChange(account, asset, beforeBalance, afterBalance) {
+        if (this.isContractCall) { //map contract=>asset proactively
+            mapSacContract(this, undefined, asset)
+        }
         const balanceChange = BigInt(afterBalance) - BigInt(beforeBalance)
         if (balanceChange < 0n) {
             this.debit((-balanceChange).toString(), asset, account, afterBalance)
@@ -922,6 +926,13 @@ class EffectsAnalyzer {
         }
         return null
     }
+
+    resolveAsset(assetOrContract) {
+        if (!assetOrContract.startsWith('C') || !this.sacMap)
+            return assetOrContract
+        //try to resolve using SAC map
+        return this.sacMap[assetOrContract] || assetOrContract
+    }
 }
 
 /**