feat:native server config fields can also be edited from UI

Author: ravitemer

lua/mcphub/ui/views/main.lua

@@ -137,17 +137,19 @@ function MainView:handle_edit()
     local cursor = vim.api.nvim_win_get_cursor(0)
     local line = cursor[1]
 
-    local type, context = self:get_line_info(line)
-    if not type or not context then
+    local line_type, context = self:get_line_info(line)
+    if not line_type or not context then
         return
     end
     local server_name = context.name
-    if type == "server" then
+    if line_type == "server" then
         local is_native = native.is_native_server(server_name)
+        local config
         if is_native then
-            return vim.notify("Native servers cannot be edited")
+            config = State.native_servers_config[server_name] or {}
+        else
+            config = State.servers_config[server_name] or {}
         end
-        local config = State.servers_config[server_name] or {}
         local text = utils.pretty_json(vim.json.encode({
             [server_name] = config,
         }) or "")
@@ -186,15 +188,27 @@ function MainView:handle_edit()
                 local new_name, new_config = next(result)
                 ---@cast new_name string
                 ---@cast new_config table
-                local valid = validation.validate_server_config(new_name, new_config)
-                if not valid.ok then
-                    vim.notify(valid.error.message, vim.log.levels.ERROR)
-                    return false
+
+                -- For native servers, we only need to validate basic structure
+                -- since they don't require command/url fields
+                if is_native then
+                    if type(new_config) ~= "table" then
+                        vim.notify("Config must be a table", vim.log.levels.ERROR)
+                        return false
+                    end
+                    -- Native servers only need basic config validation
+                    return true
+                else
+                    local valid = validation.validate_server_config(new_name, new_config)
+                    if not valid.ok then
+                        vim.notify(valid.error.message, vim.log.levels.ERROR)
+                        return false
+                    end
                 end
                 return true
             end,
         })
-    elseif (type == "customInstructions") and context then
+    elseif (line_type == "customInstructions") and context then
         self:handle_custom_instructions(context)
     end
 end
@@ -212,7 +226,7 @@ function MainView:handle_delete()
         local server_name = context.name
         local is_native = native.is_native_server(server_name)
         if is_native then
-            return vim.notify("Native servers cannot be deleted")
+            return vim.notify("Native servers cannot be deleted, only their configuration can be edited")
         end
 
         -- Using vim.ui.select instead of vim.fn.confirm

lua/mcphub/utils/renderer.lua

@@ -171,12 +171,12 @@ function M.render_server_capabilities(server, lines, current_line, config_source
     current_line = current_line + 1
 
     -- Prepare hover hint based on server status
-    local hint = is_native and "[<t> Toggle]" or "[<t> Toggle, <e> Edit, <d> Delete]"
+    local hint = is_native and "[<t> Toggle, <e> Edit]" or "[<t> Toggle, <e> Edit, <d> Delete]"
     local needs_authorization = server.status == "unauthorized"
-    local enabled_hint = is_native and "[<l> Expand, <t> Toggle]"
+    local enabled_hint = is_native and "[<l> Expand, <t> Toggle, <e> Edit]"
         or needs_authorization and "[<l> Authorize, <t> Toggle, <e> Edit, <d> Delete]"
         or "[<l> Expand, <t> Toggle, <e> Edit, <d> Delete]"
-    local expanded_hint = is_native and "[<h> Collapse, <t> Toggle]"
+    local expanded_hint = is_native and "[<h> Collapse, <t> Toggle, <e> Edit]"
         or "[<h> Collapse, <t> Toggle, <e> Edit, <d> Delete]"
     if server.status ~= "disabled" and server.status ~= "disconnected" then
         hint = view.expanded_server == server.name and expanded_hint or enabled_hint