go: upgrade to Go 1.26.4 and run go fix ./...

Author: ggilmore

.github/workflows/ci.yml

@@ -14,7 +14,7 @@ jobs:
       fail-fast: false
       matrix:
         go:
-          - 1.16
+          - 1.26.4
     name: Go ${{ matrix.go }}
     runs-on: ubuntu-latest
     steps:

call_opt.go

@@ -13,7 +13,7 @@ func (c callOptionFunc) apply(r *Request) error { return c(r) }
 // Meta returns a call option which attaches the given meta object to
 // the JSON-RPC 2.0 request (this is a Sourcegraph extension to JSON
 // RPC 2.0 for carrying metadata).
-func Meta(meta interface{}) CallOption {
+func Meta(meta any) CallOption {
 	return callOptionFunc(func(r *Request) error {
 		return r.SetMeta(meta)
 	})
@@ -22,7 +22,7 @@ func Meta(meta interface{}) CallOption {
 // ExtraField returns a call option which attaches the given name/value pair to
 // the JSON-RPC 2.0 request. This can be used to add arbitrary extensions to
 // JSON RPC 2.0.
-func ExtraField(name string, value interface{}) CallOption {
+func ExtraField(name string, value any) CallOption {
 	return callOptionFunc(func(r *Request) error {
 		return r.SetExtraField(name, value)
 	})

call_opt_test.go

@@ -9,8 +9,7 @@ import (
 )
 
 func TestPickID(t *testing.T) {
-	ctx, cancel := context.WithCancel(context.Background())
-	defer cancel()
+	ctx := t.Context()
 
 	a, b := inMemoryPeerConns()
 	defer a.Close()
@@ -27,7 +26,7 @@ func TestPickID(t *testing.T) {
 	defer connB.Close()
 
 	const n = 100
-	for i := 0; i < n; i++ {
+	for i := range n {
 		var opts []jsonrpc2.CallOption
 		id := jsonrpc2.ID{Num: uint64(i)}
 
@@ -54,8 +53,7 @@ func TestPickID(t *testing.T) {
 
 func TestStringID(t *testing.T) {
 
-	ctx, cancel := context.WithCancel(context.Background())
-	defer cancel()
+	ctx := t.Context()
 
 	a, b := inMemoryPeerConns()
 	defer a.Close()
@@ -93,8 +91,7 @@ func TestStringID(t *testing.T) {
 
 func TestExtraField(t *testing.T) {
 
-	ctx, cancel := context.WithCancel(context.Background())
-	defer cancel()
+	ctx := t.Context()
 
 	a, b := inMemoryPeerConns()
 	defer a.Close()

conn.go

@@ -86,7 +86,7 @@ func (c *Conn) Close() error {
 // waits for the response. If the response is successful, its result is stored
 // in result (a pointer to a value that can be JSON-unmarshaled into);
 // otherwise, a non-nil error is returned. See DispatchCall for more details.
-func (c *Conn) Call(ctx context.Context, method string, params, result interface{}, opts ...CallOption) error {
+func (c *Conn) Call(ctx context.Context, method string, params, result any, opts ...CallOption) error {
 	call, err := c.DispatchCall(ctx, method, params, opts...)
 	if err != nil {
 		return err
@@ -108,7 +108,7 @@ func (c *Conn) DisconnectNotify() <-chan struct{} {
 // The params member is omitted from the JSON-RPC request if the given params is
 // nil. Use json.RawMessage("null") to send a JSON-RPC request with its params
 // member set to null.
-func (c *Conn) DispatchCall(ctx context.Context, method string, params interface{}, opts ...CallOption) (Waiter, error) {
+func (c *Conn) DispatchCall(ctx context.Context, method string, params any, opts ...CallOption) (Waiter, error) {
 	req := &Request{Method: method}
 	for _, opt := range opts {
 		if opt == nil {
@@ -137,7 +137,7 @@ func (c *Conn) DispatchCall(ctx context.Context, method string, params interface
 // The params member is omitted from the JSON-RPC request if the given params is
 // nil. Use json.RawMessage("null") to send a JSON-RPC request with its params
 // member set to null.
-func (c *Conn) Notify(ctx context.Context, method string, params interface{}, opts ...CallOption) error {
+func (c *Conn) Notify(ctx context.Context, method string, params any, opts ...CallOption) error {
 	req := &Request{Method: method, Notif: true}
 	for _, opt := range opts {
 		if opt == nil {
@@ -157,7 +157,7 @@ func (c *Conn) Notify(ctx context.Context, method string, params interface{}, op
 }
 
 // Reply sends a successful response with a result.
-func (c *Conn) Reply(ctx context.Context, id ID, result interface{}) error {
+func (c *Conn) Reply(ctx context.Context, id ID, result any) error {
 	resp := &Response{ID: id}
 	if err := resp.SetResult(result); err != nil {
 		return err
@@ -346,7 +346,7 @@ type Waiter struct {
 // is successful, its result is stored in result (a pointer to a
 // value that can be JSON-unmarshaled into); otherwise, a non-nil
 // error is returned.
-func (w Waiter) Wait(ctx context.Context, result interface{}) error {
+func (w Waiter) Wait(ctx context.Context, result any) error {
 	select {
 	case <-ctx.Done():
 		return ctx.Err()
@@ -380,7 +380,7 @@ type anyMessage struct {
 }
 
 func (m anyMessage) MarshalJSON() ([]byte, error) {
-	var v interface{}
+	var v any
 	switch {
 	case m.request != nil && m.response == nil:
 		v = m.request
@@ -397,10 +397,10 @@ func (m *anyMessage) UnmarshalJSON(data []byte) error {
 	// The presence of these fields distinguishes between the 2
 	// message types.
 	type msg struct {
-		ID     interface{}              `json:"id"`
+		ID     any                      `json:"id"`
 		Method *string                  `json:"method"`
 		Result anyValueWithExplicitNull `json:"result"`
-		Error  interface{}              `json:"error"`
+		Error  any                      `json:"error"`
 	}
 
 	var isRequest, isResponse bool
@@ -441,7 +441,7 @@ func (m *anyMessage) UnmarshalJSON(data []byte) error {
 		}
 	}
 
-	var v interface{}
+	var v any
 	switch {
 	case isRequest && !isResponse:
 		v = &m.request
@@ -461,7 +461,7 @@ func (m *anyMessage) UnmarshalJSON(data []byte) error {
 // {"result":null} by anyMessage's JSON unmarshaler.
 type anyValueWithExplicitNull struct {
 	null  bool // JSON "null"
-	value interface{}
+	value any
 }
 
 func (v anyValueWithExplicitNull) MarshalJSON() ([]byte, error) {

conn_opt.go

@@ -8,7 +8,7 @@ import (
 // Logger interface implements one method - Printf.
 // You can use the stdlib logger *log.Logger
 type Logger interface {
-	Printf(format string, v ...interface{})
+	Printf(format string, v ...any)
 }
 
 // ConnOpt is the type of function that can be passed to NewConn to

conn_opt_test.go

@@ -12,8 +12,7 @@ import (
 )
 
 func TestSetLogger(t *testing.T) {
-	ctx, cancel := context.WithCancel(context.Background())
-	defer cancel()
+	ctx := t.Context()
 
 	rd, wr := io.Pipe()
 	defer rd.Close()
@@ -67,8 +66,7 @@ func (h *dummyHandler) Handle(ctx context.Context, conn *jsonrpc2.Conn, req *jso
 }
 
 func TestLogMessages(t *testing.T) {
-	ctx, cancel := context.WithCancel(context.Background())
-	defer cancel()
+	ctx := t.Context()
 
 	rd, wr := io.Pipe()
 	defer rd.Close()

conn_test.go

@@ -67,7 +67,7 @@ func TestConn(t *testing.T) {
 }
 
 var paramsTests = []struct {
-	sendParams interface{}
+	sendParams any
 	wantParams *json.RawMessage
 }{
 	{
@@ -220,8 +220,7 @@ func TestConn_Close(t *testing.T) {
 	}}
 	for _, tc := range cases {
 		t.Run(tc.name, func(t *testing.T) {
-			ctx, cancel := context.WithCancel(context.Background())
-			defer cancel()
+			ctx := t.Context()
 
 			connA, connB := net.Pipe()
 			nodeA := jsonrpc2.NewConn(

go.mod

@@ -1,5 +1,5 @@
 module github.com/sourcegraph/jsonrpc2
 
-go 1.12
+go 1.26.4
 
 require github.com/gorilla/websocket v1.4.1

handler_with_error.go

@@ -6,13 +6,13 @@ import (
 
 // HandlerWithError implements Handler by calling the func for each
 // request and handling returned errors and results.
-func HandlerWithError(handleFunc func(context.Context, *Conn, *Request) (result interface{}, err error)) *HandlerWithErrorConfigurer {
+func HandlerWithError(handleFunc func(context.Context, *Conn, *Request) (result any, err error)) *HandlerWithErrorConfigurer {
 	return &HandlerWithErrorConfigurer{handleFunc: handleFunc}
 }
 
 // HandlerWithErrorConfigurer is a handler created by HandlerWithError.
 type HandlerWithErrorConfigurer struct {
-	handleFunc        func(context.Context, *Conn, *Request) (result interface{}, err error)
+	handleFunc        func(context.Context, *Conn, *Request) (result any, err error)
 	suppressErrClosed bool
 }
 

jsonrpc2.go

@@ -16,10 +16,10 @@ import (
 // an API boundary.
 type JSONRPC2 interface {
 	// Call issues a standard request (http://www.jsonrpc.org/specification#request_object).
-	Call(ctx context.Context, method string, params, result interface{}, opt ...CallOption) error
+	Call(ctx context.Context, method string, params, result any, opt ...CallOption) error
 
 	// Notify issues a notification request (http://www.jsonrpc.org/specification#notification).
-	Notify(ctx context.Context, method string, params interface{}, opt ...CallOption) error
+	Notify(ctx context.Context, method string, params any, opt ...CallOption) error
 
 	// Close closes the underlying connection, if it exists.
 	Close() error
@@ -34,7 +34,7 @@ type Error struct {
 
 // SetError sets e.Data to the JSON encoding of v. If JSON
 // marshaling fails, it panics.
-func (e *Error) SetError(v interface{}) {
+func (e *Error) SetError(v any) {
 	b, err := json.Marshal(v)
 	if err != nil {
 		panic("Error.SetData: " + err.Error())