add useByApps field for brick details endpoint

Author: mirkoCrobu

cmd/arduino-app-cli/brick/bricks.go

@@ -17,16 +17,18 @@ package brick
 
 import (
 	"github.com/spf13/cobra"
+
+	"github.com/arduino/arduino-app-cli/internal/orchestrator/config"
 )
 
-func NewBrickCmd() *cobra.Command {
+func NewBrickCmd(cfg config.Configuration) *cobra.Command {
 	appCmd := &cobra.Command{
 		Use:   "brick",
 		Short: "Manage Arduino Bricks",
 	}
 
 	appCmd.AddCommand(newBricksListCmd())
-	appCmd.AddCommand(newBricksDetailsCmd())
+	appCmd.AddCommand(newBricksDetailsCmd(cfg))
 
 	return appCmd
 }

cmd/arduino-app-cli/brick/details.go

@@ -25,21 +25,23 @@ import (
 	"github.com/arduino/arduino-app-cli/cmd/arduino-app-cli/internal/servicelocator"
 	"github.com/arduino/arduino-app-cli/cmd/feedback"
 	"github.com/arduino/arduino-app-cli/internal/orchestrator/bricks"
+	"github.com/arduino/arduino-app-cli/internal/orchestrator/config"
 )
 
-func newBricksDetailsCmd() *cobra.Command {
+func newBricksDetailsCmd(cfg config.Configuration) *cobra.Command {
 	return &cobra.Command{
 		Use:   "details",
 		Short: "Details of a specific brick",
 		Args:  cobra.ExactArgs(1),
 		Run: func(cmd *cobra.Command, args []string) {
-			bricksDetailsHandler(args[0])
+			bricksDetailsHandler(args[0], cfg)
 		},
 	}
 }
 
-func bricksDetailsHandler(id string) {
-	res, err := servicelocator.GetBrickService().BricksDetails(id)
+func bricksDetailsHandler(id string, cfg config.Configuration) {
+	res, err := servicelocator.GetBrickService().BricksDetails(id, servicelocator.GetAppIDProvider(),
+		cfg)
 	if err != nil {
 		if errors.Is(err, bricks.ErrBrickNotFound) {
 			feedback.Fatal(err.Error(), feedback.ErrBadArgument)

cmd/arduino-app-cli/main.go

@@ -71,7 +71,7 @@ func run(configuration cfg.Configuration) error {
 
 	rootCmd.AddCommand(
 		app.NewAppCmd(configuration),
-		brick.NewBrickCmd(),
+		brick.NewBrickCmd(configuration),
 		completion.NewCompletionCommand(),
 		daemon.NewDaemonCmd(configuration, Version),
 		properties.NewPropertiesCmd(configuration),

internal/api/api.go

@@ -56,7 +56,7 @@ func NewHTTPRouter(
 	mux.Handle("GET /v1/version", handlers.HandlerVersion(version))
 	mux.Handle("GET /v1/config", handlers.HandleConfig(cfg))
 	mux.Handle("GET /v1/bricks", handlers.HandleBrickList(brickService))
-	mux.Handle("GET /v1/bricks/{brickID}", handlers.HandleBrickDetails(brickService))
+	mux.Handle("GET /v1/bricks/{brickID}", handlers.HandleBrickDetails(brickService, idProvider, cfg))
 
 	mux.Handle("GET /v1/properties", handlers.HandlePropertyKeys(cfg))
 	mux.Handle("GET /v1/properties/{key}", handlers.HandlePropertyGet(cfg))

internal/api/handlers/bricks.go

@@ -26,6 +26,7 @@ import (
 	"github.com/arduino/arduino-app-cli/internal/api/models"
 	"github.com/arduino/arduino-app-cli/internal/orchestrator/app"
 	"github.com/arduino/arduino-app-cli/internal/orchestrator/bricks"
+	"github.com/arduino/arduino-app-cli/internal/orchestrator/config"
 	"github.com/arduino/arduino-app-cli/internal/render"
 )
 
@@ -153,14 +154,15 @@ func HandleBrickCreate(
 	}
 }
 
-func HandleBrickDetails(brickService *bricks.Service) http.HandlerFunc {
+func HandleBrickDetails(brickService *bricks.Service, idProvider *app.IDProvider,
+	cfg config.Configuration) http.HandlerFunc {
 	return func(w http.ResponseWriter, r *http.Request) {
 		id := r.PathValue("brickID")
 		if id == "" {
 			render.EncodeResponse(w, http.StatusBadRequest, models.ErrorResponse{Details: "id must be set"})
 			return
 		}
-		res, err := brickService.BricksDetails(id)
+		res, err := brickService.BricksDetails(id, idProvider, cfg)
 		if err != nil {
 			if errors.Is(err, bricks.ErrBrickNotFound) {
 				details := fmt.Sprintf("brick with id %q not found", id)

internal/orchestrator/bricks/bricks.go

@@ -18,6 +18,7 @@ package bricks
 import (
 	"errors"
 	"fmt"
+	"log/slog"
 	"maps"
 	"slices"
 
@@ -26,6 +27,7 @@ import (
 
 	"github.com/arduino/arduino-app-cli/internal/orchestrator/app"
 	"github.com/arduino/arduino-app-cli/internal/orchestrator/bricksindex"
+	"github.com/arduino/arduino-app-cli/internal/orchestrator/config"
 	"github.com/arduino/arduino-app-cli/internal/orchestrator/modelsindex"
 	"github.com/arduino/arduino-app-cli/internal/store"
 )
@@ -125,7 +127,8 @@ func (s *Service) AppBrickInstanceDetails(a *app.ArduinoApp, brickID string) (Br
 	}, nil
 }
 
-func (s *Service) BricksDetails(id string) (BrickDetailsResult, error) {
+func (s *Service) BricksDetails(id string, idProvider *app.IDProvider,
+	cfg config.Configuration) (BrickDetailsResult, error) {
 	brick, found := s.bricksIndex.FindBrickByID(id)
 	if !found {
 		return BrickDetailsResult{}, ErrBrickNotFound
@@ -160,6 +163,20 @@ func (s *Service) BricksDetails(id string) (BrickDetailsResult, error) {
 		}
 	})
 
+	/*qui mi serve una funzione per calcolare questo. devo iterare su ogni app, di esempio o no.
+	ho già la funzione appList che mi può aiutare.
+	e per ogni elemento se ho il bircikc id corrente
+	mi creo un AppReference*/
+	appList, err := getAppList(cfg)
+	if err != nil {
+		slog.Error("unable to get app list", slog.String("error", err.Error()))
+		return BrickDetailsResult{}, fmt.Errorf("unable to get app list: %w", err)
+	}
+	usedByApps, err := getUsedByApps(appList, brick.ID, idProvider)
+	if err != nil {
+		slog.Error("unable to get used by apps", slog.String("error", err.Error()))
+		return BrickDetailsResult{}, fmt.Errorf("unable to get used by apps: %w", err)
+	}
 	return BrickDetailsResult{
 		ID:           id,
 		Name:         brick.Name,
@@ -171,9 +188,75 @@ func (s *Service) BricksDetails(id string) (BrickDetailsResult, error) {
 		Readme:       readme,
 		ApiDocsPath:  apiDocsPath,
 		CodeExamples: codeExamples,
+		UsedByApps:   usedByApps,
 	}, nil
 }
 
+func getAppList(
+	cfg config.Configuration,
+) ([]app.ArduinoApp, error) {
+	var (
+		pathsToExplore paths.PathList
+		appPaths       paths.PathList
+	)
+	pathsToExplore.Add(cfg.ExamplesDir())
+	pathsToExplore.Add(cfg.AppsDir())
+	arduinoApps := []app.ArduinoApp{}
+
+	for _, p := range pathsToExplore {
+		res, err := p.ReadDirRecursiveFiltered(func(file *paths.Path) bool {
+			if file.Base() == ".cache" {
+				return false
+			}
+			if file.Join("app.yaml").NotExist() && file.Join("app.yml").NotExist() {
+				return true
+			}
+			return false
+		}, paths.FilterDirectories(), paths.FilterOutNames("python", "sketch", ".cache"))
+
+		if err != nil {
+			slog.Error("unable to list apps", slog.String("error", err.Error()))
+			return arduinoApps, err
+		}
+		appPaths.AddAllMissing(res)
+	}
+
+	for _, file := range appPaths {
+		app, err := app.Load(file.String())
+		if err != nil {
+			/*			result.BrokenApps = append(result.BrokenApps, orchestrator.BrokenAppInfo{
+						Name:  file.Base(),
+						Error: fmt.Sprintf("unable to parse the app.yaml: %s", err.Error()),
+					})*/
+			continue
+		}
+
+		arduinoApps = append(arduinoApps, app)
+	}
+	return arduinoApps, nil
+}
+
+func getUsedByApps(apps []app.ArduinoApp, brickId string, idProvider *app.IDProvider) ([]AppReference, error) {
+	usedByApps := []AppReference{}
+	for _, app := range apps {
+		for _, b := range app.Descriptor.Bricks {
+			if b.ID == brickId {
+				id, err := idProvider.IDFromPath(app.FullPath)
+				if err != nil {
+					return usedByApps, fmt.Errorf("failed to get app ID for %s: %w", app.Name, err)
+				}
+				usedByApps = append(usedByApps, AppReference{
+					Name: app.Name,
+					ID:   id.String(),
+					Icon: app.Descriptor.Icon,
+				})
+				break
+			}
+		}
+	}
+	return usedByApps, nil
+}
+
 type BrickCreateUpdateRequest struct {
 	ID        string            `json:"-"`
 	Model     *string           `json:"model"`