Feedback about diagrams

[rootulp]

Context

I have some miscelaneous feedback after reading the learn section. Feel free to close if not helpful.

Feedback

• The architecture diagram here conflicts with the sequence diagram. The architecture diagram looks incorrect because the ISM doesn’t invoke ISM on the recipient. Instead the mailbox does that.
• In sequence diagram how does the relayer know which ISMs to invoke moduleType() on? Why does it need to do that given it calls process() on the mailbox without the ISM type. Maybe it’s just needed for metadata to process messages? The relayer likely needs a hard-coded list of ISM addresses deployed on each chain if it’s going to query it.
• In post dispatch how does the postDispatch hook revert if insufficient fees? Fees aren’t a parameter to the postDispatch invocation.