fix: [IDP-6590]: fix IDP toolset registration and add logging (#244)

* 8923c8 fix: [IDP-6590]:  fix IDP toolset registration and add logging

Author: yogesh-chauhan

pkg/modules/idp.go

@@ -39,15 +39,15 @@ func (m *IDPModule) Name() string {
 // Toolsets returns the names of toolsets provided by this module
 func (m *IDPModule) Toolsets() []string {
 	return []string{
-		"Internal Developer Portal",
+		"idp",
 	}
 }
 
 // RegisterToolsets registers all toolsets in the IDP module
 func (m *IDPModule) RegisterToolsets() error {
 	for _, t := range m.Toolsets() {
 		switch t {
-		case "Internal Developer Portal":
+		case "idp":
 			if err := RegisterInternalDeveloperPortal(m.config, m.tsg); err != nil {
 				return err
 			}

pkg/toolsets/toolsets.go

@@ -2,6 +2,7 @@ package toolsets
 
 import (
 	"fmt"
+	"log/slog"
 
 	"github.com/mark3labs/mcp-go/mcp"
 	"github.com/mark3labs/mcp-go/server"
@@ -131,6 +132,7 @@ func (tg *ToolsetGroup) EnableToolsets(names []string) error {
 		}
 		return nil
 	}
+
 	for _, name := range names {
 		if name == "all" {
 			tg.everythingOn = true
@@ -149,8 +151,8 @@ func (tg *ToolsetGroup) EnableToolsets(names []string) error {
 				return err
 			}
 		}
-		return nil
 	}
+
 	return nil
 }
 
@@ -160,8 +162,18 @@ func (tg *ToolsetGroup) EnableToolset(name string) error {
 	if !exists {
 		return fmt.Errorf("toolset %s does not exist", name)
 	}
-	toolset.Enabled = true
-	tg.Toolsets[name] = toolset
+
+	// Only log if not already enabled
+	if !toolset.Enabled {
+		toolset.Enabled = true
+		tg.Toolsets[name] = toolset
+
+		// Log each tool in this toolset
+		for _, tool := range toolset.GetAvailableTools() {
+			slog.Info("Tool enabled", "toolset", name, "tool", tool.Tool.Name)
+		}
+	}
+
 	return nil
 }