Use imported db_(de)serialize, add comment to clarify no-side-effect test

Use 1-2-4-8 chain as suggested

Split diff into two

magnifique

Fix comment in apply_merge_spent_by

Rebase from master

remove unneeded line

remove trailing whitespace

Turn failed merges into no-ops and handle them after updating the DB, if any

rebase on master (to get latest Rust 1.72.0)

rebase on master

Remove DEFAULT_SCRIPT_NUM_TXS_CACHE_SIZE const

Remove LRU cache, empirically, a disabled cache is faster, and the bigger the cache, the longer insert takes:

rebase from master

Write full stats to resync_stats.txt, round seconds to 3 digits, add missing n_total for GroupUtxoWriter

actually use -chronikscriptnumtxscache

In D14269#322376, @emack wrote:

@tobias_ruck - this can technically be done but would it be advisable based on how you designed this client?

factor coin finding code into getSpentCoins

add InitError when using -chronikallowpause on mainnet, fix some comments

use std::move when making the shared_ptr

rebase on D14386, fix raised issues

fix fmt

Use std::shared_ptr

Can you update the title + commit message? The deeper telos now is to move the version to one place, not to set them to the same version (which is a nice side effect now). Otherwise good work

Lovely

@bot preview-explorer

with an invalid ws url it doesn't throw any errors, so I couldn't catch it

This doesn't have to be in the client library though, it can be in the code for the airdrop.

Concur with Fabien on the comments. Note that the goal of this change is quite ambitious (which is why I haven't done it myself), so this might still take a while to get right.

Much better :) Some more changes, but this is shaping into something nice.

For reference, here's some points I raised with Joey and Fabien (redacted somewhat):

Use MakeTransactionRef, fix comment spelling

also remove the unused Config parameter

rerun tests