mirror of
https://github.com/BerriAI/litellm.git
synced 2026-09-14 23:21:35 +00:00
bc5638b38e
Some checks are pending
GitHub Actions Security Analysis / zizmor (push) Waiting to run
The "day-by-day by user and model" usage export overcounted spend by repeating each user-day total once per model. generateDailyWithModelsData iterated day.breakdown.models crossed with the entity's own api_key_breakdown and added the api key's full daily spend to every model, leaving the per-model object (modelData) unused and never matching the api key to the model. A user who called N models got N identical rows, so the column summed to N times the real spend; one reported export totaled $167,953 against a true $21,353 (7.87x). Attribute each model only the spend of the keys that actually used it by intersecting the entity's api keys with modelData.api_key_breakdown. This mirrors the already-correct pattern in activity_metrics.tsx, so per-model rows now sum back to the user-day total and models a user never called no longer appear. The existing tests passed only because their mocks omitted models[*].api_key_breakdown and never asserted numeric values. The mocks now match the real /user/daily/activity payload, and new tests assert that per-model spend sums to the user-day total and that uncalled models are omitted; both fail on the old code. Resolves LIT-3866 |
||
|---|---|---|
| .. | ||
| EntityUsageExportModal.test.tsx | ||
| EntityUsageExportModal.tsx | ||
| ExportFormatSelector.test.tsx | ||
| ExportFormatSelector.tsx | ||
| ExportSummary.test.tsx | ||
| ExportSummary.tsx | ||
| ExportTypeSelector.test.tsx | ||
| ExportTypeSelector.tsx | ||
| index.ts | ||
| types.ts | ||
| UsageExportHeader.test.tsx | ||
| UsageExportHeader.tsx | ||
| utils.test.ts | ||
| utils.ts | ||