Add TSMethodSignature to react-native-codegen

[ZihanChen-MSFT]

Summary

Refering to "Support function signature along with function type properties in commands" in #34872

Many TypeScript programmers prefer { name(arg:string):void; } to { readonly name:(arg:string)=>void; }.

In this pull request, I updated test cases in both commands and modules to cover it, with missing implementation.

Command arguments are NamedShape<T> but the optional field is missing, it is also fixed.

Changelog

[General] [Changed] - Add TSMethodSignature to react-native-codegen

Test Plan

yarn jest react-native-codegen passed

[cipolleschi]

Nice, I like it. The code read well and it is easy to follow. Thank you for this change.

Just a double check here, flow/modules already support both types of method definitions, right?

Would you mind update one test in the flow/module to keep that checked? The same change for getArray should be enough!

[analysis-bot]

Platform	Engine	Arch	Size (bytes)	Diff
android	hermes	arm64-v8a	7,101,079	+0
android	hermes	armeabi-v7a	6,470,786	+0
android	hermes	x86	7,518,920	+0
android	hermes	x86_64	7,377,544	+0
android	jsc	arm64-v8a	8,965,158	+0
android	jsc	armeabi-v7a	7,696,551	+0
android	jsc	x86	9,027,654	+0
android	jsc	x86_64	9,505,792	+0

Base commit: 29caed2
Branch: main

[ZihanChen-MSFT]

@cipolleschi Thanks for the comment! I updated test cases for Flow too but it seems the code covers it perfectly!

[facebook-github-bot]

@cipolleschi has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[analysis-bot]

Platform	Engine	Arch	Size (bytes)	Diff
ios	-	universal	n/a	--

Base commit: 29caed2
Branch: main

[pull-bot]

PR build artifact for b6fdd84 is ready.
To use, download tarball from "Artifacts" tab in this CircleCI job then run yarn add <path to tarball> in your React Native project.