Split tasks spawned by CLI commands into their own modules

[romac]

First small step towards #330

Description

Split tasks spawned by CLI commands into their own modules.

---

For contributor use:

• Unit tests written
• Added test to CI if applicable
• Updated CHANGELOG_PENDING.md
• Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
• Updated relevant documentation (docs/) and code comments
• Re-reviewed Files changed in the Github PR explorer

[adizere]

Cool, looks cleaner than before!

Some suggestion to try to further improve this (defer to next iteration of work towards #330):

• I suspect relayer-cli/src/commands/utils.rs can be deleted
• it would improve readability if all files in commands.rs have the same structure: definition of pub struct xyzCmd, then implementation of that struct, then impl Runnable for xyzCmd

[adizere]

Thanks! Looks good, I had nothing to comment.

[ancazamfir]

So start command doesn't start the monitors anymore? Just the light clients? I think the README.md needs a small update.

[romac]

So start command doesn't start the monitors anymore? Just the light clients? I think the README.md needs a small update.

It never did that AFAIK.

[ancazamfir]

So start command doesn't start the monitors anymore? Just the light clients? I think the README.md needs a small update.

It never did that AFAIK.

Just checked, it did before #255, the start command was starting them together with the event_handler from the relayer_task(). We also have the listen command that just starts the monitors and prints the events.

[ancazamfir]

Looks good!

[romac]

So start command doesn't start the monitors anymore? Just the light clients? I think the README.md needs a small update.

It never did that AFAIK.

Just checked, it did before #255, the start command was starting them together with the event_handler from the relayer_task(). We also have the listen command that just starts the monitors and prints the events.

Right, my bad! Since we are going to rewrite this code for v0 anyway, it doesn't really matter so I guess we can just go ahead and merge this.