peachsim 0.1.628 → 0.1.630

package/README.md

@@ -15,8 +15,10 @@ The executable updates itself with `peach upgrade`. Successful interactive human
 commands refresh a cached stable CLI version in a detached process at most once
 every 20 hours. A newer version produces one short notice on stderr, once per
 version and no more than weekly. CI, agents, JSON output, pipes, and
-noninteractive commands stay silent. Set `PEACH_UPDATE_CHECK=off` to disable the
-check and notice; `peach version` checks explicitly, while `peach --version` remains
+noninteractive commands stay silent. Set `PEACH_UPDATE_CHECK=off` to disable
+automatic CLI checks, runtime updates, and their notices. Start the bridge with
+this environment, or reinstall the background service with it to persist the choice.
+`peach version` checks explicitly, while `peach --version` remains
 offline.
 
 Neither artifact contains rendering code. The canvas engine (CanvasKit
@@ -233,7 +235,7 @@ Pass a target to `peach open <port-or-url>` when a script or agent needs an expl
 selection.
 
 The first `peach open` launches an isolated Chrome for Testing sim. The CLI
-resolves Playwright from the app or CLI install, asks that exact package for
+resolves Playwright from the current directory or beside its executable, asks that exact package for
 its required browser executable, and runs that package's installer once when
 the executable is absent. It never substitutes another revision found in the
 shared Playwright cache. Later opens reuse that sim and reload it at the
@@ -557,7 +559,9 @@ frame for quick art-direction passes.
 ## Skills registry
 
 `peach skill install` installs Peach agent skills in the standard
-`<skill-name>/SKILL.md` layout used by Codex and Claude Code. The bundled
+`<skill-name>/SKILL.md` layout used by Codex and Claude Code. Building the CLI
+only writes build artifacts; run `peach skill install` explicitly to install
+or refresh agent skills. The bundled
 skills cover setup, debugging/perf/accessibility, testing, visual review, and
 the generated `contrast` command index. The published CLI registry and generated
 website docs both come from `packages/peach-skills/`:

package/cli/bridge-flow-runner.ts

@@ -1,6 +1,6 @@
 import * as fs from 'fs'
 import * as path from 'path'
-import { devices, type DeviceModel } from 'peach-engine/settings'
+import { devices, type DeviceModel } from 'peach-engine/settings/devices'
 import yaml from 'yaml'
 import { REQUIRED_IDENTICAL_FLOW_FRAMES, framesIdentical } from '../detox/proof-frame.cjs'
 import { flowTimeoutScale } from '../src/flow-timeout-scale'
@@ -1314,6 +1314,9 @@ export class SootSimBridgeFlowRunner {
 
   async tapOn(target: string | FlowPointTarget) {
     const opts = typeof target === 'string' ? { text: target } : target
+    // maestro settles before resolving a tap's final position as well as after
+    // delivering it. a preceding assertion does not settle a pending update.
+    await this.waitForAppToSettle()
     const repeat = Math.max(1, Math.trunc(opts.repeat ?? 1))
     const repeatDelay = opts.delay !== undefined && opts.delay >= 0 ? opts.delay : 100
     // maestro Maestro.hierarchyBasedTap with a TapRepeat: every tap is padded
@@ -2379,14 +2382,14 @@ export class SootSimBridgeFlowRunner {
     }
   }
 
-  // maestro waits after every input for the screen to stop changing, up to
+  // maestro waits around taps for the screen to stop changing, up to
   // its ios driver's 3 s settle budget, comparing screenshots, so an action
   // still in flight (a submit spinner, a sheet closing) finishes before the
   // next command resolves its target. an app can take a few hundred ms to
   // react to a press, so stillness must hold about half a second, and it
   // counts painted frames because a spinner repaints without moving layout.
   // a screen that never stops proceeds when the budget runs out, as there.
-  private async settleAfterInput() {
+  private async waitForAppToSettle() {
     await waitForSootsimIdle({
       bridge: this.bridge,
       simId: this.opts.simId,
@@ -2396,6 +2399,10 @@ export class SootSimBridgeFlowRunner {
       strict: true,
       paintStill: true,
     })
+  }
+
+  private async settleAfterInput() {
+    await this.waitForAppToSettle()
     this.lastInteractionAt = Date.now()
   }
 

package/cli/cli-update.ts

@@ -1,6 +1,7 @@
 import fs from 'node:fs'
 import path from 'node:path'
 import { compareSemver } from '@contrast/runtime-delivery'
+import { automaticPeachUpdatesEnabled } from '../src/automatic-update-policy'
 import { getCliVersion } from '../src/cli-version'
 import { cliUpdateFilePath } from '../src/home-paths'
 import { isAgentEnv } from './commands/inspect/env'
@@ -127,12 +128,6 @@ function envFlagIsOn(value: string | undefined): boolean {
   return Boolean(value) && !envFlagIsOff(value)
 }
 
-export function automaticCliUpdateChecksEnabled(
-  env: Readonly<Record<string, string | undefined>> = process.env,
-): boolean {
-  return !envFlagIsOff(env.PEACH_UPDATE_CHECK)
-}
-
 export function shouldUseAutomaticCliUpdateCheck(
   invocation: CliUpdateInvocation,
 ): boolean {
@@ -143,7 +138,7 @@ export function shouldUseAutomaticCliUpdateCheck(
     invocation.version ||
     !invocation.stdoutIsTTY ||
     !invocation.stderrIsTTY ||
-    !automaticCliUpdateChecksEnabled(env) ||
+    !automaticPeachUpdatesEnabled(env) ||
     envFlagIsOn(env.CI) ||
     envFlagIsOn(env.GITHUB_ACTIONS) ||
     envFlagIsOn(env.PEACH_NO_PROMPT) ||

package/cli/cloud-test-tunnel.ts

@@ -26,6 +26,7 @@ import {
 import { createServer, request as httpRequest } from 'node:http'
 import { tmpdir } from 'node:os'
 import { join } from 'node:path'
+import { pipeline } from 'node:stream'
 import { sleep } from '../src/sleep'
 
 // must match packages/contrast-runner/src/hostedPorts.ts.
@@ -160,12 +161,24 @@ export function startCloudTestTokenProxy(
       },
       (upRes) => {
         res.writeHead(upRes.statusCode || 502, upRes.headers)
-        upRes.pipe(res)
+        // a broken upstream body must break the downstream response too. pipe()
+        // leaves it open after an aborted transfer, stranding the tester.
+        pipeline(upRes, res, (error) => {
+          if (error) {
+            console.error(
+              `upstream body error ${req.method} ${req.url}: ${error.message}`,
+            )
+          }
+        })
       },
     )
     up.on('error', (err) => {
-      res.writeHead(502, { 'content-type': 'text/plain' })
-      res.end(`upstream error: ${err.message}`)
+      // a transport failure is a network error, including before headers
+      // arrive; ending a partial response would report a successful transfer.
+      res.destroy(err)
+    })
+    res.once('close', () => {
+      if (!res.writableFinished) up.destroy()
     })
     req.pipe(up)
   })

package/cli/commands/config.ts

@@ -11,7 +11,7 @@ usage:
 environment:
   PEACH_HINTS=off          suppress CLI hints
   PEACH_HINTS=always       show every CLI hint every time
-  PEACH_UPDATE_CHECK=off   disable background CLI update checks and notices
+  PEACH_UPDATE_CHECK=off   disable automatic CLI checks, runtime updates, and notices
 `)
 }
 

package/cli/commands/control.ts

@@ -8,7 +8,7 @@ import {
   listSelectableDeviceModels,
   resolveDeviceFitBox,
   type DeviceModel,
-} from 'peach-engine/settings'
+} from 'peach-engine/settings/devices'
 import { WebSocket } from 'ws'
 import { probePort } from '../../scripts/dev-server-scanner'
 import { isAppDeepLinkTarget, openAppUrl } from '../../src/app-url'
@@ -36,6 +36,7 @@ import {
   runtimeDir,
 } from '../../src/home-paths'
 import { applyHotMode } from '../../src/native-dev-bundle-url'
+import { readNativeFirebaseConfig } from '../../src/native-firebase-config'
 import { ensureProfile } from '../../src/profiles'
 import { rnxPublicBrand } from '../../src/public-brand'
 import {
@@ -774,6 +775,26 @@ async function stampAppSplash(
   }
 }
 
+function stampNativeFirebaseConfig(
+  urlStr: string,
+  project?: ResolvedAppProject | null,
+): string {
+  const url = new URL(urlStr)
+  if (!url.searchParams.get('bundle') || !project) return urlStr
+  try {
+    url.searchParams.set(
+      'nativeFirebase',
+      JSON.stringify(readNativeFirebaseConfig(project.appDir)),
+    )
+  } catch (error) {
+    url.searchParams.set(
+      'nativeFirebase',
+      JSON.stringify({ error: error instanceof Error ? error.message : String(error) }),
+    )
+  }
+  return url.toString()
+}
+
 // stamp the engine page url with `?appCatalog=` so an app's asset-catalog
 // colors and its declared accent reach the tenant, which applies them to the
 // bundle this url opens. the catalog lives in the native binary, never in
@@ -867,8 +888,11 @@ export async function buildShellUrl(
   // fallback typeface, which then reads as a conformance diff.
   const project =
     opts.project ?? (await resolveAppProjectForBundleUrl(bundleUrlOf(resolved)))
-  const stamped = stampAppCatalog(
-    await stampAppSplash(await stampAppFonts(resolved, project), project),
+  const stamped = stampNativeFirebaseConfig(
+    stampAppCatalog(
+      await stampAppSplash(await stampAppFonts(resolved, project), project),
+      project,
+    ),
     project,
   )
   if (
@@ -912,6 +936,7 @@ function deriveCurrentSimBaseUrl(
     // bundle.
     url.searchParams.delete('appSplash')
     url.searchParams.delete('appCatalog')
+    url.searchParams.delete('nativeFirebase')
     return url.toString()
   } catch {
     return fallback

package/cli/commands/daemon-mac-app.ts

@@ -135,8 +135,10 @@ function renderLauncherScript(
   const quoted = parts.map(shellSingleQuote).join(' ')
   const stdout = shellSingleQuote(logDir.stdout)
   const stderr = shellSingleQuote(logDir.stderr)
-  const peachHome = process.env.PEACH_HOME
-  const environment = peachHome ? `export PEACH_HOME=${shellSingleQuote(peachHome)}\n` : ''
+  const environment = ['PEACH_HOME', 'PEACH_UPDATE_CHECK']
+    .filter((key) => process.env[key])
+    .map((key) => `export ${key}=${shellSingleQuote(process.env[key] ?? '')}\n`)
+    .join('')
   return `#!/bin/sh
 # peach daemon launcher generated by 'peach daemon install'.
 # launchd spawns this; we exec the real peach invocation in place so

package/cli/commands/daemon.ts

@@ -149,10 +149,10 @@ ${programArgs}
 function renderSystemdUnit(invocation: RnxSelfInvocation, port: number): string {
   mkdirSync(LOG_DIR_LINUX, { recursive: true })
   const exec = [invocation.executable, ...invocation.prefixArgs].map(shellQuote).join(' ')
-  const peachHome = process.env.PEACH_HOME
-  const environment = peachHome
-    ? `Environment=${shellQuote(`PEACH_HOME=${peachHome}`)}\n`
-    : ''
+  const environment = ['PEACH_HOME', 'PEACH_UPDATE_CHECK']
+    .filter((key) => process.env[key])
+    .map((key) => `Environment=${shellQuote(`${key}=${process.env[key]}`)}\n`)
+    .join('')
   return `[Unit]
 Description=peach bridge daemon
 After=default.target

package/cli/commands/device.ts

@@ -1,11 +1,10 @@
 import {
   devices,
-  DUO_REST_PROGRESS,
   getDevice,
   listSelectableDeviceModels,
   type DeviceModel,
-  type DuoFoldRest,
-} from 'peach-engine/settings'
+} from 'peach-engine/settings/devices'
+import { DUO_REST_PROGRESS, type DuoFoldRest } from 'peach-engine/settings/duo-fold-bus'
 import { SOOTSIM_SHELL_COMMAND_NAME } from '../../src/menu'
 import { rnxExit } from '../run-rnx'
 import {

package/cli/commands/inspect/actions.ts

@@ -332,6 +332,8 @@ function payloadFromResolved(
 interface TextCandidate {
   node: SimSemanticNode
   ancestorTestIDs: string[]
+  actionable: boolean
+  role?: string
 }
 
 // a non-pressable node repeating the text of its nearest labelled ancestor,
@@ -343,16 +345,28 @@ function collectTextCandidates(
   nodes: readonly SimSemanticNode[],
   ancestors: string[] = [],
   pressableOwnerText = '',
+  actionableAncestor = false,
+  actionableRole?: string,
 ): TextCandidate[] {
   const out: TextCandidate[] = []
   for (const node of nodes) {
     const text = nodeText(node)
     const caption = !!text && text === pressableOwnerText && !node.pressable
-    if (!caption) out.push({ node, ancestorTestIDs: ancestors })
+    const actionable = !!node.pressable || actionableAncestor
+    const role = node.role ?? actionableRole
+    if (!caption) out.push({ node, ancestorTestIDs: ancestors, actionable, role })
     if (node.children?.length) {
       const next = node.testID ? [...ancestors, node.testID] : ancestors
       const owner = text ? (node.pressable ? text : '') : pressableOwnerText
-      out.push(...collectTextCandidates(node.children, next, owner))
+      out.push(
+        ...collectTextCandidates(
+          node.children,
+          next,
+          owner,
+          actionable,
+          node.pressable ? node.role : actionableRole,
+        ),
+      )
     }
   }
   return out
@@ -388,93 +402,112 @@ export async function tapByText(
     agent: opts.agent,
     timing: opts.timing,
     textFallback: query,
-    resolve: async () => {
-      const { tree } = await inspectTree(bridge, 50)
-      if (!Array.isArray(tree)) return { matched: 0, total: 0 }
-      const needle = options.exact ? query : query.toLowerCase()
-      let candidates = collectTextCandidates(tree).filter(({ node, ancestorTestIDs }) => {
-        const text = nodeText(node)
-        if (!text) return false
-        if (options.exact ? text !== needle : !text.toLowerCase().includes(needle)) {
-          return false
-        }
-        if (options.role && node.role !== options.role) return false
-        if (options.within && !ancestorTestIDs.includes(options.within)) return false
-        const ax = node.geometry.x
-        const ay = node.geometry.y
-        if (options.minX != null && !(ax >= options.minX)) return false
-        if (options.maxX != null && !(ax <= options.maxX)) return false
-        if (options.minY != null && !(ay >= options.minY)) return false
-        if (options.maxY != null && !(ay <= options.maxY)) return false
-        return true
-      })
+    resolve: () => resolveTextTapTarget(bridge, query, options),
+  })
+}
 
-      if (options.near) {
-        const { x, y } = options.near
-        candidates.sort(
-          (a, b) =>
-            Math.hypot(a.node.geometry.x - x, a.node.geometry.y - y) -
-            Math.hypot(b.node.geometry.x - x, b.node.geometry.y - y),
-        )
-      } else {
-        candidates.sort((a, b) => {
-          if (Math.abs(a.node.geometry.y - b.node.geometry.y) > 2) {
-            return a.node.geometry.y - b.node.geometry.y
-          }
-          return a.node.geometry.x - b.node.geometry.x
-        })
+async function resolveTextTapTarget(
+  bridge: InspectBridge,
+  query: string,
+  options: TapTextOptions = {},
+) {
+  const { tree } = await inspectTree(bridge, 50)
+  if (!Array.isArray(tree)) return { matched: 0, total: 0 }
+  const needle = options.exact ? query : query.toLowerCase()
+  let candidates = collectTextCandidates(tree).filter(
+    ({ node, ancestorTestIDs, role }) => {
+      const text = nodeText(node)
+      if (!text) return false
+      if (options.exact ? text !== needle : !text.toLowerCase().includes(needle)) {
+        return false
       }
+      if (options.role && role !== options.role) return false
+      if (options.within && !ancestorTestIDs.includes(options.within)) return false
+      const ax = node.geometry.x
+      const ay = node.geometry.y
+      if (options.minX != null && !(ax >= options.minX)) return false
+      if (options.maxX != null && !(ax <= options.maxX)) return false
+      if (options.minY != null && !(ay >= options.minY)) return false
+      if (options.maxY != null && !(ay <= options.maxY)) return false
+      return true
+    },
+  )
 
-      const total = candidates.length
-      if (total === 0) return { matched: 0, total: 0 }
+  // default label taps prefer controls to prose mentioning the same word.
+  // explicit position selectors keep their full candidate list and order.
+  if (options.nth == null && !options.first && !options.near) {
+    const actionable = candidates.filter((candidate) => candidate.actionable)
+    if (actionable.length > 0) candidates = actionable
+  }
 
-      let idx = 0
-      if (options.nth != null) {
-        idx = options.nth < 0 ? total + options.nth : options.nth
-        if (idx < 0 || idx >= total) {
-          return { matched: 0, total, nthOutOfRange: true, nth: options.nth }
-        }
-      } else if (total > 1 && !options.first && !options.near) {
-        return {
-          ambiguous: true,
-          total,
-          candidates: candidates.slice(0, 10).map((candidate, index) => ({
-            idx: index,
-            nodeId: candidate.node.nodeId,
-            type: candidate.node.type,
-            testID: candidate.node.testID ?? null,
-            text: nodeText(candidate.node).slice(0, 60),
-            abs: { x: candidate.node.geometry.x, y: candidate.node.geometry.y },
-            layout: {
-              width: Math.round(candidate.node.geometry.width),
-              height: Math.round(candidate.node.geometry.height),
-            },
-            ancestorTestIDs: candidate.ancestorTestIDs.slice(0, 5),
-          })),
-        }
+  if (options.near) {
+    const { x, y } = options.near
+    candidates.sort(
+      (a, b) =>
+        Math.hypot(a.node.geometry.x - x, a.node.geometry.y - y) -
+        Math.hypot(b.node.geometry.x - x, b.node.geometry.y - y),
+    )
+  } else {
+    candidates.sort((a, b) => {
+      if (Math.abs(a.node.geometry.y - b.node.geometry.y) > 2) {
+        return a.node.geometry.y - b.node.geometry.y
       }
+      return a.node.geometry.x - b.node.geometry.x
+    })
+  }
 
-      const picked = candidates[idx]
-      if (!picked) return { matched: 0, total }
-      const node = picked.node
-      let resolved = node.testID
-        ? await resolveTargetCoords(bridge, { mode: 'testid', value: node.testID })
-        : await resolveTargetCoords(bridge, {
-            mode: 'text',
-            value: nodeText(node) || query,
-          })
-      if (!resolved) {
-        resolved = {
-          x: node.geometry.x + node.geometry.width / 2,
-          y: node.geometry.y + node.geometry.height / 2,
-          testID: node.testID ?? null,
-          text: nodeText(node) || query,
-          type: node.type,
-        }
-      }
-      return payloadFromResolved(resolved, node, { total, idx })
-    },
+  const total = candidates.length
+  if (total === 0) return { matched: 0, total: 0 }
+
+  let idx = 0
+  if (options.nth != null) {
+    idx = options.nth < 0 ? total + options.nth : options.nth
+    if (idx < 0 || idx >= total) {
+      return { matched: 0, total, nthOutOfRange: true, nth: options.nth }
+    }
+  } else if (total > 1 && !options.first && !options.near) {
+    return {
+      ambiguous: true,
+      total,
+      candidates: candidates.slice(0, 10).map((candidate, index) => ({
+        idx: index,
+        nodeId: candidate.node.nodeId,
+        type: candidate.node.type,
+        testID: candidate.node.testID ?? null,
+        text: nodeText(candidate.node).slice(0, 60),
+        abs: { x: candidate.node.geometry.x, y: candidate.node.geometry.y },
+        layout: {
+          width: Math.round(candidate.node.geometry.width),
+          height: Math.round(candidate.node.geometry.height),
+        },
+        ancestorTestIDs: candidate.ancestorTestIDs.slice(0, 5),
+      })),
+    }
+  }
+
+  const picked = candidates[idx]
+  if (!picked) return { matched: 0, total }
+  const node = picked.node
+  // keep the selected handle: re-searching its text or testID can pick a
+  // different control, including a covered tenant behind native UI.
+  const tap: unknown = await bridge.send({
+    type: 'call',
+    path: 'SootSim.bridges.test.resolveTapTarget',
+    args: [node.nodeId],
   })
+  if (!isTapTargetInfo(tap)) return { matched: 0, total }
+  return payloadFromResolved(
+    {
+      x: tap.cx,
+      y: tap.cy,
+      nodeId: tap.target.nodeId,
+      testID: tap.target.testID,
+      text: tap.target.text ?? tap.target.accessibilityLabel,
+      type: tap.target.type,
+    },
+    node,
+    { total, idx },
+  )
 }
 
 export type TapQuery =
@@ -570,17 +603,19 @@ export async function tapBest(
           },
         }
       }
-      const byText = await resolveTargetCoords(bridge, { mode: 'text', value: query })
-      if (byText) {
+      const byText = await resolveTextTapTarget(bridge, query)
+      if ('ambiguous' in byText) {
+        return {
+          ambiguous: true,
+          error: `multiple text matches for "${query}"; use tap-text with --nth, --within or --near`,
+        }
+      }
+      if ('cx' in byText && typeof byText.cx === 'number') {
         return {
-          ...payloadFromResolved(byText, byText),
+          ...byText,
           strategy: 'text',
           node: {
-            nodeId: byText.nodeId ?? null,
-            id: byText.testID ?? null,
-            testID: byText.testID ?? null,
-            type: byText.type ?? null,
-            text: byText.text ?? query,
+            ...byText.target,
           },
         }
       }

package/cli/commands/inspect/core.ts

@@ -544,6 +544,11 @@ export interface LayoutElement {
   lastLineWords?: number
   // last line width relative to the widest line (0-1).
   lastLineWidthRatio?: number
+  // first display-size word (24px and up, space-delimited scripts) the browser
+  // split across two lines mid-letter. words with a legal break point
+  // (hyphen, dash, slash, soft hyphen, zero-width space) and domains or emails
+  // are never reported.
+  brokenWord?: string
   ellipsisActive?: boolean
 }
 

package/cli/commands/inspect.ts

@@ -192,6 +192,24 @@ async function probeSimResponsive(bridge: WsBridge): Promise<boolean> {
   }
 }
 
+// a bare `do tap <x>` resolves a testID; users coming from maestro's
+// `tapOn: <text>` pass visible text there, so point them at `--text`.
+async function reportMissingTarget(
+  bridge: WsBridge,
+  verb: 'tap' | 'double-tap' | 'long-press',
+  target: { mode: 'testid' | 'text'; value: string },
+): Promise<void> {
+  console.error(`  not found: ${target.value}`)
+  if (target.mode !== 'testid') return
+  const byText = await inspectFind(bridge, { text: target.value }).catch(() => null)
+  if (byText?.result) {
+    console.error(`  that is visible text: peach do ${verb} --text "${target.value}"`)
+    return
+  }
+  await printSimilarTestIds(bridge, target.value)
+  maybeHint('wait-selector-for-missing-testid', target.value)
+}
+
 // surface the 5 closest candidates so callers can fix the typo without
 // dumping the full tree. uses a cheap prefix + Levenshtein score so a
 // trailing digit or near-miss casing still bubbles to the top.
@@ -1036,7 +1054,12 @@ export async function runInspect(args: string[], opts: InspectOptions) {
     args[0] === 'get' || args[0] === 'do' || args[0] === 'debug' || args[0] === 'wait'
       ? args[0]
       : null
-  const effectiveArgs = verbPrefix ? args.slice(1) : args
+  const groupedArgs = verbPrefix ? args.slice(1) : args
+  const tapTarget = groupedArgs[0] === 'tap' ? readTargetFlag(groupedArgs) : null
+  const effectiveArgs =
+    tapTarget?.mode === 'text'
+      ? ['tap-text', tapTarget.value, ...groupedArgs.slice(1)]
+      : groupedArgs
 
   const parsed = parseBridgeCliArgs(effectiveArgs, {
     port: opts.port,
@@ -1999,10 +2022,7 @@ export async function runInspect(args: string[], opts: InspectOptions) {
           })
           const payload = outcome.payload
           if (!payload || typeof payload.cx !== 'number') {
-            console.error(`  not found: ${target.value}`)
-            if (target.mode === 'testid') {
-              maybeHint('wait-selector-for-missing-testid', target.value)
-            }
+            await reportMissingTarget(bridge, 'tap', target)
             rnxExit(1)
           }
           if (!isTapSuccess(outcome.result)) {
@@ -2773,10 +2793,7 @@ export async function runInspect(args: string[], opts: InspectOptions) {
         if (target) {
           const resolved = await resolveTargetCoords(bridge, target)
           if (!resolved) {
-            console.error(`  not found: ${target.value}`)
-            if (target.mode === 'testid') {
-              maybeHint('wait-selector-for-missing-testid', target.value)
-            }
+            await reportMissingTarget(bridge, 'double-tap', target)
             rnxExit(1)
           }
           x = resolved.x
@@ -2816,10 +2833,7 @@ export async function runInspect(args: string[], opts: InspectOptions) {
         if (target) {
           const resolved = await resolveTargetCoords(bridge, target)
           if (!resolved) {
-            console.error(`  not found: ${target.value}`)
-            if (target.mode === 'testid') {
-              maybeHint('wait-selector-for-missing-testid', target.value)
-            }
+            await reportMissingTarget(bridge, 'long-press', target)
             rnxExit(1)
           }
           x = resolved.x

package/cli/commands/platform.ts

@@ -12,7 +12,7 @@ import {
   listSelectableDeviceModels,
   type DeviceModel,
   type RuntimePlatform,
-} from 'peach-engine/settings'
+} from 'peach-engine/settings/devices'
 import { settingsStore } from 'peach-engine/settings/store'
 import { rnxPublicBrand } from '../../src/public-brand'
 import { UnsupportedRemoteHostError } from '../cloud-client'

package/cli/commands/screenshot-layers.ts

@@ -9,7 +9,7 @@
 
 import { mkdirSync, writeFileSync } from 'fs'
 import { resolve } from 'path'
-import { devices } from 'peach-engine/settings'
+import { devices } from 'peach-engine/settings/devices'
 import { buildResolveRectEval, type SampleRect } from '../browser-evals'
 import { rnxExit } from '../run-rnx'
 import { callInBridge, createBridgeFromParsed, parseBridgeCliArgs } from '../ws-bridge'

package/cli/commands/screenshot.ts

@@ -8,7 +8,7 @@
 
 import { mkdirSync, writeFileSync } from 'fs'
 import { dirname, resolve } from 'path'
-import { devices, type DeviceModel } from 'peach-engine/settings'
+import { devices, type DeviceModel } from 'peach-engine/settings/devices'
 import { launchReapedChrome } from '../../../../scripts/lib/reap-browser'
 import { DEFAULT_SOOTSIM_BRIDGE_PORT } from '../../src/bridge-constants'
 import { composeFramedScreenshot } from '../../src/screenshots/frame-compose'