fix: bound harness CLI processes

.nx/version-plans/version-plan-1785922736004.md

@@ -0,0 +1,5 @@
+---
+__default__: patch
+---
+
+Harness now bounds owned CLI processes so cancelled runs do not leave console, logcat, or XCTest children behind.

CHANGELOG.md

@@ -1,3 +1,14 @@
+## 1.4.1 (2026-08-04)
+
+### 🩹 Fixes
+
+- Published packages exclude test outputs and local build artifacts. ([#179](https://github.com/callstackincubator/react-native-harness/pull/179))
+- Harness tracks pending work while preserving native Promise instances so native modules return their settled values on Hermes. ([#180](https://github.com/callstackincubator/react-native-harness/pull/180))
+
+### ❤️ Thank You
+
+- Szymon Chmal @V3RON
+
 ## 1.4.0 (2026-08-03)
 
 ### 🚀 Features

packages/babel-preset/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-harness/babel-preset",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "commonjs",
   "main": "./dist/index.js",
   "module": "./dist/index.js",

packages/bridge/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-harness/bridge",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",

packages/bundler-metro/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@react-native-harness/bundler-metro",
   "description": "Metro bundler for React Native Harness",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",

packages/bundler-metro/src/factory.ts

@@ -167,9 +167,22 @@ export const getMetroInstance = async (
     socket.on('close', () => sockets.delete(socket));
   });
 
-  abortSignal.throwIfAborted();
+  const closeServer = () =>
+    new Promise<void>((resolve) => {
+      server.close(() => resolve());
+      server.closeAllConnections();
+      for (const socket of sockets) {
+        socket.destroy();
+      }
+    });
 
-  await ready;
+  try {
+    abortSignal.throwIfAborted();
+    await ready;
+  } catch (error) {
+    await closeServer();
+    throw error;
+  }
 
   metroLogger.debug('Metro server is running');
 
@@ -265,15 +278,10 @@ export const getMetroInstance = async (
     },
     getPrewarmState: () => prewarmState,
     isBuildInFlight: () => buildsInFlight > 0,
-    dispose: () =>
-      new Promise<void>((resolve) => {
-        reporter.removeListener(onBundleRequestObserved);
-        reporter.removeListener(onBuildEvent);
-        server.close(() => resolve());
-        server.closeAllConnections();
-        for (const socket of sockets) {
-          socket.destroy();
-        }
-      }),
+    dispose: () => {
+      reporter.removeListener(onBundleRequestObserved);
+      reporter.removeListener(onBuildEvent);
+      return closeServer();
+    },
   };
 };

packages/cache/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-harness/cache",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",

packages/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-harness/cli",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",

packages/config/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-harness/config",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",

packages/coverage-ios/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@react-native-harness/coverage-ios",
   "description": "Native iOS code coverage support for React Native Harness.",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "module",
   "exports": {
     "./package.json": "./package.json",

packages/github-action/package.json

@@ -2,7 +2,7 @@
   "name": "@react-native-harness/github-action",
   "private": true,
   "description": "GitHub Actions for React Native Harness",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "commonjs",
   "exports": {
     "./package.json": "./package.json"

packages/jest/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-harness/jest",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",

packages/jest/src/harness-session.ts

@@ -38,6 +38,7 @@ import {
 import {
   createCrashArtifactWriter,
   createDiagnostics,
+  delay,
   logger,
   getTimeoutSignal,
   ensureSignalsDeliverable,
@@ -94,6 +95,25 @@ const ignorePromiseRejection = () => undefined;
 const isBridgeDisconnectError = (error: unknown) =>
   error instanceof Error && error.message === 'App bridge disconnected';
 const TEST_RUN_BRIDGE_STABILITY_WAIT_MS = 500;
+const DISPOSE_HOOK_TIMEOUT_MS = 5_000;
+
+const waitForBoundedSettlement = async <T>(
+  promise: Promise<T>,
+  timeoutMs: number
+): Promise<PromiseSettledResult<T> | null> => {
+  const timeout = delay(timeoutMs);
+  try {
+    return await Promise.race([
+      promise.then(
+        (value) => ({ status: 'fulfilled', value }) as const,
+        (reason) => ({ status: 'rejected', reason }) as const
+      ),
+      timeout.promise.then(() => null),
+    ]);
+  } finally {
+    timeout.cancel();
+  }
+};
 
 type DisconnectObservableBridge = {
   readonly connection: AppConnection | null;
@@ -487,7 +507,11 @@ export const createHarnessSession = async (
   // child the runner threads it into.
   ensureSignalsDeliverable();
   const sessionController = new AbortController();
-  const onEarlySignal = () => sessionController.abort();
+  let receivedEarlySignal = false;
+  const onEarlySignal = () => {
+    receivedEarlySignal = true;
+    sessionController.abort();
+  };
   process.once('SIGTERM', onEarlySignal);
   process.once('SIGINT', onEarlySignal);
 
@@ -584,6 +608,11 @@ export const createHarnessSession = async (
 
     let metroInstance: MetroInstance;
     let platformInstance: HarnessPlatformRunner;
+    const initialized = {
+      metro: null as MetroInstance | null,
+      platform: null as HarnessPlatformRunner | null,
+    };
+    let initializationCancelled = false;
     // Noop until the Metro instance resolves and the deriver is attached.
     let disposeMetroDiagnostics: () => void = () => undefined;
 
@@ -601,6 +630,10 @@ export const createHarnessSession = async (
             },
             sessionController.signal,
           ).then((instance) => {
+            if (initializationCancelled) {
+              return instance.dispose().then(() => instance);
+            }
+            initialized.metro = instance;
             sessionLogger.debug('Metro initialized');
             // Attach the Metro diagnostics deriver before the eager prewarm
             // fires so its bundle build/request events land in the trace.
@@ -638,6 +671,10 @@ export const createHarnessSession = async (
                     diagnostics,
                   } satisfies HarnessPlatformInitOptions),
                 ).then((instance) => {
+                  if (initializationCancelled) {
+                    return instance.dispose().then(() => instance);
+                  }
+                  initialized.platform = instance;
                   sessionLogger.debug('platform runner initialized');
                   return instance;
                 });
@@ -647,10 +684,16 @@ export const createHarnessSession = async (
         ),
       ]);
     } catch (error) {
-      // Only bridge and the Metro diagnostics listener need cleanup here;
-      // leases are released by the outer catch.
+      // Roll back every resource whose initialization won a concurrent race.
+      // Leases are released by the outer catch.
+      initializationCancelled = true;
+      sessionController.abort();
       disposeMetroDiagnostics();
-      await bridge.dispose();
+      await Promise.allSettled([
+        bridge.dispose(),
+        initialized.metro?.dispose(),
+        initialized.platform?.dispose(),
+      ]);
       throw error;
     }
 
@@ -759,7 +802,11 @@ export const createHarnessSession = async (
       const disposeSpan = diagnostics.start('dispose.total', { reason });
       let hookError: unknown;
 
-      try {
+      // Abort only cancels finite/in-flight work. Ownership remains explicit
+      // below, so this cannot race a platform or app-session disposal path.
+      sessionController.abort();
+
+      const hookWork = (async () => {
         await hooks.drain();
         await pluginManager.callHook('harness:after-run', {
           runId: currentRun?.runId,
@@ -777,8 +824,17 @@ export const createHarnessSession = async (
           error: currentRun?.error,
         });
         await hooks.drain();
-      } catch (error) {
-        hookError = error;
+      })();
+      const hookResult = await waitForBoundedSettlement(
+        hookWork,
+        DISPOSE_HOOK_TIMEOUT_MS
+      );
+      if (hookResult === null) {
+        hookError = new Error(
+          `Timed out after ${DISPOSE_HOOK_TIMEOUT_MS}ms while running disposal hooks.`
+        );
+      } else if (hookResult.status === 'rejected') {
+        hookError = hookResult.reason;
       }
 
       if (runtimeConfig.forwardClientLogs) {
@@ -809,26 +865,23 @@ export const createHarnessSession = async (
         }
       }
 
-      let cleanupError: unknown;
-      try {
-        // The app session (and its logcat/device log stream) must be fully
-        // torn down before the platform is disposed: platform disposal can
-        // shut down the device/emulator, and doing that concurrently with
-        // the app session's own cleanup races the device's log stream
-        // against the device disappearing out from under it.
-        await Promise.all([crashMonitor.dispose(), disposeCurrentAppSession()]);
-        await Promise.all([
-          bridge.dispose(),
-          platformInstance.dispose(),
-          metroInstance.dispose(),
-          metroPortLease?.release(),
-        ]);
-      } catch (error) {
-        cleanupError = error;
-      } finally {
-        await resourceLease.release();
-        sessionController.abort();
-      }
+      // The app session (and its log streams) must finish before platform
+      // disposal can remove the underlying device. Run independent cleanup
+      // all-settled so one failure cannot strand another owned resource.
+      const appCleanup = await Promise.allSettled([
+        crashMonitor.dispose(),
+        disposeCurrentAppSession(),
+      ]);
+      const infrastructureCleanup = await Promise.allSettled([
+        bridge.dispose(),
+        platformInstance.dispose(),
+        metroInstance.dispose(),
+        metroPortLease?.release(),
+      ]);
+      const leaseCleanup = await Promise.allSettled([resourceLease.release()]);
+      const cleanupError = [...appCleanup, ...infrastructureCleanup, ...leaseCleanup]
+        .find((result): result is PromiseRejectedResult => result.status === 'rejected')
+        ?.reason;
 
       sessionLogger.debug('session resources disposed');
 
@@ -1102,6 +1155,9 @@ export const createHarnessSession = async (
     // (e.g. a platform init call abandoned after a readiness timeout race).
     sessionController.abort();
     await Promise.allSettled([resourceLease.release(), metroPortLease?.release()]);
+    if (receivedEarlySignal) {
+      process.exit(1);
+    }
     throw error;
   }
 };

packages/metro/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-harness/metro",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "commonjs",
   "main": "./dist/index.js",
   "module": "./dist/index.js",

packages/platform-android/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@react-native-harness/platform-android",
   "description": "Android platform for React Native Harness",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",