@codingfactory/socialkit-vue 0.7.0 → 0.7.3

package/src/stores/discussion.ts

@@ -241,6 +241,7 @@ export function createDiscussionStoreDefinition(config: DiscussionStoreConfig) {
     const threads = ref<Thread[]>([])
     const currentThread = ref<Thread | null>(null)
     const replies = ref<Reply[]>([])
+    const currentReplySort = ref<'best' | 'top' | 'new' | 'controversial'>('best')
 
     const spacesLoadingState = createLoadingState()
     const threadsLoadingState = createLoadingState()
@@ -322,6 +323,78 @@ export function createDiscussionStoreDefinition(config: DiscussionStoreConfig) {
       }
     }
 
+    function hasQuotedReplyBody(quotedReply: Reply['quoted_reply']): boolean {
+      return typeof quotedReply?.body === 'string' && quotedReply.body.trim().length > 0
+    }
+
+    function hasQuotedReplyAuthorName(quotedReply: Reply['quoted_reply']): boolean {
+      return typeof quotedReply?.author?.name === 'string' && quotedReply.author.name.trim().length > 0
+    }
+
+    function buildQuotedReplyFromReply(sourceReply: Reply | null | undefined): Reply['quoted_reply'] {
+      if (!sourceReply?.id || !sourceReply.author_id || typeof sourceReply.body !== 'string' || sourceReply.body.trim().length === 0) {
+        return null
+      }
+
+      return {
+        id: sourceReply.id,
+        author_id: sourceReply.author_id,
+        author: sourceReply.author
+          ? {
+              id: sourceReply.author.id,
+              name: sourceReply.author.name,
+              ...(sourceReply.author.handle ? { handle: sourceReply.author.handle } : {}),
+            }
+          : null,
+        body: sourceReply.body,
+      }
+    }
+
+    function resolveQuotedReply(reply: Reply, existingReply?: Reply | null): Reply['quoted_reply'] {
+      const quotedReplyId = typeof reply.quoted_reply_id === 'string'
+        ? reply.quoted_reply_id.trim()
+        : ''
+
+      if (quotedReplyId.length === 0) {
+        return null
+      }
+
+      const incomingQuotedReply = reply.quoted_reply ?? null
+      if (hasQuotedReplyBody(incomingQuotedReply) && hasQuotedReplyAuthorName(incomingQuotedReply)) {
+        return incomingQuotedReply
+      }
+
+      const referencedReply = replies.value.find((candidate) => candidate.id === quotedReplyId)
+      const hydratedQuotedReply = buildQuotedReplyFromReply(referencedReply)
+      if (hydratedQuotedReply) {
+        return hydratedQuotedReply
+      }
+
+      const existingQuotedReply = existingReply?.quoted_reply ?? null
+      if (hasQuotedReplyBody(existingQuotedReply) && hasQuotedReplyAuthorName(existingQuotedReply)) {
+        return existingQuotedReply
+      }
+
+      return hasQuotedReplyBody(incomingQuotedReply) ? incomingQuotedReply : null
+    }
+
+    function normalizeReplyPayload(reply: Reply, existingReply?: Reply | null): Reply {
+      const normalized: Reply = { ...reply }
+      const hasQuotedReplyId = typeof normalized.quoted_reply_id === 'string'
+        && normalized.quoted_reply_id.trim().length > 0
+
+      if (!hasQuotedReplyId) {
+        normalized.quoted_reply_id = null
+        normalized.quoted_reply = null
+        normalized.quote_data = null
+        return normalized
+      }
+
+      normalized.quoted_reply = resolveQuotedReply(normalized, existingReply)
+
+      return normalized
+    }
+
     function mergeUniqueById<TItem extends { id: string }>(
       existingItems: TItem[],
       incomingItems: TItem[]
@@ -950,6 +1023,7 @@ export function createDiscussionStoreDefinition(config: DiscussionStoreConfig) {
         }
 
         const filteredReplyBatch = filterOpeningReplies(loadedReplyBatch)
+        currentReplySort.value = sortBy
 
         if (cursor) {
           replies.value = mergeUniqueById(replies.value, filteredReplyBatch)
@@ -998,6 +1072,70 @@ export function createDiscussionStoreDefinition(config: DiscussionStoreConfig) {
       }
     }
 
+    function insertTopLevelReply(reply: Reply): void {
+      if (currentReplySort.value === 'new') {
+        replies.value.unshift(reply)
+        return
+      }
+
+      replies.value.push(reply)
+    }
+
+    function insertReplyIntoActiveThread(reply: Reply, parentReplyId?: string | null): boolean {
+      const existingReplyIndex = replies.value.findIndex((candidate) => candidate.id === reply.id)
+
+      if (existingReplyIndex !== -1) {
+        const existingReply = replies.value[existingReplyIndex]
+        if (existingReply) {
+          replies.value[existingReplyIndex] = {
+            ...existingReply,
+            ...reply,
+          }
+        }
+
+        return false
+      }
+
+      if (parentReplyId) {
+        const parentIndex = replies.value.findIndex((candidate) => candidate.id === parentReplyId)
+        if (parentIndex !== -1) {
+          const parent = replies.value[parentIndex]
+          const parentDepth = parent?.depth ?? parent?.display_depth ?? 0
+
+          reply.depth = parentDepth + 1
+          reply.display_depth = parentDepth + 1
+          reply.parent_reply_id = parentReplyId
+
+          let insertIndex = parentIndex + 1
+          while (insertIndex < replies.value.length) {
+            const nextReply = replies.value[insertIndex]
+            const nextDepth = nextReply?.depth ?? nextReply?.display_depth ?? 0
+            if (nextDepth <= parentDepth) {
+              break
+            }
+            insertIndex += 1
+          }
+
+          replies.value.splice(insertIndex, 0, reply)
+
+          if (parent) {
+            parent.reply_count = (parent.reply_count ?? 0) + 1
+            parent.children_count = (parent.children_count ?? 0) + 1
+          }
+
+          return true
+        }
+
+        replies.value.push(reply)
+        return true
+      }
+
+      reply.depth = 0
+      reply.display_depth = 0
+      insertTopLevelReply(reply)
+      return true
+    }
+
     async function createThread(
       spaceSlug: string,
       input: CreateThreadInput
@@ -1144,7 +1282,8 @@ export function createDiscussionStoreDefinition(config: DiscussionStoreConfig) {
         })
 
         const responseData = toRecord(response.data)
-        const newReply = (responseData?.data ?? null) as Reply | null
+        const createdReply = (responseData?.data ?? null) as Reply | null
+        const newReply = createdReply ? normalizeReplyPayload(createdReply) : null
 
         if (newReply?.id) {
           locallyCreatedReplyIds.add(newReply.id)
@@ -1153,52 +1292,7 @@ export function createDiscussionStoreDefinition(config: DiscussionStoreConfig) {
         let didInsertReply = false
 
         if (newReply) {
-          const existingReplyIndex = replies.value.findIndex((reply) => reply.id === newReply.id)
-
-          if (existingReplyIndex !== -1) {
-            const existingReply = replies.value[existingReplyIndex]
-            if (existingReply) {
-              replies.value[existingReplyIndex] = {
-                ...existingReply,
-                ...newReply,
-              }
-            }
-          } else if (input.parent_id) {
-            const parentIndex = replies.value.findIndex((reply) => reply.id === input.parent_id)
-            if (parentIndex !== -1) {
-              const parent = replies.value[parentIndex]
-              const parentDepth = parent?.depth ?? parent?.display_depth ?? 0
-              newReply.depth = parentDepth + 1
-              newReply.display_depth = parentDepth + 1
-              newReply.parent_reply_id = input.parent_id
-
-              let insertIndex = parentIndex + 1
-              while (insertIndex < replies.value.length) {
-                const reply = replies.value[insertIndex]
-                const replyDepth = reply?.depth ?? reply?.display_depth ?? 0
-                if (replyDepth <= parentDepth) {
-                  break
-                }
-                insertIndex += 1
-              }
-
-              replies.value.splice(insertIndex, 0, newReply)
-              didInsertReply = true
-
-              if (parent) {
-                parent.reply_count = (parent.reply_count ?? 0) + 1
-                parent.children_count = (parent.children_count ?? 0) + 1
-              }
-            } else {
-              replies.value.push(newReply)
-              didInsertReply = true
-            }
-          } else {
-            newReply.depth = 0
-            newReply.display_depth = 0
-            replies.value.push(newReply)
-            didInsertReply = true
-          }
+          didInsertReply = insertReplyIntoActiveThread(newReply, input.parent_id)
         }
 
         if (didInsertReply && currentThread.value?.id === threadId) {
@@ -1380,29 +1474,18 @@ export function createDiscussionStoreDefinition(config: DiscussionStoreConfig) {
         return
       }
 
-      const normalizedPayload: Reply = (() => {
-        const normalized: Reply = { ...payload }
+      const existingReply = replies.value.find((candidate) => candidate.id === payload.id) ?? null
+      const normalizedPayload = normalizeReplyPayload(payload, existingReply)
 
-        const quotedReplyId = normalized.quoted_reply_id
-        const hasQuotedReplyId = typeof quotedReplyId === 'string' && quotedReplyId.trim().length > 0
-        if (!hasQuotedReplyId) {
-          normalized.quoted_reply_id = null
-          normalized.quoted_reply = null
-          normalized.quote_data = null
-        }
-
-        return normalized
-      })()
+      const didInsertReply = insertReplyIntoActiveThread(
+        normalizedPayload,
+        normalizedPayload.parent_reply_id ?? null
+      )
 
-      const already = replies.value.some((reply) => reply.id === payload.id)
-      if (!already) {
-        replies.value.push(normalizedPayload)
-
-        if (currentThread.value && payload.id && !realtimeCountedReplyIds.has(payload.id)) {
-          realtimeCountedReplyIds.add(payload.id)
-          currentThread.value.reply_count = (currentThread.value.reply_count || 0) + 1
-          currentThread.value.last_activity_at = payload.created_at ?? new Date().toISOString()
-        }
+      if (didInsertReply && currentThread.value && payload.id && !realtimeCountedReplyIds.has(payload.id)) {
+        realtimeCountedReplyIds.add(payload.id)
+        currentThread.value.reply_count = (currentThread.value.reply_count || 0) + 1
+        currentThread.value.last_activity_at = payload.created_at ?? new Date().toISOString()
       }
     }