mirror of
https://github.com/priyanshujain/sanderling.git
synced 2026-10-02 11:07:10 +00:00
* feat(companion): add appState, eraseText, pressKey runner handlers
The Go runner transport already calls these methods; the in-device runner
implemented them only latently. They become load-bearing on the device
path, where the hybrid's legacy-companion fallback is absent. Backward
compatible: the simulator hybrid never calls them.
* feat(ios): resolve physical devices from devicectl
ResolveDevice parses xcrun devicectl list devices into Device{Name,
HardwareUDID, CoreDeviceID}: the hardware UDID feeds xcodebuild/iproxy
and the CoreDevice id feeds devicectl install. Matches by name or either
id; errors list candidates on none/ambiguous. Fixes the stale sidecar
comment on ResolveTarget.
* feat(ioscompanion): runner-only device driver mode
NewDevice reuses Driver with d.companion set to the runner dialed over an
iproxy usbmux tunnel, hybrid=false, runnerClient=nil. The existing accessor
seams then route launch/snapshot/text/gesture to the runner with no new
DeviceDriver methods. Device seams swap clear-state to a devicectl
reinstall, container reset to a warn-once no-op, and paste grant to a no-op.
realSpawnDeviceRunner builds and signs the runner at run time via the App
Store Connect API key (no Xcode UI), caching on a source hash.
* test(ioscompanion): cover device wiring, routing, and shell-out argv
Seam-driven NewDevice wiring + gesture/text routing (asserting no keyboard
HID), devicectl/build/test/iproxy argv builders, xctestrun test-target dict
name parsing, signing-credential env checks, and source-hash cache keying.
* feat(testrun): route physical-device iOS runs to the device driver
Execute resolves a non-simulator iOS target through ios.ResolveDevice into
its hardware UDID and CoreDevice id; buildDriver constructs NewDevice via a
seam instead of rejecting the device. Generalizes the --ios-device and
--ios-app-path help to cover the device path; signing stays env-read, never
a flag.
* feat(doctor): device prereqs replace java/sidecar for ios-device
iosDeviceChecks now verifies devicectl, iproxy on PATH, a connected+paired
device (via ios.ConnectedDevices), and App Store Connect signing creds (via
ioscompanion.VerifyDeviceSigning). The retired JVM sidecar checks stay only
under android.
* feat(conformance): device backend uses iphoneos app and tunnel orphan checks
The device backend now builds via just ios-device, points --ios-app-path at
the Debug-iphoneos bundle, and reinstalls each run for clear-state. The G5
orphan scan replaces the retired sidecar.jar check with lingering iproxy and
device test-without-building sessions (destination platform=iOS,id=).
* feat(folio): device build linking the iosArm64 framework
project.yml selects the Kotlin framework slice by SDK (iosArm64 for
iphoneos, iosSimulatorArm64 for simulator) and links via -framework Shared
on the SDK-conditional search path. New ios-device/test-ios-device recipes
mirror ios/test-ios, signing the Debug-iphoneos build with the .env API key.
* docs(cli): document ios-device doctor checks and the device flags
The --ios-device flag now also selects a connected device; --ios-app-path
covers the device install; the doctor gains an ios-device platform whose
checks are devicectl, iproxy, a paired device, and signing credentials.
Corrects the --clear-data default to true.
* fix(ioscompanion): resolve signing key path to absolute
xcodebuild's -authenticationKeyPath requires an absolute path, but .env
files commonly carry a repo-relative one. Resolve it against the working
directory before the stat so a relative ASC_API_KEY_PATH still signs.
* fix(ioscompanion): re-enable signing for the device runner build
companion/project.yml disables code signing for the simulator build, so
the device build inherited it and produced an unsigned runner that the
device rejected at install (0xe8008018). build-for-testing now forces
CODE_SIGNING_ALLOWED/REQUIRED=YES so automatic provisioning signs it.
* fix(ioscompanion): key the device build cache on signing identity
The cache marker hashed only sources, so switching signing team or key
reused a runner signed with the stale identity, which the device rejects at
install (0xe8008018). Fold team + key id into the cache key so a signing
change forces a rebuild.
* docs(getting-started): document physical iOS device setup
Lists the iproxy requirement and the App Store Connect signing env vars
(SANDERLING_IOS_TEAM, ASC_API_*) a device run needs, plus the
test-ios-device recipe and the doctor check.
* feat(ios): native usbmux client and in-process tunnel forwarder
Talk to macOS usbmuxd directly instead of shelling out to iproxy, so the
device path depends on nothing beyond macOS + Xcode.
* refactor(ios): drive device tunnel via io.Closer seam
Replace the tunnelChild *exec.Cmd and spawnTunnel seam with a tunnel
io.Closer and startTunnel seam backed by the in-process usbmux forwarder.
* refactor(ios): remove iproxy spawn from device runner
* test(ios): cover tunnel close via io.Closer not child process
* feat(doctor): check usbmuxd socket instead of iproxy on PATH
* chore(conformance): drop iproxy orphan check; tunnel is in-process
* docs(ios): device tunnel uses native usbmux, nothing to install
* chore: gitignore the signing keys directory
* feat(folio): add Android launcher icon (black bg, white dot)
* feat(folio): add iOS app icon (black bg, white dot)
* feat(folio): add web favicon (black bg, white dot)
* docs(ioscompanion): fix stale const comments
* refactor(ioscompanion): inline single-use devicectl argv builders
* refactor(ioscompanion): inline xcodegenArgs, drop tautological argv tests
* refactor(ioscompanion): inline firstNonEmpty
* refactor(doctor): dedup usbmuxd socket path via ioscompanion seam
* test(doctor): trim redundant signing-check test
* refactor(ioscompanion): deliver COMPANION_PORT via TEST_RUNNER_ env
* fix(testrun): seam preflight so iOS routing tests pass on CI without xcrun
130 lines
4.0 KiB
Go
130 lines
4.0 KiB
Go
package ios
|
|
|
|
import (
|
|
"context"
|
|
"encoding/json"
|
|
"fmt"
|
|
"os"
|
|
"os/exec"
|
|
"strings"
|
|
)
|
|
|
|
// Device is a physical iOS device resolved from devicectl. A device carries two
|
|
// identifiers: HardwareUDID feeds xcodebuild -destination and the usbmux device
|
|
// match, while CoreDeviceID feeds devicectl install/uninstall.
|
|
type Device struct {
|
|
Name string
|
|
HardwareUDID string
|
|
CoreDeviceID string
|
|
}
|
|
|
|
// coreDevice mirrors the fields of one entry in devicectl's JSON device list.
|
|
type coreDevice struct {
|
|
Identifier string `json:"identifier"`
|
|
HardwareProperties struct {
|
|
UDID string `json:"udid"`
|
|
} `json:"hardwareProperties"`
|
|
DeviceProperties struct {
|
|
Name string `json:"name"`
|
|
} `json:"deviceProperties"`
|
|
}
|
|
|
|
type coreDeviceList struct {
|
|
Result struct {
|
|
Devices []coreDevice `json:"devices"`
|
|
} `json:"result"`
|
|
}
|
|
|
|
// listDevices is a seam: overridable in tests so ResolveDevice runs against
|
|
// canned devicectl output without invoking xcrun.
|
|
var listDevices = coreDevices
|
|
|
|
// ConnectedDevices lists the physical iOS devices devicectl reports. The doctor
|
|
// uses it to confirm at least one device is connected and paired.
|
|
func ConnectedDevices(ctx context.Context) ([]Device, error) {
|
|
return listDevices(ctx)
|
|
}
|
|
|
|
// ResolveDevice picks the physical iOS device a run drives. An empty query
|
|
// resolves the single connected device (an error names them all when several
|
|
// are connected). A non-empty query matches a device by name, hardware UDID, or
|
|
// CoreDevice id. No match and an ambiguous match are both errors that list the
|
|
// connected devices so the caller can refine --ios-device.
|
|
func ResolveDevice(ctx context.Context, query string) (Device, error) {
|
|
devices, err := listDevices(ctx)
|
|
if err != nil {
|
|
return Device{}, fmt.Errorf("list devices: %w", err)
|
|
}
|
|
if len(devices) == 0 {
|
|
return Device{}, fmt.Errorf("no connected iOS device found; connect and pair an iPhone, then check `xcrun devicectl list devices`")
|
|
}
|
|
|
|
if query == "" {
|
|
if len(devices) == 1 {
|
|
return devices[0], nil
|
|
}
|
|
return Device{}, fmt.Errorf("multiple connected iOS devices; pass --ios-device to select one:%s", deviceLines(devices))
|
|
}
|
|
|
|
var matches []Device
|
|
for _, device := range devices {
|
|
if device.Name == query || device.HardwareUDID == query || device.CoreDeviceID == query {
|
|
matches = append(matches, device)
|
|
}
|
|
}
|
|
switch len(matches) {
|
|
case 1:
|
|
return matches[0], nil
|
|
case 0:
|
|
return Device{}, fmt.Errorf("no connected iOS device matches %q; connected devices:%s", query, deviceLines(devices))
|
|
default:
|
|
return Device{}, fmt.Errorf("--ios-device %q matches multiple devices; select by hardware UDID or CoreDevice id:%s", query, deviceLines(matches))
|
|
}
|
|
}
|
|
|
|
func deviceLines(devices []Device) string {
|
|
var lines strings.Builder
|
|
for _, device := range devices {
|
|
fmt.Fprintf(&lines, "\n %s (udid %s, id %s)", device.Name, device.HardwareUDID, device.CoreDeviceID)
|
|
}
|
|
return lines.String()
|
|
}
|
|
|
|
// coreDevices invokes devicectl and parses its JSON device list. devicectl
|
|
// writes the JSON to a file path rather than stdout, so a temp file backs the
|
|
// --json-output flag and is read back after the command runs.
|
|
func coreDevices(ctx context.Context) ([]Device, error) {
|
|
outputFile, err := os.CreateTemp("", "sanderling-devices-*.json")
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
outputPath := outputFile.Name()
|
|
outputFile.Close()
|
|
defer os.Remove(outputPath)
|
|
|
|
if err := exec.CommandContext(ctx, "xcrun", "devicectl", "list", "devices", "--json-output", outputPath).Run(); err != nil {
|
|
return nil, fmt.Errorf("xcrun devicectl list devices: %w", err)
|
|
}
|
|
data, err := os.ReadFile(outputPath)
|
|
if err != nil {
|
|
return nil, err
|
|
}
|
|
return parseDevices(data)
|
|
}
|
|
|
|
func parseDevices(data []byte) ([]Device, error) {
|
|
var list coreDeviceList
|
|
if err := json.Unmarshal(data, &list); err != nil {
|
|
return nil, err
|
|
}
|
|
var devices []Device
|
|
for _, entry := range list.Result.Devices {
|
|
devices = append(devices, Device{
|
|
Name: entry.DeviceProperties.Name,
|
|
HardwareUDID: entry.HardwareProperties.UDID,
|
|
CoreDeviceID: entry.Identifier,
|
|
})
|
|
}
|
|
return devices, nil
|
|
}
|