feat (volcengine) : Support Volcengine responses api - #18508
Merged
2 commits merged intoJan 20, 2026
Merged
2 commits merged into
2 commits merged into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
LingXuanYin
changed the title
LingXuanYin
changed the title
LingXuanYin
changed the title
LingXuanYin
force-pushed
the
feat/volcengine_responses_api_support_25-11-14
branch
from
December 30, 2025 12:57
10057fc to
e90f96d
Compare
ghost
reviewed
Jan 3, 2026
ghost
requested a review
from AlexsanderHamir
January 3, 2026 18:57
|
@AlexsanderHamir if the lazy load changes look okay to you, please merge this PR |
Contributor
|
Acked, will review. |
LingXuanYin
force-pushed
the
feat/volcengine_responses_api_support_25-11-14
branch
from
January 4, 2026 03:36
a30df10 to
d7ff753
Compare
LingXuanYin
force-pushed
the
feat/volcengine_responses_api_support_25-11-14
branch
from
January 5, 2026 01:53
d7ff753 to
14a6064
Compare
LingXuanYin
force-pushed
the
feat/volcengine_responses_api_support_25-11-14
branch
from
January 6, 2026 02:54
14a6064 to
12fe984
Compare
Contributor
Author
|
Hi! Just checking in on this PR. Are there any blockers on the review side? Please let me know if I can help in any way. Thanks! @krrishdholakia @AlexsanderHamir |
AlexsanderHamir
approved these changes
Jan 7, 2026
Contributor
Author
|
Hi! I see the review is already approved. Could we please merge this when you have a moment? Let me know if anything else is needed from my side. Thanks! @AlexsanderHamir @krrishdholakia |
…iable `origin` is assigned to but never used fix llms/volcengine/responses/transformation.py:95: error: Argument "headers" to "VolcEngineError" has incompatible type add more supported optional params removed redundant manual logging/utils fallbacks so litellm/__init__.py uses the registry only.
LingXuanYin
force-pushed
the
feat/volcengine_responses_api_support_25-11-14
branch
from
January 11, 2026 18:09
8877949 to
0e2d414
Compare
Contributor
Author
|
rebased & confict solved |
LingXuanYin
changed the title
|
@AlexsanderHamir @krrishdholakia Could you please merge this? Thanks a lot! |
ghost
merged commit Jan 20, 2026
004bde2
into
BerriAI:litellm_staging_01_20_2026
6 of 7 checks passed
fzowl
pushed a commit
to fzowl/litellm
that referenced
this pull request
Jun 24, 2026
* Add Volcengine responses adapter * fix llms/volcengine/responses/transformation.py:507:9: F841 Local variable `origin` is assigned to but never used fix llms/volcengine/responses/transformation.py:95: error: Argument "headers" to "VolcEngineError" has incompatible type add more supported optional params removed redundant manual logging/utils fallbacks so litellm/__init__.py uses the registry only.
This pull request was closed.
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Relevant issues
#18507
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
tests/litellm/directory, Adding at least 1 test is a hard requirement - see details (seetests/test_litellm/llms/volcengine/responses/test_volcengine_responses_transformation.py)make test-unit(not run locally; only targeted pytest below)Tests executed:
poetry run pytest tests/test_litellm/llms/volcengine/responses/test_volcengine_responses_transformation.py -q(passes; pytest cache writes are blocked on this machine so pytest emits a cache warning)CI (LiteLLM team)
Branch creation CI run
Link: N/A (not requested yet)
CI run for the last commit
Link: N/A (not requested yet)
Merge / cherry-pick CI run
Links: N/A (not requested yet)
Type
Changes
extra_bodykeys) and sanitizes optional params before delegating to the OpenAI transformer.tests/test_litellm/llms/volcengine/responses/test_volcengine_responses_transformation.pyfor param filtering/dropping, URL building, auth resolution, and response parsing.