Skip to content

Comments

fix(android): revert resolveReactHost logic to use getReactHost()#121

Merged
floydkim merged 3 commits intomasterfrom
chore/drop-rn76-and-simplify-reacthost
Feb 23, 2026
Merged

fix(android): revert resolveReactHost logic to use getReactHost()#121
floydkim merged 3 commits intomasterfrom
chore/drop-rn76-and-simplify-reacthost

Conversation

@floydkim
Copy link
Collaborator

Background

Our support policy is:

  • RN 0.76 and below should use react-native-code-push@12.0.2
  • This library supports RN 0.77+ only

In a recent commit, resolveReactHost in CodePushNativeModule.java became overly complex to preserve legacy compatibility.
Given the current support range, that reflection-based path is no longer needed.

What changed

1) Reverted recent changes to simplify resolveReactHost (Android)

  • Resolve ReactDelegate
  • Return reactDelegate.getReactHost() directly

2) Removed RN <=0.76 from E2E matrix

  • Updated RN CLI E2E matrix targets to start from RN 0.77.x

Why:

  • RN <=0.76 is intentionally frozen at 12.0.2
  • Keeping those versions in this repo’s active E2E matrix creates noise and implies support outside current policy
  • Matrix coverage should reflect the actual maintained support range (0.77+)

Why this is needed

  • Align implementation and test coverage with the official support policy
  • Reduce maintenance overhead from unnecessary legacy branches
  • Keep CI/E2E signal focused on actively supported RN versions

Validation

  • local E2E passed
    • RN 0.77 ~ 0.84 (New and Legacy Architecture both)
    • Expo 54, 55 Beta

@floydkim floydkim merged commit c435a0d into master Feb 23, 2026
2 checks passed
@floydkim floydkim deleted the chore/drop-rn76-and-simplify-reacthost branch February 23, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant