Bump thriftrw-go to f40a16c86 in root and internal/examples

Bump thriftrw-go to consume the new no-wire (streaming) implementation.  Also
stop using deprecated fields (except in tests) and regenerate all of the
thriftrw relevant files.

encoding/thrift/envelope_test.go

@@ -30,7 +30,7 @@ import (
 	"time"
 
 	"github.com/stretchr/testify/assert"
-	"go.uber.org/thriftrw/protocol"
+	"go.uber.org/thriftrw/protocol/binary"
 	"go.uber.org/thriftrw/wire"
 )
 
@@ -63,7 +63,7 @@ func TestDisableEnveloperEncode(t *testing.T) {
 		generate(&e.SeqID, rand)
 
 		var buffer bytes.Buffer
-		proto := disableEnvelopingProtocol{protocol.Binary, wire.Reply}
+		proto := disableEnvelopingProtocol{binary.Default, wire.Reply}
 		if !assert.NoError(t, proto.EncodeEnveloped(e, &buffer)) {
 			continue
 		}

encoding/thrift/inbound.go

@@ -26,6 +26,7 @@ import (
 	"io"
 
 	"go.uber.org/thriftrw/protocol"
+	"go.uber.org/thriftrw/protocol/envelope"
 	"go.uber.org/thriftrw/wire"
 	encodingapi "go.uber.org/yarpc/api/encoding"
 	"go.uber.org/yarpc/api/transport"
@@ -139,7 +140,7 @@ func decodeRequest(
 	wire.Value,
 	// how to encode the response, with the enveloping
 	// strategy corresponding to the request. It is not used for oneway handlers.
-	protocol.Responder,
+	envelope.Responder,
 	error,
 ) {
 	if err := errors.ExpectEncodings(treq, Encoding); err != nil {
@@ -166,7 +167,7 @@ func decodeEnvelopedRequest(
 	reqEnvelopeType wire.EnvelopeType,
 	proto protocol.Protocol,
 	reader io.ReaderAt,
-) (wire.Value, protocol.Responder, error) {
+) (wire.Value, envelope.Responder, error) {
 	var envelope wire.Envelope
 	envelope, err := proto.DecodeEnveloped(reader)
 	if err != nil {
@@ -177,18 +178,20 @@ func decodeEnvelopedRequest(
 		return wire.Value{}, nil, err
 	}
 	reqValue := envelope.Value
+	//lint:ignore SA1019 explicit use for known enveloping
 	responder := protocol.EnvelopeV1Responder{Name: envelope.Name, SeqID: envelope.SeqID}
 	return reqValue, responder, nil
 }
 
 func decodeUnenvelopedRequest(
 	proto protocol.Protocol,
 	reader io.ReaderAt,
-) (wire.Value, protocol.Responder, error) {
+) (wire.Value, envelope.Responder, error) {
 	reqValue, err := proto.Decode(reader, wire.TStruct)
 	if err != nil {
 		return wire.Value{}, nil, err
 	}
+	//lint:ignore SA1019 explicit use for known enveloping
 	responder := protocol.NoEnvelopeResponder
 	return reqValue, responder, err
 }

encoding/thrift/inbound_test.go

@@ -18,6 +18,7 @@
 // OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 // THE SOFTWARE.
 
+//lint:file-ignore SA1019 'mock' usage
 package thrift
 
 import (