squash with core#22992

Maybe core#22992

Finished off the testing for block functionality, improved error handling and removed some questionable comments xd

more clang-tidy issues (argument names)

Let's see if the build-debug segfault is happening consistently

clang-tidy warnings

@bot build-asan

keep the first modification to close WS

• As @emack mentioned, this is 2 diffs. Please make one diff to add the close, and another to add the delay
• npm test does not actually test the behavior described in this diff. If it is impractical to add a unit or integration test, the test plan should include a description of how to repeat the issue before this change, and how to demonstrate that the issue is resolved after this change. Then npm test is demonstrating that existing behavior is preserved with this patch.
• patch version bump npm version patch (can be for just the 2nd of these 2 diffs, or can be a separate npm version patch for each diff), update changelog section of README describing the change

rebase so i can do arc land --revision D17931 (apparently to force a revision the commit must match )

nvm

prop name edit

remove accidental character in doxygen doc

nit, the offer usually refers to the actual AgoraOffer, but in this case we want to specify that we mean a count

url in RPC doc

ah sure, added

make the column logic more flexible for more whitelisted tokens

Extra comments and trailing whitespace removed

update syncing -> synced also in test

forgot to specify --update D17931

fix placeholder text for relmease notes

fix release notes placeholder

use bitcoinabc.org url for doc, don't set a short timeout in wait_until_stopped remove release notes, rename cs -> chainstate