Skip to content

Conversation

@knjiang
Copy link
Contributor

@knjiang knjiang commented Jan 14, 2026

Add Chat Completions API parameter test cases to the existing params.ts file for bidirectional mapping coverage.

New test coverage, got the request body from https://platform.openai.com/docs/api-reference/chat/create:

Category Responses API (existing) Chat Completions API (new)
Reasoning effort, summary + reasoning_effort
Text config json_object, json_schema ✓ response_format (both types)
Tools function, web_search, code_interpreter, specific choice, parallel_tool_calls ✓ function tools, tool_choice, parallel_tool_calls
Context instructions, truncation + system message (maps to instructions)
Storage store ✓ store
Caching service_tier, prompt_cache_key ✓ service_tier, prompt_cache_key
Metadata metadata, safety_identifier ✓ metadata, safety_identifier
Sampling - + temperature, top_p, frequency_penalty, presence_penalty, logprobs, top_logprobs
Output control - + n (multiple completions), stop, max_completion_tokens
Advanced - + prediction, seed, logit_bias

cross compatability coverage report -> https://github.com/braintrustdata/lingua/actions/runs/21009331121

Copy link
Contributor Author

knjiang commented Jan 14, 2026

@knjiang knjiang changed the title add chat completion test cases for parmas add chat completion parameter test cases Jan 14, 2026
@knjiang knjiang force-pushed the add_openai-chat_completion_test_cases_for_params branch from 1dca4d1 to ffa0b10 Compare January 14, 2026 20:00
@knjiang knjiang force-pushed the add_openai-responses_parameter_test_cases branch from f8ff480 to dc57d5e Compare January 14, 2026 20:00
@knjiang knjiang force-pushed the add_openai-chat_completion_test_cases_for_params branch from ffa0b10 to 38aa7e6 Compare January 14, 2026 20:48
@knjiang knjiang force-pushed the add_openai-responses_parameter_test_cases branch from dc57d5e to 84c3c24 Compare January 14, 2026 20:48
@knjiang knjiang marked this pull request as ready for review January 14, 2026 21:01
@knjiang knjiang requested review from ankrgyl and remh January 14, 2026 21:02
@knjiang knjiang force-pushed the add_openai-responses_parameter_test_cases branch from 84c3c24 to 48d812e Compare January 16, 2026 16:48
@knjiang knjiang force-pushed the add_openai-chat_completion_test_cases_for_params branch from 38aa7e6 to ab78b2e Compare January 16, 2026 16:48
@knjiang knjiang force-pushed the add_openai-chat_completion_test_cases_for_params branch from ab78b2e to e9d1772 Compare January 21, 2026 18:58
@knjiang knjiang force-pushed the add_openai-responses_parameter_test_cases branch from 48d812e to 8f18c37 Compare January 21, 2026 18:58
@knjiang knjiang changed the base branch from add_openai-responses_parameter_test_cases to graphite-base/58 January 23, 2026 00:18
@knjiang knjiang force-pushed the add_openai-chat_completion_test_cases_for_params branch from e9d1772 to 232b4d2 Compare January 23, 2026 00:18
@graphite-app graphite-app bot changed the base branch from graphite-base/58 to main January 23, 2026 00:19
@knjiang knjiang force-pushed the add_openai-chat_completion_test_cases_for_params branch 2 times, most recently from c58b1e6 to 232b4d2 Compare January 23, 2026 00:20
@knjiang knjiang force-pushed the add_openai-chat_completion_test_cases_for_params branch from 232b4d2 to befceb6 Compare January 23, 2026 00:38
@knjiang knjiang merged commit 9f28412 into main Jan 23, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants