@deeeed/metamask-harness 0.3.1 → 0.3.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.3.2 - 2026-07-04
+
+### Fixed
+- **IMP-21: FORCE_COLOR-safe pod install** — the mobile pod-triggering leaves (`yarn-setup.sh` running `yarn setup`, and `open-device.sh` running the native `yarn start:ios|android` build) now run with `FORCE_COLOR=0` / `NO_COLOR=1`. VisionCamera's podspec probes `node --print require.resolve('react-native-worklets-core')` and treats any output other than the exact string `undefined` as "found"; an inherited `FORCE_COLOR` made node emit a colorized `undefined`, enabling FrameProcessors and hard-failing on the missing worklets pod. Setup/native builds are now reliable from a `FORCE_COLOR` shell.
+- **IMP-22: `--watcher-port` beats `.js.env WATCHER_PORT`** — `verify.sh` port resolution now honors an explicit `WATCHER_PORT` in the process env (which carries `mm-harness launch --watcher-port N`) over the target's `.js.env`. Precedence is flag > process env > `.js.env` > `8081` default, so a run can be pointed at an alternate Metro/CDP port without editing a shared slot's `.js.env`.
+
 ## 0.3.1 - 2026-07-03
 
 ### Fixed

package/adapters/mobile/verify.sh

@@ -174,14 +174,21 @@ watcher_port() {
 const fs = require('fs');
 const path = require('path');
 const target = process.env.TARGET_FOR_WATCHER_PORT;
-let port = process.env.WATCHER_PORT || '8081';
-for (const file of ['.js.env', '.env', '.env.local']) {
-  const full = path.join(target, file);
-  if (!fs.existsSync(full)) continue;
-  const text = fs.readFileSync(full, 'utf8');
-  const match = text.match(/^\s*(?:export\s+)?WATCHER_PORT=(["']?)([0-9]+)\1/m);
-  if (match) { port = match[2]; break; }
+// Precedence: an explicit WATCHER_PORT in the process env (which carries the
+// --watcher-port flag applied by launch) wins over the slot's .js.env, which wins
+// over the 8081 default. This lets a run be pointed at an alternate Metro without
+// editing a shared slot .js.env.
+let port = process.env.WATCHER_PORT || '';
+if (!port) {
+  for (const file of ['.js.env', '.env', '.env.local']) {
+    const full = path.join(target, file);
+    if (!fs.existsSync(full)) continue;
+    const text = fs.readFileSync(full, 'utf8');
+    const match = text.match(/^\s*(?:export\s+)?WATCHER_PORT=(["']?)([0-9]+)\1/m);
+    if (match) { port = match[2]; break; }
+  }
 }
+if (!port) port = '8081';
 console.log(port);
 NODE
 }

package/package.json

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

package/src/adapters/mobile/prepare.ts

@@ -18,6 +18,14 @@ import {
 
 export { type MobileRuntimeDecisionReport };
 
+// Pod install (run by `yarn setup` and by the native `yarn start:ios|android`
+// build) shells `node --print require.resolve('react-native-worklets-core')` and
+// treats any output other than the exact string `undefined` as "module found".
+// An inherited FORCE_COLOR makes node emit a colorized `undefined`, so VisionCamera
+// misdetects the worklets pod, enables FrameProcessors, and fails on the missing
+// pod. Force plain output for pod-triggering spawns so the probe reads `undefined`.
+const POD_PROBE_ENV: Record<string, string> = { FORCE_COLOR: '0', NO_COLOR: '1' };
+
 export interface PrepareMobileOptions extends MobileRuntimeDecisionOptions {
   /** Pass through to leaf-script spawnScript calls so output is suppressed in --json mode. */
   json?: boolean;
@@ -159,9 +167,9 @@ function dispatchAction(
   const cwd = action.cwd ?? target;
   switch (action.id) {
     case 'yarn-setup': {
-      // Leaf: install node_modules (deps missing or stale).
+      // Leaf: install node_modules (deps missing or stale); `yarn setup` runs pods.
       const leaf = path.join(runnerDir, 'adapters/mobile/yarn-setup.sh');
-      return spawnScript(leaf, ['--target', cwd], target, json);
+      return spawnScript(leaf, ['--target', cwd], target, json, POD_PROBE_ENV);
     }
     case 'start-metro': {
       // Leaf: ensure Metro is running; argv may carry --clear for cache reset.
@@ -185,13 +193,16 @@ function dispatchAction(
       return spawnScript(leaf, ['--target', cwd, '--clear'], target, json);
     }
     case 'launch-mobile-runtime': {
-      // Leaf: open the MetaMask Mobile dev client on the simulator/device.
+      // Leaf: open the MetaMask Mobile dev client on the simulator/device. In
+      // auto/rebuild modes this runs the native build (`yarn start:*`), which
+      // triggers pod install — pass the pod-probe env so it stays FORCE_COLOR-safe.
       const leaf = path.join(runnerDir, 'adapters/mobile/open-device.sh');
       return spawnScript(
         leaf,
         ['--platform', platform, '--target', cwd, '--preflight-mode', preflightMode],
         target,
         json,
+        POD_PROBE_ENV,
       );
     }
     case 'rebuild-native-dev-client': {

package/src/commands/shared.ts

@@ -95,12 +95,14 @@ export interface ScriptResult {
 
 // Compose an adapters/ script directly. Output is always captured (needed for
 // heal classification) and, in human mode, forwarded to stderr. --json keeps
-// stdout clean for the machine summary.
+// stdout clean for the machine summary. `env` overlays extra vars onto the
+// inherited environment for spawns that need a scoped variable (e.g. color mode).
 export function spawnScript(
   script: string,
   args: string[],
   cwd: string,
   json: boolean,
+  env?: Record<string, string>,
 ): ScriptResult {
   // For node invocations (script === process.execPath) the seam stem is derived
   // from args[0] so each spawned script has its own override key.
@@ -114,7 +116,7 @@ export function spawnScript(
   const result = spawnSync(bin, spawnArgs, {
     cwd,
     encoding: 'utf8',
-    env: process.env,
+    env: env ? { ...process.env, ...env } : process.env,
     maxBuffer: 64 * 1024 * 1024,
   });
   if (result.error) {