fix(StatusChangeLogger): release mutex before calling user callback

Prevent recursive mutex locking when multiple nodes change status
during a single tree tick. The callback lambda now copies state
under the lock, releases it, then calls user code.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: facontidavide

include/behaviortree_cpp/loggers/abstract_logger.h

@@ -86,18 +86,24 @@ inline void StatusChangeLogger::subscribeToTreeChanges(TreeNode* root_node)
 
   auto subscribeCallback = [this](TimePoint timestamp, const TreeNode& node,
                                   NodeStatus prev, NodeStatus status) {
-    std::unique_lock lk(callback_mutex_);
-    if(enabled_ && (status != NodeStatus::IDLE || show_transition_to_idle_))
+    // Copy state under lock, then release before calling user code
+    // This prevents recursive mutex locking when multiple nodes change status
+    bool should_callback = false;
+    Duration adjusted_timestamp;
     {
-      if(type_ == TimestampType::absolute)
+      std::unique_lock lk(callback_mutex_);
+      if(enabled_ && (status != NodeStatus::IDLE || show_transition_to_idle_))
       {
-        this->callback(timestamp.time_since_epoch(), node, prev, status);
-      }
-      else
-      {
-        this->callback(timestamp - first_timestamp_, node, prev, status);
+        should_callback = true;
+        adjusted_timestamp = (type_ == TimestampType::absolute) ?
+                                 timestamp.time_since_epoch() :
+                                 (timestamp - first_timestamp_);
       }
     }
+    if(should_callback)
+    {
+      this->callback(adjusted_timestamp, node, prev, status);
+    }
   };
 
   auto visitor = [this, subscribeCallback](TreeNode* node) {