Merge pull request #417 from osse101/fix-itemhandler-18204570734183862940

Fix lootbox grouping for money and update bomb usage message

Author: osse101

internal/itemhandler/bomb.go

@@ -48,7 +48,7 @@ func handleBomb(ctx context.Context, ec EffectContext, _ *domain.User, inventory
 	}
 
 	log.Info("Bombs set successfully", "setter", username, "platform", platform, "count", totalQueued)
-	return fmt.Sprintf("%s set %s! Waiting for a crowd...", username, lastDisplayName), nil
+	return fmt.Sprintf("%s set %s! It will detonate when the crowd gathers...", username, lastDisplayName), nil
 }
 
 // BombHandler handles bomb items.

internal/itemhandler/lootbox.go

@@ -128,7 +128,12 @@ func aggregateDropsAndUpdateInventory(ec EffectContext, inventory *domain.Invent
 		})
 
 		// Get display name
-		itemDisplayName := ec.GetDisplayName(drop.ItemName, drop.QualityLevel)
+		var itemDisplayName string
+		if drop.ItemName == domain.ItemMoney {
+			itemDisplayName = ec.GetDisplayName(domain.ItemMoney, "")
+		} else {
+			itemDisplayName = ec.GetDisplayName(drop.ItemName, drop.QualityLevel)
+		}
 
 		if group, exists := displayGroups[itemDisplayName]; exists {
 			group.Quantity += drop.Quantity