Please use a different short name than “potions”; This name is already used here:
viewtopic.php?id=5549.
I think both potions are overpowered, because they heal you immediately. This can be very frustrating in combat, as it goes basically this way:
get hit
use full-heal potion
get hit
use full-heal potion
etc.
Unless you have a weapon which kills immediately I suppose it would be extremely hard, if not impossible, to kill a player with a lot of full-heal potions. You just have to use them faster than you are hit. I suppose you could even survive swimming in a lava lake if you just have enough full-heal potions. Since you are only hurt every second, but you can consume healing potions as fast as you can click.
I also think the current healing system in Minetest is just utterly flawed. It is obviously not optimized for gameplay. minetest.item_eat is overpowered. So it is not entirely your fault, after all. The sad thing is, currently all healing items I know are overpowered immediate healing items and it is just a matter of whoever runs out of healing items first. This is not true for very low healing items, like 1 half heart. This is because you may not click 19 times fast enough against a weapon which damages you by 4 hearts per hit.
By the way, do you plan to add more potions? You may be interested in my
Player Effects mod.
I also planned to create potions. I think I will release such a mod maybe in 1 or 2 weeks.