fix(webpack): Change NativeClass properties to enumerable (NativeScript#8813)

PeterStaev · web-flow · commit e4a2930c6a01 · 2020-09-03T14:36:55.000-07:00
diff --git a/packages/webpack/transformers/ns-transform-native-classes.ts b/packages/webpack/transformers/ns-transform-native-classes.ts
@@ -22,7 +22,7 @@ export default function (ctx: ts.TransformationContext) {
 		return ts.createIdentifier(
 			ts.transpileModule(node.getText().replace(/@NativeClass(\((.|\n)*?\))?/gm, ''), {
 				compilerOptions: { noEmitHelpers: true, module: ts.ModuleKind.CommonJS, target: ts.ScriptTarget.ES5 },
-			}).outputText
+			}).outputText.replace(/(Object\.defineProperty\(.*?{.*?)(enumerable:\s*false)(.*?}\))/gs, '$1enumerable: true$3')
 		);
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ export default function (ctx: ts.TransformationContext) {`
`22`	`22`	`return ts.createIdentifier(`
`23`	`23`	`ts.transpileModule(node.getText().replace(/@NativeClass(\((.\|\n)*?\))?/gm, ''), {`
`24`	`24`	`compilerOptions: { noEmitHelpers: true, module: ts.ModuleKind.CommonJS, target: ts.ScriptTarget.ES5 },`
`25`		`- }).outputText`
	`25`	`+ }).outputText.replace(/(Object\.defineProperty\(.?{.?)(enumerable:\sfalse)(.?}\))/gs, '$1enumerable: true$3')`
`26`	`26`	`);`
`27`	`27`	`}`
`28`	`28`