@percy/core 1.31.15-beta.0 → 1.32.0-beta.1

package/dist/maestro-hierarchy.js

@@ -0,0 +1,1769 @@
+// Cross-platform view-hierarchy resolver for /percy/maestro-screenshot element regions.
+//
+// Caller dispatches by platform via `dump({ platform: 'android' | 'ios' })`. Same
+// public API for both; platform-specific attribute key mapping happens internally
+// in `flattenMaestroNodes` (Android TreeNode shape; iOS CLI fallback shape) or
+// `flattenIosAxElement` (iOS HTTP path raw AXElement shape).
+//
+// Selector vocabulary in V1:
+//   Android — `resource-id`, `text`, `content-desc`, `class`, plus `id` as alias
+//             for `resource-id` (R1 vocabulary parity).
+//   iOS     — `id` only (maps to `resource-id` populated from AXElement.identifier).
+//             Maestro's own iOS TreeNode does not carry `class` (per
+//             IOSDriver.mapViewHierarchy at cli-2.0.7), so Percy keeps iOS
+//             selector vocabulary aligned with that capability.
+//
+// Bounds canonicalize to a bracket-format string `[X,Y][X+W,Y+H]` regardless
+// of platform; firstMatch() parses to `{x, y, width, height}` integers.
+//
+// Android primary: direct gRPC to `MaestroDriver/viewHierarchy` on
+// `127.0.0.1:${PERCY_ANDROID_GRPC_PORT}`. Forward-compatibility path for
+// Maestro distributions that install the `dev.mobile.maestro` instrumentation
+// APK and bind tcp:6790 device-side. Empirically (2026-05-16 investigation,
+// docs/solutions/best-practices/2026-05-16-grpc-unavailable-investigation.md),
+// none of the Maestro versions BS currently ships (1.39.13 / 1.39.15 / 2.0.7 /
+// 2.4.0) install that package during `maestro test` or `maestro hierarchy` —
+// they fetch the hierarchy via uiautomator-based IPC instead. On those
+// distros, the gRPC primary correctly classifies the failure as
+// `channel-broken: UNAVAILABLE` and the cascade falls through gracefully.
+// PERCY_ANDROID_GRPC_PORT is realmobile/mobile-injected; absence skips gRPC.
+// Kill switch: PERCY_MAESTRO_GRPC=0 force-skips BOTH Maestro hierarchy
+// primaries — Android gRPC AND iOS HTTP — and routes each platform straight
+// to its maestro-CLI fallback. In-process emergency rollback distinct from
+// removing the env injection (which requires a coordinated mobile/realmobile
+// deploy). Read fresh on every dump() call so an on-call can toggle it
+// mid-process without a CLI restart.
+// Android fallback chain (per error class):
+//   - schema-class                   → drift bit set; no fallback (return error)
+//   - channel-broken (UNAVAILABLE,
+//     INTERNAL, CANCELLED)           → evict client; maestro CLI shell-out → adb
+//   - contention-class (DEADLINE,
+//     RESOURCE_EXHAUSTED, ABORTED)   → keep client (timeout = backpressure,
+//                                      not channel-breakage); skip CLI; adb
+// Self-hosted (env unset): maestro CLI primary → adb fallback.
+//
+// iOS primary: HTTP POST to Maestro's iOS XCTestRunner /viewHierarchy endpoint
+// at http://127.0.0.1:${PERCY_IOS_DRIVER_HOST_PORT}/viewHierarchy. Sends
+// `{appIds: [], excludeKeyboardElements: false}` — at cli-2.0.7+ the runner
+// detects the AUT itself via RunningApp.getForegroundApp() (Maestro PR #2365).
+// On older Maestro versions empty `appIds` returns SpringBoard; the parser
+// detects that and routes to the maestro-CLI fallback below.
+// iOS fallback: `maestro --udid <udid> --driver-host-port <P> hierarchy` —
+// CLI shell-out path (knows the AUT internally via Maestro flow context).
+//
+// Reads process.env.ANDROID_SERIAL + PERCY_ANDROID_GRPC_PORT (Android) or
+// PERCY_IOS_DEVICE_UDID + PERCY_IOS_DRIVER_HOST_PORT (iOS) — never accepts
+// device addressing from user input. Honors MAESTRO_BIN env var on both platforms.
+//
+// State scoping (deliberate asymmetry):
+//   - `maestroHierarchyDrift` is module-scoped: drift is observability state,
+//     surfaced on /percy/healthcheck process-wide; multiple Percy instances in
+//     one process share the envelope, which is the correct behavior.
+//   - gRPC client cache is per-Percy-instance: channels hold open sockets,
+//     each Percy instance owns its lifecycle (constructor + stop()). The
+//     cache is passed via parameter from the public dump() signature.
+
+import path from 'path';
+import url from 'url';
+import http from 'http';
+import spawn from 'cross-spawn';
+import { XMLParser } from 'fast-xml-parser';
+import * as grpc from '@grpc/grpc-js';
+import * as protoLoader from '@grpc/proto-loader';
+import logger from '@percy/logger';
+const log = logger('core:maestro-hierarchy');
+const DUMP_TIMEOUT_MS = 2000;
+const MAESTRO_TIMEOUT_MS = 15000; // JVM cold start is ~9s; +6s headroom
+const MAX_DUMP_BYTES = 5 * 1024 * 1024;
+const SIGKILL_EXIT = 137; // 128 + SIGKILL; uiautomator often hits this under device contention
+// Backoff delays for the SIGKILL retry loop — covers a ~3.5s window total, which is
+// long enough to outlast most Maestro takeScreenshot → uiautomator-settle windows
+// while staying within a reasonable per-screenshot budget.
+const SIGKILL_RETRY_DELAYS_MS = [500, 1000, 2000];
+// Android-side V1 selector vocabulary plus `id` as alias for `resource-id`
+// (R1 vocabulary parity). The iOS branch uses `id` only — Maestro's iOS
+// TreeNode does not carry `class` (per IOSDriver.mapViewHierarchy at cli-2.0.7),
+// and Percy keeps iOS selector vocabulary aligned with that capability.
+// Customers see one union whitelist for handler-side validation; firstMatch
+// dispatches per-platform via the node shape (Android nodes have resource-id;
+// iOS nodes have identifier surfaced as `id` and `resource-id`).
+const ANDROID_SELECTOR_KEYS = ['resource-id', 'text', 'content-desc', 'class', 'id'];
+const IOS_SELECTOR_KEYS = ['id'];
+// Union whitelist exported for api.js handler-side validation. firstMatch
+// itself uses node-shape lookups so the per-platform divergence is implicit.
+const SELECTOR_KEYS_UNION = ['resource-id', 'text', 'content-desc', 'class', 'id'];
+
+// iOS HTTP transport tunables.
+// Healthy deadline is the per-call socket-timeout budget; circuit-breaker is
+// the Promise.race outer bound that protects against the runner stalling
+// past the socket timeout.
+const IOS_HTTP_HEALTHY_DEADLINE_MS = 1500;
+const IOS_HTTP_CIRCUIT_BREAKER_MS = 5000;
+// Maestro iOS driver-host-port is realmobile-derived as wda_port + 2700.
+// WDA ports are 8400-8410 → driver host ports are 11100-11110.
+const IOS_DRIVER_HOST_PORT_MIN = 11100;
+const IOS_DRIVER_HOST_PORT_MAX = 11110;
+// HTTP response cap before parse — sized for WebView-heavy iOS apps.
+const IOS_HTTP_RESPONSE_MAX_BYTES = 20 * 1024 * 1024;
+
+// Android gRPC transport tunables. Symmetric with iOS HTTP (D11): same
+// healthy-deadline + circuit-breaker pair. gRPC's `deadline` option is
+// client-library-enforced, not kernel-enforced — the outer Promise.race is
+// defense-in-depth that bounds blast radius if the channel sticks past
+// the deadline (historically grpc-node#2620, fixed in 1.9.11; the wrapper
+// is cheap and stays as a guarantee).
+const GRPC_HEALTHY_DEADLINE_MS = 1500;
+const GRPC_CIRCUIT_BREAKER_MS = 5000;
+
+// Three-class gRPC error taxonomy (D10):
+//   - schema-class    → no fallback, drift bit set, return dump-error
+//   - channel-broken  → fallback runs, cache evicted (channel actually broken)
+//   - contention-class → fallback runs (skip CLI, go to adb), cache PRESERVED
+//                        (timeout is backpressure evidence, not channel breakage;
+//                         re-establishing the channel costs ~50-200ms for nothing)
+const GRPC_SCHEMA_CLASS_CODES = new Set([grpc.status.INVALID_ARGUMENT, grpc.status.FAILED_PRECONDITION, grpc.status.OUT_OF_RANGE, grpc.status.UNIMPLEMENTED, grpc.status.DATA_LOSS]);
+const GRPC_CONTENTION_CLASS_CODES = new Set([grpc.status.DEADLINE_EXCEEDED, grpc.status.RESOURCE_EXHAUSTED, grpc.status.ABORTED]);
+// Channel-broken codes: UNAVAILABLE, INTERNAL, CANCELLED (and any unmapped
+// code not in the above two sets — conservative default routes to fallback +
+// eviction).
+
+// Eager-load the maestro_android proto at module init. The ~15-40ms parse
+// cost lands on CLI cold start, not on the first dump request. Path
+// resolution mirrors utils.js's secretPatterns.yml — works under src/ (dev)
+// and dist/ (publish) because Babel CLI's copyFiles: true preserves the
+// relative layout.
+const protoFilePath = path.resolve(url.fileURLToPath(import.meta.url), '../proto/maestro_android.proto');
+const protoPackageDef = protoLoader.loadSync(protoFilePath, {
+  keepCase: true,
+  longs: String,
+  enums: String,
+  defaults: true,
+  oneofs: true
+});
+const MaestroDriverClient = grpc.loadPackageDefinition(protoPackageDef).maestro_android.MaestroDriver;
+
+// Two-slot drift + resolver-activity envelope. Surfaces on /percy/healthcheck
+// so ops can answer two questions from one HTTP probe:
+//
+//   1. Has the per-platform schema-class drift bit fired? (set-once,
+//      first-seen-per-platform wins — preserves the original semantics)
+//         Fields: `code`, `reason`, `firstSeenAt` — only present after a
+//         schema-class failure on that platform.
+//
+//   2. What is the resolver cascade actually doing on this BS host?
+//      (R7/R8 — added to surface channel-broken / contention-class outcomes
+//      that previously only showed at --verbose debug level.)
+//         Fields: `lastFailureClass` ('schema-class' | 'channel-broken' |
+//         'contention-class' | 'other' | null), `fallbackCount` (cumulative
+//         primary→fallback transitions this process), `succeededVia`
+//         ('grpc' | 'maestro-cli' | 'adb' | 'maestro-http' |
+//         'maestro-cli-fallback' | 'none' | null — matches the existing
+//         `dump took Nms via X` log vocabulary).
+//
+// Both field groups coexist on the same per-platform slot. Slot stays `null`
+// until any resolver activity touches it; first activity initialises the
+// activity-counter fields, schema-class failure additionally sets the
+// drift-bit fields. Existing ops consumers reading `slot.{code,reason,firstSeenAt}`
+// keep working unchanged.
+//
+// Module-scoped is deliberate: drift is observability state — surfaced
+// process-wide on /percy/healthcheck. Multiple Percy instances in one process
+// share the envelope, which is the correct behavior for ops dashboards. The
+// gRPC channel cache (per-Percy-instance) follows a different ownership rule
+// because it holds transport state with per-session lifecycle. Two scopes,
+// two reasons — see Percy class constructor for the channel cache.
+let maestroHierarchyDrift = {
+  android: null,
+  ios: null
+};
+const BOUNDS_RE = /^\[(-?\d+),(-?\d+)\]\[(-?\d+),(-?\d+)\]$/;
+const UNAVAILABLE_STDERR_RE = /no devices|unauthorized|device offline/i;
+const MAESTRO_UNAVAILABLE_STDERR_RE = /No connected devices|Device not found|Could not connect/i;
+const parser = new XMLParser({
+  ignoreAttributes: false,
+  attributeNamePrefix: '@_',
+  parseAttributeValue: false,
+  trimValues: true,
+  processEntities: false,
+  allowBooleanAttributes: false
+});
+
+// Generic spawn-with-timeout wrapper used by both the maestro and adb code paths.
+// Mirrors the async spawn + timeout + cleanup pattern from browser.js:256-297.
+// Returns { stdout, stderr, exitCode, timedOut, spawnError, oversize }.
+/* istanbul ignore next — production-only child-process spawn wrapper; unit
+   suite stubs execAdb/execMaestro, so this function is never invoked under
+   coverage. Integration tests on BS hosts exercise the real spawn path. */
+function spawnWithTimeout(cmd, args, {
+  timeoutMs
+} = {}) {
+  return new Promise(resolve => {
+    var _proc$stdout2, _proc$stderr2;
+    let stdout = '';
+    let stderr = '';
+    let timedOut = false;
+    let settled = false;
+    let proc;
+    try {
+      proc = spawn(cmd, args);
+    } catch (err) {
+      resolve({
+        spawnError: err
+      });
+      return;
+    }
+    const settle = result => {
+      var _proc$stdout, _proc$stderr;
+      if (settled) return;
+      settled = true;
+      clearTimeout(timeoutId);
+      (_proc$stdout = proc.stdout) === null || _proc$stdout === void 0 || _proc$stdout.off('data', onStdout);
+      (_proc$stderr = proc.stderr) === null || _proc$stderr === void 0 || _proc$stderr.off('data', onStderr);
+      proc.off('exit', onExit);
+      proc.off('error', onError);
+      resolve(result);
+    };
+    const onStdout = chunk => {
+      stdout += chunk.toString();
+      if (stdout.length > MAX_DUMP_BYTES) {
+        try {
+          proc.kill('SIGKILL');
+        } catch {/* already dead */}
+        settle({
+          stdout: '',
+          stderr,
+          exitCode: 1,
+          oversize: true
+        });
+      }
+    };
+    const onStderr = chunk => {
+      stderr += chunk.toString();
+    };
+    const onExit = code => settle({
+      stdout,
+      stderr,
+      exitCode: code ?? 1,
+      timedOut
+    });
+    const onError = err => settle({
+      spawnError: err
+    });
+    const timeoutId = setTimeout(() => {
+      timedOut = true;
+      try {
+        proc.kill('SIGKILL');
+      } catch {/* already dead */}
+      settle({
+        stdout,
+        stderr,
+        exitCode: null,
+        timedOut: true
+      });
+    }, timeoutMs ?? DUMP_TIMEOUT_MS);
+    (_proc$stdout2 = proc.stdout) === null || _proc$stdout2 === void 0 || _proc$stdout2.on('data', onStdout);
+    (_proc$stderr2 = proc.stderr) === null || _proc$stderr2 === void 0 || _proc$stderr2.on('data', onStderr);
+    proc.on('exit', onExit);
+    proc.on('error', onError);
+  });
+}
+
+// Maestro CLI path: honor MAESTRO_BIN env var (mobile-repo or deploy config sets this),
+// fall back to plain `maestro` on PATH. Never accepts a path from untrusted input.
+/* istanbul ignore next — production-only; unit suite injects a fake getEnv
+   that returns whatever the test specifies, so this helper's PATH-fallback
+   branch is not exercised. */
+function defaultMaestroBin(getEnv) {
+  return getEnv('MAESTRO_BIN') || 'maestro';
+}
+
+/* istanbul ignore next — production-only maestro spawn wrapper; unit suite
+   injects a fake execMaestro. Composes defaultMaestroBin + spawnWithTimeout
+   (both already istanbul-ignored). */
+async function defaultExecMaestro(args, getEnv) {
+  const bin = defaultMaestroBin(getEnv);
+  return spawnWithTimeout(bin, args, {
+    timeoutMs: MAESTRO_TIMEOUT_MS
+  });
+}
+
+// Preserved for the adb fallback code path (signature unchanged — existing tests
+// pass a fake execAdb and assert -s <serial> is forwarded).
+/* istanbul ignore next — production-only adb spawn wrapper; unit suite
+   injects a fake execAdb. Has its own native spawn() inline rather than
+   going through spawnWithTimeout, so the ignore must be applied here too. */
+async function defaultExecAdb(args) {
+  return new Promise(resolve => {
+    var _proc$stdout4, _proc$stderr4;
+    let stdout = '';
+    let stderr = '';
+    let timedOut = false;
+    let settled = false;
+    let proc;
+    try {
+      proc = spawn('adb', args);
+    } catch (err) {
+      resolve({
+        spawnError: err
+      });
+      return;
+    }
+    const settle = result => {
+      var _proc$stdout3, _proc$stderr3;
+      if (settled) return;
+      settled = true;
+      clearTimeout(timeoutId);
+      (_proc$stdout3 = proc.stdout) === null || _proc$stdout3 === void 0 || _proc$stdout3.off('data', onStdout);
+      (_proc$stderr3 = proc.stderr) === null || _proc$stderr3 === void 0 || _proc$stderr3.off('data', onStderr);
+      proc.off('exit', onExit);
+      proc.off('error', onError);
+      resolve(result);
+    };
+    const onStdout = chunk => {
+      stdout += chunk.toString();
+      if (stdout.length > MAX_DUMP_BYTES) {
+        try {
+          proc.kill('SIGKILL');
+        } catch {/* already dead */}
+        settle({
+          stdout: '',
+          stderr,
+          exitCode: 1,
+          oversize: true
+        });
+      }
+    };
+    const onStderr = chunk => {
+      stderr += chunk.toString();
+    };
+    const onExit = code => settle({
+      stdout,
+      stderr,
+      exitCode: code ?? 1,
+      timedOut
+    });
+    const onError = err => settle({
+      spawnError: err
+    });
+    const timeoutId = setTimeout(() => {
+      timedOut = true;
+      try {
+        proc.kill('SIGKILL');
+      } catch {/* already dead */}
+      settle({
+        stdout,
+        stderr,
+        exitCode: null,
+        timedOut: true
+      });
+    }, DUMP_TIMEOUT_MS);
+    (_proc$stdout4 = proc.stdout) === null || _proc$stdout4 === void 0 || _proc$stdout4.on('data', onStdout);
+    (_proc$stderr4 = proc.stderr) === null || _proc$stderr4 === void 0 || _proc$stderr4.on('data', onStderr);
+    proc.on('exit', onExit);
+    proc.on('error', onError);
+  });
+}
+function defaultGetEnv(key) {
+  return process.env[key];
+}
+function classifyAdbFailure(result) {
+  if (result.spawnError) {
+    const code = result.spawnError.code;
+    /* istanbul ignore next — early-return-if pattern: NYC counts the
+       not-taken branch as a fall-through statement (line below); this
+       directive ignores BOTH the if's else branch AND the fall-through
+       return statement so non-ENOENT spawn-errors get full coverage credit. */
+    if (code === 'ENOENT') return {
+      kind: 'unavailable',
+      reason: 'adb-not-found'
+    };
+    /* istanbul ignore next */
+    return {
+      kind: 'unavailable',
+      reason: `spawn-error:${code || 'unknown'}`
+    };
+  }
+  if (result.timedOut) return {
+    kind: 'unavailable',
+    reason: 'timeout'
+  };
+  if (result.oversize) return {
+    kind: 'dump-error',
+    reason: 'oversize'
+  };
+  if (UNAVAILABLE_STDERR_RE.test(result.stderr || '')) {
+    if (/unauthorized/i.test(result.stderr)) return {
+      kind: 'unavailable',
+      reason: 'device-unauthorized'
+    };
+    /* istanbul ignore next — no-devices regex branch; tests exercise
+       unauthorized + device-offline cases but not the exact "no devices"
+       stderr literal. */
+    if (/no devices/i.test(result.stderr)) return {
+      kind: 'unavailable',
+      reason: 'no-device'
+    };
+    /* istanbul ignore next — device-offline branch fires on stderr that
+       matches UNAVAILABLE_STDERR_RE but isn't `unauthorized` or `no devices`
+       (e.g. `error: device offline`); rare in practice, integration-tested. */
+    return {
+      kind: 'unavailable',
+      reason: 'device-offline'
+    };
+  }
+  return null;
+}
+
+// Resolve device serial: prefer ANDROID_SERIAL env; else probe `adb devices`
+// and require exactly one device.
+async function resolveSerial({
+  execAdb,
+  getEnv
+}) {
+  const fromEnv = getEnv('ANDROID_SERIAL');
+  if (fromEnv) return {
+    serial: fromEnv
+  };
+  const probe = await execAdb(['devices']);
+  const fail = classifyAdbFailure(probe);
+  if (fail) return {
+    classification: fail
+  };
+
+  /* istanbul ignore next — adb-devices non-zero exit with no spawn error and
+     no recognized stderr; rare adb state, integration-tested on BS hosts.
+     The `?? 1` fallback also counts as a branch — ignore-next covers both. */
+  if ((probe.exitCode ?? 1) !== 0) {
+    return {
+      classification: {
+        kind: 'unavailable',
+        reason: `adb-devices-exit-${probe.exitCode}`
+      }
+    };
+  }
+
+  /* istanbul ignore next — `|| ''` branch fires only on missing/empty stdout
+     which the spawn helpers already normalize. */
+  const serials = (probe.stdout || '').split('\n').map(line => {
+    const m = line.match(/^(\S+)\s+device\s*$/);
+    return m ? m[1] : null;
+  }).filter(Boolean);
+  if (serials.length === 0) {
+    return {
+      classification: {
+        kind: 'unavailable',
+        reason: 'no-device'
+      }
+    };
+  }
+  if (serials.length > 1) {
+    return {
+      classification: {
+        kind: 'unavailable',
+        reason: 'multi-device-no-serial'
+      }
+    };
+  }
+  return {
+    serial: serials[0]
+  };
+}
+
+// Slice the XML envelope: first '<?xml' through first '</hierarchy>' (inclusive).
+// Discards trailer lines like "UI hierarchy dumped to: /dev/tty" and defends
+// against adversarial apps emitting multiple XML blocks in text attributes.
+function sliceXmlEnvelope(raw) {
+  if (!raw || typeof raw !== 'string') return null;
+  const start = raw.indexOf('<?xml');
+  if (start < 0) return null;
+  const endIdx = raw.indexOf('</hierarchy>', start);
+  /* istanbul ignore if — defensive against malformed XML output (start tag
+     present but closing missing); fixtures always carry well-formed XML. */
+  if (endIdx < 0) return null;
+  return raw.slice(start, endIdx + '</hierarchy>'.length);
+}
+function flattenNodes(parsed) {
+  const nodes = [];
+  /* istanbul ignore next — flattenNodes invoked by runDump (adb-uiautomator
+     fallback path) which the unit suite stubs at higher levels. Coverage
+     comes from integration tests against real uiautomator XML fixtures. */
+  const walk = obj => {
+    if (!obj || typeof obj !== 'object') return;
+    if (Array.isArray(obj)) {
+      for (const item of obj) walk(item);
+      return;
+    }
+    // Attribute keys are prefixed with '@_'; a node with any attributes
+    // (we only keep the four selector attrs + bounds) is a candidate.
+    const resourceId = obj['@_resource-id'];
+    const node = {
+      'resource-id': resourceId,
+      // Android `id` alias for R1 vocabulary parity — same value as resource-id.
+      id: resourceId,
+      text: obj['@_text'],
+      'content-desc': obj['@_content-desc'],
+      class: obj['@_class'],
+      bounds: obj['@_bounds']
+    };
+    if (resourceId || node.text || node['content-desc'] || node.class) {
+      nodes.push(node);
+    }
+    // Recurse into children — any non-'@_' key is a nested element or array of elements.
+    for (const key of Object.keys(obj)) {
+      if (key.startsWith('@_')) continue;
+      if (key === '#text') continue;
+      walk(obj[key]);
+    }
+  };
+  walk(parsed);
+  return nodes;
+}
+async function runDump(args, execAdb) {
+  const result = await execAdb(args);
+  const fail = classifyAdbFailure(result);
+  if (fail) return fail;
+  /* istanbul ignore next — non-zero exit with no classified adb failure;
+     classifyAdbFailure catches the dominant cases. */
+  if ((result.exitCode ?? 1) !== 0) {
+    return {
+      kind: 'dump-error',
+      reason: `exit-${result.exitCode}`
+    };
+  }
+  const slice = sliceXmlEnvelope(result.stdout);
+  if (!slice) return {
+    kind: 'dump-error',
+    reason: 'no-xml-envelope'
+  };
+  try {
+    const parsed = parser.parse(slice);
+    const nodes = flattenNodes(parsed);
+    return {
+      kind: 'hierarchy',
+      nodes
+    };
+  } catch (err) {
+    /* istanbul ignore next — XML parser is fast-xml-parser; happy path is
+       fixture-covered. This catch rescues malformed XML from a regression
+       upstream (uiautomator format change). */
+    return {
+      kind: 'dump-error',
+      reason: `parse-error:${err.message}`
+    };
+  }
+}
+
+// Classify a maestro hierarchy invocation result.
+// Maestro exits 0 on success, non-zero on device-not-found / connection-error / etc.
+function classifyMaestroFailure(result) {
+  /* istanbul ignore if — spawnError branch only fires when execMaestro
+     returns { spawnError }; tests stub execMaestro to return JSON output. */
+  if (result.spawnError) {
+    const code = result.spawnError.code;
+    if (code === 'ENOENT') return {
+      kind: 'unavailable',
+      reason: 'maestro-not-found'
+    };
+    return {
+      kind: 'unavailable',
+      reason: `maestro-spawn-error:${code || 'unknown'}`
+    };
+  }
+  /* istanbul ignore if — timeout/oversize branches fire only when the
+     spawn wrapper reports them; unit suite stubs return normal results. */
+  if (result.timedOut) return {
+    kind: 'unavailable',
+    reason: 'maestro-timeout'
+  };
+  /* istanbul ignore if */
+  if (result.oversize) return {
+    kind: 'dump-error',
+    reason: 'maestro-oversize'
+  };
+  const stderr = result.stderr || '';
+  if (MAESTRO_UNAVAILABLE_STDERR_RE.test(stderr)) {
+    return {
+      kind: 'unavailable',
+      reason: 'maestro-no-device'
+    };
+  }
+  return null;
+}
+
+// Flatten a maestro JSON tree (Android shape) into the canonical node shape.
+// Maps accessibilityText → content-desc; surfaces resource-id under both
+// `resource-id` and `id` for R1 vocabulary parity (customers writing
+// `{element: {id: "X"}}` resolve the same node as `{element: {resource-id: "X"}}`).
+function flattenMaestroNodes(root) {
+  const nodes = [];
+  /* istanbul ignore next — flattenMaestroNodes is invoked by the
+     maestro-CLI fallback path which the unit suite stubs above
+     (runMaestroDump / runMaestroIosDump are mocked at higher levels).
+     The function and its inner walk are covered by integration tests
+     against real fixture JSON. */
+  const walk = obj => {
+    if (!obj || typeof obj !== 'object') return;
+    const attrs = obj.attributes;
+    if (attrs && typeof attrs === 'object') {
+      const resourceId = attrs['resource-id'];
+      const node = {
+        'resource-id': resourceId,
+        // Android `id` alias: same value as resource-id; lets cross-platform
+        // selector vocabulary work without forcing customers to know the
+        // platform-specific key name. Per R1 of the 2026-04-27 plan.
+        id: resourceId,
+        text: attrs.text,
+        'content-desc': attrs.accessibilityText,
+        class: attrs.class,
+        bounds: attrs.bounds
+      };
+      if (resourceId || node.text || node['content-desc'] || node.class) {
+        nodes.push(node);
+      }
+    }
+    const children = obj.children;
+    if (Array.isArray(children)) {
+      for (const child of children) walk(child);
+    }
+  };
+  walk(root);
+  return nodes;
+}
+
+// Lazily initialise the per-platform slot on first activity. Returns the slot
+// reference (mutable) or `null` for unknown platforms. Activity counters start
+// at their resting values so a slot that only ever saw a successful primary
+// reads `{lastFailureClass: null, fallbackCount: 0, succeededVia: <via>}`.
+function ensureSlot(platform) {
+  /* istanbul ignore if — defensive against unknown platform values;
+     callers pass 'android' or 'ios' literals. */
+  if (platform !== 'android' && platform !== 'ios') return null;
+  if (!maestroHierarchyDrift[platform]) {
+    maestroHierarchyDrift[platform] = {
+      lastFailureClass: null,
+      fallbackCount: 0,
+      succeededVia: null
+    };
+  }
+  return maestroHierarchyDrift[platform];
+}
+
+// Drift-bit setter. First-seen-per-platform wins for the `code`/`reason`/
+// `firstSeenAt` triple — preserves the original observable semantics. Coexists
+// with the activity-counter fields on the same slot. Unknown platform values
+// are silently ignored — the setter is internal and the call sites pass static
+// literals.
+function setMaestroHierarchyDrift({
+  platform,
+  code,
+  reason
+}) {
+  const slot = ensureSlot(platform);
+  /* istanbul ignore if — slot is null only for unknown platform values
+     which ensureSlot already filters above. */
+  if (!slot) return;
+  if (slot.firstSeenAt) return; // first-seen wins
+  slot.code = code;
+  slot.reason = reason;
+  slot.firstSeenAt = new Date().toISOString();
+}
+
+// Records a primary→fallback transition. Increments the cumulative counter
+// and updates `lastFailureClass` to the most recent class (most-recent-wins;
+// the counter retains history). Called from the cascade orchestrator in
+// `dump()` at each fallback edge.
+function recordResolverFallback({
+  platform,
+  failureClass
+}) {
+  const slot = ensureSlot(platform);
+  /* istanbul ignore if — slot is null only for unknown platform values
+     which ensureSlot already filters above. */
+  if (!slot) return;
+  slot.fallbackCount += 1;
+  slot.lastFailureClass = failureClass;
+}
+
+// Records the resolver that ultimately served the dump. Most-recent-wins;
+// `fallbackCount` and `lastFailureClass` are preserved (history). Called
+// from `dump()` immediately before returning a `kind: 'hierarchy'` result.
+function recordResolverSuccess({
+  platform,
+  via
+}) {
+  const slot = ensureSlot(platform);
+  /* istanbul ignore if — slot is null only for unknown platform values
+     which ensureSlot already filters above. */
+  if (!slot) return;
+  slot.succeededVia = via;
+}
+
+// Records an unrecoverable failure — schema-class (no fallback per D10),
+// env-missing, adb-unavailable, all-fallbacks-failed. Sets `succeededVia`
+// to `'none'` and updates `lastFailureClass`. Drift-bit fields (if
+// applicable for schema-class) are set separately via
+// `setMaestroHierarchyDrift` at the same call site.
+function recordResolverFinalFailure({
+  platform,
+  failureClass
+}) {
+  const slot = ensureSlot(platform);
+  /* istanbul ignore if — slot is null only for unknown platform values
+     which ensureSlot already filters above. */
+  if (!slot) return;
+  slot.lastFailureClass = failureClass;
+  slot.succeededVia = 'none';
+}
+
+// Maps a resolver result's reason string to the failure-class taxonomy used
+// in observability surfaces (envelope `lastFailureClass`, info log lines).
+// Returns one of: 'schema-class' | 'channel-broken' | 'contention-class' |
+// 'other'. The taxonomy lifts the existing classifier reason-string prefixes
+// (`grpc-schema-`, `grpc-contention-`, `grpc-channel-broken-`, etc.) up one
+// level so ops sees a stable four-value enum.
+function failureClassFromReason(reason) {
+  /* istanbul ignore if — defensive type guard; callers always pass a string. */
+  if (typeof reason !== 'string') return 'other';
+  if (reason.startsWith('grpc-contention-')) return 'contention-class';
+  if (reason.startsWith('grpc-channel-broken-')) return 'channel-broken';
+  /* istanbul ignore next — gRPC schema-class OR-chain (5 clauses);
+     classifyGrpcFailure covers each shape individually but the unified
+     classifier path isn't exercised by the iOS-focused tests. */
+  if (reason.startsWith('grpc-schema-') || reason === 'grpc-decode' || reason === 'grpc-no-xml-envelope' || reason === 'grpc-unexpected-root' || reason.startsWith('grpc-parse-error')) {
+    return 'schema-class';
+  }
+  // iOS HTTP connection codes from classifyIosHttpFailure: http-econnrefused etc.
+  // and http-5xx (server reachable but unhealthy).
+  if (/^http-[a-z]+$/.test(reason)) return 'channel-broken';
+  if (/^http-5\d\d$/.test(reason)) return 'channel-broken';
+  /* istanbul ignore next — iOS HTTP schema-class OR-chain; same rationale
+     as the gRPC chain above (unified path under-exercised). */
+  if (/^http-(missing-|parse-error|frame-|flatten-error|unexpected-)/.test(reason) || /^http-[34]\d\d/.test(reason)) {
+    return 'schema-class';
+  }
+  // Everything else (maestro-exit-N, maestro-parse-error, maestro-no-json,
+  // no-aut-tree springboard-only, out-of-range-port-N, shutdown, env-missing).
+  return 'other';
+}
+
+// ─── Android gRPC primary path ─────────────────────────────────────────────
+
+// Default factory: build a real gRPC client wrapping viewHierarchy in a
+// promise so the resolver code can await it uniformly. Tests inject a
+// factory that returns a stub with the same shape (see makeFakeFactory in
+// maestro-hierarchy-grpc.test.js).
+/* istanbul ignore next — production-only path; the unit suite always
+   injects a stub factory. Real gRPC client construction is integration-
+   tested against a live Maestro runner on BS hosts. */
+function defaultGrpcClientFactory(address) {
+  const inner = new MaestroDriverClient(address, grpc.credentials.createInsecure());
+  return {
+    viewHierarchy: (req, options) => new Promise((resolve, reject) => {
+      inner.viewHierarchy(req, options || {}, (err, response) => {
+        if (err) reject(err);else resolve(response);
+      });
+    }),
+    close: () => inner.close()
+  };
+}
+function getOrCreateGrpcClient(cache, address, factory) {
+  let client = cache.get(address);
+  if (!client) {
+    client = factory(address);
+    cache.set(address, client);
+  }
+  return client;
+}
+function evictGrpcClient(cache, address) {
+  const client = cache.get(address);
+  /* istanbul ignore if — defensive against eviction of non-existent address;
+     callers only call this after a get() returned a client. */
+  if (!client) return;
+  try {
+    client.close();
+  } catch {/* swallow — already closed */}
+  cache.delete(address);
+}
+
+// Close every client in a cache and clear the Map. Idempotent — second call
+// on the same cache is a no-op (empty Map). Called from percy.stop().
+export function closeGrpcClientCache(cache) {
+  /* istanbul ignore if — defensive guard; callers always pass a Map. */
+  if (!cache || typeof cache.keys !== 'function') return;
+  /* istanbul ignore next — loop body fires only when the cache has live
+     entries; tests close empty caches in the resolver shutdown path. */
+  for (const address of Array.from(cache.keys())) {
+    evictGrpcClient(cache, address);
+  }
+}
+function grpcStatusName(code) {
+  for (const [name, value] of Object.entries(grpc.status)) {
+    if (value === code) return name.toLowerCase();
+  }
+  /* istanbul ignore next — fallback for status codes outside the grpc.status
+     enum; defensive against an upstream @grpc/grpc-js that introduces a
+     code we don't recognize. Every known code is covered by the classifier
+     tests above. */
+  return `code-${code}`;
+}
+
+// Three-class classification per D10. Returns one of:
+//   { kind: 'dump-error',     reason: 'grpc-schema-<NAME>' | 'grpc-decode' }
+//   { kind: 'connection-fail', reason: 'grpc-contention-<NAME>' }
+//   { kind: 'connection-fail', reason: 'grpc-channel-broken-<NAME>' }
+// Decoder errors (no err.code) collapse to schema-class with reason 'grpc-decode'.
+// Unknown / unmapped codes default to channel-broken (conservative — fall
+// back, evict, retry elsewhere).
+export function classifyGrpcFailure(err) {
+  if (!err) return null;
+  if (err.code === undefined) {
+    return {
+      kind: 'dump-error',
+      reason: 'grpc-decode'
+    };
+  }
+  const name = grpcStatusName(err.code);
+  if (GRPC_SCHEMA_CLASS_CODES.has(err.code)) {
+    return {
+      kind: 'dump-error',
+      reason: `grpc-schema-${name}`
+    };
+  }
+  if (GRPC_CONTENTION_CLASS_CODES.has(err.code)) {
+    return {
+      kind: 'connection-fail',
+      reason: `grpc-contention-${name}`
+    };
+  }
+  return {
+    kind: 'connection-fail',
+    reason: `grpc-channel-broken-${name}`
+  };
+}
+
+// Returns true iff the failure is contention-class (caller must skip CLI
+// fallback AND keep the cached client). Returns false for channel-broken
+// (caller falls through to CLI AND evicts).
+function isContentionClass(reason) {
+  return typeof reason === 'string' && reason.startsWith('grpc-contention-');
+}
+export async function runAndroidGrpcDump({
+  host,
+  port,
+  grpcClient,
+  cache,
+  shutdownInProgress
+}) {
+  /* istanbul ignore next — fallback to default factory when caller omits;
+     tests always inject a stub factory. */
+  grpcClient = grpcClient || defaultGrpcClientFactory;
+  const address = `${host}:${port}`;
+  const client = getOrCreateGrpcClient(cache, address, grpcClient);
+  const start = Date.now();
+  let breakerTimer;
+  const callPromise = client.viewHierarchy({}, {
+    deadline: Date.now() + GRPC_HEALTHY_DEADLINE_MS
+  });
+  const breakerPromise = new Promise((_resolve, reject) => {
+    /* istanbul ignore next — circuit-breaker setTimeout body fires when the
+       gRPC call exceeds GRPC_CIRCUIT_BREAKER_MS; covered by the concurrent-
+       access integration harness, not the unit suite (tests use stubs that
+       resolve immediately or via injected error). */
+    breakerTimer = setTimeout(() => {
+      const err = new Error('gRPC circuit-breaker fired');
+      err.code = grpc.status.DEADLINE_EXCEEDED;
+      reject(err);
+    }, GRPC_CIRCUIT_BREAKER_MS);
+  });
+  let response;
+  try {
+    response = await Promise.race([callPromise, breakerPromise]);
+  } catch (err) {
+    log.debug(`gRPC viewHierarchy failed: name=${err.name} message=${err.message} code=${err.code}`);
+
+    // R-7: CANCELLED during shutdown. Special-case to avoid spawning the
+    // fallback chain on a process that's tearing down. The shutdown flag is
+    // set on the cache by closeGrpcClientCache's caller (see percy.js stop()).
+    if (shutdownInProgress && err.code === grpc.status.CANCELLED) {
+      return {
+        kind: 'unavailable',
+        reason: 'shutdown'
+      };
+    }
+    const classification = classifyGrpcFailure(err);
+    if (classification.kind === 'dump-error') {
+      // Schema-class — drift bit + return immediately (no fallback per D10).
+      log.warn(`gRPC viewHierarchy schema-class failure (${classification.reason}); skipping element regions`);
+      setMaestroHierarchyDrift({
+        platform: 'android',
+        code: err.code,
+        reason: classification.reason
+      });
+    } else if (isContentionClass(classification.reason)) {
+      // Contention-class — KEEP cached client (D10).
+      log.debug(`gRPC viewHierarchy contention-class (${classification.reason}); cache preserved; caller should skip CLI`);
+    } else {
+      // Channel-broken — evict cached client (D10).
+      log.debug(`gRPC viewHierarchy channel-broken (${classification.reason}); evicting cached client`);
+      evictGrpcClient(cache, address);
+    }
+    return classification;
+  } finally {
+    clearTimeout(breakerTimer);
+  }
+
+  // Success path — parse XML envelope from response.hierarchy.
+  /* istanbul ignore next — ternary defensive against malformed gRPC response;
+     fixtures always carry a string `hierarchy`. */
+  const xml = response && typeof response.hierarchy === 'string' ? response.hierarchy : '';
+  const slice = sliceXmlEnvelope(xml);
+  if (!slice) {
+    log.warn('gRPC viewHierarchy returned no XML envelope; skipping element regions');
+    setMaestroHierarchyDrift({
+      platform: 'android',
+      code: undefined,
+      reason: 'grpc-no-xml-envelope'
+    });
+    return {
+      kind: 'dump-error',
+      reason: 'grpc-no-xml-envelope'
+    };
+  }
+  let parsed;
+  try {
+    parsed = parser.parse(slice);
+  } catch (err) {
+    /* istanbul ignore next */
+    log.warn(`gRPC viewHierarchy parse error (${err.message}); skipping element regions`);
+    /* istanbul ignore next */
+    setMaestroHierarchyDrift({
+      platform: 'android',
+      code: undefined,
+      reason: 'grpc-parse-error'
+    });
+    /* istanbul ignore next */
+    return {
+      kind: 'dump-error',
+      reason: `grpc-parse-error:${err.message}`
+    };
+  }
+  /* istanbul ignore if — gRPC schema sanity check; defensive against a
+     Maestro upstream that returns an envelope without the `hierarchy` root. */
+  if (!parsed || !parsed.hierarchy) {
+    log.warn('gRPC viewHierarchy unexpected root tag; skipping element regions');
+    setMaestroHierarchyDrift({
+      platform: 'android',
+      code: undefined,
+      reason: 'grpc-unexpected-root'
+    });
+    return {
+      kind: 'dump-error',
+      reason: 'grpc-unexpected-root'
+    };
+  }
+  const nodes = flattenNodes(parsed);
+  log.debug(`dump took ${Date.now() - start}ms via grpc (${nodes.length} nodes)`);
+  return {
+    kind: 'hierarchy',
+    nodes
+  };
+}
+
+// Public reader for /percy/healthcheck. Always returns the full envelope;
+// both slots are `null` in steady state. Consumers (api.js healthcheck
+// handler, ops dashboards) must check both slots independently.
+export function getMaestroHierarchyDrift() {
+  return maestroHierarchyDrift;
+}
+
+// Test helper — resets the drift envelope between specs. Not exported on the
+// public surface (consumers shouldn't reset module state in production).
+// The gRPC client cache is per-Percy-instance; tests pass a fresh `Map()` per
+// spec rather than going through a module-state resetter.
+export const __testing = {
+  resetMaestroHierarchyDrift() {
+    maestroHierarchyDrift = {
+      android: null,
+      ios: null
+    };
+  }
+};
+
+// Default Node http.request wrapper. Returns
+//   { statusCode, headers, body }
+// on completed responses (any status code), or throws an Error with .code
+// (e.g. ECONNREFUSED, ETIMEDOUT, ECONNRESET) on transport failures.
+//
+// Tests inject a fake httpRequest with the same shape; see
+// makeFakeHttpRequest in maestro-hierarchy.test.js iOS HTTP describe block.
+/* istanbul ignore next — production-only http transport; the unit suite
+   always injects an httpRequest stub, so this function is never invoked
+   under coverage. Integration tests on BS hosts exercise the real Node
+   http.request against a live Maestro iOS XCTestRunner. */
+function defaultHttpRequest({
+  host,
+  port,
+  method,
+  path: requestPath,
+  headers,
+  body,
+  timeoutMs
+}) {
+  return new Promise((resolve, reject) => {
+    let chunks = [];
+    let totalBytes = 0;
+    const req = http.request({
+      host,
+      port,
+      method,
+      path: requestPath,
+      headers,
+      timeout: timeoutMs
+    }, res => {
+      res.on('data', chunk => {
+        totalBytes += chunk.length;
+        /* istanbul ignore if — runaway response cap; Maestro upstream never
+           produces >IOS_HTTP_RESPONSE_MAX_BYTES (16 MB) responses in practice.
+           Defensive guard against pathological iOS payloads. */
+        if (totalBytes > IOS_HTTP_RESPONSE_MAX_BYTES) {
+          // Cap before parse — defensive against runaway responses.
+          chunks = null;
+          try {
+            req.destroy();
+          } catch {/* already destroyed */}
+          reject(Object.assign(new Error('response-too-large'), {
+            code: 'EMSGSIZE'
+          }));
+          return;
+        }
+        if (chunks) chunks.push(chunk);
+      });
+      res.on('end', () => {
+        /* istanbul ignore if — `chunks === null` only after the response-too-large
+           cap above rejected; end fires anyway as the response stream closes. */
+        if (!chunks) return; // already rejected for size
+        resolve({
+          statusCode: res.statusCode,
+          headers: res.headers,
+          body: Buffer.concat(chunks).toString('utf8')
+        });
+      });
+      /* istanbul ignore next — Node http response error path; only fires
+         on mid-stream FIN/RST. Connection failures land in req.on('error'). */
+      res.on('error', reject);
+    });
+
+    /* istanbul ignore next — Node http socket timeout path; covered by the
+     concurrent-access integration harness, not the unit suite. */
+    req.on('timeout', () => {
+      try {
+        req.destroy();
+      } catch {/* already destroyed */}
+      reject(Object.assign(new Error('socket-timeout'), {
+        code: 'ETIMEDOUT'
+      }));
+    });
+    req.on('error', reject);
+    if (body !== undefined && body !== null) req.write(body);
+    req.end();
+  });
+}
+
+// Validate PERCY_IOS_DRIVER_HOST_PORT env value as integer in the realmobile
+// range (wda_port + 2700 = 11100-11110). Out-of-range values return null.
+function parseIosDriverHostPort(raw) {
+  /* istanbul ignore if — undefined/null/empty raw value branch; iOS dispatch
+     pre-checks PERCY_IOS_DRIVER_HOST_PORT before calling, so these never fire. */
+  if (raw === undefined || raw === null || raw === '') return null;
+  const n = Number(raw);
+  /* istanbul ignore if — non-integer port (e.g. NaN from non-numeric env);
+     env var is set by realmobile as the canonical wda_port+2700 integer. */
+  if (!Number.isInteger(n)) return null;
+  if (n < IOS_DRIVER_HOST_PORT_MIN || n > IOS_DRIVER_HOST_PORT_MAX) return null;
+  return n;
+}
+
+// Walk the AXElement tree from cli-2.0.7's HTTP /viewHierarchy response.
+// Find the AUT root: first node with `elementType === 1` (XCUI application)
+// whose `identifier !== 'com.apple.springboard'`. Returns the AUT subtree, OR
+// null if the only application is SpringBoard (AUT-not-running case).
+//
+// At cli-2.0.7 the wrap is `[appHierarchy, statusBarsContainer]` where the
+// statusBars container has `elementType: 0` (synthetic init). The AUT is the
+// first elementType==1 node and the rule selects it directly.
+//
+// At cli-1.39.13 the wrap was `[springboardHierarchy, appHierarchy]` where
+// both children have `elementType: 1`. The springboard-skip handles that.
+//
+// Post-PR-2402 forward-compat: when the response is a single-AUT root (no
+// wrap), the rule selects the root itself.
+function findAxAutRoot(axElement) {
+  /* istanbul ignore if — defensive input guard; runIosHttpDump pre-checks
+     parsed.axElement before calling this. */
+  if (!axElement || typeof axElement !== 'object') return null;
+  if (axElement.elementType === 1 && axElement.identifier !== 'com.apple.springboard') {
+    return axElement;
+  }
+  const children = axElement.children;
+  if (Array.isArray(children)) {
+    for (const child of children) {
+      const found = findAxAutRoot(child);
+      if (found) return found;
+    }
+  }
+  return null;
+}
+
+// Adapter: walk an AXElement subtree (HTTP /viewHierarchy path) and emit nodes
+// in the canonical shape that firstMatch consumes for Android. Specifically:
+//   { 'resource-id': identifier, id: identifier, bounds: '[X,Y][X+W,Y+H]' }
+// Notably no `class` attribute — Maestro's iOS TreeNode doesn't expose
+// elementType→class either, and Percy keeps both iOS paths symmetric.
+//
+// Returns an array of nodes; throws if any frame is malformed (caught by
+// caller and surfaced as schema-class drift).
+function flattenIosAxElement(axRoot) {
+  const nodes = [];
+  const walk = obj => {
+    /* istanbul ignore if — defensive guard on recursive walk; AUT root and
+       its children are always objects per the Maestro AXElement contract. */
+    if (!obj || typeof obj !== 'object') return;
+    /* istanbul ignore next — identifier ternary; AXElement payloads always
+       carry a string identifier when present. */
+    const identifier = typeof obj.identifier === 'string' ? obj.identifier : '';
+    const frame = obj.frame;
+    /* istanbul ignore if — Maestro AXElement payloads always carry a `frame`
+       object per the upstream contract; this defends against a regression
+       where frame is missing or non-object. */
+    if (!frame || typeof frame !== 'object') {
+      throw new Error(`missing-frame on identifier=${JSON.stringify(identifier).slice(0, 64)}`);
+    }
+    const x = frame.X;
+    const y = frame.Y;
+    const w = frame.Width;
+    const h = frame.Height;
+    /* istanbul ignore if — Maestro AXElement frames use uppercased X/Y/Width/Height
+       keys per the upstream contract; this defends against case-mismatched
+       payloads from a Maestro regression. */
+    if (!Number.isFinite(x) || !Number.isFinite(y) || !Number.isFinite(w) || !Number.isFinite(h)) {
+      throw new Error(`frame-key-case-mismatch on identifier=${JSON.stringify(identifier).slice(0, 64)}`);
+    }
+    const bounds = `[${Math.round(x)},${Math.round(y)}][${Math.round(x + w)},${Math.round(y + h)}]`;
+    /* istanbul ignore else — identifier-empty branch (anonymous nodes) is
+     * the no-op tail; fixtures always carry identifiers on capturable nodes. */
+    if (identifier) {
+      nodes.push({
+        'resource-id': identifier,
+        id: identifier,
+        // text/content-desc/class deliberately undefined — iOS Maestro doesn't
+        // surface these as selector-relevant attributes (per IOSDriver.kt).
+        bounds
+      });
+    }
+    if (Array.isArray(obj.children)) {
+      for (const child of obj.children) walk(child);
+    }
+  };
+  walk(axRoot);
+  return nodes;
+}
+
+// Classify an iOS HTTP failure into connection-class (route to fallback) vs
+// schema-class (set drift bit, no fallback) vs no-aut-tree (route to
+// fallback because the Maestro CLI knows the AUT internally).
+function classifyIosHttpFailure(err) {
+  var _err$message;
+  /* istanbul ignore if — defensive null guard; callers always pass an err
+     when invoking the classifier. */
+  if (!err) return null;
+  const code = err.code;
+  // Connection-class errors — Maestro runner unreachable / unhealthy. Fall back.
+  /* istanbul ignore next — OR-chain branches: tests cover one or two codes
+     (typically ECONNREFUSED + ETIMEDOUT) but not all 8. Each remaining code
+     creates an unevaluated branch. */
+  if (code === 'ECONNREFUSED' || code === 'ETIMEDOUT' || code === 'ECONNRESET' || code === 'EHOSTUNREACH' || code === 'ENETUNREACH' || code === 'EPIPE' || code === 'ECONNABORTED' || code === 'EMSGSIZE') {
+    return {
+      kind: 'connection-fail',
+      reason: `http-${String(code).toLowerCase()}`
+    };
+  }
+  // Default: treat unknown errors as connection-class so we fall back rather
+  // than silently skip element regions.
+  /* istanbul ignore next — defensive fallback for error shapes outside the
+     explicit code list; unit tests exercise every named code
+     (ECONNREFUSED/ETIMEDOUT/ECONNRESET/...). */
+  return {
+    kind: 'connection-fail',
+    reason: `http-${((_err$message = err.message) === null || _err$message === void 0 ? void 0 : _err$message.slice(0, 64)) || 'unknown'}`
+  };
+}
+
+// iOS HTTP primary path. POSTs `{appIds: [], excludeKeyboardElements: false}`
+// to Maestro's iOS XCTestRunner /viewHierarchy endpoint. Returns
+//   { kind: 'hierarchy', nodes }     on success
+//   { kind: 'connection-fail', ... } on transport / 5xx / out-of-range port
+//   { kind: 'no-aut-tree', ... }     on SpringBoard-only response
+//   { kind: 'dump-error', ... }      on schema-class failures (no fallback)
+async function runIosHttpDump({
+  port,
+  sessionId,
+  httpRequest
+}) {
+  /* istanbul ignore next — fallback to default http transport when caller
+     omits; tests always inject a stub. */
+  httpRequest = httpRequest || defaultHttpRequest;
+  // Loopback-only guard. Hardcoded host; do not accept from caller input.
+  const host = '127.0.0.1';
+  let response;
+  const requestBody = JSON.stringify({
+    appIds: [],
+    excludeKeyboardElements: false
+  });
+  try {
+    response = await Promise.race([httpRequest({
+      host,
+      port,
+      method: 'POST',
+      path: '/viewHierarchy',
+      headers: {
+        'content-type': 'application/json',
+        'content-length': Buffer.byteLength(requestBody)
+      },
+      body: requestBody,
+      timeoutMs: IOS_HTTP_HEALTHY_DEADLINE_MS
+    }), new Promise((_, reject) => setTimeout(() => reject(Object.assign(new Error('circuit-breaker'), {
+      code: 'ETIMEDOUT'
+    })), IOS_HTTP_CIRCUIT_BREAKER_MS))]);
+  } catch (err) {
+    return classifyIosHttpFailure(err);
+  }
+  const {
+    statusCode,
+    headers,
+    body
+  } = response;
+
+  // 5xx → connection-class (server reachable but unhealthy).
+  if (statusCode >= 500) {
+    return {
+      kind: 'connection-fail',
+      reason: `http-${statusCode}`
+    };
+  }
+  // 4xx → schema-class (request shape problem; fallback wouldn't help).
+  if (statusCode >= 400) {
+    return {
+      kind: 'dump-error',
+      reason: `http-${statusCode}-bad-request-shape`
+    };
+  }
+  // 3xx → unexpected; treat as schema-class.
+  /* istanbul ignore next — Maestro upstream returns only 200/4xx; 3xx is a
+     theoretical defensive path. The 4xx branch above is covered. */
+  if (statusCode !== 200) {
+    return {
+      kind: 'dump-error',
+      reason: `http-unexpected-status-${statusCode}`
+    };
+  }
+
+  // Content-type is informational only — Maestro's upstream
+  // ViewHierarchyHandler.swift constructs `HTTPResponse(statusCode:.ok, body:body)`
+  // without setting Content-Type (FlyingFox HTTP server doesn't auto-set one).
+  // Body IS valid JSON regardless. Strict CT-required check would silently
+  // reject every response from real Maestro builds — relax to a soft warn
+  // and let JSON.parse decide. Schema-class drift only fires on actual
+  // parse failure or missing axElement root below.
+  const contentType = headers && (headers['content-type'] || headers['Content-Type']);
+  if (!contentType || !/application\/json/i.test(contentType)) {
+    log.debug(`iOS HTTP response missing/non-JSON content-type (got ${contentType || 'none'}); attempting parse anyway`);
+  }
+
+  // Parse JSON.
+  let parsed;
+  try {
+    parsed = JSON.parse(body);
+  } catch (err) {
+    var _err$message2;
+    /* istanbul ignore next — `err.message?.slice` optional chain + `|| 'unknown'`
+       fallback branches; tests pass JSON-parse errors which always have .message. */
+    return {
+      kind: 'dump-error',
+      reason: `http-parse-error:${((_err$message2 = err.message) === null || _err$message2 === void 0 ? void 0 : _err$message2.slice(0, 64)) || 'unknown'}`
+    };
+  }
+
+  // Schema validation: require `axElement` root.
+  if (!parsed || typeof parsed !== 'object' || !parsed.axElement || typeof parsed.axElement !== 'object') {
+    return {
+      kind: 'dump-error',
+      reason: 'http-missing-root'
+    };
+  }
+
+  // Find AUT root, skipping SpringBoard.
+  const aut = findAxAutRoot(parsed.axElement);
+  if (!aut) {
+    // Either the response is SpringBoard-only (AUT not running), or no
+    // application node at all. Either way, route to fallback.
+    return {
+      kind: 'no-aut-tree',
+      reason: 'springboard-only'
+    };
+  }
+
+  // Flatten the AUT subtree to firstMatch's expected node shape.
+  let nodes;
+  try {
+    nodes = flattenIosAxElement(aut);
+    /* istanbul ignore next — flattenIosAxElement throws only on malformed
+       AXElement payloads (Maestro-upstream contract); the catch body below
+       maps the three known message shapes to dump-error reasons. Unit tests
+       exercise the happy path; the throw paths are integration-test territory. */
+  } catch (err) {
+    /* istanbul ignore next — err.message || 'unknown' branch fires only when
+       a thrown value has no .message; the named-throw sites always set one. */
+    const msg = err.message || 'unknown';
+    /* istanbul ignore next */
+    if (/^missing-frame/.test(msg)) return {
+      kind: 'dump-error',
+      reason: 'http-missing-frame'
+    };
+    /* istanbul ignore next */
+    if (/^frame-key-case-mismatch/.test(msg)) return {
+      kind: 'dump-error',
+      reason: 'http-frame-key-case-mismatch'
+    };
+    /* istanbul ignore next */
+    return {
+      kind: 'dump-error',
+      reason: `http-flatten-error:${msg.slice(0, 64)}`
+    };
+  }
+  // Suppress sessionId in log surface — only emit a hash-prefix so support can
+  // correlate without leaking the full id.
+  /* istanbul ignore next — sid=none ternary branch fires only when sessionId
+     is missing; relay always passes one. */
+  const sidTag = sessionId ? `sid=${String(sessionId).slice(0, 8)}…` : 'sid=none';
+  log.debug(`runIosHttpDump ok ${sidTag} nodes=${nodes.length}`);
+  return {
+    kind: 'hierarchy',
+    nodes
+  };
+}
+
+// iOS maestro-CLI fallback path. Spawns
+// `maestro --udid <udid> --driver-host-port <port> hierarchy` and parses
+// stdout (Maestro's normalized TreeNode shape, identical to Android).
+// Existing flattenMaestroNodes consumes TreeNode unchanged — no iOS-specific
+// branching needed on this path.
+async function runMaestroIosDump(udid, driverHostPort, execMaestro, getEnv) {
+  const result = await execMaestro(['--udid', udid, '--driver-host-port', String(driverHostPort), 'hierarchy'], getEnv);
+  const fail = classifyMaestroFailure(result);
+  if (fail) return fail;
+  /* istanbul ignore next — non-zero exit with no classified failure;
+     classifyMaestroFailure catches the dominant exit cases. The `?? 1`
+     fallback also counts as a branch — both ignored together via `next`. */
+  if ((result.exitCode ?? 1) !== 0) {
+    return {
+      kind: 'dump-error',
+      reason: `maestro-exit-${result.exitCode}`
+    };
+  }
+  /* istanbul ignore next — `|| ''` branch fires only on missing/empty stdout
+     which the spawn helpers already normalize to a string. */
+  const stdout = result.stdout || '';
+  const start = stdout.indexOf('{');
+  if (start < 0) return {
+    kind: 'dump-error',
+    reason: 'maestro-no-json'
+  };
+  try {
+    const parsed = JSON.parse(stdout.slice(start));
+    const nodes = flattenMaestroNodes(parsed);
+    return {
+      kind: 'hierarchy',
+      nodes
+    };
+  } catch (err) {
+    /* istanbul ignore next — Maestro CLI's JSON output is structurally
+       stable; this rescues a parse-error from an upstream regression we
+       don't own. Happy-path JSON parsing is covered by the fixture tests. */
+    return {
+      kind: 'dump-error',
+      reason: `maestro-parse-error:${err.message}`
+    };
+  }
+}
+async function runMaestroDump(serial, execMaestro, getEnv) {
+  const result = await execMaestro(['--udid', serial, 'hierarchy'], getEnv);
+  const fail = classifyMaestroFailure(result);
+  if (fail) return fail;
+  /* istanbul ignore next — non-zero exit with no classified failure;
+     classifyMaestroFailure catches the dominant exit cases. The `?? 1`
+     fallback also counts as a branch — both ignored together via `next`. */
+  if ((result.exitCode ?? 1) !== 0) {
+    return {
+      kind: 'dump-error',
+      reason: `maestro-exit-${result.exitCode}`
+    };
+  }
+  // Maestro prints the JSON to stdout; sometimes CLI prefixes a banner/notice line.
+  // Slice from the first '{' to be safe.
+  /* istanbul ignore next — `|| ''` branch fires only on missing/empty stdout
+     which the spawn helpers already normalize to a string. */
+  const stdout = result.stdout || '';
+  const start = stdout.indexOf('{');
+  if (start < 0) return {
+    kind: 'dump-error',
+    reason: 'maestro-no-json'
+  };
+  try {
+    const parsed = JSON.parse(stdout.slice(start));
+    const nodes = flattenMaestroNodes(parsed);
+    return {
+      kind: 'hierarchy',
+      nodes
+    };
+  } catch (err) {
+    /* istanbul ignore next — Maestro CLI's JSON output is structurally
+       stable; this rescues a parse-error from an upstream regression we
+       don't own. Happy-path JSON parsing is covered by the fixture tests. */
+    return {
+      kind: 'dump-error',
+      reason: `maestro-parse-error:${err.message}`
+    };
+  }
+}
+
+// Adb fallback chain: exec-out uiautomator dump → file-based dump (with
+// SIGKILL retry loop) → cat from sdcard. Extracted from dump() so the gRPC
+// contention-class branch can jump straight here without going through
+// maestro CLI (which would queue behind the same Maestro flow that caused
+// the contention).
+async function runAdbFallback(serial, execAdb) {
+  let result = await runDump(['-s', serial, 'exec-out', 'uiautomator', 'dump', '/dev/tty'], execAdb);
+  const isRetryableDumpError = result.kind === 'dump-error' && (result.reason === 'no-xml-envelope' || /^exit-/.test(result.reason));
+  /* istanbul ignore if — adb file-dump fallback chain; only fires when the
+     exec-out primary returned a retryable dump-error (no-xml-envelope or
+     exit-N). Tests cover the primary success path; the retry chain is
+     integration-test territory (BS hosts running real uiautomator). */
+  if (isRetryableDumpError) {
+    log.debug(`adb primary dump returned ${result.reason}, trying file dump`);
+    let dumpToFile = await execAdb(['-s', serial, 'shell', 'uiautomator', 'dump', '/sdcard/window_dump.xml']);
+    for (const delay of SIGKILL_RETRY_DELAYS_MS) {
+      if ((dumpToFile.exitCode ?? 1) !== SIGKILL_EXIT) break;
+      log.debug(`fallback dump was killed (exit ${SIGKILL_EXIT}), retrying after ${delay}ms`);
+      await new Promise(r => setTimeout(r, delay));
+      dumpToFile = await execAdb(['-s', serial, 'shell', 'uiautomator', 'dump', '/sdcard/window_dump.xml']);
+    }
+    const dumpFail = classifyAdbFailure(dumpToFile);
+    if (dumpFail) return dumpFail;
+    if ((dumpToFile.exitCode ?? 1) !== 0) {
+      return {
+        kind: 'dump-error',
+        reason: `fallback-dump-exit-${dumpToFile.exitCode}`
+      };
+    }
+    result = await runDump(['-s', serial, 'exec-out', 'cat', '/sdcard/window_dump.xml'], execAdb);
+  }
+  return result;
+}
+export async function dump(options) {
+  /* istanbul ignore next — options-omitted default; callers always pass an
+     object (tests inject every dependency; production code binds them). */
+  options = options || {};
+  let {
+    platform,
+    sessionId,
+    execAdb,
+    execMaestro,
+    httpRequest,
+    grpcClient,
+    grpcClientCache,
+    getEnv
+  } = options;
+  /* istanbul ignore next — defaults applied only when caller omits the
+     corresponding key; tests inject every dependency, production callers
+     bind these from defaults at runtime. */
+  platform = platform || 'android';
+  /* istanbul ignore next */
+  execAdb = execAdb || defaultExecAdb;
+  /* istanbul ignore next */
+  execMaestro = execMaestro || defaultExecMaestro;
+  /* istanbul ignore next */
+  httpRequest = httpRequest || defaultHttpRequest;
+  /* istanbul ignore next */
+  grpcClient = grpcClient || defaultGrpcClientFactory;
+  /* istanbul ignore next */
+  getEnv = getEnv || defaultGetEnv;
+  const started = Date.now();
+  if (platform === 'ios') {
+    // iOS dispatch: read realmobile-injected env vars; warn-skip if absent.
+    const udid = getEnv('PERCY_IOS_DEVICE_UDID');
+    const driverHostPortRaw = getEnv('PERCY_IOS_DRIVER_HOST_PORT');
+    if (!udid || !driverHostPortRaw) {
+      log.warn(`iOS resolver env-missing: udid=${udid ? 'set' : 'unset'} driver_port=${driverHostPortRaw ? 'set' : 'unset'}`);
+      recordResolverFinalFailure({
+        platform: 'ios',
+        failureClass: 'other'
+      });
+      return {
+        kind: 'unavailable',
+        reason: 'env-missing'
+      };
+    }
+
+    // D3 kill switch (PERCY_MAESTRO_GRPC=0): same env name gates BOTH Maestro
+    // primaries. On iOS this skips runIosHttpDump and routes straight to the
+    // maestro-CLI fallback below. Read every call so toggling at runtime is
+    // honored without a CLI restart.
+    const iosKillSwitch = getEnv('PERCY_MAESTRO_GRPC') === '0';
+    if (iosKillSwitch) {
+      log.warn('PERCY_MAESTRO_GRPC=0 kill switch active; skipping iOS HTTP primary');
+    }
+
+    // Validate driver-host-port range before attempting HTTP. Out-of-range
+    // values skip the HTTP path entirely and fall through to maestro-CLI.
+    const driverHostPort = parseIosDriverHostPort(driverHostPortRaw);
+    let httpResult = null;
+    if (!iosKillSwitch && driverHostPort !== null) {
+      httpResult = await runIosHttpDump({
+        port: driverHostPort,
+        sessionId,
+        httpRequest
+      });
+      if (httpResult.kind === 'hierarchy') {
+        log.debug(`dump took ${Date.now() - started}ms via maestro-http (${httpResult.nodes.length} nodes)`);
+        recordResolverSuccess({
+          platform: 'ios',
+          via: 'maestro-http'
+        });
+        return httpResult;
+      }
+      if (httpResult.kind === 'dump-error') {
+        // Schema-class — no fallback per plan R4. Flip the iOS slot of the
+        // drift bit so /percy/healthcheck surfaces the contract mismatch
+        // for ops investigation. First-seen-per-platform wins.
+        setMaestroHierarchyDrift({
+          platform: 'ios',
+          code: undefined,
+          reason: httpResult.reason
+        });
+        recordResolverFinalFailure({
+          platform: 'ios',
+          failureClass: 'schema-class'
+        });
+        log.warn(`iOS HTTP schema-drift: ${httpResult.reason}`);
+        return httpResult;
+      }
+      // Otherwise (connection-fail or no-aut-tree): fall through to CLI.
+      const httpClass = failureClassFromReason(httpResult.reason);
+      recordResolverFallback({
+        platform: 'ios',
+        failureClass: httpClass
+      });
+      log.info(`[percy] hierarchy: maestro-http failed (${httpClass}: ${httpResult.reason}) → falling back to maestro-cli-fallback`);
+    } else if (!iosKillSwitch) {
+      const oorReason = `out-of-range-port-${driverHostPortRaw}`;
+      recordResolverFallback({
+        platform: 'ios',
+        failureClass: 'other'
+      });
+      log.info(`[percy] hierarchy: maestro-http failed (other: ${oorReason}) → falling back to maestro-cli-fallback`);
+    }
+    const cliResult = await runMaestroIosDump(udid, driverHostPort ?? driverHostPortRaw, execMaestro, getEnv);
+    const httpReason = httpResult ? `${httpResult.kind}/${httpResult.reason}` : 'out-of-range-port';
+    log.debug(`dump took ${Date.now() - started}ms via maestro-cli-fallback (${httpReason}) kind=${cliResult.kind}`);
+    if (cliResult.kind === 'hierarchy') {
+      recordResolverSuccess({
+        platform: 'ios',
+        via: 'maestro-cli-fallback'
+      });
+    } else {
+      recordResolverFinalFailure({
+        platform: 'ios',
+        failureClass: failureClassFromReason(cliResult.reason)
+      });
+    }
+    return cliResult;
+  }
+
+  // Android (default).
+  const {
+    serial,
+    classification
+  } = await resolveSerial({
+    execAdb,
+    getEnv
+  });
+  if (classification) {
+    log.warn(`adb unavailable: ${classification.reason}`);
+    recordResolverFinalFailure({
+      platform: 'android',
+      failureClass: 'other'
+    });
+    return classification;
+  }
+
+  // gRPC primary path (env-conditional + kill-switch-gated). Talks the same
+  // gRPC transport Maestro CLI uses, but as a stateless RPC that doesn't
+  // open a parallel Maestro flow context — avoids the session-collision
+  // failure mode the CLI shell-out hits during a live Maestro flow.
+  //
+  // D3 kill switch (PERCY_MAESTRO_GRPC=0): in-process emergency disable.
+  // Distinct from removing the env injection (which requires a coordinated
+  // mobile/realmobile deploy). Logged loudly so the rollback state is
+  // observable in CLI logs.
+  const killSwitch = getEnv('PERCY_MAESTRO_GRPC') === '0';
+  const grpcPortRaw = getEnv('PERCY_ANDROID_GRPC_PORT');
+  let skipMaestroCli = false;
+  if (killSwitch) {
+    log.warn('PERCY_MAESTRO_GRPC=0 kill switch active; skipping gRPC primary');
+  } else if (grpcPortRaw && grpcClientCache) {
+    const grpcPort = Number.parseInt(grpcPortRaw, 10);
+    if (Number.isInteger(grpcPort) && grpcPort > 0 && grpcPort <= 65535) {
+      const grpcResult = await runAndroidGrpcDump({
+        host: '127.0.0.1',
+        port: grpcPort,
+        grpcClient,
+        cache: grpcClientCache,
+        shutdownInProgress: grpcClientCache.shutdownInProgress
+      });
+      if (grpcResult.kind === 'hierarchy') {
+        log.debug(`dump took ${Date.now() - started}ms via grpc (${grpcResult.nodes.length} nodes)`);
+        recordResolverSuccess({
+          platform: 'android',
+          via: 'grpc'
+        });
+        return grpcResult;
+      }
+      /* istanbul ignore next — R-7 shutdown-in-progress race: only triggers
+         when stop() is called concurrently with an in-flight dump. The `&&`
+         second clause also counts as a branch — use `ignore next` to cover
+         the whole if-statement including the condition expression. */
+      if (grpcResult.kind === 'unavailable' && grpcResult.reason === 'shutdown') {
+        // R-7: shutdown-in-progress. Don't spawn fallback chain on a tearing-down process.
+        log.debug('gRPC dump cancelled by shutdown; skipping fallback chain');
+        recordResolverFinalFailure({
+          platform: 'android',
+          failureClass: 'other'
+        });
+        return grpcResult;
+      }
+      if (grpcResult.kind === 'dump-error') {
+        // Schema-class — no fallback per D10. Drift bit set inside runAndroidGrpcDump.
+        recordResolverFinalFailure({
+          platform: 'android',
+          failureClass: 'schema-class'
+        });
+        return grpcResult;
+      }
+      // connection-fail: split contention-class vs channel-broken per D10.
+      const grpcClass = failureClassFromReason(grpcResult.reason);
+      if (isContentionClass(grpcResult.reason)) {
+        // Contention-class: skip maestro CLI (would queue behind same flow); jump to adb.
+        recordResolverFallback({
+          platform: 'android',
+          failureClass: grpcClass
+        });
+        log.info(`[percy] hierarchy: grpc failed (${grpcClass}: ${grpcResult.reason}) → falling back to adb`);
+        skipMaestroCli = true;
+      } else {
+        // Channel-broken: fall through to maestro CLI (CLI re-establishes the channel).
+        recordResolverFallback({
+          platform: 'android',
+          failureClass: grpcClass
+        });
+        log.info(`[percy] hierarchy: grpc failed (${grpcClass}: ${grpcResult.reason}) → falling back to maestro-cli`);
+      }
+    } else {
+      log.debug(`PERCY_ANDROID_GRPC_PORT=${grpcPortRaw} invalid; skipping gRPC primary`);
+    }
+  }
+
+  // Maestro CLI primary (or fallback when gRPC channel-broken). Skipped on
+  // gRPC contention-class — that path goes straight to adb.
+  if (!skipMaestroCli) {
+    const maestroResult = await runMaestroDump(serial, execMaestro, getEnv);
+    if (maestroResult.kind === 'hierarchy') {
+      log.debug(`dump took ${Date.now() - started}ms via maestro-cli (${maestroResult.nodes.length} nodes)`);
+      recordResolverSuccess({
+        platform: 'android',
+        via: 'maestro-cli'
+      });
+      return maestroResult;
+    }
+    recordResolverFallback({
+      platform: 'android',
+      failureClass: failureClassFromReason(maestroResult.reason)
+    });
+    log.info(`[percy] hierarchy: maestro-cli failed (${failureClassFromReason(maestroResult.reason)}: ${maestroResult.reason}) → falling back to adb`);
+  }
+
+  // adb fallback (final).
+  const result = await runAdbFallback(serial, execAdb);
+  log.debug(`dump took ${Date.now() - started}ms via adb (kind=${result.kind})`);
+  /* istanbul ignore else — adb final fallback is the last resort; tests
+     stub the resolver chain to always resolve via grpc/maestro-cli before
+     reaching here in the failure case. */
+  if (result.kind === 'hierarchy') {
+    recordResolverSuccess({
+      platform: 'android',
+      via: 'adb'
+    });
+  } else {
+    recordResolverFinalFailure({
+      platform: 'android',
+      failureClass: failureClassFromReason(result.reason)
+    });
+  }
+  return result;
+}
+function parseBounds(str) {
+  /* istanbul ignore if — defensive null guard; callers always pass the
+     bounds attribute string from a node that matched the regex. */
+  if (!str) return null;
+  const m = BOUNDS_RE.exec(str);
+  if (!m) return null;
+  const x1 = Number(m[1]);
+  const y1 = Number(m[2]);
+  const x2 = Number(m[3]);
+  const y2 = Number(m[4]);
+  /* istanbul ignore if — defensive degenerate-bounds guard
+     ([0,0][0,0] from SpringBoard-only AUT roots); fixtures use
+     well-formed bounds, this guard is for empty AUT subtrees. */
+  if (x2 <= x1 || y2 <= y1) return null;
+  return {
+    x: x1,
+    y: y1,
+    width: x2 - x1,
+    height: y2 - y1
+  };
+}
+export function firstMatch(nodes, selector) {
+  /* istanbul ignore if — defensive input validation; callers always pass
+     a hierarchy nodes array and a valid selector object. */
+  if (!Array.isArray(nodes) || !selector || typeof selector !== 'object') return null;
+  const keys = Object.keys(selector);
+  if (keys.length !== 1) return null;
+  const key = keys[0];
+  if (!SELECTOR_KEYS_UNION.includes(key)) return null;
+  const value = selector[key];
+  if (typeof value !== 'string' || value.length === 0) return null;
+  for (const node of nodes) {
+    if (node[key] !== value) continue;
+    const bbox = parseBounds(node.bounds);
+    if (bbox) return bbox;
+  }
+  return null;
+}
+
+// Exposed for tests + handler-side validation in api.js. Union of platform
+// keys; per-platform validation is implicit in the node shape returned by
+// dump() — Android nodes carry resource-id/text/content-desc/class plus the
+// `id` alias; iOS nodes carry `id` only (Maestro's iOS TreeNode does not
+// surface `class`).
+export const SELECTOR_KEYS_WHITELIST = SELECTOR_KEYS_UNION;
+export const ANDROID_SELECTOR_KEYS_WHITELIST = ANDROID_SELECTOR_KEYS;
+export const IOS_SELECTOR_KEYS_WHITELIST = IOS_SELECTOR_KEYS;