🔄 refactor: Update shared links API and data handling; improve query …

…parameters and response structure

api/models/Share.js

@@ -1,5 +1,6 @@
 const { nanoid } = require('nanoid');
 const { Constants } = require('librechat-data-provider');
+const { Conversation } = require('~/models/Conversation');
 const SharedLink = require('./schema/shareSchema');
 const { getMessages } = require('./Message');
 const logger = require('~/config/winston');
@@ -94,23 +95,50 @@ async function getSharedMessages(shareId) {
   }
 }
 
-async function getSharedLinks(user, pageNumber = 1, pageSize = 25, isPublic = true) {
-  const query = { user, isPublic };
-
+async function getSharedLinks(
+  user,
+  pageNumber = 1,
+  pageSize = 25,
+  isPublic = true,
+  sortBy = 'createdAt',
+  sortDirection = 'desc',
+  search = '',
+) {
   try {
+    const query = { user, isPublic };
+
+    if (search) {
+      const searchResults = await Conversation.meiliSearch(search, {
+        attributesToHighlight: ['title'],
+      });
+
+      const conversationIds = searchResults.hits.map((hit) => hit.id);
+
+      query['conversation'] = { $in: conversationIds };
+    }
+
+    const sort = {};
+    sort[sortBy] = sortDirection === 'desc' ? -1 : 1;
+
     const [totalConvos, sharedLinks] = await Promise.all([
       SharedLink.countDocuments(query).exec(),
       SharedLink.find(query)
-        .sort({ updatedAt: -1 })
+        .sort(sort)
         .skip((pageNumber - 1) * pageSize)
         .limit(pageSize)
-        .select('-_id -__v -user')
+        .select('-__v -user')
         .lean()
         .exec(),
     ]);
 
     return {
-      sharedLinks,
+      sharedLinks: sharedLinks.map((link) => ({
+        shareId: link.shareId,
+        title: link.conversation?.title || 'Untitled',
+        isPublic: link.isPublic,
+        createdAt: link.createdAt,
+      })),
+      totalCount: totalConvos,
       pages: Math.ceil((totalConvos || 1) / pageSize),
       pageNumber,
       pageSize,
@@ -155,11 +183,14 @@ async function createSharedLink(user, conversationId) {
       throw new ShareServiceError('Share already exists', 'SHARE_EXISTS');
     }
 
+    const title = conversationMessages[0]?.title || 'Untitled';
+
     const shareId = nanoid();
     await SharedLink.create({
       shareId,
       conversationId,
       messages: conversationMessages,
+      title,
       user,
     });
 

api/models/schema/shareSchema.js

@@ -20,14 +20,6 @@ const shareSchema = mongoose.Schema(
       index: true,
     },
     isPublic: {
-      type: Boolean,
-      default: false,
-    },
-    isVisible: {
-      type: Boolean,
-      default: false,
-    },
-    isAnonymous: {
       type: Boolean,
       default: true,
     },

api/server/routes/share.js

@@ -46,23 +46,40 @@ if (allowSharedLinks) {
  */
 router.get('/', requireJwtAuth, async (req, res) => {
   try {
-    let pageNumber = req.query.pageNumber || 1;
-    pageNumber = parseInt(pageNumber, 10);
+    const params = {
+      pageNumber: Math.max(1, parseInt(req.query.pageNumber) || 1),
+      pageSize: Math.max(1, parseInt(req.query.pageSize) || 10),
+      isPublic: isEnabled(req.query.isPublic),
+      sortBy: ['createdAt', 'title'].includes(req.query.sortBy) ? req.query.sortBy : 'createdAt',
+      sortDirection: ['asc', 'desc'].includes(req.query.sortDirection)
+        ? req.query.sortDirection
+        : 'desc',
+      search: req.query.search?.trim() || undefined,
+    };
 
-    if (isNaN(pageNumber) || pageNumber < 1) {
-      return res.status(400).json({ error: 'Invalid page number' });
-    }
-
-    let pageSize = req.query.pageSize || 25;
-    pageSize = parseInt(pageSize, 10);
+    const result = await getSharedLinks(
+      req.user.id,
+      params.pageNumber,
+      params.pageSize,
+      params.isPublic,
+      params.sortBy,
+      params.sortDirection,
+      params.search,
+    );
 
-    if (isNaN(pageSize) || pageSize < 1) {
-      return res.status(400).json({ error: 'Invalid page size' });
-    }
-    const isPublic = req.query.isPublic === 'true';
-    res.status(200).send(await getSharedLinks(req.user.id, pageNumber, pageSize, isPublic));
+    res.status(200).send({
+      links: result.sharedLinks,
+      totalCount: result.totalCount,
+      pageNumber: result.pageNumber,
+      pageSize: result.pageSize,
+      pages: result.pages,
+    });
   } catch (error) {
-    res.status(500).json({ message: 'Error getting shared links' });
+    console.error('Error getting shared links:', error);
+    res.status(500).json({
+      message: 'Error getting shared links',
+      error: error.message,
+    });
   }
 });
 

client/src/components/Conversations/ConvoOptions/ConvoOptions.tsx

@@ -141,7 +141,6 @@ export default function ConvoOptions({
       />
       {showShareDialog && (
         <ShareButton
-          title={title ?? ''}
           conversationId={conversationId ?? ''}
           open={showShareDialog}
           onOpenChange={setShowShareDialog}