Right, of course, UnitIsPLayer expects a UNIT how silly of me.....

I hate the wow api

Modules/StinkyTracker.lua

@@ -212,6 +212,7 @@ function shared.StinkyTracker.Init()
 			return
 		end
 
+		local name = UnitName(unit)
 		if not UnitIsPlayer(unit) then
 			if Heimdall_Data.config.stinkyTracker.debug then
 				print(string.format("[%s] Target %s is not a player, nothing to do", ModuleName, name))
@@ -219,7 +220,6 @@ function shared.StinkyTracker.Init()
 			return
 		end
 
-		local name = UnitName(unit)
 		local enemy = UnitCanAttack("player", unit)
 		if enemy then
 			if Heimdall_Data.config.stinkyTracker.debug then