Skip to content

[router][openai] Rename prepare_mcp_payload_for_streaming and patch_streaming_response_json#16596

Merged
slin1237 merged 2 commits intomainfrom
chang/resp-refactor-7
Jan 7, 2026
Merged

[router][openai] Rename prepare_mcp_payload_for_streaming and patch_streaming_response_json#16596
slin1237 merged 2 commits intomainfrom
chang/resp-refactor-7

Conversation

@CatherineSue
Copy link
Copy Markdown
Collaborator

@CatherineSue CatherineSue commented Jan 6, 2026

Motivation

The name prepare_mcp_payload_for_streaming is a bit misleading. It is not only relevant to streaming. The function mainly removes the function calls from the payload, keep the mcp tools only.

Same applied to function patch_streaming_response_json

Modifications

  • Renamed prepare_mcp_payload_for_streamingprepare_mcp_tools_as_functions
  • Renamed patch_streaming_response_jsonpatch_response_with_request_metadata

Accuracy Tests

Benchmarking and Profiling

Checklist

Review Process

  1. Ping Merge Oncalls to start the PR flow. See the PR Merge Process.
  2. Get approvals from CODEOWNERS and other reviewers.
  3. Trigger CI tests with comments (/tag-run-ci-label, /rerun-failed-ci, /tag-and-rerun-ci) or contact authorized users to do so.
  4. After green CI and required approvals, ask Merge Oncalls to merge.

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@CatherineSue CatherineSue changed the title [router][openai] Rename prepare_mcp_payload_for_streaming [router][openai] Rename prepare_mcp_payload_for_streaming and patch_streaming_response_json Jan 6, 2026
@slin1237 slin1237 merged commit fb5b71d into main Jan 7, 2026
75 checks passed
@slin1237 slin1237 deleted the chang/resp-refactor-7 branch January 7, 2026 00:33
michaelzhang-ai pushed a commit to michaelzhang-ai/sglang that referenced this pull request Jan 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants