Implement updated mock-chronik-client, amend unit test syntax

Same issue present in D14859. See D14870 and D14871

Note: these changes cause unit tests to fail in apps/ecash-herald and apps/alias-server. This diff does not change anything about the prod behavior of these apps. These failures are expected until D14869 and D14870, which upgrade chronik-client in these apps, land.

Need to update mock-chronik-client to correct syntax or change existing unit test

looking into why it still passed in D14860

apps/mock-chronik-client does not mock this level of specifity for chronik-client

testing locally and broadcasting to a dev channel

Since, previously, ecash-herald did not have built in websocket redundancy -- it was designed to bail out on app startup if the websocket connection failed (this design is also an artifact of when the app was manually deployed).

lint

lines up according to test plan.

@bot preview-e.cash

not able to arc patch this one, is this based on master?

alt approaches are available with lower complexity

Modify unit tests so that keepAlive = false is tested

cleaning up var name changes to clean up this diff as a reference. will still be abandoned after spllitting up.

carry var name changes throughout all uses

In D14850#333540, @Fabien wrote:

I simply don't understand the problem this is solving

Bump attempt count as 6 resulted in success for only 5 of 7 blocks

will split this into smaller diffs

change _v2 naming convention to Node, layout issues

No more than 30s to wait for peername

fix comment, move wait interval to config

Clarify peername source, add unit test for emoji peername, remove unused param

FYI I get a rebase issue when I attempt to arc patch this diff

remove todo comments, the real todo there is adding features to the in-node chronik instance

or if the API fails

remove unrelated line break, mark peername api mock as generated

remove test script, fix comment

revert axios upgrade, was done troubleshooting

Sample output console.logging on a subscription to blocks and the ifp address

needs self-review, testing

gate clearinterval logic by keepalive param, reset interval to undefined after it is cleared

move url creation to helper function to avoid repeating long one-liner

In D14832#333157, @Fabien wrote:

What is the problem with the actual warning ? It feels more secure to me.

the problem is that the function does too many things. You should split it into logical pieces so you can bail early for coinbase txs without adding another nesting level that makes the code harder to follow.

add test for parsing coinbase tx

If you have a coinbase tx, dont iterate through inputs at all

this fixes the bug but is not the right implementation

remove debug logging

will print xec amount if price is unavailable, e.g.

In D14758#332754, @Fabien wrote:

Rename NNG types that will conflict with ChronikClientNode (even though these types are in separate files, bc they are used by ChronikClient and ChronikClientNode, and both objects are exported, ts will not allow them to have the same name)

Is there no kind of namespace ? Or maybe make it a member of the ChronikClient

name the generated file specific to in-node chronik

In D14758#332748, @Fabien wrote:

This could be split into several diffs, for example file/var renames could be their own diffs. Same for the @generated mark on the proto ts file. This makes it easier to spot the where the changes are.

update: draft up for review.

sharing failoverProxy, support ws iterations for in-node