Fix tool inspect animation replaying on login (SubnauticaNitrox#2634)

Author: misterbubb

NitroxClient/Communication/Packets/Processors/PlayerHeldItemChangedProcessor.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using Nitrox.Model.Core;
 using Nitrox.Model.DataStructures;
 using Nitrox.Model.Helper;
@@ -73,7 +73,7 @@ public override void Process(PlayerHeldItemChanged packet)
                 item.SetActive(true);
                 tool.SetHandIKTargetsEnabled(true);
                 SafeAnimator.SetBool(opPlayer.Value.ArmsController.GetComponent<Animator>(), $"holding_{tool.animToolName}", true);
-                opPlayer.Value.AnimationController["using_tool_first"] = packet.IsFirstTime == null;
+                opPlayer.Value.AnimationController["using_tool_first"] = packet.IsFirstTime != null;
 
                 if (item.TryGetComponent(out FPModel fpModelDraw)) //FPModel needs to be updated
                 {

NitroxClient/GameLogic/InitialSync/PlayerInitialSyncProcessor.cs

@@ -1,10 +1,12 @@
 using System.Collections;
+using System.Collections.Generic;
 using System.Text;
 using NitroxClient.GameLogic.InitialSync.Abstract;
 using NitroxClient.MonoBehaviours;
 using Nitrox.Model.DataStructures;
 using Nitrox.Model.DataStructures.GameLogic;
 using Nitrox.Model.Server;
+using Nitrox.Model.Subnautica.DataStructures;
 using Nitrox.Model.Subnautica.DataStructures.GameLogic;
 using UnityEngine;
 
@@ -35,6 +37,7 @@ public PlayerInitialSyncProcessor(Items item, ItemContainers itemContainers, Loc
         AddStep(sync => SetPlayerGameObjectId(sync.PlayerGameObjectId));
         AddStep(sync => AddStartingItemsToPlayer(sync.FirstTimeConnecting));
         AddStep(sync => SetPlayerStats(sync.PlayerStatsData));
+        AddStep(sync => SetUsedItems(sync.UsedItems));
         AddStep(sync => SetPlayerGameMode(sync.GameMode));
         AddStep(sync => ApplySettings(sync.KeepInventoryOnDeath, sync.SessionSettings.FastHatch, sync.SessionSettings.FastGrow));
     }
@@ -133,6 +136,20 @@ private static void SetPlayerStats(PlayerStatsData statsData)
         Player.main.GetPDA().Close();
     }
 
+    private static void SetUsedItems(List<NitroxTechType> usedItems)
+    {
+        if (usedItems == null)
+        {
+            return;
+        }
+
+        foreach (NitroxTechType usedItem in usedItems)
+        {
+            Player.main.usedTools.Add(usedItem.ToUnity());
+        }
+        Log.Info($"Received initial sync packet with {usedItems.Count} used items");
+    }
+
     private static void SetPlayerGameMode(NitroxGameMode gameMode)
     {
         Log.Info($"Received initial sync packet with gamemode {gameMode}");

NitroxPatcher/Patches/Dynamic/QuickSlots_SelectInternal_Patch.cs

@@ -19,7 +19,8 @@ public static void Prefix(QuickSlots __instance, int slotID, ref NitroxTechType
         {
             return;
         }
-        __state = Player.main.IsToolUsed(item.item.GetTechType()) ? item.item.GetTechType().ToDto() : null;
+        // Send the TechType if this is the first time using the tool (not yet in usedTools)
+        __state = Player.main.IsToolUsed(item.item.GetTechType()) ? null : item.item.GetTechType().ToDto();
     }
 
     public static void Postfix(InventoryItem ____heldItem, NitroxTechType __state)