@codingfactory/socialkit-vue 0.7.22 → 0.7.24

package/dist/stores/discussion.js

@@ -124,6 +124,8 @@ export function createDiscussionStoreDefinition(config) {
         const spaceMemberships = ref({});
         const spaceMembershipLoading = ref({});
         const threads = ref([]);
+        const browseThreadList = ref([]);
+        const currentBrowseMode = ref(null);
         const currentThread = ref(null);
         const replies = ref([]);
         const currentReplySort = ref('best');
@@ -131,8 +133,10 @@ export function createDiscussionStoreDefinition(config) {
         const threadsLoadingState = createLoadingState();
         const repliesLoadingState = createLoadingState();
         const loading = ref(false);
+        let activeLoadingRequests = 0;
         const error = ref(null);
         const nextCursor = ref(null);
+        const browseNextCursor = ref(null);
         const repliesNextCursor = ref(null);
         const latestSpaceSlug = ref(null);
         const latestThreadId = ref(null);
@@ -237,6 +241,17 @@ export function createDiscussionStoreDefinition(config) {
             }
             return hasQuotedReplyBody(incomingQuotedReply) ? incomingQuotedReply : null;
         }
+        /**
+         * Check whether a reply-like object carries a truthy `is_author` flag.
+         * The field is not part of the Reply TS interface (it is user-specific
+         * metadata injected by the API) so we access it reflectively.
+         */
+        function hasReplyAuthorFlag(reply) {
+            return Reflect.get(reply, 'is_author') === true;
+        }
+        function setReplyAuthorFlag(reply, value) {
+            Reflect.set(reply, 'is_author', value);
+        }
         function normalizeReplyPayload(reply, existingReply) {
             const normalized = { ...reply };
             const hasQuotedReplyId = typeof normalized.quoted_reply_id === 'string'
@@ -448,6 +463,14 @@ export function createDiscussionStoreDefinition(config) {
             THREAD: 'thread',
             REPLY: 'reply',
         };
+        function beginLoading() {
+            activeLoadingRequests += 1;
+            loading.value = true;
+        }
+        function endLoading() {
+            activeLoadingRequests = Math.max(0, activeLoadingRequests - 1);
+            loading.value = activeLoadingRequests > 0;
+        }
         function flattenTree(tree) {
             const result = [];
             function walk(nodes) {
@@ -467,12 +490,30 @@ export function createDiscussionStoreDefinition(config) {
         function leafSpaces() {
             return spaces.value.filter((space) => space.is_leaf !== false);
         }
-        async function loadSpaces() {
+        function buildSpaceQueryString(options) {
+            const params = new URLSearchParams();
+            params.set('tree', options?.tree === false ? '0' : '1');
+            if (options?.kind) {
+                params.set('kind', options.kind);
+            }
+            if (typeof options?.scope_type === 'string' && options.scope_type.trim().length > 0) {
+                params.set('scope_type', options.scope_type.trim());
+            }
+            if (typeof options?.scope_id === 'string' && options.scope_id.trim().length > 0) {
+                params.set('scope_id', options.scope_id.trim());
+            }
+            if (typeof options?.parent_id === 'string' && options.parent_id.trim().length > 0) {
+                params.set('parent_id', options.parent_id.trim());
+            }
+            return params.toString();
+        }
+        async function loadSpaces(options) {
             spacesLoadingState.setLoading(true);
-            loading.value = true;
+            beginLoading();
             error.value = null;
             try {
-                const response = await client.get('/v1/discussion/spaces?tree=1');
+                const queryString = buildSpaceQueryString(options);
+                const response = await client.get(`/v1/discussion/spaces?${queryString}`);
                 const responseData = toRecord(response.data);
                 const treeData = Array.isArray(responseData?.items)
                     ? responseData.items
@@ -481,9 +522,7 @@ export function createDiscussionStoreDefinition(config) {
                         : [];
                 spaceTree.value = treeData;
                 spaces.value = flattenTree(treeData);
-                if (treeData.length === 0) {
-                    spacesLoadingState.setEmpty(true);
-                }
+                spacesLoadingState.setEmpty(treeData.length === 0);
                 return response.data;
             }
             catch (err) {
@@ -495,7 +534,64 @@ export function createDiscussionStoreDefinition(config) {
             }
             finally {
                 spacesLoadingState.setLoading(false);
-                loading.value = false;
+                endLoading();
+            }
+        }
+        async function createSpace(input) {
+            beginLoading();
+            error.value = null;
+            try {
+                const response = await client.post('/v1/discussion/spaces', {
+                    ...input,
+                    ...(typeof input.parent_id === 'string' ? { parent_id: input.parent_id } : { parent_id: null }),
+                    ...(typeof input.description === 'string' && input.description.trim().length > 0
+                        ? { description: input.description.trim() }
+                        : {}),
+                    ...(input.meta ? { meta: input.meta } : {}),
+                });
+                return response.data;
+            }
+            catch (err) {
+                logger.error('Failed to create discussion space:', err);
+                error.value = getErrorMessage(err, 'Failed to create discussion space');
+                throw err;
+            }
+            finally {
+                endLoading();
+            }
+        }
+        async function moveSpace(spaceId, parentId) {
+            beginLoading();
+            error.value = null;
+            try {
+                const response = await client.post(`/v1/discussion/spaces/${spaceId}/move`, {
+                    parent_id: parentId ?? null,
+                });
+                return response.data;
+            }
+            catch (err) {
+                logger.error('Failed to move discussion space:', err);
+                error.value = getErrorMessage(err, 'Failed to move discussion space');
+                throw err;
+            }
+            finally {
+                endLoading();
+            }
+        }
+        async function reorderSpaces(ids) {
+            beginLoading();
+            error.value = null;
+            try {
+                const response = await client.post('/v1/discussion/spaces/reorder', { ids });
+                return response.data;
+            }
+            catch (err) {
+                logger.error('Failed to reorder discussion spaces:', err);
+                error.value = getErrorMessage(err, 'Failed to reorder discussion spaces');
+                throw err;
+            }
+            finally {
+                endLoading();
             }
         }
         async function loadSpaceDetail(slug, options) {
@@ -577,9 +673,8 @@ export function createDiscussionStoreDefinition(config) {
                 latestSpaceSlug.value = spaceSlug;
             }
             threadsLoadingState.setLoading(true);
-            loading.value = true;
+            beginLoading();
             error.value = null;
-            let isStale = false;
             try {
                 const queryParams = new URLSearchParams();
                 if (cursor) {
@@ -588,6 +683,9 @@ export function createDiscussionStoreDefinition(config) {
                 if (options?.sort) {
                     queryParams.set('sort', options.sort);
                 }
+                if (options?.filter && options.filter !== 'all') {
+                    queryParams.set('filter', options.filter);
+                }
                 const queryString = queryParams.toString();
                 const url = `/v1/discussion/spaces/${spaceSlug}/threads${queryString.length > 0 ? `?${queryString}` : ''}`;
                 const requestConfig = {
@@ -604,7 +702,6 @@ export function createDiscussionStoreDefinition(config) {
                     response = await client.get(url, requestConfig);
                 }
                 if (latestSpaceSlug.value !== spaceSlug) {
-                    isStale = true;
                     return response.data;
                 }
                 const newThreads = getThreadsFromPayload(response.data).map((thread) => normalizeThreadReplyCount(thread));
@@ -630,7 +727,6 @@ export function createDiscussionStoreDefinition(config) {
             }
             catch (err) {
                 if (options?.signal?.aborted || latestSpaceSlug.value !== spaceSlug) {
-                    isStale = true;
                     return undefined;
                 }
                 if (isAxiosError(err) && err.response?.status === 404) {
@@ -654,10 +750,67 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                if (!isStale) {
-                    threadsLoadingState.setLoading(false);
-                    loading.value = false;
+                threadsLoadingState.setLoading(false);
+                endLoading();
+            }
+        }
+        async function browseThreads(view, cursor, options) {
+            if (!cursor || currentBrowseMode.value !== view) {
+                currentBrowseMode.value = view;
+            }
+            threadsLoadingState.setLoading(true);
+            beginLoading();
+            error.value = null;
+            try {
+                const queryParams = new URLSearchParams();
+                if (cursor) {
+                    queryParams.set('cursor', cursor);
                 }
+                const queryString = queryParams.toString();
+                const url = `/v1/discussion/threads/${view}${queryString.length > 0 ? `?${queryString}` : ''}`;
+                const requestConfig = {
+                    ...(options?.signal ? { signal: options.signal } : {}),
+                };
+                let response;
+                try {
+                    response = await client.get(url, requestConfig);
+                }
+                catch (requestError) {
+                    if (!isTransientThreadListError(requestError) || options?.signal?.aborted) {
+                        throw requestError;
+                    }
+                    response = await client.get(url, requestConfig);
+                }
+                if (currentBrowseMode.value !== view) {
+                    return response.data;
+                }
+                const newThreads = getThreadsFromPayload(response.data).map((thread) => normalizeThreadReplyCount(thread));
+                if (cursor) {
+                    browseThreadList.value = mergeUniqueById(browseThreadList.value, newThreads);
+                }
+                else {
+                    browseThreadList.value = newThreads;
+                    if (newThreads.length === 0) {
+                        threadsLoadingState.setEmpty(true);
+                    }
+                }
+                browseNextCursor.value = getThreadNextCursorFromPayload(response.data);
+                currentSpace.value = null;
+                return response.data;
+            }
+            catch (err) {
+                if (options?.signal?.aborted || currentBrowseMode.value !== view) {
+                    return undefined;
+                }
+                logger.error('Failed to browse discussion threads:', err);
+                const errorMessage = getErrorMessage(err, 'Failed to load discussion threads');
+                threadsLoadingState.setError(new Error(errorMessage));
+                error.value = errorMessage;
+                throw err;
+            }
+            finally {
+                threadsLoadingState.setLoading(false);
+                endLoading();
             }
         }
         function isValidUUID(uuid) {
@@ -672,23 +825,25 @@ export function createDiscussionStoreDefinition(config) {
         async function loadThread(spaceSlug, threadId) {
             latestThreadId.value = threadId;
             cleanupRealtimeChannels();
-            loading.value = true;
+            beginLoading();
             error.value = null;
+            replies.value = [];
+            repliesNextCursor.value = null;
+            currentReplySort.value = 'best';
             if (!threadId) {
                 error.value = 'Missing thread identifier';
-                loading.value = false;
+                endLoading();
                 return null;
             }
-            if (!isValidUUID(threadId)) {
+            // Accept both UUID and slug identifiers — the backend resolves by ID or slug
+            if (!/^[\w-]+$/u.test(threadId)) {
                 error.value = 'Invalid thread identifier format';
-                loading.value = false;
+                endLoading();
                 return null;
             }
-            let isStale = false;
             try {
                 const response = await client.get(`/v1/discussion/threads/${threadId}`);
                 if (latestThreadId.value !== threadId) {
-                    isStale = true;
                     return null;
                 }
                 const responseData = toRecord(response.data);
@@ -738,7 +893,6 @@ export function createDiscussionStoreDefinition(config) {
             }
             catch (err) {
                 if (latestThreadId.value !== threadId) {
-                    isStale = true;
                     return null;
                 }
                 const errorResponseData = getErrorResponseData(err);
@@ -764,18 +918,15 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                if (!isStale) {
-                    loading.value = false;
-                }
+                endLoading();
             }
         }
         async function loadReplies(threadId, cursor, sortBy = 'best') {
             if (latestThreadId.value === null) {
                 latestThreadId.value = threadId;
             }
-            loading.value = true;
+            beginLoading();
             error.value = null;
-            let isStale = false;
             try {
                 const queryParts = [];
                 if (cursor) {
@@ -787,7 +938,6 @@ export function createDiscussionStoreDefinition(config) {
                 const queryString = queryParts.length > 0 ? `?${queryParts.join('&')}` : '';
                 const response = await client.get(`/v1/discussion/threads/${threadId}/replies${queryString}`);
                 if (latestThreadId.value !== threadId) {
-                    isStale = true;
                     return undefined;
                 }
                 const responseData = toRecord(response.data);
@@ -847,7 +997,6 @@ export function createDiscussionStoreDefinition(config) {
             }
             catch (err) {
                 if (latestThreadId.value !== threadId) {
-                    isStale = true;
                     return undefined;
                 }
                 logger.error('Failed to load replies:', err);
@@ -855,9 +1004,7 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                if (!isStale) {
-                    loading.value = false;
-                }
+                endLoading();
             }
         }
         function insertTopLevelReply(reply) {
@@ -872,10 +1019,17 @@ export function createDiscussionStoreDefinition(config) {
             if (existingReplyIndex !== -1) {
                 const existingReply = replies.value[existingReplyIndex];
                 if (existingReply) {
-                    replies.value[existingReplyIndex] = {
+                    const merged = {
                         ...existingReply,
                         ...reply,
                     };
+                    // Broadcast payloads lack an authenticated user context, so
+                    // is_author is always false in them.  Preserve the value from
+                    // the original authenticated API response when it was true.
+                    if (hasReplyAuthorFlag(existingReply) && !hasReplyAuthorFlag(reply)) {
+                        setReplyAuthorFlag(merged, true);
+                    }
+                    replies.value[existingReplyIndex] = merged;
                 }
                 return false;
             }
@@ -913,7 +1067,7 @@ export function createDiscussionStoreDefinition(config) {
         }
         async function createThread(spaceSlug, input) {
             cleanupRealtimeChannels();
-            loading.value = true;
+            beginLoading();
             error.value = null;
             try {
                 const response = await client.post(`/v1/discussion/spaces/${spaceSlug}/threads`, {
@@ -941,11 +1095,11 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
         async function loadTagCategories(query = '') {
-            loading.value = true;
+            beginLoading();
             error.value = null;
             try {
                 const normalizedQuery = query.trim();
@@ -979,7 +1133,7 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
         function cleanupRealtimeChannels() {
@@ -1221,10 +1375,17 @@ export function createDiscussionStoreDefinition(config) {
             if (!existingReply) {
                 return;
             }
-            replies.value[replyIndex] = normalizeReplyPayload({
+            const merged = {
                 ...existingReply,
                 ...payload,
-            }, existingReply);
+            };
+            // Broadcast payloads lack an authenticated user context, so
+            // is_author is always false in them.  Preserve the value from
+            // the original authenticated API response when it was true.
+            if (hasReplyAuthorFlag(existingReply) && !hasReplyAuthorFlag(payload)) {
+                setReplyAuthorFlag(merged, true);
+            }
+            replies.value[replyIndex] = normalizeReplyPayload(merged, existingReply);
         }
         function handleRealtimeReplyReaction(payload) {
             if (!payload?.reply_id || !payload.thread_id || !payload.user_id) {
@@ -1343,40 +1504,49 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
         }
-        async function searchThreads(query, spaceSlug) {
-            loading.value = true;
+        async function searchThreads(query, spaceSlug, options) {
+            beginLoading();
             error.value = null;
             try {
                 const params = new URLSearchParams({ q: query });
                 if (spaceSlug) {
                     params.append('space', spaceSlug);
                 }
-                const response = await client.get(`/v1/discussion/search/threads?${params.toString()}`);
+                const requestConfig = options?.signal
+                    ? { signal: options.signal }
+                    : undefined;
+                const response = await client.get(`/v1/discussion/search/threads?${params.toString()}`, requestConfig);
                 return response.data;
             }
             catch (err) {
+                if (options?.signal?.aborted) {
+                    return undefined;
+                }
                 logger.error('Failed to search threads:', err);
                 error.value = getErrorMessage(err, 'Failed to search');
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
-        async function searchThreadsInSpace(query, spaceId) {
+        async function searchThreadsInSpace(query, spaceId, options) {
             const normalizedQuery = query.trim();
             if (normalizedQuery.length < 2) {
                 return [];
             }
-            loading.value = true;
+            beginLoading();
             error.value = null;
             try {
+                const requestConfig = options?.signal
+                    ? { signal: options.signal }
+                    : undefined;
                 const response = await client.post('/v1/discussion/search', {
                     query: normalizedQuery,
                     space_id: spaceId,
                     sort_by: 'relevance',
                     limit: 50,
-                });
+                }, requestConfig);
                 const rows = extractThreadSearchRows(response.data);
                 if (rows.length > 0) {
                     return rows.map((row) => mapSearchRowToThread(row, spaceId));
@@ -1384,6 +1554,9 @@ export function createDiscussionStoreDefinition(config) {
                 return filterLoadedThreadsByQuery(normalizedQuery, spaceId);
             }
             catch (err) {
+                if (options?.signal?.aborted) {
+                    return [];
+                }
                 logger.error('Failed to search threads in space:', err);
                 error.value = isAxiosError(err)
                     ? err.response?.data?.message ?? 'Failed to search threads'
@@ -1391,25 +1564,31 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
-        async function searchThreadsGlobally(query) {
+        async function searchThreadsGlobally(query, options) {
             const normalizedQuery = query.trim();
             if (normalizedQuery.length < 2) {
                 return [];
             }
-            loading.value = true;
+            beginLoading();
             error.value = null;
             try {
+                const requestConfig = options?.signal
+                    ? { signal: options.signal }
+                    : undefined;
                 const response = await client.post('/v1/discussion/search', {
                     query: normalizedQuery,
                     sort_by: 'relevance',
                     limit: 50,
-                });
+                }, requestConfig);
                 return extractThreadSearchRows(response.data).map((row) => mapSearchRowToThread(row));
             }
             catch (err) {
+                if (options?.signal?.aborted) {
+                    return [];
+                }
                 logger.error('Failed to search threads globally:', err);
                 error.value = isAxiosError(err)
                     ? err.response?.data?.message ?? 'Failed to search threads'
@@ -1417,7 +1596,7 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
         function mapSearchRowToThread(row, fallbackSpaceId) {
@@ -1445,6 +1624,14 @@ export function createDiscussionStoreDefinition(config) {
             if (typeof row.slug === 'string') {
                 thread.slug = row.slug;
             }
+            const spaceId = row.space_id ?? fallbackSpaceId ?? '';
+            if (typeof row.space_name === 'string' && typeof row.space_slug === 'string' && spaceId) {
+                thread.space = {
+                    id: spaceId,
+                    slug: row.space_slug,
+                    name: row.space_name,
+                };
+            }
             return thread;
         }
         function extractThreadSearchRows(responseBody) {
@@ -1480,7 +1667,7 @@ export function createDiscussionStoreDefinition(config) {
             return spaces.value.filter((space) => space.meta?.is_featured);
         }
         async function setThreadPinned(threadId, pinned) {
-            loading.value = true;
+            beginLoading();
             error.value = null;
             try {
                 await client.post(`/v1/discussion/threads/${threadId}/pin`, { pinned });
@@ -1499,11 +1686,11 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
         async function setThreadLocked(threadId, locked) {
-            loading.value = true;
+            beginLoading();
             error.value = null;
             try {
                 await client.post(`/v1/discussion/threads/${threadId}/lock`, { locked });
@@ -1522,11 +1709,11 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
         async function moveThread(threadId, toSpaceSlug) {
-            loading.value = true;
+            beginLoading();
             error.value = null;
             try {
                 const response = await client.post(`/v1/discussion/threads/${threadId}/move`, {
@@ -1535,11 +1722,21 @@ export function createDiscussionStoreDefinition(config) {
                 const responseData = toRecord(response.data);
                 const movedThread = (responseData?.data ?? null);
                 const destinationSpace = spaces.value.find((space) => space.slug === toSpaceSlug);
+                const destinationSpaceSummary = destinationSpace
+                    ? {
+                        id: destinationSpace.id,
+                        slug: destinationSpace.slug,
+                        name: destinationSpace.name,
+                    }
+                    : null;
                 if (currentThread.value?.id === threadId && currentThread.value) {
                     currentThread.value = {
                         ...currentThread.value,
                         ...(movedThread ?? {}),
-                        ...(destinationSpace ? { space_id: destinationSpace.id } : {}),
+                        ...(destinationSpaceSummary ? {
+                            space_id: destinationSpaceSummary.id,
+                            space: destinationSpaceSummary,
+                        } : {}),
                     };
                 }
                 const currentSpaceSlug = currentSpace.value?.slug ?? null;
@@ -1556,7 +1753,10 @@ export function createDiscussionStoreDefinition(config) {
                         threads.value[threadIndex] = {
                             ...existingThread,
                             ...(movedThread ?? {}),
-                            ...(destinationSpace ? { space_id: destinationSpace.id } : {}),
+                            ...(destinationSpaceSummary ? {
+                                space_id: destinationSpaceSummary.id,
+                                space: destinationSpaceSummary,
+                            } : {}),
                         };
                     }
                 }
@@ -1568,11 +1768,11 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
         async function updateThread(threadId, updates) {
-            loading.value = true;
+            beginLoading();
             error.value = null;
             try {
                 const response = await client.patch(`/v1/discussion/threads/${threadId}`, {
@@ -1602,11 +1802,11 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
         async function deleteThread(threadId) {
-            loading.value = true;
+            beginLoading();
             error.value = null;
             try {
                 const response = await client.delete(`/v1/discussion/threads/${threadId}`);
@@ -1622,11 +1822,11 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
         async function restoreThread(threadId) {
-            loading.value = true;
+            beginLoading();
             error.value = null;
             try {
                 const response = await client.post(`/v1/discussion/threads/${threadId}/restore`);
@@ -1667,11 +1867,11 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
         async function deleteReply(replyId) {
-            loading.value = true;
+            beginLoading();
             error.value = null;
             const replyIndex = replies.value.findIndex((reply) => reply.id === replyId);
             const replyToRestore = replyIndex >= 0 ? replies.value[replyIndex] ?? null : null;
@@ -1729,7 +1929,7 @@ export function createDiscussionStoreDefinition(config) {
                 throw err;
             }
             finally {
-                loading.value = false;
+                endLoading();
             }
         }
         async function updateReply(replyId, body) {
@@ -1962,16 +2162,23 @@ export function createDiscussionStoreDefinition(config) {
             spaceMembershipLoading,
             cleanupRealtimeChannels,
             threads,
+            browseThreadList,
+            currentBrowseMode,
             currentThread,
             replies,
+            currentReplySort,
             loading,
             error,
             nextCursor,
+            browseNextCursor,
             repliesNextCursor,
             spacesLoadingState,
             threadsLoadingState,
             repliesLoadingState,
             loadSpaces,
+            createSpace,
+            moveSpace,
+            reorderSpaces,
             loadSpaceDetail,
             loadSpaceMembership,
             joinSpace,
@@ -1981,6 +2188,7 @@ export function createDiscussionStoreDefinition(config) {
             rootSpaces,
             leafSpaces,
             loadThreads,
+            browseThreads,
             loadThread,
             loadReplies,
             createThread,