fix: wire onGuiCreated callback in HelperIPC constructor to send gui_ready notification

Source/Helper/HelperIPC.cpp

@@ -15,10 +15,17 @@ HelperIPC::HelperIPC(int parentPid, HelperPluginHost& host)
     : parentPid(parentPid), host(host)
 {
     signal(SIGPIPE, SIG_IGN);
+
+    // Wire the GUI created callback so we send notification to host process
+    host.onGuiCreated = [this](const juce::String& portName, int width, int height)
+    {
+        sendGuiReady(portName, width, height);
+    };
 }
 
 HelperIPC::~HelperIPC()
 {
+    host.onGuiCreated = nullptr;
     disconnect();
 }
 
@@ -133,7 +140,6 @@ void HelperIPC::listenLoop()
         {
             auto msg = buffer.substr(0, pos);
             buffer.erase(0, pos + 1);
-
             if (!msg.empty())
                 handleMessage(juce::String(msg));
         }
@@ -152,15 +158,12 @@ void HelperIPC::handleMessage(const juce::String& message)
             auto path = j.value("path", "");
             double sr = j.value("sampleRate", 44100.0);
             int bs = j.value("blockSize", 512);
-
             bool ok = host.loadPlugin(juce::String(path), sr, bs);
-
             json response;
             if (ok)
                 response = {{"ok", true}, {"name", host.getPluginName().toStdString()}, {"params", host.getParameterCount()}};
             else
                 response = {{"ok", false}, {"error", "Failed to load plugin"}};
-
             sendMessage(juce::String(response.dump()));
         }
         else if (cmd == "unload")
@@ -171,8 +174,7 @@ void HelperIPC::handleMessage(const juce::String& message)
         else if (cmd == "show_gui")
         {
             host.showGui();
-            // GUI ready notification is sent asynchronously after GUI is created
-            // (see HelperPluginHost::showGui which calls ipc->sendGuiReady)
+            // gui_ready notification sent via onGuiCreated callback (async)
             sendMessage("{\"ok\":true}");
         }
         else if (cmd == "hide_gui")
@@ -184,64 +186,38 @@ void HelperIPC::handleMessage(const juce::String& message)
         {
             auto keyword = j.value("keyword", "");
             auto results = host.searchParams(juce::String(keyword));
-
             json arr = json::array();
             for (auto& p : results)
-            {
-                arr.push_back({
-                    {"i", p.index},
-                    {"id", p.id.toStdString()},
-                    {"n", p.name.toStdString()},
-                    {"v", std::round(p.value * 1000.0f) / 1000.0f},
-                    {"t", p.displayText.toStdString()}
-                });
-            }
-
-            json response = {{"ok", true}, {"results", arr}};
-            sendMessage(juce::String(response.dump()));
+                arr.push_back({{"i",p.index},{"id",p.id.toStdString()},{"n",p.name.toStdString()},
+                               {"v",std::round(p.value*1000.0f)/1000.0f},{"t",p.displayText.toStdString()}});
+            sendMessage(juce::String(json({{"ok",true},{"results",arr}}).dump()));
         }
         else if (cmd == "get_params")
         {
             std::vector<int> ids;
-            for (auto& id : j["ids"])
-                ids.push_back(id.get<int>());
-
+            for (auto& id : j["ids"]) ids.push_back(id.get<int>());
             auto results = host.getParams(ids);
-
             json values = json::object();
-            for (auto& [idx, val] : results)
-                values[std::to_string(idx)] = std::round(val * 1000.0f) / 1000.0f;
-
-            json response = {{"ok", true}, {"values", values}};
-            sendMessage(juce::String(response.dump()));
+            for (auto& [idx, val] : results) values[std::to_string(idx)] = std::round(val*1000.0f)/1000.0f;
+            sendMessage(juce::String(json({{"ok",true},{"values",values}}).dump()));
         }
         else if (cmd == "set_params")
         {
             std::map<int, float> values;
-            for (auto& [key, val] : j["values"].items())
-                values[std::stoi(key)] = val.get<float>();
-
+            for (auto& [key, val] : j["values"].items()) values[std::stoi(key)] = val.get<float>();
             bool ok = host.setParams(values);
             sendMessage(ok ? "{\"ok\":true}" : "{\"ok\":false}");
         }
         else if (cmd == "mouse")
         {
             auto type = j.value("type", "");
-            float x = j.value("x", 0.0f);
-            float y = j.value("y", 0.0f);
+            float x = j.value("x", 0.0f), y = j.value("y", 0.0f);
             int button = j.value("button", 0);
-
             if (type == "down")       host.injectMouseDown(x, y, button);
             else if (type == "up")    host.injectMouseUp(x, y, button);
             else if (type == "move")  host.injectMouseMove(x, y);
             else if (type == "drag")  host.injectMouseDrag(x, y, button);
-            else if (type == "scroll")
-            {
-                float dx = j.value("dx", 0.0f);
-                float dy = j.value("dy", 0.0f);
-                host.injectMouseScroll(x, y, dx, dy);
-            }
-            // No response for mouse events (fire-and-forget for low latency)
+            else if (type == "scroll") host.injectMouseScroll(x, y, j.value("dx",0.0f), j.value("dy",0.0f));
         }
         else if (cmd == "key")
         {
@@ -249,28 +225,22 @@ void HelperIPC::handleMessage(const juce::String& message)
             int keyCode = j.value("keyCode", 0);
             auto chars = j.value("chars", "");
             int modifiers = j.value("modifiers", 0);
-
-            if (type == "down")      host.injectKeyDown(keyCode, juce::String(chars), modifiers);
-            else if (type == "up")   host.injectKeyUp(keyCode, juce::String(chars), modifiers);
+            if (type == "down") host.injectKeyDown(keyCode, juce::String(chars), modifiers);
+            else if (type == "up") host.injectKeyUp(keyCode, juce::String(chars), modifiers);
         }
         else if (cmd == "get_state")
         {
             auto state = host.getStateBase64();
-            json response = {{"ok", true}, {"state", state.toStdString()}};
-            sendMessage(juce::String(response.dump()));
+            sendMessage(juce::String(json({{"ok",true},{"state",state.toStdString()}}).dump()));
         }
         else if (cmd == "set_state")
         {
-            auto state = j.value("state", "");
-            bool ok = host.setStateBase64(juce::String(state));
+            bool ok = host.setStateBase64(juce::String(j.value("state", "")));
             sendMessage(ok ? "{\"ok\":true}" : "{\"ok\":false}");
         }
         else if (cmd == "configure")
         {
-            double sr = j.value("sampleRate", 44100.0);
-            int bs = j.value("blockSize", 512);
-            int ch = j.value("channels", 2);
-            host.configure(sr, bs, ch);
+            host.configure(j.value("sampleRate",44100.0), j.value("blockSize",512), j.value("channels",2));
             sendMessage("{\"ok\":true}");
         }
         else if (cmd == "quit")