fix: print crash artifact paths relative to cwd

Add a shared tools formatPath helper and use it when rendering native crash artifact paths in Jest error output.

Author: V3RON

actions/shared/index.cjs

@@ -633,8 +633,8 @@ function getErrorMap() {
 
 // ../../node_modules/zod/dist/esm/v3/helpers/parseUtil.js
 var makeIssue = (params) => {
-  const { data, path: path7, errorMaps, issueData } = params;
-  const fullPath = [...path7, ...issueData.path || []];
+  const { data, path: path8, errorMaps, issueData } = params;
+  const fullPath = [...path8, ...issueData.path || []];
   const fullIssue = {
     ...issueData,
     path: fullPath
@@ -750,11 +750,11 @@ var errorUtil;
 
 // ../../node_modules/zod/dist/esm/v3/types.js
 var ParseInputLazyPath = class {
-  constructor(parent, value, path7, key) {
+  constructor(parent, value, path8, key) {
     this._cachedPath = [];
     this.parent = parent;
     this.data = value;
-    this._path = path7;
+    this._path = path8;
     this._key = key;
   }
   get path() {
@@ -4345,14 +4345,17 @@ var HarnessError = class extends Error {
 var import_node_path3 = __toESM(require("path"), 1);
 var import_node_fs3 = __toESM(require("fs"), 1);
 
+// ../tools/dist/path.js
+var import_node_path4 = __toESM(require("path"), 1);
+
 // ../tools/dist/crash-artifacts.js
 var import_node_fs4 = __toESM(require("fs"), 1);
-var import_node_path4 = __toESM(require("path"), 1);
-var DEFAULT_ARTIFACT_ROOT = import_node_path4.default.join(process.cwd(), ".harness", "crash-reports");
+var import_node_path5 = __toESM(require("path"), 1);
+var DEFAULT_ARTIFACT_ROOT = import_node_path5.default.join(process.cwd(), ".harness", "crash-reports");
 
 // ../tools/dist/harness-artifacts.js
 var import_node_fs5 = __toESM(require("fs"), 1);
-var import_node_path5 = __toESM(require("path"), 1);
+var import_node_path6 = __toESM(require("path"), 1);
 
 // ../plugins/dist/utils.js
 var isHookTree = (value) => {
@@ -4512,13 +4515,13 @@ var ConfigLoadError = class extends HarnessError {
 };
 
 // ../config/dist/reader.js
-var import_node_path6 = __toESM(require("path"), 1);
+var import_node_path7 = __toESM(require("path"), 1);
 var import_node_fs6 = __toESM(require("fs"), 1);
 var import_node_module2 = require("module");
 var import_meta = {};
 var extensions = [".js", ".mjs", ".cjs", ".json"];
 var importUp = async (dir, name) => {
-  const filePath = import_node_path6.default.join(dir, name);
+  const filePath = import_node_path7.default.join(dir, name);
   for (const ext of extensions) {
     const filePathWithExt = `${filePath}${ext}`;
     if (import_node_fs6.default.existsSync(filePathWithExt)) {
@@ -4539,16 +4542,16 @@ var importUp = async (dir, name) => {
       } catch (error) {
         if (error instanceof ZodError) {
           const validationErrors = error.errors.map((err) => {
-            const path7 = err.path.length > 0 ? ` at "${err.path.join(".")}"` : "";
-            return `${err.message}${path7}`;
+            const path8 = err.path.length > 0 ? ` at "${err.path.join(".")}"` : "";
+            return `${err.message}${path8}`;
           });
           throw new ConfigValidationError(filePathWithExt, validationErrors);
         }
         throw error;
       }
     }
   }
-  const parentDir = import_node_path6.default.dirname(dir);
+  const parentDir = import_node_path7.default.dirname(dir);
   if (parentDir === dir) {
     throw new ConfigNotFoundError(dir);
   }
@@ -4563,7 +4566,7 @@ var getConfig = async (dir) => {
 };
 
 // src/shared/index.ts
-var import_node_path7 = __toESM(require("path"));
+var import_node_path8 = __toESM(require("path"));
 var import_node_fs7 = __toESM(require("fs"));
 var getHostAndroidSystemImageArch = () => {
   switch (process.arch) {
@@ -4632,7 +4635,7 @@ var run = async () => {
     if (!runnerInput) {
       throw new Error("Runner input is required");
     }
-    const projectRoot = projectRootInput ? import_node_path7.default.resolve(projectRootInput) : process.cwd();
+    const projectRoot = projectRootInput ? import_node_path8.default.resolve(projectRootInput) : process.cwd();
     console.info(`Loading React Native Harness config from: ${projectRoot}`);
     const { config, projectRoot: resolvedProjectRoot } = await getConfig(
       projectRoot
@@ -4646,7 +4649,7 @@ var run = async () => {
       throw new Error("GITHUB_OUTPUT environment variable is not set");
     }
     const resolvedRunner = getResolvedRunner(runner);
-    const relativeProjectRoot = import_node_path7.default.relative(process.cwd(), resolvedProjectRoot) || ".";
+    const relativeProjectRoot = import_node_path8.default.relative(process.cwd(), resolvedProjectRoot) || ".";
     const output = `config=${JSON.stringify(
       resolvedRunner
     )}

packages/jest/src/__tests__/crash-monitor.test.ts

@@ -1,4 +1,5 @@
 import { describe, expect, it, vi } from 'vitest';
+import path from 'node:path';
 import type {
   AppSession,
   AppSessionEvent,
@@ -166,7 +167,10 @@ describe('createCrashMonitor', () => {
       '/tmp/.harness/crash-reports/crash-logcat.txt'
     );
     expect(error.message).toContain(
-      'Harness extracted the crash log: /tmp/.harness/crash-reports/crash-logcat.txt'
+      `Harness extracted the crash log: ${path.relative(
+        process.cwd(),
+        '/tmp/.harness/crash-reports/crash-logcat.txt'
+      )}`
     );
   });
 

packages/jest/src/__tests__/errors.test.ts

@@ -1,4 +1,5 @@
 import { describe, expect, it } from 'vitest';
+import path from 'node:path';
 import { NativeCrashError, PlatformReadyTimeoutError } from '../errors.js';
 
 describe('PlatformReadyTimeoutError', () => {
@@ -13,29 +14,48 @@ describe('NativeCrashError', () => {
   it('reports the extracted crash log path when available', () => {
     const error = new NativeCrashError('/tmp/crash.harness.ts', {
       phase: 'execution',
-      artifactPath: '/tmp/.harness/crash-reports/crash.ips',
+      artifactPath: path.join(
+        process.cwd(),
+        '.harness',
+        'crash-reports',
+        'crash.ips'
+      ),
     });
 
     expect(error.message).toContain(
-      'Harness extracted the crash log: /tmp/.harness/crash-reports/crash.ips'
+      `Harness extracted the crash log: ${path.join(
+        '.harness',
+        'crash-reports',
+        'crash.ips'
+      )}`
     );
   });
 
   it('lists enrichment artifact paths when available', () => {
     const error = new NativeCrashError('/tmp/crash.harness.ts', {
       phase: 'execution',
-      artifactPath: '/tmp/.harness/crash-reports/logcat.txt',
+      artifactPath: path.join(
+        process.cwd(),
+        '.harness',
+        'crash-reports',
+        'logcat.txt'
+      ),
       enrichmentArtifacts: [
         {
           artifactType: 'dropbox-native-crash',
-          artifactPath: '/tmp/.harness/crash-reports/dropbox-native-crash.txt',
+          artifactPath: path.join(
+            process.cwd(),
+            '.harness',
+            'crash-reports',
+            'dropbox-native-crash.txt'
+          ),
         },
       ],
     });
 
     expect(error.message).toContain('Additional crash artifacts:');
     expect(error.message).toContain(
-      '/tmp/.harness/crash-reports/dropbox-native-crash.txt'
+      path.join('.harness', 'crash-reports', 'dropbox-native-crash.txt')
     );
   });
 

packages/jest/src/errors.ts

@@ -1,4 +1,4 @@
-import { HarnessError } from '@react-native-harness/tools';
+import { formatPath, HarnessError } from '@react-native-harness/tools';
 import type { AppCrashDetails } from '@react-native-harness/platforms';
 export {
   StartupStallError,
@@ -79,14 +79,14 @@ const buildNativeCrashMessage = ({
 
   lines.push(
     artifactPath
-      ? `Harness extracted the crash log: ${artifactPath}`
+      ? `Harness extracted the crash log: ${formatPath(artifactPath)}`
       : "Harness couldn't extract the crash log."
   );
 
   if (enrichmentArtifacts && enrichmentArtifacts.length > 0) {
     lines.push('Additional crash artifacts:');
     for (const artifact of enrichmentArtifacts) {
-      lines.push(`  - ${artifact.artifactPath}`);
+      lines.push(`  - ${formatPath(artifact.artifactPath)}`);
     }
   }
 

packages/tools/src/__tests__/path.test.ts

@@ -0,0 +1,26 @@
+import { describe, expect, it } from 'vitest';
+import path from 'node:path';
+import { formatPath } from '../path.js';
+
+describe('formatPath', () => {
+  it('formats absolute paths relative to the current working directory', () => {
+    expect(
+      formatPath(
+        path.join('/project', '.harness', 'crash-reports', 'log.txt'),
+        {
+          cwd: '/project',
+        }
+      )
+    ).toBe(path.join('.harness', 'crash-reports', 'log.txt'));
+  });
+
+  it('leaves relative paths unchanged', () => {
+    expect(formatPath(path.join('.harness', 'crash-reports', 'log.txt'))).toBe(
+      path.join('.harness', 'crash-reports', 'log.txt')
+    );
+  });
+
+  it('formats the working directory as a dot', () => {
+    expect(formatPath('/project', { cwd: '/project' })).toBe('.');
+  });
+});

packages/tools/src/index.ts

@@ -8,6 +8,7 @@ export * from './react-native.js';
 export * from './error.js';
 export * from './events.js';
 export * from './packages.js';
+export * from './path.js';
 export * from './crash-artifacts.js';
 export * from './harness-artifacts.js';
 export * from './regex.js';

packages/tools/src/path.ts

@@ -0,0 +1,11 @@
+import path from 'node:path';
+
+export type FormatPathOptions = {
+  cwd?: string;
+};
+
+export const formatPath = (
+  filePath: string,
+  { cwd = process.cwd() }: FormatPathOptions = {}
+) =>
+  path.isAbsolute(filePath) ? path.relative(cwd, filePath) || '.' : filePath;