feat: Add wireless Apple TV devices to created tunnels - #2755
Merged
Conversation
Contributor
Author
|
@KazuCocoa Could you please test this update with your wireless AppleTV device? If everything works well I'll update related docs. The next step would be to try full e2e driver test with such device and a tunnel to make sure we don't have any gaps in the driver implementation. |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds Apple TV (tvOS) WiFi tunnel support to the existing tunnel-creation script so the tunnel registry can include both USB iOS devices and paired Apple TVs.
Changes:
- Extend tunnel registry creation to merge USB tunnel results with Apple TV (WiFi) tunnel entries.
- Add Apple TV tunnel setup/cleanup logic (packet stream server + resource teardown).
- Update script reference docs to mention paired Apple TVs and linking to the pairing guide.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| scripts/tunnel-creation.mjs | Adds Apple TV WiFi tunnel creation, merges results into a single registry, and expands cleanup to cover Apple TV resources. |
| docs/reference/scripts.md | Updates the tunnel-creation script description to include paired Apple TVs and pairing guide reference. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Member
|
I assume This is logs I saw for the tunnel creation command itself - maybe good? I was confused error - expected as no sudo:
So, this script itself is ok, but still I need to learn the tools' behavior. I also ran the https://github.com/appium/appium-ios-remotexpc a bit. Then, the like this log as well. Not sure what this means, but it let me think Apple TV would need to run cc @navin772 |
KazuCocoa
reviewed
Mar 14, 2026
| log.info('\n Available endpoints:'); | ||
| log.info(' - GET /remotexpc/tunnels - List all tunnels'); | ||
| log.info(' - GET /remotexpc/tunnels/:udid - Get tunnel by UDID'); | ||
| log.info(' - GET /remotexpc/tunnels/metadata - Get registry metadata'); |
Member
There was a problem hiding this comment.
Looks like this endpoint is wrong. I'll create a removal PR to the repo
Contributor
Author
|
@KazuCocoa I assume we need also to update the logic responsible for listing devices in xcuitest driver. Currently it relies on the legacy ios-device lib instead of the remotexpc one. I assume tunnels are ok, we just need now to update the driver flow |
Member
|
Yes, it will need to be updated. This apple tv behavior is kind of WiFi only connected one specific one so not necessary to remove the legacy one - which is for USB connected devices |
KazuCocoa
approved these changes
Mar 14, 2026
Contributor
Author
|
@KazuCocoa I have created appium/appium-ios-remotexpc#158 and will try to wire it up into xcuitest driver device selection as soon as it's merged. Will need some help with testing it later |
github-actions Bot
pushed a commit
that referenced
this pull request
Mar 14, 2026
## [10.30.0](v10.29.0...v10.30.0) (2026-03-14) ### Features * Add wireless Apple TV devices to created tunnels ([#2755](#2755)) ([07911a7](07911a7))
Contributor
|
🎉 This PR is included in version 10.30.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
cc @navin772