@react-native-harness/jest 1.5.0 → 1.7.0

package/dist/index.js

@@ -1,4 +1,5 @@
 import { createHarnessSession } from './harness-session.js';
+import { getWatchSession } from './watch-session.js';
 import { executeRun } from './execute-run.js';
 import { HarnessError } from '@react-native-harness/tools';
 import { getErrorMessage } from './logs.js';
@@ -6,7 +7,6 @@ export default class JestHarness {
     isSerial = true;
     supportsEventEmitters = true;
     #globalConfig;
-    #session = null;
     #listeners = new Map();
     constructor(globalConfig) {
         this.#globalConfig = globalConfig;
@@ -27,11 +27,12 @@ export default class JestHarness {
             throw new Error('Parallel test running is not supported');
         }
         const isWatchMode = this.#globalConfig.watch || this.#globalConfig.watchAll;
+        let session = null;
         try {
-            if (!this.#session) {
-                this.#session = await createHarnessSession(this.#globalConfig);
-            }
-            await executeRun(this.#session, tests, watcher, this.#emit, this.#globalConfig);
+            session = isWatchMode
+                ? await getWatchSession(this.#globalConfig)
+                : await createHarnessSession(this.#globalConfig);
+            await executeRun(session, tests, watcher, this.#emit, this.#globalConfig);
         }
         catch (error) {
             if (error instanceof HarnessError) {
@@ -41,8 +42,7 @@ export default class JestHarness {
         }
         finally {
             if (!isWatchMode) {
-                await this.#session?.dispose();
-                this.#session = null;
+                await session?.dispose();
             }
         }
     }

package/dist/jest-preset.d.ts

@@ -1,5 +1,8 @@
 declare const _default: {
     runner: string;
+    maxWorkers: number;
+    modulePathIgnorePatterns: string[];
+    watchman: boolean;
 };
 export default _default;
 //# sourceMappingURL=jest-preset.d.ts.map

package/dist/jest-preset.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"jest-preset.d.ts","sourceRoot":"","sources":["../src/jest-preset.ts"],"names":[],"mappings":";;;AAAA,wBAEE"}
+{"version":3,"file":"jest-preset.d.ts","sourceRoot":"","sources":["../src/jest-preset.ts"],"names":[],"mappings":";;;;;;AAAA,wBAoBE"}

package/dist/jest-preset.js

@@ -1,3 +1,21 @@
 export default {
     runner: '@react-native-harness/jest',
+    // Tests run serially on the device, so Jest only uses its workers to build
+    // the haste map. On a cold cache or after any file change that starts a pool
+    // of `availableParallelism() - 1` processes, which roughly doubles Jest's
+    // CPU time without making the run faster.
+    maxWorkers: 1,
+    // Build and cache output never contains tests, but it can hold tens of
+    // thousands of files. Jest checks these patterns before descending into a
+    // directory, so its crawl skips them entirely. Jest appends a project's own
+    // patterns to the preset's.
+    modulePathIgnorePatterns: [
+        '/\\.harness(/|$)',
+        '/ios/(Pods|build)(/|$)',
+        '/android/(build|\\.gradle|app/build|app/\\.cxx)(/|$)',
+    ],
+    // With the build output skipped, Jest's own crawl is cheaper than querying
+    // watchman, which costs two CLI process spawns per run and still lists every
+    // file. Metro keeps using watchman.
+    watchman: false,
 };

package/dist/relative-path.d.ts

@@ -0,0 +1,8 @@
+/**
+ * Returns `filePath` relative to `rootDir`, always using `/` as the separator.
+ *
+ * These paths leave the host (they are sent to the device and used in Metro
+ * bundle URLs and plugin hooks), so they must not depend on the host OS.
+ */
+export declare const toRelativePosixPath: (rootDir: string, filePath: string) => string;
+//# sourceMappingURL=relative-path.d.ts.map

package/dist/relative-path.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"relative-path.d.ts","sourceRoot":"","sources":["../src/relative-path.ts"],"names":[],"mappings":"AAEA;;;;;GAKG;AACH,eAAO,MAAM,mBAAmB,GAAI,SAAS,MAAM,EAAE,UAAU,MAAM,KAAG,MACZ,CAAC"}

package/dist/relative-path.js

@@ -0,0 +1,8 @@
+import path from 'node:path';
+/**
+ * Returns `filePath` relative to `rootDir`, always using `/` as the separator.
+ *
+ * These paths leave the host (they are sent to the device and used in Metro
+ * bundle URLs and plugin hooks), so they must not depend on the host OS.
+ */
+export const toRelativePosixPath = (rootDir, filePath) => path.relative(rootDir, filePath).split(path.sep).join('/');

package/dist/resource-lock.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"resource-lock.d.ts","sourceRoot":"","sources":["../src/resource-lock.ts"],"names":[],"mappings":"AAIA,OAAO,EAAU,KAAK,aAAa,EAAE,MAAM,6BAA6B,CAAC;AAiBzE,MAAM,MAAM,0BAA0B,GAAG;IACvC,MAAM,CAAC,EAAE,WAAW,CAAC;IACrB,MAAM,CAAC,EAAE,MAAM,IAAI,CAAC;IACpB,cAAc,CAAC,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;CAC9C,CAAC;AAEF,MAAM,MAAM,aAAa,GAAG;IAC1B,OAAO,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC9B,CAAC;AAEF,MAAM,MAAM,mBAAmB,GAAG;IAChC,OAAO,EAAE,CACP,GAAG,EAAE,MAAM,EACX,OAAO,CAAC,EAAE,0BAA0B,KACjC,OAAO,CAAC,aAAa,CAAC,CAAC;CAC7B,CAAC;AAEF,KAAK,0BAA0B,GAAG;IAChC,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,aAAa,CAAC;IACvB,eAAe,CAAC,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC;CAC5C,CAAC;AAqQF,eAAO,MAAM,yBAAyB,GACpC,UAAS,0BAA+B,KACvC,mBA+KF,CAAC"}
+{"version":3,"file":"resource-lock.d.ts","sourceRoot":"","sources":["../src/resource-lock.ts"],"names":[],"mappings":"AAIA,OAAO,EAAU,KAAK,aAAa,EAAE,MAAM,6BAA6B,CAAC;AAiBzE,MAAM,MAAM,0BAA0B,GAAG;IACvC,MAAM,CAAC,EAAE,WAAW,CAAC;IACrB,MAAM,CAAC,EAAE,MAAM,IAAI,CAAC;IACpB,cAAc,CAAC,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAC;CAC9C,CAAC;AAEF,MAAM,MAAM,aAAa,GAAG;IAC1B,OAAO,EAAE,MAAM,OAAO,CAAC,IAAI,CAAC,CAAC;CAC9B,CAAC;AAEF,MAAM,MAAM,mBAAmB,GAAG;IAChC,OAAO,EAAE,CACP,GAAG,EAAE,MAAM,EACX,OAAO,CAAC,EAAE,0BAA0B,KACjC,OAAO,CAAC,aAAa,CAAC,CAAC;CAC7B,CAAC;AAEF,KAAK,0BAA0B,GAAG;IAChC,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,kBAAkB,CAAC,EAAE,MAAM,CAAC;IAC5B,GAAG,CAAC,EAAE,MAAM,CAAC;IACb,MAAM,CAAC,EAAE,aAAa,CAAC;IACvB,eAAe,CAAC,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC;CAC5C,CAAC;AAsSF,eAAO,MAAM,yBAAyB,GACpC,UAAS,0BAA+B,KACvC,mBA6KF,CAAC"}

package/dist/resource-lock.js

@@ -35,6 +35,28 @@ const isMissingFileError = (error) => {
         typeof error.code === 'string' &&
         error.code === 'ENOENT');
 };
+// On Windows, a file being deleted by another process can briefly fail
+// reads and deletes with EPERM/EBUSY instead of ENOENT.
+const isTransientFsError = (error) => {
+    return (error instanceof Error &&
+        'code' in error &&
+        (error.code === 'EPERM' || error.code === 'EBUSY'));
+};
+const TRANSIENT_FS_RETRIES = 5;
+const TRANSIENT_FS_RETRY_DELAY_MS = 20;
+const retryTransientFsError = async (operation) => {
+    for (let attempt = 1;; attempt += 1) {
+        try {
+            return await operation();
+        }
+        catch (error) {
+            if (!isTransientFsError(error) || attempt >= TRANSIENT_FS_RETRIES) {
+                throw error;
+            }
+            await new Promise((resolve) => setTimeout(resolve, TRANSIENT_FS_RETRY_DELAY_MS));
+        }
+    }
+};
 const isExclusiveCreateError = (error) => {
     return (error instanceof Error &&
         'code' in error &&
@@ -46,7 +68,7 @@ const ensureLockDirectories = async (paths) => {
 };
 const readJsonFile = async (filePath) => {
     try {
-        const content = await fs.readFile(filePath, 'utf8');
+        const content = await retryTransientFsError(() => fs.readFile(filePath, 'utf8'));
         return JSON.parse(content);
     }
     catch (error) {
@@ -69,7 +91,7 @@ const writeJsonFileAtomic = async (filePath, value) => {
 };
 const removeFileIfPresent = async (filePath) => {
     try {
-        await fs.rm(filePath, { force: true });
+        await retryTransientFsError(() => fs.rm(filePath, { force: true }));
     }
     catch (error) {
         if (!isMissingFileError(error)) {
@@ -200,7 +222,7 @@ export const createResourceLockManager = (options = {}) => {
             await fs.writeFile(ticketPath, JSON.stringify(metadata), 'utf8');
             scopedLogger.debug('queued ticket %s for key %s', ticketId, key);
             let heartbeatTimer = null;
-            let heartbeatInFlight = false;
+            let heartbeatInFlight = null;
             let released = false;
             let didNotifyWait = false;
             const waitStartedAt = Date.now();
@@ -213,6 +235,9 @@ export const createResourceLockManager = (options = {}) => {
                     clearInterval(heartbeatTimer);
                     heartbeatTimer = null;
                 }
+                // Let an in-flight refresh finish so it cannot recreate owner.json
+                // after we remove it.
+                await heartbeatInFlight;
                 const owner = await readJsonFile(paths.ownerFilePath);
                 if (owner?.ticketId === ticketId) {
                     await removeFileIfPresent(paths.ownerFilePath);
@@ -221,40 +246,35 @@ export const createResourceLockManager = (options = {}) => {
                 scopedLogger.debug('released ticket %s for key %s', ticketId, key);
             };
             const startHeartbeat = () => {
-                heartbeatTimer = setInterval(async () => {
-                    if (heartbeatInFlight) {
-                        return;
-                    }
-                    heartbeatInFlight = true;
+                const refreshHeartbeat = async () => {
                     try {
                         const nextHeartbeatAt = Date.now();
                         const owner = await readJsonFile(paths.ownerFilePath);
                         if (released || owner?.ticketId !== ticketId) {
                             return;
                         }
-                        const nextMetadata = {
+                        await writeJsonFileAtomic(paths.ownerFilePath, {
                             ...owner,
                             heartbeatAt: nextHeartbeatAt,
-                        };
-                        if (released) {
-                            return;
-                        }
-                        try {
-                            await writeJsonFileAtomic(paths.ownerFilePath, nextMetadata);
-                            scopedLogger.debug('refreshed heartbeat for ticket %s', ticketId);
-                        }
-                        catch (error) {
-                            // A failed refresh is not fatal by design: the lock goes stale
-                            // and another holder reclaims it. Swallow it so a transient
-                            // write error (e.g. the owner file racing a concurrent release,
-                            // or an EPERM on Windows when the directory is being torn down)
-                            // never surfaces as an unhandled rejection from this interval.
-                            scopedLogger.debug('heartbeat refresh for ticket %s failed: %s', ticketId, error instanceof Error ? error.message : String(error));
-                        }
+                        });
+                        scopedLogger.debug('refreshed heartbeat for ticket %s', ticketId);
                     }
-                    finally {
-                        heartbeatInFlight = false;
+                    catch (error) {
+                        // A failed refresh is not fatal by design: the lock goes stale
+                        // and another holder reclaims it. Swallow it so a transient
+                        // error (e.g. the owner file racing a concurrent release, or an
+                        // EPERM on Windows while a file is being deleted) never surfaces
+                        // as an unhandled rejection from this interval.
+                        scopedLogger.debug('heartbeat refresh for ticket %s failed: %s', ticketId, error instanceof Error ? error.message : String(error));
                     }
+                };
+                heartbeatTimer = setInterval(() => {
+                    if (heartbeatInFlight) {
+                        return;
+                    }
+                    heartbeatInFlight = refreshHeartbeat().finally(() => {
+                        heartbeatInFlight = null;
+                    });
                 }, heartbeatIntervalMs);
                 heartbeatTimer.unref?.();
             };

package/dist/run.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"run.d.ts","sourceRoot":"","sources":["../src/run.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,MAAM,IAAI,UAAU,EAAE,MAAM,aAAa,CAAC;AACxD,OAAO,EAAE,UAAU,IAAI,cAAc,EAAE,MAAM,mBAAmB,CAAC;AACjE,OAAO,KAAK,EACV,eAAe,IAAI,sBAAsB,EAE1C,MAAM,8BAA8B,CAAC;AACtC,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,sBAAsB,CAAC;AAmE3D,MAAM,MAAM,kBAAkB,GAAG,CAAC,OAAO,EAAE;IACzC,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,cAAc,CAAC;IACxB,aAAa,EAAE,UAAU,CAAC,aAAa,CAAC;IACxC,YAAY,EAAE,UAAU,CAAC,YAAY,CAAC;CACvC,KAAK,OAAO,CAAC;IACZ,UAAU,EAAE,cAAc,CAAC;IAC3B,aAAa,EAAE,sBAAsB,CAAC;IACtC,gBAAgB,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,MAAM,CAAC;CAClB,CAAC,CAAC;AAEH,eAAO,MAAM,kBAAkB,EAAE,kBA0FhC,CAAC"}
+{"version":3,"file":"run.d.ts","sourceRoot":"","sources":["../src/run.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,IAAI,UAAU,EAAE,MAAM,aAAa,CAAC;AACxD,OAAO,EAAE,UAAU,IAAI,cAAc,EAAE,MAAM,mBAAmB,CAAC;AACjE,OAAO,KAAK,EACV,eAAe,IAAI,sBAAsB,EAE1C,MAAM,8BAA8B,CAAC;AACtC,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,sBAAsB,CAAC;AAoE3D,MAAM,MAAM,kBAAkB,GAAG,CAAC,OAAO,EAAE;IACzC,QAAQ,EAAE,MAAM,CAAC;IACjB,OAAO,EAAE,cAAc,CAAC;IACxB,aAAa,EAAE,UAAU,CAAC,aAAa,CAAC;IACxC,YAAY,EAAE,UAAU,CAAC,YAAY,CAAC;CACvC,KAAK,OAAO,CAAC;IACZ,UAAU,EAAE,cAAc,CAAC;IAC3B,aAAa,EAAE,sBAAsB,CAAC;IACtC,gBAAgB,EAAE,MAAM,CAAC;IACzB,QAAQ,EAAE,MAAM,CAAC;CAClB,CAAC,CAAC;AAEH,eAAO,MAAM,kBAAkB,EAAE,kBA0FhC,CAAC"}

package/dist/run.js

@@ -1,7 +1,7 @@
-import path from 'node:path';
 import { formatResultsErrors } from 'jest-message-util';
 import { toTestResult } from './toTestResult.js';
 import { formatHarnessErrorMessage } from './format-harness-error.js';
+import { toRelativePosixPath } from './relative-path.js';
 // Helper function to flatten nested test suites into a flat array of tests with hierarchy
 const flattenTests = (suiteResult, ancestorTitles = []) => {
     const tests = [];
@@ -55,10 +55,10 @@ const calculateStats = (tests) => {
 };
 export const runHarnessTestFile = async ({ testPath, globalConfig, projectConfig, session, }) => {
     const start = Date.now();
-    const relativeTestPath = path.relative(globalConfig.rootDir, testPath);
+    const relativeTestPath = toRelativePosixPath(globalConfig.rootDir, testPath);
     // Extract setup files from Jest config and convert to relative paths
-    const setupFiles = projectConfig.setupFiles?.map((setupFile) => path.relative(globalConfig.rootDir, setupFile));
-    const setupFilesAfterEnv = projectConfig.setupFilesAfterEnv?.map((setupFile) => path.relative(globalConfig.rootDir, setupFile));
+    const setupFiles = projectConfig.setupFiles?.map((setupFile) => toRelativePosixPath(globalConfig.rootDir, setupFile));
+    const setupFilesAfterEnv = projectConfig.setupFilesAfterEnv?.map((setupFile) => toRelativePosixPath(globalConfig.rootDir, setupFile));
     const testTimeout = session.config.testTimeout ??
         projectConfig
             .testTimeout ??

package/dist/watch-session.d.ts

@@ -0,0 +1,6 @@
+import type { Config } from 'jest-runner';
+import { type HarnessSession } from './harness-session.js';
+export declare const getWatchSession: (globalConfig: Config.GlobalConfig) => Promise<HarnessSession>;
+/** @internal Test-only: forget the module-level session without disposing. */
+export declare const resetWatchSessionForTests: () => void;
+//# sourceMappingURL=watch-session.d.ts.map

package/dist/watch-session.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"watch-session.d.ts","sourceRoot":"","sources":["../src/watch-session.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,aAAa,CAAC;AAC1C,OAAO,EAEL,KAAK,cAAc,EACpB,MAAM,sBAAsB,CAAC;AAqB9B,eAAO,MAAM,eAAe,GAC1B,cAAc,MAAM,CAAC,YAAY,KAChC,OAAO,CAAC,cAAc,CAgCxB,CAAC;AAEF,8EAA8E;AAC9E,eAAO,MAAM,yBAAyB,QAAO,IAE5C,CAAC"}

package/dist/watch-session.js

@@ -0,0 +1,37 @@
+import { createHarnessSession, } from './harness-session.js';
+let current = null;
+// Everything createHarnessSession reads from the Jest config.
+const getSessionKey = (globalConfig) => JSON.stringify([globalConfig.rootDir, Boolean(globalConfig.collectCoverage)]);
+export const getWatchSession = (globalConfig) => {
+    const key = getSessionKey(globalConfig);
+    if (current?.key === key) {
+        return current.promise;
+    }
+    const stale = current;
+    // The entry is installed synchronously so concurrent callers share it, even
+    // while a stale session is still being disposed.
+    const entry = {
+        key,
+        promise: (async () => {
+            if (stale) {
+                // A failed disposal must not fail the run.
+                await stale.promise
+                    .then((session) => session.dispose())
+                    .catch(() => undefined);
+            }
+            return createHarnessSession(globalConfig);
+        })(),
+    };
+    current = entry;
+    // Never cache a failed creation; the next run retries.
+    entry.promise.catch(() => {
+        if (current === entry) {
+            current = null;
+        }
+    });
+    return entry.promise;
+};
+/** @internal Test-only: forget the module-level session without disposing. */
+export const resetWatchSessionForTests = () => {
+    current = null;
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-harness/jest",
-  "version": "1.5.0",
+  "version": "1.7.0",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",
@@ -39,13 +39,13 @@
     "jest-util": "^30.2.0",
     "tslib": "^2.3.0",
     "yargs": "^17.7.2",
-    "@react-native-harness/bridge": "1.5.0",
-    "@react-native-harness/config": "1.5.0",
-    "@react-native-harness/plugins": "1.5.0",
-    "@react-native-harness/cache": "1.5.0",
-    "@react-native-harness/bundler-metro": "1.5.0",
-    "@react-native-harness/platforms": "1.5.0",
-    "@react-native-harness/tools": "1.5.0"
+    "@react-native-harness/config": "1.7.0",
+    "@react-native-harness/tools": "1.7.0",
+    "@react-native-harness/platforms": "1.7.0",
+    "@react-native-harness/bundler-metro": "1.7.0",
+    "@react-native-harness/plugins": "1.7.0",
+    "@react-native-harness/cache": "1.7.0",
+    "@react-native-harness/bridge": "1.7.0"
   },
   "devDependencies": {
     "@types/yargs": "^17.0.32",

package/src/cli-args.ts

@@ -1,25 +1,17 @@
-import yargs from 'yargs';
+import { harnessCliOptions } from '@react-native-harness/config';
+import yargs, { type InferredOptionTypes } from 'yargs';
 import { hideBin } from 'yargs/helpers';
 
-export type HarnessCliArgs = {
-  harnessRunner?: string;
-  metroPort?: number;
-};
+export type HarnessCliArgs = InferredOptionTypes<typeof harnessCliOptions>;
 
 export const getAdditionalCliArgs = (): HarnessCliArgs => {
   const argv = yargs(hideBin(process.argv))
-    .option('harnessRunner', {
-      type: 'string',
-      description: 'Specify which Harness runner to use',
-    })
-    .option('metroPort', {
-      type: 'number',
-      description: 'Override the Metro bundler port',
-    })
+    .options(harnessCliOptions)
     .strict(false)
     .help(false)
     .version(false)
     .exitProcess(false)
+    .fail(false)
     .parseSync();
 
   return {

package/src/diagnostics/index.ts

@@ -1,2 +1,13 @@
 export { observeMetroEvents, observeBridgeEvents } from './derivers.js';
-export { writeTraceFile, printSummary } from './report.js';
+export {
+  writeTraceFile,
+  appendToTraceFile,
+  printSummary,
+  createDiagnosticsAppender,
+  summarizeRunKpis,
+  writeSummaryFile,
+  appendStepSummary,
+} from './report.js';
+export type { DiagnosticsAppender, RunKpis } from './report.js';
+export { startSessionTelemetry } from './telemetry.js';
+export type { SessionTelemetry } from './telemetry.js';