Revert "Change target to ES5 for targeting Hermes Engine, disable noEmitHelpers, don't crash if registerOnGlobalContext fails, and disable the special transform for NativeClass decorator"

This reverts commit 7e0497c8b15c96e440a9809642e0f3d441e4e871.

packages/core/globals/index.ts

@@ -15,27 +15,23 @@ interface ExtensionMap {
 }
 
 function registerOnGlobalContext(moduleName: string, exportName: string): void {
-	try {
-		Object.defineProperty(global, exportName, {
-			get: function () {
-				// We do not need to cache require() call since it is already cached in the runtime.
-				const m = global.loadModule(moduleName);
+	Object.defineProperty(global, exportName, {
+		get: function () {
+			// We do not need to cache require() call since it is already cached in the runtime.
+			const m = global.loadModule(moduleName);
 
-				// Redefine the property to make sure the above code is executed only once.
-				const resolvedValue = m[exportName];
-				Object.defineProperty(global, exportName, {
-					value: resolvedValue,
-					configurable: true,
-					writable: true,
-				});
+			// Redefine the property to make sure the above code is executed only once.
+			const resolvedValue = m[exportName];
+			Object.defineProperty(global, exportName, {
+				value: resolvedValue,
+				configurable: true,
+				writable: true,
+			});
 
-				return resolvedValue;
-			},
-			configurable: true,
-		});
-	} catch (_e) {
-		// TODO: On Hermes, some globals are non-configurable and we cannot override them.
-	}
+			return resolvedValue;
+		},
+		configurable: true,
+	});
 }
 
 /**

packages/core/tsconfig.lib.json

@@ -2,7 +2,7 @@
 	"extends": "./tsconfig.json",
 	"compilerOptions": {
 		"noEmitOnError": true,
-		"noEmitHelpers": false,
+		"noEmitHelpers": true,
 		"declaration": true,
 		"ignoreDeprecations": "5.0",
 		"removeComments": false,

packages/core/utils/platform-check.ts

@@ -2,8 +2,7 @@
  * @internal Util used for exporting opposing platform utils and warning the user if they are trying to access them.
  */
 export function platformCheck(parent?: string) {
-	// @ts-ignore
-	if (globalThis.__DEV__) {
+	if (__DEV__) {
 		return new Proxy(
 			{},
 			{

packages/webpack5/src/configuration/base.ts

@@ -274,7 +274,7 @@ export default function (config: Config, env: IWebpackEnv = _env): Config {
 			},
 			getCustomTransformers() {
 				return {
-					before: [],
+					before: [require('../transformers/NativeClass').default],
 				};
 			},
 		});

packages/webpack5/src/transformers/NativeClass/index.ts

@@ -5,8 +5,6 @@ import ts from 'typescript';
  */
 export default function (ctx: ts.TransformationContext) {
 	function isNativeClassExtension(node: ts.ClassDeclaration) {
-		return false;
-
 		let decorators: Readonly<ts.Decorator[]>;
 
 		if ('canHaveDecorators' in ts && ts.canHaveDecorators(node)) {
@@ -37,7 +35,7 @@ export default function (ctx: ts.TransformationContext) {
 					node.getText().replace(/@NativeClass(\((.|\n)*?\))?/gm, ''),
 					{
 						compilerOptions: {
-							noEmitHelpers: false,
+							noEmitHelpers: true,
 							module: ts.ModuleKind.ESNext,
 							target: ts.ScriptTarget.ES5,
 							experimentalDecorators: true,

packages/webpack5/tsconfig.json

@@ -2,7 +2,7 @@
 	"compilerOptions": {
 		"rootDir": ".",
 		"baseUrl": ".",
-		"target": "es5",
+		"target": "es2017",
 		"module": "commonjs",
 		"outDir": "./dist",
 		"declaration": true,

tsconfig.base.json

@@ -2,16 +2,15 @@
 	"compileOnSave": false,
 	"compilerOptions": {
 		"rootDir": ".",
-		"target": "ES5",
+		"target": "ES2020",
 		"module": "esnext",
-		"downlevelIteration": true,
 		"sourceMap": true,
 		"declaration": true,
 		"moduleResolution": "node",
 		"emitDecoratorMetadata": true,
 		"experimentalDecorators": true,
 		"noEmitOnError": true,
-		"noEmitHelpers": false,
+		"noEmitHelpers": true,
 		"diagnostics": true,
 		"skipLibCheck": true,
 		"skipDefaultLibCheck": true,