unbonding with ygg funds can leave rune behind on bond module
If rune is included on an unbond transaction, it is added to the node account after unbonding. If the node has yggdrasil funds, the funds are recalled and the handler returns before updating the node account with the extra bond. In the ygg case, the bond is left on the bond module but never credited to the node, leaving an oversolvency on the module.
Here's an example transaction: https://viewblock.io/thorchain/tx/3D707DD0E8C0F26B26B80BE07C508EF31738A7877D22CD22F5F81611FEBB47CA
Think we should update the node account bond at the top of the function. Simple enough, but I'm not sure if that has any other implications, so just opening an issue for now.
Adding coin to node account at end of function: https://gitlab.com/thorchain/thornode/-/blob/develop/x/thorchain/handler_unbond.go#L246-252
Returning early for ygg funds, skipping the node account update: https://gitlab.com/thorchain/thornode/-/blob/develop/x/thorchain/handler_unbond.go#L178
Related to #1258 (closed)