Skip to content

feat(displays): commands to display vault and config transactions#181

Merged
sean-sqds merged 6 commits into
mainfrom
feat/show-decoded-transaction-data
Apr 15, 2026
Merged

feat(displays): commands to display vault and config transactions#181
sean-sqds merged 6 commits into
mainfrom
feat/show-decoded-transaction-data

Conversation

@sean-sqds
Copy link
Copy Markdown
Contributor

@sean-sqds sean-sqds commented Apr 14, 2026

This pull request introduces two new display commands to the CLI for inspecting Vault and Config transactions, updates the documentation to describe these features, and improves code clarity by adding doc comments to command structs. Additionally, it adds a new dependency and increments the CLI version.

New CLI display commands:

  • Added display-transaction command to fetch and display details of a VaultTransaction account, including decoded instructions, accounts (with flags), base58-encoded data, and address lookup tables. (cli/src/command/display_transaction.rs, cli/src/command/mod.rs, cli/README.md) [1] [2] [3] [4] [5] [6]
  • Added display-config-transaction command to fetch and display details of a ConfigTransaction account, decoding actions such as member management, threshold changes, spending limits, and rent collector updates. (cli/src/command/display_config_transaction.rs, cli/src/command/mod.rs, cli/README.md) [1] [2] [3] [4] [5] [6]

Documentation updates:

  • Expanded the CLI README.md to document the new display commands, including usage, parameters, and examples. [1] [2]

Code clarity and maintainability:

Dependency and versioning:

  • Added the bs58 crate as a new dependency for base58 encoding. (cli/Cargo.toml)
  • Bumped the CLI version from 0.1.7 to 0.1.8. (cli/Cargo.toml)

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 14, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
v4-sdk-typedoc Ready Ready Preview, Comment Apr 15, 2026 11:47am

Request Review

Comment thread cli/Cargo.toml Outdated
Copy link
Copy Markdown

@palmaswell-squads palmaswell-squads left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need all those empty prints?

Comment thread cli/src/command/display_config_transaction.rs
@sean-sqds sean-sqds merged commit edbca83 into main Apr 15, 2026
4 of 5 checks passed
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.

5 participants