Why does this function not get called but after its his turn again it gets called this time - func

Question

I am making a turn based game, currently I am trying to add a critical hit system it shows that it is doing critical hit damage but doesnt play the critical hit function,the thing that doesnt make sense too me is, it gets called when its his turn again so something in the first turn is making the function not be called but gets called at the second time, It seems to be who ever attacks first doesnt get the dialogue function to be called, but plays fine after the first turn.
func check_if_player(attacker, target):
if attacker == player_node:
print(attacker.move_slot[move_index].name)
dialogue_text.text = (attacker.name + " is attacking")
dialogue_ani.play("TextScroll")
target.take_damage(attacker.move_slot[move_index],attacker)
yield(get_tree().create_timer(2), "timeout")
_update_view()
if attacker.crit:
attacker.crit = false # Where I turn it off
print(attacker.crit)
yield(critical_hit(attacker),"completed")
else:
yield(attack(attacker, target), "completed")
func attack(attacker, target):
var random_move = randi() % 3 + 1
print(attacker.move_slot[move_index].name)
dialogue_text.text = (attacker.name + " is attacking")
dialogue_ani.play("TextScroll")
target.take_damage(attacker.move_slot[random_move],attacker) # Calls the function to take damage
yield(get_tree().create_timer(2), "timeout") #Wait for 2 seconds
_update_view()
if attacker.crit:
attacker.crit = false # Where I turn it off
print(attacker.crit)
yield(critical_hit(attacker),"completed")
these two functions is how I check if the player is attacking I need to do this because I want the player to use a selected move, if isnt player then play enemy attack
and this is my monster script, where I detect if a critical hit happens, I should say this the monster script is a resource and is separate from the other script, where player, enemy and critical hit function is all in one script.
func take_damage(move,attacker):
randomize()
var critical = 1
#var critical_chance = randi() % 100 + 1
var critical_chance = 5
if critical_chance <= 99.25:
crit = true
critical = 2
var type : float = Get_effectiveness(move.type,type_1) * Get_effectiveness(move.type,type_2)
var modifier : float = rand_range(0.85,1.0) * type * critical
var a : float = (2.0 * attacker.level / 5.0 + 2.0)
var b : float = (a * attacker.attack * move.power / defence) / 50.0
var c : float = (b + 2.0) * modifier
var damage = int(c)
current_health -= damage
print(str(attacker.name) + " = " + str(damage))
#print(a)
#print(b)
Currently I have it pretty much be true every time the player or enemy attacks so it should call/ show the critical hit function, I turn the crit bool to false in the first few functions so it should only happen once
This is the function that adds text to the screen it also prints it to tell me its working but it doesnt print it, if its the first attacking monster, am I making a noobie mistake here?
func critical_hit(attacker):
dialogue_text.text = (attacker.name + " has landed a critical hit!")
dialogue_ani.play("TextScroll")
print(attacker.name + " has landed a critical hit!")
yield(get_tree().create_timer(3), "timeout")