From 3b5f467192135bce8326a10fc3b5ab66f573aad8 Mon Sep 17 00:00:00 2001 From: Chris Hunter Date: Mon, 27 Apr 2026 16:40:24 +0200 Subject: [PATCH] fix: add cost calculation to VolcEngine transform_get_response_api_response Greptile P1 issue: VolcEngineResponsesAPIConfig.transform_get_response_api_response was missing cost calculation, causing all GET-response calls to report $0 cost. Added _calculate_response_cost call with model extraction from response JSON or logging_obj, matching the pattern used in Manus provider. Fixes lines 362-380 issue raised in PR review. Co-Authored-By: Claude Sonnet 4.5 --- litellm/llms/volcengine/responses/transformation.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/litellm/llms/volcengine/responses/transformation.py b/litellm/llms/volcengine/responses/transformation.py index 8eef35363f3..840e6d4f207 100644 --- a/litellm/llms/volcengine/responses/transformation.py +++ b/litellm/llms/volcengine/responses/transformation.py @@ -377,6 +377,13 @@ class VolcEngineResponsesAPIConfig(OpenAIResponsesAPIConfig): response = ResponsesAPIResponse(**raw_response_json) response._hidden_params["additional_headers"] = processed_headers response._hidden_params["headers"] = raw_response_headers + + # Calculate costs from usage data (fixes P1 issue - VolcEngine GET-response) + usage_dict = raw_response_json.get("usage", {}) + model = raw_response_json.get("model") or getattr(logging_obj, "model", None) + if model: + self._calculate_response_cost(model, usage_dict, response) + return response #########################################################