feat: migrate staking-cli to CWP + add wallet-cli and companion-wallet#13
Merged
feat: migrate staking-cli to CWP + add wallet-cli and companion-wallet#13
Conversation
…i packages Replace WalletConnect SDK dependency in staking-cli with CLI Wallet Protocol (CWP) for provider-agnostic wallet connectivity. Add wallet-cli for CWP provider discovery and interactive selection. Add companion-wallet as a local key CWP provider. Extend cli-sdk with send-transaction, --chain, --json flags and wallet-walletconnect CWP adapter. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Move CWP discovery, exec, and select modules from the standalone @anthropic-ai/wallet-cli package into @walletconnect/cli-sdk/src/cwp/. This eliminates a separate package and fixes the scope mismatch. - Add src/cwp/ barrel with exec, discovery, select modules - Add wallet binary as cwp-cli.ts entry point in cli-sdk - Update staking-cli imports from @anthropic-ai/wallet-cli → @walletconnect/cli-sdk - Remove @anthropic-ai/wallet-cli from changeset fixed group - Delete packages/wallet-cli/ Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
cli-sdk) to CLI Wallet Protocol (CWP) viawallet-cli— wallet commands now use stateless CWP provider subprocesses instead of QR code sessions@anthropic-ai/wallet-clipackage — CWP provider discovery (discoverProviders), execution (walletExec), and interactive selection (selectProvider) with capability/chain filtering@walletconnect/companion-walletpackage — local key CWP provider with AES-256-GCM encrypted keystore, signing, transaction broadcast, fund/drain commands@walletconnect/cli-sdk—send-transactioncommand,--chainflag for connect,--jsonflag for whoami,wallet-walletconnectCWP adapter binary, stderr for log outputBreaking Changes (staking-cli)
stake/unstake/claimacceptWalletSenderinterface instead ofWalletConnectCLI--browserflag removed,--wallet <name>flag addedstatusandbalancenow require--address=0x...(no longer discover from wallet)CLI_METADATAexport removedTest plan
npm run build— all 4 packages compilevitest— cli-sdk (41 tests), staking-cli (25 tests) all pass🤖 Generated with Claude Code