Skip to content

CLI: Print TX info for Squads#64

Open
aoikurokawa wants to merge 2 commits into
masterfrom
ak/squads
Open

CLI: Print TX info for Squads#64
aoikurokawa wants to merge 2 commits into
masterfrom
ak/squads

Conversation

@aoikurokawa

Copy link
Copy Markdown

No description provided.

@aoikurokawa aoikurokawa marked this pull request as ready for review April 21, 2026 23:42
@aoikurokawa aoikurokawa requested a review from Copilot April 21, 2026 23:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds CLI support to output instruction data suitable for composing transactions in Squads, and updates CLI docs/examples to reflect current mainnet IDs/flows.

Changes:

  • Add --print-tx support to update-stake-deposit-authority, including base58 instruction data printing.
  • Update hardcoded withdraw_authority pubkey used by whitelisted deposit/withdraw builders.
  • Update CLI README examples and add bs58 dependency (workspace + CLI crate).

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
cli/src/interceptor_handler.rs Adds base58 instruction printing for Squads and adjusts update + whitelisted instruction construction details.
cli/README.md Adds an example for printing the update instruction and refreshes example addresses/flags.
cli/Cargo.toml Adds bs58 dependency via workspace.
Cargo.toml Adds bs58 to workspace dependencies.
Cargo.lock Lockfile update for bs58.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cli/src/interceptor_handler.rs
Comment thread cli/src/interceptor_handler.rs
Comment thread cli/src/interceptor_handler.rs
Comment thread cli/src/interceptor_handler.rs
Comment thread cli/README.md Outdated
@aoikurokawa aoikurokawa requested a review from ebatsell June 10, 2026 19:38
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.

2 participants