refactor: share string normalization helper

packages/api/src/langfuse/config.ts

@@ -1,7 +1,8 @@
 import type { AppConfig } from '@librechat/data-schemas';
 import type { RunConfig } from '@librechat/agents';
 import { resolveLangfuseTenantDestination } from './tenantDestinations';
-import { isTrueEnv, normalizeString } from './utils';
+import { normalizeString } from '~/utils/text';
+import { isTrueEnv } from './utils';
 
 type LangfuseRunConfig = NonNullable<RunConfig['langfuse']>;
 type LangfuseRunConfigWithTraceAttributes = LangfuseRunConfig & {

packages/api/src/langfuse/destinations.ts

@@ -1,6 +1,7 @@
 import type { AppConfig } from '@librechat/data-schemas';
-import { isFalseEnv, isTrueEnv, normalizeString, toBasicAuthorization } from './utils';
 import { resolveLangfuseTenantDestination } from './tenantDestinations';
+import { isFalseEnv, isTrueEnv, toBasicAuthorization } from './utils';
+import { normalizeString } from '~/utils/text';
 
 const DEFAULT_BASE_URL = 'https://cloud.langfuse.com';
 

packages/api/src/langfuse/tenantDestinations.ts

@@ -1,4 +1,4 @@
-import { normalizeString } from './utils';
+import { normalizeString } from '~/utils/text';
 
 const DEFAULT_TENANT_DESTINATIONS: Array<[string, string]> = [
   ['eu', 'https://cloud.langfuse.com'],

packages/api/src/langfuse/utils.ts

@@ -1,7 +1,3 @@
-export function normalizeString(value: unknown): string | undefined {
-  return typeof value === 'string' && value.trim() !== '' ? value.trim() : undefined;
-}
-
 export function isFalseEnv(value?: string): boolean {
   return value != null && ['0', 'false', 'no', 'off'].includes(value.trim().toLowerCase());
 }

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

@@ -1,4 +1,4 @@
-import { processTextWithTokenLimit, TokenCountFn } from './text';
+import { normalizeString, processTextWithTokenLimit, TokenCountFn } from './text';
 import Tokenizer, { countTokens } from './tokenizer';
 
 jest.mock('@librechat/data-schemas', () => ({
@@ -9,6 +9,15 @@ jest.mock('@librechat/data-schemas', () => ({
   },
 }));
 
+describe('normalizeString', () => {
+  it('trims non-empty strings and treats blank or non-string values as undefined', () => {
+    expect(normalizeString('  value  ')).toBe('value');
+    expect(normalizeString('   ')).toBeUndefined();
+    expect(normalizeString(null)).toBeUndefined();
+    expect(normalizeString(123)).toBeUndefined();
+  });
+});
+
 /**
  * OLD IMPLEMENTATION (Binary Search) - kept for comparison testing
  * This is the original algorithm that caused CPU spikes

packages/api/src/utils/text.ts

@@ -3,6 +3,10 @@ import { logger } from '@librechat/data-schemas';
 /** Token count function that can be sync or async */
 export type TokenCountFn = (text: string) => number | Promise<number>;
 
+export function normalizeString(value: unknown): string | undefined {
+  return typeof value === 'string' && value.trim() !== '' ? value.trim() : undefined;
+}
+
 /**
  * Safety buffer multiplier applied to character position estimates during truncation.
  *