From c4a8941938858808382a7b6be75c1710a40bfad6 Mon Sep 17 00:00:00 2001
From: Abdelrahman Shawki Hassan <shawki.hassan@snyk.io>
Date: Mon, 22 Jun 2026 12:37:12 +0200
Subject: [PATCH] fix: promote breakability tool to default profile

---
 internal/mcp/profiles_test.go | 2 +-
 internal/mcp/snyk_tools.json  | 2 +-
 internal/mcp/tools_test.go    | 8 ++++----
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/internal/mcp/profiles_test.go b/internal/mcp/profiles_test.go
index 6020a1c..aea9c48 100644
--- a/internal/mcp/profiles_test.go
+++ b/internal/mcp/profiles_test.go
@@ -178,10 +178,10 @@ func TestIsToolInProfile(t *testing.T) {
 		{"snyk_sbom_scan", false, true, true},
 		{"snyk_aibom", false, true, true},
 		{"snyk_package_health_check", false, true, true},
+		{"snyk_breakability_check", false, true, true},
 
 		// Tools in experimental only
 		{"snyk_secret_scan", false, false, true},
-		{"snyk_breakability_check", false, false, true},
 	}
 
 	// Load actual tools from JSON
diff --git a/internal/mcp/snyk_tools.json b/internal/mcp/snyk_tools.json
index 1a82275..dd6cb39 100644
--- a/internal/mcp/snyk_tools.json
+++ b/internal/mcp/snyk_tools.json
@@ -792,7 +792,7 @@
       "description": "Runs a breaking change assessment for a package version upgrade.",
       "command": [],
       "standardParams": [],
-      "profiles": ["experimental"],
+      "profiles": ["full","experimental"],
       "ignoreTrust": true,
       "annotations": {
         "readOnlyHint": true,
diff --git a/internal/mcp/tools_test.go b/internal/mcp/tools_test.go
index 5efe4d7..2963dc9 100644
--- a/internal/mcp/tools_test.go
+++ b/internal/mcp/tools_test.go
@@ -2093,10 +2093,10 @@ func TestAddSnykToolsWithProfile(t *testing.T) {
 				"snyk_sbom_scan",
 				"snyk_aibom",
 				"snyk_package_health_check",
+				"snyk_breakability_check",
 			},
 			unexpectedTools: []string{
 				"snyk_secret_scan",
-				"snyk_breakability_check",
 			},
 		},
 		{
@@ -2203,7 +2203,7 @@ func TestToolProfileAssignmentsInJson(t *testing.T) {
 				require.True(t, IsToolInProfile(tool, ProfileExperimental),
 					"Tool %s should be in experimental profile", tool.Name)
 
-			case "snyk_container_scan", "snyk_iac_scan", "snyk_sbom_scan", "snyk_aibom", "snyk_package_health_check":
+			case "snyk_container_scan", "snyk_iac_scan", "snyk_sbom_scan", "snyk_aibom", "snyk_package_health_check", "snyk_breakability_check":
 				// These should be in full but not lite
 				require.False(t, IsToolInProfile(tool, ProfileLite),
 					"Tool %s should NOT be in lite profile", tool.Name)
@@ -2212,7 +2212,7 @@ func TestToolProfileAssignmentsInJson(t *testing.T) {
 				require.True(t, IsToolInProfile(tool, ProfileExperimental),
 					"Tool %s should be in experimental profile", tool.Name)
 
-			case "snyk_secret_scan", "snyk_breakability_check":
+			case "snyk_secret_scan":
 				// These should be experimental only
 				require.False(t, IsToolInProfile(tool, ProfileLite),
 					"Tool %s should NOT be in lite profile", tool.Name)
@@ -2505,7 +2505,7 @@ func TestSnykBreakabilityHandler_GracefulFailure(t *testing.T) {
 
 		respBody := map[string]interface{}{
 			"jsonapi": map[string]interface{}{"version": "1.0"},
-			"errors": []interface{}{map[string]interface{}{"status": "500", "detail": "boom"}},
+			"errors":  []interface{}{map[string]interface{}{"status": "500", "detail": "boom"}},
 		}
 		apiURL := startBreakabilityMockServer(t, orgID, http.StatusInternalServerError, respBody, nil)
 		configureBreakabilityFixture(t, fixture, apiURL, orgID)