rebase

Author: ioanarm

README.md

@@ -285,12 +285,12 @@ For even more efficient context window usage, you can enable **dynamic toolset d
 
 Protocol Support:
 - ✅ **stdio protocol** - Fully supported
-- ❌ **SSE (Server-Sent Events)** - Not yet supported
-- ❌ **Streamable HTTP** - Not yet supported
+- ✅ **SSE (Server-Sent Events)** - Fully supported
+- ✅ **Streamable HTTP** - Fully supported
 
 Client Support:
-- ✅ **Cursor** - Fully supported (supports notifications via stdio)
-- ✅ **VS Code** - Fully supported (supports notifications via stdio)
+- ✅ **Cursor** - Fully supported (supports notifications via stdio, SSE, and streamable-http)
+- ✅ **VS Code** - Fully supported (supports notifications via stdio, SSE, and streamable-http)
 - ❌ **Claude Desktop** - Not yet supported (no notification support, but open issues exist)
 - ❌ **Claude Code** - Not yet supported (no notification support, but open issues exist)
 

dynamic_tools.go

@@ -62,10 +62,6 @@ 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)
 	}
@@ -169,26 +165,3 @@ 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) {
-	// Create notification parameters as map[string]any
-	params := map[string]any{
-		"_meta": map[string]any{
-			"message": "Tool list has been updated. New tools are now available.",
-		},
-	}
-
-	// Send notification to all connected clients
-	dtm.server.SendNotificationToAllClients("tools/list_changed", params)
-
-	// Also try to send to the current context if available
-	if err := dtm.server.SendNotificationToClient(ctx, "tools/list_changed", params); err != nil {
-		slog.Debug("Failed to send notification to client", "error", err)
-	}
-
-	slog.Info("Sent tool list changed notification to clients")
-}
-=======
->>>>>>> 4df4345 (add a comment on where the dynamic tool capability is triggered)