Use bufout instead of concurrently

Author: kraenhansen

.github/workflows/check.yml

@@ -134,5 +134,3 @@ jobs:
             adb uninstall com.microsoft.reacttestapp || true
             # Build, install and run the app
             npm run test:android
-            # Wait a bit for the sub-process to terminate, before terminating the emulator
-            sleep 5

apps/test-app/package.json

@@ -1,5 +1,6 @@
 {
   "name": "react-native-node-api-test-app",
+  "type": "commonjs",
   "private": true,
   "version": "0.1.0",
   "scripts": {
@@ -8,8 +9,8 @@
     "build:android": "react-native bundle --entry-file index.js --platform android --dev true --bundle-output dist/main.android.jsbundle --assets-dest dist/res",
     "ios": "react-native run-ios --no-packager",
     "pod-install": "cd ios && pod install",
-    "test:android": "mocha-remote --exit-on-error -- concurrently --kill-timeout 1000 npm:start npm:android",
-    "test:ios": "mocha-remote --exit-on-error -- concurrently --kill-timeout 1000 npm:start npm:ios"
+    "test:android": "mocha-remote --exit-on-error -- tsx ./scripts/run-tests.ts android",
+    "test:ios": "mocha-remote --exit-on-error -- tsx ./scripts/run-tests.ts ios"
   },
   "dependencies": {
     "@babel/core": "^7.26.10",
@@ -23,7 +24,7 @@
     "@react-native/typescript-config": "0.79.0",
     "@rnx-kit/metro-config": "^2.0.1",
     "@types/react": "^19.0.0",
-    "concurrently": "^9.1.2",
+    "bufout": "^0.3.2",
     "ferric-example": "^0.1.0",
     "mocha": "^11.6.0",
     "mocha-remote-cli": "^1.13.1",

apps/test-app/scripts/run-tests.ts

@@ -0,0 +1,45 @@
+import assert from "node:assert/strict";
+import path from "node:path";
+
+import { spawn } from "bufout";
+
+// Ideally, we would just use "concurrently" or "npm-run-all" to run these in parallel but:
+// - "concurrently" hangs the emulator action on Ubuntu
+// - "npm-run-all" shows symptoms of not closing metro when Mocha Remote sends a SIGTERM
+
+const platform = process.argv[2];
+assert(
+  platform === "android" || platform === "ios",
+  "Platform must be 'android' or 'ios'"
+);
+
+const cwd = path.resolve(__dirname, "..");
+const env = {
+  ...process.env,
+  FORCE_COLOR: "1",
+};
+
+const metro = spawn("npx", ["react-native", "start", "--no-interactive"], {
+  cwd,
+  stdio: "inherit",
+  outputPrefix: "[metro] ",
+  env,
+});
+
+const build = spawn(
+  "npx",
+  [
+    "react-native",
+    `run-${platform}`,
+    "--no-packager",
+    ...(platform === "android" ? ["--active-arch-only"] : []),
+  ],
+  {
+    cwd,
+    stdio: "inherit",
+    outputPrefix: `[${platform}] `,
+    env,
+  }
+);
+
+Promise.all([metro, build]).catch(console.error);

apps/test-app/tsconfig.json

@@ -1,3 +1,8 @@
 {
-  "extends": "@react-native/typescript-config/tsconfig.json"
+  "extends": "@react-native/typescript-config/tsconfig.json",
+  "compilerOptions": {
+    "types": ["react-native"]
+  },
+  "files": ["App.tsx", "index.ts"],
+  "references": [{ "path": "./tsconfig.node-scripts.json" }]
 }

apps/test-app/tsconfig.node-scripts.json

@@ -0,0 +1,12 @@
+{
+  "extends": "@tsconfig/node22/tsconfig.json",
+  "compilerOptions": {
+    "composite": true,
+    "emitDeclarationOnly": true,
+    "outDir": "dist",
+    "rootDir": "scripts",
+    "types": ["node", "bufout"]
+  },
+  "include": ["scripts/**/*.ts"],
+  "exclude": []
+}