Fix: Editor reload button uses getLastPluginPath() (retained after crash)

Source/Plugin/PluginBridgeEditor.cpp

@@ -2,7 +2,7 @@
 
 static const juce::Colour kBgDark       (0xff0d0d1a);
 static const juce::Colour kBgPanel      (0xff1a1a2e);
-static const juce::Colour kAccent       (0xff6c63ff);  // Purple accent
+static const juce::Colour kAccent       (0xff6c63ff);
 static const juce::Colour kAccentDim    (0xff4a42cc);
 static const juce::Colour kTextPrimary  (0xffe8e8f0);
 static const juce::Colour kTextSecondary(0xff8888aa);
@@ -16,48 +16,40 @@ PluginBridgeEditor::PluginBridgeEditor(PluginBridgeProcessor& p)
 {
     setSize(340, 200);
 
-    // Title
     titleLabel.setText("PluginBridge", juce::dontSendNotification);
     titleLabel.setFont(juce::FontOptions(18.0f));
     titleLabel.setJustificationType(juce::Justification::centredLeft);
     titleLabel.setColour(juce::Label::textColourId, kTextPrimary);
     addAndMakeVisible(titleLabel);
 
-    // MCP status
     mcpStatusLabel.setFont(juce::FontOptions(11.0f));
     mcpStatusLabel.setJustificationType(juce::Justification::centredRight);
     addAndMakeVisible(mcpStatusLabel);
 
-    // Plugin status
     pluginStatusLabel.setFont(juce::FontOptions(13.0f));
     pluginStatusLabel.setJustificationType(juce::Justification::centredLeft);
     pluginStatusLabel.setColour(juce::Label::textColourId, kTextSecondary);
     addAndMakeVisible(pluginStatusLabel);
 
-    // Connection status
     connectionLabel.setFont(juce::FontOptions(10.0f));
     connectionLabel.setJustificationType(juce::Justification::centredLeft);
     connectionLabel.setColour(juce::Label::textColourId, kTextSecondary);
     addAndMakeVisible(connectionLabel);
 
-    // Select plugin button
     selectButton.onClick = [this]() { showPluginMenu(); };
     selectButton.setColour(juce::TextButton::buttonColourId, kAccent);
     selectButton.setColour(juce::TextButton::textColourOffId, kTextPrimary);
     addAndMakeVisible(selectButton);
 
-    // Open GUI button
     guiButton.onClick = [this]() { processorRef.showPluginGui(); };
     guiButton.setColour(juce::TextButton::buttonColourId, kBtnBg);
     guiButton.setColour(juce::TextButton::textColourOffId, kTextPrimary);
     addAndMakeVisible(guiButton);
 
-    // Reload button (shown after crash)
     reloadButton.onClick = [this]() {
-        auto path = processorRef.getLoadedPluginName();
-        // Re-attempt load with same plugin
-        // For now, just respawn helper
-        processorRef.loadPlugin(processorRef.getLoadedPluginName());
+        auto path = processorRef.getLastPluginPath();
+        if (path.isNotEmpty())
+            processorRef.loadPlugin(path);
     };
     reloadButton.setColour(juce::TextButton::buttonColourId, kRed);
     reloadButton.setColour(juce::TextButton::textColourOffId, kTextPrimary);
@@ -76,12 +68,10 @@ void PluginBridgeEditor::paint(juce::Graphics& g)
 {
     g.fillAll(kBgDark);
 
-    // Main panel
     auto panelArea = getLocalBounds().reduced(8);
     g.setColour(kBgPanel);
     g.fillRoundedRectangle(panelArea.toFloat(), 8.0f);
 
-    // Subtle border
     g.setColour(kAccent.withAlpha(0.3f));
     g.drawRoundedRectangle(panelArea.toFloat(), 8.0f, 1.0f);
 }
@@ -90,20 +80,15 @@ void PluginBridgeEditor::resized()
 {
     auto area = getLocalBounds().reduced(16);
 
-    // Top row: title + MCP status
     auto topRow = area.removeFromTop(24);
     titleLabel.setBounds(topRow.removeFromLeft(160));
     mcpStatusLabel.setBounds(topRow);
 
     area.removeFromTop(8);
-
-    // Plugin status line
     pluginStatusLabel.setBounds(area.removeFromTop(20));
     connectionLabel.setBounds(area.removeFromTop(16));
-
     area.removeFromTop(12);
 
-    // Button row
     auto buttonRow = area.removeFromTop(32);
     selectButton.setBounds(buttonRow.removeFromLeft(140));
     buttonRow.removeFromLeft(8);
@@ -114,7 +99,6 @@ void PluginBridgeEditor::resized()
 
 void PluginBridgeEditor::timerCallback()
 {
-    // MCP status
     if (processorRef.isMcpServerRunning())
     {
         mcpStatusLabel.setText("MCP:" + juce::String(processorRef.getMcpServerPort()), juce::dontSendNotification);
@@ -126,7 +110,6 @@ void PluginBridgeEditor::timerCallback()
         mcpStatusLabel.setColour(juce::Label::textColourId, kRed);
     }
 
-    // Plugin + connection status
     auto connState = processorRef.getConnectionState();
 
     if (processorRef.isPluginLoaded())
@@ -153,15 +136,15 @@ void PluginBridgeEditor::timerCallback()
             connectionLabel.setText("", juce::dontSendNotification);
             break;
         case HelperConnection::State::Starting:
-            connectionLabel.setText("⏳ Starting helper...", juce::dontSendNotification);
+            connectionLabel.setText("Starting helper...", juce::dontSendNotification);
             connectionLabel.setColour(juce::Label::textColourId, kYellow);
             break;
         case HelperConnection::State::Connected:
-            connectionLabel.setText("⚡ Helper connected", juce::dontSendNotification);
+            connectionLabel.setText("Helper connected", juce::dontSendNotification);
             connectionLabel.setColour(juce::Label::textColourId, kGreen);
             break;
         case HelperConnection::State::Crashed:
-            connectionLabel.setText("💥 Plugin crashed — click Reload", juce::dontSendNotification);
+            connectionLabel.setText("Plugin crashed - click Reload", juce::dontSendNotification);
             connectionLabel.setColour(juce::Label::textColourId, kRed);
             reloadButton.setVisible(true);
             guiButton.setVisible(false);