Skip to content

Add evm token-holders command with token address as positional arg#36

Open
ivpusic wants to merge 1 commit intosim/evm-token-infofrom
sim/evm-token-holders
Open

Add evm token-holders command with token address as positional arg#36
ivpusic wants to merge 1 commit intosim/evm-token-infofrom
sim/evm-token-holders

Conversation

@ivpusic
Copy link
Member

@ivpusic ivpusic commented Mar 12, 2026

  • GET /v1/evm/token-holders/{chain_id}/{address} with --chain-id (required), --limit, --offset
  • Token address is positional arg, chain-id is a required flag for clarity
  • Table output: WALLET_ADDRESS, BALANCE, FIRST_ACQUIRED, HAS_TRANSFERRED (Y/N)
  • Chain ID validated as numeric
  • 5 E2E tests: text, JSON, pagination, invalid chain-id, required chain-id flag

@cursor
Copy link

cursor bot commented Mar 12, 2026

PR Summary

Low Risk
Low risk: adds a new CLI subcommand and associated output/parsing logic without changing existing API client behavior; main risk is regressions in argument/flag validation or formatting.

Overview
Adds a new sim evm token-holders <token_address> command that calls GET /v1/evm/token-holders/{chain_id}/{address} with required --chain-id plus optional --limit and --offset pagination.

The command supports -o json passthrough or a table view (wallet, balance, first acquired, and a Y/N transfer flag) and validates --chain-id is numeric.

Includes end-to-end tests covering text output, JSON output, pagination via next_offset, invalid --chain-id, and missing required --chain-id.

Written by Cursor Bugbot for commit d12b226. Configure here.

@ivpusic ivpusic force-pushed the sim/evm-token-info branch from d78a42c to b393895 Compare March 12, 2026 15:34
@ivpusic ivpusic force-pushed the sim/evm-token-holders branch from d12b226 to 2381d5d Compare March 12, 2026 15:34
@ivpusic ivpusic force-pushed the sim/evm-token-info branch from b393895 to 755ff88 Compare March 12, 2026 15:39
@ivpusic ivpusic force-pushed the sim/evm-token-holders branch from 2381d5d to 2eaa889 Compare March 12, 2026 15:39
@ivpusic ivpusic force-pushed the sim/evm-token-info branch from 755ff88 to e904424 Compare March 12, 2026 16:07
@ivpusic ivpusic force-pushed the sim/evm-token-holders branch from 2eaa889 to 07686bb Compare March 12, 2026 16:07
- GET /v1/evm/token-holders/{chain_id}/{address} with --chain-id (required), --limit, --offset
- Token address is positional arg, chain-id is a required flag for clarity
- Table output: WALLET_ADDRESS, BALANCE, FIRST_ACQUIRED, HAS_TRANSFERRED (Y/N)
- Chain ID validated as numeric
- 5 E2E tests: text, JSON, pagination, invalid chain-id, required chain-id flag
@ivpusic ivpusic force-pushed the sim/evm-token-holders branch from 07686bb to 99a071c Compare March 12, 2026 16:10
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