@okwasniewski I believe it is ready

Hey @itsramiel,

Thanks for this PR!

I'm going to review and test it over the weekend 🙌

I've compiled the app and for some reason I can see my macbook there:

Here is the output of xctrace list devices

❯ xctrace list devices
== Devices ==
MacBook Pro (Oskar) (123)
iPhone (Oskar) (17.6.1) (123)

== Devices Offline ==
Apple Vision Pro (2.0) (123)

It would be nice to support the naming convention of iPhone (Name) as this is the default naming in my country and probably others too.

Also I found this: xcrun devicectl list devices which shows the output for listing only devices. Maybe faster (as it doesn't list simulators)

Looks like Android Devices are working smoothly!

I've compiled the app and for some reason I can see my macbook there:

Here is the output of xctrace list devices

❯ xctrace list devices
== Devices ==
MacBook Pro (Oskar) (123)
iPhone (Oskar) (17.6.1) (123)

== Devices Offline ==
Apple Vision Pro (2.0) (123)

It would be nice to support the naming convention of iPhone (Name) as this is the default naming in my country and probably others too.

Would be great if we can setup a unit test for parsing results, so you know nothing breaks when fixing for other cases. I am not familiar with swift unit testing but that's an idea

Also I found this: xcrun devicectl list devices which shows the output for listing only devices. Maybe faster (as it doesn't list simulators)

That is a pretty good command that I didnt know about. One issue I see is that it doesnt provide the ios version 🙃.

EDIT:
So while the command doesnt provide os version directly, we can get more information by querying each device:
xcrun devicectl device info details --device <uuid>

This provies way more info though that maybe could be useful? Try it out with your device.

You can know if user is connected by usb or not, connected(tunnel state) and way more

EDIT 2:

For the future you can also interact with ios devices? Wow seems like more ideas for features

Would be great if we can setup a unit test for parsing results, so you know nothing breaks when fixing for other cases. I am not familiar with swift unit testing but that's an idea

Yeah, this is great idea. I can write tests for current implementation in separate PR and then you can add it for real devices

I also learned about devicectl today, really handy and definitely bunch of cool stuff to implement in upcoming PRs!

I also noticed that you can pass -j flag and specify json file to output the results. On the other hand not having version near the device is not such a big issue because you can't change the version anyways.

For simulators is much more useful because you can change their versions.

@itsramiel Just checked and it looks like Xcode 15 introduced devicectl in that case let's stay with your current approach. We can add features that use devicectl in upcoming PRs and check if user has Xcode 15 but let's add real device listing for everyone.

Im going to write tests for parsers today.

@itsramiel Just checked and it looks like Xcode 15 introduced devicectl in that case let's stay with your current approach. We can add features that use devicectl in upcoming PRs and check if user has Xcode 15 but let's add real device listing for everyone.

Im going to write tests for parsers today.

I just pushed the devicectl changes 😅

EDIT: do I revert it?

@itsramiel Ugh, sorry.. okay i guess let's make this feature available only for Xcode >= 15.

We will need to make sure that it doesn't break on older Xcode versions (if xcrun devicectl doesn't exist lets just don't display the iOS Devices section)

I've extracted DeviceParser to another class and wrote some tests here: #125

Thanks for working on this!