Better function signature for `select_coins_to_spend`

```
          i think we've reached a point with this function that it is no longer maintainable / readable due to the number of arguments. this can be a follow up / tech debt. we should investigate a better way to structure it.
```