Chore/extract codegen parser more than one module exception

[Marcoo09]

Summary

This PR is part of #34872
This PR extracts MoreThanOneModuleInterfaceParserError exception to a separate function inside an error-utils.js file

Changelog

[Internal] [Changed] - Extract MoreThanOneModuleInterfaceParserError to a seperate function inside error-utils.js

Test Plan

[cipolleschi]

Can you rebase on top of main? The #34896 finally landed!

[analysis-bot]

Platform	Engine	Arch	Size (bytes)	Diff
android	hermes	arm64-v8a	7,777,102	+118
android	hermes	armeabi-v7a	7,180,438	+1,489
android	hermes	x86	8,091,826	+1,176
android	hermes	x86_64	8,062,787	+341
android	jsc	arm64-v8a	9,635,938	+0
android	jsc	armeabi-v7a	8,401,130	+0
android	jsc	x86	9,585,857	+0
android	jsc	x86_64	10,179,063	+0

Base commit: 49cb7f2
Branch: main

[Marcoo09]

Done @cipolleschi
Let me know any feedback, thanks!

[Marcoo09]

@cipolleschi I fixed some of the flow-checks errors. Now is missing the following one:

But I'm not able to fix because it starts to trigger the following one:

Should I type the parserType param as string, $FlowFixMe, 'Flow' | 'Typescript' or which is the correct way?

[cipolleschi]

Hi @Marcoo09!
The problem is in how you are trying to export the ParserType. As the compiler tells you, Types are erased at runtime. If you want to reuse that type, you have to export it with this syntax in the errors.js

export type ParserType

and then you have to use it with the import type syntax:

import type {ParserType} from 

In this way, it should work!

[Marcoo09]

Great @cipolleschi
I've just done the changes. Thanks!

[facebook-github-bot]

@motiz88 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: 49cb7f2
Branch: main

[cipolleschi]

@motiz88 could you please reimport this PR?

[facebook-github-bot]

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

[cipolleschi]

/rebase

[cipolleschi]

@Marcoo09 could you rebase and resolve the conflicts, please?

[Marcoo09]

@cipolleschi sorry, just rebased

[cipolleschi]

Sorry to ask again, could you rebase, please? 🙏

[github-actions]

	Warnings
⚠️	packages/react-native-codegen/src/parsers/tests/error-utils-test.js#L471 - packages/react-native-codegen/src/parsers/tests/error-utils-test.js line 471 – Describe block title is used multiple times in the same describe block. (jest/no-identical-title)
⚠️	packages/react-native-codegen/src/parsers/typescript/modules/index.js#L56 - packages/react-native-codegen/src/parsers/typescript/modules/index.js line 56 – 'MoreThanOneModuleInterfaceParserError' is assigned a value but never used. (no-unused-vars)
⚠️	packages/react-native-codegen/src/parsers/typescript/modules/index.js#L57 - packages/react-native-codegen/src/parsers/typescript/modules/index.js line 57 – 'ModuleInterfaceNotFoundParserError' is assigned a value but never used. (no-unused-vars)

Generated by 🚫 dangerJS against 9f60605

[react-native-bot]

This pull request was successfully merged by @Marcoo09 in f628edc.

^{When will my fix make it into a release? | Upcoming Releases}