Skip to content

[Split #94595] Final PR - Remove useOnyx dependencies arg and bump Onyx #95582

Description

@mountiny

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

  • Full npm run typecheck (compiler enforces no 3rd arg)
  • npm run lint passes
  • npm run test passes
Issue OwnerCurrent Issue Owner: @hoangzinh

Metadata

Metadata

Labels

#qualityRelates to work in the #quality roomReviewingHas a PR in reviewTaskWeeklyKSv2

Type

No type

Projects

Status
SUBISSUE

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions