Merge pull request #640 from devoxx/feat-635

Removed old MCP Env dialog

Author: stephanj

src/main/java/com/devoxx/genie/ui/settings/mcp/MCPSettingsComponent.java

@@ -4,16 +4,10 @@
 import com.devoxx.genie.service.mcp.MCPService;
 import com.devoxx.genie.ui.settings.AbstractSettingsComponent;
 import com.devoxx.genie.ui.settings.DevoxxGenieStateService;
-import com.devoxx.genie.ui.settings.mcp.dialog.MCPEnvironmentVariablesDialog;
 import com.devoxx.genie.ui.settings.mcp.dialog.MCPServerDialog;
-import com.devoxx.genie.ui.util.NotificationUtil;
 import com.intellij.icons.AllIcons;
-import com.intellij.openapi.actionSystem.AnActionEvent;
-import com.intellij.openapi.project.Project;
-import com.intellij.openapi.project.ProjectManager;
 import com.intellij.openapi.ui.DialogWrapper;
 import com.intellij.openapi.ui.Messages;
-import com.intellij.ui.AnActionButton;
 import com.intellij.ui.ToolbarDecorator;
 import com.intellij.ui.components.JBScrollPane;
 import com.intellij.ui.table.JBTable;
@@ -179,20 +173,6 @@ private void removeMcpServer() {
         }
     }
 
-    private void editEnvironmentVariables() {
-        int selectedRow = mcpTable.getSelectedRow();
-        if (selectedRow >= 0) {
-            MCPServer selectedServer = tableModel.getMcpServerAt(selectedRow);
-            MCPEnvironmentVariablesDialog dialog = new MCPEnvironmentVariablesDialog(selectedServer);
-            
-            if (dialog.showAndGet()) {
-                selectedServer.setEnv(dialog.getEnvironmentVariables());
-                tableModel.fireTableRowsUpdated(selectedRow, selectedRow);
-                isModified = true;
-            }
-        }
-    }
-
     /**
      * Apply the settings from the UI to the DevoxxGenieStateService
      */

src/main/java/com/devoxx/genie/ui/settings/mcp/dialog/MCPEnvironmentVariablesDialog.java

@@ -437,7 +437,7 @@ public Class<?> getColumnClass(int columnIndex) {
     /**
      * Dialog for adding/editing environment variables
      */
-    private class EnvVarDialog extends DialogWrapper {
+    private static class EnvVarDialog extends DialogWrapper {
         private final JTextField keyField = new JTextField();
         private final JTextField valueField = new JTextField();