Jake Kidd

- domain indexer and queue still need forge unit tests working on that on branch `lib-message-tests`, which also includes a fix for: https://hackmd.io/@-HV50kYcRqOjl_7du8m1AA/HJbJOh6Go#Finding-MerklebranchRoot-can-attribute-a-leaf-to-its-root-at-multiple-indices 

- "iff delay block == 0, just skip the whole validation queue" needs an issue opened

- looks like we are doing a redundant verification period check with the mainnet spoke connector (separate from above issue) 

- delay block needs for validation period technically could vary on a chain by chain basis. rollups, for example, don't even need verification period (for one of the directions, either...

two separate issues that are likely *both* out of scope. the first one is technically more pressing, but both are harmless sec-wise as far as I can tell, only are...

- inconsistent use of interface IERC20 in bridgefacet: noting this bc we literally use SafeERC20 for IERC20 under libraries in this contract

this should just be the offchain component, related to #1729 ~~should use the merkletreelib.js library to generate proofs~~ EDIT: should NOT use the merkletreejs library, see edited og comment