remove name prefix from ConsumerMessage / ConsumerMessageBatch

_examples/queues/main.go

@@ -105,7 +105,7 @@ func produceBytes(q *queues.Producer, req *http.Request) error {
 	return nil
 }
 
-func consumeBatch(batch *queues.ConsumerMessageBatch) error {
+func consumeBatch(batch *queues.MessageBatch) error {
 	for _, msg := range batch.Messages {
 		log.Printf("Received message: %v\n", msg.Body.Get("name").String())
 	}

cloudflare/queues/consumer.go

@@ -12,7 +12,7 @@ import (
 // A returned error will cause the batch to be retried (unless the batch or individual messages are acked).
 // NOTE: to do long-running message processing task within the Consumer, use cloudflare.WaitUntil, this will postpone the message
 // acknowledgment until the task is completed witout blocking the queue consumption.
-type Consumer func(batch *ConsumerMessageBatch) error
+type Consumer func(batch *MessageBatch) error
 
 var consumer Consumer
 
@@ -44,7 +44,7 @@ func init() {
 }
 
 func consumeBatch(batch js.Value) error {
-	b, err := newConsumerMessageBatch(batch)
+	b, err := newMessageBatch(batch)
 	if err != nil {
 		return fmt.Errorf("failed to parse message batch: %v", err)
 	}

cloudflare/queues/message.go

@@ -8,9 +8,9 @@ import (
 	"github.com/syumai/workers/internal/jsutil"
 )
 
-// ConsumerMessage represents a message of the batch received by the consumer.
+// Message represents a message of the batch received by the consumer.
 //   - https://developers.cloudflare.com/queues/configuration/javascript-apis/#message
-type ConsumerMessage struct {
+type Message struct {
 	// instance - The underlying instance of the JS message object passed by the cloudflare
 	instance js.Value
 
@@ -24,13 +24,13 @@ type ConsumerMessage struct {
 	Attempts int
 }
 
-func newConsumerMessage(obj js.Value) (*ConsumerMessage, error) {
+func newMessage(obj js.Value) (*Message, error) {
 	timestamp, err := jsutil.DateToTime(obj.Get("timestamp"))
 	if err != nil {
 		return nil, fmt.Errorf("failed to parse message timestamp: %v", err)
 	}
 
-	return &ConsumerMessage{
+	return &Message{
 		instance:  obj,
 		Id:        obj.Get("id").String(),
 		Body:      obj.Get("body"),
@@ -41,13 +41,13 @@ func newConsumerMessage(obj js.Value) (*ConsumerMessage, error) {
 
 // Ack acknowledges the message as successfully delivered despite the result returned from the consuming function.
 //   - https://developers.cloudflare.com/queues/configuration/javascript-apis/#message
-func (m *ConsumerMessage) Ack() {
+func (m *Message) Ack() {
 	m.instance.Call("ack")
 }
 
 // Retry marks the message to be re-delivered.
 // The message will be retried after the optional delay configured with RetryOption.
-func (m *ConsumerMessage) Retry(opts ...RetryOption) {
+func (m *Message) Retry(opts ...RetryOption) {
 	var o *retryOptions
 	if len(opts) > 0 {
 		o = &retryOptions{}
@@ -59,14 +59,14 @@ func (m *ConsumerMessage) Retry(opts ...RetryOption) {
 	m.instance.Call("retry", o.toJS())
 }
 
-func (m *ConsumerMessage) StringBody() (string, error) {
+func (m *Message) StringBody() (string, error) {
 	if m.Body.Type() != js.TypeString {
 		return "", fmt.Errorf("message body is not a string: %v", m.Body)
 	}
 	return m.Body.String(), nil
 }
 
-func (m *ConsumerMessage) BytesBody() ([]byte, error) {
+func (m *Message) BytesBody() ([]byte, error) {
 	if m.Body.Type() != js.TypeObject ||
 		!(m.Body.InstanceOf(jsutil.Uint8ArrayClass) || m.Body.InstanceOf(jsutil.Uint8ClampedArrayClass)) {
 		return nil, fmt.Errorf("message body is not a byte array: %v", m.Body)

cloudflare/queues/message_test.go

@@ -20,9 +20,9 @@ func TestNewConsumerMessage(t *testing.T) {
 		"attempts":  1,
 	}
 
-	got, err := newConsumerMessage(js.ValueOf(m))
+	got, err := newMessage(js.ValueOf(m))
 	if err != nil {
-		t.Fatalf("newConsumerMessage failed: %v", err)
+		t.Fatalf("newMessage failed: %v", err)
 	}
 
 	if body := got.Body.String(); body != "hello" {
@@ -49,7 +49,7 @@ func TestConsumerMessage_Ack(t *testing.T) {
 		ackCalled = true
 		return nil
 	}))
-	m := &ConsumerMessage{
+	m := &Message{
 		instance: jsObj,
 	}
 
@@ -67,7 +67,7 @@ func TestConsumerMessage_Retry(t *testing.T) {
 		retryCalled = true
 		return nil
 	}))
-	m := &ConsumerMessage{
+	m := &Message{
 		instance: jsObj,
 	}
 
@@ -99,7 +99,7 @@ func TestConsumerMessage_RetryWithDelay(t *testing.T) {
 		return nil
 	}))
 
-	m := &ConsumerMessage{
+	m := &Message{
 		instance: jsObj,
 	}
 
@@ -151,7 +151,7 @@ func TestNewConsumerMessage_StringBody(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			m := &ConsumerMessage{
+			m := &Message{
 				Body: tt.body(),
 			}
 
@@ -203,7 +203,7 @@ func TestConsumerMessage_BytesBody(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			m := &ConsumerMessage{
+			m := &Message{
 				Body: tt.body(),
 			}
 

cloudflare/queues/messagebatch.go

@@ -5,11 +5,11 @@ import (
 	"syscall/js"
 )
 
-// ConsumerMessageBatch represents a batch of messages received by the consumer. The size of the batch is determined by the
+// MessageBatch represents a batch of messages received by the consumer. The size of the batch is determined by the
 // worker configuration.
 //   - https://developers.cloudflare.com/queues/configuration/configure-queues/#consumer
 //   - https://developers.cloudflare.com/queues/configuration/javascript-apis/#messagebatch
-type ConsumerMessageBatch struct {
+type MessageBatch struct {
 	// instance - The underlying instance of the JS message object passed by the cloudflare
 	instance js.Value
 
@@ -17,21 +17,21 @@ type ConsumerMessageBatch struct {
 	Queue string
 
 	// Messages - The messages in the batch
-	Messages []*ConsumerMessage
+	Messages []*Message
 }
 
-func newConsumerMessageBatch(obj js.Value) (*ConsumerMessageBatch, error) {
+func newMessageBatch(obj js.Value) (*MessageBatch, error) {
 	msgArr := obj.Get("messages")
-	messages := make([]*ConsumerMessage, msgArr.Length())
+	messages := make([]*Message, msgArr.Length())
 	for i := 0; i < msgArr.Length(); i++ {
-		m, err := newConsumerMessage(msgArr.Index(i))
+		m, err := newMessage(msgArr.Index(i))
 		if err != nil {
 			return nil, fmt.Errorf("failed to parse message %d: %v", i, err)
 		}
 		messages[i] = m
 	}
 
-	return &ConsumerMessageBatch{
+	return &MessageBatch{
 		instance: obj,
 		Queue:    obj.Get("queue").String(),
 		Messages: messages,
@@ -40,14 +40,14 @@ func newConsumerMessageBatch(obj js.Value) (*ConsumerMessageBatch, error) {
 
 // AckAll acknowledges all messages in the batch as successfully delivered despite the result returned from the consuming function.
 //   - https://developers.cloudflare.com/queues/configuration/javascript-apis/#messagebatch
-func (b *ConsumerMessageBatch) AckAll() {
+func (b *MessageBatch) AckAll() {
 	b.instance.Call("ackAll")
 }
 
 // RetryAll marks all messages in the batch to be re-delivered.
 // The messages will be retried after the optional delay configured with RetryOption.
 //   - https://developers.cloudflare.com/queues/configuration/javascript-apis/#messagebatch
-func (b *ConsumerMessageBatch) RetryAll(opts ...RetryOption) {
+func (b *MessageBatch) RetryAll(opts ...RetryOption) {
 	var o *retryOptions
 	if len(opts) > 0 {
 		o = &retryOptions{}

cloudflare/queues/messagebatch_test.go

@@ -24,9 +24,9 @@ func TestNewConsumerMessageBatch(t *testing.T) {
 		},
 	}
 
-	got, err := newConsumerMessageBatch(js.ValueOf(m))
+	got, err := newMessageBatch(js.ValueOf(m))
 	if err != nil {
-		t.Fatalf("newConsumerMessageBatch failed: %v", err)
+		t.Fatalf("newMessageBatch failed: %v", err)
 	}
 
 	if got.Queue != "some-queue" {
@@ -62,7 +62,7 @@ func TestConsumerMessageBatch_AckAll(t *testing.T) {
 		ackAllCalled = true
 		return nil
 	}))
-	b := &ConsumerMessageBatch{
+	b := &MessageBatch{
 		instance: jsObj,
 	}
 
@@ -80,7 +80,7 @@ func TestConsumerMessageBatch_RetryAll(t *testing.T) {
 		retryAllCalled = true
 		return nil
 	}))
-	b := &ConsumerMessageBatch{
+	b := &MessageBatch{
 		instance: jsObj,
 	}
 
@@ -112,7 +112,7 @@ func TestConsumerMessageBatch_RetryAllWithRetryOption(t *testing.T) {
 		return nil
 	}))
 
-	b := &ConsumerMessageBatch{
+	b := &MessageBatch{
 		instance: jsObj,
 	}