Fix L’Hiver Mod: Monster HP sometimes increases during a fight

[blackus3r]

This should fix the issue described here: #813
I've tested it and it worked with L'hiver + Uriziel Mod.

Fix Npc_CanSeeNpc logic to use logical orientation instead of visual bone rotation

Previously, Npc_CanSeeNpc (via Npc::canRayHitPoint) relied on visual.viewDirection(), which calculates the viewing angle dynamically from the BIP01 root bone. This caused the perceived line of sight to fluctuate significantly during combat animations (e.g., when a character turns sideways to dodge or stumble).

As a result, mods relying on line of sight checks during combat (like the anti-exploit healing mechanic in L'Hiver) would break, causing enemies to regenerate mid-combat because they temporarily "lost sight" of the player due to their own animation.

Using the NPC's logical angle instead prevents these animation-based FOV blindspots and aligns the engine's behavior correctly with original ZenGin logic.

[Try]

Hi, @blackus3r and thanks for PR!

Use of head orientation is intended in this place. In original scripts, when npc sits on a chair/bench his face is oriented opposed to logical orientation, and then if you talk to them this script runs:

		if (Npc_CanSeeNpc(self, other))
		{
			AI_StartState 	(self, ZS_Talk, 0, ""); 	// keep talking while sit
		}	
		else 
		{
			...
			AI_StartState 	(self, ZS_Talk, 1, ""); 	// standup and turn around
		};

[blackus3r]

Hey @Try,
thanks for the important heads up!
That explains why the visual orientation was originally used in this place.
To preserve this intended behavior for chairs/benches while still fixing the combat regeneration bug in the L'Hiver mod, I've added a check for interactive objects.

What do you think about this hybrid approach?

Edit: Okay, whatever. The bug is back. I need to do more testing. I'll get back to you.

[Try]

I'll move PR into drafts for now.

one theory: maybe in L'Hiver script never meant to reach Npc_CanSeeNpc check, and was suppose to early-out earlier

[blackus3r]

@Try

I built and tested another fix last night. However, it was very late, so I'll continue testing over the next few days to see if the bug has been fixed.
In any case, I've gotten much closer to identifying the problem thanks to additional logging:

Npc_CanSeeNpc computed the view-cone direction from the NPC's head bone to the target. In melee, an attacking monster's head dives into the player model — the measured distance drops to ~35 cm and the direction vector becomes numerical noise. The ±100° view-angle check then fails randomly, even with the player directly in front and a clear line of sight. L'Hiver's anti-exploit script interprets Npc_CanSeeNpc == false as "player ran away", heals the monster (+20 HP per trigger) and drops it out of its attack state.

Fix 1: Measure the view cone from the NPC position instead of the head bone — collision keeps positions ~1 m apart in melee, so the direction stays stable. The line-of-sight ray still starts at the head bone. Matches original engine behavior.

Fix 2 : ATR_REGENERATEHP/MANA was treated as points per second; the original engine defines it as 1 point every N seconds. A value of 20 healed 20 HP/s instead of 1 HP per 20 s. Not the cause for the wolf (attribute was 0), but would make any monster that uses it effectively unkillable.

Verified: Debug logging showed 14× script heals triggered by failed sight checks after the fix, wolf fights behave normally.