Skip to content

Refactor CLI command modules for exploration#19

Merged
rosterloh merged 1 commit into
devfrom
refactor/parameter-modularity
Jun 10, 2026
Merged

Refactor CLI command modules for exploration#19
rosterloh merged 1 commit into
devfrom
refactor/parameter-modularity

Conversation

@rosterloh

@rosterloh rosterloh commented Jun 10, 2026

Copy link
Copy Markdown
Owner

Summary

  • Split all CLI command handlers into focused command modules: topics, nodes, echo, and parameters.
  • Keep talos-cli/src/main.rs as the composition root for arg parsing, client setup, and dispatch.
  • Split parameter protocol types and TUI parameter input handling into focused modules.
  • Make list-topics and list-nodes propagate agent error responses instead of printing and exiting successfully.

Verification

  • rtk cargo fmt -- --check
  • rtk cargo check -p talos-common -p talos-cli -p talos-tui --offline
  • rtk cargo test -p talos-common --offline
  • rtk cargo test -p talos-cli --offline
  • rtk cargo test -p talos-tui --offline
  • with rclrs_ws/install/setup.zsh sourced: rtk cargo check --workspace --offline
  • with rclrs_ws/install/setup.zsh sourced: rtk cargo test --workspace --offline

@rosterloh
rosterloh force-pushed the refactor/parameter-modularity branch from 119fa36 to 85800c7 Compare June 10, 2026 09:04
@rosterloh rosterloh changed the title Refactor parameter modules for exploration Refactor CLI command modules for exploration Jun 10, 2026
@rosterloh
rosterloh merged commit cfe1d33 into dev Jun 10, 2026
2 checks passed
@rosterloh
rosterloh deleted the refactor/parameter-modularity branch June 10, 2026 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant