mirror of
https://github.com/danny-avila/LibreChat.git
synced 2026-08-27 20:24:21 +00:00
* 🧾 fix: Honor Disabled Transactions on the Assistants Usage Path Thread the resolved transactions config through `recordUsage` from each of its callers, so `transactions.enabled: false` is honored on the assistants token spend path. * 🧾 fix: Thread the transactions config through the vision-request caller Address review: `ToolService.processVisionRequest` also records usage without the resolved config, and `recordUsage`'s documented return type did not match the function. * 🧾 fix: Set the resolved transactions config after the usage spread - provider usage could carry a `transactions` key that overwrote the trusted value - matches the ordering the other `recordUsage` callers already use
91 lines
2.4 KiB
JavaScript
91 lines
2.4 KiB
JavaScript
/**
|
|
* Tests for recordUsage - the assistants-side token spend path.
|
|
*
|
|
* `createTransaction` reads the guard out of caller-supplied data, so an
|
|
* omitted `transactions` is indistinguishable from an enabled one and the
|
|
* write proceeds even when the resolved config says `enabled: false`.
|
|
*/
|
|
|
|
const mockSpendTokens = jest.fn().mockResolvedValue();
|
|
|
|
jest.mock('@librechat/api', () => ({
|
|
countTokens: jest.fn().mockResolvedValue(0),
|
|
}));
|
|
|
|
jest.mock('@librechat/data-schemas', () => ({
|
|
escapeRegExp: jest.fn((str) => str),
|
|
}));
|
|
|
|
jest.mock('~/models', () => ({
|
|
recordMessage: jest.fn(),
|
|
getMessages: jest.fn(),
|
|
saveConvo: jest.fn(),
|
|
spendTokens: (...args) => mockSpendTokens(...args),
|
|
}));
|
|
|
|
jest.mock('~/server/services/Files/process', () => ({
|
|
retrieveAndProcessFile: jest.fn(),
|
|
}));
|
|
|
|
const { recordUsage } = require('./manage');
|
|
|
|
describe('recordUsage', () => {
|
|
beforeEach(() => {
|
|
jest.clearAllMocks();
|
|
});
|
|
|
|
it('forwards the resolved transactions config to spendTokens', async () => {
|
|
await recordUsage({
|
|
prompt_tokens: 100,
|
|
completion_tokens: 50,
|
|
model: 'gpt-4',
|
|
user: 'user-123',
|
|
conversationId: 'convo-123',
|
|
transactions: { enabled: false },
|
|
});
|
|
|
|
expect(mockSpendTokens).toHaveBeenCalledTimes(1);
|
|
expect(mockSpendTokens).toHaveBeenCalledWith(
|
|
{
|
|
user: 'user-123',
|
|
model: 'gpt-4',
|
|
context: 'message',
|
|
conversationId: 'convo-123',
|
|
transactions: { enabled: false },
|
|
},
|
|
{ promptTokens: 100, completionTokens: 50 },
|
|
);
|
|
});
|
|
|
|
it('forwards the config alongside an explicit context', async () => {
|
|
await recordUsage({
|
|
prompt_tokens: 10,
|
|
completion_tokens: 5,
|
|
model: 'gpt-4',
|
|
user: 'user-123',
|
|
conversationId: 'convo-123',
|
|
context: 'incomplete',
|
|
transactions: { enabled: true },
|
|
});
|
|
|
|
expect(mockSpendTokens).toHaveBeenCalledWith(
|
|
expect.objectContaining({ context: 'incomplete', transactions: { enabled: true } }),
|
|
expect.any(Object),
|
|
);
|
|
});
|
|
|
|
it('leaves the call unchanged when no config is supplied', async () => {
|
|
await recordUsage({
|
|
prompt_tokens: 10,
|
|
completion_tokens: 5,
|
|
model: 'gpt-4',
|
|
user: 'user-123',
|
|
conversationId: 'convo-123',
|
|
});
|
|
|
|
expect(mockSpendTokens).toHaveBeenCalledWith(
|
|
expect.objectContaining({ transactions: undefined }),
|
|
expect.any(Object),
|
|
);
|
|
});
|
|
});
|