Add evm transactions command and fix OpenAPI spec gaps across all EVM commands

[ivpusic]

• Add dune sim evm transactions with --chain-ids, --decode, --limit, --offset flags
• Add --decode text-mode stderr hint and E2E tests (text, JSON, decode-text, decode-JSON, pagination)
• Fix spec gaps: add missing struct fields across all EVM commands
  • transactions: block_version, max_fee_per_gas, max_priority_fee_per_gas, decoded, logs, errors
  • balances/balance/stablecoins: historical_prices, token_metadata, pool, errors + printBalanceErrors
  • activity: tokenMetadata.standard, functionInfo.inputs
  • balance: --metadata and --historical-prices CLI flags
• Use json.RawMessage for decoded input values to handle non-string ABI types safely

[ivpusic]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• Add svm transactions command with signature extraction and block time formatting #39
• Add svm balances command with SimClient wiring #38
• Add evm defi-positions command for DeFi portfolio positions #37
• Add evm token-holders command with token address as positional arg #36
• Add evm token-info command for token metadata and pricing #35
• Add evm collectibles command for NFT listings with spam filtering #34
• Add evm transactions command and fix OpenAPI spec gaps across all EVM commands #33 👈 (View in Graphite)
• EVM activity #31
• Sim evm stablecoins #30
• Get balance #29
• Sim evm balances #28
• Supported chains #27
• Sim auth #26
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cursor]

PR Summary

Medium Risk
Adds a new sim evm transactions CLI surface and expands JSON decoding structs/flags across existing EVM commands; moderate risk of breaking output/unmarshal expectations if the API schema differs. Changes are mostly additive but touch multiple commands’ response parsing and stderr messaging.

Overview
Adds a new sim evm transactions command with pagination, chain filtering, and optional --decode support (JSON-only), including stderr hints and end-to-end tests for text/JSON/decode/pagination.

Updates existing EVM commands to close API schema gaps by extending decoded response structs (e.g., balance errors, historical_prices, token_metadata, pool; activity token standard and function inputs with json.RawMessage values), and introduces printBalanceErrors to surface partial-failure details on stderr. Also extends sim evm balance to pass through --metadata and --historical-prices query params.

^{Written by Cursor Bugbot for commit bcff9e1. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix prepared a fix for the issue found in the latest run.

• ✅ Fixed: Balance command silently drops errors in text mode
  • Added missing printBalanceErrors call to runBalance function in balance.go, matching the pattern used in balances and stablecoins commands.

Or push these changes by commenting:

@cursor push a207014ce9

Preview (a207014ce9)

diff --git a/cmd/sim/evm/balance.go b/cmd/sim/evm/balance.go
--- a/cmd/sim/evm/balance.go
+++ b/cmd/sim/evm/balance.go
@@ -73,6 +73,8 @@
 			return fmt.Errorf("parsing response: %w", err)
 		}
 
+		printBalanceErrors(cmd, resp.Errors)
+
 		if len(resp.Balances) == 0 {
 			fmt.Fprintln(w, "No balance found.")
 			return nil

Comment @cursor review or bugbot run to trigger another review on this PR

[cursor]

Balance command silently drops errors in text mode

Medium Severity

The runBalance text-mode path in balance.go deserializes into balancesResponse (which now includes the new Errors field) but never calls printBalanceErrors. Both runBalances and runStablecoins call printBalanceErrors(cmd, resp.Errors) before rendering, so balance is the only command that silently swallows API-reported errors in text mode.

Additional Locations (2)

• cmd/sim/evm/balances.go#L153-L156
• cmd/sim/evm/stablecoins.go#L86-L88

 

[cursor]

Duplicated error types and printing functions across commands

Low Severity

transactionErrorInfo and balanceErrorInfo have identical fields, and printTransactionErrors and printBalanceErrors contain nearly identical logic. Both are introduced in this PR and could share a single error-info type and a common printing helper, reducing duplication and the risk of divergent fixes.

Additional Locations (2)

• cmd/sim/evm/transactions.go#L167-L187
• cmd/sim/evm/balances.go#L234-L254