@deeeed/metamask-harness 0.94.0 → 0.94.2

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 ## Unreleased
 
+## 0.94.2 - 2026-10-24
+
+### Fixed
+
+- Treat disappeared or reused companion processes as already stopped, preserving the unrelated PID and allowing repeated slot cleanup.
+
+## 0.94.1 - 2026-10-22
+
+### Fixed
+
+- Mobile scroll-into-view checks the complete target bounds before treating it as already on screen.
+- iOS simulator launch restarts app processes that have run for six hours or reached 1.5 GiB RSS before opening the dev client, including when the runtime otherwise looks ready.
+
 ## 0.94.0 - 2026-10-22
 
 ### Added

package/adapters/mobile/lib/idb-companion.sh

@@ -41,8 +41,6 @@ stop_checkout_idb_companions() {
     else
       status=$?
       [ "$status" -eq 1 ] && continue
-      printf 'Companion ownership is unknown. Next: mm-harness status --target "%s"\n' "$repo" >&2
-      return 1
     fi
     if ! stop_verified_process "$pid" "$identity" 'TERM KILL' 0.1 _idb_companion_owned "$repo" "$udid" "$pid" "$identity"; then
       printf 'Companion %s cleanup was not confirmed. Next: mm-harness status --target "%s"\n' "$pid" "$repo" >&2

package/adapters/shared/process-identity.sh

@@ -15,7 +15,9 @@ stop_verified_process() {
   fi
   for signal in $signals; do
     if ! "$@"; then
-      if [ -z "$(process_identity "$pid")" ] && ! kill -0 "$pid" 2>/dev/null; then return 0; fi
+      current="$(process_identity "$pid")"
+      if [ -n "$current" ] && [ "$current" != "$identity" ]; then return 0; fi
+      if [ -z "$current" ] && ! kill -0 "$pid" 2>/dev/null; then return 0; fi
       printf 'Process %s ownership changed before stop.\n' "$pid" >&2
       return 1
     fi
@@ -23,6 +25,7 @@ stop_verified_process() {
     for attempt in $(seq 1 20); do
       current="$(process_identity "$pid")"
       if [ -z "$current" ] && ! kill -0 "$pid" 2>/dev/null; then return 0; fi
+      if [ -n "$current" ] && [ "$current" != "$identity" ]; then return 0; fi
       if [ "$current" != "$identity" ]; then
         printf 'Process %s identity changed during stop.\n' "$pid" >&2
         return 1

package/dist/adapters/mobile/lifecycle.js

@@ -127,6 +127,38 @@ function mobileSourceAwareLifecycleAdapters(lifecycle, readinessProbe = probeMob
     }
   }));
 }
+function staleMobileAppProcessReason(usage) {
+  if (usage.elapsedSeconds !== void 0 && usage.elapsedSeconds >= 6 * 60 * 60) {
+    return `app process ${String(usage.pid)} has run for ${Math.floor(usage.elapsedSeconds / 60)} minutes`;
+  }
+  if (usage.residentBytes !== void 0 && usage.residentBytes >= 1.5 * 1024 ** 3) {
+    return `app process ${String(usage.pid)} uses ${Math.round(usage.residentBytes / 1024 ** 2)} MiB RSS`;
+  }
+  return void 0;
+}
+function readIosAppProcessUsage(env = process.env) {
+  const device = env.SIM_UDID || env.IOS_SIMULATOR || "booted";
+  const pid = readIosAppPid({ simulator: device }, { env });
+  if (pid === null) return {};
+  try {
+    const output = execFileSync("ps", ["-p", String(pid), "-o", "etime=,rss="], {
+      encoding: "utf8",
+      timeout: 5e3,
+      stdio: ["ignore", "pipe", "ignore"]
+    });
+    const row = output.trim().split("\n").at(-1) ?? "";
+    const match = /^(?:(\d+)-)?(?:(\d+):)?(\d+):(\d+)\s+(\d+)$/u.exec(row.trim());
+    if (!match) return { pid };
+    const [, days, hours, minutes, seconds, rssKiB] = match;
+    return {
+      pid,
+      elapsedSeconds: Number(days ?? 0) * 86400 + Number(hours ?? 0) * 3600 + Number(minutes) * 60 + Number(seconds),
+      residentBytes: Number(rssKiB) * 1024
+    };
+  } catch {
+    return { pid };
+  }
+}
 function readAndroidAppPid(node, context) {
   const env = context.env ?? {};
   const device = node.adb_serial ?? node.android_device ?? node.device ?? env.ADB_SERIAL ?? env.ANDROID_SERIAL ?? process.env.ADB_SERIAL ?? process.env.ANDROID_SERIAL;
@@ -296,5 +328,7 @@ function asRecord(value) {
 }
 export {
   mobileSourceAwareLifecycleAdapters,
-  resolveMobileLifecycleReadinessTimeout
+  readIosAppProcessUsage,
+  resolveMobileLifecycleReadinessTimeout,
+  staleMobileAppProcessReason
 };

package/dist/adapters/mobile/ui.js

@@ -567,7 +567,16 @@ async function queryMobileViewportTarget(input, testId) {
       "Mobile ui.scroll scroll_into_view requires the current __AGENTIC__.queryUiTarget bridge."
     );
   }
-  return result;
+  const { rect, viewport } = result;
+  return {
+    ...result,
+    visible: result.visible === true && isRecord(rect) && isRecord(viewport) && scrollTargetFillsAxis(rect.x, rect.width, viewport.width) && scrollTargetFillsAxis(rect.y, rect.height, viewport.height)
+  };
+}
+function scrollTargetFillsAxis(start, size, viewport) {
+  if (typeof start !== "number" || typeof size !== "number" || typeof viewport !== "number" || !Number.isFinite(start) || !Number.isFinite(size) || !Number.isFinite(viewport) || size <= 0 || viewport <= 0) return false;
+  const visible = Math.min(start + size, viewport) - Math.max(start, 0);
+  return visible >= Math.min(size, viewport);
 }
 async function waitForMobileViewportTarget(input, testId, timeoutMs) {
   const deadline = Date.now() + Math.max(0, Math.min(timeoutMs, 5e3));

package/dist/commands/launch/mobile.js

@@ -8,6 +8,7 @@ import { ensureHarnessFresh } from "../../adapters/harness-freshness.js";
 import { pinMobileFeatureFlags } from "../../adapters/mobile/feature-flags.js";
 import { toRemoteFlagSpec } from "../../feature-flags.js";
 import { clearMobileReleaseArtifactState } from "../../adapters/mobile/release-artifact-state.js";
+import { readIosAppProcessUsage, staleMobileAppProcessReason } from "../../adapters/mobile/lifecycle.js";
 async function launchMobile(target, mobileTarget, tier, json, restartApp = false, clearMetro = false, remoteFlags) {
   clearMobileReleaseArtifactState(target);
   await ensureHarnessFresh(target, "mobile");
@@ -15,12 +16,15 @@ async function launchMobile(target, mobileTarget, tier, json, restartApp = false
   const watcherPort = process.env.WATCHER_PORT ? parseInt(process.env.WATCHER_PORT, 10) : void 0;
   const preflightMode = tier === "build" ? "rebuild-native" : "fast";
   const source = mobileSourceCheck(target);
+  const staleProcess = !restartApp && source.status === "current" && platform === "ios" ? staleMobileAppProcessReason(readIosAppProcessUsage()) : void 0;
+  if (staleProcess) process.stderr.write(`Mobile launch: ${staleProcess}; restarting the app process.
+`);
   const result = await prepareMobile(target, {
     platform,
     json,
     watcherPort,
     preflightMode,
-    restartApp: restartApp || source.status !== "current",
+    restartApp: restartApp || source.status !== "current" || staleProcess !== void 0,
     clearMetro
   });
   if (result.status !== 0) return result;

package/package.json

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