forked from Hopsken/gpt-port
-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add support for OpenAI Realtime and Responses API #1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
chrisehlee
wants to merge
2
commits into
main
Choose a base branch
from
support-realtime-responses-api
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
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
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
tomasmcm
reviewed
Oct 7, 2025
|
Also, from the build logs: |
tomasmcm
reviewed
Oct 13, 2025
| export const processRequest = async (c:Context) => { | ||
| const result = await c.req.valid('json') | ||
| export const processRequest = async (c: Context) => { | ||
| const result = (c.req as any).valid('json') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
are you sure about removing await here?
|
It is my understanding that zValidator is already cached and not a promise so no need for awaiting I might be wrong but the TS types also corroborate
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
________________________________
From: Tomás Correia Marques ***@***.***>
Sent: Monday, October 13, 2025 1:46:50 PM
To: whitesmith/gpt-port ***@***.***>
Cc: chrisehlee ***@***.***>; Assign ***@***.***>
Subject: Re: [whitesmith/gpt-port] feat: add support for OpenAI Realtime and Responses API (PR #1)
@tomasmcm commented on this pull request.
________________________________
In src/server/routes/proxy.route.ts<#1 (comment)>:
@@ -8,8 +8,8 @@ import { validateAPIToken } from '../middleware/validate-token.middleware'
import { MiddlewareHandler, Context } from 'hono'
import { Env } from '../types'
…-export const processRequest = async (c:Context) => {
- const result = await c.req.valid('json')
+export const processRequest = async (c: Context) => {
+ const result = (c.req as any).valid('json')
are you sure about removing await here?
—
Reply to this email directly, view it on GitHub<#1 (review)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ADBIMFTCS43ZURSORBIFFG33XONLVAVCNFSM6AAAAACIPT6D42VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZTGMZRGQ4TEMRWGM>.
You are receiving this because you were assigned.Message ID: ***@***.***>
|
tomasmcm
approved these changes
Oct 15, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
added support for openai's responses api and realtime api with webrtc.
responses api works the same as other endpoints - just pass model and input and it routes to the provider.
for realtime added the webrtc call endpoints. most importantly the create calls endpoint accepts both multipart form data (with session + sdp fields) or direct sdp data with model in query param. also added client secrets for ephemeral tokens and accept/reject/hangup calls.
updated proxy middleware to handle formdata properly and added the openai-beta header for realtime endpoints.