Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55696
Changelog: [Internal]
Unifies different methods of parsing doxygen text outputs into a single method, able to resolve nested references.
Reviewed By: cipolleschi
Differential Revision: D94077808
fbshipit-source-id: 494ee4da7380e17a60bbdae8e552f1c3827c1130
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55650
Changelog: [Internal]
Updates the C++ snapshot generation script to correctly handle brace-initialized values. It also adds `STORE_INITIALIZERS_IN_SNAPSHOT` which can be used to enable/disable presence of initializers in the API snapshot.
Reviewed By: cortinico
Differential Revision: D93845131
fbshipit-source-id: 06872e705305ced0c46779f4a97c674f26417ff5
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55697
Changelog: [Internal]
Implements fallback name qualification for cases where doxygen doesn't extract the full qualified name, and instead passes on the string it found in the header file.
- Adds type_qualification.py with qualify_type_str(), qualify_arguments(), and qualify_parsed_type()
- Updates member.py with close() methods for VariableMember, FunctionMember, and TypedefMember to apply type qualification
- Updates main.py with get_doxygen_params() to extract structured parameter info
Reviewed By: cipolleschi
Differential Revision: D94077825
fbshipit-source-id: 2265f5573164af1af129014082fc44fb8fceb135
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55698
Changelog: [Internal]
Refactors utils.py into a proper package structure and adds comprehensive argument string parsing utilities.
- Converts utils.py to utils/ package with submodules
- Adds argument_parsing.py with parse_arg_string() and related utilities
- Adds qualified_path.py with parse_qualified_path moved from old utils.py
- Renames test_member.py to test_parse_arg_string.py with updated tests for the new 4-tuple Argument format
Reviewed By: cipolleschi
Differential Revision: D94077819
fbshipit-source-id: 460d7a44e6d3e55580d996877ca63fa0c777bc23
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55642
Changelog: [Internal]
Adds a configuration file for the snapshot generation, allowing to create multiple views of the API and different flavors.
It also adds option to include codegen in the snapshot generation. Codegen needs to be executed first, and the path to its output needs to be passed when running the generation.
Reviewed By: cipolleschi
Differential Revision: D92958731
fbshipit-source-id: 42780d25b2bc04f213e697cca0b6bde07d94ca59
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55695
Changelog: [Internal]
Adds support for handling concepts in the C++ API parser.
Reviewed By: NickGerleman
Differential Revision: D94074054
fbshipit-source-id: 5e8ef0b0d4d1eeb93554558bb7f80cd67d622a72
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55693
Changelog: [Internal]
Adds support for handling template params in the C++ API parser.
Reviewed By: cipolleschi
Differential Revision: D94074045
fbshipit-source-id: b758a8ceb7f95e8a1acf0983afa0c65aeec6b756
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55694
Changelog: [Internal]
Adds support for handling type aliases in the c++ API snapshot
Reviewed By: cipolleschi
Differential Revision: D94074032
fbshipit-source-id: 8af8bc88e884f4b1b7f7394d1316776cc9d50d6b
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55692
Changelog: [Internal]
Adds support for handling functions and methods in the C++ API parser
Reviewed By: cipolleschi
Differential Revision: D94073981
fbshipit-source-id: 1c22a8cd4584868a220974335b321d523e6a3dc2
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55691
Changelog: [Internal]
Adds support for handling enums in the C++ API parser.
Reviewed By: cipolleschi
Differential Revision: D94073960
fbshipit-source-id: 2c726f5ac5c33c66950463760e2592064d812400
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/55640
Changelog: [Internal]
Adds a simple script going over the doxygen output and a test harness for snapshot testing.
Reviewed By: cipolleschi
Differential Revision: D90959837
fbshipit-source-id: fde0c1116b7c0fc10e43484e2b57ead79af706b8
Summary:
This replaces `glob@^7.0.0` with `tinyglobby@^0.2.15`. `glob@7` has been deprecated for a while and some versions after had security notices released for them. The plan is to backport this PR to `0.81.x` and onwards.
> [!NOTE]
> This is a stopgap solution until `fs.glob` becomes generally available with the EOL of Node v20
Succeeds:
- https://github.com/facebook/react-native/issues/54669
- https://github.com/facebook/react-native/issues/48875
## Changelog:
[GENERAL] [SECURITY] - Replace `glob@^7.0.0` with `tinyglobby@^0.2.15`
Pull Request resolved: https://github.com/facebook/react-native/pull/54737
Test Plan:
- Ran all modified commands manually and `pod install in `rn-tester`
- NOTE: `ios-prebuild`-related scripts haven't been run manually yet
Reviewed By: robhogan
Differential Revision: D88069145
Pulled By: huntie
fbshipit-source-id: 0c455342a4c6d1d6605fd09fe47b418e5d751491
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/54704
This class is coming from Legacy Architecture and is no longer used so can be cleaned up.
Changelog:
[Internal] [Changed] -
Reviewed By: mdvacca
Differential Revision: D87925037
fbshipit-source-id: 30faffef0da2b7055a6d381b41dc72bead2dd8de
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/54684
This diff removes the `RCTTurboModuleSyncVoidMethodsEnabled` feature flag and all related code that allowed TurboModule void methods to execute synchronously.
Changelog: [Internal]
Reviewed By: philIip
Differential Revision: D87865883
fbshipit-source-id: fa64e8105e8329a0b20bb1718dbc01b17464cd06
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/53028
Changelog: [internal]
Now that we have mocking at a more fundamental level (`HighResTimeStamp` API) we can replace other timing mocks with that one.
This does it for `PerformanceEntryReporter` and the `NativePerformance` module.
Reviewed By: hoxyq
Differential Revision: D79557640
fbshipit-source-id: 86579b8bb586190ab7cc8721f30e60b3ef789798
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/52771
Changelog: [Internal]
It seems we can simply use a std::shared_ptr here
(This was some initial over-engineering which isn't needed as it turns out)
Reviewed By: cipolleschi
Differential Revision: D78771904
fbshipit-source-id: 2925c424d2061ca727636c683ec783ed56e3f0c9
Summary:
Pull Request resolved: https://github.com/facebook/react-native/pull/52543
Prefixing "eager" to the name makes it less confusing. React native already has main queue modules: they're just lazy.
Changelog: [Internal]
Reviewed By: lunaleaps
Differential Revision: D78109287
fbshipit-source-id: 5fa6095d2dd8fcf74fdda64e05483eb487bc8f56
Jakub Piasecki