Fix derivable-descriptor detection during wallet sync

This fix is for electrum-based blockchains.

Previously, it was assumed that containing > 1 spks in database means
the descriptor is derivable, however each database tracks two
descriptors so it should really check > 2.

.vscode/settings.json

@@ -0,0 +1,9 @@
+{
+    "rust-analyzer.cargo.noDefaultFeatures": true,
+    "rust-analyzer.cargo.features": [
+        "test-blockchains",
+        "test-rpc",
+        // "compact_filters",
+        "verify",
+    ]
+}

src/blockchain/script_sync.rs

@@ -131,8 +131,9 @@ impl<'a, D: BatchDatabase> ScriptReq<'a, D> {
         let current_gap = self.script_index - last;
 
         // this is a hack to check whether the scripts are coming from a derivable descriptor
-        // we assume for non-derivable descriptors, the initial script count is always 1
-        let is_derivable = self.initial_scripts_needed > 1;
+        // we assume for non-derivable descriptors, the initial script count is always 1 and
+        // as we are working with (at-most) 2 descriptors, we use 2 to determine derivability
+        let is_derivable = self.initial_scripts_needed > 2;
 
         debug!(
             "sync: last={}, remaining={}, diff={}, stop_gap={}",