-
Notifications
You must be signed in to change notification settings - Fork 552
Add chatgpt-web provider: route ChatGPT Web models via codex-chatgpt-web bridge #876
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -1282,6 +1282,35 @@ export const PROVIDER_REGISTRY: readonly ProviderRegistryEntry[] = [ | |
| // A self-hosted proxy may legitimately run without a master key. | ||
| keyOptional: true, | ||
| }, | ||
| { | ||
| id: "chatgpt-web", | ||
| label: "ChatGPT Web (codex-chatgpt-web)", | ||
| baseUrl: "http://127.0.0.1:17841/v1", | ||
| adapter: "openai-responses", | ||
| authKind: "local", | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
When a user opens the dashboard's Add Provider catalog, this entry is absent because AGENTS.md reference: src/AGENTS.md:L18-L18 Useful? React with 👍 / 👎. |
||
| allowPrivateNetworkByDefault: true, | ||
| allowBaseUrlOverride: true, | ||
| keyOptional: true, | ||
|
Comment on lines
+1292
to
+1293
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
With this entry present, AGENTS.md reference: src/AGENTS.md:L24-L26 Useful? React with 👍 / 👎. |
||
| liveModels: true, | ||
| models: ["chatgpt-web/light", "chatgpt-web/medium", "chatgpt-web/high", "chatgpt-web/extra-high", "chatgpt-web/pro"], | ||
| defaultModel: "chatgpt-web/high", | ||
| modelContextWindows: { | ||
| "chatgpt-web/light": 256_000, | ||
| "chatgpt-web/medium": 256_000, | ||
| "chatgpt-web/high": 256_000, | ||
| "chatgpt-web/extra-high": 256_000, | ||
| "chatgpt-web/pro": 256_000, | ||
| }, | ||
| modelInputModalities: { | ||
| "chatgpt-web/light": ["text", "image"], | ||
| "chatgpt-web/medium": ["text", "image"], | ||
| "chatgpt-web/high": ["text", "image"], | ||
| "chatgpt-web/extra-high": ["text", "image"], | ||
| "chatgpt-web/pro": ["text", "image"], | ||
| }, | ||
| dashboardUrl: "https://github.com/miuuyy/codex-chatgpt-web", | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
This provider requires a separately installed loopback bridge, but the repository-wide docs search finds no AGENTS.md reference: AGENTS.md:L192-L193 Useful? React with 👍 / 👎. |
||
| note: "Local ChatGPT Web browser bridge. Requires codex-chatgpt-web running on loopback (default port 17841). Routes ChatGPT Web Instant–Pro through an embedded browser session; no API key needed.", | ||
| }, | ||
| { | ||
| id: "ollama-cloud", | ||
| label: "Ollama Cloud", | ||
|
|
||
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.
The reviewed commit is based directly on
88589c9(release: v2.9.1) rather than the integration line, and its own subject marks it[WRONG BRANCH]. Rebase or recreate this change ondevbefore opening or merging it so the feature does not bypass the repository's single integration branch and current development changes.AGENTS.md reference: AGENTS.md:L177-L178
Useful? React with 👍 / 👎.