remove compatible models from brick instance list endpoint

Author: mirkoCrobu

internal/api/docs/openapi.yaml

@@ -1192,7 +1192,7 @@ components:
       properties:
         bricks:
           items:
-            $ref: '#/components/schemas/BrickInstance'
+            $ref: '#/components/schemas/BrickInstanceListItem'
           nullable: true
           type: array
       type: object
@@ -1374,6 +1374,31 @@ components:
             for backward compatibility.'
           type: object
       type: object
+    BrickInstanceListItem:
+      properties:
+        author:
+          type: string
+        category:
+          type: string
+        config_variables:
+          items:
+            $ref: '#/components/schemas/BrickConfigVariable'
+          type: array
+        id:
+          type: string
+        model:
+          type: string
+        name:
+          type: string
+        status:
+          type: string
+        variables:
+          additionalProperties:
+            type: string
+          description: 'Deprecated: use config_variables instead. This field is kept
+            for backward compatibility.'
+          type: object
+      type: object
     BrickListItem:
       properties:
         author:

internal/e2e/client/client.gen.go

@@ -70,7 +70,7 @@ func (s *Service) List() (BrickListResult, error) {
 }
 
 func (s *Service) AppBrickInstancesList(a *app.ArduinoApp) (AppBrickInstancesResult, error) {
-	res := AppBrickInstancesResult{BrickInstances: make([]BrickInstance, len(a.Descriptor.Bricks))}
+	res := AppBrickInstancesResult{BrickInstances: make([]BrickInstanceListItem, len(a.Descriptor.Bricks))}
 	for i, brickInstance := range a.Descriptor.Bricks {
 		brick, found := s.bricksIndex.FindBrickByID(brickInstance.ID)
 		if !found {
@@ -79,7 +79,7 @@ func (s *Service) AppBrickInstancesList(a *app.ArduinoApp) (AppBrickInstancesRes
 
 		variablesMap, configVariables := getBrickConfigDetails(brick, brickInstance.Variables)
 
-		res.BrickInstances[i] = BrickInstance{
+		res.BrickInstances[i] = BrickInstanceListItem{
 			ID:              brick.ID,
 			Name:            brick.Name,
 			Author:          "Arduino", // TODO: for now we only support our bricks

internal/orchestrator/bricks/bricks.go

@@ -29,7 +29,18 @@ type BrickListItem struct {
 }
 
 type AppBrickInstancesResult struct {
-	BrickInstances []BrickInstance `json:"bricks"`
+	BrickInstances []BrickInstanceListItem `json:"bricks"`
+}
+
+type BrickInstanceListItem struct {
+	ID              string                `json:"id"`
+	Name            string                `json:"name"`
+	Author          string                `json:"author"`
+	Category        string                `json:"category"`
+	Status          string                `json:"status"`
+	Variables       map[string]string     `json:"variables,omitempty" description:"Deprecated: use config_variables instead. This field is kept for backward compatibility."`
+	ConfigVariables []BrickConfigVariable `json:"config_variables,omitempty"`
+	ModelID         string                `json:"model,omitempty"`
 }
 
 type BrickInstance struct {