chore: Add Cursor Bugbot PR review guidelines (JAVA-721) - #6046
Open
runningcode wants to merge 3 commits into
Open
chore: Add Cursor Bugbot PR review guidelines (JAVA-721)#6046runningcode wants to merge 3 commits into
runningcode wants to merge 3 commits into
Conversation
Bugbot reviews PRs against generic Java advice unless the repo tells it what this SDK actually cares about. This adds .cursor/BUGBOT.md, the same location sentry-javascript, sentry-dart, and sentry-react-native use. The rules are drawn from this repo's own invariants: the narrow-catch rule and ExceptionUtils.rethrowIfFatal, binary compatibility and the IScope/IScopes implementation fan-out, opt-in-by-default options, cost added to the Android main-thread init path, span origin and op conventions, and monotonic-vs-wall clock selection. Formatting, PR title format, and changelog entries are listed as out of scope, since Spotless, validate-pr, and Danger already enforce them. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
📲 Install BuildsAndroid
|
Re-wrap the bullets that ran long and add the missing blank line after the Concurrency and Clocks headings, so the file matches the wrapping used throughout. No wording changes. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
runningcode
changed the title
runningcode
marked this pull request as ready for review
September 2, 2026 14:25
runningcode
requested review from
0xadam-brown,
adinauer,
markushi and
romtsn
as code owners
September 2, 2026 14:25
runningcode
added
the
sanity-check
runningcode
commented
Sep 3, 2026
runningcode
commented
Sep 3, 2026
runningcode
commented
Sep 3, 2026
runningcode
commented
Sep 3, 2026
runningcode
commented
Sep 3, 2026
runningcode
commented
Sep 3, 2026
runningcode
commented
Sep 3, 2026
runningcode
commented
Sep 3, 2026
runningcode
commented
Sep 3, 2026
Add checks for reflection keep rules, SAGP bytecode manipulation, threading models on new classes, clock arithmetic misuse, dependency bump intent, and contract-focused tests. Require broad catches to justify themselves in a code comment rather than the PR description, so the reasoning stays with the code. Narrow the test requirement to customer-facing behavior, and drop the `.api` and IScope/IScopes rules now covered elsewhere. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
runningcode
commented
Sep 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📜 Description
Adds
.cursor/BUGBOT.md, a rules file for Cursor Bugbot PR review.Read the file, not much explaining here to do. We should probably update the AGENTS.md file to be consistent with this too.
💡 Motivation and Context
💚 How did you test it?
Not applicable — no code changes. Behavior will be visible on the next PRs Bugbot reviews, and the rules can be tuned from there.
📝 Checklist
sendDefaultPIIis enabled.🔮 Next steps
#skip-changelog