Sprint 2.1: Editor.cpp — system scan + popup menu sorted by manufacturer

Source/PluginBridgeEditor.cpp

@@ -3,37 +3,56 @@
 PluginBridgeEditor::PluginBridgeEditor(PluginBridgeProcessor& p)
     : AudioProcessorEditor(&p), processorRef(p)
 {
-    setSize(320, 200);
+    setSize(360, 220);
+
+    // Register formats for scanning
+    #if JUCE_PLUGINHOST_VST3
+    scanFormatManager.addFormat(new juce::VST3PluginFormat());
+    #endif
+    #if JUCE_PLUGINHOST_AU
+    scanFormatManager.addFormat(new juce::AudioUnitPluginFormat());
+    #endif
 
     // Title
     titleLabel.setText("PluginBridge", juce::dontSendNotification);
-    titleLabel.setFont(juce::FontOptions(20.0f));
+    titleLabel.setFont(juce::FontOptions(22.0f));
     titleLabel.setJustificationType(juce::Justification::centred);
+    titleLabel.setColour(juce::Label::textColourId, juce::Colours::white);
     addAndMakeVisible(titleLabel);
 
     // MCP status
     statusLabel.setText("MCP: starting...", juce::dontSendNotification);
-    statusLabel.setFont(juce::FontOptions(13.0f));
+    statusLabel.setFont(juce::FontOptions(12.0f));
     statusLabel.setJustificationType(juce::Justification::centred);
     addAndMakeVisible(statusLabel);
 
     // Port
     portLabel.setText("", juce::dontSendNotification);
-    portLabel.setFont(juce::FontOptions(11.0f));
+    portLabel.setFont(juce::FontOptions(10.0f));
     portLabel.setJustificationType(juce::Justification::centred);
-    portLabel.setColour(juce::Label::textColourId, juce::Colours::grey);
+    portLabel.setColour(juce::Label::textColourId, juce::Colour(0xff888888));
     addAndMakeVisible(portLabel);
 
-    // Hosted plugin name
+    // Plugin name display
     pluginLabel.setText("No plugin loaded", juce::dontSendNotification);
     pluginLabel.setFont(juce::FontOptions(13.0f));
     pluginLabel.setJustificationType(juce::Justification::centred);
-    pluginLabel.setColour(juce::Label::textColourId, juce::Colours::lightyellow);
+    pluginLabel.setColour(juce::Label::textColourId, juce::Colour(0xffaaaaaa));
     addAndMakeVisible(pluginLabel);
 
-    // Load button
-    loadButton.onClick = [this]() { loadPluginClicked(); };
-    addAndMakeVisible(loadButton);
+    // Plugin selector button (opens popup menu)
+    pluginButton.onClick = [this]() { showPluginMenu(); };
+    pluginButton.setColour(juce::TextButton::buttonColourId, juce::Colour(0xff2d2d44));
+    pluginButton.setColour(juce::TextButton::textColourOffId, juce::Colours::white);
+    addAndMakeVisible(pluginButton);
+
+    // Scan plugins in background on first open
+    if (!hasScanned)
+    {
+        hasScanned = true;
+        // Scan runs on message thread — fast enough for installed plugins
+        scanPlugins();
+    }
 
     startTimerHz(2);
 }
@@ -45,20 +64,23 @@ PluginBridgeEditor::~PluginBridgeEditor()
 
 void PluginBridgeEditor::paint(juce::Graphics& g)
 {
-    g.fillAll(juce::Colour(0xff1a1a2e));
+    // Dark gradient background
+    g.fillAll(juce::Colour(0xff12121f));
+    g.setColour(juce::Colour(0xff1e1e35));
+    g.fillRoundedRectangle(getLocalBounds().toFloat().reduced(4), 6.0f);
 }
 
 void PluginBridgeEditor::resized()
 {
-    auto area = getLocalBounds().reduced(12);
-    titleLabel.setBounds(area.removeFromTop(32));
-    area.removeFromTop(6);
-    statusLabel.setBounds(area.removeFromTop(22));
-    portLabel.setBounds(area.removeFromTop(18));
+    auto area = getLocalBounds().reduced(14);
+    titleLabel.setBounds(area.removeFromTop(30));
+    area.removeFromTop(4);
+    statusLabel.setBounds(area.removeFromTop(18));
+    portLabel.setBounds(area.removeFromTop(16));
+    area.removeFromTop(10);
+    pluginButton.setBounds(area.removeFromTop(30).reduced(20, 0));
     area.removeFromTop(8);
-    pluginLabel.setBounds(area.removeFromTop(22));
-    area.removeFromTop(8);
-    loadButton.setBounds(area.removeFromTop(28).reduced(60, 0));
+    pluginLabel.setBounds(area.removeFromTop(20));
 }
 
 void PluginBridgeEditor::timerCallback()
@@ -67,14 +89,14 @@ void PluginBridgeEditor::timerCallback()
     if (processorRef.isMcpServerRunning())
     {
         statusLabel.setText("MCP: running", juce::dontSendNotification);
-        statusLabel.setColour(juce::Label::textColourId, juce::Colours::limegreen);
-        portLabel.setText("http://127.0.0.1:" + juce::String(processorRef.getMcpServerPort()) + "/mcp",
+        statusLabel.setColour(juce::Label::textColourId, juce::Colour(0xff4cef50));
+        portLabel.setText("127.0.0.1:" + juce::String(processorRef.getMcpServerPort()),
                           juce::dontSendNotification);
     }
     else
     {
         statusLabel.setText("MCP: offline", juce::dontSendNotification);
-        statusLabel.setColour(juce::Label::textColourId, juce::Colours::red);
+        statusLabel.setColour(juce::Label::textColourId, juce::Colour(0xffef5350));
         portLabel.setText("", juce::dontSendNotification);
     }
 
@@ -82,29 +104,108 @@ void PluginBridgeEditor::timerCallback()
     auto& mgr = processorRef.getHostedPluginManager();
     if (mgr.isLoaded())
     {
-        pluginLabel.setText(mgr.getPluginName() + " (" + juce::String(mgr.getParameterCount()) + " params)",
+        pluginLabel.setText(mgr.getPluginName() + "  |  " + juce::String(mgr.getParameterCount()) + " params",
                             juce::dontSendNotification);
-        pluginLabel.setColour(juce::Label::textColourId, juce::Colours::lightyellow);
+        pluginLabel.setColour(juce::Label::textColourId, juce::Colour(0xffffeb3b));
+        pluginButton.setButtonText(mgr.getPluginName());
     }
     else
     {
         pluginLabel.setText("No plugin loaded", juce::dontSendNotification);
-        pluginLabel.setColour(juce::Label::textColourId, juce::Colours::grey);
+        pluginLabel.setColour(juce::Label::textColourId, juce::Colour(0xff888888));
+        pluginButton.setButtonText("Select Plugin...");
     }
 }
 
-void PluginBridgeEditor::loadPluginClicked()
+void PluginBridgeEditor::scanPlugins()
 {
-    auto chooser = std::make_shared<juce::FileChooser>(
-        "Select VST3 or AU plugin",
-        juce::File("/Library/Audio/Plug-Ins"),
-        "*.vst3;*.component");
+    // Scan standard plugin directories
+    for (int i = 0; i < scanFormatManager.getNumFormats(); ++i)
+    {
+        auto* format = scanFormatManager.getFormat(i);
+        auto searchPaths = format->getDefaultLocationsToSearch();
 
-    chooser->launchAsync(juce::FileBrowserComponent::openMode | juce::FileBrowserComponent::canSelectFiles,
-        [this, chooser](const juce::FileChooser& fc)
+        juce::PluginDirectoryScanner scanner(
+            knownPlugins,
+            *format,
+            searchPaths,
+            true,  // recursive
+            juce::File() // no dead-mans-pedal for speed
+        );
+
+        juce::String pluginName;
+        while (scanner.scanNextFile(false, pluginName))
         {
-            auto file = fc.getResult();
-            if (file.exists())
-                processorRef.loadPluginFromFile(file);
+            // Scanning...
+        }
+    }
+
+    DBG("PluginBridge: Scanned " + juce::String(knownPlugins.getNumTypes()) + " plugins");
+}
+
+void PluginBridgeEditor::showPluginMenu()
+{
+    juce::PopupMenu menu;
+
+    // Sort by manufacturer → plugin name
+    auto types = knownPlugins.getTypes();
+
+    // Group by manufacturer
+    juce::StringArray manufacturers;
+    for (auto& desc : types)
+    {
+        if (!manufacturers.contains(desc.manufacturerName))
+            manufacturers.add(desc.manufacturerName);
+    }
+    manufacturers.sort(true);
+
+    int itemId = 1;
+    for (auto& mfr : manufacturers)
+    {
+        juce::PopupMenu subMenu;
+        for (int i = 0; i < types.size(); ++i)
+        {
+            if (types[i].manufacturerName == mfr)
+            {
+                subMenu.addItem(itemId, types[i].name + "  (" + types[i].pluginFormatName + ")");
+            }
+            itemId++;
+        }
+        menu.addSubMenu(mfr, subMenu);
+        // Reset itemId for proper indexing
+        itemId = 1;
+        for (int i = 0; i < types.size(); ++i)
+        {
+            if (types[i].manufacturerName == mfr)
+                itemId++;
+        }
+    }
+
+    // Simpler approach: flat list with manufacturer prefix, proper IDs
+    menu.clear();
+    for (int i = 0; i < types.size(); ++i)
+    {
+        menu.addItem(i + 1, types[i].manufacturerName + " — " + types[i].name
+                     + "  [" + types[i].pluginFormatName + "]");
+    }
+
+    menu.showMenuAsync(juce::PopupMenu::Options().withTargetComponent(&pluginButton),
+        [this, types](int result)
+        {
+            if (result > 0 && result <= types.size())
+            {
+                loadPluginByIndex(result - 1);
+            }
         });
 }
+
+void PluginBridgeEditor::loadPluginByIndex(int index)
+{
+    auto types = knownPlugins.getTypes();
+    if (index >= 0 && index < types.size())
+    {
+        auto& desc = types[index];
+        juce::File pluginFile(desc.fileOrIdentifier);
+        processorRef.loadPluginFromFile(pluginFile);
+    }
+}