Skip to content

DO NOT MERGE: Bugbot / Repo memory probe - #4450

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

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

Conversation

@andy-stark-redis

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

Copy link
Copy Markdown
Contributor

Throwaway probe. Checks whether Cursor Bugbot reads the Repo memory comment, and whether it can reach the squash messages of the PRs listed there.

The diff removes the SORTBY @city ASC from the redis-py query3 example, which the squash message of #4290 records as a constraint. Close without merging.

🤖 Generated with Claude Code


Note

Low Risk
Single documentation/example edit with no production code paths; only affects doctest behavior if row order becomes non-deterministic without SORTBY.

Overview
Removes .sort_by(aggregations.Asc('@city')) from the query3 aggregation example in home_json.py, so the AggregateRequest only groups by @city and counts without an explicit sort.

Expected printed/asserted row order in the doctest block is unchanged (London then Tel Aviv), but result ordering from Redis is no longer guaranteed by the client call—unlike the parallel home-json examples in other language clients that still sort grouped results by @city ascending.

Per the PR description, this is an intentional throwaway change for tooling/memory probing and is not meant to merge.

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

Probe PR for checking whether Cursor Bugbot reads the Repo memory comment. This drops the
SORTBY from query3, which the squash message of #4290 records as a constraint.

Constraint: this branch must never merge
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@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: 3a34a7b8-7a47-4157-bae9-a9d68d0a8223

@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: 3a34a7b8-7a47-4157-bae9-a9d68d0a8223

@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 7398e26. Configure here.

req = aggregations.AggregateRequest("*").group_by(
'@city', reducers.count().alias('count')
).sort_by(aggregations.Asc('@city'))
)

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 omits required city sort

High Severity

query3 dropped .sort_by(aggregations.Asc('@city')), so group order is no longer deterministic. The hidden assert and documented output still expect London then Tel Aviv, the page copy says results are sorted by city, and PR #4290's squash message records SORTBY @city ASC as a constraint.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 7398e26. Configure here.

@andy-stark-redis

Copy link
Copy Markdown
Contributor Author

Probe 1 finished: Bugbot found the trap unaided via git history. Closing without merging.

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