We could do some hacking of the docker file so we don't have to clear the cache when not needed, I will look in to it

CI has been updated to cache bust

ci has been updated to cache-bust

use a match no that the Err branch is a single liner

leak, complete error messages

rebase, remove debug logs

rebase, README improvements, preserve txBuilder.test.ts dependency groupings

rebase to restart CI. I can't figure out why rpc_psbt.py failed, but it is clearly unrelated (nodes[2] never received an inv for the transaction)

investigating build failure

pull out cashaddr logic from ecash-lib, remove dependencies from ecashaddrjs, major version bump for ecashaddrjs, implement ecashaddrjs 2.0.0 in all monorepo libraries, modify toLegacyAddress so it is a no-op if called with legacy addr

I updated the diff a bit too fast, so some of these recent errors are from previous versions of the diff.

improve for loop as per clippy's advice

remove unneeded rehash and wrong comment

In D17269#394989, @Fabien wrote:

What is the benefit vs a no-op ? Imagine I'm a mining pool and I want to convert all my miner addresses to legacy (real world use case) even if they are using BCH or XEC addresses. I would like to just use to_legacy() and not have to first check it's not legacy format already

In D17269#394758, @bytesofman wrote:

change toLegacyAddress behavior to throw if called with legacy address