Skip to content

add openai voice models - #1268

Merged
Ken Jiang (knjiang) merged 2 commits into
mainfrom
09-17-add_voice_openai_models
Sep 17, 2026
Merged

Ken Jiang (knjiang) merged 2 commits into
mainfrom
09-17-add_voice_openai_models

Conversation

@knjiang

@knjiang Ken Jiang (knjiang) commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

This PR adds openai voice models

@vercel

vercel Bot commented Sep 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
ai-proxy Ready Ready Preview Sep 17, 2026 7:34pm UTC

Request Review

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@knjiang Ken Jiang (knjiang) changed the title add voice openai models add openai voice models Sep 17, 2026
@knjiang
Ken Jiang (knjiang) marked this pull request as ready for review September 17, 2026 19:34
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 17, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-17T19:38:40.931830Z 2510607 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

"gpt-realtime-2.1": {
"format": "openai",
"flavor": "realtime",
"displayName": "GPT Realtime 2.1",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are we able to handle pricing for these yet?

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2510607c4a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "Codex (@codex) review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "Codex (@codex) address that feedback".

Comment on lines +204 to +205
"gpt-realtime-2.1": ["openai"],
"gpt-live-1": ["openai"],

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Move provider routing to the active gateway

These AvailableEndpointTypes entries are routing changes in the deprecated proxy, while the repository instructions reserve routing and provider translation for the parent gateway/ and permit only catalog changes here. Shipping this mapping changes the legacy package without updating the active gateway; keep the catalog metadata here and implement the provider routing in gateway/.

AGENTS.md reference: AGENTS.md:L6-L15

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gateway only

Comment on lines +1788 to +1792
"gpt-live-1": {
"format": "openai",
"flavor": "live",
"displayName": "GPT Live 1",
"available_providers": [

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Implement live-session support before advertising gpt-live-1

When a client selects gpt-live-1, this entry reports OpenAI as available even though packages/proxy/scripts/sync_models.ts:204-207 still explicitly classifies that exact model as not invocable through supported proxy endpoints. A repository-wide search also finds no /live/sessions handler or runtime handling for the new live flavor, so catalog-driven clients can select a model that cannot complete a session; remove the entry until the corresponding endpoint is implemented in the active gateway.

AGENTS.md reference: AGENTS.md:L6-L15

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gateway only

@knjiang
Ken Jiang (knjiang) merged commit ad7eb7d into main Sep 17, 2026
9 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.

2 participants