Skip to content

Conversation

@thomasttvo
Copy link
Collaborator

@thomasttvo thomasttvo commented Dec 24, 2025

Reanimated has a hard requirement that a worklet must be defined before the worklet that calls it is. Otherwise, the app will crash. At the same time there's no linter that would apply the rule only for worklets, so we have to turn it on for everything. Quite inconvenient, but gets us one step toward reanimated for now, and can be revisited for future improvements.

There's no logic change in this PR. Only functions getting moved around.


Note

Refactors and reorders code to comply with no-use-before-define, extracting shared logic into hooks/helpers and adjusting call sites without changing behavior.

  • Introduces useZoomSubject (measurement) and useDebugPoints (pinch debug) hooks, and moves zoom step calc to helper/getNextZoomStep
  • Updates ReactNativeZoomableView to use new hooks, replaces inline helpers, and reorders pan/gesture handlers and setup
  • Minor formatting and eslint-disable annotations in AnimatedTouchFeedback, StaticPin, and debug helpers; map files added

Written by Cursor Bugbot for commit 16483a8. Configure here.

@thomasttvo thomasttvo changed the base branch from master to thomas/functional December 24, 2025 19:58
@thomasttvo thomasttvo added the enhancement New feature or request label Jan 2, 2026
@thomasttvo
Copy link
Collaborator Author

bugbot run

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no bugs!


Comment @cursor review or bugbot run to trigger another review on this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants