te.js 2.1.5 → 2.1.6

package/utils/errors-llm-config.js

@@ -7,6 +7,8 @@
 import { env } from 'tej-env';
 
 const MESSAGE_TYPES = /** @type {const} */ (['endUser', 'developer']);
+const LLM_MODES = /** @type {const} */ (['sync', 'async']);
+const LLM_CHANNELS = /** @type {const} */ (['console', 'log', 'both']);
 
 /**
  * Normalize messageType to 'endUser' | 'developer'.
@@ -14,14 +16,56 @@ const MESSAGE_TYPES = /** @type {const} */ (['endUser', 'developer']);
  * @returns {'endUser'|'developer'}
  */
 function normalizeMessageType(v) {
-  const s = String(v ?? '').trim().toLowerCase();
+  const s = String(v ?? '')
+    .trim()
+    .toLowerCase();
   if (s === 'developer' || s === 'dev') return 'developer';
   return 'endUser'; // endUser, end_user, default
 }
 
+/**
+ * Normalize mode to 'sync' | 'async'.
+ * @param {string} v
+ * @returns {'sync'|'async'}
+ */
+function normalizeMode(v) {
+  const s = String(v ?? '')
+    .trim()
+    .toLowerCase();
+  if (s === 'async') return 'async';
+  return 'sync';
+}
+
+/**
+ * Normalize channel to 'console' | 'log' | 'both'.
+ * @param {string} v
+ * @returns {'console'|'log'|'both'}
+ */
+function normalizeChannel(v) {
+  const s = String(v ?? '')
+    .trim()
+    .toLowerCase();
+  if (s === 'log') return 'log';
+  if (s === 'both') return 'both';
+  return 'console';
+}
+
 /**
  * Resolve errors.llm config from env (feature-specific then LLM_ fallback).
- * @returns {{ enabled: boolean, baseURL: string, apiKey: string, model: string, messageType: 'endUser'|'developer' }}
+ * @returns {{
+ *   enabled: boolean,
+ *   baseURL: string,
+ *   apiKey: string,
+ *   model: string,
+ *   messageType: 'endUser'|'developer',
+ *   mode: 'sync'|'async',
+ *   timeout: number,
+ *   channel: 'console'|'log'|'both',
+ *   logFile: string,
+ *   rateLimit: number,
+ *   cache: boolean,
+ *   cacheTTL: number,
+ * }}
  */
 export function getErrorsLlmConfig() {
   const enabledRaw = env('ERRORS_LLM_ENABLED') ?? '';
@@ -45,11 +89,50 @@ export function getErrorsLlmConfig() {
     env('LLM_APIKEY') ??
     '';
 
-  const model =
-    env('ERRORS_LLM_MODEL') ?? env('LLM_MODEL') ?? '';
+  const model = env('ERRORS_LLM_MODEL') ?? env('LLM_MODEL') ?? '';
 
   const messageTypeRaw =
-    env('ERRORS_LLM_MESSAGE_TYPE') ?? env('ERRORS_LLM_MESSAGETYPE') ?? env('LLM_MESSAGE_TYPE') ?? env('LLM_MESSAGETYPE') ?? '';
+    env('ERRORS_LLM_MESSAGE_TYPE') ??
+    env('ERRORS_LLM_MESSAGETYPE') ??
+    env('LLM_MESSAGE_TYPE') ??
+    env('LLM_MESSAGETYPE') ??
+    '';
+
+  const modeRaw = env('ERRORS_LLM_MODE') ?? env('LLM_MODE') ?? '';
+
+  const timeoutRaw = env('ERRORS_LLM_TIMEOUT') ?? env('LLM_TIMEOUT') ?? '';
+  const timeoutNum = Number(timeoutRaw);
+  const timeout =
+    !timeoutRaw || isNaN(timeoutNum) || timeoutNum <= 0 ? 10000 : timeoutNum;
+
+  const channelRaw = env('ERRORS_LLM_CHANNEL') ?? env('LLM_CHANNEL') ?? '';
+
+  const logFile =
+    String(env('ERRORS_LLM_LOG_FILE') ?? '').trim() || './errors.llm.log';
+
+  const rateLimitRaw =
+    env('ERRORS_LLM_RATE_LIMIT') ?? env('LLM_RATE_LIMIT') ?? '';
+  const rateLimitNum = Number(rateLimitRaw);
+  const rateLimit =
+    !rateLimitRaw || isNaN(rateLimitNum) || rateLimitNum <= 0
+      ? 10
+      : Math.floor(rateLimitNum);
+
+  const cacheRaw = env('ERRORS_LLM_CACHE') ?? '';
+  const cache =
+    cacheRaw === false ||
+    cacheRaw === 'false' ||
+    cacheRaw === '0' ||
+    cacheRaw === 0
+      ? false
+      : true;
+
+  const cacheTTLRaw = env('ERRORS_LLM_CACHE_TTL') ?? '';
+  const cacheTTLNum = Number(cacheTTLRaw);
+  const cacheTTL =
+    !cacheTTLRaw || isNaN(cacheTTLNum) || cacheTTLNum <= 0
+      ? 3600000
+      : cacheTTLNum;
 
   return {
     enabled: Boolean(enabled),
@@ -57,18 +140,35 @@ export function getErrorsLlmConfig() {
     apiKey: String(apiKey ?? '').trim(),
     model: String(model ?? '').trim(),
     messageType: normalizeMessageType(messageTypeRaw || 'endUser'),
+    mode: normalizeMode(modeRaw),
+    timeout,
+    channel: normalizeChannel(channelRaw),
+    logFile,
+    rateLimit,
+    cache,
+    cacheTTL,
   };
 }
 
-export { MESSAGE_TYPES };
+export { MESSAGE_TYPES, LLM_MODES, LLM_CHANNELS };
 
 /**
  * Validate errors.llm when enabled: require baseURL, apiKey, and model (after LLM_ fallback).
+ * Also warns about misconfigurations (e.g. channel set with sync mode).
  * Call at takeoff. Throws if enabled but config is invalid; no-op otherwise.
  * @throws {Error} If errors.llm.enabled is true but any of baseURL, apiKey, or model is missing
  */
 export function validateErrorsLlmAtTakeoff() {
-  const { enabled, baseURL, apiKey, model } = getErrorsLlmConfig();
+  const {
+    enabled,
+    baseURL,
+    apiKey,
+    model,
+    mode,
+    channel,
+    rateLimit,
+    cacheTTL,
+  } = getErrorsLlmConfig();
   if (!enabled) return;
 
   const missing = [];
@@ -81,4 +181,34 @@ export function validateErrorsLlmAtTakeoff() {
       `errors.llm is enabled but required config is missing: ${missing.join(', ')}. Set these env vars or disable errors.llm.enabled.`,
     );
   }
+
+  // Warn about channel set while mode is sync (channel only applies in async mode).
+  const channelRaw = String(
+    env('ERRORS_LLM_CHANNEL') ?? env('LLM_CHANNEL') ?? '',
+  ).trim();
+  if (mode === 'sync' && channelRaw) {
+    console.warn(
+      `[Tejas] errors.llm: channel="${channel}" is set but mode is "sync" — channel output only applies in async mode. Set ERRORS_LLM_MODE=async to use it.`,
+    );
+  }
+
+  // Warn about invalid numeric values that were silently reset to defaults.
+  const rateLimitRaw = String(
+    env('ERRORS_LLM_RATE_LIMIT') ?? env('LLM_RATE_LIMIT') ?? '',
+  ).trim();
+  if (
+    rateLimitRaw &&
+    (isNaN(Number(rateLimitRaw)) || Number(rateLimitRaw) <= 0)
+  ) {
+    console.warn(
+      `[Tejas] errors.llm: rateLimit value "${rateLimitRaw}" is invalid; defaulting to 10.`,
+    );
+  }
+
+  const cacheTTLRaw = String(env('ERRORS_LLM_CACHE_TTL') ?? '').trim();
+  if (cacheTTLRaw && (isNaN(Number(cacheTTLRaw)) || Number(cacheTTLRaw) <= 0)) {
+    console.warn(
+      `[Tejas] errors.llm: cacheTTL value "${cacheTTLRaw}" is invalid; defaulting to 3600000.`,
+    );
+  }
 }