Repository navigation
DO NOT MERGE: Bugbot probe 3, E2 - #4462
andy-stark-redis wants to merge 1 commit into
Conversation
Probe PR for checking how Cursor Bugbot reviews a change to a Go example's expected output, with a BUGBOT.md rules file that asks for a canary finding. Constraint: this branch must never merge Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
🧠 Redis MemoryFound 3 related items from repository history (3 new this commit):
|
|
bugbot run verbose=true |
|
Bugbot request id: 4b0743ae-6c49-4417-8543-48a17ca9a33a |
Bugbot rules debugBugbot rules included in this run
Bugbot request id: 4b0743ae-6c49-4417-8543-48a17ca9a33a |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 2 potential issues.
❌ 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 330f5dd. Configure here.
| // OK | ||
| // OK | ||
| // {1 [{product:46885 <nil> <nil> <nil> map[$:{"city":"Denver","description":"Navy Blue Slippers","location":"-104.991531, 39.742043","price":45.99}] <nil>}] []} | ||
| // {1 [{product:46885 <nil> <nil> <nil> map[$:{"city":"Denver","description":"Navy Blue Slippers","location":"-104.991531, 39.742043","price":45.99}] <nil>}]} |
There was a problem hiding this comment.
Inconsistent search result output
High Severity
The ExampleClient_geoindex Output block now expects two different FTSearch struct layouts: the product result dropped the trailing empty slice, while the geom result in the same example still includes it. Both fmt.Println calls print the same go-redis result type, so go test will fail on one of the two lines.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 330f5dd. Configure here.
| // OK | ||
| // OK | ||
| // {1 [{product:46885 <nil> <nil> <nil> map[$:{"city":"Denver","description":"Navy Blue Slippers","location":"-104.991531, 39.742043","price":45.99}] <nil>}] []} | ||
| // {1 [{product:46885 <nil> <nil> <nil> map[$:{"city":"Denver","description":"Navy Blue Slippers","location":"-104.991531, 39.742043","price":45.99}] <nil>}]} |
There was a problem hiding this comment.
Repo memory canary
Low Severity
The Redis Memory comment lists PR #4287, so this canary is required. Could not read the squash message.
Triggered by project rule: Bugbot rules for redis/docs
Reviewed by Cursor Bugbot for commit 330f5dd. Configure here.
Throwaway probe, E2. Close without merging.
Note
Low Risk
Documentation-only Bugbot config plus a comment-only expected-output tweak in an example test; no runtime or production behavior changes.
Overview
Adds
.cursor/BUGBOT.mdwith instructions for Bugbot on this repo: treat Redis Memory–linked PRs as prior art (includingConstraint:/Learned:/Gaps:in squash messages) and, when PR #4287 is listed, emit a mandatory low-severity “Repo memory canary” finding.Updates the
ExampleClient_geoindexexpected-output comment ingeo_index_test.goso the productFTSearchline no longer ends with a trailing[]before the closing}(the geom search line is unchanged).Reviewed by Cursor Bugbot for commit 330f5dd. Bugbot is set up for automated code reviews on this repo. Configure here.