Skip to lesson
Exit
Bugbot PR review1 / 2

1 min lesson

Review the SQL finding

Reproduce one Bugbot finding, fix the cited line, rerun its test and confirm the new review covers the latest commit.

Step 1 of 2

Verify the finding on the current commit

Interactive diagram. Step through it with the Next and Previous controls below, or Tab to a region to read its detail.

diagram: flow

Keep the cited line, focused test and new Bugbot run in one chain.

Learn more

Full explanation

Latest commit, cited line, test, rerun, check status

Review a SQL interpolation finding on PR #482
Latest commit, cited line, test, rerun, check status
SayPR #482 changes how src/search/query.ts builds a search query. BugbotCursor's automated PR reviewer that posts inline findings and can push fix commits from isolated VMs. Press Enter for the full definition. is enabled for the repository and runs after each pull request update.
SeeThe Cursor BugbotCursor's automated PR reviewer that posts inline findings and can push fix commits from isolated VMs. Press Enter for the full definition. check is neutral and an inline finding points to the changed query. Neutral can mean findings, cancellation or an internal error, so open the run instead of treating it as a pass.
DoOpen the finding beside the diff. Run the focused search test with an input that contains a quote and inspect the generated query.
SeeThe test reproduces direct string interpolation at the cited line. The finding is real and its scope is clear.
DoReplace the interpolation with a parameterized query, rerun the focused test and push the fix. Wait for BugbotCursor's automated PR reviewer that posts inline findings and can push fix commits from isolated VMs. Press Enter for the full definition. to review the new commit.
SeeThe test passes and the new run covers the latest pull request diff rather than the earlier commit.
DoOpen the new Cursor BugbotCursor's automated PR reviewer that posts inline findings and can push fix commits from isolated VMs. Press Enter for the full definition. check and confirm that no current or earlier Bugbot comment remains unresolved. Then read the full diff, tests and other required checks.
SeeBugbotCursor's automated PR reviewer that posts inline findings and can push fix commits from isolated VMs. Press Enter for the full definition. reports success for the latest commit. That proves this review found no issue and no earlier Bugbot comment remains unresolved; it does not replace the merge review.
Learn more

Optional practice

Test yourself on Review the SQL finding

QWhich result proves the SQL finding is resolved on the current pull request?