Skip to content

Fix node inspect block argument and reject unknown commands - #29

Open
geroldsteiner67 wants to merge 1 commit into
mainfrom
fix/node-inspect-unknown-command-20261023
Open

geroldsteiner67 wants to merge 1 commit into
mainfrom
fix/node-inspect-unknown-command-20261023

Conversation

@geroldsteiner67

Copy link
Copy Markdown
Contributor

Summary

  • Make modal node inspect block <INDEX> consume the documented second positional argument.
  • Reject unknown inspect commands and invalid arguments before opening the datastore.
  • Add regression tests for the CLI positional parse and invalid commands.

Validation

  • cargo test -p modality-cli-node inspect_command_tests (2 passed)
  • cargo check -p modality-cli-node
  • cargo clippy -p modality-cli-node -- -D warnings -A clippy::too_many_arguments
  • RUSTFLAGS="-D warnings" cargo check --all
  • RUSTFLAGS="-D warnings" cargo check --all --tests --examples
  • cargo clippy --all -- -D warnings -A clippy::too_many_arguments

This branch has not been deployed

No deployments
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