Skip to content

Comments

chore: Restore the legacy method of killing a process#2731

Merged
mykola-mokhnach merged 3 commits intoappium:masterfrom
mykola-mokhnach:restore-16
Feb 17, 2026
Merged

chore: Restore the legacy method of killing a process#2731
mykola-mokhnach merged 3 commits intoappium:masterfrom
mykola-mokhnach:restore-16

Conversation

@mykola-mokhnach
Copy link
Contributor

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR restores support for terminating apps on iOS versions below 17 by reintroducing the legacy instrument service method. PR #2730 had simplified the terminateApp logic but dropped support for iOS 16 and lower by only using devicectl. This change adds conditional logic to handle iOS < 17 devices using the instrument service while keeping the devicectl path for iOS 17+.

Changes:

  • Added imports for services and INSTRUMENT_CHANNEL from appium-ios-device
  • Modified terminateAppLegacy to check platform version and branch accordingly
  • iOS >= 17: uses devicectl to list and kill processes (existing behavior)
  • iOS < 17: uses instrument service to find and kill processes (restored legacy behavior)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mykola-mokhnach mykola-mokhnach merged commit 395e8bc into appium:master Feb 17, 2026
14 of 15 checks passed
@mykola-mokhnach mykola-mokhnach deleted the restore-16 branch February 17, 2026 18:39
github-actions bot pushed a commit that referenced this pull request Feb 17, 2026
## [10.23.1](v10.23.0...v10.23.1) (2026-02-17)

### Miscellaneous Chores

* Restore the legacy method of killing a process ([#2731](#2731)) ([395e8bc](395e8bc))
@github-actions
Copy link
Contributor

🎉 This PR is included in version 10.23.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants