feat(activation): disable tx history when using external strategy

packages/komodo_defi_sdk/lib/src/activation/protocol_strategies/eth_task_activation_strategy.dart

@@ -1,5 +1,7 @@
 import 'package:komodo_defi_rpc_methods/komodo_defi_rpc_methods.dart';
 import 'package:komodo_defi_sdk/src/activation/_activation.dart';
+import 'package:komodo_defi_sdk/src/transaction_history/strategies/etherscan_transaction_history_strategy.dart'
+    show EtherscanProtocolHelper;
 import 'package:komodo_defi_types/komodo_defi_types.dart';
 
 class EthTaskActivationStrategy extends ProtocolActivationStrategy {
@@ -76,7 +78,8 @@ class EthTaskActivationStrategy extends ProtocolActivationStrategy {
           erc20Tokens:
               children?.map((e) => TokensRequest(ticker: e.id.id)).toList() ??
               [],
-          txHistory: true,
+          txHistory: const EtherscanProtocolHelper()
+              .shouldEnableTransactionHistory(asset),
           privKeyPolicy: privKeyPolicy,
         ),
       );

packages/komodo_defi_sdk/lib/src/activation/protocol_strategies/eth_with_tokens_activation_strategy.dart

@@ -1,5 +1,7 @@
 import 'package:komodo_defi_rpc_methods/komodo_defi_rpc_methods.dart';
 import 'package:komodo_defi_sdk/src/activation/_activation.dart';
+import 'package:komodo_defi_sdk/src/transaction_history/strategies/etherscan_transaction_history_strategy.dart'
+    show EtherscanProtocolHelper;
 import 'package:komodo_defi_types/komodo_defi_types.dart';
 
 class EthWithTokensActivationStrategy extends ProtocolActivationStrategy {
@@ -95,7 +97,8 @@ class EthWithTokensActivationStrategy extends ProtocolActivationStrategy {
           erc20Tokens:
               children?.map((e) => TokensRequest(ticker: e.id.id)).toList() ??
               [],
-          txHistory: true,
+          txHistory: const EtherscanProtocolHelper()
+              .shouldEnableTransactionHistory(asset),
           privKeyPolicy: privKeyPolicy,
         ),
       );

packages/komodo_defi_sdk/lib/src/activation/protocol_strategies/tendermint_activation_strategy.dart

@@ -1,5 +1,7 @@
 import 'package:komodo_defi_rpc_methods/komodo_defi_rpc_methods.dart';
 import 'package:komodo_defi_sdk/src/activation/_activation.dart';
+import 'package:komodo_defi_sdk/src/transaction_history/strategies/etherscan_transaction_history_strategy.dart'
+    show EtherscanProtocolHelper;
 import 'package:komodo_defi_types/komodo_defi_types.dart';
 
 class TendermintActivationStrategy extends ProtocolActivationStrategy {
@@ -73,7 +75,8 @@ class TendermintActivationStrategy extends ProtocolActivationStrategy {
                     .toList() ??
                 [],
             getBalances: true,
-            txHistory: true,
+            txHistory: const EtherscanProtocolHelper()
+                .shouldEnableTransactionHistory(asset),
             privKeyPolicy: privKeyPolicy,
           ),
         );

packages/komodo_defi_sdk/lib/src/transaction_history/strategies/etherscan_transaction_history_strategy.dart

@@ -13,8 +13,8 @@ class EtherscanTransactionStrategy extends TransactionHistoryStrategy {
     required this.pubkeyManager,
     http.Client? httpClient,
     String? baseUrl,
-  })  : _client = httpClient ?? http.Client(),
-        _protocolHelper = EtherscanProtocolHelper(baseUrl: baseUrl);
+  }) : _client = httpClient ?? http.Client(),
+       _protocolHelper = EtherscanProtocolHelper(baseUrl: baseUrl);
 
   final http.Client _client;
   final EtherscanProtocolHelper _protocolHelper;
@@ -23,9 +23,9 @@ class EtherscanTransactionStrategy extends TransactionHistoryStrategy {
 
   @override
   Set<Type> get supportedPaginationModes => {
-        PagePagination,
-        TransactionBasedPagination,
-      };
+    PagePagination,
+    TransactionBasedPagination,
+  };
 
   @override
   bool supportsAsset(Asset asset) => _protocolHelper.supportsProtocol(asset);
@@ -49,7 +49,8 @@ class EtherscanTransactionStrategy extends TransactionHistoryStrategy {
 
     validatePagination(pagination);
 
-    final url = _protocolHelper.getApiUrlForAsset(asset) ??
+    final url =
+        _protocolHelper.getApiUrlForAsset(asset) ??
         (throw UnsupportedError(
           'No API URL found for asset ${asset.id.toJson()}',
         ));
@@ -78,16 +79,17 @@ class EtherscanTransactionStrategy extends TransactionHistoryStrategy {
       // Apply pagination based on type
       final paginatedResults = switch (pagination) {
         final PagePagination p => _applyPagePagination(
-            allTransactions,
-            p.pageNumber,
-            p.itemsPerPage,
-          ),
+          allTransactions,
+          p.pageNumber,
+          p.itemsPerPage,
+        ),
         final TransactionBasedPagination t => _applyTransactionPagination(
-            allTransactions,
-            t.fromId,
-            t.itemCount,
-          ),
-        _ => throw UnsupportedError(
+          allTransactions,
+          t.fromId,
+          t.itemCount,
+        ),
+        _ =>
+          throw UnsupportedError(
             'Unsupported pagination type: ${pagination.runtimeType}',
           ),
       };
@@ -110,9 +112,7 @@ class EtherscanTransactionStrategy extends TransactionHistoryStrategy {
         transactions: paginatedResults.transactions,
       );
     } catch (e) {
-      throw HttpException(
-        'Error fetching transaction history: $e',
-      );
+      throw HttpException('Error fetching transaction history: $e');
     }
   }
 
@@ -152,12 +152,13 @@ class EtherscanTransactionStrategy extends TransactionHistoryStrategy {
             blockHeight: tx.value<int>('block_height'),
             confirmations: tx.value<int>('confirmations'),
             timestamp: tx.value<int>('timestamp'),
-            feeDetails: tx.valueOrNull<JsonMap>('fee_details') != null
-                ? FeeInfo.fromJson(
-                    tx.value<JsonMap>('fee_details')
-                      ..setIfAbsentOrEmpty('type', 'Eth'),
-                  )
-                : null,
+            feeDetails:
+                tx.valueOrNull<JsonMap>('fee_details') != null
+                    ? FeeInfo.fromJson(
+                      tx.value<JsonMap>('fee_details')
+                        ..setIfAbsentOrEmpty('type', 'Eth'),
+                    )
+                    : null,
             coin: coinId,
             internalId: tx.value<String>('internal_id'),
             memo: tx.valueOrNull<String>('memo'),
@@ -166,11 +167,8 @@ class EtherscanTransactionStrategy extends TransactionHistoryStrategy {
         .toList();
   }
 
-  ({
-    List<TransactionInfo> transactions,
-    int skipped,
-    int pageSize,
-  }) _applyPagePagination(
+  ({List<TransactionInfo> transactions, int skipped, int pageSize})
+  _applyPagePagination(
     List<TransactionInfo> transactions,
     int pageNumber,
     int itemsPerPage,
@@ -183,11 +181,8 @@ class EtherscanTransactionStrategy extends TransactionHistoryStrategy {
     );
   }
 
-  ({
-    List<TransactionInfo> transactions,
-    int skipped,
-    int pageSize,
-  }) _applyTransactionPagination(
+  ({List<TransactionInfo> transactions, int skipped, int pageSize})
+  _applyTransactionPagination(
     List<TransactionInfo> transactions,
     String fromId,
     int itemCount,
@@ -211,9 +206,8 @@ class EtherscanTransactionStrategy extends TransactionHistoryStrategy {
 
 /// Helper class for managing Etherscan protocol endpoints and URL construction
 class EtherscanProtocolHelper {
-  const EtherscanProtocolHelper({
-    String? baseUrl,
-  }) : _baseUrl = baseUrl ?? 'https://etherscan-proxy-v2.komodo.earth/api';
+  const EtherscanProtocolHelper({String? baseUrl})
+    : _baseUrl = baseUrl ?? 'https://etherscan-proxy-v2.komodo.earth/api';
 
   final String _baseUrl;
 
@@ -222,6 +216,12 @@ class EtherscanProtocolHelper {
     return asset.protocol is Erc20Protocol && getApiUrlForAsset(asset) != null;
   }
 
+  /// Whether transaction history should also be fetched via mm2.
+  ///
+  /// When Etherscan does not support the provided [asset], transaction history
+  /// must fall back to mm2 RPC calls.
+  bool shouldEnableTransactionHistory(Asset asset) => !supportsProtocol(asset);
+
   /// Constructs the appropriate API URL for a given asset
   Uri? getApiUrlForAsset(Asset asset) {
     if (asset.protocol is! Erc20Protocol) return null;