From 6fbe33ede7ec8d9e46eb3f55f0ffb217cc830c99 Mon Sep 17 00:00:00 2001 From: zsxh1990 <445655361@qq.com> Date: Fri, 14 Aug 2026 00:47:34 +0800 Subject: [PATCH] fix(agents): remove unreachable recordUsage code The recordUsage call in agents/errors.js was unreachable because: 1. `run` variable was declared but never assigned 2. `run.usage` and `run.model` would always throw TypeError 3. The catch block swallowed the error with misleading log message This was a copy/paste regression from assistants/errors.js where `run` is properly assigned via `openai.beta.threads.runs.retrieve()`. Since agents have no equivalent API, the entire block was dead code. Closes #14792 Signed-off-by: zsxh1990 Signed-off-by: zsxh1990 <445655361@qq.com> --- api/server/controllers/agents/errors.js | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/api/server/controllers/agents/errors.js b/api/server/controllers/agents/errors.js index b16ce75591c..66fdda19d9c 100644 --- a/api/server/controllers/agents/errors.js +++ b/api/server/controllers/agents/errors.js @@ -2,7 +2,6 @@ const { logger } = require('@librechat/data-schemas'); const { CacheKeys, ViolationTypes } = require('librechat-data-provider'); const { sendResponse } = require('~/server/middleware/error'); -const { recordUsage } = require('~/server/services/Threads'); const getLogStores = require('~/cache/getLogStores'); const { getConvo } = require('~/models'); @@ -111,18 +110,6 @@ const createErrorHandler = ({ req, res, getContext, originPath = '/assistants/ch await new Promise((resolve) => setTimeout(resolve, 2000)); - let run; - try { - await recordUsage({ - ...run.usage, - model: run.model, - user: req.user.id, - conversationId, - }); - } catch (error) { - logger.error(`[${originPath}] Error fetching or processing run`, error); - } - let finalEvent; try { finalEvent = {