fix: move defaulted 'state' param after non-default params in list_code_scanning_alerts (#488)

ph3t · web-flow · commit c141bf4da4e5 · 2025-06-07T15:33:41.000+02:00
diff --git a/pkg/github/code_scanning.go b/pkg/github/code_scanning.go
@@ -90,14 +90,14 @@ func ListCodeScanningAlerts(getClient GetClientFn, t translations.TranslationHel
 				mcp.Required(),
 				mcp.Description("The name of the repository."),
 			),
-			mcp.WithString("ref",
-				mcp.Description("The Git reference for the results you want to list."),
-			),
 			mcp.WithString("state",
 				mcp.Description("Filter code scanning alerts by state. Defaults to open"),
 				mcp.DefaultString("open"),
 				mcp.Enum("open", "closed", "dismissed", "fixed"),
 			),
+			mcp.WithString("ref",
+				mcp.Description("The Git reference for the results you want to list."),
+			),
 			mcp.WithString("severity",
 				mcp.Description("Filter code scanning alerts by severity"),
 				mcp.Enum("critical", "high", "medium", "low", "warning", "note", "error"),
