Commit7404ba5

committed

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

Fixes `SyntaxError: parameter without a default follows parameter with a default` when using github-mcp-server with the Python MCP SDK

1 parentc17ebfe commit7404ba5Copy full SHA for 7404ba5

File tree

-3

lines changed

-3

lines changed

Lines changed: 3 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -90,14 +90,14 @@ func ListCodeScanningAlerts(getClient GetClientFn, t translations.TranslationHel`
`90`	`90`	`mcp.Required(),`
`91`	`91`	`mcp.Description("The name of the repository."),`
`92`	`92`	`),`
`93`		`-mcp.WithString("ref",`
`94`		`-mcp.Description("The Git reference for the results you want to list."),`
`95`		`-),`
`96`	`93`	`mcp.WithString("state",`
`97`	`94`	`mcp.Description("Filter code scanning alerts by state. Defaults to open"),`
`98`	`95`	`mcp.DefaultString("open"),`
`99`	`96`	`mcp.Enum("open","closed","dismissed","fixed"),`
`100`	`97`	`),`
	`98`	`+mcp.WithString("ref",`
	`99`	`+mcp.Description("The Git reference for the results you want to list."),`
	`100`	`+),`
`101`	`101`	`mcp.WithString("severity",`
`102`	`102`	`mcp.Description("Filter code scanning alerts by severity"),`
`103`	`103`	`mcp.Enum("critical","high","medium","low","warning","note","error"),`

Comments

(0)