Fix JS build on Windows (#1648)

Summary:
Pull Request resolved: https://github.com/facebook/yoga/pull/1648

Node made a breaking change in a security release for 18/20 where `spawn()` no longer loads `.bat` files by default. 69ffc6d50d. Execute command in shell.

Reviewed By: javache

Differential Revision: D56230965

fbshipit-source-id: 52e9bd8a76664bd07ea25b6355ac54fcb24cbb9a

.github/workflows/validate-tests.yml

@@ -5,7 +5,6 @@ on:
   push:
     branches:
       - main
-      - 'release-*'
   workflow_dispatch:
 
 jobs:

Yoga.podspec

@@ -6,7 +6,7 @@
 
 Pod::Spec.new do |spec|
   spec.name = 'Yoga'
-  spec.version = '3.0.3'
+  spec.version = '3.0.4'
   spec.license =  { :type => 'MIT', :file => "LICENSE" }
   spec.homepage = 'https://yogalayout.dev/'
   spec.documentation_url = 'https://yogalayout.dev/docs'

gradle.properties

@@ -11,4 +11,4 @@ android.useAndroidX=true
 
 org.gradle.jvmargs=-Xmx1536M
 
-VERSION_NAME=3.0.3
+VERSION_NAME=3.0.4

javascript/just.config.cjs

@@ -13,11 +13,11 @@ const {
   logger,
   jestTask,
   option,
-  parallel,
   series,
   spawn,
   task,
   tscTask,
+  copyTask,
 } = require('just-scripts');
 
 const {existsSync} = require('fs');
@@ -58,16 +58,32 @@ task('prepack-package-json', async () => {
   const packageJsonContents = await readFile(packageJsonPath);
   const packageJson = JSON.parse(packageJsonContents.toString('utf-8'));
 
-  recursiveReplace(packageJson, /(.\/src\/.*)\.ts/, '$1.js');
-  packageJson.types = packageJson.main.replace(/(.\/src\/.*)\.js/, '$1.d.ts');
+  packageJson.main = packageJson.main.replace(
+    /^.\/src\/(.*)\.ts/,
+    './dist/src/$1.js',
+  );
+  packageJson.types = packageJson.main.replace(/(.*)\.js/, '$1.d.ts');
+
+  recursiveReplace(
+    packageJson.exports,
+    /^.\/src\/(.*)\.ts/,
+    './dist/src/$1.js',
+  );
+
   await writeFile(packageJsonPath, JSON.stringify(packageJson, null, 2));
 });
 
 task(
   'prepack',
   series(
-    parallel('build', tscTask({emitDeclarationOnly: true})),
-    babelTransformTask({dir: 'src'}),
+    'build',
+    copyTask({paths: ['binaries'], dest: 'dist/binaries'}),
+    tscTask({
+      emitDeclarationOnly: true,
+      rootDir: '.',
+      declarationDir: 'dist',
+    }),
+    babelTransformTask({src: 'src', dst: 'dist/src'}),
     'prepack-package-json',
   ),
 );
@@ -85,14 +101,14 @@ function recursiveReplace(obj, pattern, replacement) {
 function babelTransformTask(opts) {
   return () => {
     const args = [
-      opts.dir,
+      opts.src,
       '--source-maps',
       '--out-dir',
-      opts.dir,
+      opts.dst,
       '--extensions',
       '.js,.cjs,.mjs,.ts,.cts,.mts',
     ];
-    logger.info(`Transforming "${path.resolve(opts.dir)}"`);
+    logger.info(`Transforming "${path.resolve(opts.src)}"`);
 
     return spawn(node, [require.resolve('@babel/cli/bin/babel'), ...args], {
       cwd: __dirname,
@@ -152,9 +168,9 @@ function installEmsdkTask() {
       {stdio: 'inherit'},
     );
 
-    await spawn(emsdkBin, ['install', emsdkVersion], {stdio: 'inherit'});
+    await spawnShell(emsdkBin, ['install', emsdkVersion], {stdio: 'inherit'});
 
-    await spawn(emsdkBin, ['activate', emsdkVersion], {
+    await spawnShell(emsdkBin, ['activate', emsdkVersion], {
       stdio: logger.enableVerbose ? 'inherit' : 'ignore',
     });
   };
@@ -200,7 +216,7 @@ function emcmakeGenerateTask() {
     ];
     logger.info(['emcmake', ...args].join(' '));
 
-    return spawn(emcmakeBin, args, {
+    return spawnShell(emcmakeBin, args, {
       stdio: logger.enableVerbose ? 'inherit' : 'ignore',
     });
   };
@@ -218,7 +234,7 @@ function cmakeBuildTask(opts) {
     ];
     logger.info(['cmake', ...args].join(' '));
 
-    return spawn(cmake, args, {stdio: 'inherit'});
+    return spawnShell(cmake, args, {stdio: 'inherit'});
   };
 }
 
@@ -230,8 +246,13 @@ function clangFormatTask(opts) {
     ];
     logger.info(['clang-format', ...args].join(' '));
 
-    return spawn(node, [require.resolve('clang-format'), ...args], {
+    return spawnShell(node, [require.resolve('clang-format'), ...args], {
       stdio: 'inherit',
     });
   };
 }
+
+function spawnShell(cmd, args, opts) {
+  // https://github.com/nodejs/node/issues/52554
+  return spawn(cmd, args, {...opts, shell: true});
+}

javascript/package.json

@@ -1,6 +1,6 @@
 {
   "name": "yoga-layout",
-  "version": "3.0.3",
+  "version": "3.0.4",
   "description": "An embeddable and performant flexbox layout engine with bindings for multiple languages",
   "license": "MIT",
   "author": "Meta Open Source",
@@ -11,12 +11,14 @@
   },
   "type": "module",
   "main": "./src/index.ts",
+  "types": "./src/index.ts",
   "exports": {
     ".": "./src/index.ts",
     "./load": "./src/load.ts"
   },
   "files": [
-    "binaries/**",
+    "dist/binaries/**",
+    "dist/src/**",
     "src/**"
   ],
   "scripts": {

website-next/package.json

@@ -27,7 +27,7 @@
     "react": "^18.0.0",
     "react-dom": "^18.0.0",
     "react-live": "^4.1.5",
-    "yoga-layout": "3.0.3"
+    "yoga-layout": "3.0.4"
   },
   "devDependencies": {
     "@docusaurus/module-type-aliases": "3.0.0",