fix(logs): respect NO_COLOR for partition names

The partition name colors (e.g., Application:, NetworkOPs:) were
ignoring the NO_COLOR environment variable and always applying
color codes. Now both partition names and location suffixes
properly respect should_log_use_colors() which honors NO_COLOR
and terminal detection.

src/ripple/basics/impl/Log.cpp

@@ -360,6 +360,7 @@ Logs::format(
     if (!partition.empty())
     {
 #ifdef BEAST_ENHANCED_LOGGING
+        if (beast::detail::should_log_use_colors())
             output += beast::detail::get_log_highlight_color();
 #endif
         output += partition + ":";
@@ -392,6 +393,7 @@ Logs::format(
     }
 
 #ifdef BEAST_ENHANCED_LOGGING
+    if (beast::detail::should_log_use_colors())
         output += "\033[0m";
 #endif
 

src/ripple/beast/utility/src/beast_Journal.cpp

@@ -155,14 +155,43 @@ Journal::ScopedStream::~ScopedStream()
 
 #ifdef BEAST_ENHANCED_LOGGING
     // Add suffix if location is enabled
-    if (file_ && detail::should_show_location() && !s.empty() && s != "\n")
+    if (file_ && detail::should_show_location() && !s.empty())
     {
-        std::ostringstream combined;
+        // Single optimized scan from the end
-        combined << s;
+        size_t const lastNonWhitespace = s.find_last_not_of(" \n\r\t");
-        if (!s.empty() && s.back() != ' ')
+
-            combined << " ";
+        // Skip if message is only whitespace (e.g., just "\n" or "  \n\n")
-        detail::log_write_location_string(combined, file_, line_);
+        if (lastNonWhitespace != std::string::npos)
-        s = combined.str();
+        {
+            // Count only the trailing newlines (tiny range)
+            size_t trailingNewlines = 0;
+            for (size_t i = lastNonWhitespace + 1; i < s.length(); ++i)
+            {
+                if (s[i] == '\n')
+                    ++trailingNewlines;
+            }
+
+            // Build location string once
+            std::ostringstream locStream;
+            detail::log_write_location_string(locStream, file_, line_);
+            std::string const location = locStream.str();
+
+            // Pre-allocate exact size → zero reallocations
+            size_t const finalSize = lastNonWhitespace + 1 + 1 +
+                location.length() + trailingNewlines;
+
+            std::string result;
+            result.reserve(finalSize);
+
+            // Direct string ops (no ostringstream overhead)
+            result.append(s, 0, lastNonWhitespace + 1);
+            result.push_back(' ');
+            result += location;
+            if (trailingNewlines > 0)
+                result.append(trailingNewlines, '\n');
+
+            s = std::move(result);  // Move, no copy
+        }
     }
 #endif