fix(platform): address review comments on PR #53

- Add mutex locks to hasProducer/hasConsumer to prevent data race
  with the reconcile service thread modifying _actual* maps
- Fix format string typo '%''' → '%s' in deployment verify()

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: lterrac

platform/include/modules/channelController/module.hpp

@@ -75,8 +75,16 @@ class Module final : public serving::modules::Module
     return _desiredConsumers[edgeName];
   }
 
-  [[nodiscard]] __INLINE__ bool hasProducer(const edgeName_t edgeName) const { return _actualProducers.contains(edgeName); }
-  [[nodiscard]] __INLINE__ bool hasConsumer(const edgeName_t edgeName) const { return _actualConsumers.contains(edgeName); }
+  [[nodiscard]] __INLINE__ bool hasProducer(const edgeName_t edgeName) const
+  {
+    std::lock_guard lock(_producerMutex);
+    return _actualProducers.contains(edgeName);
+  }
+  [[nodiscard]] __INLINE__ bool hasConsumer(const edgeName_t edgeName) const
+  {
+    std::lock_guard lock(_consumerMutex);
+    return _actualConsumers.contains(edgeName);
+  }
 
   [[nodiscard]] __INLINE__ std::weak_ptr<output_t> getProducer(const edgeName_t edgeName) const
   {

platform/include/modules/configuration/deployment.hpp

@@ -242,7 +242,7 @@ class Deployment final
 
         // Make sure all tasks have at least one output
         if (task->getOutputs().size() == 0 && tasksWithDependencies.contains(task->getFunctionName()) == false)
-          HICR_THROW_LOGIC("Deployment specifies task in partition '%'' with function name '%s' without any outputs or dependents\n",
+          HICR_THROW_LOGIC("Deployment specifies task in partition '%s' with function name '%s' without any outputs or dependents\n",
                            partition->getName().c_str(),
                            task->getFunctionName().c_str());