feat: add x-padding to cells in conversation list and inset outline for selected conversation

.env.example

@@ -623,8 +623,6 @@ AZURE_CONTAINER_NAME=files
 
 ALLOW_SHARED_LINKS=true
 ALLOW_SHARED_LINKS_PUBLIC=true
-# Default expiration time for shared links in hours (default: 0 = never expires)
-# SHARED_LINK_DEFAULT_TTL_HOURS=0
 
 #==============================#
 # Static File Cache Control    #

.github/workflows/dev-staging-images.yml

@@ -1,66 +0,0 @@
-name: Docker Dev Staging Images Build
-
-on:
-  workflow_dispatch:
-
-jobs:
-  build:
-    runs-on: ubuntu-latest
-    strategy:
-      matrix:
-        include:
-          - target: api-build
-            file: Dockerfile.multi
-            image_name: lc-dev-staging-api
-          - target: node
-            file: Dockerfile
-            image_name: lc-dev-staging
-
-    steps:
-      # Check out the repository
-      - name: Checkout
-        uses: actions/checkout@v4
-
-      # Set up QEMU
-      - name: Set up QEMU
-        uses: docker/setup-qemu-action@v3
-
-      # Set up Docker Buildx
-      - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v3
-
-      # Log in to GitHub Container Registry
-      - name: Log in to GitHub Container Registry
-        uses: docker/login-action@v2
-        with:
-          registry: ghcr.io
-          username: ${{ github.actor }}
-          password: ${{ secrets.GITHUB_TOKEN }}
-
-      # Login to Docker Hub
-      - name: Login to Docker Hub
-        uses: docker/login-action@v3
-        with:
-          username: ${{ secrets.DOCKERHUB_USERNAME }}
-          password: ${{ secrets.DOCKERHUB_TOKEN }}
-
-      # Prepare the environment
-      - name: Prepare environment
-        run: |
-          cp .env.example .env
-
-      # Build and push Docker images for each target
-      - name: Build and push Docker images
-        uses: docker/build-push-action@v5
-        with:
-          context: .
-          file: ${{ matrix.file }}
-          push: true
-          tags: |
-            ghcr.io/${{ github.repository_owner }}/${{ matrix.image_name }}:${{ github.sha }}
-            ghcr.io/${{ github.repository_owner }}/${{ matrix.image_name }}:latest
-            ${{ secrets.DOCKERHUB_USERNAME }}/${{ matrix.image_name }}:${{ github.sha }}
-            ${{ secrets.DOCKERHUB_USERNAME }}/${{ matrix.image_name }}:latest
-          platforms: linux/amd64,linux/arm64
-          target: ${{ matrix.target }}
-

Dockerfile

@@ -1,4 +1,4 @@
-# v0.8.1-rc2
+# v0.8.1-rc1
 
 # Base node image
 FROM node:20-alpine AS node

Dockerfile.multi

@@ -1,5 +1,5 @@
 # Dockerfile.multi
-# v0.8.1-rc2
+# v0.8.1-rc1
 
 # Base for all builds
 FROM node:20-alpine AS base-min

api/app/clients/BaseClient.js

@@ -2,7 +2,6 @@ const crypto = require('crypto');
 const fetch = require('node-fetch');
 const { logger } = require('@librechat/data-schemas');
 const {
-  countTokens,
   getBalanceConfig,
   extractFileContext,
   encodeAndFormatAudios,
@@ -24,6 +23,7 @@ const { getMessages, saveMessage, updateMessage, saveConvo, getConvo } = require
 const { getStrategyFunctions } = require('~/server/services/Files/strategies');
 const { checkBalance } = require('~/models/balanceMethods');
 const { truncateToolCallOutputs } = require('./prompts');
+const countTokens = require('~/server/utils/countTokens');
 const { getFiles } = require('~/models/File');
 const TextStream = require('./TextStream');
 
@@ -1213,8 +1213,8 @@ class BaseClient {
       this.options.req,
       attachments,
       {
-        provider: this.options.agent?.provider ?? this.options.endpoint,
-        endpoint: this.options.agent?.endpoint ?? this.options.endpoint,
+        provider: this.options.agent?.provider,
+        endpoint: this.options.agent?.endpoint,
         useResponsesApi: this.options.agent?.model_parameters?.useResponsesApi,
       },
       getStrategyFunctions,
@@ -1231,8 +1231,8 @@ class BaseClient {
       this.options.req,
       attachments,
       {
-        provider: this.options.agent?.provider ?? this.options.endpoint,
-        endpoint: this.options.agent?.endpoint ?? this.options.endpoint,
+        provider: this.options.agent?.provider,
+        endpoint: this.options.agent?.endpoint,
       },
       getStrategyFunctions,
     );
@@ -1246,8 +1246,8 @@ class BaseClient {
       this.options.req,
       attachments,
       {
-        provider: this.options.agent?.provider ?? this.options.endpoint,
-        endpoint: this.options.agent?.endpoint ?? this.options.endpoint,
+        provider: this.options.agent?.provider,
+        endpoint: this.options.agent?.endpoint,
       },
       getStrategyFunctions,
     );

api/app/clients/tools/structured/TavilySearch.js

@@ -1,5 +1,4 @@
 const { z } = require('zod');
-const { ProxyAgent, fetch } = require('undici');
 const { tool } = require('@langchain/core/tools');
 const { getApiKey } = require('./credentials');
 
@@ -20,19 +19,13 @@ function createTavilySearchTool(fields = {}) {
         ...kwargs,
       };
 
-      const fetchOptions = {
+      const response = await fetch('https://api.tavily.com/search', {
         method: 'POST',
         headers: {
           'Content-Type': 'application/json',
         },
         body: JSON.stringify(requestBody),
-      };
-
-      if (process.env.PROXY) {
-        fetchOptions.dispatcher = new ProxyAgent(process.env.PROXY);
-      }
-
-      const response = await fetch('https://api.tavily.com/search', fetchOptions);
+      });
 
       const json = await response.json();
       if (!response.ok) {

api/app/clients/tools/structured/TavilySearchResults.js

@@ -1,5 +1,4 @@
 const { z } = require('zod');
-const { ProxyAgent, fetch } = require('undici');
 const { Tool } = require('@langchain/core/tools');
 const { getEnvironmentVariable } = require('@langchain/core/utils/env');
 
@@ -103,19 +102,13 @@ class TavilySearchResults extends Tool {
       ...this.kwargs,
     };
 
-    const fetchOptions = {
+    const response = await fetch('https://api.tavily.com/search', {
       method: 'POST',
       headers: {
         'Content-Type': 'application/json',
       },
       body: JSON.stringify(requestBody),
-    };
-
-    if (process.env.PROXY) {
-      fetchOptions.dispatcher = new ProxyAgent(process.env.PROXY);
-    }
-
-    const response = await fetch('https://api.tavily.com/search', fetchOptions);
+    });
 
     const json = await response.json();
     if (!response.ok) {

api/app/clients/tools/structured/specs/TavilySearchResults.spec.js

@@ -1,7 +1,6 @@
-const { fetch, ProxyAgent } = require('undici');
 const TavilySearchResults = require('../TavilySearchResults');
 
-jest.mock('undici');
+jest.mock('node-fetch');
 jest.mock('@langchain/core/utils/env');
 
 describe('TavilySearchResults', () => {
@@ -14,7 +13,6 @@ describe('TavilySearchResults', () => {
 
   beforeEach(() => {
     jest.resetModules();
-    jest.clearAllMocks();
     process.env = {
       ...originalEnv,
       TAVILY_API_KEY: mockApiKey,
@@ -22,6 +20,7 @@ describe('TavilySearchResults', () => {
   });
 
   afterEach(() => {
+    jest.clearAllMocks();
     process.env = originalEnv;
   });
 
@@ -36,49 +35,4 @@ describe('TavilySearchResults', () => {
     });
     expect(instance.apiKey).toBe(mockApiKey);
   });
-
-  describe('proxy support', () => {
-    const mockResponse = {
-      ok: true,
-      json: jest.fn().mockResolvedValue({ results: [] }),
-    };
-
-    beforeEach(() => {
-      fetch.mockResolvedValue(mockResponse);
-    });
-
-    it('should use ProxyAgent when PROXY env var is set', async () => {
-      const proxyUrl = 'http://proxy.example.com:8080';
-      process.env.PROXY = proxyUrl;
-
-      const mockProxyAgent = { type: 'proxy-agent' };
-      ProxyAgent.mockImplementation(() => mockProxyAgent);
-
-      const instance = new TavilySearchResults({ TAVILY_API_KEY: mockApiKey });
-      await instance._call({ query: 'test query' });
-
-      expect(ProxyAgent).toHaveBeenCalledWith(proxyUrl);
-      expect(fetch).toHaveBeenCalledWith(
-        'https://api.tavily.com/search',
-        expect.objectContaining({
-          dispatcher: mockProxyAgent,
-        }),
-      );
-    });
-
-    it('should not use ProxyAgent when PROXY env var is not set', async () => {
-      delete process.env.PROXY;
-
-      const instance = new TavilySearchResults({ TAVILY_API_KEY: mockApiKey });
-      await instance._call({ query: 'test query' });
-
-      expect(ProxyAgent).not.toHaveBeenCalled();
-      expect(fetch).toHaveBeenCalledWith(
-        'https://api.tavily.com/search',
-        expect.not.objectContaining({
-          dispatcher: expect.anything(),
-        }),
-      );
-    });
-  });
 });

api/models/Prompt.js

@@ -1,5 +1,4 @@
 const { ObjectId } = require('mongodb');
-const { escapeRegExp } = require('@librechat/api');
 const { logger } = require('@librechat/data-schemas');
 const {
   Constants,
@@ -15,6 +14,7 @@ const {
 } = require('./Project');
 const { removeAllPermissions } = require('~/server/services/PermissionService');
 const { PromptGroup, Prompt, AclEntry } = require('~/db/models');
+const { escapeRegExp } = require('~/server/utils');
 
 /**
  * Create a pipeline for the aggregation to get prompt groups

api/models/tx.js

@@ -141,7 +141,6 @@ const tokenValues = Object.assign(
     'command-r': { prompt: 0.5, completion: 1.5 },
     'command-r-plus': { prompt: 3, completion: 15 },
     'command-text': { prompt: 1.5, completion: 2.0 },
-    'deepseek-chat': { prompt: 0.28, completion: 0.42 },
     'deepseek-reasoner': { prompt: 0.28, completion: 0.42 },
     'deepseek-r1': { prompt: 0.4, completion: 2.0 },
     'deepseek-v3': { prompt: 0.2, completion: 0.8 },
@@ -174,9 +173,6 @@ const tokenValues = Object.assign(
     'grok-3-mini': { prompt: 0.3, completion: 0.5 },
     'grok-3-mini-fast': { prompt: 0.6, completion: 4 },
     'grok-4': { prompt: 3.0, completion: 15.0 },
-    'grok-4-fast': { prompt: 0.2, completion: 0.5 },
-    'grok-4-1-fast': { prompt: 0.2, completion: 0.5 }, // covers reasoning & non-reasoning variants
-    'grok-code-fast': { prompt: 0.2, completion: 1.5 },
     codestral: { prompt: 0.3, completion: 0.9 },
     'ministral-3b': { prompt: 0.04, completion: 0.04 },
     'ministral-8b': { prompt: 0.1, completion: 0.1 },
@@ -247,10 +243,6 @@ const cacheTokenValues = {
   'claude-sonnet-4': { write: 3.75, read: 0.3 },
   'claude-opus-4': { write: 18.75, read: 1.5 },
   'claude-opus-4-5': { write: 6.25, read: 0.5 },
-  // DeepSeek models - cache hit: $0.028/1M, cache miss: $0.28/1M
-  deepseek: { write: 0.28, read: 0.028 },
-  'deepseek-chat': { write: 0.28, read: 0.028 },
-  'deepseek-reasoner': { write: 0.28, read: 0.028 },
 };
 
 /**

api/models/tx.spec.js

@@ -766,78 +766,6 @@ describe('Deepseek Model Tests', () => {
     const result = tokenValues[valueKey].prompt && multiplier === tokenValues[valueKey].prompt;
     expect(result).toBe(true);
   });
-
-  it('should return correct pricing for deepseek-chat', () => {
-    expect(getMultiplier({ model: 'deepseek-chat', tokenType: 'prompt' })).toBe(
-      tokenValues['deepseek-chat'].prompt,
-    );
-    expect(getMultiplier({ model: 'deepseek-chat', tokenType: 'completion' })).toBe(
-      tokenValues['deepseek-chat'].completion,
-    );
-    expect(tokenValues['deepseek-chat'].prompt).toBe(0.28);
-    expect(tokenValues['deepseek-chat'].completion).toBe(0.42);
-  });
-
-  it('should return correct pricing for deepseek-reasoner', () => {
-    expect(getMultiplier({ model: 'deepseek-reasoner', tokenType: 'prompt' })).toBe(
-      tokenValues['deepseek-reasoner'].prompt,
-    );
-    expect(getMultiplier({ model: 'deepseek-reasoner', tokenType: 'completion' })).toBe(
-      tokenValues['deepseek-reasoner'].completion,
-    );
-    expect(tokenValues['deepseek-reasoner'].prompt).toBe(0.28);
-    expect(tokenValues['deepseek-reasoner'].completion).toBe(0.42);
-  });
-
-  it('should handle DeepSeek model name variations with provider prefixes', () => {
-    const modelVariations = [
-      'deepseek/deepseek-chat',
-      'openrouter/deepseek-chat',
-      'deepseek/deepseek-reasoner',
-    ];
-
-    modelVariations.forEach((model) => {
-      const promptMultiplier = getMultiplier({ model, tokenType: 'prompt' });
-      const completionMultiplier = getMultiplier({ model, tokenType: 'completion' });
-      expect(promptMultiplier).toBe(0.28);
-      expect(completionMultiplier).toBe(0.42);
-    });
-  });
-
-  it('should return correct cache multipliers for DeepSeek models', () => {
-    expect(getCacheMultiplier({ model: 'deepseek-chat', cacheType: 'write' })).toBe(
-      cacheTokenValues['deepseek-chat'].write,
-    );
-    expect(getCacheMultiplier({ model: 'deepseek-chat', cacheType: 'read' })).toBe(
-      cacheTokenValues['deepseek-chat'].read,
-    );
-    expect(getCacheMultiplier({ model: 'deepseek-reasoner', cacheType: 'write' })).toBe(
-      cacheTokenValues['deepseek-reasoner'].write,
-    );
-    expect(getCacheMultiplier({ model: 'deepseek-reasoner', cacheType: 'read' })).toBe(
-      cacheTokenValues['deepseek-reasoner'].read,
-    );
-  });
-
-  it('should return correct cache pricing values for DeepSeek models', () => {
-    expect(cacheTokenValues['deepseek-chat'].write).toBe(0.28);
-    expect(cacheTokenValues['deepseek-chat'].read).toBe(0.028);
-    expect(cacheTokenValues['deepseek-reasoner'].write).toBe(0.28);
-    expect(cacheTokenValues['deepseek-reasoner'].read).toBe(0.028);
-    expect(cacheTokenValues['deepseek'].write).toBe(0.28);
-    expect(cacheTokenValues['deepseek'].read).toBe(0.028);
-  });
-
-  it('should handle DeepSeek cache multipliers with model variations', () => {
-    const modelVariations = ['deepseek/deepseek-chat', 'openrouter/deepseek-reasoner'];
-
-    modelVariations.forEach((model) => {
-      const writeMultiplier = getCacheMultiplier({ model, cacheType: 'write' });
-      const readMultiplier = getCacheMultiplier({ model, cacheType: 'read' });
-      expect(writeMultiplier).toBe(0.28);
-      expect(readMultiplier).toBe(0.028);
-    });
-  });
 });
 
 describe('Qwen3 Model Tests', () => {
@@ -1277,39 +1205,6 @@ describe('Grok Model Tests - Pricing', () => {
       );
     });
 
-    test('should return correct prompt and completion rates for Grok 4 Fast model', () => {
-      expect(getMultiplier({ model: 'grok-4-fast', tokenType: 'prompt' })).toBe(
-        tokenValues['grok-4-fast'].prompt,
-      );
-      expect(getMultiplier({ model: 'grok-4-fast', tokenType: 'completion' })).toBe(
-        tokenValues['grok-4-fast'].completion,
-      );
-    });
-
-    test('should return correct prompt and completion rates for Grok 4.1 Fast models', () => {
-      expect(getMultiplier({ model: 'grok-4-1-fast-reasoning', tokenType: 'prompt' })).toBe(
-        tokenValues['grok-4-1-fast'].prompt,
-      );
-      expect(getMultiplier({ model: 'grok-4-1-fast-reasoning', tokenType: 'completion' })).toBe(
-        tokenValues['grok-4-1-fast'].completion,
-      );
-      expect(getMultiplier({ model: 'grok-4-1-fast-non-reasoning', tokenType: 'prompt' })).toBe(
-        tokenValues['grok-4-1-fast'].prompt,
-      );
-      expect(getMultiplier({ model: 'grok-4-1-fast-non-reasoning', tokenType: 'completion' })).toBe(
-        tokenValues['grok-4-1-fast'].completion,
-      );
-    });
-
-    test('should return correct prompt and completion rates for Grok Code Fast model', () => {
-      expect(getMultiplier({ model: 'grok-code-fast-1', tokenType: 'prompt' })).toBe(
-        tokenValues['grok-code-fast'].prompt,
-      );
-      expect(getMultiplier({ model: 'grok-code-fast-1', tokenType: 'completion' })).toBe(
-        tokenValues['grok-code-fast'].completion,
-      );
-    });
-
     test('should return correct prompt and completion rates for Grok 3 models with prefixes', () => {
       expect(getMultiplier({ model: 'xai/grok-3', tokenType: 'prompt' })).toBe(
         tokenValues['grok-3'].prompt,
@@ -1345,39 +1240,6 @@ describe('Grok Model Tests - Pricing', () => {
         tokenValues['grok-4'].completion,
       );
     });
-
-    test('should return correct prompt and completion rates for Grok 4 Fast model with prefixes', () => {
-      expect(getMultiplier({ model: 'xai/grok-4-fast', tokenType: 'prompt' })).toBe(
-        tokenValues['grok-4-fast'].prompt,
-      );
-      expect(getMultiplier({ model: 'xai/grok-4-fast', tokenType: 'completion' })).toBe(
-        tokenValues['grok-4-fast'].completion,
-      );
-    });
-
-    test('should return correct prompt and completion rates for Grok 4.1 Fast models with prefixes', () => {
-      expect(getMultiplier({ model: 'xai/grok-4-1-fast-reasoning', tokenType: 'prompt' })).toBe(
-        tokenValues['grok-4-1-fast'].prompt,
-      );
-      expect(getMultiplier({ model: 'xai/grok-4-1-fast-reasoning', tokenType: 'completion' })).toBe(
-        tokenValues['grok-4-1-fast'].completion,
-      );
-      expect(getMultiplier({ model: 'xai/grok-4-1-fast-non-reasoning', tokenType: 'prompt' })).toBe(
-        tokenValues['grok-4-1-fast'].prompt,
-      );
-      expect(
-        getMultiplier({ model: 'xai/grok-4-1-fast-non-reasoning', tokenType: 'completion' }),
-      ).toBe(tokenValues['grok-4-1-fast'].completion);
-    });
-
-    test('should return correct prompt and completion rates for Grok Code Fast model with prefixes', () => {
-      expect(getMultiplier({ model: 'xai/grok-code-fast-1', tokenType: 'prompt' })).toBe(
-        tokenValues['grok-code-fast'].prompt,
-      );
-      expect(getMultiplier({ model: 'xai/grok-code-fast-1', tokenType: 'completion' })).toBe(
-        tokenValues['grok-code-fast'].completion,
-      );
-    });
   });
 });
 

api/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@librechat/backend",
-  "version": "v0.8.1-rc2",
+  "version": "v0.8.1-rc1",
   "description": "",
   "scripts": {
     "start": "echo 'please run this from the root directory'",
@@ -47,7 +47,7 @@
     "@langchain/google-genai": "^0.2.13",
     "@langchain/google-vertexai": "^0.2.13",
     "@langchain/textsplitters": "^0.1.0",
-    "@librechat/agents": "^3.0.36",
+    "@librechat/agents": "^3.0.32",
     "@librechat/api": "*",
     "@librechat/data-schemas": "*",
     "@microsoft/microsoft-graph-client": "^3.0.7",
@@ -92,7 +92,7 @@
     "multer": "^2.0.2",
     "nanoid": "^3.3.7",
     "node-fetch": "^2.7.0",
-    "nodemailer": "^7.0.11",
+    "nodemailer": "^7.0.9",
     "ollama": "^0.5.0",
     "openai": "5.8.2",
     "openid-client": "^6.5.0",

api/server/controllers/assistants/chatV1.js

@@ -1,7 +1,7 @@
 const { v4 } = require('uuid');
 const { sleep } = require('@librechat/agents');
 const { logger } = require('@librechat/data-schemas');
-const { sendEvent, getBalanceConfig, getModelMaxTokens, countTokens } = require('@librechat/api');
+const { sendEvent, getBalanceConfig, getModelMaxTokens } = require('@librechat/api');
 const {
   Time,
   Constants,
@@ -33,6 +33,7 @@ const { getTransactions } = require('~/models/Transaction');
 const { checkBalance } = require('~/models/balanceMethods');
 const { getConvo } = require('~/models/Conversation');
 const getLogStores = require('~/cache/getLogStores');
+const { countTokens } = require('~/server/utils');
 const { getOpenAIClient } = require('./helpers');
 
 /**

api/server/controllers/assistants/chatV2.js

@@ -1,7 +1,7 @@
 const { v4 } = require('uuid');
 const { sleep } = require('@librechat/agents');
 const { logger } = require('@librechat/data-schemas');
-const { sendEvent, getBalanceConfig, getModelMaxTokens, countTokens } = require('@librechat/api');
+const { sendEvent, getBalanceConfig, getModelMaxTokens } = require('@librechat/api');
 const {
   Time,
   Constants,
@@ -30,6 +30,7 @@ const { getTransactions } = require('~/models/Transaction');
 const { checkBalance } = require('~/models/balanceMethods');
 const { getConvo } = require('~/models/Conversation');
 const getLogStores = require('~/cache/getLogStores');
+const { countTokens } = require('~/server/utils');
 const { getOpenAIClient } = require('./helpers');
 
 /**

api/server/experimental.js

@@ -16,7 +16,6 @@ const {
   isEnabled,
   ErrorController,
   performStartupChecks,
-  handleJsonParseError,
   initializeFileStorage,
 } = require('@librechat/api');
 const { connectDb, indexSync } = require('~/db');
@@ -246,7 +245,6 @@ if (cluster.isMaster) {
     app.use(noIndex);
     app.use(express.json({ limit: '3mb' }));
     app.use(express.urlencoded({ extended: true, limit: '3mb' }));
-    app.use(handleJsonParseError);
     app.use(mongoSanitize());
     app.use(cors());
     app.use(cookieParser());
@@ -292,6 +290,7 @@ if (cluster.isMaster) {
     app.use('/api/presets', routes.presets);
     app.use('/api/prompts', routes.prompts);
     app.use('/api/categories', routes.categories);
+    app.use('/api/tokenizer', routes.tokenizer);
     app.use('/api/endpoints', routes.endpoints);
     app.use('/api/balance', routes.balance);
     app.use('/api/models', routes.models);

api/server/index.js

@@ -14,7 +14,6 @@ const {
   isEnabled,
   ErrorController,
   performStartupChecks,
-  handleJsonParseError,
   initializeFileStorage,
 } = require('@librechat/api');
 const { connectDb, indexSync } = require('~/db');
@@ -82,7 +81,6 @@ const startServer = async () => {
   app.use(noIndex);
   app.use(express.json({ limit: '3mb' }));
   app.use(express.urlencoded({ extended: true, limit: '3mb' }));
-  app.use(handleJsonParseError);
   app.use(mongoSanitize());
   app.use(cors());
   app.use(cookieParser());
@@ -128,6 +126,7 @@ const startServer = async () => {
   app.use('/api/presets', routes.presets);
   app.use('/api/prompts', routes.prompts);
   app.use('/api/categories', routes.categories);
+  app.use('/api/tokenizer', routes.tokenizer);
   app.use('/api/endpoints', routes.endpoints);
   app.use('/api/balance', routes.balance);
   app.use('/api/models', routes.models);

api/server/middleware/buildEndpointOption.js

@@ -61,24 +61,18 @@ async function buildEndpointOption(req, res, next) {
 
     try {
       currentModelSpec.preset.spec = spec;
+      if (currentModelSpec.iconURL != null && currentModelSpec.iconURL !== '') {
+        currentModelSpec.preset.iconURL = currentModelSpec.iconURL;
+      }
       parsedBody = parseCompactConvo({
         endpoint,
         endpointType,
         conversation: currentModelSpec.preset,
       });
-      if (currentModelSpec.iconURL != null && currentModelSpec.iconURL !== '') {
-        parsedBody.iconURL = currentModelSpec.iconURL;
-      }
     } catch (error) {
       logger.error(`Error parsing model spec for endpoint ${endpoint}`, error);
       return handleError(res, { text: 'Error parsing model spec' });
     }
-  } else if (parsedBody.spec && appConfig.modelSpecs?.list) {
-    // Non-enforced mode: if spec is selected, derive iconURL from model spec
-    const modelSpec = appConfig.modelSpecs.list.find((s) => s.name === parsedBody.spec);
-    if (modelSpec?.iconURL) {
-      parsedBody.iconURL = modelSpec.iconURL;
-    }
   }
 
   try {

api/server/routes/index.js

@@ -1,6 +1,7 @@
 const accessPermissions = require('./accessPermissions');
 const assistants = require('./assistants');
 const categories = require('./categories');
+const tokenizer = require('./tokenizer');
 const endpoints = require('./endpoints');
 const staticRoute = require('./static');
 const messages = require('./messages');
@@ -52,6 +53,7 @@ module.exports = {
   messages,
   memories,
   endpoints,
+  tokenizer,
   assistants,
   categories,
   staticRoute,

api/server/routes/messages.js

@@ -1,7 +1,7 @@
 const express = require('express');
+const { unescapeLaTeX } = require('@librechat/api');
 const { logger } = require('@librechat/data-schemas');
 const { ContentTypes } = require('librechat-data-provider');
-const { unescapeLaTeX, countTokens } = require('@librechat/api');
 const {
   saveConvo,
   getMessage,
@@ -14,6 +14,7 @@ const { findAllArtifacts, replaceArtifactContent } = require('~/server/services/
 const { requireJwtAuth, validateMessageReq } = require('~/server/middleware');
 const { cleanUpPrimaryKeyValue } = require('~/lib/utils/misc');
 const { getConvosQueried } = require('~/models/Conversation');
+const { countTokens } = require('~/server/utils');
 const { Message } = require('~/db/models');
 
 const router = express.Router();

api/server/routes/prompts.js

@@ -5,7 +5,6 @@ const {
   markPublicPromptGroups,
   buildPromptGroupFilter,
   formatPromptGroupsResponse,
-  safeValidatePromptGroupUpdate,
   createEmptyPromptGroupsResponse,
   filterAccessibleIdsBySharedLogic,
 } = require('@librechat/api');
@@ -345,16 +344,7 @@ const patchPromptGroup = async (req, res) => {
     if (req.user.role === SystemRoles.ADMIN) {
       delete filter.author;
     }
-
-    const validationResult = safeValidatePromptGroupUpdate(req.body);
-    if (!validationResult.success) {
-      return res.status(400).send({
-        error: 'Invalid request body',
-        details: validationResult.error.errors,
-      });
-    }
-
-    const promptGroup = await updatePromptGroup(filter, validationResult.data);
+    const promptGroup = await updatePromptGroup(filter, req.body);
     res.status(200).send(promptGroup);
   } catch (error) {
     logger.error(error);

api/server/routes/prompts.test.js

@@ -544,169 +544,6 @@ describe('Prompt Routes - ACL Permissions', () => {
     });
   });
 
-  describe('PATCH /api/prompts/groups/:groupId - Update Prompt Group Security', () => {
-    let testGroup;
-
-    beforeEach(async () => {
-      // Create a prompt group
-      testGroup = await PromptGroup.create({
-        name: 'Security Test Group',
-        category: 'security-test',
-        author: testUsers.owner._id,
-        authorName: testUsers.owner.name,
-        productionId: new ObjectId(),
-      });
-
-      // Grant owner permissions
-      await grantPermission({
-        principalType: PrincipalType.USER,
-        principalId: testUsers.owner._id,
-        resourceType: ResourceType.PROMPTGROUP,
-        resourceId: testGroup._id,
-        accessRoleId: AccessRoleIds.PROMPTGROUP_OWNER,
-        grantedBy: testUsers.owner._id,
-      });
-    });
-
-    afterEach(async () => {
-      await PromptGroup.deleteMany({});
-      await AclEntry.deleteMany({});
-    });
-
-    it('should allow updating allowed fields (name, category, oneliner)', async () => {
-      const updateData = {
-        name: 'Updated Group Name',
-        category: 'updated-category',
-        oneliner: 'Updated description',
-      };
-
-      const response = await request(app)
-        .patch(`/api/prompts/groups/${testGroup._id}`)
-        .send(updateData)
-        .expect(200);
-
-      expect(response.body.name).toBe(updateData.name);
-      expect(response.body.category).toBe(updateData.category);
-      expect(response.body.oneliner).toBe(updateData.oneliner);
-    });
-
-    it('should reject request with author field (400 Bad Request)', async () => {
-      const maliciousUpdate = {
-        name: 'Legit Update',
-        author: testUsers.noAccess._id.toString(), // Try to change ownership
-      };
-
-      const response = await request(app)
-        .patch(`/api/prompts/groups/${testGroup._id}`)
-        .send(maliciousUpdate)
-        .expect(400);
-
-      // Verify the request was rejected
-      expect(response.body.error).toBe('Invalid request body');
-      expect(response.body.details).toBeDefined();
-    });
-
-    it('should reject request with authorName field (400 Bad Request)', async () => {
-      const maliciousUpdate = {
-        name: 'Legit Update',
-        authorName: 'Malicious Author Name',
-      };
-
-      const response = await request(app)
-        .patch(`/api/prompts/groups/${testGroup._id}`)
-        .send(maliciousUpdate)
-        .expect(400);
-
-      // Verify the request was rejected
-      expect(response.body.error).toBe('Invalid request body');
-    });
-
-    it('should reject request with _id field (400 Bad Request)', async () => {
-      const newId = new ObjectId();
-      const maliciousUpdate = {
-        name: 'Legit Update',
-        _id: newId.toString(),
-      };
-
-      const response = await request(app)
-        .patch(`/api/prompts/groups/${testGroup._id}`)
-        .send(maliciousUpdate)
-        .expect(400);
-
-      // Verify the request was rejected
-      expect(response.body.error).toBe('Invalid request body');
-    });
-
-    it('should reject request with productionId field (400 Bad Request)', async () => {
-      const newProductionId = new ObjectId();
-      const maliciousUpdate = {
-        name: 'Legit Update',
-        productionId: newProductionId.toString(),
-      };
-
-      const response = await request(app)
-        .patch(`/api/prompts/groups/${testGroup._id}`)
-        .send(maliciousUpdate)
-        .expect(400);
-
-      // Verify the request was rejected
-      expect(response.body.error).toBe('Invalid request body');
-    });
-
-    it('should reject request with createdAt field (400 Bad Request)', async () => {
-      const maliciousDate = new Date('2020-01-01');
-      const maliciousUpdate = {
-        name: 'Legit Update',
-        createdAt: maliciousDate.toISOString(),
-      };
-
-      const response = await request(app)
-        .patch(`/api/prompts/groups/${testGroup._id}`)
-        .send(maliciousUpdate)
-        .expect(400);
-
-      // Verify the request was rejected
-      expect(response.body.error).toBe('Invalid request body');
-    });
-
-    it('should reject request with __v field (400 Bad Request)', async () => {
-      const maliciousUpdate = {
-        name: 'Legit Update',
-        __v: 999,
-      };
-
-      const response = await request(app)
-        .patch(`/api/prompts/groups/${testGroup._id}`)
-        .send(maliciousUpdate)
-        .expect(400);
-
-      // Verify the request was rejected
-      expect(response.body.error).toBe('Invalid request body');
-    });
-
-    it('should reject request with multiple sensitive fields (400 Bad Request)', async () => {
-      const maliciousUpdate = {
-        name: 'Legit Update',
-        author: testUsers.noAccess._id.toString(),
-        authorName: 'Hacker',
-        _id: new ObjectId().toString(),
-        productionId: new ObjectId().toString(),
-        createdAt: new Date('2020-01-01').toISOString(),
-        __v: 999,
-      };
-
-      const response = await request(app)
-        .patch(`/api/prompts/groups/${testGroup._id}`)
-        .send(maliciousUpdate)
-        .expect(400);
-
-      // Verify the request was rejected with validation errors
-      expect(response.body.error).toBe('Invalid request body');
-      expect(response.body.details).toBeDefined();
-      expect(Array.isArray(response.body.details)).toBe(true);
-    });
-  });
-
   describe('Pagination', () => {
     beforeEach(async () => {
       // Create multiple prompt groups for pagination testing

api/server/routes/share.js

@@ -99,13 +99,8 @@ router.get('/link/:conversationId', requireJwtAuth, async (req, res) => {
 
 router.post('/:conversationId', requireJwtAuth, async (req, res) => {
   try {
-    const { targetMessageId, expirationHours } = req.body;
-    const created = await createSharedLink(
-      req.user.id,
-      req.params.conversationId,
-      targetMessageId,
-      expirationHours,
-    );
+    const { targetMessageId } = req.body;
+    const created = await createSharedLink(req.user.id, req.params.conversationId, targetMessageId);
     if (created) {
       res.status(200).json(created);
     } else {

api/server/routes/tokenizer.js

@@ -0,0 +1,19 @@
+const express = require('express');
+const { logger } = require('@librechat/data-schemas');
+const requireJwtAuth = require('~/server/middleware/requireJwtAuth');
+const { countTokens } = require('~/server/utils');
+
+const router = express.Router();
+
+router.post('/', requireJwtAuth, async (req, res) => {
+  try {
+    const { arg } = req.body;
+    const count = await countTokens(arg?.text ?? arg);
+    res.send({ count });
+  } catch (e) {
+    logger.error('[/tokenizer] Error counting tokens', e);
+    res.status(500).json('Error counting tokens');
+  }
+});
+
+module.exports = router;

api/server/services/Endpoints/assistants/initalize.js

@@ -1,6 +1,5 @@
 const OpenAI = require('openai');
 const { ProxyAgent } = require('undici');
-const { isUserProvided } = require('@librechat/api');
 const { ErrorTypes, EModelEndpoint } = require('librechat-data-provider');
 const {
   getUserKeyValues,
@@ -8,6 +7,7 @@ const {
   checkUserKeyExpiry,
 } = require('~/server/services/UserService');
 const OAIClient = require('~/app/clients/OpenAIClient');
+const { isUserProvided } = require('~/server/utils');
 
 const initializeClient = async ({ req, res, endpointOption, version, initAppClient = false }) => {
   const { PROXY, OPENAI_ORGANIZATION, ASSISTANTS_API_KEY, ASSISTANTS_BASE_URL } = process.env;

api/server/services/Endpoints/index.js

@@ -12,13 +12,14 @@ const initGoogle = require('~/server/services/Endpoints/google/initialize');
  * @returns {boolean} - True if the provider is a known custom provider, false otherwise
  */
 function isKnownCustomProvider(provider) {
-  return [Providers.XAI, Providers.DEEPSEEK, Providers.OPENROUTER].includes(
+  return [Providers.XAI, Providers.OLLAMA, Providers.DEEPSEEK, Providers.OPENROUTER].includes(
     provider?.toLowerCase() || '',
   );
 }
 
 const providerConfigMap = {
   [Providers.XAI]: initCustom,
+  [Providers.OLLAMA]: initCustom,
   [Providers.DEEPSEEK]: initCustom,
   [Providers.OPENROUTER]: initCustom,
   [EModelEndpoint.openAI]: initOpenAI,

api/server/services/ModelService.js

@@ -1,14 +1,11 @@
 const axios = require('axios');
+const { Providers } = require('@librechat/agents');
 const { logger } = require('@librechat/data-schemas');
 const { HttpsProxyAgent } = require('https-proxy-agent');
-const { logAxiosError, inputSchema, processModelData, isUserProvided } = require('@librechat/api');
-const {
-  CacheKeys,
-  defaultModels,
-  KnownEndpoints,
-  EModelEndpoint,
-} = require('librechat-data-provider');
+const { logAxiosError, inputSchema, processModelData } = require('@librechat/api');
+const { EModelEndpoint, defaultModels, CacheKeys } = require('librechat-data-provider');
 const { OllamaClient } = require('~/app/clients/OllamaClient');
+const { isUserProvided } = require('~/server/utils');
 const getLogStores = require('~/cache/getLogStores');
 const { extractBaseURL } = require('~/utils');
 
@@ -71,7 +68,7 @@ const fetchModels = async ({
     return models;
   }
 
-  if (name && name.toLowerCase().startsWith(KnownEndpoints.ollama)) {
+  if (name && name.toLowerCase().startsWith(Providers.OLLAMA)) {
     try {
       return await OllamaClient.fetchModels(baseURL, { headers, user: userObject });
     } catch (ollamaError) {
@@ -106,7 +103,7 @@ const fetchModels = async ({
       options.headers['OpenAI-Organization'] = process.env.OPENAI_ORGANIZATION;
     }
 
-    const url = new URL(`${baseURL.replace(/\/+$/, '')}${azure ? '' : '/models'}`);
+    const url = new URL(`${baseURL}${azure ? '' : '/models'}`);
     if (user && userIdQuery) {
       url.searchParams.append('user', user);
     }

api/server/services/ModelService.spec.js

@@ -436,68 +436,6 @@ describe('fetchModels with Ollama specific logic', () => {
   });
 });
 
-describe('fetchModels URL construction with trailing slashes', () => {
-  beforeEach(() => {
-    axios.get.mockResolvedValue({
-      data: {
-        data: [{ id: 'model-1' }, { id: 'model-2' }],
-      },
-    });
-  });
-
-  afterEach(() => {
-    jest.clearAllMocks();
-  });
-
-  it('should not create double slashes when baseURL has a trailing slash', async () => {
-    await fetchModels({
-      user: 'user123',
-      apiKey: 'testApiKey',
-      baseURL: 'https://api.test.com/v1/',
-      name: 'TestAPI',
-    });
-
-    expect(axios.get).toHaveBeenCalledWith('https://api.test.com/v1/models', expect.any(Object));
-  });
-
-  it('should handle baseURL without trailing slash normally', async () => {
-    await fetchModels({
-      user: 'user123',
-      apiKey: 'testApiKey',
-      baseURL: 'https://api.test.com/v1',
-      name: 'TestAPI',
-    });
-
-    expect(axios.get).toHaveBeenCalledWith('https://api.test.com/v1/models', expect.any(Object));
-  });
-
-  it('should handle baseURL with multiple trailing slashes', async () => {
-    await fetchModels({
-      user: 'user123',
-      apiKey: 'testApiKey',
-      baseURL: 'https://api.test.com/v1///',
-      name: 'TestAPI',
-    });
-
-    expect(axios.get).toHaveBeenCalledWith('https://api.test.com/v1/models', expect.any(Object));
-  });
-
-  it('should correctly append query params after stripping trailing slashes', async () => {
-    await fetchModels({
-      user: 'user123',
-      apiKey: 'testApiKey',
-      baseURL: 'https://api.test.com/v1/',
-      name: 'TestAPI',
-      userIdQuery: true,
-    });
-
-    expect(axios.get).toHaveBeenCalledWith(
-      'https://api.test.com/v1/models?user=user123',
-      expect.any(Object),
-    );
-  });
-});
-
 describe('splitAndTrim', () => {
   it('should split a string by commas and trim each value', () => {
     const input = ' model1, model2 , model3,model4 ';

api/server/services/PermissionService.js

@@ -292,7 +292,7 @@ const ensurePrincipalExists = async function (principal) {
     let existingUser = await findUser({ idOnTheSource: principal.idOnTheSource });
 
     if (!existingUser) {
-      existingUser = await findUser({ email: principal.email });
+      existingUser = await findUser({ email: principal.email.toLowerCase() });
     }
 
     if (existingUser) {

api/server/services/Threads/manage.js

@@ -1,6 +1,5 @@
 const path = require('path');
 const { v4 } = require('uuid');
-const { countTokens, escapeRegExp } = require('@librechat/api');
 const {
   Constants,
   ContentTypes,
@@ -9,6 +8,7 @@ const {
 } = require('librechat-data-provider');
 const { retrieveAndProcessFile } = require('~/server/services/Files/process');
 const { recordMessage, getMessages } = require('~/models/Message');
+const { countTokens, escapeRegExp } = require('~/server/utils');
 const { spendTokens } = require('~/models/spendTokens');
 const { saveConvo } = require('~/models/Conversation');
 

api/server/utils/countTokens.js

@@ -0,0 +1,37 @@
+const { Tiktoken } = require('tiktoken/lite');
+const { logger } = require('@librechat/data-schemas');
+const p50k_base = require('tiktoken/encoders/p50k_base.json');
+const cl100k_base = require('tiktoken/encoders/cl100k_base.json');
+
+/**
+ * Counts the number of tokens in a given text using a specified encoding model.
+ *
+ * This function utilizes the 'Tiktoken' library to encode text based on the selected model.
+ * It supports two models, 'text-davinci-003' and 'gpt-3.5-turbo', each with its own encoding strategy.
+ * For 'text-davinci-003', the 'p50k_base' encoder is used, whereas for other models, the 'cl100k_base' encoder is applied.
+ * In case of an error during encoding, the error is logged, and the function returns 0.
+ *
+ * @async
+ * @param {string} text - The text to be tokenized. Defaults to an empty string if not provided.
+ * @param {string} modelName - The name of the model used for tokenizing. Defaults to 'gpt-3.5-turbo'.
+ * @returns {Promise<number>} The number of tokens in the provided text. Returns 0 if an error occurs.
+ * @throws Logs the error to a logger and rethrows if any error occurs during tokenization.
+ */
+const countTokens = async (text = '', modelName = 'gpt-3.5-turbo') => {
+  let encoder = null;
+  try {
+    const model = modelName.includes('text-davinci-003') ? p50k_base : cl100k_base;
+    encoder = new Tiktoken(model.bpe_ranks, model.special_tokens, model.pat_str);
+    const tokens = encoder.encode(text);
+    encoder.free();
+    return tokens.length;
+  } catch (e) {
+    logger.error('[countTokens]', e);
+    if (encoder) {
+      encoder.free();
+    }
+    return 0;
+  }
+};
+
+module.exports = countTokens;

api/server/utils/handleText.js

@@ -10,6 +10,14 @@ const {
 const { sendEvent } = require('@librechat/api');
 const partialRight = require('lodash/partialRight');
 
+/** Helper function to escape special characters in regex
+ * @param {string} string - The string to escape.
+ * @returns {string} The escaped string.
+ */
+function escapeRegExp(string) {
+  return string.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+}
+
 const addSpaceIfNeeded = (text) => (text.length > 0 && !text.endsWith(' ') ? text + ' ' : text);
 
 const base = { message: true, initial: true };
@@ -173,6 +181,7 @@ function generateConfig(key, baseURL, endpoint) {
 module.exports = {
   handleText,
   formatSteps,
+  escapeRegExp,
   formatAction,
   isUserProvided,
   generateConfig,

api/server/utils/index.js

@@ -1,4 +1,5 @@
 const removePorts = require('./removePorts');
+const countTokens = require('./countTokens');
 const handleText = require('./handleText');
 const sendEmail = require('./sendEmail');
 const queue = require('./queue');
@@ -6,6 +7,7 @@ const files = require('./files');
 
 module.exports = {
   ...handleText,
+  countTokens,
   removePorts,
   sendEmail,
   ...files,

api/strategies/socialLogin.test.js

@@ -172,7 +172,6 @@ describe('socialLogin', () => {
 
       /** Verify both searches happened */
       expect(findUser).toHaveBeenNthCalledWith(1, { googleId: googleId });
-      /** Email passed as-is; findUser implementation handles case normalization */
       expect(findUser).toHaveBeenNthCalledWith(2, { email: email });
       expect(findUser).toHaveBeenCalledTimes(2);
 

api/utils/tokens.spec.js

@@ -665,7 +665,7 @@ describe('Meta Models Tests', () => {
 
     test('should match Deepseek model variations', () => {
       expect(getModelMaxTokens('deepseek-chat')).toBe(
-        maxTokensMap[EModelEndpoint.openAI]['deepseek-chat'],
+        maxTokensMap[EModelEndpoint.openAI]['deepseek'],
       );
       expect(getModelMaxTokens('deepseek-coder')).toBe(
         maxTokensMap[EModelEndpoint.openAI]['deepseek'],
@@ -677,20 +677,6 @@ describe('Meta Models Tests', () => {
         maxTokensMap[EModelEndpoint.openAI]['deepseek.r1'],
       );
     });
-
-    test('should return 128000 context tokens for all DeepSeek models', () => {
-      expect(getModelMaxTokens('deepseek-chat')).toBe(128000);
-      expect(getModelMaxTokens('deepseek-reasoner')).toBe(128000);
-      expect(getModelMaxTokens('deepseek-r1')).toBe(128000);
-      expect(getModelMaxTokens('deepseek-v3')).toBe(128000);
-      expect(getModelMaxTokens('deepseek.r1')).toBe(128000);
-    });
-
-    test('should handle DeepSeek models with provider prefixes', () => {
-      expect(getModelMaxTokens('deepseek/deepseek-chat')).toBe(128000);
-      expect(getModelMaxTokens('openrouter/deepseek-reasoner')).toBe(128000);
-      expect(getModelMaxTokens('openai/deepseek-v3')).toBe(128000);
-    });
   });
 
   describe('matchModelName', () => {
@@ -719,42 +705,11 @@ describe('Meta Models Tests', () => {
     });
 
     test('should match Deepseek model variations', () => {
-      expect(matchModelName('deepseek-chat')).toBe('deepseek-chat');
+      expect(matchModelName('deepseek-chat')).toBe('deepseek');
       expect(matchModelName('deepseek-coder')).toBe('deepseek');
     });
   });
 
-  describe('DeepSeek Max Output Tokens', () => {
-    const { getModelMaxOutputTokens } = require('@librechat/api');
-
-    test('should return correct max output tokens for deepseek-chat', () => {
-      expect(getModelMaxOutputTokens('deepseek-chat')).toBe(8000);
-      expect(getModelMaxOutputTokens('deepseek-chat', EModelEndpoint.openAI)).toBe(8000);
-      expect(getModelMaxOutputTokens('deepseek-chat', EModelEndpoint.custom)).toBe(8000);
-    });
-
-    test('should return correct max output tokens for deepseek-reasoner', () => {
-      expect(getModelMaxOutputTokens('deepseek-reasoner')).toBe(64000);
-      expect(getModelMaxOutputTokens('deepseek-reasoner', EModelEndpoint.openAI)).toBe(64000);
-      expect(getModelMaxOutputTokens('deepseek-reasoner', EModelEndpoint.custom)).toBe(64000);
-    });
-
-    test('should return correct max output tokens for deepseek-r1', () => {
-      expect(getModelMaxOutputTokens('deepseek-r1')).toBe(64000);
-      expect(getModelMaxOutputTokens('deepseek-r1', EModelEndpoint.openAI)).toBe(64000);
-    });
-
-    test('should return correct max output tokens for deepseek base pattern', () => {
-      expect(getModelMaxOutputTokens('deepseek')).toBe(8000);
-      expect(getModelMaxOutputTokens('deepseek-v3')).toBe(8000);
-    });
-
-    test('should handle DeepSeek models with provider prefixes for max output tokens', () => {
-      expect(getModelMaxOutputTokens('deepseek/deepseek-chat')).toBe(8000);
-      expect(getModelMaxOutputTokens('openrouter/deepseek-reasoner')).toBe(64000);
-    });
-  });
-
   describe('processModelData with Meta models', () => {
     test('should process Meta model data correctly', () => {
       const input = {
@@ -823,16 +778,6 @@ describe('Grok Model Tests - Tokens', () => {
       expect(getModelMaxTokens('grok-4-0709')).toBe(256000);
     });
 
-    test('should return correct tokens for Grok 4 Fast and Grok 4.1 Fast models', () => {
-      expect(getModelMaxTokens('grok-4-fast')).toBe(2000000);
-      expect(getModelMaxTokens('grok-4-1-fast-reasoning')).toBe(2000000);
-      expect(getModelMaxTokens('grok-4-1-fast-non-reasoning')).toBe(2000000);
-    });
-
-    test('should return correct tokens for Grok Code Fast model', () => {
-      expect(getModelMaxTokens('grok-code-fast-1')).toBe(256000);
-    });
-
     test('should handle partial matches for Grok models with prefixes', () => {
       // Vision models should match before general models
       expect(getModelMaxTokens('xai/grok-2-vision-1212')).toBe(32768);
@@ -852,12 +797,6 @@ describe('Grok Model Tests - Tokens', () => {
       expect(getModelMaxTokens('xai/grok-3-mini-fast')).toBe(131072);
       // Grok 4 model
       expect(getModelMaxTokens('xai/grok-4-0709')).toBe(256000);
-      // Grok 4 Fast and 4.1 Fast models
-      expect(getModelMaxTokens('xai/grok-4-fast')).toBe(2000000);
-      expect(getModelMaxTokens('xai/grok-4-1-fast-reasoning')).toBe(2000000);
-      expect(getModelMaxTokens('xai/grok-4-1-fast-non-reasoning')).toBe(2000000);
-      // Grok Code Fast model
-      expect(getModelMaxTokens('xai/grok-code-fast-1')).toBe(256000);
     });
   });
 
@@ -881,12 +820,6 @@ describe('Grok Model Tests - Tokens', () => {
       expect(matchModelName('grok-3-mini-fast')).toBe('grok-3-mini-fast');
       // Grok 4 model
       expect(matchModelName('grok-4-0709')).toBe('grok-4');
-      // Grok 4 Fast and 4.1 Fast models
-      expect(matchModelName('grok-4-fast')).toBe('grok-4-fast');
-      expect(matchModelName('grok-4-1-fast-reasoning')).toBe('grok-4-1-fast');
-      expect(matchModelName('grok-4-1-fast-non-reasoning')).toBe('grok-4-1-fast');
-      // Grok Code Fast model
-      expect(matchModelName('grok-code-fast-1')).toBe('grok-code-fast');
     });
 
     test('should match Grok model variations with prefixes', () => {
@@ -908,12 +841,6 @@ describe('Grok Model Tests - Tokens', () => {
       expect(matchModelName('xai/grok-3-mini-fast')).toBe('grok-3-mini-fast');
       // Grok 4 model
       expect(matchModelName('xai/grok-4-0709')).toBe('grok-4');
-      // Grok 4 Fast and 4.1 Fast models
-      expect(matchModelName('xai/grok-4-fast')).toBe('grok-4-fast');
-      expect(matchModelName('xai/grok-4-1-fast-reasoning')).toBe('grok-4-1-fast');
-      expect(matchModelName('xai/grok-4-1-fast-non-reasoning')).toBe('grok-4-1-fast');
-      // Grok Code Fast model
-      expect(matchModelName('xai/grok-code-fast-1')).toBe('grok-code-fast');
     });
   });
 });

client/jest.config.cjs

@@ -1,4 +1,4 @@
-/** v0.8.1-rc2 */
+/** v0.8.1-rc1 */
 module.exports = {
   roots: ['<rootDir>/src'],
   testEnvironment: 'jsdom',

client/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@librechat/frontend",
-  "version": "v0.8.1-rc2",
+  "version": "v0.8.1-rc1",
   "description": "",
   "type": "module",
   "scripts": {
@@ -64,7 +64,6 @@
     "copy-to-clipboard": "^3.3.3",
     "cross-env": "^7.0.3",
     "date-fns": "^3.3.1",
-    "dompurify": "^3.3.0",
     "downloadjs": "^1.4.7",
     "export-from-json": "^1.7.2",
     "filenamify": "^6.0.0",

client/src/components/Bookmarks/BookmarkForm.tsx

@@ -72,7 +72,7 @@ const BookmarkForm = ({
     }
     const allTags =
       queryClient.getQueryData<TConversationTag[]>([QueryKeys.conversationTags]) ?? [];
-    if (allTags.some((tag) => tag.tag === data.tag && tag.tag !== bookmark?.tag)) {
+    if (allTags.some((tag) => tag.tag === data.tag)) {
       showToast({
         message: localize('com_ui_bookmarks_create_exists'),
         status: 'warning',

client/src/components/Bookmarks/__tests__/BookmarkForm.test.tsx

@@ -1,499 +0,0 @@
-import React, { createRef } from 'react';
-import { render, screen, fireEvent, waitFor, act } from '@testing-library/react';
-import '@testing-library/jest-dom/extend-expect';
-import BookmarkForm from '../BookmarkForm';
-import type { TConversationTag } from 'librechat-data-provider';
-
-const mockMutate = jest.fn();
-const mockShowToast = jest.fn();
-const mockGetQueryData = jest.fn();
-const mockSetOpen = jest.fn();
-
-jest.mock('~/hooks', () => ({
-  useLocalize: () => (key: string, params?: Record<string, unknown>) => {
-    const translations: Record<string, string> = {
-      com_ui_bookmarks_title: 'Title',
-      com_ui_bookmarks_description: 'Description',
-      com_ui_bookmarks_edit: 'Edit Bookmark',
-      com_ui_bookmarks_new: 'New Bookmark',
-      com_ui_bookmarks_create_exists: 'This bookmark already exists',
-      com_ui_bookmarks_add_to_conversation: 'Add to current conversation',
-      com_ui_bookmarks_tag_exists: 'A bookmark with this title already exists',
-      com_ui_field_required: 'This field is required',
-      com_ui_field_max_length: `${params?.field || 'Field'} must be less than ${params?.length || 0} characters`,
-    };
-    return translations[key] || key;
-  },
-}));
-
-jest.mock('@librechat/client', () => {
-  const ActualReact = jest.requireActual<typeof import('react')>('react');
-  return {
-    Checkbox: ({
-      checked,
-      onCheckedChange,
-      value,
-      ...props
-    }: {
-      checked: boolean;
-      onCheckedChange: (checked: boolean) => void;
-      value: string;
-    }) =>
-      ActualReact.createElement('input', {
-        type: 'checkbox',
-        checked,
-        onChange: (e: React.ChangeEvent<HTMLInputElement>) => onCheckedChange(e.target.checked),
-        value,
-        ...props,
-      }),
-    Label: ({ children, ...props }: { children: React.ReactNode }) =>
-      ActualReact.createElement('label', props, children),
-    TextareaAutosize: ActualReact.forwardRef<
-      HTMLTextAreaElement,
-      React.TextareaHTMLAttributes<HTMLTextAreaElement>
-    >((props, ref) => ActualReact.createElement('textarea', { ref, ...props })),
-    Input: ActualReact.forwardRef<HTMLInputElement, React.InputHTMLAttributes<HTMLInputElement>>(
-      (props, ref) => ActualReact.createElement('input', { ref, ...props }),
-    ),
-    useToastContext: () => ({
-      showToast: mockShowToast,
-    }),
-  };
-});
-
-jest.mock('~/Providers/BookmarkContext', () => ({
-  useBookmarkContext: () => ({
-    bookmarks: [],
-  }),
-}));
-
-jest.mock('@tanstack/react-query', () => ({
-  useQueryClient: () => ({
-    getQueryData: mockGetQueryData,
-  }),
-}));
-
-jest.mock('~/utils', () => ({
-  cn: (...classes: (string | undefined | null | boolean)[]) => classes.filter(Boolean).join(' '),
-  logger: {
-    log: jest.fn(),
-  },
-}));
-
-const createMockBookmark = (overrides?: Partial<TConversationTag>): TConversationTag => ({
-  _id: 'bookmark-1',
-  user: 'user-1',
-  tag: 'Test Bookmark',
-  description: 'Test description',
-  createdAt: '2024-01-01T00:00:00.000Z',
-  updatedAt: '2024-01-01T00:00:00.000Z',
-  count: 1,
-  position: 0,
-  ...overrides,
-});
-
-const createMockMutation = (isLoading = false) => ({
-  mutate: mockMutate,
-  isLoading,
-  isError: false,
-  isSuccess: false,
-  data: undefined,
-  error: null,
-  reset: jest.fn(),
-  mutateAsync: jest.fn(),
-  status: 'idle' as const,
-  variables: undefined,
-  context: undefined,
-  failureCount: 0,
-  failureReason: null,
-  isPaused: false,
-  isIdle: true,
-  submittedAt: 0,
-});
-
-describe('BookmarkForm - Bookmark Editing', () => {
-  const formRef = createRef<HTMLFormElement>();
-
-  beforeEach(() => {
-    jest.clearAllMocks();
-    mockGetQueryData.mockReturnValue([]);
-  });
-
-  describe('Editing only the description (tag unchanged)', () => {
-    it('should allow submitting when only the description is changed', async () => {
-      const existingBookmark = createMockBookmark({
-        tag: 'My Bookmark',
-        description: 'Original description',
-      });
-
-      mockGetQueryData.mockReturnValue([existingBookmark]);
-
-      render(
-        <BookmarkForm
-          bookmark={existingBookmark}
-          mutation={
-            createMockMutation() as ReturnType<
-              typeof import('~/data-provider').useConversationTagMutation
-            >
-          }
-          setOpen={mockSetOpen}
-          formRef={formRef}
-        />,
-      );
-
-      const descriptionInput = screen.getByRole('textbox', { name: /description/i });
-
-      await act(async () => {
-        fireEvent.change(descriptionInput, { target: { value: 'Updated description' } });
-      });
-
-      await act(async () => {
-        fireEvent.submit(formRef.current!);
-      });
-
-      await waitFor(() => {
-        expect(mockMutate).toHaveBeenCalledWith(
-          expect.objectContaining({
-            tag: 'My Bookmark',
-            description: 'Updated description',
-          }),
-        );
-      });
-      expect(mockShowToast).not.toHaveBeenCalled();
-      expect(mockSetOpen).toHaveBeenCalledWith(false);
-    });
-
-    it('should not submit when both tag and description are unchanged', async () => {
-      const existingBookmark = createMockBookmark({
-        tag: 'My Bookmark',
-        description: 'Same description',
-      });
-
-      mockGetQueryData.mockReturnValue([existingBookmark]);
-
-      render(
-        <BookmarkForm
-          bookmark={existingBookmark}
-          mutation={
-            createMockMutation() as ReturnType<
-              typeof import('~/data-provider').useConversationTagMutation
-            >
-          }
-          setOpen={mockSetOpen}
-          formRef={formRef}
-        />,
-      );
-
-      await act(async () => {
-        fireEvent.submit(formRef.current!);
-      });
-
-      await waitFor(() => {
-        expect(mockMutate).not.toHaveBeenCalled();
-      });
-      expect(mockSetOpen).not.toHaveBeenCalled();
-    });
-  });
-
-  describe('Renaming a tag to an existing tag (should show error)', () => {
-    it('should show error toast when renaming to an existing tag name (via allTags)', async () => {
-      const existingBookmark = createMockBookmark({
-        tag: 'Original Tag',
-        description: 'Description',
-      });
-
-      const otherBookmark = createMockBookmark({
-        _id: 'bookmark-2',
-        tag: 'Existing Tag',
-        description: 'Other description',
-      });
-
-      mockGetQueryData.mockReturnValue([existingBookmark, otherBookmark]);
-
-      render(
-        <BookmarkForm
-          bookmark={existingBookmark}
-          mutation={
-            createMockMutation() as ReturnType<
-              typeof import('~/data-provider').useConversationTagMutation
-            >
-          }
-          setOpen={mockSetOpen}
-          formRef={formRef}
-        />,
-      );
-
-      const tagInput = screen.getByLabelText('Edit Bookmark');
-
-      await act(async () => {
-        fireEvent.change(tagInput, { target: { value: 'Existing Tag' } });
-      });
-
-      await act(async () => {
-        fireEvent.submit(formRef.current!);
-      });
-
-      await waitFor(() => {
-        expect(mockShowToast).toHaveBeenCalledWith({
-          message: 'This bookmark already exists',
-          status: 'warning',
-        });
-      });
-      expect(mockMutate).not.toHaveBeenCalled();
-      expect(mockSetOpen).not.toHaveBeenCalled();
-    });
-
-    it('should show error toast when renaming to an existing tag name (via tags prop)', async () => {
-      const existingBookmark = createMockBookmark({
-        tag: 'Original Tag',
-        description: 'Description',
-      });
-
-      mockGetQueryData.mockReturnValue([existingBookmark]);
-
-      render(
-        <BookmarkForm
-          tags={['Existing Tag', 'Another Tag']}
-          bookmark={existingBookmark}
-          mutation={
-            createMockMutation() as ReturnType<
-              typeof import('~/data-provider').useConversationTagMutation
-            >
-          }
-          setOpen={mockSetOpen}
-          formRef={formRef}
-        />,
-      );
-
-      const tagInput = screen.getByLabelText('Edit Bookmark');
-
-      await act(async () => {
-        fireEvent.change(tagInput, { target: { value: 'Existing Tag' } });
-      });
-
-      await act(async () => {
-        fireEvent.submit(formRef.current!);
-      });
-
-      await waitFor(() => {
-        expect(mockShowToast).toHaveBeenCalledWith({
-          message: 'This bookmark already exists',
-          status: 'warning',
-        });
-      });
-      expect(mockMutate).not.toHaveBeenCalled();
-      expect(mockSetOpen).not.toHaveBeenCalled();
-    });
-  });
-
-  describe('Renaming a tag to a new tag (should succeed)', () => {
-    it('should allow renaming to a completely new tag name', async () => {
-      const existingBookmark = createMockBookmark({
-        tag: 'Original Tag',
-        description: 'Description',
-      });
-
-      mockGetQueryData.mockReturnValue([existingBookmark]);
-
-      render(
-        <BookmarkForm
-          bookmark={existingBookmark}
-          mutation={
-            createMockMutation() as ReturnType<
-              typeof import('~/data-provider').useConversationTagMutation
-            >
-          }
-          setOpen={mockSetOpen}
-          formRef={formRef}
-        />,
-      );
-
-      const tagInput = screen.getByLabelText('Edit Bookmark');
-
-      await act(async () => {
-        fireEvent.change(tagInput, { target: { value: 'Brand New Tag' } });
-      });
-
-      await act(async () => {
-        fireEvent.submit(formRef.current!);
-      });
-
-      await waitFor(() => {
-        expect(mockMutate).toHaveBeenCalledWith(
-          expect.objectContaining({
-            tag: 'Brand New Tag',
-            description: 'Description',
-          }),
-        );
-      });
-      expect(mockShowToast).not.toHaveBeenCalled();
-      expect(mockSetOpen).toHaveBeenCalledWith(false);
-    });
-
-    it('should allow keeping the same tag name when editing (not trigger duplicate error)', async () => {
-      const existingBookmark = createMockBookmark({
-        tag: 'My Bookmark',
-        description: 'Original description',
-      });
-
-      mockGetQueryData.mockReturnValue([existingBookmark]);
-
-      render(
-        <BookmarkForm
-          bookmark={existingBookmark}
-          mutation={
-            createMockMutation() as ReturnType<
-              typeof import('~/data-provider').useConversationTagMutation
-            >
-          }
-          setOpen={mockSetOpen}
-          formRef={formRef}
-        />,
-      );
-
-      const descriptionInput = screen.getByRole('textbox', { name: /description/i });
-
-      await act(async () => {
-        fireEvent.change(descriptionInput, { target: { value: 'New description' } });
-      });
-
-      await act(async () => {
-        fireEvent.submit(formRef.current!);
-      });
-
-      await waitFor(() => {
-        expect(mockMutate).toHaveBeenCalledWith(
-          expect.objectContaining({
-            tag: 'My Bookmark',
-            description: 'New description',
-          }),
-        );
-      });
-      expect(mockShowToast).not.toHaveBeenCalled();
-    });
-  });
-
-  describe('Validation interaction between different data sources', () => {
-    it('should check both tags prop and allTags query data for duplicates', async () => {
-      const existingBookmark = createMockBookmark({
-        tag: 'Original Tag',
-        description: 'Description',
-      });
-
-      const queryDataBookmark = createMockBookmark({
-        _id: 'bookmark-query',
-        tag: 'Query Data Tag',
-      });
-
-      mockGetQueryData.mockReturnValue([existingBookmark, queryDataBookmark]);
-
-      render(
-        <BookmarkForm
-          tags={['Props Tag']}
-          bookmark={existingBookmark}
-          mutation={
-            createMockMutation() as ReturnType<
-              typeof import('~/data-provider').useConversationTagMutation
-            >
-          }
-          setOpen={mockSetOpen}
-          formRef={formRef}
-        />,
-      );
-
-      const tagInput = screen.getByLabelText('Edit Bookmark');
-
-      await act(async () => {
-        fireEvent.change(tagInput, { target: { value: 'Props Tag' } });
-      });
-
-      await act(async () => {
-        fireEvent.submit(formRef.current!);
-      });
-
-      await waitFor(() => {
-        expect(mockShowToast).toHaveBeenCalledWith({
-          message: 'This bookmark already exists',
-          status: 'warning',
-        });
-      });
-      expect(mockMutate).not.toHaveBeenCalled();
-    });
-
-    it('should not trigger mutation when mutation is loading', async () => {
-      const existingBookmark = createMockBookmark({
-        tag: 'My Bookmark',
-        description: 'Description',
-      });
-
-      mockGetQueryData.mockReturnValue([existingBookmark]);
-
-      render(
-        <BookmarkForm
-          bookmark={existingBookmark}
-          mutation={
-            createMockMutation(true) as ReturnType<
-              typeof import('~/data-provider').useConversationTagMutation
-            >
-          }
-          setOpen={mockSetOpen}
-          formRef={formRef}
-        />,
-      );
-
-      const descriptionInput = screen.getByRole('textbox', { name: /description/i });
-
-      await act(async () => {
-        fireEvent.change(descriptionInput, { target: { value: 'Updated description' } });
-      });
-
-      await act(async () => {
-        fireEvent.submit(formRef.current!);
-      });
-
-      await waitFor(() => {
-        expect(mockMutate).not.toHaveBeenCalled();
-      });
-    });
-
-    it('should handle empty allTags gracefully', async () => {
-      const existingBookmark = createMockBookmark({
-        tag: 'My Bookmark',
-        description: 'Description',
-      });
-
-      mockGetQueryData.mockReturnValue(null);
-
-      render(
-        <BookmarkForm
-          bookmark={existingBookmark}
-          mutation={
-            createMockMutation() as ReturnType<
-              typeof import('~/data-provider').useConversationTagMutation
-            >
-          }
-          setOpen={mockSetOpen}
-          formRef={formRef}
-        />,
-      );
-
-      const tagInput = screen.getByLabelText('Edit Bookmark');
-
-      await act(async () => {
-        fireEvent.change(tagInput, { target: { value: 'New Tag' } });
-      });
-
-      await act(async () => {
-        fireEvent.submit(formRef.current!);
-      });
-
-      await waitFor(() => {
-        expect(mockMutate).toHaveBeenCalledWith(
-          expect.objectContaining({
-            tag: 'New Tag',
-          }),
-        );
-      });
-    });
-  });
-});

client/src/components/Chat/Input/ChatForm.tsx

@@ -260,50 +260,37 @@ const ChatForm = memo(({ index = 0 }: { index?: number }) => {
             <FileFormChat conversation={conversation} />
             {endpoint && (
               <div className={cn('flex', isRTL ? 'flex-row-reverse' : 'flex-row')}>
-                <div className="relative flex-1">
-                  <TextareaAutosize
-                    {...registerProps}
-                    ref={(e) => {
-                      ref(e);
-                      (textAreaRef as React.MutableRefObject<HTMLTextAreaElement | null>).current =
-                        e;
-                    }}
-                    disabled={disableInputs || isNotAppendable}
-                    onPaste={handlePaste}
-                    onKeyDown={handleKeyDown}
-                    onKeyUp={handleKeyUp}
-                    onCompositionStart={handleCompositionStart}
-                    onCompositionEnd={handleCompositionEnd}
-                    id={mainTextareaId}
-                    tabIndex={0}
-                    data-testid="text-input"
-                    rows={1}
-                    onFocus={() => {
-                      handleFocusOrClick();
-                      setIsTextAreaFocused(true);
-                    }}
-                    onBlur={setIsTextAreaFocused.bind(null, false)}
-                    aria-label={localize('com_ui_message_input')}
-                    onClick={handleFocusOrClick}
-                    style={{ height: 44, overflowY: 'auto' }}
-                    className={cn(
-                      baseClasses,
-                      removeFocusRings,
-                      'scrollbar-hover transition-[max-height] duration-200 disabled:cursor-not-allowed',
-                    )}
-                  />
-                  {isCollapsed && (
-                    <div
-                      className="pointer-events-none absolute bottom-0 left-0 right-0 h-10 transition-all duration-200"
-                      style={{
-                        backdropFilter: 'blur(2px)',
-                        WebkitMaskImage: 'linear-gradient(to top, black 15%, transparent 75%)',
-                        maskImage: 'linear-gradient(to top, black 15%, transparent 75%)',
-                      }}
-                    />
+                <TextareaAutosize
+                  {...registerProps}
+                  ref={(e) => {
+                    ref(e);
+                    (textAreaRef as React.MutableRefObject<HTMLTextAreaElement | null>).current = e;
+                  }}
+                  disabled={disableInputs || isNotAppendable}
+                  onPaste={handlePaste}
+                  onKeyDown={handleKeyDown}
+                  onKeyUp={handleKeyUp}
+                  onCompositionStart={handleCompositionStart}
+                  onCompositionEnd={handleCompositionEnd}
+                  id={mainTextareaId}
+                  tabIndex={0}
+                  data-testid="text-input"
+                  rows={1}
+                  onFocus={() => {
+                    handleFocusOrClick();
+                    setIsTextAreaFocused(true);
+                  }}
+                  onBlur={setIsTextAreaFocused.bind(null, false)}
+                  aria-label={localize('com_ui_message_input')}
+                  onClick={handleFocusOrClick}
+                  style={{ height: 44, overflowY: 'auto' }}
+                  className={cn(
+                    baseClasses,
+                    removeFocusRings,
+                    'transition-[max-height] duration-200 disabled:cursor-not-allowed',
                   )}
-                </div>
-                <div className="flex flex-col items-start justify-start pr-2.5 pt-1.5">
+                />
+                <div className="flex flex-col items-start justify-start pt-1.5">
                   <CollapseChat
                     isCollapsed={isCollapsed}
                     isScrollable={isMoreThanThreeRows}

client/src/components/Chat/Messages/Content/Parts/OpenAIImageGen/OpenAIImageGen.tsx

@@ -145,7 +145,8 @@ export default function OpenAIImageGen({
         clearInterval(intervalRef.current);
       }
     };
-  }, [isSubmitting, initialProgress, quality]);
+    // eslint-disable-next-line react-hooks/exhaustive-deps
+  }, [initialProgress, quality]);
 
   useEffect(() => {
     if (initialProgress >= 1 || cancelled) {

client/src/components/Chat/Messages/HoverButtons.tsx

@@ -45,9 +45,6 @@ const extractMessageContent = (message: TMessage): string => {
   if (Array.isArray(message.content)) {
     return message.content
       .map((part) => {
-        if (part == null) {
-          return '';
-        }
         if (typeof part === 'string') {
           return part;
         }

client/src/components/Conversations/Conversations.tsx

@@ -150,7 +150,7 @@ const Conversations: FC<ConversationsProps> = ({
       return (
         <CellMeasurer cache={cache} columnIndex={0} key={key} parent={parent} rowIndex={index}>
           {({ registerChild }) => (
-            <div ref={registerChild} style={style}>
+            <div ref={registerChild} style={style} className="px-1">
               {rendering}
             </div>
           )}

client/src/components/Conversations/Convo.tsx

@@ -133,7 +133,9 @@ export default function Conversation({ conversation, retainView, toggleNav }: Co
     <div
       className={cn(
         'group relative flex h-12 w-full items-center rounded-lg transition-colors duration-200 md:h-9',
-        isActiveConvo ? 'bg-surface-active-alt' : 'hover:bg-surface-active-alt',
+        isActiveConvo
+          ? 'bg-surface-active-alt outline outline-2 outline-offset-[-2px]'
+          : 'hover:bg-surface-active-alt',
       )}
       role="button"
       tabIndex={renaming ? -1 : 0}

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

@@ -55,11 +55,6 @@ export function DeleteConversationDialog({
       }
       setMenuOpen?.(false);
       retainView();
-      showToast({
-        message: localize('com_ui_convo_delete_success'),
-        severity: NotificationSeverity.SUCCESS,
-        showIcon: true,
-      });
     },
     onError: () => {
       showToast({

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

@@ -6,7 +6,6 @@ import {
   Spinner,
   TooltipAnchor,
   Label,
-  Dropdown,
   OGDialogTemplate,
   useToastContext,
 } from '@librechat/client';
@@ -18,7 +17,6 @@ import {
 } from '~/data-provider';
 import { NotificationSeverity } from '~/common';
 import { useLocalize } from '~/hooks';
-import { SHARE_EXPIRY } from '~/utils/shareExpiry';
 
 export default function SharedLinkButton({
   share,
@@ -40,12 +38,8 @@ export default function SharedLinkButton({
   const localize = useLocalize();
   const { showToast } = useToastContext();
   const [showDeleteDialog, setShowDeleteDialog] = useState(false);
-  const [expiryLabel, setExpiryLabel] = useState(SHARE_EXPIRY.THIRTY_DAYS.label);
   const shareId = share?.shareId ?? '';
 
-  const expirationOptions = Object.values(SHARE_EXPIRY);
-  const localizedOptions = expirationOptions.map((option) => localize(option.label));
-
   const { mutateAsync: mutate, isLoading: isCreateLoading } = useCreateSharedLinkMutation({
     onError: () => {
       showToast({
@@ -94,9 +88,7 @@ export default function SharedLinkButton({
   };
 
   const createShareLink = async () => {
-    const selectedOption = expirationOptions.find((option) => option.label === expiryLabel);
-    const expirationHours = selectedOption ? selectedOption.hours : undefined;
-    const share = await mutate({ conversationId, targetMessageId, expirationHours });
+    const share = await mutate({ conversationId, targetMessageId });
     const newLink = generateShareLink(share.shareId);
     setSharedLink(newLink);
   };
@@ -125,33 +117,12 @@ export default function SharedLinkButton({
 
   return (
     <>
-      <div className="flex flex-col gap-2">
+      <div className="flex gap-2">
         {!shareId && (
-          <>
-            <div className="flex items-center gap-2">
-              <Label htmlFor="expiry-select" className="text-sm">
-                {localize('com_ui_expires')}:
-              </Label>
-              <Dropdown
-                id="expiry-select"
-                label=""
-                value={localize(expiryLabel)}
-                onChange={(value) => {
-                  const option = expirationOptions.find((opt) => localize(opt.label) === value);
-                  if (option) {
-                    setExpiryLabel(option.label);
-                  }
-                }}
-                options={localizedOptions}
-                sizeClasses="flex-1"
-                portal={false}
-              />
-            </div>
-            <Button disabled={isCreateLoading} variant="submit" onClick={createShareLink}>
-              {!isCreateLoading && localize('com_ui_create_link')}
-              {isCreateLoading && <Spinner className="size-4" />}
-            </Button>
-          </>
+          <Button disabled={isCreateLoading} variant="submit" onClick={createShareLink}>
+            {!isCreateLoading && localize('com_ui_create_link')}
+            {isCreateLoading && <Spinner className="size-4" />}
+          </Button>
         )}
         {shareId && (
           <div className="flex items-center gap-2">

client/src/components/MCP/CustomUserVarsSection.tsx

@@ -1,7 +1,6 @@
 import React, { useMemo } from 'react';
-import DOMPurify from 'dompurify';
 import { useForm, Controller } from 'react-hook-form';
-import { Input, Label, Button } from '@librechat/client';
+import { Input, Label, Button, TooltipAnchor, CircleHelpIcon } from '@librechat/client';
 import { useMCPAuthValuesQuery } from '~/data-provider/Tools/queries';
 import { useLocalize } from '~/hooks';
 
@@ -28,40 +27,21 @@ interface AuthFieldProps {
 function AuthField({ name, config, hasValue, control, errors }: AuthFieldProps) {
   const localize = useLocalize();
 
-  const sanitizer = useMemo(() => {
-    const instance = DOMPurify();
-    instance.addHook('afterSanitizeAttributes', (node) => {
-      if (node.tagName && node.tagName === 'A') {
-        node.setAttribute('target', '_blank');
-        node.setAttribute('rel', 'noopener noreferrer');
-      }
-    });
-    return instance;
-  }, []);
-
-  const sanitizedDescription = useMemo(() => {
-    if (!config.description) {
-      return '';
-    }
-    try {
-      return sanitizer.sanitize(config.description, {
-        ALLOWED_TAGS: ['a', 'strong', 'b', 'em', 'i', 'br', 'code'],
-        ALLOWED_ATTR: ['href', 'class', 'target', 'rel'],
-        ALLOW_DATA_ATTR: false,
-        ALLOW_ARIA_ATTR: false,
-      });
-    } catch (error) {
-      console.error('Sanitization failed', error);
-      return config.description;
-    }
-  }, [config.description, sanitizer]);
-
   return (
     <div className="space-y-2">
       <div className="flex items-center justify-between">
-        <Label htmlFor={name} className="text-sm font-medium">
-          {config.title}
-        </Label>
+        <TooltipAnchor
+          enableHTML={true}
+          description={config.description || ''}
+          render={
+            <div className="flex items-center gap-2">
+              <Label htmlFor={name} className="text-sm font-medium">
+                {config.title}
+              </Label>
+              <CircleHelpIcon className="h-6 w-6 cursor-help text-text-secondary transition-colors hover:text-text-primary" />
+            </div>
+          }
+        />
         {hasValue ? (
           <div className="flex min-w-fit items-center gap-2 whitespace-nowrap rounded-full border border-border-light px-2 py-0.5 text-xs font-medium text-text-secondary">
             <div className="h-1.5 w-1.5 rounded-full bg-green-500" />
@@ -86,18 +66,12 @@ function AuthField({ name, config, hasValue, control, errors }: AuthFieldProps)
             placeholder={
               hasValue
                 ? localize('com_ui_mcp_update_var', { 0: config.title })
-                : localize('com_ui_mcp_enter_var', { 0: config.title })
+                : `${localize('com_ui_mcp_enter_var', { 0: config.title })} ${localize('com_ui_optional')}`
             }
             className="w-full rounded border border-border-medium bg-transparent px-2 py-1 text-text-primary placeholder:text-text-secondary focus:outline-none sm:text-sm"
           />
         )}
       />
-      {sanitizedDescription && (
-        <p
-          className="text-xs text-text-secondary [&_a]:text-blue-500 [&_a]:hover:underline"
-          dangerouslySetInnerHTML={{ __html: sanitizedDescription }}
-        />
-      )}
       {errors[name] && <p className="text-xs text-red-500">{errors[name]?.message}</p>}
     </div>
   );

client/src/components/Nav/SettingsTabs/Data/SharedLinks.tsx

@@ -232,28 +232,6 @@ export default function SharedLinks() {
           mobileSize: '20%',
         },
       },
-      {
-        accessorKey: 'expiresAt',
-        header: () => <Label className="px-2 text-xs sm:text-sm">{localize('com_ui_expires')}</Label>,
-        cell: ({ row }) => {
-          const expiresAt = row.original.expiresAt;
-          if (!expiresAt) {
-            return <span className="text-xs text-text-secondary sm:text-sm">{localize('com_ui_never')}</span>;
-          }
-          const expiryDate = new Date(expiresAt);
-          const isExpired = expiryDate < new Date();
-          return (
-            <span className={`text-xs sm:text-sm ${isExpired ? 'text-red-500' : 'text-text-secondary'}`}>
-              {isExpired ? `${localize('com_ui_expired')} ` : ''}
-              {formatDate(expiresAt.toString(), isSmallScreen)}
-            </span>
-          );
-        },
-        meta: {
-          size: '10%',
-          mobileSize: '20%',
-        },
-      },
       {
         accessorKey: 'actions',
         header: () => (

client/src/components/Nav/SettingsTabs/General/ArchivedChatsTable.tsx

@@ -93,11 +93,6 @@ export default function ArchivedChatsTable({
     onSuccess: async () => {
       setIsDeleteOpen(false);
       await refetch();
-      showToast({
-        message: localize('com_ui_convo_delete_success'),
-        severity: NotificationSeverity.SUCCESS,
-        showIcon: true,
-      });
     },
     onError: (error: unknown) => {
       showToast({

client/src/data-provider/mutations.ts

@@ -170,32 +170,24 @@ export const useArchiveConvoMutation = (
 export const useCreateSharedLinkMutation = (
   options?: t.MutationOptions<
     t.TCreateShareLinkRequest,
-    { conversationId: string; targetMessageId?: string; expirationHours?: number }
+    { conversationId: string; targetMessageId?: string }
   >,
 ): UseMutationResult<
   t.TSharedLinkResponse,
   unknown,
-  { conversationId: string; targetMessageId?: string; expirationHours?: number },
+  { conversationId: string; targetMessageId?: string },
   unknown
 > => {
   const queryClient = useQueryClient();
 
   const { onSuccess, ..._options } = options || {};
   return useMutation(
-    ({
-      conversationId,
-      targetMessageId,
-      expirationHours,
-    }: {
-      conversationId: string;
-      targetMessageId?: string;
-      expirationHours?: number;
-    }) => {
+    ({ conversationId, targetMessageId }: { conversationId: string; targetMessageId?: string }) => {
       if (!conversationId) {
         throw new Error('Conversation ID is required');
       }
 
-      return dataService.createSharedLink(conversationId, targetMessageId, expirationHours);
+      return dataService.createSharedLink(conversationId, targetMessageId);
     },
     {
       onSuccess: (_data: t.TSharedLinkResponse, vars, context) => {

client/src/hooks/Chat/useChatFunctions.ts

@@ -168,7 +168,6 @@ export default function useChatFunctions({
 
     const endpointsConfig = queryClient.getQueryData<TEndpointsConfig>([QueryKeys.endpoints]);
     const endpointType = getEndpointField(endpointsConfig, endpoint, 'type');
-    const iconURL = conversation?.iconURL;
 
     /** This becomes part of the `endpointOption` */
     const convo = parseCompactConvo({
@@ -249,9 +248,9 @@ export default function useChatFunctions({
       conversationId,
       unfinished: false,
       isCreatedByUser: false,
+      iconURL: convo?.iconURL,
       model: convo?.model,
       error: false,
-      iconURL,
     };
 
     if (isAssistantsEndpoint(endpoint)) {

client/src/hooks/Conversations/useExportConversation.ts

@@ -73,9 +73,7 @@ export default function useExportConversation({
     }
 
     return message.content
-      .filter((content) => content != null)
       .map((content) => getMessageContent(message.sender || '', content))
-      .filter((text) => text.length > 0)
       .map((text) => {
         return formatText(text[0], text[1]);
       })
@@ -105,7 +103,7 @@ export default function useExportConversation({
     if (content.type === ContentTypes.TEXT) {
       // TEXT
       const textPart = content[ContentTypes.TEXT];
-      const text = typeof textPart === 'string' ? textPart : (textPart?.value ?? '');
+      const text = typeof textPart === 'string' ? textPart : textPart.value;
       return [sender, text];
     }
 
@@ -367,10 +365,12 @@ export default function useExportConversation({
       data['messages'] = messages;
     }
 
-    /** Use JSON.stringify without indentation to minimize file size for deeply nested recursive exports */
-    const jsonString = JSON.stringify(data);
-    const blob = new Blob([jsonString], { type: 'application/json;charset=utf-8' });
-    download(blob, `${filename}.json`, 'application/json');
+    exportFromJSON({
+      data: data,
+      fileName: filename,
+      extension: 'json',
+      exportType: exportFromJSON.types.json,
+    });
   };
 
   const exportConversation = () => {

client/src/hooks/SSE/useContentHandler.ts

@@ -33,8 +33,9 @@ export default function useContentHandler({ setMessages, getMessages }: TUseCont
 
       const _messages = getMessages();
       const messages =
-        _messages?.filter((m) => m.messageId !== messageId).map((msg) => ({ ...msg, thread_id })) ??
-        [];
+        _messages
+          ?.filter((m) => m.messageId !== messageId)
+          .map((msg) => ({ ...msg, thread_id })) ?? [];
       const userMessage = messages[messages.length - 1] as TMessage | undefined;
 
       const { initialResponse } = submission;
@@ -65,17 +66,14 @@ export default function useContentHandler({ setMessages, getMessages }: TUseCont
 
       response.content[index] = { type, [type]: part } as TMessageContentParts;
 
-      const lastContentPart = response.content[response.content.length - 1];
-      const initialContentPart = initialResponse.content?.[0];
       if (
         type !== ContentTypes.TEXT &&
-        initialContentPart != null &&
-        lastContentPart != null &&
-        ((lastContentPart.type === ContentTypes.TOOL_CALL &&
-          lastContentPart[ContentTypes.TOOL_CALL]?.progress === 1) ||
-          lastContentPart.type === ContentTypes.IMAGE_FILE)
+        initialResponse.content &&
+        ((response.content[response.content.length - 1].type === ContentTypes.TOOL_CALL &&
+          response.content[response.content.length - 1][ContentTypes.TOOL_CALL].progress === 1) ||
+          response.content[response.content.length - 1].type === ContentTypes.IMAGE_FILE)
       ) {
-        response.content.push(initialContentPart);
+        response.content.push(initialResponse.content[0]);
       }
 
       setMessages([...messages, response]);

client/src/hooks/SSE/useEventHandlers.ts

@@ -87,14 +87,12 @@ const createErrorMessage = ({
   let isValidContentPart = false;
   if (latestContent.length > 0) {
     const latestContentPart = latestContent[latestContent.length - 1];
-    if (latestContentPart != null) {
-      const latestPartValue = latestContentPart[latestContentPart.type ?? ''];
-      isValidContentPart =
-        latestContentPart.type !== ContentTypes.TEXT ||
-        (latestContentPart.type === ContentTypes.TEXT && typeof latestPartValue === 'string')
-          ? true
-          : latestPartValue?.value !== '';
-    }
+    const latestPartValue = latestContentPart?.[latestContentPart.type ?? ''];
+    isValidContentPart =
+      latestContentPart.type !== ContentTypes.TEXT ||
+      (latestContentPart.type === ContentTypes.TEXT && typeof latestPartValue === 'string')
+        ? true
+        : latestPartValue?.value !== '';
   }
   if (
     latestMessage?.conversationId &&
@@ -457,145 +455,141 @@ export default function useEventHandlers({
         isTemporary = false,
       } = submission;
 
-      try {
-        if (responseMessage?.attachments && responseMessage.attachments.length > 0) {
-          // Process each attachment through the attachmentHandler
-          responseMessage.attachments.forEach((attachment) => {
-            const attachmentData = {
-              ...attachment,
-              messageId: responseMessage.messageId,
-            };
+      if (responseMessage?.attachments && responseMessage.attachments.length > 0) {
+        // Process each attachment through the attachmentHandler
+        responseMessage.attachments.forEach((attachment) => {
+          const attachmentData = {
+            ...attachment,
+            messageId: responseMessage.messageId,
+          };
 
-            attachmentHandler({
-              data: attachmentData,
-              submission: submission as EventSubmission,
-            });
+          attachmentHandler({
+            data: attachmentData,
+            submission: submission as EventSubmission,
           });
-        }
+        });
+      }
 
-        setCompleted((prev) => new Set(prev.add(submission.initialResponse.messageId)));
+      setShowStopButton(false);
+      setCompleted((prev) => new Set(prev.add(submission.initialResponse.messageId)));
 
-        const currentMessages = getMessages();
-        /* Early return if messages are empty; i.e., the user navigated away */
-        if (!currentMessages || currentMessages.length === 0) {
-          return;
-        }
+      const currentMessages = getMessages();
+      /* Early return if messages are empty; i.e., the user navigated away */
+      if (!currentMessages || currentMessages.length === 0) {
+        setIsSubmitting(false);
+        return;
+      }
 
-        /* a11y announcements */
-        announcePolite({ message: 'end', isStatus: true });
-        announcePolite({ message: getAllContentText(responseMessage) });
+      /* a11y announcements */
+      announcePolite({ message: 'end', isStatus: true });
+      announcePolite({ message: getAllContentText(responseMessage) });
 
-        const isNewConvo = conversation.conversationId !== submissionConvo.conversationId;
+      const isNewConvo = conversation.conversationId !== submissionConvo.conversationId;
 
-        const setFinalMessages = (id: string | null, _messages: TMessage[]) => {
-          setMessages(_messages);
-          queryClient.setQueryData<TMessage[]>([QueryKeys.messages, id], _messages);
-        };
+      const setFinalMessages = (id: string | null, _messages: TMessage[]) => {
+        setMessages(_messages);
+        queryClient.setQueryData<TMessage[]>([QueryKeys.messages, id], _messages);
+      };
 
-        const hasNoResponse =
-          responseMessage?.content?.[0]?.['text']?.value ===
-            submission.initialResponse?.content?.[0]?.['text']?.value ||
-          !!responseMessage?.content?.[0]?.['tool_call']?.auth;
-
-        /** Handle edge case where stream is cancelled before any response, which creates a blank page */
-        if (!conversation.conversationId && hasNoResponse) {
-          const currentConvoId =
-            (submissionConvo.conversationId ?? conversation.conversationId) || Constants.NEW_CONVO;
-          if (isNewConvo && submissionConvo.conversationId) {
-            removeConvoFromAllQueries(queryClient, submissionConvo.conversationId);
-          }
-
-          const isNewChat =
-            location.pathname === `/c/${Constants.NEW_CONVO}` &&
-            currentConvoId === Constants.NEW_CONVO;
-
-          setFinalMessages(currentConvoId, isNewChat ? [] : [...messages]);
-          setDraft({ id: currentConvoId, value: requestMessage?.text });
-          if (isNewChat) {
-            navigate(`/c/${Constants.NEW_CONVO}`, { replace: true, state: { focusChat: true } });
-          }
-          return;
-        }
-
-        /* Update messages; if assistants endpoint, client doesn't receive responseMessage */
-        let finalMessages: TMessage[] = [];
-        if (runMessages) {
-          finalMessages = [...runMessages];
-        } else if (isRegenerate && responseMessage) {
-          finalMessages = [...messages, responseMessage];
-        } else if (requestMessage != null && responseMessage != null) {
-          finalMessages = [...messages, requestMessage, responseMessage];
-        }
-        if (finalMessages.length > 0) {
-          setFinalMessages(conversation.conversationId, finalMessages);
-        } else if (
-          isAssistantsEndpoint(submissionConvo.endpoint) &&
-          (!submissionConvo.conversationId ||
-            submissionConvo.conversationId === Constants.NEW_CONVO)
-        ) {
-          queryClient.setQueryData<TMessage[]>(
-            [QueryKeys.messages, conversation.conversationId],
-            [...currentMessages],
-          );
-        }
+      const hasNoResponse =
+        responseMessage?.content?.[0]?.['text']?.value ===
+          submission.initialResponse?.content?.[0]?.['text']?.value ||
+        !!responseMessage?.content?.[0]?.['tool_call']?.auth;
 
+      /** Handle edge case where stream is cancelled before any response, which creates a blank page */
+      if (!conversation.conversationId && hasNoResponse) {
+        const currentConvoId =
+          (submissionConvo.conversationId ?? conversation.conversationId) || Constants.NEW_CONVO;
         if (isNewConvo && submissionConvo.conversationId) {
           removeConvoFromAllQueries(queryClient, submissionConvo.conversationId);
         }
 
-        /* Refresh title */
-        if (
-          genTitle &&
-          isNewConvo &&
-          !isTemporary &&
-          requestMessage &&
-          requestMessage.parentMessageId === Constants.NO_PARENT
-        ) {
-          setTimeout(() => {
-            genTitle.mutate({ conversationId: conversation.conversationId as string });
-          }, 2500);
-        }
+        const isNewChat =
+          location.pathname === `/c/${Constants.NEW_CONVO}` &&
+          currentConvoId === Constants.NEW_CONVO;
 
-        if (setConversation && isAddedRequest !== true) {
-          setConversation((prevState) => {
-            const update = {
-              ...prevState,
-              ...(conversation as TConversation),
-            };
-            if (prevState?.model != null && prevState.model !== submissionConvo.model) {
-              update.model = prevState.model;
-            }
-            const cachedConvo = queryClient.getQueryData<TConversation>([
-              QueryKeys.conversation,
-              conversation.conversationId,
-            ]);
-            if (!cachedConvo) {
-              queryClient.setQueryData(
-                [QueryKeys.conversation, conversation.conversationId],
-                update,
-              );
-            }
-            return update;
-          });
-
-          if (conversation.conversationId && submission.ephemeralAgent) {
-            applyAgentTemplate({
-              targetId: conversation.conversationId,
-              sourceId: submissionConvo.conversationId,
-              ephemeralAgent: submission.ephemeralAgent,
-              specName: submission.conversation?.spec,
-              startupConfig: queryClient.getQueryData<TStartupConfig>([QueryKeys.startupConfig]),
-            });
-          }
-
-          if (location.pathname === `/c/${Constants.NEW_CONVO}`) {
-            navigate(`/c/${conversation.conversationId}`, { replace: true });
-          }
-        }
-      } finally {
-        setShowStopButton(false);
+        setFinalMessages(currentConvoId, isNewChat ? [] : [...messages]);
+        setDraft({ id: currentConvoId, value: requestMessage?.text });
         setIsSubmitting(false);
+        if (isNewChat) {
+          navigate(`/c/${Constants.NEW_CONVO}`, { replace: true, state: { focusChat: true } });
+        }
+        return;
       }
+
+      /* Update messages; if assistants endpoint, client doesn't receive responseMessage */
+      let finalMessages: TMessage[] = [];
+      if (runMessages) {
+        finalMessages = [...runMessages];
+      } else if (isRegenerate && responseMessage) {
+        finalMessages = [...messages, responseMessage];
+      } else if (requestMessage != null && responseMessage != null) {
+        finalMessages = [...messages, requestMessage, responseMessage];
+      }
+      if (finalMessages.length > 0) {
+        setFinalMessages(conversation.conversationId, finalMessages);
+      } else if (
+        isAssistantsEndpoint(submissionConvo.endpoint) &&
+        (!submissionConvo.conversationId || submissionConvo.conversationId === Constants.NEW_CONVO)
+      ) {
+        queryClient.setQueryData<TMessage[]>(
+          [QueryKeys.messages, conversation.conversationId],
+          [...currentMessages],
+        );
+      }
+
+      if (isNewConvo && submissionConvo.conversationId) {
+        removeConvoFromAllQueries(queryClient, submissionConvo.conversationId);
+      }
+
+      /* Refresh title */
+      if (
+        genTitle &&
+        isNewConvo &&
+        !isTemporary &&
+        requestMessage &&
+        requestMessage.parentMessageId === Constants.NO_PARENT
+      ) {
+        setTimeout(() => {
+          genTitle.mutate({ conversationId: conversation.conversationId as string });
+        }, 2500);
+      }
+
+      if (setConversation && isAddedRequest !== true) {
+        setConversation((prevState) => {
+          const update = {
+            ...prevState,
+            ...(conversation as TConversation),
+          };
+          if (prevState?.model != null && prevState.model !== submissionConvo.model) {
+            update.model = prevState.model;
+          }
+          const cachedConvo = queryClient.getQueryData<TConversation>([
+            QueryKeys.conversation,
+            conversation.conversationId,
+          ]);
+          if (!cachedConvo) {
+            queryClient.setQueryData([QueryKeys.conversation, conversation.conversationId], update);
+          }
+          return update;
+        });
+
+        if (conversation.conversationId && submission.ephemeralAgent) {
+          applyAgentTemplate({
+            targetId: conversation.conversationId,
+            sourceId: submissionConvo.conversationId,
+            ephemeralAgent: submission.ephemeralAgent,
+            specName: submission.conversation?.spec,
+            startupConfig: queryClient.getQueryData<TStartupConfig>([QueryKeys.startupConfig]),
+          });
+        }
+
+        if (location.pathname === `/c/${Constants.NEW_CONVO}`) {
+          navigate(`/c/${conversation.conversationId}`, { replace: true });
+        }
+      }
+
+      setIsSubmitting(false);
     },
     [
       navigate,
@@ -728,37 +722,26 @@ export default function useEventHandlers({
         messages[messages.length - 2] != null
       ) {
         let requestMessage = messages[messages.length - 2];
-        const _responseMessage = messages[messages.length - 1];
-        if (requestMessage.messageId !== _responseMessage.parentMessageId) {
+        const responseMessage = messages[messages.length - 1];
+        if (requestMessage.messageId !== responseMessage.parentMessageId) {
           // the request message is the parent of response, which we search for backwards
           for (let i = messages.length - 3; i >= 0; i--) {
-            if (messages[i].messageId === _responseMessage.parentMessageId) {
+            if (messages[i].messageId === responseMessage.parentMessageId) {
               requestMessage = messages[i];
               break;
             }
           }
         }
-        /** Sanitize content array to remove undefined parts from interrupted streaming */
-        const responseMessage = {
-          ..._responseMessage,
-          content: _responseMessage.content?.filter((part) => part != null),
-        };
-        try {
-          finalHandler(
-            {
-              conversation: {
-                conversationId,
-              },
-              requestMessage,
-              responseMessage,
+        finalHandler(
+          {
+            conversation: {
+              conversationId,
             },
-            submission,
-          );
-        } catch (error) {
-          console.error('Error in finalHandler during abort:', error);
-          setShowStopButton(false);
-          setIsSubmitting(false);
-        }
+            requestMessage,
+            responseMessage,
+          },
+          submission,
+        );
         return;
       } else if (!isAssistantsEndpoint(endpoint)) {
         const convoId = conversationId || `_${v4()}`;
@@ -826,14 +809,13 @@ export default function useEventHandlers({
       }
     },
     [
-      token,
-      getMessages,
-      setMessages,
       finalHandler,
-      cancelHandler,
       newConversation,
       setIsSubmitting,
-      setShowStopButton,
+      token,
+      cancelHandler,
+      getMessages,
+      setMessages,
     ],
   );
 

client/src/hooks/SSE/useSSE.ts

@@ -124,13 +124,7 @@ export default function useSSE(
       if (data.final != null) {
         clearDraft(submission.conversation?.conversationId);
         const { plugins } = data;
-        try {
-          finalHandler(data, { ...submission, plugins } as EventSubmission);
-        } catch (error) {
-          console.error('Error in finalHandler:', error);
-          setIsSubmitting(false);
-          setShowStopButton(false);
-        }
+        finalHandler(data, { ...submission, plugins } as EventSubmission);
         (startupConfig?.balance?.enabled ?? false) && balanceQuery.refetch();
         console.log('final', data);
         return;
@@ -193,20 +187,14 @@ export default function useSSE(
       setCompleted((prev) => new Set(prev.add(streamKey)));
       const latestMessages = getMessages();
       const conversationId = latestMessages?.[latestMessages.length - 1]?.conversationId;
-      try {
-        await abortConversation(
-          conversationId ??
-            userMessage.conversationId ??
-            submission.conversation?.conversationId ??
-            '',
-          submission as EventSubmission,
-          latestMessages,
-        );
-      } catch (error) {
-        console.error('Error during abort:', error);
-        setIsSubmitting(false);
-        setShowStopButton(false);
-      }
+      return await abortConversation(
+        conversationId ??
+          userMessage.conversationId ??
+          submission.conversation?.conversationId ??
+          '',
+        submission as EventSubmission,
+        latestMessages,
+      );
     });
 
     sse.addEventListener('error', async (e: MessageEvent) => {

client/src/hooks/SSE/useStepHandler.ts

@@ -313,10 +313,6 @@ export default function useStepHandler({
             ? messageDelta.delta.content[0]
             : messageDelta.delta.content;
 
-          if (contentPart == null) {
-            return;
-          }
-
           const currentIndex = calculateContentIndex(
             runStep.index,
             initialContent,
@@ -349,10 +345,6 @@ export default function useStepHandler({
             ? reasoningDelta.delta.content[0]
             : reasoningDelta.delta.content;
 
-          if (contentPart == null) {
-            return;
-          }
-
           const currentIndex = calculateContentIndex(
             runStep.index,
             initialContent,

client/src/locales/en/translation.json

@@ -799,7 +799,6 @@
   "com_ui_continue_oauth": "Continue with OAuth",
   "com_ui_controls": "Controls",
   "com_ui_convo_delete_error": "Failed to delete conversation",
-  "com_ui_convo_delete_success": "Conversation successfully deleted",
   "com_ui_copied": "Copied!",
   "com_ui_copied_to_clipboard": "Copied to clipboard",
   "com_ui_copy_code": "Copy code",
@@ -900,13 +899,6 @@
   "com_ui_everyone_permission_level": "Everyone's permission level",
   "com_ui_examples": "Examples",
   "com_ui_expand_chat": "Expand Chat",
-  "com_ui_expired": "Expired",
-  "com_ui_expires": "Expires",
-  "com_ui_expiry_hour": "Hour",
-  "com_ui_expiry_12_hours": "12 Hours",
-  "com_ui_expiry_day": "Day",
-  "com_ui_expiry_week": "Week",
-  "com_ui_expiry_month": "Month",
   "com_ui_export_convo_modal": "Export Conversation Modal",
   "com_ui_feedback_more": "More...",
   "com_ui_feedback_more_information": "Provide additional feedback",
@@ -1056,7 +1048,6 @@
   "com_ui_more_info": "More info",
   "com_ui_my_prompts": "My Prompts",
   "com_ui_name": "Name",
-  "com_ui_never": "Never",
   "com_ui_new": "New",
   "com_ui_new_chat": "New chat",
   "com_ui_new_conversation_title": "New Conversation Title",

client/src/style.css

@@ -1487,26 +1487,6 @@ button {
   background-color: transparent;
 }
 
-/* Show scrollbar only on hover */
-.scrollbar-hover {
-  scrollbar-width: thin;
-  scrollbar-color: transparent transparent;
-}
-
-.scrollbar-hover:hover {
-  scrollbar-color: var(--border-medium) transparent;
-}
-
-.scrollbar-hover::-webkit-scrollbar-thumb {
-  background-color: transparent;
-  transition: background-color 0.3s ease 0.5s;
-}
-
-.scrollbar-hover:hover::-webkit-scrollbar-thumb {
-  background-color: var(--border-medium);
-  transition-delay: 0s;
-}
-
 body,
 html {
   height: 100%;

client/src/utils/messages.ts

@@ -44,7 +44,7 @@ export const getAllContentText = (message?: TMessage | null): string => {
 
   if (message.content && message.content.length > 0) {
     return message.content
-      .filter((part) => part != null && part.type === ContentTypes.TEXT)
+      .filter((part) => part.type === ContentTypes.TEXT)
       .map((part) => {
         if (!('text' in part)) return '';
         const text = part.text;

client/src/utils/shareExpiry.ts

@@ -1,32 +0,0 @@
-export const SHARE_EXPIRY = {
-  ONE_HOUR: {
-    label: 'com_ui_expiry_hour',
-    value: 60 * 60 * 1000, // 1 hour in ms
-    hours: 1,
-  },
-  TWELVE_HOURS: {
-    label: 'com_ui_expiry_12_hours',
-    value: 12 * 60 * 60 * 1000,
-    hours: 12,
-  },
-  ONE_DAY: {
-    label: 'com_ui_expiry_day',
-    value: 24 * 60 * 60 * 1000,
-    hours: 24,
-  },
-  SEVEN_DAYS: {
-    label: 'com_ui_expiry_week',
-    value: 7 * 24 * 60 * 60 * 1000,
-    hours: 168,
-  },
-  THIRTY_DAYS: {
-    label: 'com_ui_expiry_month',
-    value: 30 * 24 * 60 * 60 * 1000,
-    hours: 720,
-  },
-  NEVER: {
-    label: 'com_ui_never',
-    value: 0,
-    hours: 0,
-  },
-} as const;

config/add-balance.js

@@ -1,6 +1,6 @@
 const path = require('path');
 const mongoose = require('mongoose');
-const { getBalanceConfig } = require('@librechat/api');
+const { isEnabled, getBalanceConfig } = require('@librechat/api');
 const { User } = require('@librechat/data-schemas').createModels(mongoose);
 require('module-alias')({ base: path.resolve(__dirname, '..', 'api') });
 const { createTransaction } = require('~/models/Transaction');
@@ -33,12 +33,15 @@ const connect = require('./connect');
     // console.purple(`[DEBUG] Args Length: ${process.argv.length}`);
   }
 
-  const appConfig = await getAppConfig();
-  const balanceConfig = getBalanceConfig(appConfig);
-
-  if (!balanceConfig?.enabled) {
+  if (!process.env.CHECK_BALANCE) {
     console.red(
-      'Error: Balance is not enabled. Use librechat.yaml to enable it',
+      'Error: CHECK_BALANCE environment variable is not set! Configure it to use it: `CHECK_BALANCE=true`',
+    );
+    silentExit(1);
+  }
+  if (isEnabled(process.env.CHECK_BALANCE) === false) {
+    console.red(
+      'Error: CHECK_BALANCE environment variable is set to `false`! Please configure: `CHECK_BALANCE=true`',
     );
     silentExit(1);
   }
@@ -77,6 +80,8 @@ const connect = require('./connect');
    */
   let result;
   try {
+    const appConfig = await getAppConfig();
+    const balanceConfig = getBalanceConfig(appConfig);
     result = await createTransaction({
       user: user._id,
       tokenType: 'credits',

config/set-balance.js

@@ -1,6 +1,6 @@
 const path = require('path');
 const mongoose = require('mongoose');
-const { getBalanceConfig } = require('@librechat/api');
+const { isEnabled } = require('@librechat/api');
 const { User, Balance } = require('@librechat/data-schemas').createModels(mongoose);
 require('module-alias')({ base: path.resolve(__dirname, '..', 'api') });
 const { askQuestion, silentExit } = require('./helpers');
@@ -31,10 +31,15 @@ const connect = require('./connect');
     // console.purple(`[DEBUG] Args Length: ${process.argv.length}`);
   }
 
-  const balanceConfig = getBalanceConfig();
-  if (!balanceConfig?.enabled) {
+  if (!process.env.CHECK_BALANCE) {
     console.red(
-      'Error: Balance is not enabled. Use librechat.yaml to enable it',
+      'Error: CHECK_BALANCE environment variable is not set! Configure it to use it: `CHECK_BALANCE=true`',
+    );
+    silentExit(1);
+  }
+  if (isEnabled(process.env.CHECK_BALANCE) === false) {
+    console.red(
+      'Error: CHECK_BALANCE environment variable is set to `false`! Please configure: `CHECK_BALANCE=true`',
     );
     silentExit(1);
   }

e2e/jestSetup.js

@@ -1,3 +1,3 @@
-// v0.8.1-rc2
+// v0.8.1-rc1
 // See .env.test.example for an example of the '.env.test' file.
 require('dotenv').config({ path: './e2e/.env.test' });

eslint.config.mjs

@@ -269,16 +269,6 @@ export default [
         project: './packages/data-provider/tsconfig.json',
       },
     },
-    rules: {
-      '@typescript-eslint/no-unused-vars': [
-        'warn',
-        {
-          argsIgnorePattern: '^_',
-          varsIgnorePattern: '^_',
-          caughtErrorsIgnorePattern: '^_',
-        },
-      ],
-    },
   },
   {
     files: ['./api/demo/**/*.ts'],

helm/librechat/Chart.yaml

@@ -15,7 +15,7 @@ type: application
 # This is the chart version. This version number should be incremented each time you make changes
 # to the chart and its templates, including the app version.
 # Versions are expected to follow Semantic Versioning (https://semver.org/)
-version: 1.9.3
+version: 1.9.2
 
 # This is the version number of the application being deployed. This version number should be
 # incremented each time you make changes to the application. Versions are not expected to
@@ -23,7 +23,7 @@ version: 1.9.3
 # It is recommended to use it with quotes.
 
 # renovate: image=ghcr.io/danny-avila/librechat
-appVersion: "v0.8.1-rc2"
+appVersion: "v0.8.1-rc1"
 
 home: https://www.librechat.ai
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "LibreChat",
-  "version": "v0.8.1-rc2",
+  "version": "v0.8.1-rc1",
   "description": "",
   "workspaces": [
     "api",

packages/api/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@librechat/api",
-  "version": "1.6.0",
+  "version": "1.5.0",
   "type": "commonjs",
   "description": "MCP services for LibreChat",
   "main": "dist/index.js",
@@ -84,7 +84,7 @@
     "@azure/storage-blob": "^12.27.0",
     "@keyv/redis": "^4.3.3",
     "@langchain/core": "^0.3.79",
-    "@librechat/agents": "^3.0.36",
+    "@librechat/agents": "^3.0.32",
     "@librechat/data-schemas": "*",
     "@modelcontextprotocol/sdk": "^1.21.0",
     "axios": "^1.12.1",

packages/api/src/agents/run.ts

@@ -16,9 +16,9 @@ import { resolveHeaders, createSafeUser } from '~/utils/env';
 
 const customProviders = new Set([
   Providers.XAI,
+  Providers.OLLAMA,
   Providers.DEEPSEEK,
   Providers.OPENROUTER,
-  KnownEndpoints.ollama,
 ]);
 
 export function getReasoningKey(

packages/api/src/auth/openid.spec.ts

@@ -394,34 +394,6 @@ describe('findOpenIDUser', () => {
       expect(mockFindUser).toHaveBeenCalledWith({ email: 'user@example.com' });
     });
 
-    it('should pass email to findUser for case-insensitive lookup (findUser handles normalization)', async () => {
-      const mockUser: IUser = {
-        _id: 'user123',
-        provider: 'openid',
-        openidId: 'openid_456',
-        email: 'user@example.com',
-        username: 'testuser',
-      } as IUser;
-
-      mockFindUser
-        .mockResolvedValueOnce(null) // Primary condition fails
-        .mockResolvedValueOnce(mockUser); // Email search succeeds
-
-      const result = await findOpenIDUser({
-        openidId: 'openid_123',
-        findUser: mockFindUser,
-        email: 'User@Example.COM',
-      });
-
-      /** Email is passed as-is; findUser implementation handles normalization */
-      expect(mockFindUser).toHaveBeenNthCalledWith(2, { email: 'User@Example.COM' });
-      expect(result).toEqual({
-        user: mockUser,
-        error: null,
-        migration: false,
-      });
-    });
-
     it('should handle findUser throwing an error', async () => {
       mockFindUser.mockRejectedValueOnce(new Error('Database error'));
 

packages/api/src/endpoints/openai/config.spec.ts

@@ -26,7 +26,7 @@ describe('getOpenAIConfig', () => {
 
   it('should apply model options', () => {
     const modelOptions = {
-      model: 'gpt-4',
+      model: 'gpt-5',
       temperature: 0.7,
       max_tokens: 1000,
     };
@@ -34,11 +34,14 @@ describe('getOpenAIConfig', () => {
     const result = getOpenAIConfig(mockApiKey, { modelOptions });
 
     expect(result.llmConfig).toMatchObject({
-      model: 'gpt-4',
+      model: 'gpt-5',
       temperature: 0.7,
-      maxTokens: 1000,
+      modelKwargs: {
+        max_completion_tokens: 1000,
+      },
     });
     expect((result.llmConfig as Record<string, unknown>).max_tokens).toBeUndefined();
+    expect((result.llmConfig as Record<string, unknown>).maxTokens).toBeUndefined();
   });
 
   it('should separate known and unknown params from addParams', () => {
@@ -283,7 +286,7 @@ describe('getOpenAIConfig', () => {
 
   it('should ignore non-boolean web_search values in addParams', () => {
     const modelOptions = {
-      model: 'gpt-4',
+      model: 'gpt-5',
       web_search: true,
     };
 
@@ -396,7 +399,7 @@ describe('getOpenAIConfig', () => {
 
   it('should handle verbosity parameter in modelKwargs', () => {
     const modelOptions = {
-      model: 'gpt-4',
+      model: 'gpt-5',
       temperature: 0.7,
       verbosity: Verbosity.high,
     };
@@ -404,7 +407,7 @@ describe('getOpenAIConfig', () => {
     const result = getOpenAIConfig(mockApiKey, { modelOptions });
 
     expect(result.llmConfig).toMatchObject({
-      model: 'gpt-4',
+      model: 'gpt-5',
       temperature: 0.7,
     });
     expect(result.llmConfig.modelKwargs).toEqual({
@@ -414,7 +417,7 @@ describe('getOpenAIConfig', () => {
 
   it('should allow addParams to override verbosity in modelKwargs', () => {
     const modelOptions = {
-      model: 'gpt-4',
+      model: 'gpt-5',
       verbosity: Verbosity.low,
     };
 
@@ -448,7 +451,7 @@ describe('getOpenAIConfig', () => {
 
   it('should nest verbosity under text when useResponsesApi is enabled', () => {
     const modelOptions = {
-      model: 'gpt-4',
+      model: 'gpt-5',
       temperature: 0.7,
       verbosity: Verbosity.low,
       useResponsesApi: true,
@@ -457,7 +460,7 @@ describe('getOpenAIConfig', () => {
     const result = getOpenAIConfig(mockApiKey, { modelOptions });
 
     expect(result.llmConfig).toMatchObject({
-      model: 'gpt-4',
+      model: 'gpt-5',
       temperature: 0.7,
       useResponsesApi: true,
     });
@@ -493,6 +496,7 @@ describe('getOpenAIConfig', () => {
   it('should move maxTokens to modelKwargs.max_completion_tokens for GPT-5+ models', () => {
     const modelOptions = {
       model: 'gpt-5',
+      temperature: 0.7,
       max_tokens: 2048,
     };
 
@@ -500,6 +504,7 @@ describe('getOpenAIConfig', () => {
 
     expect(result.llmConfig).toMatchObject({
       model: 'gpt-5',
+      temperature: 0.7,
     });
     expect(result.llmConfig.maxTokens).toBeUndefined();
     expect(result.llmConfig.modelKwargs).toEqual({
@@ -1679,7 +1684,7 @@ describe('getOpenAIConfig', () => {
       it('should not override existing modelOptions with defaultParams', () => {
         const result = getOpenAIConfig(mockApiKey, {
           modelOptions: {
-            model: 'gpt-4',
+            model: 'gpt-5',
             temperature: 0.9,
           },
           customParams: {
@@ -1692,7 +1697,7 @@ describe('getOpenAIConfig', () => {
         });
 
         expect(result.llmConfig.temperature).toBe(0.9);
-        expect(result.llmConfig.maxTokens).toBe(1000);
+        expect(result.llmConfig.modelKwargs?.max_completion_tokens).toBe(1000);
       });
 
       it('should allow addParams to override defaultParams', () => {
@@ -1840,7 +1845,7 @@ describe('getOpenAIConfig', () => {
       it('should preserve order: defaultParams < addParams < modelOptions', () => {
         const result = getOpenAIConfig(mockApiKey, {
           modelOptions: {
-            model: 'gpt-4',
+            model: 'gpt-5',
             temperature: 0.9,
           },
           customParams: {
@@ -1858,7 +1863,7 @@ describe('getOpenAIConfig', () => {
 
         expect(result.llmConfig.temperature).toBe(0.9);
         expect(result.llmConfig.topP).toBe(0.8);
-        expect(result.llmConfig.maxTokens).toBe(500);
+        expect(result.llmConfig.modelKwargs?.max_completion_tokens).toBe(500);
       });
     });
   });

packages/api/src/endpoints/openai/llm.spec.ts

@@ -1,602 +0,0 @@
-import {
-  Verbosity,
-  EModelEndpoint,
-  ReasoningEffort,
-  ReasoningSummary,
-} from 'librechat-data-provider';
-import { getOpenAILLMConfig, extractDefaultParams, applyDefaultParams } from './llm';
-import type * as t from '~/types';
-
-describe('getOpenAILLMConfig', () => {
-  describe('Basic Configuration', () => {
-    it('should create a basic configuration with required fields', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('apiKey', 'test-api-key');
-      expect(result.llmConfig).toHaveProperty('model', 'gpt-4');
-      expect(result.llmConfig).toHaveProperty('streaming', true);
-      expect(result.tools).toEqual([]);
-    });
-
-    it('should handle model options including temperature and penalties', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-          temperature: 0.7,
-          frequency_penalty: 0.5,
-          presence_penalty: 0.3,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('temperature', 0.7);
-      expect(result.llmConfig).toHaveProperty('frequencyPenalty', 0.5);
-      expect(result.llmConfig).toHaveProperty('presencePenalty', 0.3);
-    });
-
-    it('should handle max_tokens conversion to maxTokens', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-          max_tokens: 4096,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('maxTokens', 4096);
-      expect(result.llmConfig).not.toHaveProperty('max_tokens');
-    });
-  });
-
-  describe('OpenAI Reasoning Models (o1/o3/gpt-5)', () => {
-    const reasoningModels = [
-      'o1',
-      'o1-mini',
-      'o1-preview',
-      'o1-pro',
-      'o3',
-      'o3-mini',
-      'gpt-5',
-      'gpt-5-pro',
-      'gpt-5-turbo',
-    ];
-
-    const excludedParams = [
-      'frequencyPenalty',
-      'presencePenalty',
-      'temperature',
-      'topP',
-      'logitBias',
-      'n',
-      'logprobs',
-    ];
-
-    it.each(reasoningModels)(
-      'should exclude unsupported parameters for reasoning model: %s',
-      (model) => {
-        const result = getOpenAILLMConfig({
-          apiKey: 'test-api-key',
-          streaming: true,
-          modelOptions: {
-            model,
-            temperature: 0.7,
-            frequency_penalty: 0.5,
-            presence_penalty: 0.3,
-            topP: 0.9,
-            logitBias: { '50256': -100 },
-            n: 2,
-            logprobs: true,
-          } as Partial<t.OpenAIParameters>,
-        });
-
-        excludedParams.forEach((param) => {
-          expect(result.llmConfig).not.toHaveProperty(param);
-        });
-
-        expect(result.llmConfig).toHaveProperty('model', model);
-        expect(result.llmConfig).toHaveProperty('streaming', true);
-      },
-    );
-
-    it('should preserve maxTokens for reasoning models', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'o1',
-          max_tokens: 4096,
-          temperature: 0.7,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('maxTokens', 4096);
-      expect(result.llmConfig).not.toHaveProperty('temperature');
-    });
-
-    it('should preserve other valid parameters for reasoning models', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'o1',
-          max_tokens: 8192,
-          stop: ['END'],
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('maxTokens', 8192);
-      expect(result.llmConfig).toHaveProperty('stop', ['END']);
-    });
-
-    it('should handle GPT-5 max_tokens conversion to max_completion_tokens', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-5',
-          max_tokens: 8192,
-          stop: ['END'],
-        },
-      });
-
-      expect(result.llmConfig.modelKwargs).toHaveProperty('max_completion_tokens', 8192);
-      expect(result.llmConfig).not.toHaveProperty('maxTokens');
-      expect(result.llmConfig).toHaveProperty('stop', ['END']);
-    });
-
-    it('should combine user dropParams with reasoning exclusion params', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'o3-mini',
-          temperature: 0.7,
-          stop: ['END'],
-        },
-        dropParams: ['stop'],
-      });
-
-      expect(result.llmConfig).not.toHaveProperty('temperature');
-      expect(result.llmConfig).not.toHaveProperty('stop');
-    });
-
-    it('should NOT exclude parameters for non-reasoning models', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4-turbo',
-          temperature: 0.7,
-          frequency_penalty: 0.5,
-          presence_penalty: 0.3,
-          topP: 0.9,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('temperature', 0.7);
-      expect(result.llmConfig).toHaveProperty('frequencyPenalty', 0.5);
-      expect(result.llmConfig).toHaveProperty('presencePenalty', 0.3);
-      expect(result.llmConfig).toHaveProperty('topP', 0.9);
-    });
-
-    it('should NOT exclude parameters for gpt-5.x versioned models (they support sampling params)', () => {
-      const versionedModels = ['gpt-5.1', 'gpt-5.1-turbo', 'gpt-5.2', 'gpt-5.5-preview'];
-
-      versionedModels.forEach((model) => {
-        const result = getOpenAILLMConfig({
-          apiKey: 'test-api-key',
-          streaming: true,
-          modelOptions: {
-            model,
-            temperature: 0.7,
-            frequency_penalty: 0.5,
-            presence_penalty: 0.3,
-            topP: 0.9,
-          },
-        });
-
-        expect(result.llmConfig).toHaveProperty('temperature', 0.7);
-        expect(result.llmConfig).toHaveProperty('frequencyPenalty', 0.5);
-        expect(result.llmConfig).toHaveProperty('presencePenalty', 0.3);
-        expect(result.llmConfig).toHaveProperty('topP', 0.9);
-      });
-    });
-
-    it('should NOT exclude parameters for gpt-5-chat (it supports sampling params)', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-5-chat',
-          temperature: 0.7,
-          frequency_penalty: 0.5,
-          presence_penalty: 0.3,
-          topP: 0.9,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('temperature', 0.7);
-      expect(result.llmConfig).toHaveProperty('frequencyPenalty', 0.5);
-      expect(result.llmConfig).toHaveProperty('presencePenalty', 0.3);
-      expect(result.llmConfig).toHaveProperty('topP', 0.9);
-    });
-
-    it('should handle reasoning models with reasoning_effort parameter', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        endpoint: EModelEndpoint.openAI,
-        modelOptions: {
-          model: 'o1',
-          reasoning_effort: ReasoningEffort.high,
-          temperature: 0.7,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('reasoning_effort', ReasoningEffort.high);
-      expect(result.llmConfig).not.toHaveProperty('temperature');
-    });
-  });
-
-  describe('OpenAI Web Search Models', () => {
-    it('should exclude parameters for gpt-4o search models', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4o-search-preview',
-          temperature: 0.7,
-          top_p: 0.9,
-          seed: 42,
-        } as Partial<t.OpenAIParameters>,
-      });
-
-      expect(result.llmConfig).not.toHaveProperty('temperature');
-      expect(result.llmConfig).not.toHaveProperty('top_p');
-      expect(result.llmConfig).not.toHaveProperty('seed');
-    });
-
-    it('should preserve max_tokens for search models', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4o-search',
-          max_tokens: 4096,
-          temperature: 0.7,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('maxTokens', 4096);
-      expect(result.llmConfig).not.toHaveProperty('temperature');
-    });
-  });
-
-  describe('Web Search Functionality', () => {
-    it('should enable web search with Responses API', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-          web_search: true,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('useResponsesApi', true);
-      expect(result.tools).toContainEqual({ type: 'web_search' });
-    });
-
-    it('should handle web search with OpenRouter', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        useOpenRouter: true,
-        modelOptions: {
-          model: 'gpt-4',
-          web_search: true,
-        },
-      });
-
-      expect(result.llmConfig.modelKwargs).toHaveProperty('plugins', [{ id: 'web' }]);
-      expect(result.llmConfig).toHaveProperty('include_reasoning', true);
-    });
-
-    it('should disable web search via dropParams', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-          web_search: true,
-        },
-        dropParams: ['web_search'],
-      });
-
-      expect(result.tools).not.toContainEqual({ type: 'web_search' });
-    });
-  });
-
-  describe('GPT-5 max_tokens Handling', () => {
-    it('should convert maxTokens to max_completion_tokens for GPT-5 models', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-5',
-          max_tokens: 8192,
-        },
-      });
-
-      expect(result.llmConfig.modelKwargs).toHaveProperty('max_completion_tokens', 8192);
-      expect(result.llmConfig).not.toHaveProperty('maxTokens');
-    });
-
-    it('should convert maxTokens to max_output_tokens for GPT-5 with Responses API', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-5',
-          max_tokens: 8192,
-        },
-        addParams: {
-          useResponsesApi: true,
-        },
-      });
-
-      expect(result.llmConfig.modelKwargs).toHaveProperty('max_output_tokens', 8192);
-      expect(result.llmConfig).not.toHaveProperty('maxTokens');
-    });
-  });
-
-  describe('Reasoning Parameters', () => {
-    it('should handle reasoning_effort for OpenAI endpoint', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        endpoint: EModelEndpoint.openAI,
-        modelOptions: {
-          model: 'o1',
-          reasoning_effort: ReasoningEffort.high,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('reasoning_effort', ReasoningEffort.high);
-    });
-
-    it('should use reasoning object for non-OpenAI endpoints', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        endpoint: 'custom',
-        modelOptions: {
-          model: 'o1',
-          reasoning_effort: ReasoningEffort.high,
-          reasoning_summary: ReasoningSummary.concise,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('reasoning');
-      expect(result.llmConfig.reasoning).toEqual({
-        effort: ReasoningEffort.high,
-        summary: ReasoningSummary.concise,
-      });
-    });
-
-    it('should use reasoning object when useResponsesApi is true', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        endpoint: EModelEndpoint.openAI,
-        modelOptions: {
-          model: 'o1',
-          reasoning_effort: ReasoningEffort.medium,
-          reasoning_summary: ReasoningSummary.detailed,
-        },
-        addParams: {
-          useResponsesApi: true,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('reasoning');
-      expect(result.llmConfig.reasoning).toEqual({
-        effort: ReasoningEffort.medium,
-        summary: ReasoningSummary.detailed,
-      });
-    });
-  });
-
-  describe('Default and Add Parameters', () => {
-    it('should apply default parameters when fields are undefined', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-        },
-        defaultParams: {
-          temperature: 0.5,
-          topP: 0.9,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('temperature', 0.5);
-      expect(result.llmConfig).toHaveProperty('topP', 0.9);
-    });
-
-    it('should NOT override existing values with default parameters', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-          temperature: 0.8,
-        },
-        defaultParams: {
-          temperature: 0.5,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('temperature', 0.8);
-    });
-
-    it('should apply addParams and override defaults', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-        },
-        defaultParams: {
-          temperature: 0.5,
-        },
-        addParams: {
-          temperature: 0.9,
-          seed: 42,
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('temperature', 0.9);
-      expect(result.llmConfig).toHaveProperty('seed', 42);
-    });
-
-    it('should handle unknown params via modelKwargs', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-        },
-        addParams: {
-          custom_param: 'custom_value',
-        },
-      });
-
-      expect(result.llmConfig.modelKwargs).toHaveProperty('custom_param', 'custom_value');
-    });
-  });
-
-  describe('Drop Parameters', () => {
-    it('should drop specified parameters', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-          temperature: 0.7,
-          topP: 0.9,
-        },
-        dropParams: ['temperature'],
-      });
-
-      expect(result.llmConfig).not.toHaveProperty('temperature');
-      expect(result.llmConfig).toHaveProperty('topP', 0.9);
-    });
-  });
-
-  describe('OpenRouter Configuration', () => {
-    it('should include include_reasoning for OpenRouter', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        useOpenRouter: true,
-        modelOptions: {
-          model: 'gpt-4',
-        },
-      });
-
-      expect(result.llmConfig).toHaveProperty('include_reasoning', true);
-    });
-  });
-
-  describe('Verbosity Handling', () => {
-    it('should add verbosity to modelKwargs', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-          verbosity: Verbosity.high,
-        },
-      });
-
-      expect(result.llmConfig.modelKwargs).toHaveProperty('verbosity', Verbosity.high);
-    });
-
-    it('should convert verbosity to text object with Responses API', () => {
-      const result = getOpenAILLMConfig({
-        apiKey: 'test-api-key',
-        streaming: true,
-        modelOptions: {
-          model: 'gpt-4',
-          verbosity: Verbosity.low,
-        },
-        addParams: {
-          useResponsesApi: true,
-        },
-      });
-
-      expect(result.llmConfig.modelKwargs).toHaveProperty('text', { verbosity: Verbosity.low });
-      expect(result.llmConfig.modelKwargs).not.toHaveProperty('verbosity');
-    });
-  });
-});
-
-describe('extractDefaultParams', () => {
-  it('should extract default values from param definitions', () => {
-    const paramDefinitions = [
-      { key: 'temperature', default: 0.7 },
-      { key: 'maxTokens', default: 4096 },
-      { key: 'noDefault' },
-    ];
-
-    const result = extractDefaultParams(paramDefinitions);
-
-    expect(result).toEqual({
-      temperature: 0.7,
-      maxTokens: 4096,
-    });
-  });
-
-  it('should return undefined for undefined or non-array input', () => {
-    expect(extractDefaultParams(undefined)).toBeUndefined();
-    expect(extractDefaultParams(null as unknown as undefined)).toBeUndefined();
-  });
-
-  it('should handle empty array', () => {
-    const result = extractDefaultParams([]);
-    expect(result).toEqual({});
-  });
-});
-
-describe('applyDefaultParams', () => {
-  it('should apply defaults only when field is undefined', () => {
-    const target: Record<string, unknown> = {
-      temperature: 0.8,
-      maxTokens: undefined,
-    };
-
-    const defaults = {
-      temperature: 0.5,
-      maxTokens: 4096,
-      topP: 0.9,
-    };
-
-    applyDefaultParams(target, defaults);
-
-    expect(target).toEqual({
-      temperature: 0.8,
-      maxTokens: 4096,
-      topP: 0.9,
-    });
-  });
-});

packages/api/src/endpoints/openai/llm.ts

@@ -259,35 +259,9 @@ export function getOpenAILLMConfig({
   }
 
   /**
-   * Note: OpenAI reasoning models (o1/o3/gpt-5) do not support temperature and other sampling parameters
-   * Exception: gpt-5-chat and versioned models like gpt-5.1 DO support these parameters
+   * Note: OpenAI Web Search models do not support any known parameters besides `max_tokens`
    */
-  if (
-    modelOptions.model &&
-    /\b(o[13]|gpt-5)(?!\.|-chat)(?:-|$)/.test(modelOptions.model as string)
-  ) {
-    const reasoningExcludeParams = [
-      'frequencyPenalty',
-      'presencePenalty',
-      'temperature',
-      'topP',
-      'logitBias',
-      'n',
-      'logprobs',
-    ];
-
-    const updatedDropParams = dropParams || [];
-    const combinedDropParams = [...new Set([...updatedDropParams, ...reasoningExcludeParams])];
-
-    combinedDropParams.forEach((param) => {
-      if (param in llmConfig) {
-        delete llmConfig[param as keyof t.OAIClientOptions];
-      }
-    });
-  } else if (modelOptions.model && /gpt-4o.*search/.test(modelOptions.model as string)) {
-    /**
-     * Note: OpenAI Web Search models do not support any known parameters besides `max_tokens`
-     */
+  if (modelOptions.model && /gpt-4o.*search/.test(modelOptions.model as string)) {
     const searchExcludeParams = [
       'frequency_penalty',
       'presence_penalty',

packages/api/src/middleware/index.ts

@@ -1,4 +1,3 @@
 export * from './access';
 export * from './error';
 export * from './balance';
-export * from './json';

packages/api/src/middleware/json.spec.ts

@@ -1,158 +0,0 @@
-import { handleJsonParseError } from './json';
-import type { Request, Response, NextFunction } from 'express';
-
-describe('handleJsonParseError', () => {
-  let req: Partial<Request>;
-  let res: Partial<Response>;
-  let next: NextFunction;
-  let jsonSpy: jest.Mock;
-  let statusSpy: jest.Mock;
-
-  beforeEach(() => {
-    req = {
-      path: '/api/test',
-      method: 'POST',
-      ip: '127.0.0.1',
-    };
-
-    jsonSpy = jest.fn();
-    statusSpy = jest.fn().mockReturnValue({ json: jsonSpy });
-
-    res = {
-      status: statusSpy,
-      json: jsonSpy,
-    };
-
-    next = jest.fn();
-  });
-
-  describe('JSON parse errors', () => {
-    it('should handle JSON SyntaxError with 400 status', () => {
-      const err = new SyntaxError('Unexpected token < in JSON at position 0') as SyntaxError & {
-        status?: number;
-        body?: unknown;
-      };
-      err.status = 400;
-      err.body = {};
-
-      handleJsonParseError(err, req as Request, res as Response, next);
-
-      expect(statusSpy).toHaveBeenCalledWith(400);
-      expect(jsonSpy).toHaveBeenCalledWith({
-        error: 'Invalid JSON format',
-        message: 'The request body contains malformed JSON',
-      });
-      expect(next).not.toHaveBeenCalled();
-    });
-
-    it('should not reflect user input in error message', () => {
-      const maliciousInput = '<script>alert("xss")</script>';
-      const err = new SyntaxError(
-        `Unexpected token < in JSON at position 0: ${maliciousInput}`,
-      ) as SyntaxError & {
-        status?: number;
-        body?: unknown;
-      };
-      err.status = 400;
-      err.body = maliciousInput;
-
-      handleJsonParseError(err, req as Request, res as Response, next);
-
-      expect(statusSpy).toHaveBeenCalledWith(400);
-      const errorResponse = jsonSpy.mock.calls[0][0];
-      expect(errorResponse.message).not.toContain(maliciousInput);
-      expect(errorResponse.message).toBe('The request body contains malformed JSON');
-      expect(next).not.toHaveBeenCalled();
-    });
-
-    it('should handle JSON parse error with HTML tags in body', () => {
-      const err = new SyntaxError('Invalid JSON') as SyntaxError & {
-        status?: number;
-        body?: unknown;
-      };
-      err.status = 400;
-      err.body = '<html><body><h1>XSS</h1></body></html>';
-
-      handleJsonParseError(err, req as Request, res as Response, next);
-
-      expect(statusSpy).toHaveBeenCalledWith(400);
-      const errorResponse = jsonSpy.mock.calls[0][0];
-      expect(errorResponse.message).not.toContain('<html>');
-      expect(errorResponse.message).not.toContain('<script>');
-      expect(next).not.toHaveBeenCalled();
-    });
-  });
-
-  describe('non-JSON errors', () => {
-    it('should pass through non-SyntaxError errors', () => {
-      const err = new Error('Some other error');
-
-      handleJsonParseError(err, req as Request, res as Response, next);
-
-      expect(next).toHaveBeenCalledWith(err);
-      expect(statusSpy).not.toHaveBeenCalled();
-      expect(jsonSpy).not.toHaveBeenCalled();
-    });
-
-    it('should pass through SyntaxError without status 400', () => {
-      const err = new SyntaxError('Some syntax error') as SyntaxError & { status?: number };
-      err.status = 500;
-
-      handleJsonParseError(err, req as Request, res as Response, next);
-
-      expect(next).toHaveBeenCalledWith(err);
-      expect(statusSpy).not.toHaveBeenCalled();
-    });
-
-    it('should pass through SyntaxError without body property', () => {
-      const err = new SyntaxError('Some syntax error') as SyntaxError & { status?: number };
-      err.status = 400;
-
-      handleJsonParseError(err, req as Request, res as Response, next);
-
-      expect(next).toHaveBeenCalledWith(err);
-      expect(statusSpy).not.toHaveBeenCalled();
-    });
-
-    it('should pass through TypeError', () => {
-      const err = new TypeError('Type error');
-
-      handleJsonParseError(err, req as Request, res as Response, next);
-
-      expect(next).toHaveBeenCalledWith(err);
-      expect(statusSpy).not.toHaveBeenCalled();
-    });
-  });
-
-  describe('security verification', () => {
-    it('should return generic error message for all JSON parse errors', () => {
-      const testCases = [
-        'Unexpected token < in JSON',
-        'Unexpected end of JSON input',
-        'Invalid or unexpected token',
-        '<script>alert(1)</script>',
-        '"><img src=x onerror=alert(1)>',
-      ];
-
-      testCases.forEach((errorMsg) => {
-        const err = new SyntaxError(errorMsg) as SyntaxError & {
-          status?: number;
-          body?: unknown;
-        };
-        err.status = 400;
-        err.body = errorMsg;
-
-        jsonSpy.mockClear();
-        statusSpy.mockClear();
-        (next as jest.Mock).mockClear();
-
-        handleJsonParseError(err, req as Request, res as Response, next);
-
-        const errorResponse = jsonSpy.mock.calls[0][0];
-        // Verify the generic message is always returned, not the user input
-        expect(errorResponse.message).toBe('The request body contains malformed JSON');
-        expect(errorResponse.error).toBe('Invalid JSON format');
-      });
-    });
-  });
-});

packages/api/src/middleware/json.ts

@@ -1,40 +0,0 @@
-import { logger } from '@librechat/data-schemas';
-import type { Request, Response, NextFunction } from 'express';
-
-/**
- * Middleware to handle JSON parsing errors from express.json()
- * Prevents user input from being reflected in error messages (XSS prevention)
- *
- * This middleware should be placed immediately after express.json() middleware.
- *
- * @param err - Error object from express.json()
- * @param req - Express request object
- * @param res - Express response object
- * @param next - Express next function
- *
- * @example
- * app.use(express.json({ limit: '3mb' }));
- * app.use(handleJsonParseError);
- */
-export function handleJsonParseError(
-  err: Error & { status?: number; body?: unknown },
-  req: Request,
-  res: Response,
-  next: NextFunction,
-): void {
-  if (err instanceof SyntaxError && err.status === 400 && 'body' in err) {
-    logger.warn('[JSON Parse Error] Invalid JSON received', {
-      path: req.path,
-      method: req.method,
-      ip: req.ip,
-    });
-
-    res.status(400).json({
-      error: 'Invalid JSON format',
-      message: 'The request body contains malformed JSON',
-    });
-    return;
-  }
-
-  next(err);
-}

packages/api/src/prompts/format.ts

@@ -2,7 +2,6 @@ import { SystemCategories } from 'librechat-data-provider';
 import type { IPromptGroupDocument as IPromptGroup } from '@librechat/data-schemas';
 import type { Types } from 'mongoose';
 import type { PromptGroupsListResponse } from '~/types';
-import { escapeRegExp } from '~/utils/common';
 
 /**
  * Formats prompt groups for the paginated /groups endpoint response
@@ -102,6 +101,7 @@ export function buildPromptGroupFilter({
 
   // Handle name filter - convert to regex for case-insensitive search
   if (name) {
+    const escapeRegExp = (str: string) => str.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
     filter.name = new RegExp(escapeRegExp(name), 'i');
   }
 

packages/api/src/prompts/index.ts

@@ -1,3 +1,2 @@
 export * from './format';
 export * from './migration';
-export * from './schemas';

packages/api/src/prompts/schemas.spec.ts

@@ -1,222 +0,0 @@
-import {
-  updatePromptGroupSchema,
-  validatePromptGroupUpdate,
-  safeValidatePromptGroupUpdate,
-} from './schemas';
-
-describe('updatePromptGroupSchema', () => {
-  describe('allowed fields', () => {
-    it('should accept valid name field', () => {
-      const result = updatePromptGroupSchema.safeParse({ name: 'Test Group' });
-      expect(result.success).toBe(true);
-      if (result.success) {
-        expect(result.data.name).toBe('Test Group');
-      }
-    });
-
-    it('should accept valid oneliner field', () => {
-      const result = updatePromptGroupSchema.safeParse({ oneliner: 'A short description' });
-      expect(result.success).toBe(true);
-      if (result.success) {
-        expect(result.data.oneliner).toBe('A short description');
-      }
-    });
-
-    it('should accept valid category field', () => {
-      const result = updatePromptGroupSchema.safeParse({ category: 'testing' });
-      expect(result.success).toBe(true);
-      if (result.success) {
-        expect(result.data.category).toBe('testing');
-      }
-    });
-
-    it('should accept valid projectIds array', () => {
-      const result = updatePromptGroupSchema.safeParse({
-        projectIds: ['proj1', 'proj2'],
-      });
-      expect(result.success).toBe(true);
-      if (result.success) {
-        expect(result.data.projectIds).toEqual(['proj1', 'proj2']);
-      }
-    });
-
-    it('should accept valid removeProjectIds array', () => {
-      const result = updatePromptGroupSchema.safeParse({
-        removeProjectIds: ['proj1'],
-      });
-      expect(result.success).toBe(true);
-      if (result.success) {
-        expect(result.data.removeProjectIds).toEqual(['proj1']);
-      }
-    });
-
-    it('should accept valid command field', () => {
-      const result = updatePromptGroupSchema.safeParse({ command: 'my-command-123' });
-      expect(result.success).toBe(true);
-      if (result.success) {
-        expect(result.data.command).toBe('my-command-123');
-      }
-    });
-
-    it('should accept null command field', () => {
-      const result = updatePromptGroupSchema.safeParse({ command: null });
-      expect(result.success).toBe(true);
-      if (result.success) {
-        expect(result.data.command).toBeNull();
-      }
-    });
-
-    it('should accept multiple valid fields', () => {
-      const input = {
-        name: 'Updated Name',
-        category: 'new-category',
-        oneliner: 'New description',
-      };
-      const result = updatePromptGroupSchema.safeParse(input);
-      expect(result.success).toBe(true);
-      if (result.success) {
-        expect(result.data).toEqual(input);
-      }
-    });
-
-    it('should accept empty object', () => {
-      const result = updatePromptGroupSchema.safeParse({});
-      expect(result.success).toBe(true);
-    });
-  });
-
-  describe('security - strips sensitive fields', () => {
-    it('should reject author field', () => {
-      const result = updatePromptGroupSchema.safeParse({
-        name: 'Test',
-        author: '507f1f77bcf86cd799439011',
-      });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject authorName field', () => {
-      const result = updatePromptGroupSchema.safeParse({
-        name: 'Test',
-        authorName: 'Malicious Author',
-      });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject _id field', () => {
-      const result = updatePromptGroupSchema.safeParse({
-        name: 'Test',
-        _id: '507f1f77bcf86cd799439011',
-      });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject productionId field', () => {
-      const result = updatePromptGroupSchema.safeParse({
-        name: 'Test',
-        productionId: '507f1f77bcf86cd799439011',
-      });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject createdAt field', () => {
-      const result = updatePromptGroupSchema.safeParse({
-        name: 'Test',
-        createdAt: new Date().toISOString(),
-      });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject updatedAt field', () => {
-      const result = updatePromptGroupSchema.safeParse({
-        name: 'Test',
-        updatedAt: new Date().toISOString(),
-      });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject __v field', () => {
-      const result = updatePromptGroupSchema.safeParse({
-        name: 'Test',
-        __v: 999,
-      });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject multiple sensitive fields in a single request', () => {
-      const result = updatePromptGroupSchema.safeParse({
-        name: 'Legit Name',
-        author: '507f1f77bcf86cd799439011',
-        authorName: 'Hacker',
-        _id: 'newid123',
-        productionId: 'prodid456',
-        createdAt: '2020-01-01T00:00:00.000Z',
-        __v: 999,
-      });
-      expect(result.success).toBe(false);
-    });
-  });
-
-  describe('validation rules', () => {
-    it('should reject empty name', () => {
-      const result = updatePromptGroupSchema.safeParse({ name: '' });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject name exceeding max length', () => {
-      const result = updatePromptGroupSchema.safeParse({ name: 'a'.repeat(256) });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject oneliner exceeding max length', () => {
-      const result = updatePromptGroupSchema.safeParse({ oneliner: 'a'.repeat(501) });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject category exceeding max length', () => {
-      const result = updatePromptGroupSchema.safeParse({ category: 'a'.repeat(101) });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject command with invalid characters (uppercase)', () => {
-      const result = updatePromptGroupSchema.safeParse({ command: 'MyCommand' });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject command with invalid characters (spaces)', () => {
-      const result = updatePromptGroupSchema.safeParse({ command: 'my command' });
-      expect(result.success).toBe(false);
-    });
-
-    it('should reject command with invalid characters (special)', () => {
-      const result = updatePromptGroupSchema.safeParse({ command: 'my_command!' });
-      expect(result.success).toBe(false);
-    });
-  });
-});
-
-describe('validatePromptGroupUpdate', () => {
-  it('should return validated data for valid input', () => {
-    const input = { name: 'Test', category: 'testing' };
-    const result = validatePromptGroupUpdate(input);
-    expect(result).toEqual(input);
-  });
-
-  it('should throw ZodError for invalid input', () => {
-    expect(() => validatePromptGroupUpdate({ author: 'malicious-id' })).toThrow();
-  });
-});
-
-describe('safeValidatePromptGroupUpdate', () => {
-  it('should return success true for valid input', () => {
-    const result = safeValidatePromptGroupUpdate({ name: 'Test' });
-    expect(result.success).toBe(true);
-  });
-
-  it('should return success false for invalid input with errors', () => {
-    const result = safeValidatePromptGroupUpdate({ author: 'malicious-id' });
-    expect(result.success).toBe(false);
-    if (!result.success) {
-      expect(result.error.errors.length).toBeGreaterThan(0);
-    }
-  });
-});

packages/api/src/prompts/schemas.ts

@@ -1,53 +0,0 @@
-import { z } from 'zod';
-import { Constants } from 'librechat-data-provider';
-
-/**
- * Schema for validating prompt group update payloads.
- * Only allows fields that users should be able to modify.
- * Sensitive fields like author, authorName, _id, productionId, etc. are excluded.
- */
-export const updatePromptGroupSchema = z
-  .object({
-    /** The name of the prompt group */
-    name: z.string().min(1).max(255).optional(),
-    /** Short description/oneliner for the prompt group */
-    oneliner: z.string().max(500).optional(),
-    /** Category for organizing prompt groups */
-    category: z.string().max(100).optional(),
-    /** Project IDs to add for sharing */
-    projectIds: z.array(z.string()).optional(),
-    /** Project IDs to remove from sharing */
-    removeProjectIds: z.array(z.string()).optional(),
-    /** Command shortcut for the prompt group */
-    command: z
-      .string()
-      .max(Constants.COMMANDS_MAX_LENGTH as number)
-      .regex(/^[a-z0-9-]*$/, {
-        message: 'Command must only contain lowercase alphanumeric characters and hyphens',
-      })
-      .optional()
-      .nullable(),
-  })
-  .strict();
-
-export type TUpdatePromptGroupSchema = z.infer<typeof updatePromptGroupSchema>;
-
-/**
- * Validates and sanitizes a prompt group update payload.
- * Returns only the allowed fields, stripping any sensitive fields.
- * @param data - The raw request body to validate
- * @returns The validated and sanitized payload
- * @throws ZodError if validation fails
- */
-export function validatePromptGroupUpdate(data: unknown): TUpdatePromptGroupSchema {
-  return updatePromptGroupSchema.parse(data);
-}
-
-/**
- * Safely validates a prompt group update payload without throwing.
- * @param data - The raw request body to validate
- * @returns A SafeParseResult with either the validated data or validation errors
- */
-export function safeValidatePromptGroupUpdate(data: unknown) {
-  return updatePromptGroupSchema.safeParse(data);
-}

packages/api/src/utils/common.ts

@@ -48,12 +48,3 @@ export function optionalChainWithEmptyCheck(
   }
   return values[values.length - 1];
 }
-
-/**
- * Escapes special characters in a string for use in a regular expression.
- * @param str - The string to escape.
- * @returns The escaped string safe for use in RegExp.
- */
-export function escapeRegExp(str: string): string {
-  return str.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-}

packages/api/src/utils/index.ts

@@ -17,7 +17,7 @@ export * from './promise';
 export * from './sanitizeTitle';
 export * from './tempChatRetention';
 export * from './text';
-export { default as Tokenizer, countTokens } from './tokenizer';
+export { default as Tokenizer } from './tokenizer';
 export * from './yaml';
 export * from './http';
 export * from './tokens';

packages/api/src/utils/oidc.ts

@@ -36,16 +36,30 @@ const OPENID_TOKEN_FIELDS = [
 
 export function extractOpenIDTokenInfo(user: IUser | null | undefined): OpenIDTokenInfo | null {
   if (!user) {
+    logger.debug('[extractOpenIDTokenInfo] No user provided');
     return null;
   }
 
   try {
+    logger.debug(
+      '[extractOpenIDTokenInfo] User provider:',
+      user.provider,
+      'openidId:',
+      user.openidId,
+    );
+
     if (user.provider !== 'openid' && !user.openidId) {
+      logger.debug('[extractOpenIDTokenInfo] User not authenticated via OpenID');
       return null;
     }
 
     const tokenInfo: OpenIDTokenInfo = {};
 
+    logger.debug(
+      '[extractOpenIDTokenInfo] Checking for federatedTokens in user object:',
+      'federatedTokens' in user,
+    );
+
     if ('federatedTokens' in user && isFederatedTokens(user.federatedTokens)) {
       const tokens = user.federatedTokens;
       logger.debug('[extractOpenIDTokenInfo] Found federatedTokens:', {

packages/api/src/utils/text.spec.ts

@@ -1,851 +0,0 @@
-import { processTextWithTokenLimit, TokenCountFn } from './text';
-import Tokenizer, { countTokens } from './tokenizer';
-
-jest.mock('@librechat/data-schemas', () => ({
-  logger: {
-    debug: jest.fn(),
-    warn: jest.fn(),
-    error: jest.fn(),
-  },
-}));
-
-/**
- * OLD IMPLEMENTATION (Binary Search) - kept for comparison testing
- * This is the original algorithm that caused CPU spikes
- */
-async function processTextWithTokenLimitOLD({
-  text,
-  tokenLimit,
-  tokenCountFn,
-}: {
-  text: string;
-  tokenLimit: number;
-  tokenCountFn: TokenCountFn;
-}): Promise<{ text: string; tokenCount: number; wasTruncated: boolean }> {
-  const originalTokenCount = await tokenCountFn(text);
-
-  if (originalTokenCount <= tokenLimit) {
-    return {
-      text,
-      tokenCount: originalTokenCount,
-      wasTruncated: false,
-    };
-  }
-
-  let low = 0;
-  let high = text.length;
-  let bestText = '';
-
-  while (low <= high) {
-    const mid = Math.floor((low + high) / 2);
-    const truncatedText = text.substring(0, mid);
-    const tokenCount = await tokenCountFn(truncatedText);
-
-    if (tokenCount <= tokenLimit) {
-      bestText = truncatedText;
-      low = mid + 1;
-    } else {
-      high = mid - 1;
-    }
-  }
-
-  const finalTokenCount = await tokenCountFn(bestText);
-
-  return {
-    text: bestText,
-    tokenCount: finalTokenCount,
-    wasTruncated: true,
-  };
-}
-
-/**
- * Creates a wrapper around Tokenizer.getTokenCount that tracks call count
- */
-const createRealTokenCounter = () => {
-  let callCount = 0;
-  const tokenCountFn = (text: string): number => {
-    callCount++;
-    return Tokenizer.getTokenCount(text, 'cl100k_base');
-  };
-  return {
-    tokenCountFn,
-    getCallCount: () => callCount,
-    resetCallCount: () => {
-      callCount = 0;
-    },
-  };
-};
-
-/**
- * Creates a wrapper around the async countTokens function that tracks call count
- */
-const createCountTokensCounter = () => {
-  let callCount = 0;
-  const tokenCountFn = async (text: string): Promise<number> => {
-    callCount++;
-    return countTokens(text);
-  };
-  return {
-    tokenCountFn,
-    getCallCount: () => callCount,
-    resetCallCount: () => {
-      callCount = 0;
-    },
-  };
-};
-
-describe('processTextWithTokenLimit', () => {
-  /**
-   * Creates a mock token count function that simulates realistic token counting.
-   * Roughly 4 characters per token (common for English text).
-   * Tracks call count to verify efficiency.
-   */
-  const createMockTokenCounter = () => {
-    let callCount = 0;
-    const tokenCountFn = (text: string): number => {
-      callCount++;
-      return Math.ceil(text.length / 4);
-    };
-    return {
-      tokenCountFn,
-      getCallCount: () => callCount,
-      resetCallCount: () => {
-        callCount = 0;
-      },
-    };
-  };
-
-  /** Creates a string of specified character length */
-  const createTextOfLength = (charLength: number): string => {
-    return 'a'.repeat(charLength);
-  };
-
-  /** Creates realistic text content with varied token density */
-  const createRealisticText = (approximateTokens: number): string => {
-    const words = [
-      'the',
-      'quick',
-      'brown',
-      'fox',
-      'jumps',
-      'over',
-      'lazy',
-      'dog',
-      'lorem',
-      'ipsum',
-      'dolor',
-      'sit',
-      'amet',
-      'consectetur',
-      'adipiscing',
-      'elit',
-      'sed',
-      'do',
-      'eiusmod',
-      'tempor',
-      'incididunt',
-      'ut',
-      'labore',
-      'et',
-      'dolore',
-      'magna',
-      'aliqua',
-      'enim',
-      'ad',
-      'minim',
-      'veniam',
-      'authentication',
-      'implementation',
-      'configuration',
-      'documentation',
-    ];
-    const result: string[] = [];
-    for (let i = 0; i < approximateTokens; i++) {
-      result.push(words[i % words.length]);
-    }
-    return result.join(' ');
-  };
-
-  describe('tokenCountFn flexibility (sync and async)', () => {
-    it('should work with synchronous tokenCountFn', async () => {
-      const syncTokenCountFn = (text: string): number => Math.ceil(text.length / 4);
-      const text = 'Hello, world! This is a test message.';
-      const tokenLimit = 5;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: syncTokenCountFn,
-      });
-
-      expect(result.wasTruncated).toBe(true);
-      expect(result.tokenCount).toBeLessThanOrEqual(tokenLimit);
-    });
-
-    it('should work with asynchronous tokenCountFn', async () => {
-      const asyncTokenCountFn = async (text: string): Promise<number> => {
-        await new Promise((resolve) => setTimeout(resolve, 1));
-        return Math.ceil(text.length / 4);
-      };
-      const text = 'Hello, world! This is a test message.';
-      const tokenLimit = 5;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: asyncTokenCountFn,
-      });
-
-      expect(result.wasTruncated).toBe(true);
-      expect(result.tokenCount).toBeLessThanOrEqual(tokenLimit);
-    });
-
-    it('should produce equivalent results with sync and async tokenCountFn', async () => {
-      const syncTokenCountFn = (text: string): number => Math.ceil(text.length / 4);
-      const asyncTokenCountFn = async (text: string): Promise<number> => Math.ceil(text.length / 4);
-      const text = 'a'.repeat(8000);
-      const tokenLimit = 1000;
-
-      const syncResult = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: syncTokenCountFn,
-      });
-
-      const asyncResult = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: asyncTokenCountFn,
-      });
-
-      expect(syncResult.tokenCount).toBe(asyncResult.tokenCount);
-      expect(syncResult.wasTruncated).toBe(asyncResult.wasTruncated);
-      expect(syncResult.text.length).toBe(asyncResult.text.length);
-    });
-  });
-
-  describe('when text is under the token limit', () => {
-    it('should return original text unchanged', async () => {
-      const { tokenCountFn } = createMockTokenCounter();
-      const text = 'Hello, world!';
-      const tokenLimit = 100;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(result.text).toBe(text);
-      expect(result.wasTruncated).toBe(false);
-    });
-
-    it('should return correct token count', async () => {
-      const { tokenCountFn } = createMockTokenCounter();
-      const text = 'Hello, world!';
-      const tokenLimit = 100;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(result.tokenCount).toBe(Math.ceil(text.length / 4));
-    });
-
-    it('should only call tokenCountFn once when under limit', async () => {
-      const { tokenCountFn, getCallCount } = createMockTokenCounter();
-      const text = 'Hello, world!';
-      const tokenLimit = 100;
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(getCallCount()).toBe(1);
-    });
-  });
-
-  describe('when text is exactly at the token limit', () => {
-    it('should return original text unchanged', async () => {
-      const { tokenCountFn } = createMockTokenCounter();
-      const text = createTextOfLength(400);
-      const tokenLimit = 100;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(result.text).toBe(text);
-      expect(result.wasTruncated).toBe(false);
-      expect(result.tokenCount).toBe(tokenLimit);
-    });
-  });
-
-  describe('when text exceeds the token limit', () => {
-    it('should truncate text to fit within limit', async () => {
-      const { tokenCountFn } = createMockTokenCounter();
-      const text = createTextOfLength(8000);
-      const tokenLimit = 1000;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(result.wasTruncated).toBe(true);
-      expect(result.tokenCount).toBeLessThanOrEqual(tokenLimit);
-      expect(result.text.length).toBeLessThan(text.length);
-    });
-
-    it('should truncate text to be close to but not exceed the limit', async () => {
-      const { tokenCountFn } = createMockTokenCounter();
-      const text = createTextOfLength(8000);
-      const tokenLimit = 1000;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(result.tokenCount).toBeLessThanOrEqual(tokenLimit);
-      expect(result.tokenCount).toBeGreaterThan(tokenLimit * 0.9);
-    });
-  });
-
-  describe('efficiency - tokenCountFn call count', () => {
-    it('should call tokenCountFn at most 7 times for large text (vs ~17 for binary search)', async () => {
-      const { tokenCountFn, getCallCount } = createMockTokenCounter();
-      const text = createTextOfLength(400000);
-      const tokenLimit = 50000;
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(getCallCount()).toBeLessThanOrEqual(7);
-    });
-
-    it('should typically call tokenCountFn only 2-3 times for standard truncation', async () => {
-      const { tokenCountFn, getCallCount } = createMockTokenCounter();
-      const text = createTextOfLength(40000);
-      const tokenLimit = 5000;
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(getCallCount()).toBeLessThanOrEqual(3);
-    });
-
-    it('should call tokenCountFn only once when text is under limit', async () => {
-      const { tokenCountFn, getCallCount } = createMockTokenCounter();
-      const text = createTextOfLength(1000);
-      const tokenLimit = 10000;
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(getCallCount()).toBe(1);
-    });
-
-    it('should handle very large text (100k+ tokens) efficiently', async () => {
-      const { tokenCountFn, getCallCount } = createMockTokenCounter();
-      const text = createTextOfLength(500000);
-      const tokenLimit = 100000;
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(getCallCount()).toBeLessThanOrEqual(7);
-    });
-  });
-
-  describe('edge cases', () => {
-    it('should handle empty text', async () => {
-      const { tokenCountFn } = createMockTokenCounter();
-      const text = '';
-      const tokenLimit = 100;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(result.text).toBe('');
-      expect(result.tokenCount).toBe(0);
-      expect(result.wasTruncated).toBe(false);
-    });
-
-    it('should handle token limit of 1', async () => {
-      const { tokenCountFn } = createMockTokenCounter();
-      const text = createTextOfLength(1000);
-      const tokenLimit = 1;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(result.wasTruncated).toBe(true);
-      expect(result.tokenCount).toBeLessThanOrEqual(tokenLimit);
-    });
-
-    it('should handle text that is just slightly over the limit', async () => {
-      const { tokenCountFn } = createMockTokenCounter();
-      const text = createTextOfLength(404);
-      const tokenLimit = 100;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn,
-      });
-
-      expect(result.wasTruncated).toBe(true);
-      expect(result.tokenCount).toBeLessThanOrEqual(tokenLimit);
-    });
-  });
-
-  describe('correctness with variable token density', () => {
-    it('should handle text with varying token density', async () => {
-      const variableDensityTokenCounter = (text: string): number => {
-        const shortWords = (text.match(/\s+/g) || []).length;
-        return Math.ceil(text.length / 4) + shortWords;
-      };
-
-      const text = 'This is a test with many short words and some longer concatenated words too';
-      const tokenLimit = 10;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: variableDensityTokenCounter,
-      });
-
-      expect(result.wasTruncated).toBe(true);
-      expect(result.tokenCount).toBeLessThanOrEqual(tokenLimit);
-    });
-  });
-
-  describe('direct comparison with OLD binary search implementation', () => {
-    it('should produce equivalent results to the old implementation', async () => {
-      const oldCounter = createMockTokenCounter();
-      const newCounter = createMockTokenCounter();
-      const text = createTextOfLength(8000);
-      const tokenLimit = 1000;
-
-      const oldResult = await processTextWithTokenLimitOLD({
-        text,
-        tokenLimit,
-        tokenCountFn: oldCounter.tokenCountFn,
-      });
-
-      const newResult = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: newCounter.tokenCountFn,
-      });
-
-      expect(newResult.wasTruncated).toBe(oldResult.wasTruncated);
-      expect(newResult.tokenCount).toBeLessThanOrEqual(tokenLimit);
-      expect(oldResult.tokenCount).toBeLessThanOrEqual(tokenLimit);
-    });
-
-    it('should use significantly fewer tokenCountFn calls than old implementation (400k chars)', async () => {
-      const oldCounter = createMockTokenCounter();
-      const newCounter = createMockTokenCounter();
-      const text = createTextOfLength(400000);
-      const tokenLimit = 50000;
-
-      await processTextWithTokenLimitOLD({
-        text,
-        tokenLimit,
-        tokenCountFn: oldCounter.tokenCountFn,
-      });
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: newCounter.tokenCountFn,
-      });
-
-      const oldCalls = oldCounter.getCallCount();
-      const newCalls = newCounter.getCallCount();
-
-      console.log(
-        `[400k chars] OLD implementation: ${oldCalls} calls, NEW implementation: ${newCalls} calls`,
-      );
-      console.log(`[400k chars] Reduction: ${((1 - newCalls / oldCalls) * 100).toFixed(1)}%`);
-
-      expect(newCalls).toBeLessThan(oldCalls);
-      expect(newCalls).toBeLessThanOrEqual(7);
-    });
-
-    it('should use significantly fewer tokenCountFn calls than old implementation (500k chars, 100k token limit)', async () => {
-      const oldCounter = createMockTokenCounter();
-      const newCounter = createMockTokenCounter();
-      const text = createTextOfLength(500000);
-      const tokenLimit = 100000;
-
-      await processTextWithTokenLimitOLD({
-        text,
-        tokenLimit,
-        tokenCountFn: oldCounter.tokenCountFn,
-      });
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: newCounter.tokenCountFn,
-      });
-
-      const oldCalls = oldCounter.getCallCount();
-      const newCalls = newCounter.getCallCount();
-
-      console.log(
-        `[500k chars] OLD implementation: ${oldCalls} calls, NEW implementation: ${newCalls} calls`,
-      );
-      console.log(`[500k chars] Reduction: ${((1 - newCalls / oldCalls) * 100).toFixed(1)}%`);
-
-      expect(newCalls).toBeLessThan(oldCalls);
-    });
-
-    it('should achieve at least 70% reduction in tokenCountFn calls', async () => {
-      const oldCounter = createMockTokenCounter();
-      const newCounter = createMockTokenCounter();
-      const text = createTextOfLength(500000);
-      const tokenLimit = 100000;
-
-      await processTextWithTokenLimitOLD({
-        text,
-        tokenLimit,
-        tokenCountFn: oldCounter.tokenCountFn,
-      });
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: newCounter.tokenCountFn,
-      });
-
-      const oldCalls = oldCounter.getCallCount();
-      const newCalls = newCounter.getCallCount();
-      const reduction = 1 - newCalls / oldCalls;
-
-      console.log(
-        `Efficiency improvement: ${(reduction * 100).toFixed(1)}% fewer tokenCountFn calls`,
-      );
-
-      expect(reduction).toBeGreaterThanOrEqual(0.7);
-    });
-
-    it('should simulate the reported scenario (122k tokens, 100k limit)', async () => {
-      const oldCounter = createMockTokenCounter();
-      const newCounter = createMockTokenCounter();
-      const text = createTextOfLength(489564);
-      const tokenLimit = 100000;
-
-      await processTextWithTokenLimitOLD({
-        text,
-        tokenLimit,
-        tokenCountFn: oldCounter.tokenCountFn,
-      });
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: newCounter.tokenCountFn,
-      });
-
-      const oldCalls = oldCounter.getCallCount();
-      const newCalls = newCounter.getCallCount();
-
-      console.log(`[User reported scenario: ~122k tokens]`);
-      console.log(`OLD implementation: ${oldCalls} tokenCountFn calls`);
-      console.log(`NEW implementation: ${newCalls} tokenCountFn calls`);
-      console.log(`Reduction: ${((1 - newCalls / oldCalls) * 100).toFixed(1)}%`);
-
-      expect(newCalls).toBeLessThan(oldCalls);
-      expect(newCalls).toBeLessThanOrEqual(7);
-    });
-  });
-
-  describe('direct comparison with REAL tiktoken tokenizer', () => {
-    beforeEach(() => {
-      Tokenizer.freeAndResetAllEncoders();
-    });
-
-    it('should produce valid truncation with real tokenizer', async () => {
-      const counter = createRealTokenCounter();
-      const text = createRealisticText(5000);
-      const tokenLimit = 1000;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: counter.tokenCountFn,
-      });
-
-      expect(result.wasTruncated).toBe(true);
-      expect(result.tokenCount).toBeLessThanOrEqual(tokenLimit);
-      expect(result.text.length).toBeLessThan(text.length);
-    });
-
-    it('should use fewer tiktoken calls than old implementation (realistic text)', async () => {
-      const oldCounter = createRealTokenCounter();
-      const newCounter = createRealTokenCounter();
-      const text = createRealisticText(15000);
-      const tokenLimit = 5000;
-
-      await processTextWithTokenLimitOLD({
-        text,
-        tokenLimit,
-        tokenCountFn: oldCounter.tokenCountFn,
-      });
-
-      Tokenizer.freeAndResetAllEncoders();
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: newCounter.tokenCountFn,
-      });
-
-      const oldCalls = oldCounter.getCallCount();
-      const newCalls = newCounter.getCallCount();
-
-      console.log(`[Real tiktoken ~15k tokens] OLD: ${oldCalls} calls, NEW: ${newCalls} calls`);
-      console.log(`[Real tiktoken] Reduction: ${((1 - newCalls / oldCalls) * 100).toFixed(1)}%`);
-
-      expect(newCalls).toBeLessThan(oldCalls);
-    });
-
-    it('should handle the reported user scenario with real tokenizer (~120k tokens)', async () => {
-      const oldCounter = createRealTokenCounter();
-      const newCounter = createRealTokenCounter();
-      const text = createRealisticText(120000);
-      const tokenLimit = 100000;
-
-      const startOld = performance.now();
-      await processTextWithTokenLimitOLD({
-        text,
-        tokenLimit,
-        tokenCountFn: oldCounter.tokenCountFn,
-      });
-      const timeOld = performance.now() - startOld;
-
-      Tokenizer.freeAndResetAllEncoders();
-
-      const startNew = performance.now();
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: newCounter.tokenCountFn,
-      });
-      const timeNew = performance.now() - startNew;
-
-      const oldCalls = oldCounter.getCallCount();
-      const newCalls = newCounter.getCallCount();
-
-      console.log(`\n[REAL TIKTOKEN - User reported scenario: ~120k tokens]`);
-      console.log(`OLD implementation: ${oldCalls} tiktoken calls, ${timeOld.toFixed(0)}ms`);
-      console.log(`NEW implementation: ${newCalls} tiktoken calls, ${timeNew.toFixed(0)}ms`);
-      console.log(`Call reduction: ${((1 - newCalls / oldCalls) * 100).toFixed(1)}%`);
-      console.log(`Time reduction: ${((1 - timeNew / timeOld) * 100).toFixed(1)}%`);
-      console.log(
-        `Result: truncated=${result.wasTruncated}, tokens=${result.tokenCount}/${tokenLimit}\n`,
-      );
-
-      expect(newCalls).toBeLessThan(oldCalls);
-      expect(result.tokenCount).toBeLessThanOrEqual(tokenLimit);
-      expect(newCalls).toBeLessThanOrEqual(7);
-    });
-
-    it('should achieve at least 70% reduction with real tokenizer', async () => {
-      const oldCounter = createRealTokenCounter();
-      const newCounter = createRealTokenCounter();
-      const text = createRealisticText(50000);
-      const tokenLimit = 10000;
-
-      await processTextWithTokenLimitOLD({
-        text,
-        tokenLimit,
-        tokenCountFn: oldCounter.tokenCountFn,
-      });
-
-      Tokenizer.freeAndResetAllEncoders();
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: newCounter.tokenCountFn,
-      });
-
-      const oldCalls = oldCounter.getCallCount();
-      const newCalls = newCounter.getCallCount();
-      const reduction = 1 - newCalls / oldCalls;
-
-      console.log(
-        `[Real tiktoken 50k tokens] OLD: ${oldCalls}, NEW: ${newCalls}, Reduction: ${(reduction * 100).toFixed(1)}%`,
-      );
-
-      expect(reduction).toBeGreaterThanOrEqual(0.7);
-    });
-  });
-
-  describe('using countTokens async function from @librechat/api', () => {
-    beforeEach(() => {
-      Tokenizer.freeAndResetAllEncoders();
-    });
-
-    it('countTokens should return correct token count', async () => {
-      const text = 'Hello, world!';
-      const count = await countTokens(text);
-
-      expect(count).toBeGreaterThan(0);
-      expect(typeof count).toBe('number');
-    });
-
-    it('countTokens should handle empty string', async () => {
-      const count = await countTokens('');
-      expect(count).toBe(0);
-    });
-
-    it('should work with processTextWithTokenLimit using countTokens', async () => {
-      const counter = createCountTokensCounter();
-      const text = createRealisticText(5000);
-      const tokenLimit = 1000;
-
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: counter.tokenCountFn,
-      });
-
-      expect(result.wasTruncated).toBe(true);
-      expect(result.tokenCount).toBeLessThanOrEqual(tokenLimit);
-      expect(result.text.length).toBeLessThan(text.length);
-    });
-
-    it('should use fewer countTokens calls than old implementation', async () => {
-      const oldCounter = createCountTokensCounter();
-      const newCounter = createCountTokensCounter();
-      const text = createRealisticText(15000);
-      const tokenLimit = 5000;
-
-      await processTextWithTokenLimitOLD({
-        text,
-        tokenLimit,
-        tokenCountFn: oldCounter.tokenCountFn,
-      });
-
-      Tokenizer.freeAndResetAllEncoders();
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: newCounter.tokenCountFn,
-      });
-
-      const oldCalls = oldCounter.getCallCount();
-      const newCalls = newCounter.getCallCount();
-
-      console.log(`[countTokens ~15k tokens] OLD: ${oldCalls} calls, NEW: ${newCalls} calls`);
-      console.log(`[countTokens] Reduction: ${((1 - newCalls / oldCalls) * 100).toFixed(1)}%`);
-
-      expect(newCalls).toBeLessThan(oldCalls);
-    });
-
-    it('should handle user reported scenario with countTokens (~120k tokens)', async () => {
-      const oldCounter = createCountTokensCounter();
-      const newCounter = createCountTokensCounter();
-      const text = createRealisticText(120000);
-      const tokenLimit = 100000;
-
-      const startOld = performance.now();
-      await processTextWithTokenLimitOLD({
-        text,
-        tokenLimit,
-        tokenCountFn: oldCounter.tokenCountFn,
-      });
-      const timeOld = performance.now() - startOld;
-
-      Tokenizer.freeAndResetAllEncoders();
-
-      const startNew = performance.now();
-      const result = await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: newCounter.tokenCountFn,
-      });
-      const timeNew = performance.now() - startNew;
-
-      const oldCalls = oldCounter.getCallCount();
-      const newCalls = newCounter.getCallCount();
-
-      console.log(`\n[countTokens - User reported scenario: ~120k tokens]`);
-      console.log(`OLD implementation: ${oldCalls} countTokens calls, ${timeOld.toFixed(0)}ms`);
-      console.log(`NEW implementation: ${newCalls} countTokens calls, ${timeNew.toFixed(0)}ms`);
-      console.log(`Call reduction: ${((1 - newCalls / oldCalls) * 100).toFixed(1)}%`);
-      console.log(`Time reduction: ${((1 - timeNew / timeOld) * 100).toFixed(1)}%`);
-      console.log(
-        `Result: truncated=${result.wasTruncated}, tokens=${result.tokenCount}/${tokenLimit}\n`,
-      );
-
-      expect(newCalls).toBeLessThan(oldCalls);
-      expect(result.tokenCount).toBeLessThanOrEqual(tokenLimit);
-      expect(newCalls).toBeLessThanOrEqual(7);
-    });
-
-    it('should achieve at least 70% reduction with countTokens', async () => {
-      const oldCounter = createCountTokensCounter();
-      const newCounter = createCountTokensCounter();
-      const text = createRealisticText(50000);
-      const tokenLimit = 10000;
-
-      await processTextWithTokenLimitOLD({
-        text,
-        tokenLimit,
-        tokenCountFn: oldCounter.tokenCountFn,
-      });
-
-      Tokenizer.freeAndResetAllEncoders();
-
-      await processTextWithTokenLimit({
-        text,
-        tokenLimit,
-        tokenCountFn: newCounter.tokenCountFn,
-      });
-
-      const oldCalls = oldCounter.getCallCount();
-      const newCalls = newCounter.getCallCount();
-      const reduction = 1 - newCalls / oldCalls;
-
-      console.log(
-        `[countTokens 50k tokens] OLD: ${oldCalls}, NEW: ${newCalls}, Reduction: ${(reduction * 100).toFixed(1)}%`,
-      );
-
-      expect(reduction).toBeGreaterThanOrEqual(0.7);
-    });
-  });
-});

packages/api/src/utils/text.ts

@@ -1,39 +1,11 @@
 import { logger } from '@librechat/data-schemas';
 
-/** Token count function that can be sync or async */
-export type TokenCountFn = (text: string) => number | Promise<number>;
-
-/**
- * Safety buffer multiplier applied to character position estimates during truncation.
- *
- * We use 98% (0.98) rather than 100% to intentionally undershoot the target on the first attempt.
- * This is necessary because:
- * - Token density varies across text (some regions may have more tokens per character than the average)
- * - The ratio-based estimate assumes uniform token distribution, which is rarely true
- * - Undershooting is safer than overshooting: exceeding the limit requires another iteration,
- *   while being slightly under is acceptable
- * - In practice, this buffer reduces refinement iterations from 2-3 down to 0-1 in most cases
- *
- * @example
- * // If text has 1000 chars and 250 tokens (4 chars/token average), targeting 100 tokens:
- * // Without buffer: estimate = 1000 * (100/250) = 400 chars → might yield 105 tokens (over!)
- * // With 0.98 buffer: estimate = 400 * 0.98 = 392 chars → likely yields 97-99 tokens (safe)
- */
-const TRUNCATION_SAFETY_BUFFER = 0.98;
-
 /**
  * Processes text content by counting tokens and truncating if it exceeds the specified limit.
- * Uses ratio-based estimation to minimize expensive tokenCountFn calls.
- *
  * @param text - The text content to process
  * @param tokenLimit - The maximum number of tokens allowed
- * @param tokenCountFn - Function to count tokens (can be sync or async)
+ * @param tokenCountFn - Function to count tokens
  * @returns Promise resolving to object with processed text, token count, and truncation status
- *
- * @remarks
- * This function uses a ratio-based estimation algorithm instead of binary search.
- * Binary search would require O(log n) tokenCountFn calls (~17 for 100k chars),
- * while this approach typically requires only 2-3 calls for a 90%+ reduction in CPU usage.
  */
 export async function processTextWithTokenLimit({
   text,
@@ -42,7 +14,7 @@ export async function processTextWithTokenLimit({
 }: {
   text: string;
   tokenLimit: number;
-  tokenCountFn: TokenCountFn;
+  tokenCountFn: (text: string) => number;
 }): Promise<{ text: string; tokenCount: number; wasTruncated: boolean }> {
   const originalTokenCount = await tokenCountFn(text);
 
@@ -54,34 +26,40 @@ export async function processTextWithTokenLimit({
     };
   }
 
+  /**
+   * Doing binary search here to find the truncation point efficiently
+   * (May be a better way to go about this)
+   */
+  let low = 0;
+  let high = text.length;
+  let bestText = '';
+
   logger.debug(
     `[textTokenLimiter] Text content exceeds token limit: ${originalTokenCount} > ${tokenLimit}, truncating...`,
   );
 
-  const ratio = tokenLimit / originalTokenCount;
-  let charPosition = Math.floor(text.length * ratio * TRUNCATION_SAFETY_BUFFER);
+  while (low <= high) {
+    const mid = Math.floor((low + high) / 2);
+    const truncatedText = text.substring(0, mid);
+    const tokenCount = await tokenCountFn(truncatedText);
 
-  let truncatedText = text.substring(0, charPosition);
-  let tokenCount = await tokenCountFn(truncatedText);
-
-  const maxIterations = 5;
-  let iterations = 0;
-
-  while (tokenCount > tokenLimit && iterations < maxIterations && charPosition > 0) {
-    const overageRatio = tokenLimit / tokenCount;
-    charPosition = Math.floor(charPosition * overageRatio * TRUNCATION_SAFETY_BUFFER);
-    truncatedText = text.substring(0, charPosition);
-    tokenCount = await tokenCountFn(truncatedText);
-    iterations++;
+    if (tokenCount <= tokenLimit) {
+      bestText = truncatedText;
+      low = mid + 1;
+    } else {
+      high = mid - 1;
+    }
   }
 
+  const finalTokenCount = await tokenCountFn(bestText);
+
   logger.warn(
-    `[textTokenLimiter] Text truncated from ${originalTokenCount} to ${tokenCount} tokens (limit: ${tokenLimit})`,
+    `[textTokenLimiter] Text truncated from ${originalTokenCount} to ${finalTokenCount} tokens (limit: ${tokenLimit})`,
   );
 
   return {
-    text: truncatedText,
-    tokenCount,
+    text: bestText,
+    tokenCount: finalTokenCount,
     wasTruncated: true,
   };
 }

packages/api/src/utils/tokenizer.ts

@@ -75,14 +75,4 @@ class Tokenizer {
 
 const TokenizerSingleton = new Tokenizer();
 
-/**
- * Counts the number of tokens in a given text using tiktoken.
- * This is an async wrapper around Tokenizer.getTokenCount for compatibility.
- * @param text - The text to be tokenized. Defaults to an empty string if not provided.
- * @returns The number of tokens in the provided text.
- */
-export async function countTokens(text = ''): Promise<number> {
-  return TokenizerSingleton.getTokenCount(text, 'cl100k_base');
-}
-
 export default TokenizerSingleton;

packages/api/src/utils/tokens.ts

@@ -140,7 +140,6 @@ const anthropicModels = {
 
 const deepseekModels = {
   deepseek: 128000,
-  'deepseek-chat': 128000,
   'deepseek-reasoner': 128000,
   'deepseek-r1': 128000,
   'deepseek-v3': 128000,
@@ -281,9 +280,6 @@ const xAIModels = {
   'grok-3-mini': 131072,
   'grok-3-mini-fast': 131072,
   'grok-4': 256000, // 256K context
-  'grok-4-fast': 2000000, // 2M context
-  'grok-4-1-fast': 2000000, // 2M context (covers reasoning & non-reasoning variants)
-  'grok-code-fast': 256000, // 256K context
 };
 
 const aggregateModels = {
@@ -348,21 +344,11 @@ const anthropicMaxOutputs = {
   'claude-3-7-sonnet': 128000,
 };
 
-/** Outputs from https://api-docs.deepseek.com/quick_start/pricing */
-const deepseekMaxOutputs = {
-  deepseek: 8000, // deepseek-chat default: 4K, max: 8K
-  'deepseek-chat': 8000,
-  'deepseek-reasoner': 64000, // default: 32K, max: 64K
-  'deepseek-r1': 64000,
-  'deepseek-v3': 8000,
-  'deepseek.r1': 64000,
-};
-
 export const maxOutputTokensMap = {
   [EModelEndpoint.anthropic]: anthropicMaxOutputs,
   [EModelEndpoint.azureOpenAI]: modelMaxOutputs,
-  [EModelEndpoint.openAI]: { ...modelMaxOutputs, ...deepseekMaxOutputs },
-  [EModelEndpoint.custom]: { ...modelMaxOutputs, ...deepseekMaxOutputs },
+  [EModelEndpoint.openAI]: modelMaxOutputs,
+  [EModelEndpoint.custom]: modelMaxOutputs,
 };
 
 /**

packages/client/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@librechat/client",
-  "version": "0.4.0",
+  "version": "0.3.2",
   "description": "React components for LibreChat",
   "main": "dist/index.js",
   "module": "dist/index.es.js",
@@ -51,7 +51,7 @@
     "@tanstack/react-virtual": "^3.0.0",
     "class-variance-authority": "^0.7.1",
     "clsx": "^2.1.1",
-    "dompurify": "^3.3.0",
+    "dompurify": "^3.2.6",
     "framer-motion": "^12.23.6",
     "i18next": "^24.2.2 || ^25.3.2",
     "i18next-browser-languagedetector": "^8.2.0",

packages/data-provider/package.json

@@ -1,6 +1,6 @@
 {
   "name": "librechat-data-provider",
-  "version": "0.8.100",
+  "version": "0.8.020",
   "description": "data services for librechat apps",
   "main": "dist/index.js",
   "module": "dist/index.es.js",

packages/data-provider/specs/parsers.spec.ts

@@ -1,7 +1,6 @@
-import { replaceSpecialVars, parseCompactConvo } from '../src/parsers';
+import { replaceSpecialVars } from '../src/parsers';
 import { specialVariables } from '../src/config';
-import { EModelEndpoint } from '../src/schemas';
-import type { TUser, TConversation } from '../src/types';
+import type { TUser } from '../src/types';
 
 // Mock dayjs module with consistent date/time values regardless of environment
 jest.mock('dayjs', () => {
@@ -124,138 +123,3 @@ describe('replaceSpecialVars', () => {
     expect(result).toContain('Test User'); // current_user
   });
 });
-
-describe('parseCompactConvo', () => {
-  describe('iconURL security sanitization', () => {
-    test('should strip iconURL from OpenAI endpoint conversation input', () => {
-      const maliciousIconURL = 'https://evil-tracker.example.com/pixel.png?user=victim';
-      const conversation: Partial<TConversation> = {
-        model: 'gpt-4',
-        iconURL: maliciousIconURL,
-        endpoint: EModelEndpoint.openAI,
-      };
-
-      const result = parseCompactConvo({
-        endpoint: EModelEndpoint.openAI,
-        conversation,
-      });
-
-      expect(result).not.toBeNull();
-      expect(result?.iconURL).toBeUndefined();
-      expect(result?.model).toBe('gpt-4');
-    });
-
-    test('should strip iconURL from agents endpoint conversation input', () => {
-      const maliciousIconURL = 'https://evil-tracker.example.com/pixel.png';
-      const conversation: Partial<TConversation> = {
-        agent_id: 'agent_123',
-        iconURL: maliciousIconURL,
-        endpoint: EModelEndpoint.agents,
-      };
-
-      const result = parseCompactConvo({
-        endpoint: EModelEndpoint.agents,
-        conversation,
-      });
-
-      expect(result).not.toBeNull();
-      expect(result?.iconURL).toBeUndefined();
-      expect(result?.agent_id).toBe('agent_123');
-    });
-
-    test('should strip iconURL from anthropic endpoint conversation input', () => {
-      const maliciousIconURL = 'https://tracker.malicious.com/beacon.gif';
-      const conversation: Partial<TConversation> = {
-        model: 'claude-3-opus',
-        iconURL: maliciousIconURL,
-        endpoint: EModelEndpoint.anthropic,
-      };
-
-      const result = parseCompactConvo({
-        endpoint: EModelEndpoint.anthropic,
-        conversation,
-      });
-
-      expect(result).not.toBeNull();
-      expect(result?.iconURL).toBeUndefined();
-      expect(result?.model).toBe('claude-3-opus');
-    });
-
-    test('should strip iconURL from google endpoint conversation input', () => {
-      const maliciousIconURL = 'https://tracking.example.com/spy.png';
-      const conversation: Partial<TConversation> = {
-        model: 'gemini-pro',
-        iconURL: maliciousIconURL,
-        endpoint: EModelEndpoint.google,
-      };
-
-      const result = parseCompactConvo({
-        endpoint: EModelEndpoint.google,
-        conversation,
-      });
-
-      expect(result).not.toBeNull();
-      expect(result?.iconURL).toBeUndefined();
-      expect(result?.model).toBe('gemini-pro');
-    });
-
-    test('should strip iconURL from assistants endpoint conversation input', () => {
-      const maliciousIconURL = 'https://evil.com/track.png';
-      const conversation: Partial<TConversation> = {
-        assistant_id: 'asst_123',
-        iconURL: maliciousIconURL,
-        endpoint: EModelEndpoint.assistants,
-      };
-
-      const result = parseCompactConvo({
-        endpoint: EModelEndpoint.assistants,
-        conversation,
-      });
-
-      expect(result).not.toBeNull();
-      expect(result?.iconURL).toBeUndefined();
-      expect(result?.assistant_id).toBe('asst_123');
-    });
-
-    test('should preserve other conversation properties while stripping iconURL', () => {
-      const conversation: Partial<TConversation> = {
-        model: 'gpt-4',
-        iconURL: 'https://malicious.com/track.png',
-        endpoint: EModelEndpoint.openAI,
-        temperature: 0.7,
-        top_p: 0.9,
-        promptPrefix: 'You are a helpful assistant.',
-        maxContextTokens: 4000,
-      };
-
-      const result = parseCompactConvo({
-        endpoint: EModelEndpoint.openAI,
-        conversation,
-      });
-
-      expect(result).not.toBeNull();
-      expect(result?.iconURL).toBeUndefined();
-      expect(result?.model).toBe('gpt-4');
-      expect(result?.temperature).toBe(0.7);
-      expect(result?.top_p).toBe(0.9);
-      expect(result?.promptPrefix).toBe('You are a helpful assistant.');
-      expect(result?.maxContextTokens).toBe(4000);
-    });
-
-    test('should handle conversation without iconURL (no error)', () => {
-      const conversation: Partial<TConversation> = {
-        model: 'gpt-4',
-        endpoint: EModelEndpoint.openAI,
-      };
-
-      const result = parseCompactConvo({
-        endpoint: EModelEndpoint.openAI,
-        conversation,
-      });
-
-      expect(result).not.toBeNull();
-      expect(result?.iconURL).toBeUndefined();
-      expect(result?.model).toBe('gpt-4');
-    });
-  });
-});

packages/data-provider/src/config.ts

@@ -1585,7 +1585,7 @@ export enum TTSProviders {
 /** Enum for app-wide constants */
 export enum Constants {
   /** Key for the app's version. */
-  VERSION = 'v0.8.1-rc2',
+  VERSION = 'v0.8.1-rc1',
   /** Key for the Custom Config's version (librechat.yaml). */
   CONFIG_VERSION = '1.3.1',
   /** Standard value for the first message's `parentMessageId` value, to indicate no parent exists. */

packages/data-provider/src/data-service.ts

@@ -45,12 +45,8 @@ export function getSharedLink(conversationId: string): Promise<t.TSharedLinkGetR
 export function createSharedLink(
   conversationId: string,
   targetMessageId?: string,
-  expirationHours?: number,
 ): Promise<t.TSharedLinkResponse> {
-  return request.post(endpoints.createSharedLink(conversationId), {
-    targetMessageId,
-    expirationHours,
-  });
+  return request.post(endpoints.createSharedLink(conversationId), { targetMessageId });
 }
 
 export function updateSharedLink(shareId: string): Promise<t.TSharedLinkResponse> {

packages/data-provider/src/parsers.ts

@@ -326,7 +326,7 @@ export const parseCompactConvo = ({
   possibleValues?: TPossibleValues;
   // TODO: POC for default schema
   // defaultSchema?: Partial<EndpointSchema>,
-}): Omit<s.TConversation, 'iconURL'> | null => {
+}) => {
   if (!endpoint) {
     throw new Error(`undefined endpoint: ${endpoint}`);
   }
@@ -343,11 +343,7 @@ export const parseCompactConvo = ({
     throw new Error(`Unknown endpointType: ${endpointType}`);
   }
 
-  // Strip iconURL from input before parsing - it should only be derived server-side
-  // from model spec configuration, not accepted from client requests
-  const { iconURL: _clientIconURL, ...conversationWithoutIconURL } = conversation;
-
-  const convo = schema.parse(conversationWithoutIconURL) as s.TConversation | null;
+  const convo = schema.parse(conversation) as s.TConversation | null;
   // const { models, secondaryModels } = possibleValues ?? {};
   const { models } = possibleValues ?? {};
 

packages/data-provider/src/schemas.ts

@@ -41,6 +41,7 @@ export enum Providers {
   BEDROCK = 'bedrock',
   MISTRALAI = 'mistralai',
   MISTRAL = 'mistral',
+  OLLAMA = 'ollama',
   DEEPSEEK = 'deepseek',
   OPENROUTER = 'openrouter',
   XAI = 'xai',
@@ -58,6 +59,7 @@ export const documentSupportedProviders = new Set<string>([
   Providers.VERTEXAI,
   Providers.MISTRALAI,
   Providers.MISTRAL,
+  Providers.OLLAMA,
   Providers.DEEPSEEK,
   Providers.OPENROUTER,
   Providers.XAI,
@@ -69,6 +71,7 @@ const openAILikeProviders = new Set<string>([
   EModelEndpoint.custom,
   Providers.MISTRALAI,
   Providers.MISTRAL,
+  Providers.OLLAMA,
   Providers.DEEPSEEK,
   Providers.OPENROUTER,
   Providers.XAI,

packages/data-schemas/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@librechat/data-schemas",
-  "version": "0.0.30",
+  "version": "0.0.23",
   "description": "Mongoose schemas and models for LibreChat",
   "type": "module",
   "main": "dist/index.cjs",

packages/data-schemas/src/methods/share.ts

@@ -4,7 +4,6 @@ import type { FilterQuery, Model } from 'mongoose';
 import type { SchemaWithMeiliMethods } from '~/models/plugins/mongoMeili';
 import type * as t from '~/types';
 import logger from '~/config/winston';
-import { createShareExpirationDate, isShareExpired } from '~/utils/shareExpiration';
 
 class ShareServiceError extends Error {
   code: string;
@@ -174,15 +173,6 @@ export function createShareMethods(mongoose: typeof import('mongoose')) {
         return null;
       }
 
-      // Check if share has expired
-      if (isShareExpired(share.expiresAt)) {
-        logger.warn('[getSharedMessages] Share has expired', {
-          shareId,
-          expiresAt: share.expiresAt,
-        });
-        throw new ShareServiceError('This shared link has expired', 'SHARE_EXPIRED');
-      }
-
       /** Filtered messages based on targetMessageId if present (branch-specific sharing) */
       let messagesToShare: t.IMessage[] = share.messages;
       if (share.targetMessageId) {
@@ -194,7 +184,6 @@ export function createShareMethods(mongoose: typeof import('mongoose')) {
         shareId: share.shareId || shareId,
         title: share.title,
         isPublic: share.isPublic,
-        expiresAt: share.expiresAt,
         createdAt: share.createdAt,
         updatedAt: share.updatedAt,
         conversationId: newConvoId,
@@ -286,7 +275,6 @@ export function createShareMethods(mongoose: typeof import('mongoose')) {
           shareId: link.shareId || '',
           title: link?.title || 'Untitled',
           isPublic: link.isPublic,
-          expiresAt: link.expiresAt,
           createdAt: link.createdAt || new Date(),
           conversationId: link.conversationId,
         })),
@@ -357,7 +345,6 @@ export function createShareMethods(mongoose: typeof import('mongoose')) {
     user: string,
     conversationId: string,
     targetMessageId?: string,
-    expirationHours?: number,
   ): Promise<t.CreateShareResult> {
     if (!user || !conversationId) {
       throw new ShareServiceError('Missing required parameters', 'INVALID_PARAMS');
@@ -414,7 +401,6 @@ export function createShareMethods(mongoose: typeof import('mongoose')) {
       const title = conversation.title || 'Untitled';
 
       const shareId = nanoid();
-      const expiresAt = createShareExpirationDate(expirationHours);
       await SharedLink.create({
         shareId,
         conversationId,
@@ -422,7 +408,6 @@ export function createShareMethods(mongoose: typeof import('mongoose')) {
         title,
         user,
         ...(targetMessageId && { targetMessageId }),
-        ...(expiresAt && { expiresAt }),
       });
 
       return { shareId, conversationId };

packages/data-schemas/src/methods/token.spec.ts

@@ -418,41 +418,6 @@ describe('Token Methods - Detailed Tests', () => {
 
       expect(updated).toBeNull();
     });
-
-    test('should update expiresAt when expiresIn is provided', async () => {
-      const beforeUpdate = Date.now();
-      const newExpiresIn = 7200;
-
-      const updated = await methods.updateToken(
-        { token: 'update-token' },
-        { expiresIn: newExpiresIn },
-      );
-
-      const afterUpdate = Date.now();
-
-      expect(updated).toBeDefined();
-      expect(updated?.expiresAt).toBeDefined();
-
-      const expectedMinExpiry = beforeUpdate + newExpiresIn * 1000;
-      const expectedMaxExpiry = afterUpdate + newExpiresIn * 1000;
-
-      expect(updated!.expiresAt.getTime()).toBeGreaterThanOrEqual(expectedMinExpiry);
-      expect(updated!.expiresAt.getTime()).toBeLessThanOrEqual(expectedMaxExpiry);
-    });
-
-    test('should not modify expiresAt when expiresIn is not provided', async () => {
-      const original = await Token.findOne({ token: 'update-token' });
-      const originalExpiresAt = original!.expiresAt.getTime();
-
-      const updated = await methods.updateToken(
-        { token: 'update-token' },
-        { email: 'changed@example.com' },
-      );
-
-      expect(updated).toBeDefined();
-      expect(updated?.email).toBe('changed@example.com');
-      expect(updated!.expiresAt.getTime()).toBe(originalExpiresAt);
-    });
   });
 
   describe('deleteTokens', () => {
@@ -652,171 +617,4 @@ describe('Token Methods - Detailed Tests', () => {
       expect(remainingTokens.find((t) => t.token === 'email-verify-token-2')).toBeUndefined();
     });
   });
-
-  describe('Email Normalization', () => {
-    let normUserId: mongoose.Types.ObjectId;
-
-    beforeEach(async () => {
-      normUserId = new mongoose.Types.ObjectId();
-
-      // Create token with lowercase email (as stored in DB)
-      await Token.create({
-        token: 'norm-token-1',
-        userId: normUserId,
-        email: 'john.doe@example.com',
-        createdAt: new Date(),
-        expiresAt: new Date(Date.now() + 3600000),
-      });
-    });
-
-    describe('findToken email normalization', () => {
-      test('should find token by email with different case (case-insensitive)', async () => {
-        const foundUpper = await methods.findToken({ email: 'JOHN.DOE@EXAMPLE.COM' });
-        const foundMixed = await methods.findToken({ email: 'John.Doe@Example.COM' });
-        const foundLower = await methods.findToken({ email: 'john.doe@example.com' });
-
-        expect(foundUpper).toBeDefined();
-        expect(foundUpper?.token).toBe('norm-token-1');
-
-        expect(foundMixed).toBeDefined();
-        expect(foundMixed?.token).toBe('norm-token-1');
-
-        expect(foundLower).toBeDefined();
-        expect(foundLower?.token).toBe('norm-token-1');
-      });
-
-      test('should find token by email with leading/trailing whitespace', async () => {
-        const foundWithSpaces = await methods.findToken({ email: '  john.doe@example.com  ' });
-        const foundWithTabs = await methods.findToken({ email: '\tjohn.doe@example.com\t' });
-
-        expect(foundWithSpaces).toBeDefined();
-        expect(foundWithSpaces?.token).toBe('norm-token-1');
-
-        expect(foundWithTabs).toBeDefined();
-        expect(foundWithTabs?.token).toBe('norm-token-1');
-      });
-
-      test('should find token by email with both case difference and whitespace', async () => {
-        const found = await methods.findToken({ email: '  JOHN.DOE@EXAMPLE.COM  ' });
-
-        expect(found).toBeDefined();
-        expect(found?.token).toBe('norm-token-1');
-      });
-
-      test('should find token with combined email and other criteria', async () => {
-        const found = await methods.findToken({
-          userId: normUserId.toString(),
-          email: 'John.Doe@Example.COM',
-        });
-
-        expect(found).toBeDefined();
-        expect(found?.token).toBe('norm-token-1');
-      });
-    });
-
-    describe('deleteTokens email normalization', () => {
-      test('should delete token by email with different case', async () => {
-        const result = await methods.deleteTokens({ email: 'JOHN.DOE@EXAMPLE.COM' });
-
-        expect(result.deletedCount).toBe(1);
-
-        const remaining = await Token.find({});
-        expect(remaining).toHaveLength(0);
-      });
-
-      test('should delete token by email with whitespace', async () => {
-        const result = await methods.deleteTokens({ email: '  john.doe@example.com  ' });
-
-        expect(result.deletedCount).toBe(1);
-
-        const remaining = await Token.find({});
-        expect(remaining).toHaveLength(0);
-      });
-
-      test('should delete token by email with case and whitespace combined', async () => {
-        const result = await methods.deleteTokens({ email: '  John.Doe@EXAMPLE.COM  ' });
-
-        expect(result.deletedCount).toBe(1);
-
-        const remaining = await Token.find({});
-        expect(remaining).toHaveLength(0);
-      });
-
-      test('should only delete matching token when using normalized email', async () => {
-        // Create additional token with different email
-        await Token.create({
-          token: 'norm-token-2',
-          userId: new mongoose.Types.ObjectId(),
-          email: 'jane.doe@example.com',
-          createdAt: new Date(),
-          expiresAt: new Date(Date.now() + 3600000),
-        });
-
-        const result = await methods.deleteTokens({ email: 'JOHN.DOE@EXAMPLE.COM' });
-
-        expect(result.deletedCount).toBe(1);
-
-        const remaining = await Token.find({});
-        expect(remaining).toHaveLength(1);
-        expect(remaining[0].email).toBe('jane.doe@example.com');
-      });
-    });
-
-    describe('Email verification flow with normalization', () => {
-      test('should handle OpenID provider email case mismatch scenario', async () => {
-        /**
-         * Simulate the exact bug scenario:
-         * 1. User registers with email stored as lowercase
-         * 2. OpenID provider returns email with different casing
-         * 3. System should still find and delete the correct token
-         */
-        const userId = new mongoose.Types.ObjectId();
-
-        // Token created during registration (email stored lowercase)
-        await Token.create({
-          token: 'verification-token',
-          userId: userId,
-          email: 'user@company.com',
-          createdAt: new Date(),
-          expiresAt: new Date(Date.now() + 86400000),
-        });
-
-        // OpenID provider returns email with different case
-        const emailFromProvider = 'User@Company.COM';
-
-        // Should find the token despite case mismatch
-        const found = await methods.findToken({ email: emailFromProvider });
-        expect(found).toBeDefined();
-        expect(found?.token).toBe('verification-token');
-
-        // Should delete the token despite case mismatch
-        const deleted = await methods.deleteTokens({ email: emailFromProvider });
-        expect(deleted.deletedCount).toBe(1);
-      });
-
-      test('should handle resend verification email with case mismatch', async () => {
-        const userId = new mongoose.Types.ObjectId();
-
-        // Old verification token
-        await Token.create({
-          token: 'old-verification',
-          userId: userId,
-          email: 'john.smith@enterprise.com',
-          createdAt: new Date(Date.now() - 3600000),
-          expiresAt: new Date(Date.now() + 82800000),
-        });
-
-        // User requests resend with different email casing
-        const userInputEmail = '  John.Smith@ENTERPRISE.COM  ';
-
-        // Delete old tokens for this email
-        const deleted = await methods.deleteTokens({ email: userInputEmail });
-        expect(deleted.deletedCount).toBe(1);
-
-        // Verify token was actually deleted
-        const remaining = await Token.find({ userId });
-        expect(remaining).toHaveLength(0);
-      });
-    });
-  });
 });

packages/data-schemas/src/methods/token.ts

@@ -35,13 +35,7 @@ export function createTokenMethods(mongoose: typeof import('mongoose')) {
   ): Promise<IToken | null> {
     try {
       const Token = mongoose.models.Token;
-
-      const dataToUpdate = { ...updateData };
-      if (updateData?.expiresIn !== undefined) {
-        dataToUpdate.expiresAt = new Date(Date.now() + updateData.expiresIn * 1000);
-      }
-
-      return await Token.findOneAndUpdate(query, dataToUpdate, { new: true });
+      return await Token.findOneAndUpdate(query, updateData, { new: true });
     } catch (error) {
       logger.debug('An error occurred while updating token:', error);
       throw error;
@@ -50,7 +44,6 @@ export function createTokenMethods(mongoose: typeof import('mongoose')) {
 
   /**
    * Deletes all Token documents that match the provided token, user ID, or email.
-   * Email is automatically normalized to lowercase for case-insensitive matching.
    */
   async function deleteTokens(query: TokenQuery): Promise<TokenDeleteResult> {
     try {
@@ -64,7 +57,7 @@ export function createTokenMethods(mongoose: typeof import('mongoose')) {
         conditions.push({ token: query.token });
       }
       if (query.email !== undefined) {
-        conditions.push({ email: query.email.trim().toLowerCase() });
+        conditions.push({ email: query.email });
       }
       if (query.identifier !== undefined) {
         conditions.push({ identifier: query.identifier });
@@ -88,7 +81,6 @@ export function createTokenMethods(mongoose: typeof import('mongoose')) {
 
   /**
    * Finds a Token document that matches the provided query.
-   * Email is automatically normalized to lowercase for case-insensitive matching.
    */
   async function findToken(query: TokenQuery, options?: QueryOptions): Promise<IToken | null> {
     try {
@@ -102,7 +94,7 @@ export function createTokenMethods(mongoose: typeof import('mongoose')) {
         conditions.push({ token: query.token });
       }
       if (query.email) {
-        conditions.push({ email: query.email.trim().toLowerCase() });
+        conditions.push({ email: query.email });
       }
       if (query.identifier) {
         conditions.push({ identifier: query.identifier });

packages/data-schemas/src/methods/user.methods.spec.ts

@@ -1,623 +0,0 @@
-import mongoose from 'mongoose';
-import { MongoMemoryServer } from 'mongodb-memory-server';
-import type * as t from '~/types';
-import { createUserMethods } from './user';
-import userSchema from '~/schema/user';
-import balanceSchema from '~/schema/balance';
-
-/** Mocking crypto for generateToken */
-jest.mock('~/crypto', () => ({
-  signPayload: jest.fn().mockResolvedValue('mocked-token'),
-}));
-
-let mongoServer: MongoMemoryServer;
-let User: mongoose.Model<t.IUser>;
-let Balance: mongoose.Model<t.IBalance>;
-let methods: ReturnType<typeof createUserMethods>;
-
-beforeAll(async () => {
-  mongoServer = await MongoMemoryServer.create();
-  const mongoUri = mongoServer.getUri();
-  await mongoose.connect(mongoUri);
-
-  /** Register models */
-  User = mongoose.models.User || mongoose.model<t.IUser>('User', userSchema);
-  Balance = mongoose.models.Balance || mongoose.model<t.IBalance>('Balance', balanceSchema);
-
-  /** Initialize methods */
-  methods = createUserMethods(mongoose);
-});
-
-afterAll(async () => {
-  await mongoose.disconnect();
-  await mongoServer.stop();
-});
-
-beforeEach(async () => {
-  await mongoose.connection.dropDatabase();
-});
-
-describe('User Methods - Database Tests', () => {
-  describe('findUser', () => {
-    test('should find user by exact email', async () => {
-      await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'local',
-      });
-
-      const found = await methods.findUser({ email: 'test@example.com' });
-
-      expect(found).toBeDefined();
-      expect(found?.email).toBe('test@example.com');
-    });
-
-    test('should find user by email with different case (case-insensitive)', async () => {
-      await User.create({
-        name: 'Test User',
-        email: 'test@example.com', // stored lowercase by schema
-        provider: 'local',
-      });
-
-      /** Test various case combinations - all should find the same user */
-      const foundUpper = await methods.findUser({ email: 'TEST@EXAMPLE.COM' });
-      const foundMixed = await methods.findUser({ email: 'Test@Example.COM' });
-      const foundLower = await methods.findUser({ email: 'test@example.com' });
-
-      expect(foundUpper).toBeDefined();
-      expect(foundUpper?.email).toBe('test@example.com');
-
-      expect(foundMixed).toBeDefined();
-      expect(foundMixed?.email).toBe('test@example.com');
-
-      expect(foundLower).toBeDefined();
-      expect(foundLower?.email).toBe('test@example.com');
-    });
-
-    test('should find user by email with leading/trailing whitespace (trimmed)', async () => {
-      await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'local',
-      });
-
-      const foundWithSpaces = await methods.findUser({ email: '  test@example.com  ' });
-      const foundWithTabs = await methods.findUser({ email: '\ttest@example.com\t' });
-
-      expect(foundWithSpaces).toBeDefined();
-      expect(foundWithSpaces?.email).toBe('test@example.com');
-
-      expect(foundWithTabs).toBeDefined();
-      expect(foundWithTabs?.email).toBe('test@example.com');
-    });
-
-    test('should find user by email with both case difference and whitespace', async () => {
-      await User.create({
-        name: 'Test User',
-        email: 'john.doe@example.com',
-        provider: 'local',
-      });
-
-      const found = await methods.findUser({ email: '  John.Doe@EXAMPLE.COM  ' });
-
-      expect(found).toBeDefined();
-      expect(found?.email).toBe('john.doe@example.com');
-    });
-
-    test('should normalize email in $or conditions', async () => {
-      await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'openid',
-        openidId: 'openid-123',
-      });
-
-      const found = await methods.findUser({
-        $or: [{ openidId: 'different-id' }, { email: 'TEST@EXAMPLE.COM' }],
-      });
-
-      expect(found).toBeDefined();
-      expect(found?.email).toBe('test@example.com');
-    });
-
-    test('should find user by non-email criteria without affecting them', async () => {
-      await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'openid',
-        openidId: 'openid-123',
-      });
-
-      const found = await methods.findUser({ openidId: 'openid-123' });
-
-      expect(found).toBeDefined();
-      expect(found?.openidId).toBe('openid-123');
-    });
-
-    test('should apply field selection correctly', async () => {
-      await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'local',
-        username: 'testuser',
-      });
-
-      const found = await methods.findUser({ email: 'test@example.com' }, 'email name');
-
-      expect(found).toBeDefined();
-      expect(found?.email).toBe('test@example.com');
-      expect(found?.name).toBe('Test User');
-      expect(found?.username).toBeUndefined();
-      expect(found?.provider).toBeUndefined();
-    });
-
-    test('should return null for non-existent user', async () => {
-      const found = await methods.findUser({ email: 'nonexistent@example.com' });
-
-      expect(found).toBeNull();
-    });
-  });
-
-  describe('createUser', () => {
-    test('should create a user and return ObjectId by default', async () => {
-      const result = await methods.createUser({
-        name: 'New User',
-        email: 'new@example.com',
-        provider: 'local',
-      });
-
-      expect(result).toBeInstanceOf(mongoose.Types.ObjectId);
-
-      const user = await User.findById(result);
-      expect(user).toBeDefined();
-      expect(user?.name).toBe('New User');
-      expect(user?.email).toBe('new@example.com');
-    });
-
-    test('should create a user and return user object when returnUser is true', async () => {
-      const result = await methods.createUser(
-        {
-          name: 'New User',
-          email: 'new@example.com',
-          provider: 'local',
-        },
-        undefined,
-        true,
-        true,
-      );
-
-      expect(result).toHaveProperty('_id');
-      expect(result).toHaveProperty('name', 'New User');
-      expect(result).toHaveProperty('email', 'new@example.com');
-    });
-
-    test('should store email as lowercase regardless of input case', async () => {
-      await methods.createUser({
-        name: 'New User',
-        email: 'NEW@EXAMPLE.COM',
-        provider: 'local',
-      });
-
-      const user = await User.findOne({ email: 'new@example.com' });
-      expect(user).toBeDefined();
-      expect(user?.email).toBe('new@example.com');
-    });
-
-    test('should create user with TTL when disableTTL is false', async () => {
-      const result = await methods.createUser(
-        {
-          name: 'TTL User',
-          email: 'ttl@example.com',
-          provider: 'local',
-        },
-        undefined,
-        false,
-        true,
-      );
-
-      expect(result).toHaveProperty('expiresAt');
-      const expiresAt = (result as t.IUser).expiresAt;
-      expect(expiresAt).toBeInstanceOf(Date);
-
-      /** Should expire in approximately 1 week */
-      const oneWeekMs = 604800 * 1000;
-      const expectedExpiry = Date.now() + oneWeekMs;
-      expect(expiresAt!.getTime()).toBeGreaterThan(expectedExpiry - 10000);
-      expect(expiresAt!.getTime()).toBeLessThan(expectedExpiry + 10000);
-    });
-
-    test('should create balance record when balanceConfig is provided', async () => {
-      const userId = await methods.createUser(
-        {
-          name: 'Balance User',
-          email: 'balance@example.com',
-          provider: 'local',
-        },
-        {
-          enabled: true,
-          startBalance: 1000,
-        },
-      );
-
-      const balance = await Balance.findOne({ user: userId });
-      expect(balance).toBeDefined();
-      expect(balance?.tokenCredits).toBe(1000);
-    });
-  });
-
-  describe('updateUser', () => {
-    test('should update user fields', async () => {
-      const user = await User.create({
-        name: 'Original Name',
-        email: 'test@example.com',
-        provider: 'local',
-      });
-
-      const updated = await methods.updateUser(user._id?.toString() ?? '', {
-        name: 'Updated Name',
-      });
-
-      expect(updated).toBeDefined();
-      expect(updated?.name).toBe('Updated Name');
-      expect(updated?.email).toBe('test@example.com');
-    });
-
-    test('should remove expiresAt field on update', async () => {
-      const user = await User.create({
-        name: 'TTL User',
-        email: 'ttl@example.com',
-        provider: 'local',
-        expiresAt: new Date(Date.now() + 604800 * 1000),
-      });
-
-      const updated = await methods.updateUser(user._id?.toString() || '', {
-        name: 'No longer TTL',
-      });
-
-      expect(updated).toBeDefined();
-      expect(updated?.expiresAt).toBeUndefined();
-    });
-
-    test('should return null for non-existent user', async () => {
-      const fakeId = new mongoose.Types.ObjectId();
-      const result = await methods.updateUser(fakeId.toString(), { name: 'Test' });
-
-      expect(result).toBeNull();
-    });
-  });
-
-  describe('getUserById', () => {
-    test('should get user by ID', async () => {
-      const user = await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'local',
-      });
-
-      const found = await methods.getUserById(user._id?.toString() || '');
-
-      expect(found).toBeDefined();
-      expect(found?.name).toBe('Test User');
-    });
-
-    test('should apply field selection', async () => {
-      const user = await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'local',
-        username: 'testuser',
-      });
-
-      const found = await methods.getUserById(user._id?.toString() || '', 'name email');
-
-      expect(found).toBeDefined();
-      expect(found?.name).toBe('Test User');
-      expect(found?.email).toBe('test@example.com');
-      expect(found?.username).toBeUndefined();
-    });
-
-    test('should return null for non-existent ID', async () => {
-      const fakeId = new mongoose.Types.ObjectId();
-      const found = await methods.getUserById(fakeId.toString());
-
-      expect(found).toBeNull();
-    });
-  });
-
-  describe('deleteUserById', () => {
-    test('should delete user by ID', async () => {
-      const user = await User.create({
-        name: 'To Delete',
-        email: 'delete@example.com',
-        provider: 'local',
-      });
-
-      const result = await methods.deleteUserById(user._id?.toString() || '');
-
-      expect(result.deletedCount).toBe(1);
-      expect(result.message).toBe('User was deleted successfully.');
-
-      const found = await User.findById(user._id);
-      expect(found).toBeNull();
-    });
-
-    test('should return zero count for non-existent user', async () => {
-      const fakeId = new mongoose.Types.ObjectId();
-      const result = await methods.deleteUserById(fakeId.toString());
-
-      expect(result.deletedCount).toBe(0);
-      expect(result.message).toBe('No user found with that ID.');
-    });
-  });
-
-  describe('countUsers', () => {
-    test('should count all users', async () => {
-      await User.create([
-        { name: 'User 1', email: 'user1@example.com', provider: 'local' },
-        { name: 'User 2', email: 'user2@example.com', provider: 'local' },
-        { name: 'User 3', email: 'user3@example.com', provider: 'openid' },
-      ]);
-
-      const count = await methods.countUsers();
-
-      expect(count).toBe(3);
-    });
-
-    test('should count users with filter', async () => {
-      await User.create([
-        { name: 'User 1', email: 'user1@example.com', provider: 'local' },
-        { name: 'User 2', email: 'user2@example.com', provider: 'local' },
-        { name: 'User 3', email: 'user3@example.com', provider: 'openid' },
-      ]);
-
-      const count = await methods.countUsers({ provider: 'local' });
-
-      expect(count).toBe(2);
-    });
-
-    test('should return zero for empty collection', async () => {
-      const count = await methods.countUsers();
-
-      expect(count).toBe(0);
-    });
-  });
-
-  describe('searchUsers', () => {
-    beforeEach(async () => {
-      await User.create([
-        { name: 'John Doe', email: 'john@example.com', username: 'johnd', provider: 'local' },
-        { name: 'Jane Smith', email: 'jane@example.com', username: 'janes', provider: 'local' },
-        {
-          name: 'Bob Johnson',
-          email: 'bob@example.com',
-          username: 'bobbyj',
-          provider: 'local',
-        },
-        {
-          name: 'Alice Wonder',
-          email: 'alice@test.com',
-          username: 'alice',
-          provider: 'openid',
-        },
-      ]);
-    });
-
-    test('should search by name', async () => {
-      const results = await methods.searchUsers({ searchPattern: 'John' });
-
-      expect(results).toHaveLength(2); // John Doe and Bob Johnson
-    });
-
-    test('should search by email', async () => {
-      const results = await methods.searchUsers({ searchPattern: 'example.com' });
-
-      expect(results).toHaveLength(3);
-    });
-
-    test('should search by username', async () => {
-      const results = await methods.searchUsers({ searchPattern: 'alice' });
-
-      expect(results).toHaveLength(1);
-      expect((results[0] as unknown as t.IUser)?.username).toBe('alice');
-    });
-
-    test('should be case-insensitive', async () => {
-      const results = await methods.searchUsers({ searchPattern: 'JOHN' });
-
-      expect(results.length).toBeGreaterThan(0);
-    });
-
-    test('should respect limit', async () => {
-      const results = await methods.searchUsers({ searchPattern: 'example', limit: 2 });
-
-      expect(results).toHaveLength(2);
-    });
-
-    test('should return empty array for empty search pattern', async () => {
-      const results = await methods.searchUsers({ searchPattern: '' });
-
-      expect(results).toEqual([]);
-    });
-
-    test('should return empty array for whitespace-only pattern', async () => {
-      const results = await methods.searchUsers({ searchPattern: '   ' });
-
-      expect(results).toEqual([]);
-    });
-
-    test('should apply field selection', async () => {
-      const results = await methods.searchUsers({
-        searchPattern: 'john',
-        fieldsToSelect: 'name email',
-      });
-
-      expect(results.length).toBeGreaterThan(0);
-      expect(results[0]).toHaveProperty('name');
-      expect(results[0]).toHaveProperty('email');
-      expect(results[0]).not.toHaveProperty('username');
-    });
-
-    test('should sort by relevance (exact match first)', async () => {
-      const results = await methods.searchUsers({ searchPattern: 'alice' });
-
-      /** 'alice' username should score highest due to exact match */
-      expect((results[0] as unknown as t.IUser).username).toBe('alice');
-    });
-  });
-
-  describe('toggleUserMemories', () => {
-    test('should enable memories for user', async () => {
-      const user = await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'local',
-      });
-
-      const updated = await methods.toggleUserMemories(user._id?.toString() || '', true);
-
-      expect(updated).toBeDefined();
-      expect(updated?.personalization?.memories).toBe(true);
-    });
-
-    test('should disable memories for user', async () => {
-      const user = await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'local',
-        personalization: { memories: true },
-      });
-
-      const updated = await methods.toggleUserMemories(user._id?.toString() || '', false);
-
-      expect(updated).toBeDefined();
-      expect(updated?.personalization?.memories).toBe(false);
-    });
-
-    test('should update personalization.memories field', async () => {
-      const user = await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'local',
-      });
-
-      /** Toggle memories to true */
-      const updated = await methods.toggleUserMemories(user._id?.toString() || '', true);
-
-      expect(updated?.personalization).toBeDefined();
-      expect(updated?.personalization?.memories).toBe(true);
-
-      /** Toggle back to false */
-      const updatedAgain = await methods.toggleUserMemories(user._id?.toString() || '', false);
-      expect(updatedAgain?.personalization?.memories).toBe(false);
-    });
-
-    test('should return null for non-existent user', async () => {
-      const fakeId = new mongoose.Types.ObjectId();
-      const result = await methods.toggleUserMemories(fakeId.toString(), true);
-
-      expect(result).toBeNull();
-    });
-  });
-
-  describe('Email Normalization Edge Cases', () => {
-    test('should handle email with multiple spaces', async () => {
-      await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'local',
-      });
-
-      const found = await methods.findUser({ email: '    test@example.com    ' });
-
-      expect(found).toBeDefined();
-      expect(found?.email).toBe('test@example.com');
-    });
-
-    test('should handle mixed case with international characters', async () => {
-      await User.create({
-        name: 'Test User',
-        email: 'user@example.com',
-        provider: 'local',
-      });
-
-      const found = await methods.findUser({ email: 'USER@EXAMPLE.COM' });
-
-      expect(found).toBeDefined();
-    });
-
-    test('should handle email normalization in complex $or queries', async () => {
-      const user1 = await User.create({
-        name: 'User One',
-        email: 'user1@example.com',
-        provider: 'openid',
-        openidId: 'openid-1',
-      });
-
-      await User.create({
-        name: 'User Two',
-        email: 'user2@example.com',
-        provider: 'openid',
-        openidId: 'openid-2',
-      });
-
-      /** Search with mixed case email in $or */
-      const found = await methods.findUser({
-        $or: [{ openidId: 'nonexistent' }, { email: 'USER1@EXAMPLE.COM' }],
-      });
-
-      expect(found).toBeDefined();
-      expect(found?._id?.toString()).toBe(user1._id?.toString());
-    });
-
-    test('should not normalize non-string email values', async () => {
-      await User.create({
-        name: 'Test User',
-        email: 'test@example.com',
-        provider: 'local',
-      });
-
-      /** Using regex for email (should not be normalized) */
-      const found = await methods.findUser({ email: /test@example\.com/i });
-
-      expect(found).toBeDefined();
-      expect(found?.email).toBe('test@example.com');
-    });
-
-    test('should handle OpenID provider migration scenario', async () => {
-      /** Simulate user stored with lowercase email */
-      await User.create({
-        name: 'John Doe',
-        email: 'john.doe@company.com',
-        provider: 'openid',
-        openidId: 'old-provider-id',
-      });
-
-      /**
-       * New OpenID provider returns email with different casing
-       * This simulates the exact bug reported in the GitHub issue
-       */
-      const emailFromNewProvider = 'John.Doe@Company.COM';
-
-      const found = await methods.findUser({ email: emailFromNewProvider });
-
-      expect(found).toBeDefined();
-      expect(found?.email).toBe('john.doe@company.com');
-      expect(found?.name).toBe('John Doe');
-    });
-
-    test('should handle SAML provider email normalization', async () => {
-      await User.create({
-        name: 'SAML User',
-        email: 'saml.user@enterprise.com',
-        provider: 'saml',
-        samlId: 'saml-123',
-      });
-
-      /** SAML providers sometimes return emails in different formats */
-      const found = await methods.findUser({ email: '  SAML.USER@ENTERPRISE.COM  ' });
-
-      expect(found).toBeDefined();
-      expect(found?.provider).toBe('saml');
-    });
-  });
-});

packages/data-schemas/src/methods/user.ts

@@ -4,37 +4,15 @@ import { signPayload } from '~/crypto';
 
 /** Factory function that takes mongoose instance and returns the methods */
 export function createUserMethods(mongoose: typeof import('mongoose')) {
-  /**
-   * Normalizes email fields in search criteria to lowercase and trimmed.
-   * Handles both direct email fields and $or arrays containing email conditions.
-   */
-  function normalizeEmailInCriteria<T extends FilterQuery<IUser>>(criteria: T): T {
-    const normalized = { ...criteria };
-    if (typeof normalized.email === 'string') {
-      normalized.email = normalized.email.trim().toLowerCase();
-    }
-    if (Array.isArray(normalized.$or)) {
-      normalized.$or = normalized.$or.map((condition) => {
-        if (typeof condition.email === 'string') {
-          return { ...condition, email: condition.email.trim().toLowerCase() };
-        }
-        return condition;
-      });
-    }
-    return normalized;
-  }
-
   /**
    * Search for a single user based on partial data and return matching user document as plain object.
-   * Email fields in searchCriteria are automatically normalized to lowercase for case-insensitive matching.
    */
   async function findUser(
     searchCriteria: FilterQuery<IUser>,
     fieldsToSelect?: string | string[] | null,
   ): Promise<IUser | null> {
     const User = mongoose.models.User;
-    const normalizedCriteria = normalizeEmailInCriteria(searchCriteria);
-    const query = User.findOne(normalizedCriteria);
+    const query = User.findOne(searchCriteria);
     if (fieldsToSelect) {
       query.select(fieldsToSelect);
     }

packages/data-schemas/src/schema/share.ts

@@ -8,7 +8,6 @@ export interface ISharedLink extends Document {
   shareId?: string;
   targetMessageId?: string;
   isPublic: boolean;
-  expiresAt?: Date;
   createdAt?: Date;
   updatedAt?: Date;
 }
@@ -41,15 +40,10 @@ const shareSchema: Schema<ISharedLink> = new Schema(
       type: Boolean,
       default: true,
     },
-    expiresAt: {
-      type: Date,
-      required: false,
-    },
   },
   { timestamps: true },
 );
 
 shareSchema.index({ conversationId: 1, user: 1, targetMessageId: 1 });
-shareSchema.index({ expiresAt: 1 }, { expireAfterSeconds: 0 });
 
 export default shareSchema;