feat: Update Logger and BaseComponent to return tag/name as const…

… ref (#245)

* feat: Update `Logger` and `BaseComponent` to return tag/name as const ref

* fix sa

components/base_component/include/base_component.hpp

@@ -11,9 +11,9 @@ namespace espp {
 class BaseComponent {
 public:
   /// Get the name of the component
-  /// \return The name of the component
+  /// \return A const reference to the name of the component
   /// \note This is the tag of the logger
-  std::string get_name() { return logger_.get_tag(); }
+  const std::string &get_name() const { return logger_.get_tag(); }
 
   /// Set the tag for the logger
   /// \param tag The tag to use for the logger

components/logger/include/logger.hpp

@@ -84,12 +84,9 @@ class Logger {
 
   /**
    * @brief Get the current tag for the logger.
-   * @return The current tag.
+   * @return A const reference to the current tag.
    */
-  std::string get_tag() {
-    std::lock_guard<std::mutex> lock(tag_mutex_);
-    return tag_;
-  }
+  const std::string &get_tag() const { return tag_; }
 
   /**
    * @brief Whether to include the time in the log.