Fix the bad minification due to ternary expression

Author: arthur791004

projects/packages/jetpack-mu-wpcom/bin/sync-newspack-blocks-formatter.js

@@ -0,0 +1,46 @@
+module.exports = function transformer( file, api ) {
+	const j = api.jscodeshift;
+
+	return j( file.source )
+		.find( j.ConditionalExpression )
+		.filter( path => {
+			const { consequent, alternate } = path.value;
+
+			// Check if the consequent or alternate is a function call to targetFunctionName
+			return (
+				isFunctionCallWithName( consequent, '__' ) || isFunctionCallWithName( alternate, '__' )
+			);
+		} )
+		.forEach( path => {
+			const { alternate } = path.value;
+
+			// Add the new argument to the alternate function call
+			const newAlternateArguments = [
+				...alternate.arguments,
+				j.literal( 0 ), // Create a literal AST node for the new argument
+			];
+
+			// Create a new CallExpression node with the updated arguments
+			const newAlternate = j.callExpression( alternate.callee, newAlternateArguments );
+
+			// Update the alternate branch of the ternary expression
+			path.value.alternate = newAlternate;
+		} )
+		.toSource();
+};
+
+/**
+ * Whether the node is a function call with the specific name
+ *
+ * @param node - The AST node
+ * @param functionName - The function name we want to check
+ */
+function isFunctionCallWithName( node, functionName ) {
+	return (
+		node &&
+		node.type === 'CallExpression' &&
+		node.callee &&
+		node.callee.type === 'Identifier' &&
+		node.callee.name === functionName
+	);
+}

projects/packages/jetpack-mu-wpcom/bin/sync-newspack-blocks.sh

@@ -147,6 +147,9 @@ sed "${sedi[@]}" -e "s| function| Function|g" "$TARGET/types/index.d.ts"
 echo "Changing JS textdomain to match jetpack-mu-wpcom..."
 pnpm --package=eslint@8.57.0 dlx eslint --no-ignore --rule '"@wordpress/i18n-text-domain":["error",{"allowedTextDomain":"jetpack-mu-wpcom"}]' --fix $TARGET > /dev/null
 
+echo "Changing JS translation function call to avoid bad minification..."
+pnpm --package=jscodeshift dlx jscodeshift -t ./bin/sync-newspack-blocks-formatter.js --extensions=js $TARGET
+
 echo "Changing PHP textdomain to match jetpack-mu-wpcom..."
 ../../../vendor/bin/phpcbf --standard=./.phpcs.dir.xml --filter=../../../vendor/automattic/jetpack-phpcs-filter/src/PhpcsFilter.php --runtime-set jetpack-filter-no-ignore -q $TARGET