chore: Add Cursor Bugbot PR review guidelines (JAVA-721) by runningcode · Pull Request #6046 · getsentry/sentry-java

runningcode · 2026-09-02T14:14:30Z

📜 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

resolves: JAVA-721
resolves: Add bugbot to repo #6044

💚 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

I added GH Issue ID & Linear ID
I added tests to verify the changes. — n/a, no code changes
No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
I updated the docs if needed. — this is the doc
I updated the wizard if needed. — n/a
Review from the native team if needed. — n/a
No breaking change or entry added to the changelog.
No breaking change for hybrid SDKs or communicated to hybrid SDKs.
Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec. — no public API changes

🔮 Next steps

#skip-changelog

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>

sentry · 2026-09-02T14:15:47Z

📲 Install Builds

Android

🔗 App Name	App ID	Version	Configuration
SDK Size	io.sentry.tests.size	8.55.0 (1)	release

⚙️ sentry-android Build Distribution Settings

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>

linear-code · 2026-09-02T14:18:22Z

JAVA-721

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>

markushi

Look great, already curious about the upcoming bot feedback we'll get.

style: Wrap BUGBOT.md lines at 100 characters

4d32a67

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 ~~chore: Add Cursor Bugbot PR review guidelines~~ chore: Add Cursor Bugbot PR review guidelines (JAVA-721) Sep 2, 2026

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 PR needs a lightweight review for obvious issues label Sep 2, 2026