Commitda9c97b

authored

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 commitda9c97bCopy full SHA for da9c97b

File tree

-2

lines changed

-2

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -115,11 +115,11 @@ func ListCodeScanningAlerts(getClient GetClientFn, t translations.TranslationHel`
`115`	`115`	`iferr!=nil {`
`116`	`116`	`returnmcp.NewToolResultError(err.Error()),nil`
`117`	`117`	`}`
`118`		`-ref,err:=OptionalParam[string](request,"ref")`
	`118`	`+state,err:=OptionalParam[string](request,"state")`
`119`	`119`	`iferr!=nil {`
`120`	`120`	`returnmcp.NewToolResultError(err.Error()),nil`
`121`	`121`	`}`
`122`		`-state,err:=OptionalParam[string](request,"state")`
	`122`	`+ref,err:=OptionalParam[string](request,"ref")`
`123`	`123`	`iferr!=nil {`
`124`	`124`	`returnmcp.NewToolResultError(err.Error()),nil`
`125`	`125`	`}`

Comments

(0)