Improve list devices (#216)

***NO_CI***

Author: josesimoes

README.md

@@ -343,6 +343,14 @@ To get the details of a nano device connected to a serial port.
 nanoff --nanodevice --devicedetails --serialport COM9
 ```
 
+### List connected nano devices
+
+To get a list of connected nano devices. If more details are required pass verbose option above normal.
+
+```console
+nanoff --listdevices | -v d
+```
+
 ## Common options
 
 ### Pre-check if target fits connected device

nanoFirmwareFlasher.Library/NanoDeviceOperations.cs

@@ -35,14 +35,56 @@ public NanoDeviceOperations()
         /// <summary>
         /// List connected .NET nanoFramework devices.
         /// </summary>
+        /// <param name="getDeviceDetails">Set to <see langword="true"/> to get details from devices.</param>
         /// <returns>An observable collection of <see cref="NanoDeviceBase"/>. devices</returns>
-        public ObservableCollection<NanoDeviceBase> ListDevices()
+        public ObservableCollection<NanoDeviceBase> ListDevices(bool getDeviceDetails)
         {
+            // start device watchers
+            _serialDebuggerPort.StartDeviceWatchers();
+
             while (!_serialDebuggerPort.IsDevicesEnumerationComplete)
             {
                 Thread.Sleep(100);
             }
 
+            if (getDeviceDetails)
+            {
+                // get device details
+                foreach (var device in _serialDebuggerPort.NanoFrameworkDevices)
+                {
+                    _ = device.DebugEngine.Connect(
+                        false,
+                        true);
+
+                    // check that we are in CLR
+                    if (device.DebugEngine.IsConnectedTonanoCLR)
+                    {
+                        try
+                        {
+                            // get device info
+                            _ = device.GetDeviceInfo(true);
+                        }
+                        catch
+                        {
+                            // no need to report this, just move on
+                        }
+                    }
+                    else
+                    {
+                        // we are in booter, can only get TargetInfo
+                        try
+                        {
+                            // get device info
+                            _ = device.DebugEngine?.TargetInfo;
+                        }
+                        catch
+                        {
+                            // no need to report this, just move on
+                        }
+                    }
+                }
+            }
+
             return _serialDebuggerPort.NanoFrameworkDevices;
         }
 

nanoFirmwareFlasher.Tool/Program.cs

@@ -313,7 +313,7 @@ static async Task RunOptionsAndReturnExitCodeAsync(Options o)
 
                 try
                 {
-                    var connectedDevices = _nanoDeviceOperations.ListDevices();
+                    var connectedDevices = _nanoDeviceOperations.ListDevices(_verbosityLevel > VerbosityLevel.Normal);
 
                     if (connectedDevices.Count() == 0)
                     {
@@ -327,6 +327,38 @@ static async Task RunOptionsAndReturnExitCodeAsync(Options o)
                         foreach (var nanoDevice in connectedDevices)
                         {
                             Console.WriteLine($"{nanoDevice.Description}");
+
+                            if (_verbosityLevel >= VerbosityLevel.Normal)
+                            {
+                                // check that we are in CLR
+                                if (nanoDevice.DebugEngine.IsConnectedTonanoCLR)
+                                {
+                                    // we have to have a valid device info
+                                    if (nanoDevice.DeviceInfo.Valid)
+                                    {
+                                        Console.WriteLine($"  Target:      {nanoDevice.DeviceInfo.TargetName?.ToString()}");
+                                        Console.WriteLine($"  Platform:    {nanoDevice.DeviceInfo.Platform?.ToString()}");
+                                        Console.WriteLine($"  Date:        {nanoDevice.DebugEngine.Capabilities.SoftwareVersion.BuildDate ?? "unknown"}");
+                                        Console.WriteLine($"  Type:        {nanoDevice.DebugEngine.Capabilities.SolutionReleaseInfo.VendorInfo ?? "unknown"}");
+                                        Console.WriteLine($"  CLR Version: {nanoDevice.DeviceInfo.SolutionBuildVersion}");
+                                    }
+                                }
+                                else
+                                {
+                                    // we are in booter, can only get TargetInfo
+                                    // we have to have a valid device info
+                                    if (nanoDevice.DebugEngine.TargetInfo != null)
+                                    {
+                                        Console.WriteLine($"  Target:         {nanoDevice.DebugEngine.TargetInfo.TargetName}");
+                                        Console.WriteLine($"  Platform:       {nanoDevice.DebugEngine.TargetInfo.PlatformName}");
+                                        Console.WriteLine($"  Type:           {nanoDevice.DebugEngine.TargetInfo.PlatformInfo}");
+                                        Console.WriteLine($"  CLR Version:    {nanoDevice.DebugEngine.TargetInfo.CLRVersion}");
+                                        Console.WriteLine($"  Booter Version: {nanoDevice.DebugEngine.TargetInfo.CLRVersion}");
+                                    }
+                                }
+
+                                Console.WriteLine("");
+                            }
                         }
 
                         Console.WriteLine("------------------------------------------");