scripts/components/CollectableComponent.cs

@@ -13,10 +13,16 @@ public partial class CollectableComponent : Node
     [Export] public Area2D Area2D { get; set; }
     [Export] public CollisionShape2D CollisionShape { get; set; }
     [Export] public CollectableResource Data { get; set; }
-    [Export] public AudioStreamPlayer2D Sfx {get; set; }
+    [Export] public AudioStreamPlayer2D Sfx { get; set; }
 
     [Signal] public delegate void CollectedEventHandler(float amount, CollectableType type, Node2D body);
 
+    /// <summary>
+    /// Delegate for checking if collection should be allowed.
+    /// Return false to prevent collection.
+    /// </summary>
+    public Func<Node2D, bool> CanCollect { get; set; }
+
     private FloatingTextManager _floatingTextManager;
 
     public override void _Ready()
@@ -38,6 +44,9 @@ public partial class CollectableComponent : Node
         {
             if (!body.HasNode("CanPickUpComponent")) return;
 
+            // Allow components to veto collection (e.g., full health for potions)
+            if (CanCollect != null && !CanCollect(body)) return;
+
             if (Owner is Node2D ownerNode)
             {
                 switch (Data.Type)

scripts/components/HealComponent.cs

@@ -17,9 +17,20 @@ public partial class HealComponent : Node
             return;
         }
 
+        // Register check to prevent collecting when at full health
+        Collectable.CanCollect = CanCollectHealth;
         Collectable.Collected += OnCollected;
     }
 
+    private bool CanCollectHealth(Node2D body)
+    {
+        var healthComponent = body.GetNodeOrNull<HealthComponent>("HealthComponent");
+        if (healthComponent == null) return true; // Allow collection if no health component
+
+        // Prevent collection if already at full health
+        return healthComponent.Health < healthComponent.MaxHealth;
+    }
+
     private void OnCollected(float amount, CollectableType type, Node2D body)
     {
         if (type != CollectableType.Health) return;