@deeeed/metamask-harness 0.23.0 → 0.23.1

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## Unreleased
 
+## 0.23.1 - 2026-07-29
+
+### Fixed
+
+- Mobile recipe evidence no longer triggers Metro Fast Refresh: harness-managed Metro ignores checkout-local `temp` artifacts, and screenshot scratch files stay outside the watched project.
+
 ## 0.23.0 - 2026-07-28
 
 ### Added

package/adapters/manifest.json

@@ -42,6 +42,14 @@
       "inputs": "--target --port --log --pid-file [--workers] [--clear]",
       "outputs": "Metro PID/launch metadata and redirected log; exit 0/1/2"
     },
+    {
+      "id": "mobile/metro-config",
+      "entry": "adapters/mobile/metro-config.cjs",
+      "kind": "module",
+      "purpose": "Preserve the product Metro config while excluding harness-owned checkout temp artifacts.",
+      "inputs": "base Metro config; env MM_HARNESS_METRO_PROJECT_ROOT, MM_HARNESS_METRO_BASE_CONFIG",
+      "outputs": "merged Metro config with resolver.blockList for <projectRoot>/temp"
+    },
     {
       "id": "mobile/stop-metro",
       "entry": "adapters/mobile/stop-metro.sh",

package/adapters/mobile/metro-config.cjs

@@ -0,0 +1,93 @@
+'use strict';
+
+const fs = require('node:fs');
+const { createRequire } = require('node:module');
+const path = require('node:path');
+const { pathToFileURL } = require('node:url');
+
+function escapeRegExp(value) {
+  return value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+}
+
+function appendBlockList(blockList, pattern) {
+  if (Array.isArray(blockList)) return [...blockList, pattern];
+  return blockList instanceof RegExp ? [blockList, pattern] : [pattern];
+}
+
+function blockListFlags(blockList) {
+  const first = Array.isArray(blockList) ? blockList[0] : blockList;
+  return first instanceof RegExp ? first.flags : '';
+}
+
+function resolveExpoConfigLoader(projectRoot) {
+  const requireFromProject = createRequire(path.join(projectRoot, 'package.json'));
+  let requireUtilsPath;
+  try {
+    requireUtilsPath = requireFromProject.resolve('@expo/require-utils');
+  } catch (error) {
+    if (error?.code !== 'MODULE_NOT_FOUND') throw error;
+    return null;
+  }
+  const { loadModuleSync } = requireFromProject(requireUtilsPath);
+  return typeof loadModuleSync === 'function' ? loadModuleSync : null;
+}
+
+async function loadProductConfig(configPath, baseConfig, projectRoot) {
+  let loaded;
+  const expoConfigLoader = resolveExpoConfigLoader(projectRoot);
+  if (expoConfigLoader) {
+    loaded = expoConfigLoader(configPath);
+  } else {
+    try {
+      loaded = require(configPath);
+    } catch (error) {
+      if (error?.code !== 'ERR_REQUIRE_ESM') throw error;
+      loaded = await import(pathToFileURL(configPath).href);
+    }
+  }
+
+  const moduleValue = await loaded;
+  const exported =
+    moduleValue?.__esModule ||
+    moduleValue?.[Symbol.toStringTag] === 'Module'
+      ? moduleValue.default
+      : moduleValue;
+  const config = await exported;
+  return typeof config === 'function' ? config(baseConfig) : config;
+}
+
+module.exports = async function harnessMetroConfig(baseConfig) {
+  const projectRoot = path.resolve(
+    process.env.MM_HARNESS_METRO_PROJECT_ROOT || process.cwd(),
+  );
+  const productConfigPath = path.resolve(
+    process.env.MM_HARNESS_METRO_BASE_CONFIG ||
+      path.join(projectRoot, 'metro.config.js'),
+  );
+  let productConfig = baseConfig;
+  if (fs.existsSync(productConfigPath)) {
+    productConfig = await loadProductConfig(
+      productConfigPath,
+      baseConfig,
+      projectRoot,
+    );
+  }
+
+  const tempRoot = path.join(projectRoot, 'temp');
+  const existingBlockList =
+    productConfig.resolver?.blockList ?? baseConfig.resolver?.blockList;
+  const tempPattern = new RegExp(
+    `^${escapeRegExp(tempRoot)}(?:${escapeRegExp(path.sep)}|$)`,
+    blockListFlags(existingBlockList),
+  );
+  return {
+    ...productConfig,
+    resolver: {
+      ...productConfig.resolver,
+      blockList: appendBlockList(
+        existingBlockList,
+        tempPattern,
+      ),
+    },
+  };
+};

package/adapters/mobile/start-metro.sh

@@ -49,6 +49,7 @@ fi
 # only ever inspect or signal a Metro listening on $PORT.
 # shellcheck disable=SC1091
 . "$SCRIPT_DIR/lib/metro-listener.sh"
+MOBILE_METRO_REQUIRED_ENV="${MOBILE_METRO_REQUIRED_ENV:-MM_INFURA_PROJECT_ID} MM_HARNESS_METRO_PROJECT_ROOT EXPO_OVERRIDE_METRO_CONFIG"
 
 # RECIPE_RUNTIME_DIR (relative, validated) makes runtime state per-run so jobs
 # sharing one checkout do not collide on metro.log / metro.pid / metro.tmux.
@@ -98,6 +99,13 @@ set +a
 export EXPO_NO_TYPESCRIPT_SETUP=1
 export WATCHER_PORT="$(printf '%q' "$PORT")" METRO_PORT="$(printf '%q' "$PORT")"
 export METRO_MAX_WORKERS="$(printf '%q' "$METRO_WORKERS")"
+export MM_HARNESS_METRO_PROJECT_ROOT="$(printf '%q' "$TARGET")"
+if [ -n "\${EXPO_OVERRIDE_METRO_CONFIG:-}" ]; then
+  export MM_HARNESS_METRO_BASE_CONFIG="\$EXPO_OVERRIDE_METRO_CONFIG"
+else
+  export MM_HARNESS_METRO_BASE_CONFIG=$(printf '%q' "$TARGET/metro.config.js")
+fi
+export EXPO_OVERRIDE_METRO_CONFIG="$(printf '%q' "$SCRIPT_DIR/metro-config.cjs")"
 yarn expo start --port "$(printf '%q' "$PORT")"$clear_flag 2>&1 | tee -a "$(printf '%q' "$LOG_FILE")"
 EOF
   chmod +x "$runner"
@@ -184,6 +192,9 @@ if ! start_metro_tmux; then
     export EXPO_NO_TYPESCRIPT_SETUP=1
     export WATCHER_PORT="${PORT}" METRO_PORT="${PORT}"
     export METRO_MAX_WORKERS="${METRO_WORKERS}"
+    export MM_HARNESS_METRO_PROJECT_ROOT="$TARGET"
+    export MM_HARNESS_METRO_BASE_CONFIG="${EXPO_OVERRIDE_METRO_CONFIG:-$TARGET/metro.config.js}"
+    export EXPO_OVERRIDE_METRO_CONFIG="$SCRIPT_DIR/metro-config.cjs"
     launcher_args=(
       --target "$TARGET"
       --port "$PORT"

package/library/actions/mobile/platform/bridge.mjs

@@ -1,8 +1,9 @@
 import { randomUUID } from 'node:crypto';
 import { constants as fsConstants } from 'node:fs';
-import { mkdir, open, readFile, rename, rm, writeFile } from 'node:fs/promises';
+import { mkdir, mkdtemp, open, readFile, rename, rm, writeFile } from 'node:fs/promises';
 import { execFile, spawn } from 'node:child_process';
 import { promisify } from 'node:util';
+import os from 'node:os';
 import path from 'node:path';
 import { fileURLToPath } from 'node:url';
 import bridgeErrors from '../../../../adapters/mobile/bridge-runtime/lib/bridge-errors.cjs';
@@ -420,7 +421,7 @@ export async function simulatorScreenshot(input, relPath) {
   await rm(absolute, { force: true });
   let temporaryDirectory;
   try {
-    temporaryDirectory = await createScreenshotTemporaryDirectory(absolute);
+    temporaryDirectory = await createScreenshotTemporaryDirectory();
     const captured = path.join(temporaryDirectory, 'captured.png');
     await createPrivateScreenshotFile(captured);
     let result;
@@ -446,7 +447,7 @@ export async function simulatorScreenshot(input, relPath) {
         `  Next: mm-harness doctor --adapter mobile --device ${target} --json`,
       );
     }
-    await publishPrivateScreenshot(png, temporaryDirectory, absolute);
+    await publishPrivateScreenshot(png, absolute);
     await rm(temporaryDirectory, { recursive: true, force: true });
   } catch (error) {
     await removeScreenshotResidue(temporaryDirectory, absolute);
@@ -477,7 +478,7 @@ async function androidScreenshot(input, relPath) {
   await rm(absolute, { force: true });
   let temporaryDirectory;
   try {
-    temporaryDirectory = await createScreenshotTemporaryDirectory(absolute);
+    temporaryDirectory = await createScreenshotTemporaryDirectory();
     const adbSerial = await resolveAndroidScreenshotSerial(input);
     let result;
     try {
@@ -501,7 +502,7 @@ async function androidScreenshot(input, relPath) {
         `  Next: mm-harness doctor --adapter mobile --device ${adbSerial} --json`,
       );
     }
-    await publishPrivateScreenshot(result.stdout, temporaryDirectory, absolute);
+    await publishPrivateScreenshot(result.stdout, absolute);
     await rm(temporaryDirectory, { recursive: true, force: true });
     return {
       path: relative,
@@ -672,10 +673,8 @@ function pngCrc32(bytes, start, end) {
   return (crc ^ 0xffffffff) >>> 0;
 }
 
-async function createScreenshotTemporaryDirectory(absolute) {
-  const temporary = path.join(path.dirname(absolute), `.${path.basename(absolute)}.${process.pid}.${randomUUID()}.tmp`);
-  await mkdir(temporary, { mode: 0o700 });
-  return temporary;
+async function createScreenshotTemporaryDirectory() {
+  return mkdtemp(path.join(os.tmpdir(), 'mm-harness-screenshot-'));
 }
 
 async function createPrivateScreenshotFile(file) {
@@ -706,21 +705,28 @@ async function readPrivateScreenshotFile(file) {
   }
 }
 
-async function publishPrivateScreenshot(png, temporaryDirectory, absolute) {
-  const publication = path.join(temporaryDirectory, 'validated.png');
-  const handle = await open(
-    publication,
-    fsConstants.O_WRONLY | fsConstants.O_CREAT | fsConstants.O_EXCL | fsConstants.O_NOFOLLOW,
-    0o600,
+async function publishPrivateScreenshot(png, absolute) {
+  const publication = path.join(
+    path.dirname(absolute),
+    `.${path.basename(absolute)}.${process.pid}.${randomUUID()}.tmp`,
   );
+  let handle;
   try {
+    handle = await open(
+      publication,
+      fsConstants.O_WRONLY | fsConstants.O_CREAT | fsConstants.O_EXCL | fsConstants.O_NOFOLLOW,
+      0o600,
+    );
     const stats = await handle.stat();
     if (!stats.isFile() || stats.nlink !== 1) throw new Error('validated screenshot is not a private regular file');
     await handle.writeFile(png);
     await handle.sync();
+    await handle.close();
+    handle = undefined;
     await rename(publication, absolute);
   } finally {
-    await handle.close();
+    await handle?.close();
+    await rm(publication, { force: true });
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@deeeed/metamask-harness",
-  "version": "0.23.0",
+  "version": "0.23.1",
   "type": "module",
   "bin": {
     "mm-harness": "bin/mm-harness"