Skip to content

feat: Drop legacy APIs#1137

Open
mykola-mokhnach wants to merge 3 commits into
masterfrom
no-uninst
Open

feat: Drop legacy APIs#1137
mykola-mokhnach wants to merge 3 commits into
masterfrom
no-uninst

Conversation

@mykola-mokhnach
Copy link
Copy Markdown

Summary

Removes device uninstall logic, deprecated session teardown helpers, dead code paths, and the appium-ios-device dependency from the TypeScript driver layer. WDA lifecycle is now limited to quit (stop processes) and cache-or-skip (reuse an existing instance when compatible).

Breaking changes

  • BREAKING CHANGE: quitAndUninstall() removed — use quit() only. App uninstall is out of scope for this module.
  • BREAKING CHANGE: uninstall() removed — WDA must not be uninstalled from this package; callers (e.g. xcuitest-driver) should own that if needed.
  • BREAKING CHANGE: setupCaching() no longer uninstalls WDA — on bundle-id or version mismatch it logs and skips caching instead of removing apps from the device. Also, it now returns the cached url on success.
  • BREAKING CHANGE: appium-ios-device dependency removed — preinstalled WDA on real devices always launches via devicectl (no iOS < 17 Xctest fallback).
  • BREAKING CHANGE: Unused private helpers removed — parseBundleId, fetchWDABundle, isRunning.

Other changes

  • Export WDA_RUNNER_APP from the package entry point (lib/index.ts).
  • Delete lib/appium-ios-device.d.ts (no longer needed).
  • Simplify quit() for usePreinstalledWDA: always uses simctl.terminateApp.
  • Unit tests updated to assert cache vs. no-cache behavior instead of uninstall calls.

Migration

Before After
await wda.quitAndUninstall() await wda.quit()
await wda.uninstall() (if used via cast) Handle uninstall in the host driver
Expect setupCaching() to remove mismatched WDA Expect it to skip caching; uninstall elsewhere if required
Preinstalled WDA on real device iOS < 17 via USB Xctest Requires device.devicectl (iOS 17+ style launch path)

@mykola-mokhnach mykola-mokhnach requested a review from KazuCocoa May 16, 2026 15:26
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.

2 participants