Skip to content

Fix query Insights errors and pagination - #1388

Merged
no-itsbackpack merged 2 commits into
mainfrom
egomez/fix-insights-regressions
Aug 26, 2026
Merged

Fix query Insights errors and pagination#1388
no-itsbackpack merged 2 commits into
mainfrom
egomez/fix-insights-regressions

Conversation

@no-itsbackpack

Copy link
Copy Markdown
Contributor

Summary

  • report missing query executions and fingerprints instead of claiming the branch is missing
  • add --page and --per-page to query traffic-budget results without walking all pages
  • remove the nested ls alias that intercepted insights queries ls

Test plan

  • go test ./internal/cmd/insights ./internal/planetscale -count=1

Name missing query resources accurately, expose traffic-budget page controls, and remove the alias that hijacked `insights queries ls`.

Co-authored-by: Cursor <cursoragent@cursor.com>
@no-itsbackpack
no-itsbackpack requested a review from a team as a code owner August 26, 2026 01:06

@cursor cursor Bot left a comment

Copy link
Copy Markdown

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 using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1f1c80e. Configure here.

Comment thread internal/cmd/insights/query_traffic_budgets.go

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Risk: low. Left a non-blocking comment; no reviewers were assigned. Cursor Bugbot finished as skipped with one unresolved pagination empty-state finding, so this run is not approving.

Open in Web View Automation 

Sent by Cursor Approval Agent: cli-reviewer

Match other list commands: omit page unless requested, and only claim no budgets exist on the first page.

Co-authored-by: Cursor <cursoragent@cursor.com>
@no-itsbackpack
no-itsbackpack merged commit 085a7f5 into main Aug 26, 2026
4 checks passed
@no-itsbackpack
no-itsbackpack deleted the egomez/fix-insights-regressions branch August 26, 2026 01:49
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