apply fixes from PR comments

imessage/bluebubbles/api.go

@@ -316,11 +316,11 @@ func (bb *blueBubbles) GetChatsWithMessagesAfter(minDate time.Time) (resp []imes
 	request := ChatQueryRequest{
 		Limit:  limit,
 		Offset: offset,
-		With: []string{
-			"lastMessage",
-			"sms",
+		With: []ChatQueryWith{
+			ChatQueryWithLastMessage,
+			ChatQueryWithSMS,
 		},
-		Sort: "lastmessage",
+		Sort: QuerySortLastMessage,
 	}
 	var response ChatQueryResponse
 
@@ -329,10 +329,10 @@ func (bb *blueBubbles) GetChatsWithMessagesAfter(minDate time.Time) (resp []imes
 		return nil, err
 	}
 
-	for _, chat := range *response.Data {
+	for _, chat := range response.Data {
 		resp = append(resp, imessage.ChatIdentifier{
-			ChatGUID: fmt.Sprintf("%v", chat.GUID),
-			ThreadID: fmt.Sprintf("%v", chat.ChatIdentifier), // TODO Is this the right one to use?
+			ChatGUID: chat.GUID,
+			ThreadID: chat.GroupId,
 		})
 	}
 

imessage/bluebubbles/interface.go

@@ -7,18 +7,31 @@ type PageMetadata struct {
 	Limit  int64 `json:"limit"`
 }
 
+type QuerySort string
+
+const (
+	QuerySortLastMessage QuerySort = "lastmessage"
+)
+
 type ChatQueryRequest struct {
 	// TODO Other Fields
-	Limit  int64    `json:"limit"`
-	Offset int64    `json:"offset"`
-	With   []string `json:"with"`
-	Sort   string   `json:"sort"`
+	Limit  int64           `json:"limit"`
+	Offset int64           `json:"offset"`
+	With   []ChatQueryWith `json:"with"`
+	Sort   QuerySort       `json:"sort"`
 }
 
+type ChatQueryWith string
+
+const (
+	ChatQueryWithSMS         ChatQueryWith = "sms"
+	ChatQueryWithLastMessage ChatQueryWith = "lastMessage"
+)
+
 type ChatQueryResponse struct {
 	Status   int64        `json:"status"`
 	Message  string       `json:"message"`
-	Data     *[]Chat      `json:"data"`
+	Data     []Chat       `json:"data"`
 	Metadata PageMetadata `json:"metadata"`
 }