Skip to content

MOB-99: honor the documented settle delay after iOS accessibility activation #46

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
GenericJam merged 2 commits into from
Aug 26, 2026
Merged

MOB-99: honor the documented settle delay after iOS accessibility activation #46

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a63e948
MOB-99: honor the documented settle delay after iOS accessibility act…
GenericJam Aug 25, 2026
e85ede4
MOB-99 review fix: gate the accessibility settle sleep on simulator, …
GenericJam Aug 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 15 additions & 4 deletions lib/mob_dev/connector.ex
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,9 @@ defmodule MobDev.Connector do
@connect_timeout 25_000
# ms between polls
@connect_interval 500
# ms to let SwiftUI's accessibility tree rebuild after enable_accessibility's
# notifyutil broadcast — see MOB-99.
@ios_accessibility_settle_ms 500

@doc """
Discovers all connected devices, sets up tunnels, restarts apps, and waits
Expand Down Expand Up @@ -63,10 +66,18 @@ defmodule MobDev.Connector do

# Activate accessibility on iOS simulators so ui_tree() returns elements.
# SwiftUI lazily populates its a11y tree; this one-time activation persists
# for the simulator session (survives app restarts).
tunneled
|> Enum.filter(&(&1.platform == :ios))
|> Enum.each(fn d -> IOS.enable_accessibility(d.serial) end)
# for the simulator session (survives app restarts). MOB-99: give it a
# beat to propagate before any caller can start driving taps — the app
# itself is still booting (wait_for_nodes below) and SwiftUI needs a
# moment after the notifyutil broadcast to actually rebuild its tree.
#
# Simulator only, not just iOS: IOS.enable_accessibility/1 shells out to
# `xcrun simctl spawn <udid> ...`, which is simulator-only tooling — a
# no-op (or error) against a physical device's UDID. Same predicate
# kill_stale_simulator_apps/1 above already uses for the same reason.
ios_sim_targets = Enum.filter(tunneled, &(&1.platform == :ios && &1.type == :simulator))
Enum.each(ios_sim_targets, fn d -> IOS.enable_accessibility(d.serial) end)
if ios_sim_targets != [], do: Process.sleep(@ios_accessibility_settle_ms)

# Wait for nodes to come online
IO.puts("\n Waiting for nodes...")
Expand Down
Loading