refactor: attach Tor listeners lazily in open() instead of constructor

Author: sneurlax

lib/wallets/wallet/intermediate/lib_monero_wallet.dart

@@ -53,7 +53,8 @@ abstract class LibMoneroWallet<T extends CryptonoteCurrency>
   int get isarTransactionVersion => 2;
 
   LibMoneroWallet(super.currency, this.compatType) {
-    _attachTorListeners();
+    // Tor listeners are attached lazily in open() so events cannot drive
+    // updateNode() before a wallet has been loaded.
 
     // Potentially dangerous hack. See comments in _startInit()
     _startInit();

lib/wallets/wallet/intermediate/lib_salvium_wallet.dart

@@ -51,7 +51,8 @@ abstract class LibSalviumWallet<T extends CryptonoteCurrency>
   int get isarTransactionVersion => 2;
 
   LibSalviumWallet(super.currency) {
-    _attachTorListeners();
+    // Tor listeners are attached lazily in open() so events cannot drive
+    // updateNode() before a wallet has been loaded.
 
     // Potentially dangerous hack. See comments in _startInit()
     _startInit();

lib/wallets/wallet/intermediate/lib_wownero_wallet.dart

@@ -55,7 +55,8 @@ abstract class LibWowneroWallet<T extends CryptonoteCurrency>
   int get isarTransactionVersion => 2;
 
   LibWowneroWallet(super.currency, this.compatType) {
-    _attachTorListeners();
+    // Tor listeners are attached lazily in open() so events cannot drive
+    // updateNode() before a wallet has been loaded.
 
     // Potentially dangerous hack. See comments in _startInit()
     _startInit();