log warning if local items have conflicting names

cmd/crowdsec-cli/clihub/items.go

@@ -63,7 +63,7 @@ func ListItems(out io.Writer, wantColor string, itemTypes []string, items map[st
 				continue
 			}
 
-			listHubItemTable(out, wantColor, "\n"+strings.ToUpper(itemType), items[itemType])
+			listHubItemTable(out, wantColor, strings.ToUpper(itemType), items[itemType])
 
 			nothingToDisplay = false
 		}

pkg/cwhub/hub.go

@@ -171,6 +171,7 @@ func (h *Hub) Update(ctx context.Context, indexProvider IndexProvider, withConte
 	return nil
 }
 
+// addItem adds an item to the hub. It silently replaces an existing item with the same type and name.
 func (h *Hub) addItem(item *Item) {
 	if h.items[item.Type] == nil {
 		h.items[item.Type] = make(map[string]*Item)

pkg/cwhub/state.go

@@ -11,6 +11,7 @@ type ItemState struct {
 	LocalVersion         string   `json:"local_version,omitempty" yaml:"local_version,omitempty"`
 	LocalHash            string   `json:"local_hash,omitempty" yaml:"local_hash,omitempty"`
 	Installed            bool     `json:"installed"`
+	local                bool
 	Downloaded           bool     `json:"downloaded"`
 	UpToDate             bool     `json:"up_to_date"`
 	Tainted              bool     `json:"tainted"`
@@ -20,7 +21,7 @@ type ItemState struct {
 
 // IsLocal returns true if the item has been create by a user (not downloaded from the hub).
 func (s *ItemState) IsLocal() bool {
-	return s.Installed && !s.Downloaded
+	return s.local
 }
 
 // Text returns the status of the item as a string (eg. "enabled,update-available").

pkg/cwhub/state_test.go

@@ -40,6 +40,7 @@ func TestItemStateText(t *testing.T) {
 			ItemState{
 				Installed:  true,
 				UpToDate:   false,
+				local:      true,
 				Tainted:    false,
 				Downloaded: false,
 			},