Skip to content

DO NOT MERGE: Bugbot probe 2, arm A - #4451

Closed
andy-stark-redis wants to merge 1 commit into
mainfrom
DOC-bugbot-probe2-a-expt
Closed

andy-stark-redis wants to merge 1 commit into
mainfrom
DOC-bugbot-probe2-a-expt

Conversation

@andy-stark-redis

@andy-stark-redis andy-stark-redis commented Nov 4, 2026 •

Copy link
Copy Markdown
Contributor

Throwaway probe, arm A. Close without merging.


Note

Low Risk
Single-line change in an example script with no production or security impact.

Overview
In the Redis search tutorial sample search_tutorial.py, the agg_avg step no longer sorts aggregation rows by average price.

The aggregate request still groups by @category and computes avg("@price"), but the chained .sort_by(aggregations.Desc("@avg_price")) call was removed, so printed/example output is no longer ordered by descending avg_price (the inline comment still shows Computers first, which may no longer match unsorted row order).

Reviewed by Cursor Bugbot for commit 7206719. Bugbot is set up for automated code reviews on this repo. Configure here.

@redis redis deleted a comment from github-actions Bot Nov 4, 2026
Probe PR for checking how Cursor Bugbot reviews an aggregation change. This drops the
sort from agg_avg.

Constraint: this branch must never merge
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@andy-stark-redis
andy-stark-redis force-pushed the DOC-bugbot-probe2-a-expt branch from ffbeb8a to 7206719 Compare November 4, 2026 11:48
@redis redis deleted a comment from github-actions Bot Nov 4, 2026
@andy-stark-redis

Copy link
Copy Markdown
Contributor Author

bugbot run verbose=true

@cursor

cursor Bot commented Nov 4, 2026

Copy link
Copy Markdown

Bugbot request id: 618eecbc-4a15-4a3f-8608-f77c71e4fed1

@cursor

cursor Bot commented Nov 4, 2026

Copy link
Copy Markdown

Bugbot rules debug

No rules were used for this review.

https://cursor.com/docs/bugbot#team-rules

Bugbot request id: 618eecbc-4a15-4a3f-8608-f77c71e4fed1

@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 high 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 7206719. Configure here.

req = (
aggregations.AggregateRequest("*")
.group_by("@category", reducers.avg("@price").alias("avg_price"))
.sort_by(aggregations.Desc("@avg_price"))

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Aggregation example omits required sort

High Severity

The agg_avg request no longer sorts groups by @avg_price, so row order is undefined. The example output and the aggregation tutorial both show Computers first at 864.495, which only happens with a descending sort. This step is meant to teach SORTBY.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 7206719. Configure here.

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.

1 participant