Apply suggestions from code review

Co-authored-by: Nader Ziada <[email protected]>
Signed-off-by: Calum Murray <[email protected]>

Author: Cali0707

pkg/mcp/resources_test.go

@@ -630,13 +630,13 @@ func (s *ResourcesSuite) TestResourcesScale() {
 	s.Run("resources_scale with missing apiVersion returns error", func() {
 		toolResult, _ := s.CallTool("resources_scale", map[string]interface{}{})
 		s.Truef(toolResult.IsError, "call tool should fail")
-		s.Equalf("failed get/update resource scale, missing argument apiVersion", toolResult.Content[0].(mcp.TextContent).Text,
+		s.Equalf("failed to get/update resource scale, missing argument apiVersion", toolResult.Content[0].(mcp.TextContent).Text,
 			"invalid error message, got %v", toolResult.Content[0].(mcp.TextContent).Text)
 	})
 	s.Run("resources_scale with missing kind returns error", func() {
 		toolResult, _ := s.CallTool("resources_scale", map[string]interface{}{"apiVersion": "apps/v1"})
 		s.Truef(toolResult.IsError, "call tool should fail")
-		s.Equalf("failed get/update resource scale, missing argument kind", toolResult.Content[0].(mcp.TextContent).Text,
+		s.Equalf("failed to get/update resource scale, missing argument kind", toolResult.Content[0].(mcp.TextContent).Text,
 			"invalid error message, got %v", toolResult.Content[0].(mcp.TextContent).Text)
 	})
 	s.Run("resources_scale with missing name returns error", func() {

pkg/toolsets/core/resources.go

@@ -306,7 +306,7 @@ func resourcesScale(params api.ToolHandlerParams) (*api.ToolCallResult, error) {
 
 	gvk, err := parseGroupVersionKind(params.GetArguments())
 	if err != nil {
-		return api.NewToolCallResult("", fmt.Errorf("failed get/update resource scale, %w", err)), nil
+		return api.NewToolCallResult("", fmt.Errorf("failed to get/update resource scale, %w", err)), nil
 	}
 
 	name := params.GetArguments()["name"]
@@ -373,7 +373,7 @@ func getResourceClient(params api.ToolHandlerParams, gvk *schema.GroupVersionKin
 
 	isNamespaced, err := params.IsNamespaced(gvk)
 	if err != nil {
-		return nil, fmt.Errorf("failed to determine if resource is namespaceable")
+		return nil, fmt.Errorf("failed to determine if resource is namespaced")
 	}
 
 	if isNamespaced {