CLI Rework TODO List

Code Refactoring

• Split CLI commands into separate submodules under commands/
• Centralize shared utilities in common.rs
• Consistent user interface for different operations (e.g. coordinator addr & port specification)

Documentation

• Write a tutorial guide for typical workflows and beginner onboarding
• Write a reference manual covering all commands, options, and usage examples

Node Hub Design & Architecture

• Discuss and define the role of a centralized Node Hub
• Design Node Hub related commands

Testing

• Add unit and integration tests for CLI components
  • Set up a mock CLI environment to validate behavior and user experience

Migration

• Ensure compatibility with existing CLI usage patterns

Related: #551

So I feel like you want to make a lot of changes on the structure of the CI.

Could we try to make as small PR as possible as it might be very hard to debug the CLI if something breaks.

And I feel like this PR is already pretty good and might be conlicting with Philipps #901

So I feel like you want to make a lot of changes on the structure of the CI.

Not exactly, I'm mentioning the CI just for tests (which is not presented yet) to make sure this PR does not break anything.

Could we try to make as small PR as possible as it might be very hard to debug the CLI if something breaks.

Totally agree, I think I should put the TODO list in an issue or discussion for tracking, instead of leaving it here. This PR should only change the CLI crate file structure.

And I feel like this PR is already pretty good and might be conflicting with Philipp's #901.

Sorry for not investigating the existing work in advance but I actually started working on this earlier than the first commit. I don't think it's fully tested to be OK (as I stated before), so I kept it as drafted.

I can do some compatibility fixes for #901 once it has been merged. 😊

Sorry I meant CLI :D

But okay sounds good!

@phil-opp @haixuanTao Would it be possible for someone to review this PR when convenient?

I looked into it and it looks good to me.

I’ve just merged the latest main into this refactor branch after #901 was merged.
That PR introduced extensive renames/moves, so I resolved several non-trivial merge conflicts locally and pushed the result. I’ve reviewed the diff, but I’m not 100 % certain every new feature made it through intact, so a quick sanity-check would be great. Thanks!

I pushed a commit to your branch to rename the commands module to command, in order to reduce the number of changes in the diff.

@phil-opp Please review

@phil-opp I modified the related part, plz review