add a comment on where the dynamic tool capability is triggered

Author: ioanarm

cmd/mcp-grafana/main.go

@@ -109,10 +109,8 @@ func (dt *disabledTools) addTools(s *server.MCPServer) {
 func (dt *disabledTools) addToolsDynamically(s *server.MCPServer) *mcpgrafana.DynamicToolManager {
 	dtm := mcpgrafana.NewDynamicToolManager(s)
 
-	// Parse enabled tools list
 	enabledTools := strings.Split(dt.enabledTools, ",")
 
-	// Helper function to check if a tool is enabled
 	isEnabled := func(toolName string) bool {
 		// If enabledTools is empty string, no tools should be available
 		if dt.enabledTools == "" {
@@ -252,7 +250,7 @@ Available Capabilities:
 		instructions += "\nNote that some of these capabilities may be disabled. Do not try to use features that are not available via tools."
 	}
 
-	// Create server with tool capabilities enabled for dynamic tool discovery
+	// Create server with tool capabilities based on whether dynamic tools are enabled
 	var s *server.MCPServer
 	if dt.dynamicTools {
 		s = server.NewMCPServer("mcp-grafana", mcpgrafana.Version(),

dynamic_tools.go

@@ -62,7 +62,10 @@ func (dtm *DynamicToolManager) EnableToolset(ctx context.Context, name string) e
 	// when AddTool is called (via the Register method), so we don't need to manually
 	// send notifications here. This happens because WithToolCapabilities(true) was set
 	// during server initialization.
+<<<<<<< HEAD
 
+=======
+>>>>>>> 4df4345 (add a comment on where the dynamic tool capability is triggered)
 	if toolset.AddFunc != nil {
 		toolset.AddFunc(dtm.server)
 	}
@@ -166,6 +169,7 @@ func (dtm *DynamicToolManager) getToolsetInfo(name string) *Toolset {
 	defer dtm.mu.RUnlock()
 	return dtm.toolsets[name]
 }
+<<<<<<< HEAD
 
 // sendToolListChangedNotification sends a proper MCP ToolListChangedNotification
 func (dtm *DynamicToolManager) sendToolListChangedNotification(ctx context.Context) {
@@ -186,3 +190,5 @@ func (dtm *DynamicToolManager) sendToolListChangedNotification(ctx context.Conte
 
 	slog.Info("Sent tool list changed notification to clients")
 }
+=======
+>>>>>>> 4df4345 (add a comment on where the dynamic tool capability is triggered)