Sub-issue of #94595.
Source plan: fabioh8010 comment.
Goal
Complete the migration after all batches merge and Onyx PR #800 is published.
Scope
- Bump
react-native-onyx to the deps-removed version (package.json + lockfile)
src/hooks/useOnyx.ts: drop the dependencies param + stop forwarding it to originalUseOnyx
- Delete
eslint-plugin-local-rules/no-useonyx-dependencies-arg.js + its config entry + seatbelt rows
- Grep for any
useOnyx(..., ..., ...) that landed after the batches and migrate
- Check App doesn't reference
useLiveRef; adopt createMemoizedSelector only where genuinely needed
Dependencies
- PR 1 + Batches A–H merged
- Onyx PR #800 merged/published
- Seatbelt baseline for
rulesdir/no-useonyx-dependencies-arg = 0
Acceptance criteria
Issue Owner
Current Issue Owner: @hoangzinh
Sub-issue of #94595.
Source plan: fabioh8010 comment.
Goal
Complete the migration after all batches merge and Onyx PR #800 is published.
Scope
react-native-onyxto the deps-removed version (package.json+ lockfile)src/hooks/useOnyx.ts: drop thedependenciesparam + stop forwarding it tooriginalUseOnyxeslint-plugin-local-rules/no-useonyx-dependencies-arg.js+ its config entry + seatbelt rowsuseOnyx(..., ..., ...)that landed after the batches and migrateuseLiveRef; adoptcreateMemoizedSelectoronly where genuinely neededDependencies
rulesdir/no-useonyx-dependencies-arg= 0Acceptance criteria
npm run typecheck(compiler enforces no 3rd arg)npm run lintpassesnpm run testpassesIssue Owner
Current Issue Owner: @hoangzinh