Skip to content

Supported chains#27

Merged
ivpusic merged 1 commit intomainfrom
sim/supported-chains
Mar 12, 2026
Merged

Supported chains#27
ivpusic merged 1 commit intomainfrom
sim/supported-chains

Conversation

@ivpusic
Copy link
Member

@ivpusic ivpusic commented Mar 11, 2026

Add dune sim evm supported-chains command listing all supported EVM chains and their endpoint availability (balances, activity, transactions, etc.). Works without auth via a bare HTTP client fallback.

@cursor
Copy link

cursor bot commented Mar 12, 2026

PR Summary

Medium Risk
Adds a new CLI command that hits a public Sim API endpoint (with a built-in unauthenticated HTTP fallback), plus tests that execute the command and may rely on live network responses, which can introduce flakiness.

Overview
Adds dune sim evm supported-chains, a new public (no-auth) command that fetches /v1/evm/supported-chains and prints either raw JSON or a human-readable table of per-chain endpoint availability.

Refactors cmd/sim/evm to consume a small SimClient interface sourced from command context when available, and otherwise uses a new minimal HTTP client for this public endpoint; includes basic command-level tests for both text and JSON output.

Written by Cursor Bugbot for commit 6641c15. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

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

Bugbot Autofix is ON. A cloud agent has been kicked off to fix the reported issues.

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

}

func (c *bareSimClient) Get(ctx context.Context, path string, params url.Values) ([]byte, error) {
u, err := url.Parse("https://api.sim.dune.com" + path)
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: It would be nice to have the URL prefix in a variable

I see you already have defaultBaseURL from the previous PR

@ivpusic ivpusic force-pushed the sim/supported-chains branch from 6641c15 to e26d221 Compare March 12, 2026 15:34
@ivpusic ivpusic changed the base branch from sim/auth to graphite-base/27 March 12, 2026 15:39
@ivpusic ivpusic force-pushed the sim/supported-chains branch from e26d221 to 4669a46 Compare March 12, 2026 15:39
@ivpusic ivpusic changed the base branch from graphite-base/27 to main March 12, 2026 15:39
@ivpusic ivpusic force-pushed the sim/supported-chains branch from 4669a46 to c65dd10 Compare March 12, 2026 16:07
@ivpusic ivpusic merged commit 3e29f6e into main Mar 12, 2026
2 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.

2 participants