Skip to content

fix(cli): allow --performance small for query run and run-sql#54

Merged
charisra merged 1 commit intomainfrom
fix-cli-allow-performance-small-for-query-run
Apr 21, 2026
Merged

fix(cli): allow --performance small for query run and run-sql#54
charisra merged 1 commit intomainfrom
fix-cli-allow-performance-small-for-query-run

Conversation

@charisra
Copy link
Copy Markdown
Contributor

@charisra charisra commented Apr 21, 2026

  • Extend parsePerformance to accept small alongside medium and large (matches public API / Dune API client).
  • Update dune query run and dune query run-sql flag help and README so docs match the allowed values.
  • No change to default performance (still medium).

@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 21, 2026

PR Summary

Low Risk
Small, localized change to CLI flag validation and documentation; minimal behavioral impact beyond accepting an additional enum value.

Overview
The CLI now accepts --performance small for dune query run and dune query run-sql by extending parsePerformance validation to allow small in addition to medium/large.

Help text and the README usage tables are updated so documented --performance values match the newly accepted tier; the default remains medium.

Reviewed by Cursor Bugbot for commit 854691d. Configure here.

Copy link
Copy Markdown
Contributor Author

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

@charisra
Copy link
Copy Markdown
Contributor Author

bugbot run

Copy link
Copy Markdown

@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.

✅ Bugbot reviewed your changes and found no new issues!

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

Reviewed by Cursor Bugbot for commit 854691d. Configure here.

Comment thread cmd/query/run_sql.go
@charisra charisra merged commit 133c977 into main Apr 21, 2026
3 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