Skip to content

Migrate NativeSampleTurboModule to codegen integration - RFC (#56723)#56723

Open
christophpurrer wants to merge 1 commit intofacebook:mainfrom
christophpurrer:export-D104323543
Open

Migrate NativeSampleTurboModule to codegen integration - RFC (#56723)#56723
christophpurrer wants to merge 1 commit intofacebook:mainfrom
christophpurrer:export-D104323543

Conversation

@christophpurrer
Copy link
Copy Markdown
Contributor

@christophpurrer christophpurrer commented May 8, 2026

Summary:

Replace hand-written spec files for NativeSampleTurboModule with proper codegen integration by removing exclusion rules and configuring the module in package.json. This eliminates ~1600 lines of manual boilerplate (Java, C++, Objective-C) across Android, iOS, and macOS platforms, allowing the codegen system to generate these files automatically from the JavaScript spec.

Changelog: [Internal]

Differential Revision: D104323543

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 8, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented May 8, 2026

@christophpurrer has exported this pull request. If you are a Meta employee, you can view the originating Diff in D104323543.

@facebook-github-tools facebook-github-tools Bot added p: Facebook Partner: Facebook Partner labels May 8, 2026
@meta-codesync meta-codesync Bot changed the title Migrate NativeSampleTurboModule to codegen integration - RFC Migrate NativeSampleTurboModule to codegen integration - RFC (#56723) May 8, 2026
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request May 8, 2026
…k#56723)

Summary:

Replace hand-written spec files for `NativeSampleTurboModule` with proper codegen integration by removing exclusion rules and configuring the module in `package.json`. This eliminates ~1600 lines of manual boilerplate (Java, C++, Objective-C) across Android, iOS, and macOS platforms, allowing the codegen system to generate these files automatically from the JavaScript spec.

Changelog: [Internal]

Differential Revision: D104323543
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request May 8, 2026
…k#56723)

Summary:

Replace hand-written spec files for `NativeSampleTurboModule` with proper codegen integration by removing exclusion rules and configuring the module in `package.json`. This eliminates ~1600 lines of manual boilerplate (Java, C++, Objective-C) across Android, iOS, and macOS platforms, allowing the codegen system to generate these files automatically from the JavaScript spec.

Changelog: [Internal]

Differential Revision: D104323543
christophpurrer added a commit to christophpurrer/react-native-macos that referenced this pull request May 8, 2026
…k#56723)

Summary:

Replace hand-written spec files for `NativeSampleTurboModule` with proper codegen integration by removing exclusion rules and configuring the module in `package.json`. This eliminates ~1600 lines of manual boilerplate (Java, C++, Objective-C) across Android, iOS, and macOS platforms, allowing the codegen system to generate these files automatically from the JavaScript spec.

Changelog: [Internal]

Differential Revision: D104323543
…k#56723)

Summary:

Replace hand-written spec files for `NativeSampleTurboModule` with proper codegen integration by removing exclusion rules and configuring the module in `package.json`. This eliminates ~1600 lines of manual boilerplate (Java, C++, Objective-C) across Android, iOS, and macOS platforms, allowing the codegen system to generate these files automatically from the JavaScript spec.

Changelog: [Internal]

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

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant