Core search/list now return boards in a platform (#268)

Signed-off-by: Christian Weichel <[email protected]>

Author: Christian Weichel

Diff for: cli/core/list.go

@@ -69,7 +69,7 @@ func runListCommand(cmd *cobra.Command, args []string) {
 	}
 }
 
-func outputInstalledCores(cores []*rpc.InstalledPlatform) {
+func outputInstalledCores(cores []*rpc.Platform) {
 	table := output.NewTable()
 	table.AddRow("ID", "Installed", "Latest", "Name")
 	sort.Slice(cores, func(i, j int) bool {

Diff for: cli/core/search.go

@@ -71,14 +71,14 @@ func runSearchCommand(cmd *cobra.Command, args []string) {
 	}
 }
 
-func outputSearchCores(cores []*rpc.SearchOutput) {
+func outputSearchCores(cores []*rpc.Platform) {
 	table := output.NewTable()
 	table.AddRow("ID", "Version", "Name")
 	sort.Slice(cores, func(i, j int) bool {
 		return cores[i].ID < cores[j].ID
 	})
 	for _, item := range cores {
-		table.AddRow(item.GetID(), item.GetVersion(), item.GetName())
+		table.AddRow(item.GetID(), item.GetLatest(), item.GetName())
 	}
 	fmt.Print(table.Render())
 }

Diff for: commands/core/core.go

@@ -0,0 +1,54 @@
+/*
+ * This file is part of arduino-cli.
+ *
+ * Copyright 2019 ARDUINO SA (http://www.arduino.cc/)
+ *
+ * This software is released under the GNU General Public License version 3,
+ * which covers the main part of arduino-cli.
+ * The terms of this license can be found at:
+ * https://www.gnu.org/licenses/gpl-3.0.en.html
+ *
+ * You can be released from the requirements of the above licenses by purchasing
+ * a commercial license. Buying such a license is mandatory if you want to modify or
+ * otherwise use the software for commercial activities involving the Arduino
+ * software without disclosing the source code of your own applications. To purchase
+ * a commercial license, send an email to [email protected].
+ */
+
+package core
+
+import (
+	"github.com/arduino/arduino-cli/arduino/cores"
+	rpc "github.com/arduino/arduino-cli/rpc/commands"
+)
+
+// platformReleaseToRPC converts our internal structure to the RPC structure.
+// Note: this function does not touch the "Installed" field of rpc.Platform as it's not always clear that the
+// platformRelease we're currently converting is actually installed.
+func platformReleaseToRPC(platformRelease *cores.PlatformRelease) *rpc.Platform {
+	boards := make([]*rpc.Board, len(platformRelease.Boards))
+	i := 0
+	for _, b := range platformRelease.Boards {
+		boards[i] = &rpc.Board{
+			Name: b.Name(),
+			Fqbn: b.FQBN(),
+		}
+		i++
+	}
+
+	result := &rpc.Platform{
+		ID:         platformRelease.Platform.String(),
+		Name:       platformRelease.Platform.Name,
+		Maintainer: platformRelease.Platform.Package.Maintainer,
+		Website:    platformRelease.Platform.Package.WebsiteURL,
+		Email:      platformRelease.Platform.Package.Email,
+		Boards:     boards,
+	}
+
+	latest := platformRelease.Platform.GetLatestRelease()
+	if latest != nil {
+		result.Latest = latest.Version.String()
+	}
+
+	return result
+}

Diff for: commands/core/list.go

@@ -32,7 +32,7 @@ func PlatformList(ctx context.Context, req *rpc.PlatformListReq) (*rpc.PlatformL
 		return nil, errors.New("invalid instance")
 	}
 
-	installed := []*rpc.InstalledPlatform{}
+	installed := []*rpc.Platform{}
 	for _, targetPackage := range pm.GetPackages().Packages {
 		for _, platform := range targetPackage.Platforms {
 			if platformRelease := pm.GetInstalledPlatformRelease(platform); platformRelease != nil {
@@ -41,14 +41,9 @@ func PlatformList(ctx context.Context, req *rpc.PlatformListReq) (*rpc.PlatformL
 						continue
 					}
 				}
-				p := &rpc.InstalledPlatform{
-					ID:        platformRelease.String(),
-					Installed: platformRelease.Version.String(),
-					Name:      platformRelease.Platform.Name,
-				}
-				if latest := platformRelease.Platform.GetLatestRelease(); latest != nil {
-					p.Latest = latest.Version.String()
-				}
+
+				p := platformReleaseToRPC(platformRelease)
+				p.Installed = platformRelease.Version.String()
 				installed = append(installed, p)
 			}
 		}

Diff for: commands/core/search.go

@@ -65,13 +65,9 @@ func PlatformSearch(ctx context.Context, req *rpc.PlatformSearchReq) (*rpc.Platf
 		}
 	}
 
-	out := []*rpc.SearchOutput{}
-	for _, platformRelease := range res {
-		out = append(out, &rpc.SearchOutput{
-			ID:      platformRelease.Platform.String(),
-			Name:    platformRelease.Platform.Name,
-			Version: platformRelease.Version.String(),
-		})
+	out := make([]*rpc.Platform, len(res))
+	for i, platformRelease := range res {
+		out[i] = platformReleaseToRPC(platformRelease)
 	}
 	return &rpc.PlatformSearchResp{SearchOutput: out}, nil
 }