Skip to content

Don't explicitly set perf tier, it's plan dependent#56

Merged
charisra merged 1 commit intomainfrom
optional_performance
Apr 22, 2026
Merged

Don't explicitly set perf tier, it's plan dependent#56
charisra merged 1 commit intomainfrom
optional_performance

Conversation

@ivpusic
Copy link
Copy Markdown
Member

@ivpusic ivpusic commented Apr 21, 2026

Unless explicitly provided, perf tier will be auto selected for the used

@ivpusic ivpusic requested a review from charisra April 21, 2026 22:10
@cursor
Copy link
Copy Markdown

cursor Bot commented Apr 21, 2026

PR Summary

Medium Risk
Behavior change for CLI executions that previously implicitly used medium performance; may affect users/scripts relying on the old default or the removed small tier.

Overview
query run and query run-sql no longer default --performance to medium; the flag now defaults to empty so the API can auto-select the appropriate tier unless the user explicitly sets one.

Validation and help text were updated accordingly (dropping support for small and clarifying override semantics), and new tests assert that omitted --performance results in an empty Performance field being sent.

Reviewed by Cursor Bugbot for commit 41c7c2f. Configure here.

@charisra
Copy link
Copy Markdown
Contributor

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 41c7c2f. Configure here.

@charisra charisra merged commit 865e42f into main Apr 22, 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