fix: move defaulted 'state' param after non-default params in list_code_scanning_alerts

ph3t · ph3t · commit 7404ba57f6d1 · 2025-06-06T15:51:10.000+02:00
Fixes `SyntaxError: parameter without a default follows parameter with a default` when using github-mcp-server with the Python MCP SDK
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"),
