Skip to content

[skill-drift] fix(sentry-react-native-sdk): add includeWebFeedback Metro option and rage tap detection docs #108

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
HazAT merged 1 commit into from
May 4, 2026
Merged

[skill-drift] fix(sentry-react-native-sdk): add includeWebFeedback Metro option and rage tap detection docs #108

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
21 changes: 20 additions & 1 deletion skills/sentry-react-native-sdk/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -309,7 +309,12 @@ const { getDefaultConfig } = require("@react-native/metro-config");
const { withSentryConfig } = require("@sentry/react-native/metro");

const config = getDefaultConfig(__dirname);
module.exports = withSentryConfig(config);
module.exports = withSentryConfig(config, {
// Set to false to exclude @sentry-internal/replay from the native bundle (web only).
// includeWebReplay: true,
// Set to false to exclude @sentry-internal/feedback from the native bundle (web only).
// includeWebFeedback: true,
});
```

**Step 3 — iOS: Modify Xcode build phase**
Expand Down Expand Up @@ -724,6 +729,20 @@ These integrations are enabled automatically — no config needed:
| `feedbackIntegration()` | Add to `integrations` array (user feedback widget; supports `enableShakeToReport` for native shake detection) |
| `deeplinkIntegration()` | Add to `integrations` array (auto-captures deep link URLs as breadcrumbs; opt-in) |

### Rage Tap Detection (TouchEventBoundary)

`TouchEventBoundary` (wraps your app root) includes built-in rage tap detection. When a user taps the same element 3+ times within 1 second, a `ui.multiClick` breadcrumb is emitted and shown on the replay timeline. Configure via props:
Copy link
Copy Markdown

@sentry sentry Bot Apr 27, 2026

Choose a reason for hiding this comment

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

Bug: The documentation for rage tap detection incorrectly specifies the breadcrumb type as ui.multiClick instead of ui.frustration.
Severity: LOW

Suggested Fix

Update the documentation in SKILL.md to replace the incorrect breadcrumb type ui.multiClick with the correct type, ui.frustration, for rage tap detection.

Prompt for AI Agent 
Review the code at the location below. A potential bug has been identified by an AI
agent. Verify if this is a real issue. If it is, propose a fix; if not, explain why it's
not valid.

Location: skills/sentry-react-native-sdk/SKILL.md#L734

Potential issue: The documentation at `skills/sentry-react-native-sdk/SKILL.md`
incorrectly states that rage tap detection in the React Native SDK emits a breadcrumb
with the type `ui.multiClick`. The correct breadcrumb type for this feature is
`ui.frustration`. The `ui.multiClick` type is associated with rage-clicks in the web
SDK, not rage-taps in the React Native SDK. This discrepancy can cause confusion for
developers trying to use or filter these breadcrumbs.

Did we get this right? 👍 / 👎 to inform future reviews.


```tsx
<Sentry.TouchEventBoundary
enableRageTapDetection={true} // default: true — set false to disable
rageTapThreshold={3} // taps required to trigger (default: 3)
rageTapTimeWindow={1000} // detection window in ms (default: 1000)
>
<App />
</Sentry.TouchEventBoundary>
```

---

## Expo Config Plugin Reference
Expand Down
Loading