Add Serena as built-in MCP tool for listing and inspection (#7055)

Author: Copilot

.github/workflows/agent-performance-analyzer.lock.yml

@@ -206,7 +206,7 @@ The command will:
 }
 
 // commonMCPServerNames contains commonly used MCP server names for shell completion
-var commonMCPServerNames = []string{"github", "playwright", "tavily", "safe-outputs"}
+var commonMCPServerNames = []string{"github", "playwright", "serena", "tavily", "safe-outputs"}
 
 // completeMCPListToolsArgs provides completion for mcp list-tools command arguments
 func completeMCPListToolsArgs(cmd *cobra.Command, args []string, toComplete string) ([]string, cobra.ShellCompDirective) {

.github/workflows/campaign-manager.lock.yml

@@ -199,8 +199,8 @@ func ExtractMCPConfigurations(frontmatter map[string]any, serverFilter string) (
 		if hasTools {
 			if tools, ok := toolsSection.(map[string]any); ok {
 				for toolName, toolValue := range tools {
-					// Only handle built-in MCP tools (github and playwright)
-					if toolName == "github" || toolName == "playwright" {
+					// Only handle built-in MCP tools (github, playwright, and serena)
+					if toolName == "github" || toolName == "playwright" || toolName == "serena" {
 						config, err := processBuiltinMCPTool(toolName, toolValue, serverFilter)
 						if err != nil {
 							return nil, err
@@ -227,8 +227,8 @@ func ExtractMCPConfigurations(frontmatter map[string]any, serverFilter string) (
 	if hasTools {
 		if tools, ok := toolsSection.(map[string]any); ok {
 			for toolName, toolValue := range tools {
-				// Only handle built-in MCP tools (github and playwright)
-				if toolName == "github" || toolName == "playwright" {
+				// Only handle built-in MCP tools (github, playwright, and serena)
+				if toolName == "github" || toolName == "playwright" || toolName == "serena" {
 					config, err := processBuiltinMCPTool(toolName, toolValue, serverFilter)
 					if err != nil {
 						return nil, err
@@ -268,7 +268,7 @@ func ExtractMCPConfigurations(frontmatter map[string]any, serverFilter string) (
 	return configs, nil
 }
 
-// processBuiltinMCPTool handles built-in MCP tools (github and playwright)
+// processBuiltinMCPTool handles built-in MCP tools (github, playwright, and serena)
 func processBuiltinMCPTool(toolName string, toolValue any, serverFilter string) (*MCPServerConfig, error) {
 	// Apply server filter if specified
 	if serverFilter != "" && !strings.Contains(strings.ToLower(toolName), strings.ToLower(serverFilter)) {
@@ -478,6 +478,57 @@ func processBuiltinMCPTool(toolName string, toolValue any, serverFilter string)
 			config.Env["PLAYWRIGHT_BLOCK_ALL_DOMAINS"] = "true"
 		}
 
+		return &config, nil
+	} else if toolName == "serena" {
+		// Handle Serena MCP server - uses uvx to install and run from GitHub
+		config := MCPServerConfig{
+			Name:    "serena",
+			Type:    "stdio",
+			Command: "uvx",
+			Args: []string{
+				"--from", "git+https://github.com/oraios/serena",
+				"serena", "start-mcp-server",
+				"--context", "codex",
+				"--project", "${GITHUB_WORKSPACE}",
+			},
+			Env: make(map[string]string),
+		}
+
+		// Check for custom Serena configuration
+		if toolConfig, ok := toolValue.(map[string]any); ok {
+			// Handle custom args - these would be appended to the default args
+			if argsValue, exists := toolConfig["args"]; exists {
+				// Handle []any format
+				if argsSlice, ok := argsValue.([]any); ok {
+					for _, arg := range argsSlice {
+						if argStr, ok := arg.(string); ok {
+							config.Args = append(config.Args, argStr)
+						}
+					}
+				}
+				// Handle []string format
+				if argsSlice, ok := argsValue.([]string); ok {
+					config.Args = append(config.Args, argsSlice...)
+				}
+			}
+
+			// Handle allowed tools configuration
+			if allowed, hasAllowed := toolConfig["allowed"]; hasAllowed {
+				if allowedSlice, ok := allowed.([]any); ok {
+					for _, item := range allowedSlice {
+						if str, ok := item.(string); ok {
+							config.Allowed = append(config.Allowed, str)
+						}
+					}
+				}
+			}
+		}
+
+		// If no specific allowed tools are configured, allow all tools (*)
+		if len(config.Allowed) == 0 {
+			config.Allowed = []string{"*"}
+		}
+
 		return &config, nil
 	}