Skip to content

Add xcrs crate - #41

Open
setoelkahfi wants to merge 10 commits into
developmentfrom
feature/xcrs
Open

Add xcrs crate#41
setoelkahfi wants to merge 10 commits into
developmentfrom
feature/xcrs

Conversation

@setoelkahfi

Copy link
Copy Markdown
Collaborator

Related Issue

N/A

Description

Type of Change

  • Bug fix
  • New feature
  • Refactor (no functional change)
  • Documentation update
  • CI / tooling change

How Has This Been Tested?

  • cargo test --all-features
  • Manual testing (smb <command>)

Checklist

  • cargo check --workspace passes
  • cargo clippy --workspace --tests -- -D warnings passes
  • cargo fmt --all -- --check passes
  • No new unwrap() or expect() calls in production code
  • No new mod.rs files introduced
  • New dependencies added to root Cargo.toml and inherited with { workspace = true }
  • Error messages are user-friendly and surfaced to the terminal

Release Notes

@setoelkahfi setoelkahfi self-assigned this Jul 30, 2026
@setoelkahfi
setoelkahfi requested a review from paydii July 30, 2026 20:02
smb_simulator_press_button, launch_app, terminate_app, and related
tools required simulator_name/simulator_udid resolved via simctl, so
they couldn't reach a ControlKit runner on a real iOS/tvOS device over
the network. Add an optional host field (matching the pattern already
used by smb_macos_click/smb_visionos_spatial_tap/smb_watchos_tap) and
route app launch/terminate through the RPC methods instead of simctl
when a host is given.
…port

Simctl has no equivalent for a real device, so smb_simulator_screenshot
couldn't capture one. Add a devicectl-backed Devicectl::screenshot next
to Simctl::screenshot and expose it as smb_device_screenshot/
xcrs_device_screenshot. Also document the host field's physical-device
usage across the ControlKit tool surface, which the previous commit
added but left undocumented.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants