refactor(scripting): extract PublishConsoleOutputMessage helper

- Add PublishConsoleOutputMessage() using BML_ConsoleOutputEvent and PublishBuffer
- Replace inline ad-hoc struct publish in LogScriptException with helper
- Replace inline publish in Script_Print (BindImc.cpp) with helper
- Include bml_console.h for BML_ConsoleOutputEvent type

Author: doyaGu

modules/BML_Scripting/src/ScriptExceptionHelper.h

@@ -1,17 +1,59 @@
 #ifndef BML_SCRIPTING_SCRIPT_EXCEPTION_HELPER_H
 #define BML_SCRIPTING_SCRIPT_EXCEPTION_HELPER_H
 
+#include <cstdlib>
+#include <cstring>
 #include <cstdio>
+#include <string_view>
 
 #include <angelscript.h>
 
+#include "bml_console.h"
 #include "bml_logging.h"
 #include "bml_topics.h"
 #include "ModuleScope.h"
 #include "ScriptInstance.h"
 
 namespace BML::Scripting {
 
+inline void PublishConsoleOutputMessage(BML_Mod owner, std::string_view message, uint32_t flags) {
+    if (!g_Services || !g_Services->ImcBus || !g_Services->ImcBus->PublishBuffer || !owner || message.empty()) {
+        return;
+    }
+
+    BML_TopicId topicId = BML_TOPIC_ID_INVALID;
+    if (g_Services->ImcBus->GetTopicId(
+            g_Services->ImcBus->Context,
+            BML_TOPIC_CONSOLE_OUTPUT,
+            &topicId) != BML_RESULT_OK) {
+        return;
+    }
+
+    const size_t textSize = message.size() + 1;
+    const size_t totalSize = sizeof(BML_ConsoleOutputEvent) + textSize;
+    auto *storage = static_cast<char *>(std::malloc(totalSize));
+    if (!storage) {
+        return;
+    }
+
+    auto *event = reinterpret_cast<BML_ConsoleOutputEvent *>(storage);
+    *event = BML_CONSOLE_OUTPUT_EVENT_INIT;
+    event->message_utf8 = storage + sizeof(BML_ConsoleOutputEvent);
+    event->flags = flags;
+
+    std::memcpy(const_cast<char *>(event->message_utf8), message.data(), message.size());
+    const_cast<char *>(event->message_utf8)[message.size()] = '\0';
+
+    BML_ImcBuffer buffer = BML_IMC_BUFFER_INIT;
+    buffer.data = event;
+    buffer.size = totalSize;
+    buffer.cleanup = [](const void *, size_t, void *userData) {
+        std::free(userData);
+    };
+    buffer.cleanup_user_data = storage;
+    (void) g_Services->ImcBus->PublishBuffer(owner, topicId, &buffer);
+}
+
 // Log an AngelScript exception with full context (function, file, line).
 // Must be called before ReleaseContext because the context holds exception info.
 inline void LogScriptException(asIScriptContext *ctx, const ScriptInstance &inst) {
@@ -36,24 +78,13 @@ inline void LogScriptException(asIScriptContext *ctx, const ScriptInstance &inst
                              exception_str);
 
     // Also forward to console output
-    if (g_Services->ImcBus && g_Services->ImcBus->Publish) {
+    if (g_Services->ImcBus && g_Services->ImcBus->PublishBuffer) {
         char buf[512];
         int n = std::snprintf(buf, sizeof(buf),
                               "\x1b[31m[%s] %s (%s:%d): %s\x1b[0m",
                               inst.mod_id.c_str(), func_decl, section, line, exception_str);
         if (n > 0 && static_cast<size_t>(n) < sizeof(buf)) {
-            BML_TopicId topic_id = BML_TOPIC_ID_INVALID;
-            if (g_Services->ImcBus->GetTopicId(
-                    g_Services->ImcBus->Context,
-                    BML_TOPIC_CONSOLE_OUTPUT,
-                    &topic_id) == BML_RESULT_OK) {
-                struct {
-                    size_t struct_size;
-                    const char *msg;
-                    uint32_t flags;
-                } event{sizeof(event), buf, 0};
-                g_Services->ImcBus->Publish(inst.mod_handle, topic_id, &event, sizeof(event));
-            }
+            PublishConsoleOutputMessage(inst.mod_handle, std::string_view(buf, static_cast<size_t>(n)), 0);
         }
     }
 }

modules/BML_Scripting/src/bindings/BindImc.cpp

@@ -212,23 +212,10 @@ static void Script_PublishInt(const std::string &topic, int value) {
 
 static void Script_Print(const std::string &message) {
     const BML_Mod owner = CurrentScriptOwner();
-    if (!g_Services || !g_Services->ImcBus || !g_Services->ImcBus->Publish || !owner) {
+    if (!owner) {
         return;
     }
-
-    BML_TopicId id = BML_TOPIC_ID_INVALID;
-    if (g_Services->ImcBus->GetTopicId(
-            g_Services->ImcBus->Context, BML_TOPIC_CONSOLE_OUTPUT, &id) != BML_RESULT_OK) return;
-
-    struct {
-        size_t struct_size;
-        const char *message_utf8;
-        uint32_t flags;
-    } event;
-    event.struct_size = sizeof(event);
-    event.message_utf8 = message.c_str();
-    event.flags = 0;
-    g_Services->ImcBus->Publish(owner, id, &event, sizeof(event));
+    PublishConsoleOutputMessage(owner, message, 0);
 }
 
 void RegisterImcBindings(asIScriptEngine *engine, ScriptInstanceManager *manager) {