went ahead and removed comments on self-explanatory code

Author: aydencook03

electrum/gui/qml/components/wizard/WCHaveMasterKey.qml

@@ -27,9 +27,6 @@ WizardComponent {
             wizard_data['multisig_cosigner_data'][cosigner.toString()]['master_key'] = key
         } else {
             wizard_data['master_key'] = key
-            // Pass key-origin fields through to create_storage() so the
-            // keystore is stored with the correct derivation prefix and master
-            // fingerprint.  Both fields are optional; empty strings are ignored.
             wizard_data['key_origin_derivation'] = derivation_tf.text.trim()
             wizard_data['key_origin_fingerprint'] = fingerprint_tf.text.trim().toLowerCase()
         }
@@ -65,12 +62,9 @@ WizardComponent {
         return valid = true
     }
 
-    // Validate the key-origin fields and update the inline error label.
-    // Returns true when everything is OK (or when the section is not shown).
     function validateKeyOrigin() {
         if (cosigner || wizard_data['wallet_type'] === 'multisig')
             return true
-        // Section is hidden for depth <= 1: no validation needed.
         if (bitcoin.masterKeyDepth(masterkey_ta.text.trim()) <= 1)
             return true
         var msg = bitcoin.verifyKeyOriginInfo(
@@ -82,8 +76,6 @@ WizardComponent {
         return msg === ''
     }
 
-    // Revalidate both the master key and the key-origin fields together,
-    // then commit the current values to wizard_data.
     function revalidate() {
         var keyOk = verifyMasterKey(masterkey_ta.text)
         var originOk = validateKeyOrigin()
@@ -222,30 +214,8 @@ WizardComponent {
             }
         }
 
-        // ── Key-origin section ─────────────────────────────────────────────
-        // Only shown for standard single-sig imports (not multisig/cosigner)
-        // where the key depth is > 1.
-        //
-        // Both fields are optional.
-        // 
-        // Hardware wallets export xpubs at derivation depth > 1 (e.g.
-        // Coldcard exports a zpub at m/84'/0'/0', depth=3).  Electrum
-        // cannot infer the master fingerprint or the full derivation path
-        // from such an xpub alone, so without these values the generated
-        // PSBTs will be missing key-origin metadata and hardware signers
-        // that verify it will refuse to sign the transaction.
-        // 
-        // At depth 0 the node IS the root -> fingerprint is computed locally.
-        // At depth 1 the node encodes the master fingerprint directly in its
-        // parent_fingerprint bytes, so Electrum can infer it automatically.
-        // At depth > 1 the xpub only carries the *immediate parent's*
-        // fingerprint (e.g. Coldcard zpub at m/84'/0'/0' has depth 3), so
-        // Electrum cannot recover the master fingerprint or full path on its
-        // own -> the user must supply them.
         ColumnLayout {
             id: keyOriginSection
-            // masterKeyDepth returns -1 for invalid/empty input, so the
-            // section stays hidden until a valid key of depth > 1 is entered.
             visible: !cosigner
                      && wizard_data['wallet_type'] !== 'multisig'
                      && bitcoin.masterKeyDepth(masterkey_ta.text.trim()) > 1

electrum/gui/qml/qebitcoin.py

@@ -100,17 +100,6 @@ def verifyDerivationPath(self, path):
 
     @pyqtSlot(str, str, str, result=str)
     def verifyKeyOriginInfo(self, xpub: str, derivation: str, fingerprint: str) -> str:
-        """Validate optional key-origin fields against the given xpub.
-
-        Both fields are optional, passing empty strings is valid.  When
-        non-empty each field is checked for format, and if the xpub is also
-        provided the combination is checked for internal consistency
-        (derivation path depth must match the xpub depth, last child number
-        must match, etc.).
-
-        Returns an empty string if everything is valid, or a translated
-        human-readable error message describing the problem.
-        """
         derivation = derivation.strip()
         fingerprint = fingerprint.strip().lower()
         if not derivation and not fingerprint:
@@ -136,17 +125,6 @@ def verifyKeyOriginInfo(self, xpub: str, derivation: str, fingerprint: str) -> s
 
     @pyqtSlot(str, result=int)
     def masterKeyDepth(self, key: str) -> int:
-        """Return the BIP32 depth of *key* (xpub or xprv), or -1 if invalid.
-
-        Depth 0: key is the root; fingerprint computed locally.
-        Depth 1: parent_fingerprint in the xpub IS the master fingerprint;
-                 Electrum can recover it automatically.
-        Depth > 1: only the immediate parent fingerprint is encoded (e.g.
-                   a Coldcard zpub at m/84'/0'/0' has depth 3); neither the
-                   master fingerprint nor the full path can be recovered,
-                   the user must supply them.
-        Returns -1 for empty or invalid input.
-        """
         try:
             return BIP32Node.from_xkey(key.strip()).depth
         except Exception:

electrum/wizard.py

@@ -749,13 +749,6 @@ def create_storage(self, path: str, data: dict):
                 else:
                     if t1 not in ['standard', 'p2wpkh', 'p2wpkh-p2sh']:
                         raise UserFacingException(_('Wrong key type {}').format(t1))
-                # Apply optional key-origin metadata supplied by the user.
-                # A raw xpub exported from a hardware wallet (e.g. Coldcard at
-                # m/84'/0'/0', depth=3) cannot encode the master fingerprint or
-                # the full derivation path by itself.  Without these values the
-                # keystore generates PSBTs whose bip32_derivations use only the
-                # xpub's own intermediate fingerprint, which hardware signers
-                # that verify key-origin (Coldcard fw >= 3.2.1) will reject.
                 _der = data.get('key_origin_derivation', '').strip()
                 _fp  = data.get('key_origin_fingerprint', '').strip().lower()
                 if _der or _fp: