Skip to content

Comments

fix: modify the format of listApps to follow xcuitest's format#1059

Merged
KazuCocoa merged 10 commits intomasterfrom
fix-format
Feb 20, 2026
Merged

fix: modify the format of listApps to follow xcuitest's format#1059
KazuCocoa merged 10 commits intomasterfrom
fix-format

Conversation

@KazuCocoa
Copy link
Member

@KazuCocoa KazuCocoa commented Feb 19, 2026 •
edited by mykola-mokhnach
Loading

BREAKING CHANGE: modify the mobile:listApps style to follow XCUITest driver format

I was wrong for the format. listApps in xcuitest is like below, so let me modify the return format to be similar. We can add more info into them later:

{"com.clearme.Clear"=>
  {"CFBundlePackageType"=>"APPL",
   "UILaunchStoryboardName"=>"LaunchScreen",
   "NSBluetoothAlwaysUsageDescription"=>
    "Our app uses bluetooth to share your CLEAR ID with the TSA",
   "GroupContainers"=>
    {"group.com.clearme.Clear"=>
      "/private/var/mobile/Containers/Shared/AppGroup/1E3BC78E-3B2E-4401-9C3B-E81A3C21178E"},
   "DTPlatformVersion"=>"18.5",
   "DTSDKBuild"=>"22F76",
   "UIStatusBarStyle"=>"UIStatusBarStyleDefault",
   "NFCReaderUsageDescription"=>"We use NFC scan to read ePassport data",
   "IsHostBackupEligible"=>true,
   "LSRequiresIPhoneOS"=>true,
   "NSCameraUsageDescription"=>
    "We use the camera to help validate your identity during enrollment.",
   "Appboy"=>{"Endpoint"=>"sdk.iad-03.braze.com"},
   "UISupportedDevices"=>["iPhone12,8", "iPhone14,6"],
   "CFBundleDisplayName"=>"CLEAR",
   "com.apple.developer.nfc.readersession.iso7816.select-identifiers"=>
    ["A0000002471001", "A00000045645444C2D3031"],
   "SignerIdentity"=>"Apple iPhone OS Application Signing",
   "ITSDRMScheme"=>"v2",
   "DTXcodeBuild"=>"16F6",
   "Path"=>
    "/private/var/containers/Bundle/Application/E56ABD3A-664A-4A55-AD7F-1970A6BE758F/Clear.app",
   "EnvironmentVariables"=>
    {"CFFIXED_USER_HOME"=>
      "/private/var/mobile/Containers/Data/Application/D4331DA2-95F4-4609-99B3-44114FA7486D",
     "TMPDIR"=>
      "/private/var/mobile/Containers/Data/Application/D4331DA2-95F4-4609-99B3-44114FA7486D/tmp"
,
     "HOME"=>
      "/private/var/mobile/Containers/Data/Application/D4331DA2-95F4-4609-99B3-44114FA7486D"},
   "CFBundleNumericVersion"=>0,
   "NSPrivacyTrackingDomains"=>["api-safetrack-eu.branch.io", "api-safetrack.branch.io"],
   "IsDemotedApp"=>false,
   "SequenceNumber"=>1428,
   "PHPhotoLibraryPreventAutomaticLimitedAccessAlert"=>true,

KazuCocoa
KazuCocoa commented Feb 19, 2026
View reviewed changes
lib/commands/app-management.ts Outdated
return (await this.adb.listInstalledPackages(opts)).map((pkg) => pkg.appPackage);
return (await this.adb.listInstalledPackages(opts)).map((pkg) => {
const packageName = pkg.appPackage;
const versionCode = pkg.versionCode ? pkg.versionCode : '';
Copy link
Member Author

@KazuCocoa KazuCocoa Feb 19, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

Maybe will take a look later if we have another way to get this info as well via another method and combine the results (as another PR)

Copy link
Contributor

@mykola-mokhnach mykola-mokhnach Feb 20, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

how about

const versionCode = pkg.versionCode && !Number.isNaN(+pkg.versionCode) ? +pkg.versionCode : null

Copy link
Member Author

@KazuCocoa KazuCocoa Feb 20, 2026

Choose a reason for hiding this comment

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

Sure. I didn't observe weird values as the versionCode output in my tests, so this would work well.

mykola-mokhnach
mykola-mokhnach reviewed Feb 19, 2026
View reviewed changes
mykola-mokhnach
mykola-mokhnach reviewed Feb 19, 2026
View reviewed changes
mykola-mokhnach
mykola-mokhnach reviewed Feb 19, 2026
View reviewed changes
mykola-mokhnach
mykola-mokhnach reviewed Feb 19, 2026
View reviewed changes
@mykola-mokhnach
Copy link
Contributor

mykola-mokhnach commented Feb 19, 2026

This is a breaking change. We must bump a major version of this module in order to avoid further issues in drivers that use this module if mobile: listApps extension has been already integrated and published at least once

@KazuCocoa KazuCocoa changed the title fix: modify the format of listApps fix: modify the format of listApps to follow xcuitest's format Feb 20, 2026
@KazuCocoa
Copy link
Member Author

KazuCocoa commented Feb 20, 2026

Updated. I'll add BREAKING: modify the mobile:listApps style to follow XCUITest driver format in the description as the commit message when i merge this

@KazuCocoa KazuCocoa mentioned this pull request Feb 20, 2026
Merged
@KazuCocoa KazuCocoa merged commit 0a683f9 into master Feb 20, 2026
10 checks passed
@KazuCocoa KazuCocoa deleted the fix-format branch February 20, 2026 08:32
github-actions bot pushed a commit that referenced this pull request Feb 20, 2026
@semantic-release-bot
chore(release): 13.0.0 [skip ci]
e193e65 
## [13.0.0](v12.7.0...v13.0.0) (2026-02-20)

### ⚠ BREAKING CHANGES

* modify the mobile:listApps style to follow XCUITest driver format

### Bug Fixes

* modify the format of listApps to follow xcuitest's format ([#1059](#1059)) ([0a683f9](0a683f9))
@github-actions
Copy link

github-actions bot commented Feb 20, 2026

🎉 This PR is included in version 13.0.0 🎉

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

Reviewers

@mykola-mokhnach mykola-mokhnach mykola-mokhnach approved these changes

Assignees

No one assigned

Labels

released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@KazuCocoa @mykola-mokhnach