Skip to content

Conversation

mshsheikh
Copy link
Contributor

This update replaces all openai_client.responses.create(...) invocations with openai_client.chat.completions.create(...) calls, leveraging the fully‑typed Chat Completions endpoint introduced in openai>=1.96.0.

  • No changes were made to chunking, code‑block handling, or translation logic.
  • Only the request surface has been updated to satisfy mypy/type‑stub requirements.
  • Dependencies remain untouched, and the script continues to function identically while passing CI type checks.

This update replaces all `openai_client.responses.create(...)` invocations with `openai_client.chat.completions.create(...)` calls, leveraging the fully‑typed Chat Completions endpoint introduced in `openai>=1.96.0`.

- No changes were made to chunking, code‑block handling, or translation logic.
- Only the request surface has been updated to satisfy mypy/type‑stub requirements.
- Dependencies remain untouched, and the script continues to function identically while passing CI type checks.
This update replaces all `openai_client.responses.create(...)` invocations with `openai_client.chat.completions.create(...)` calls, leveraging the fully‑typed Chat Completions endpoint introduced in `openai>=1.96.0`.

- No changes were made to chunking, code‑block handling, or translation logic.
- Only the request surface has been updated to satisfy mypy/type‑stub requirements.
- Dependencies remain untouched, and the script continues to function identically while passing CI type checks.
@seratch seratch added the wontfix This will not be worked on label Jul 15, 2025
@seratch
Copy link
Member

seratch commented Jul 15, 2025

thanks for sending this but we prefer using responses api over chat completions.

@seratch seratch closed this Jul 15, 2025
@mshsheikh mshsheikh deleted the patch-14 branch July 15, 2025 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants