format

Author: barnjamin

DESIGN.md

@@ -44,5 +44,3 @@ Several interfaces should be available for each platform supported.
 A Protocol (fka `Module`) is a specific application, it provides a set of methods that can be called to accomplish some action (e.g. `TokenBridge` allows `transfer`/`redeem`/`getWrappedAsset`, etc...)
 
 To allow platform agnostic access to Protocols, each Platform that provides the protocol should have its own implementation. 
-
-

notes.md

@@ -10,16 +10,18 @@ The [token registry](./tokenRegistry/) directory is unused except for importing
 
 It is still possible to screw up a transfer (blackhole funds) if the higher level constructs are not used and care is not taken to produce a transfer that is acceptable by the destination chain. I have no suggestions for this that would prevent the platform-specific-oddities from leaking into packages they have no business in besides "better documentation".
 
-Invalid ATA receiver:
-https://wormholescan.io/#/tx/0x1605bd06e15d46398c061bd2fc24e65c2b580d07c3e7a4ca9a0643bf91d16a7c
-https://wormholescan.io/#/tx/DnEXtm2NdLhT5RszHc7nR7LPNATzGSP3kmbC2GQkAJyK
+Examples: 
 
-An [issue](https://github.com/wormhole-foundation/wormhole/issues/3992) has been filed to reduce the danger for this one.
+- Invalid ATA receiver for Solana destined transfer. An [issue](https://github.com/wormhole-foundation/wormhole/issues/3992) has been filed to reduce the danger for this one.
 
-Incorrect contract address in payload:
-https://wormholescan.io/#/tx/2g4nn6fWZCkphxeMGfqjzViZYo3XAYarYTSvTcxYY95JAtZvwVd7MKY6RWHhhHd8oBzeFWjuTkXNo4tdVVTwBWfo
+    - https://wormholescan.io/#/tx/0x1605bd06e15d46398c061bd2fc24e65c2b580d07c3e7a4ca9a0643bf91d16a7c
+    - https://wormholescan.io/#/tx/DnEXtm2NdLhT5RszHc7nR7LPNATzGSP3kmbC2GQkAJyK
+
+
+- Incorrect contract address in payload, caused by a bug that allowed an empty address to be passed for the remote contract.
+
+    - https://wormholescan.io/#/tx/2g4nn6fWZCkphxeMGfqjzViZYo3XAYarYTSvTcxYY95JAtZvwVd7MKY6RWHhhHd8oBzeFWjuTkXNo4tdVVTwBWfo
 
-Caused by a bug that allowed an empty address to be passed for the remote contract.
 
 ## Core
 
@@ -61,7 +63,7 @@ The classes that implement the [WormholeTransfer](./connect/src/protocols/wormho
 
 Every Protocol implementation defines its own private `createUnsignedTransaction` function, which, kinda sucks. 
 
-The Signer implementations are bad at things like gas estimation or handling errors.
+The Signer implementations are bad at things like gas estimation or handling errors. It should also provide better support for transaction review prior to signing and sending.
 
 
 ### Evm