Simplify RendererProxy and RendererImplementation to Fabric-only (#56556)#56556
Closed
rubennorte wants to merge 2 commits into
Closed
Simplify RendererProxy and RendererImplementation to Fabric-only (#56556)#56556rubennorte wants to merge 2 commits into
rubennorte wants to merge 2 commits into
Conversation
|
@rubennorte has exported this pull request. If you are a Meta employee, you can view the originating Diff in D101353113. |
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Differential Revision: D101353113
accdcfb to
0820958
Compare
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Differential Revision: D101353113
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Differential Revision: D101353113
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Differential Revision: D101353113
0820958 to
2dd046e
Compare
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Pull Request resolved: facebook#56556 Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Differential Revision: D101353113
2dd046e to
04db788
Compare
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Pull Request resolved: facebook#56556 Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Differential Revision: D101353113
04db788 to
d8a53a4
Compare
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Differential Revision: D101353113
d8a53a4 to
a431905
Compare
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Pull Request resolved: facebook#56556 Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Differential Revision: D101353113
2757e47 to
2b2e4c7
Compare
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
2b2e4c7 to
5e476b7
Compare
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Pull Request resolved: facebook#56556 Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
5e476b7 to
66e5bad
Compare
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
66e5bad to
f62c3e8
Compare
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 22, 2026
…ebook#56556) Summary: Pull Request resolved: facebook#56556 Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
f62c3e8 to
4817807
Compare
Summary:
Migrates the remaining meaningful coverage from `TextInput-test.js` (Jest with snapshots) over to `TextInput-itest.js` (Fantom) so we can rely on a single test file that exercises the real renderer instead of a heavily-mocked one.
Changes in `TextInput-itest.js`:
- Add a `focus()` test verifying that focusing a `TextInput` unfocuses any previously focused one (mirrors Jest's "should unfocus when other TextInput is focused").
- Add a top-level `has the correct displayName` test.
- Add a new set of `props` describes that assert the props from the original Jest snapshot tests reach the mounting layer via `getRenderedOutput({props: [...]}).toJSX()`:
- `id and nativeID`: `id` → `nativeID` mapping, `nativeID` pass-through, and `id` precedence over `nativeID`.
- `testID`: pass-through.
- `aria-label and accessibilityLabel`: `aria-label` → `accessibilityLabel` mapping and `accessibilityLabel` pass-through.
- `aria-* and accessibilityState`: `aria-busy/checked/disabled/expanded/selected` mapped into `accessibilityState`.
- `accessibilityRole`: pass-through.
- `style`: `backgroundColor` propagated as a native color value.
Tests intentionally NOT ported:
- Inherited `HostInstance` method presence checks (`focus`, `blur`, `setNativeProps`, `measure`, `measureInWindow`, `measureLayout`): they are guaranteed by Flow typing on `ReactNativeElement`, exercised in `ReactNativeElement-itest.js`, and Flow's `method-unbinding` rule prevents the simple `toBeInstanceOf(Function)` check without `$FlowFixMe` (which is forbidden by project rules).
- `focus() should not do anything if the TextInput is not editable`: `TextInputState.focusTextInput` reads `textField.currentProps?.editable`, but in the new architecture `currentProps` is not exposed on the public `ReactNativeElement` instance, so the check is effectively bypassed. The Jest test only passed by monkey-patching `currentProps`. Documenting this as expected behavior in Fantom would mask a real bug.
- `should give precedence to textContentType when set`: `textContentType` is iOS-only and Fantom defaults to Android (`AndroidTextInput`).
Files removed:
- `Libraries/Components/TextInput/__tests__/TextInput-test.js`
- `Libraries/Components/TextInput/__tests__/__snapshots__/TextInput-test.js.snap`
Changelog: [Internal]
Differential Revision: D101977225
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 23, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
4817807 to
53ab47e
Compare
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 23, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 23, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 23, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
rubennorte
added a commit
to rubennorte/react-native
that referenced
this pull request
Apr 23, 2026
…ebook#56556) Summary: Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
…ebook#56556) Summary: Pull Request resolved: facebook#56556 Changelog: [internal] All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure. This commit simplifies: - RendererImplementation.js: Rewritten to only use Fabric renderer - RendererProxy.fb.js: Removed Paper-related exports - RendererProxy.js: Updated comment to reflect Fabric variants (profiling vs production) - FabricRendererImplementationWithProfiling.js: Removed Paper shims - Jest mock RendererProxy.js: Removed Paper exports Reviewed By: javache Differential Revision: D101353113
53ab47e to
ddac353
Compare
|
This pull request has been merged in 1a623a8. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary:
Changelog: [internal]
All Meta apps have migrated to Fabric, so we no longer need to support Paper in the renderer infrastructure.
This commit simplifies:
Reviewed By: javache
Differential Revision: D101353113