minimal-xec-wallet 1.0.2 → 1.0.4

package/lib/consolidate-utxos.js

@@ -72,29 +72,37 @@ class ConsolidateUtxos {
 
   async analyzeUtxos (options = {}) {
     try {
-      const spendableUtxos = this.utxos.getSpendableXecUtxos()
+      const allUtxos = this.utxos.getSpendableXecUtxos()
 
-      if (spendableUtxos.length < this.minUtxosForConsolidation) {
+      // CRITICAL: Filter out token UTXOs to prevent token burning
+      const pureXecUtxos = allUtxos.filter(utxo => !utxo.token)
+      const tokenUtxos = allUtxos.filter(utxo => utxo.token)
+
+      if (pureXecUtxos.length < this.minUtxosForConsolidation) {
         return {
           shouldConsolidate: false,
-          reason: `Not enough UTXOs for consolidation (${spendableUtxos.length} < ${this.minUtxosForConsolidation})`,
-          totalUtxos: spendableUtxos.length,
-          totalValue: this._calculateTotalValue(spendableUtxos)
+          reason: `Not enough pure XEC UTXOs for consolidation (${pureXecUtxos.length} < ${this.minUtxosForConsolidation})`,
+          totalUtxos: pureXecUtxos.length,
+          totalValue: this._calculateTotalValue(pureXecUtxos),
+          tokenUtxos: tokenUtxos.length,
+          tokenUtxosSkipped: tokenUtxos.length > 0
         }
       }
 
       // Filter UTXOs that should be consolidated (smaller ones first)
-      const utxosToConsolidate = spendableUtxos
-        .filter(utxo => utxo.value <= options.consolidationThreshold)
-        .sort((a, b) => a.value - b.value) // Sort by value ascending
+      const utxosToConsolidate = pureXecUtxos
+        .filter(utxo => this._getUtxoValue(utxo) <= options.consolidationThreshold)
+        .sort((a, b) => this._getUtxoValue(a) - this._getUtxoValue(b)) // Sort by value ascending
 
       if (utxosToConsolidate.length < this.minUtxosForConsolidation) {
         return {
           shouldConsolidate: false,
-          reason: `Not enough small UTXOs to consolidate (${utxosToConsolidate.length} below ${options.consolidationThreshold} satoshis)`,
-          totalUtxos: spendableUtxos.length,
-          totalValue: this._calculateTotalValue(spendableUtxos),
-          smallUtxos: utxosToConsolidate.length
+          reason: `Not enough small pure XEC UTXOs to consolidate (${utxosToConsolidate.length} below ${options.consolidationThreshold} satoshis)`,
+          totalUtxos: pureXecUtxos.length,
+          totalValue: this._calculateTotalValue(pureXecUtxos),
+          smallUtxos: utxosToConsolidate.length,
+          tokenUtxos: tokenUtxos.length,
+          tokenUtxosSkipped: tokenUtxos.length > 0
         }
       }
 
@@ -117,6 +125,8 @@ class ConsolidateUtxos {
         totalValue: this._calculateTotalValue(utxosToConsolidate),
         consolidationFee,
         potentialSavings: totalSavings,
+        tokenUtxos: tokenUtxos.length,
+        tokenUtxosSkipped: tokenUtxos.length > 0,
         consolidationPlans
       }
     } catch (err) {
@@ -167,6 +177,12 @@ class ConsolidateUtxos {
 
       for (const plan of consolidationPlans) {
         try {
+          // CRITICAL SAFETY CHECK: Ensure no token UTXOs are being consolidated
+          const tokenUtxosInPlan = plan.inputUtxos.filter(utxo => utxo.token)
+          if (tokenUtxosInPlan.length > 0) {
+            throw new Error(`SAFETY ABORT: Plan contains ${tokenUtxosInPlan.length} token UTXOs. Consolidation would burn tokens!`)
+          }
+
           // Create consolidation transaction - send all value to same address
           const outputs = [{
             address: this.wallet.walletInfo.xecAddress,
@@ -251,7 +267,17 @@ class ConsolidateUtxos {
   // Helper methods
 
   _calculateTotalValue (utxos) {
-    return utxos.reduce((total, utxo) => total + utxo.value, 0)
+    return utxos.reduce((total, utxo) => total + this._getUtxoValue(utxo), 0)
+  }
+
+  _getUtxoValue (utxo) {
+    if (utxo.sats !== undefined) {
+      return typeof utxo.sats === 'bigint' ? Number(utxo.sats) : parseInt(utxo.sats)
+    }
+    if (utxo.value !== undefined) {
+      return typeof utxo.value === 'bigint' ? Number(utxo.value) : parseInt(utxo.value)
+    }
+    return 0
   }
 
   _calculateConsolidationFee (numInputs, numOutputs, satsPerByte) {
@@ -283,21 +309,30 @@ class ConsolidateUtxos {
 
   getUtxoDistribution () {
     try {
-      const utxos = this.utxos.getSpendableXecUtxos()
+      const allUtxos = this.utxos.getSpendableXecUtxos()
+
+      // Separate pure XEC from token UTXOs
+      const pureXecUtxos = allUtxos.filter(utxo => !utxo.token)
+      const tokenUtxos = allUtxos.filter(utxo => utxo.token)
+
       const distribution = {
         dust: 0, // < 1000 sats
         small: 0, // 1000 - 10000 sats
         medium: 0, // 10000 - 100000 sats
         large: 0, // > 100000 sats
-        total: utxos.length
+        total: pureXecUtxos.length,
+        tokenUtxos: tokenUtxos.length,
+        tokenUtxosSkipped: tokenUtxos.length > 0
       }
 
-      for (const utxo of utxos) {
-        if (utxo.value < 1000) {
+      // Only analyze pure XEC UTXOs for consolidation
+      for (const utxo of pureXecUtxos) {
+        const value = this._getUtxoValue(utxo)
+        if (value < 1000) {
           distribution.dust++
-        } else if (utxo.value < 10000) {
+        } else if (value < 10000) {
           distribution.small++
-        } else if (utxo.value < 100000) {
+        } else if (value < 100000) {
           distribution.medium++
         } else {
           distribution.large++
@@ -312,22 +347,33 @@ class ConsolidateUtxos {
 
   estimateOptimizationSavings () {
     try {
-      const utxos = this.utxos.getSpendableXecUtxos()
+      const allUtxos = this.utxos.getSpendableXecUtxos()
 
-      if (utxos.length < 2) {
-        return { savings: 0, reason: 'No optimization needed' }
+      // Only analyze pure XEC UTXOs (tokens are preserved separately)
+      const pureXecUtxos = allUtxos.filter(utxo => !utxo.token)
+      const tokenUtxos = allUtxos.filter(utxo => utxo.token)
+
+      if (pureXecUtxos.length < 2) {
+        return {
+          savings: 0,
+          reason: 'No optimization needed for pure XEC UTXOs',
+          currentUtxos: pureXecUtxos.length,
+          tokenUtxos: tokenUtxos.length
+        }
       }
 
-      const currentFee = this._estimateCurrentSpendingFee(utxos, this.defaultSatsPerByte)
-      const optimalUtxoCount = Math.max(1, Math.ceil(utxos.length / 50)) // Optimal: ~50 UTXOs max
+      const currentFee = this._estimateCurrentSpendingFee(pureXecUtxos, this.defaultSatsPerByte)
+      const optimalUtxoCount = Math.max(1, Math.ceil(pureXecUtxos.length / 50)) // Optimal: ~50 UTXOs max
       const optimizedFee = this._estimateFutureSpendingFee(optimalUtxoCount, this.defaultSatsPerByte)
 
       return {
         savings: currentFee - optimizedFee,
-        currentUtxos: utxos.length,
+        currentUtxos: pureXecUtxos.length,
         optimalUtxos: optimalUtxoCount,
         currentEstimatedFee: currentFee,
-        optimizedEstimatedFee: optimizedFee
+        optimizedEstimatedFee: optimizedFee,
+        tokenUtxos: tokenUtxos.length,
+        tokenUtxosPreserved: tokenUtxos.length
       }
     } catch (err) {
       throw new Error(`Optimization savings estimation failed: ${err.message}`)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "minimal-xec-wallet",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "description": "A minimalist eCash (XEC) wallet npm library, for use in web apps. Supports eTokens.",
   "main": "./index.js",
   "module": "./dist/minimal-xec-wallet.min.js",