Skip to content

feat(tooling): add Cursor BUGBOT.md review guidelines#5716

Merged
antonis merged 2 commits intomainfrom
add-cursor-instructions
Feb 25, 2026
Merged

feat(tooling): add Cursor BUGBOT.md review guidelines#5716
antonis merged 2 commits intomainfrom
add-cursor-instructions

Conversation

@antonis
Copy link
Copy Markdown
Contributor

@antonis antonis commented Feb 24, 2026

Summary

Closes #5657

Adds .cursor/BUGBOT.md with PR review guidelines for the Cursor bot, adapted from sentry-dart's equivalent and tailored for the React Native SDK.

Key sections:

  • Critical issues: security/PII, public API stability, and dependency update review (reads PR description changelog for native SDK bumps)
  • TypeScript/JS idioms: null safety, async patterns, tree-shakeability
  • React Native bridge: iOS (Obj-C/Swift) and Android (Java/Kotlin) native module conventions, TurboModule/New Architecture awareness
  • SDK-specific: span lifecycle, sentry.origin, replay masking, safe init paths

Test plan

  • Verify .cursor/BUGBOT.md is picked up by Cursor in this repo
  • Open a dependency bump PR and confirm the bot checks the PR description for changelog info

🤖 Generated with Claude Code

Adds .cursor/BUGBOT.md with PR review instructions for the Cursor bot,
covering security/privacy, public API stability, dependency update checks
(including native SDK changelogs), TypeScript idioms, React Native bridge
patterns, and SDK-specific tracing/replay rules.

Closes #5657
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 24, 2026

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


  • feat(tooling): add Cursor BUGBOT.md review guidelines by antonis in #5716
  • chore(ci): Use runner group 10 by itaybre in #5717
  • chore(deps): bump qs to ^6.14.2 by antonis in #5707
  • chore(deps): bump lodash to ^4.17.23 by antonis in #5702
  • chore(deps): bump getsentry/craft from 2.21.4 to 2.21.7 by dependabot in #5694
  • chore(deps): bump getsentry/craft/.github/workflows/changelog-preview.yml from 2.21.4 to 2.21.7 by dependabot in #5695
  • chore(deps): update CLI to v3.2.2 by github-actions in #5692
  • chore(deps): bump github/codeql-action from 4.32.3 to 4.32.4 by dependabot in #5693
  • chore(deps): update Maestro to v2.2.0 by github-actions in #5691
  • chore(deps): update Cocoa SDK to v9.5.0 by github-actions in #5685
  • chore(deps): update Android SDK Stubs to v8.33.0 by github-actions in #5697
  • chore(deps): update Android SDK to v8.33.0 by github-actions in #5684
  • chore(deps): update Sentry Android Gradle Plugin to v6.1.0 by github-actions in #5687
  • Ref(CI): Add android sdk version check by lucas-zimerman in #5686

🤖 This preview updates automatically when you update the PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 24, 2026

Fails
🚫 Pull request is not ready for merge, please add the "ready-to-merge" label to the pull request
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 1233a70

Copy link
Copy Markdown
Collaborator

@lucas-zimerman lucas-zimerman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good!

@antonis antonis merged commit d4ebf2b into main Feb 25, 2026
34 of 39 checks passed
@antonis antonis deleted the add-cursor-instructions branch February 25, 2026 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add cursor instructions

2 participants