Repository navigation
Conversation
🎉 All green!🧪 All tests passed 🔗 Commit SHA: 106c799 | Docs | View more details | Give us feedback! |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 04c8796616
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
04c8796 to
c3abd77
Compare
c3abd77 to
106c799
Compare

What does this PR do?
Preserves
undefinedandnullhandler props when applying Babel RUM instrumentation. Referenced handlers are evaluated once and wrapped only when present.This restores
Pressablebehavior: when noonLongPresshandler is provided, holding and releasing still triggersonPress.Fixes #1509.
Motivation
The plugin previously replaced absent handlers with wrapper functions. React Native treated the generated
onLongPresswrapper as a defined handler and suppressedonPressafter the long-press threshold, silently dropping button presses.Additional Notes
Pressabilitystate machine, covering absent and defined handlers with and without tracking.example-new-architecturewith visible press counters.Review checklist (to be filled by reviewers)