Clear pending timers on unmount in StatefulPressable

[m-bert]

Description

The stateful Pressable engine keeps four pending timers - longPressTimeoutRef, pressDelayTimeoutRef, hoverInTimeout and hoverOutTimeout - but never clears them on unmount. If the component unmounts while one is pending, the scheduled callback still fires (onPressIn / onLongPress / onHoverIn / onHoverOut, plus a setState), acting on a torn-down component.

This adds a cleanup effect that clears all four on unmount, matching the cleanup the Touchable-based engine already has.

Follow-up to #4411, flagged by CodeRabbit.

Test plan

Existing v3 suite passes (yarn test in packages/react-native-gesture-handler). No behavior change while mounted — the effect only cancels timers that would otherwise fire after unmount.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0a2c6bc3-b1f0-4be4-ac44-31331630ba27

📥 Commits

Reviewing files that changed from the base of the PR and between cb144a3 and 90a5007.

📒 Files selected for processing (1)

• packages/react-native-gesture-handler/src/v3/components/StatefulPressable.tsx

---

📝 Walkthrough

Summary by CodeRabbit

• Bug Fixes
  • Fixed pending press and hover timers continuing after a pressable component is removed.
  • Prevented delayed long-press, press, and hover interactions from triggering after unmounting.

Walkthrough

StatefulPressable now clears pending press and hover timers when the component unmounts.

Changes

StatefulPressable cleanup

Layer / File(s)	Summary
Unmount timer cleanup packages/react-native-gesture-handler/src/v3/components/StatefulPressable.tsx	An unmount effect clears long-press, delayed-press, hover-in, and hover-out timeout handles.

Possibly related PRs

• software-mansion/react-native-gesture-handler#4411: Adds the timers that this PR clears during StatefulPressable unmount.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title clearly and concisely describes the main change: clearing pending timers when StatefulPressable unmounts.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[Copilot]

Pull request overview

This PR prevents StatefulPressable’s delayed callbacks from firing after the component unmounts by adding an unmount cleanup that clears pending press/hover timers, aligning behavior with the Touchable-based engine.

Changes:

• Add a useEffect unmount cleanup to clear longPressTimeoutRef, pressDelayTimeoutRef, hoverInTimeout, and hoverOutTimeout.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.