mcp/server: fix nil PingParams in server.Ping (#163)

samthanawalla · web-flow · commit 94b88a26e033 · 2025-07-23T15:40:57.000-04:00
Use OrZero in case PingParams is nil to avoid typed nil. Fixes: #162
diff --git a/mcp/server.go b/mcp/server.go
@@ -610,7 +610,7 @@ func (ss *ServerSession) ID() string {
 
 // Ping pings the client.
 func (ss *ServerSession) Ping(ctx context.Context, params *PingParams) error {
-	_, err := handleSend[*emptyResult](ctx, ss, methodPing, params)
+	_, err := handleSend[*emptyResult](ctx, ss, methodPing, orZero[Params](params))
 	return err
 }
 

Original file line number	Diff line number	Diff line change
`@@ -610,7 +610,7 @@ func (ss *ServerSession) ID() string {`
`610`	`610`
`611`	`611`	`// Ping pings the client.`
`612`	`612`	`func (ss ServerSession) Ping(ctx context.Context, params PingParams) error {`
`613`		`- _, err := handleSend[*emptyResult](ctx, ss, methodPing, params)`
	`613`	`+ _, err := handleSend[*emptyResult](ctx, ss, methodPing, orZero[Params](params))`
`614`	`614`	`return err`
`615`	`615`	`}`
`616`	`616`