Add partial sync for MeleeAttack

Author: dartasen

NitroxClient/Communication/Packets/Processors/PlayerStatsProcessor.cs

@@ -18,7 +18,7 @@ public override void Process(PlayerStats playerStats)
     {
         if (playerManager.TryFind(playerStats.PlayerId, out RemotePlayer remotePlayer))
         {
-            RemotePlayerVitals vitals = remotePlayer.vitals;
+            RemotePlayerVitals vitals = remotePlayer.Vitals;
             vitals.SetOxygen(playerStats.Oxygen, playerStats.MaxOxygen);
             vitals.SetHealth(playerStats.Health);
             vitals.SetFood(playerStats.Food);

NitroxClient/GameLogic/RemotePlayer.cs

@@ -40,7 +40,7 @@ public class RemotePlayer : INitroxPlayer
     public AnimationController AnimationController { get; private set; }
     public ItemsContainer Inventory { get; private set; }
     public Transform ItemAttachPoint { get; private set; }
-    public RemotePlayerVitals vitals { get; private set; }
+    public RemotePlayerVitals Vitals { get; private set; }
 
     public ushort PlayerId => PlayerContext.PlayerId;
     public string PlayerName => PlayerContext.PlayerName;
@@ -104,7 +104,7 @@ public void InitializeGameObject(GameObject playerBody)
         SetupPlayerSounds();
         SetupMixins();
 
-        vitals = playerVitalsManager.CreateOrFindForPlayer(this);
+        Vitals = playerVitalsManager.CreateOrFindForPlayer(this);
         RefreshVitalsVisibility();
 
         PlayerDisconnectEvent.AddHandler(Body, _ =>
@@ -549,10 +549,10 @@ public void SetGameMode(NitroxGameMode gameMode)
 
     private void RefreshVitalsVisibility()
     {
-        if (vitals)
+        if (Vitals)
         {
             bool visible = PlayerContext.GameMode != NitroxGameMode.CREATIVE;
-            vitals.SetStatsVisible(visible);
+            Vitals.SetStatsVisible(visible);
         }
     }
 

NitroxPatcher/Patches/Dynamic/MeleeAttack_CanDealDamageTo_Patch.cs

@@ -14,7 +14,7 @@ namespace NitroxPatcher.Patches.Dynamic;
 /// </summary>
 public sealed partial class MeleeAttack_CanDealDamageTo_Patch : NitroxPatch, IDynamicPatch
 {
-    private static readonly MethodInfo TARGET_METHOD = Reflect.Method((MeleeAttack t) => t.CanDealDamageTo(default));
+    internal static readonly MethodInfo TARGET_METHOD = Reflect.Method((MeleeAttack t) => t.CanDealDamageTo(default));
 
     /*
      * REPLACE:

NitroxPatcher/Patches/Dynamic/MeleeAttack_OnTouch_Patch.cs

@@ -0,0 +1,36 @@
+using System.Collections.Generic;
+using System.Reflection;
+using System.Reflection.Emit;
+using HarmonyLib;
+using NitroxClient.GameLogic;
+using NitroxClient.MonoBehaviours.Cyclops;
+using NitroxModel.DataStructures;
+using NitroxModel.Helper;
+using UnityEngine;
+
+namespace NitroxPatcher.Patches.Dynamic;
+
+
+/// <summary>
+/// Prevents non simulating players from running locally <see cref="MeleeAttack.OnTouch(Collider)"/>.
+/// 
+/// Adds RemotePlayer support for the simulating player
+/// </summary>
+public sealed partial class MeleeAttack_OnTouch_Patch : NitroxPatch, IDynamicPatch
+{
+    internal static readonly MethodInfo TARGET_METHOD = Reflect.Method((MeleeAttack t) => t.OnTouch(default));
+
+    public static bool Prefix(MeleeAttack __instance)
+    {
+        if (!__instance.TryGetNitroxId(out NitroxId creatureId) ||
+            Resolve<SimulationOwnership>().HasAnyLockType(creatureId))
+        {
+            return true;
+        }
+
+        return false;
+    }
+
+    // TODO: Add transpiler to add support for held item eat from remote players, we might need to add an equivalent of Inventory.GetHeldItem()
+    // For the MeleeAttack part , it'll natively work since RemotePlayer should have a LiveMixin
+}