Small updates to dispatchers in items and puzzles

Assets/Data/Items/ExampleAss/Ass1.asset

@@ -12,6 +12,6 @@ MonoBehaviour:
   m_Script: {fileID: 11500000, guid: 951b5c8af5114086a865d4bb7eae4548, type: 3}
   m_Name: Ass1
   m_EditorClassIdentifier: 
-  itemName: TestAss
+  itemName: TestAss1
   description: Well... how do you like 'em?
   mapSprite: {fileID: -5471482676633547895, guid: 77ab3d770c92d5344b36eee3293a0f94, type: 3}

Assets/Data/Items/ExampleAss/Ass2.asset

@@ -12,6 +12,6 @@ MonoBehaviour:
   m_Script: {fileID: 11500000, guid: 951b5c8af5114086a865d4bb7eae4548, type: 3}
   m_Name: Ass2
   m_EditorClassIdentifier: 
-  itemName: TestAss
+  itemName: TestAss2
   description: Well... how do you like 'em?
   mapSprite: {fileID: 4013237533433454447, guid: 77ab3d770c92d5344b36eee3293a0f94, type: 3}

Assets/Data/Items/ExampleAss/Ass3.asset

@@ -12,6 +12,6 @@ MonoBehaviour:
   m_Script: {fileID: 11500000, guid: 951b5c8af5114086a865d4bb7eae4548, type: 3}
   m_Name: Ass3
   m_EditorClassIdentifier: 
-  itemName: TestAss
+  itemName: TestAss3
   description: Well... how do you like 'em?
   mapSprite: {fileID: -8109988653212156562, guid: 77ab3d770c92d5344b36eee3293a0f94, type: 3}

Assets/Dialogue/TestDialogue.dialoguegraph

@@ -36,6 +36,9 @@ MonoBehaviour:
         - rid: 1219994508087787764
         - rid: 1219994508087787770
         - rid: 1219994508087787776
+        - rid: 1219994508087787907
+        - rid: 1219994508087787922
+        - rid: 1219994508087787946
         m_GraphWireModels:
         - rid: 1219994508087787755
         - rid: 1219994508087787773
@@ -55,7 +58,7 @@ MonoBehaviour:
           x: 123
           y: -65
           width: 1698
-          height: 242
+          height: 763
         m_GraphElementMetaData:
         - m_Guid:
             m_Value0: 13346176596883742728
@@ -153,6 +156,30 @@ MonoBehaviour:
             Hash: 1b76d4d1d2059b91f866d271199acbfd
           m_Category: 2
           m_Index: 4
+        - m_Guid:
+            m_Value0: 17687616821000604665
+            m_Value1: 11418671301582040963
+          m_HashGuid:
+            serializedVersion: 2
+            Hash: f90bf722c90977f583afbed10c45779e
+          m_Category: 0
+          m_Index: 6
+        - m_Guid:
+            m_Value0: 5830891262905902794
+            m_Value1: 8531471494669057217
+          m_HashGuid:
+            serializedVersion: 2
+            Hash: ca666fd1b57ceb50c1bc1e6b1fe06576
+          m_Category: 0
+          m_Index: 7
+        - m_Guid:
+            m_Value0: 16523742220227669027
+            m_Value1: 12072672733572090161
+          m_HashGuid:
+            serializedVersion: 2
+            Hash: 236ccbb4002050e531c91b2ee3bf8aa7
+          m_Category: 0
+          m_Index: 8
         m_EntryPoint:
           rid: 1219994508087787747
         m_Graph:
@@ -693,3 +720,287 @@ MonoBehaviour:
       type: {class: 'Constant`1[[System.Boolean, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
       data:
         m_Value: 0
+    - rid: 1219994508087787907
+      type: {class: UserNodeModelImp, ns: Unity.GraphToolkit.Editor.Implementation, asm: Unity.GraphToolkit.Editor}
+      data:
+        m_Guid:
+          m_Value0: 17687616821000604665
+          m_Value1: 11418671301582040963
+        m_HashGuid:
+          serializedVersion: 2
+          Hash: f90bf722c90977f583afbed10c45779e
+        m_Version: 2
+        m_Position: {x: 337.83755, y: 301.66}
+        m_Title: 
+        m_Tooltip: 
+        m_NodePreviewModel:
+          rid: -2
+        m_State: 0
+        m_InputConstantsById:
+          m_KeyList:
+          - __option_DialogueLineType
+          - __option_NoLines
+          - DefaultDialogueLine
+          - LoopThroughDefaultLines
+          - RequiredPickup
+          m_ValueList:
+          - rid: 1219994508087787908
+          - rid: 1219994508087787909
+          - rid: 1219994508087787910
+          - rid: 1219994508087787911
+          - rid: 1219994508087787916
+        m_InputPortInfos:
+          expandedPortsById:
+            m_KeyList: []
+            m_ValueList: 
+        m_OutputPortInfos:
+          expandedPortsById:
+            m_KeyList: []
+            m_ValueList: 
+        m_Collapsed: 0
+        m_CurrentModeIndex: 0
+        m_ElementColor:
+          m_Color: {r: 0, g: 0, b: 0, a: 0}
+          m_HasUserColor: 0
+        m_Node:
+          rid: 1219994508087787913
+    - rid: 1219994508087787908
+      type: {class: EnumConstant, ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value:
+          m_EnumType:
+            m_Identification: Editor.Dialogue.DialogueType, AppleHillsEditor, Version=0.0.0.0,
+              Culture=neutral, PublicKeyToken=null
+          m_Value: 0
+        m_EnumType:
+          m_Identification: Editor.Dialogue.DialogueType, AppleHillsEditor, Version=0.0.0.0,
+            Culture=neutral, PublicKeyToken=null
+    - rid: 1219994508087787909
+      type: {class: 'Constant`1[[System.Int32, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 1
+    - rid: 1219994508087787910
+      type: {class: 'Constant`1[[System.String, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 
+    - rid: 1219994508087787911
+      type: {class: 'Constant`1[[System.Boolean, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 0
+    - rid: 1219994508087787913
+      type: {class: WaitOnPickup, ns: Editor.Dialogue, asm: AppleHillsEditor}
+      data: 
+    - rid: 1219994508087787916
+      type: {class: 'Constant`1[[PickupItemData, AppleHillsScripts]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: {fileID: 0}
+    - rid: 1219994508087787922
+      type: {class: UserNodeModelImp, ns: Unity.GraphToolkit.Editor.Implementation, asm: Unity.GraphToolkit.Editor}
+      data:
+        m_Guid:
+          m_Value0: 5830891262905902794
+          m_Value1: 8531471494669057217
+        m_HashGuid:
+          serializedVersion: 2
+          Hash: ca666fd1b57ceb50c1bc1e6b1fe06576
+        m_Version: 2
+        m_Position: {x: 806, y: 304}
+        m_Title: 
+        m_Tooltip: 
+        m_NodePreviewModel:
+          rid: -2
+        m_State: 0
+        m_InputConstantsById:
+          m_KeyList:
+          - __option_DialogueLineType
+          - __option_NoLines
+          - __option_IncorrectItemDialogueLineType
+          - __option_IncorrectItemNoLines
+          - __option_ForbiddenItemDialogueLineType
+          - __option_ForbiddenItemNoLines
+          - DefaultDialogueLine
+          - LoopThroughDefaultLines
+          - RequiredSlot
+          - LoopThroughIncorrectItemLines
+          - ForbiddenItemDialogueLine
+          - LoopThroughForbiddenItemLines
+          - IncorrectItemDialogueLine1
+          m_ValueList:
+          - rid: 1219994508087787923
+          - rid: 1219994508087787924
+          - rid: 1219994508087787925
+          - rid: 1219994508087787926
+          - rid: 1219994508087787927
+          - rid: 1219994508087787928
+          - rid: 1219994508087787929
+          - rid: 1219994508087787930
+          - rid: 1219994508087787931
+          - rid: 1219994508087787933
+          - rid: 1219994508087787934
+          - rid: 1219994508087787935
+          - rid: 1219994508087787938
+        m_InputPortInfos:
+          expandedPortsById:
+            m_KeyList: []
+            m_ValueList: 
+        m_OutputPortInfos:
+          expandedPortsById:
+            m_KeyList: []
+            m_ValueList: 
+        m_Collapsed: 0
+        m_CurrentModeIndex: 0
+        m_ElementColor:
+          m_Color: {r: 0, g: 0, b: 0, a: 0}
+          m_HasUserColor: 0
+        m_Node:
+          rid: 1219994508087787936
+    - rid: 1219994508087787923
+      type: {class: EnumConstant, ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value:
+          m_EnumType:
+            m_Identification: Editor.Dialogue.DialogueType, AppleHillsEditor, Version=0.0.0.0,
+              Culture=neutral, PublicKeyToken=null
+          m_Value: 0
+        m_EnumType:
+          m_Identification: Editor.Dialogue.DialogueType, AppleHillsEditor, Version=0.0.0.0,
+            Culture=neutral, PublicKeyToken=null
+    - rid: 1219994508087787924
+      type: {class: 'Constant`1[[System.Int32, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 1
+    - rid: 1219994508087787925
+      type: {class: EnumConstant, ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value:
+          m_EnumType:
+            m_Identification: Editor.Dialogue.DialogueType, AppleHillsEditor, Version=0.0.0.0,
+              Culture=neutral, PublicKeyToken=null
+          m_Value: 1
+        m_EnumType:
+          m_Identification: Editor.Dialogue.DialogueType, AppleHillsEditor, Version=0.0.0.0,
+            Culture=neutral, PublicKeyToken=null
+    - rid: 1219994508087787926
+      type: {class: 'Constant`1[[System.Int32, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 1
+    - rid: 1219994508087787927
+      type: {class: EnumConstant, ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value:
+          m_EnumType:
+            m_Identification: Editor.Dialogue.DialogueType, AppleHillsEditor, Version=0.0.0.0,
+              Culture=neutral, PublicKeyToken=null
+          m_Value: 0
+        m_EnumType:
+          m_Identification: Editor.Dialogue.DialogueType, AppleHillsEditor, Version=0.0.0.0,
+            Culture=neutral, PublicKeyToken=null
+    - rid: 1219994508087787928
+      type: {class: 'Constant`1[[System.Int32, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 1
+    - rid: 1219994508087787929
+      type: {class: 'Constant`1[[System.String, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 
+    - rid: 1219994508087787930
+      type: {class: 'Constant`1[[System.Boolean, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 0
+    - rid: 1219994508087787931
+      type: {class: 'Constant`1[[PickupItemData, AppleHillsScripts]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: {fileID: 0}
+    - rid: 1219994508087787933
+      type: {class: 'Constant`1[[System.Boolean, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 0
+    - rid: 1219994508087787934
+      type: {class: 'Constant`1[[System.String, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 
+    - rid: 1219994508087787935
+      type: {class: 'Constant`1[[System.Boolean, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 0
+    - rid: 1219994508087787936
+      type: {class: WaitOnSlot, ns: Editor.Dialogue, asm: AppleHillsEditor}
+      data: 
+    - rid: 1219994508087787938
+      type: {class: 'Constant`1[[System.String, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 
+    - rid: 1219994508087787946
+      type: {class: UserNodeModelImp, ns: Unity.GraphToolkit.Editor.Implementation, asm: Unity.GraphToolkit.Editor}
+      data:
+        m_Guid:
+          m_Value0: 16523742220227669027
+          m_Value1: 12072672733572090161
+        m_HashGuid:
+          serializedVersion: 2
+          Hash: 236ccbb4002050e531c91b2ee3bf8aa7
+        m_Version: 2
+        m_Position: {x: 1347.0507, y: 296.67874}
+        m_Title: 
+        m_Tooltip: 
+        m_NodePreviewModel:
+          rid: -2
+        m_State: 0
+        m_InputConstantsById:
+          m_KeyList:
+          - __option_DialogueLineType
+          - __option_NoLines
+          - RequiredPuzzleStep
+          - DefaultDialogueLine
+          - LoopThroughDefaultLines
+          m_ValueList:
+          - rid: 1219994508087787947
+          - rid: 1219994508087787948
+          - rid: 1219994508087787949
+          - rid: 1219994508087787950
+          - rid: 1219994508087787951
+        m_InputPortInfos:
+          expandedPortsById:
+            m_KeyList: []
+            m_ValueList: 
+        m_OutputPortInfos:
+          expandedPortsById:
+            m_KeyList: []
+            m_ValueList: 
+        m_Collapsed: 0
+        m_CurrentModeIndex: 0
+        m_ElementColor:
+          m_Color: {r: 0, g: 0, b: 0, a: 0}
+          m_HasUserColor: 0
+        m_Node:
+          rid: 1219994508087787952
+    - rid: 1219994508087787947
+      type: {class: EnumConstant, ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value:
+          m_EnumType:
+            m_Identification: Editor.Dialogue.DialogueType, AppleHillsEditor, Version=0.0.0.0,
+              Culture=neutral, PublicKeyToken=null
+          m_Value: 0
+        m_EnumType:
+          m_Identification: Editor.Dialogue.DialogueType, AppleHillsEditor, Version=0.0.0.0,
+            Culture=neutral, PublicKeyToken=null
+    - rid: 1219994508087787948
+      type: {class: 'Constant`1[[System.Int32, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 1
+    - rid: 1219994508087787949
+      type: {class: 'Constant`1[[PuzzleStepSO, AppleHillsScripts]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: {fileID: 0}
+    - rid: 1219994508087787950
+      type: {class: 'Constant`1[[System.String, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 
+    - rid: 1219994508087787951
+      type: {class: 'Constant`1[[System.Boolean, mscorlib]]', ns: Unity.GraphToolkit.Editor, asm: Unity.GraphToolkit.Internal.Editor}
+      data:
+        m_Value: 0
+    - rid: 1219994508087787952
+      type: {class: WaitOnPuzzleStep, ns: Editor.Dialogue, asm: AppleHillsEditor}
+      data: 

Assets/Scripts/Interactions/ItemSlot.cs

@@ -1,18 +1,38 @@
 using System.Collections.Generic;
 using UnityEngine;
 using UnityEngine.Events;
+using System; // for Action<T>
 
 namespace Interactions
 {
+    // New enum describing possible states for the slotted item
+    public enum ItemSlotState
+    {
+        None,
+        Correct,
+        Incorrect,
+        Forbidden
+    }
+
     /// <summary>
     /// Interaction requirement that allows slotting, swapping, or picking up items in a slot.
     /// </summary>
     [RequireComponent(typeof(Interactable))]
     public class ItemSlot : Pickup
     {
+        // Tracks the current state of the slotted item
+        private ItemSlotState _currentState = ItemSlotState.None;
+
+        /// <summary>
+        /// Read-only access to the current slotted item state.
+        /// </summary>
+        public ItemSlotState CurrentSlottedState => _currentState;
+
         public UnityEvent onItemSlotted;
         public UnityEvent onItemSlotRemoved;
         public UnityEvent onCorrectItemSlotted;
+        // Native C# event alternative to the UnityEvent for code-only subscribers
+        public event Action<PickupItemData, PickupItemData> OnCorrectItemSlotted;
         public UnityEvent onIncorrectItemSlotted;
         public UnityEvent onForbiddenItemSlotted;
         private PickupItemData _currentlySlottedItemData;
@@ -48,6 +68,7 @@ namespace Interactions
                 {
                     DebugUIMessage.Show("Can't place that here.", Color.red);
                     onForbiddenItemSlotted?.Invoke();
+                    _currentState = ItemSlotState.Forbidden;
                     Interactable.BroadcastInteractionComplete(false);
                     return;
                 }
@@ -62,6 +83,7 @@ namespace Interactions
             {
                 FollowerController.TryPickupItem(_currentlySlottedItemObject, _currentlySlottedItemData, false);
                 onItemSlotRemoved?.Invoke();
+                _currentState = ItemSlotState.None;
                 SlotItem(heldItemObj, heldItemData, _currentlySlottedItemObject == null);
                 return;
             }
@@ -109,6 +131,8 @@ namespace Interactions
             {
                 _currentlySlottedItemObject = null;
                 _currentlySlottedItemData = null;
+                // Clear state when no item is slotted
+                _currentState = ItemSlotState.None;
             }
             else
             {
@@ -133,6 +157,8 @@ namespace Interactions
                 {
                     DebugUIMessage.Show("You correctly slotted " + itemToSlotData.itemName + " into: " + itemData.itemName, Color.green);
                     onCorrectItemSlotted?.Invoke();
+                    OnCorrectItemSlotted?.Invoke(itemData, _currentlySlottedItemData);
+                    _currentState = ItemSlotState.Correct;
                 }
                     
                 Interactable.BroadcastInteractionComplete(true);
@@ -143,6 +169,7 @@ namespace Interactions
                 {
                     DebugUIMessage.Show("I'm not sure this works.", Color.yellow);
                     onIncorrectItemSlotted?.Invoke();
+                    _currentState = ItemSlotState.Incorrect;
                 }
                 Interactable.BroadcastInteractionComplete(false);
             }

Assets/Scripts/Interactions/Pickup.cs

@@ -1,5 +1,6 @@
 using Input;
 using UnityEngine;
+using System; // added for Action<T>
 
 namespace Interactions
 {
@@ -12,6 +13,9 @@ namespace Interactions
         private PlayerTouchController _playerRef;
         protected FollowerController FollowerController;
 
+        // Event: invoked when the item was picked up successfully
+        public event Action<PickupItemData> OnItemPickedUp;
+        
         /// <summary>
         /// Unity Awake callback. Sets up icon, interactable, and event handlers.
         /// </summary>
@@ -89,7 +93,14 @@ namespace Interactions
             }
             
             FollowerController?.TryPickupItem(gameObject, itemData);
-            Interactable.BroadcastInteractionComplete(combinationResult == FollowerController.CombinationResult.NotApplicable);
+            bool wasPickedUp = (combinationResult == FollowerController.CombinationResult.NotApplicable);
+            Interactable.BroadcastInteractionComplete(wasPickedUp);
+
+            // Invoke native C# event when the item was picked up successfully
+            if (wasPickedUp)
+            {
+                OnItemPickedUp?.Invoke(itemData);
+            }
         }
     }
 }

Assets/Scripts/PuzzleS/ObjectiveStepBehaviour.cs

@@ -95,7 +95,7 @@ namespace PuzzleS
             if (success)
             {
                 Debug.Log($"[Puzzles] Step interacted: {stepData?.stepId} on {gameObject.name}");
-                PuzzleManager.Instance?.OnStepCompleted(stepData);
+                PuzzleManager.Instance?.MarkPuzzleStepCompleted(stepData);
             }
         }
     }

Assets/Scripts/PuzzleS/PuzzleManager.cs

@@ -1,16 +1,17 @@
-using UnityEngine;
+using System;
 using System.Collections.Generic;
-using PuzzleS;
+using UnityEngine;
 using UnityEngine.SceneManagement;
-using System; // added for Action<T>
 
+namespace PuzzleS
+{
     /// <summary>
     /// Manages puzzle step registration, dependency management, and step completion for the puzzle system.
     /// </summary>
     public class PuzzleManager : MonoBehaviour
     {
         private static PuzzleManager _instance;
-    private static bool _isQuitting = false;
+        private static bool _isQuitting;
 
         /// <summary>
         /// Singleton instance of the PuzzleManager.
@@ -34,17 +35,15 @@ public class PuzzleManager : MonoBehaviour
         }
 
         // Events to notify about step lifecycle
-    public event Action<PuzzleStepSO> StepCompleted;
+        public event Action<PuzzleStepSO> OnStepCompleted;
-    public event Action<PuzzleStepSO> StepUnlocked;
+        public event Action<PuzzleStepSO> OnStepUnlocked;
-
-    private HashSet<PuzzleStepSO> completedSteps = new HashSet<PuzzleStepSO>();
-    private HashSet<PuzzleStepSO> unlockedSteps = new HashSet<PuzzleStepSO>();
 
+        private HashSet<PuzzleStepSO> _completedSteps = new HashSet<PuzzleStepSO>();
+        private HashSet<PuzzleStepSO> _unlockedSteps = new HashSet<PuzzleStepSO>();
         // Registration for ObjectiveStepBehaviour
-    private Dictionary<PuzzleStepSO, ObjectiveStepBehaviour> stepBehaviours = new Dictionary<PuzzleStepSO, ObjectiveStepBehaviour>();
+        private Dictionary<PuzzleStepSO, ObjectiveStepBehaviour> _stepBehaviours = new Dictionary<PuzzleStepSO, ObjectiveStepBehaviour>();
-
         // Runtime dependency graph
-    private Dictionary<PuzzleStepSO, List<PuzzleStepSO>> runtimeDependencies = new Dictionary<PuzzleStepSO, List<PuzzleStepSO>>();
+        private Dictionary<PuzzleStepSO, List<PuzzleStepSO>> _runtimeDependencies = new Dictionary<PuzzleStepSO, List<PuzzleStepSO>>();
 
         void Awake()
         {
@@ -71,9 +70,9 @@ public class PuzzleManager : MonoBehaviour
         public void RegisterStepBehaviour(ObjectiveStepBehaviour behaviour)
         {
             if (behaviour?.stepData == null) return;
-        if (!stepBehaviours.ContainsKey(behaviour.stepData))
+            if (!_stepBehaviours.ContainsKey(behaviour.stepData))
             {
-            stepBehaviours.Add(behaviour.stepData, behaviour);
+                _stepBehaviours.Add(behaviour.stepData, behaviour);
                 Debug.Log($"[Puzzles] Registered step: {behaviour.stepData.stepId} on {behaviour.gameObject.name}");
             }
         }
@@ -85,7 +84,7 @@ public class PuzzleManager : MonoBehaviour
         public void UnregisterStepBehaviour(ObjectiveStepBehaviour behaviour)
         {
             if (behaviour?.stepData == null) return;
-        stepBehaviours.Remove(behaviour.stepData);
+            _stepBehaviours.Remove(behaviour.stepData);
             Debug.Log($"[Puzzles] Unregistered step: {behaviour.stepData.stepId} on {behaviour.gameObject.name}");
         }
 
@@ -94,15 +93,15 @@ public class PuzzleManager : MonoBehaviour
         /// </summary>
         private void BuildRuntimeDependencies()
         {
-        runtimeDependencies = PuzzleGraphUtility.BuildDependencyGraph(stepBehaviours.Keys);
+            _runtimeDependencies = PuzzleGraphUtility.BuildDependencyGraph(_stepBehaviours.Keys);
-        foreach (var step in runtimeDependencies.Keys)
+            foreach (var step in _runtimeDependencies.Keys)
             {
-            foreach (var dep in runtimeDependencies[step])
+                foreach (var dep in _runtimeDependencies[step])
                 {
                     Debug.Log($"[Puzzles] Step {step.stepId} depends on {dep.stepId}");
                 }
             }
-        Debug.Log($"[Puzzles] Runtime dependencies built. Total steps: {stepBehaviours.Count}");
+            Debug.Log($"[Puzzles] Runtime dependencies built. Total steps: {_stepBehaviours.Count}");
         }
 
         /// <summary>
@@ -110,7 +109,7 @@ public class PuzzleManager : MonoBehaviour
         /// </summary>
         private void UnlockInitialSteps()
         {
-        var initialSteps = PuzzleGraphUtility.FindInitialSteps(runtimeDependencies);
+            var initialSteps = PuzzleGraphUtility.FindInitialSteps(_runtimeDependencies);
             foreach (var step in initialSteps)
             {
                 Debug.Log($"[Puzzles] Initial step unlocked: {step.stepId}");
@@ -122,14 +121,14 @@ public class PuzzleManager : MonoBehaviour
         /// Called when a step is completed. Unlocks dependent steps if their dependencies are met.
         /// </summary>
         /// <param name="step">The completed step.</param>
-    public void OnStepCompleted(PuzzleStepSO step)
+        public void MarkPuzzleStepCompleted(PuzzleStepSO step)
         {
-        if (completedSteps.Contains(step)) return;
+            if (_completedSteps.Contains(step)) return;
-        completedSteps.Add(step);
+            _completedSteps.Add(step);
             Debug.Log($"[Puzzles] Step completed: {step.stepId}");
 
             // Broadcast completion
-        StepCompleted?.Invoke(step);
+            OnStepCompleted?.Invoke(step);
 
             foreach (var unlock in step.unlocks)
             {
@@ -153,10 +152,10 @@ public class PuzzleManager : MonoBehaviour
         /// <returns>True if all dependencies are met, false otherwise.</returns>
         private bool AreRuntimeDependenciesMet(PuzzleStepSO step)
         {
-        if (!runtimeDependencies.ContainsKey(step) || runtimeDependencies[step].Count == 0) return true;
+            if (!_runtimeDependencies.ContainsKey(step) || _runtimeDependencies[step].Count == 0) return true;
-        foreach (var dep in runtimeDependencies[step])
+            foreach (var dep in _runtimeDependencies[step])
             {
-            if (!completedSteps.Contains(dep)) return false;
+                if (!_completedSteps.Contains(dep)) return false;
             }
             return true;
         }
@@ -167,16 +166,16 @@ public class PuzzleManager : MonoBehaviour
         /// <param name="step">The step to unlock.</param>
         private void UnlockStep(PuzzleStepSO step)
         {
-        if (unlockedSteps.Contains(step)) return;
+            if (_unlockedSteps.Contains(step)) return;
-        unlockedSteps.Add(step);
+            _unlockedSteps.Add(step);
-        if (stepBehaviours.TryGetValue(step, out var behaviour))
+            if (_stepBehaviours.TryGetValue(step, out var behaviour))
             {
                 behaviour.UnlockStep();
             }
             Debug.Log($"[Puzzles] Step unlocked: {step.stepId}");
 
             // Broadcast unlock
-        StepUnlocked?.Invoke(step);
+            OnStepUnlocked?.Invoke(step);
         }
 
         /// <summary>
@@ -184,7 +183,7 @@ public class PuzzleManager : MonoBehaviour
         /// </summary>
         private void CheckPuzzleCompletion()
         {
-        if (completedSteps.Count == stepBehaviours.Count)
+            if (_completedSteps.Count == _stepBehaviours.Count)
             {
                 Debug.Log("[Puzzles] Puzzle complete! All steps finished.");
                 // TODO: Fire puzzle complete event or trigger outcome logic
@@ -198,7 +197,7 @@ public class PuzzleManager : MonoBehaviour
         {
             BuildRuntimeDependencies();
             UnlockInitialSteps();
-        return unlockedSteps.Contains(step);
+            return _unlockedSteps.Contains(step);
         }
 
         void OnApplicationQuit()
@@ -206,3 +205,4 @@ public class PuzzleManager : MonoBehaviour
             _isQuitting = true;
         }
     }
+}

Assets/Settings/InteractionSettings.asset

@@ -21,8 +21,8 @@ MonoBehaviour:
   basePickupPrefab: {fileID: 7447346505753002421, guid: bf4b9d7045397f946b2125b1ad4a3fbd, type: 3}
   levelSwitchMenuPrefab: {fileID: 4062459998181038721, guid: de2ed28e4200a4340a5af4086c98a0dc, type: 3}
   combinationRules:
-  - itemA: {fileID: 11400000, guid: 33e7ca06b22108d4e802486e08bcdfd1, type: 2}
+  - itemA: {fileID: 11400000, guid: 8b2616beb14825a46b9b1ed85ad3cb25, type: 2}
-    itemB: {fileID: 11400000, guid: 8b2616beb14825a46b9b1ed85ad3cb25, type: 2}
+    itemB: {fileID: 11400000, guid: 33e7ca06b22108d4e802486e08bcdfd1, type: 2}
     resultPrefab: {fileID: 1610562450228973293, guid: 58654125374567147839eb382fcde422, type: 3}
   - itemA: {fileID: 11400000, guid: 983414276ae3f004c854e9c450f27f88, type: 2}
     itemB: {fileID: 11400000, guid: 0c6986639ca176a419c92f5a327d95ce, type: 2}