Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat(site): use /api/v2 chat API paths - #28498

Merged
ibetitsmike merged 2 commits into
mainfrom
mike/chat-api-v2-site
Aug 26, 2026
Merged

feat(site): use /api/v2 chat API paths#28498
ibetitsmike merged 2 commits into
mainfrom
mike/chat-api-v2-site

Conversation

@ibetitsmike

@ibetitsmike ibetitsmike commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator

Stack context

This is the final PR in the 3-PR chat API promotion stack: server compatibility mounts (#28496), codersdk promotion (#28497), and frontend path updates (this PR).

Summary

Switch promoted frontend chat and MCP REST and WebSocket calls to /api/v2. Debug runs, virtual desktop streaming, advisor, and computer-use provider routes remain on /api/experimental because those surfaces were not promoted.

Update the matching tests, stories, helpers, and end-to-end route expectations. Remote dogfood UAT passed with chat traffic verified on the v2 routes.

Note

Xum acted on Mike's behalf in this pull request.

@ibetitsmike

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

Reviewed commit: e13b39310b

ℹ️ 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 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 address that feedback".

@ibetitsmike
ibetitsmike force-pushed the mike/chat-api-v2-site branch from e13b393 to 06cd67d Compare August 24, 2026 18:19
@ibetitsmike

Copy link
Copy Markdown
Collaborator Author

@codex review

@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: 06cd67db91

ℹ️ 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 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 address that feedback".

Comment thread site/src/api/api.ts
@ibetitsmike
ibetitsmike force-pushed the mike/chat-api-v2-site branch from 06cd67d to dc814bb Compare August 24, 2026 18:41
@ibetitsmike

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. What shall we delve into next?

Reviewed commit: dc814bba3d

ℹ️ 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 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 address that feedback".

@ibetitsmike
ibetitsmike force-pushed the mike/chat-api-v2-site branch from dc814bb to 429364c Compare August 24, 2026 18:57
@ibetitsmike

Copy link
Copy Markdown
Collaborator Author

@codex review

@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: 429364c7e4

ℹ️ 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 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 address that feedback".

Comment thread site/src/api/api.ts
@ibetitsmike

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Delightful!

Reviewed commit: 429364c7e4

ℹ️ 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 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 address that feedback".

@ibetitsmike
ibetitsmike force-pushed the mike/chat-api-v2-site branch from 429364c to a9fc7a8 Compare August 25, 2026 01:08
@ibetitsmike

Copy link
Copy Markdown
Collaborator Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep it up!

Reviewed commit: a9fc7a8de1

ℹ️ 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 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 address that feedback".

@ibetitsmike
ibetitsmike force-pushed the mike/chat-api-v2-site branch from a9fc7a8 to 4e4af91 Compare August 25, 2026 02:12
@ibetitsmike

Copy link
Copy Markdown
Collaborator Author

@codex review

Please review current head 4e4af912c21736f1ffcd601566e03d355f66d692.

Xum acted on Mike's behalf.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep it up!

Reviewed commit: 4e4af912c2

ℹ️ 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 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 address that feedback".

@ibetitsmike
ibetitsmike force-pushed the mike/chat-api-v2-site branch from 4e4af91 to 5150063 Compare August 25, 2026 03:07
@ibetitsmike

Copy link
Copy Markdown
Collaborator Author

Xum acted on Mike's behalf in requesting a fresh Codex review for head 515006357e3f.

@ibetitsmike
ibetitsmike force-pushed the mike/chat-api-v2-site branch from 694adc8 to e5dc867 Compare August 25, 2026 05:03
@ibetitsmike

Copy link
Copy Markdown
Collaborator Author

@codex review

Please review current head e5dc8673f36ac17d6b1df3cac8037a2a81ea3331.

Xum acted on Mike's behalf.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Chef's kiss.

Reviewed commit: e5dc8673f3

ℹ️ 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 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 address that feedback".

ibetitsmike added a commit that referenced this pull request Aug 26, 2026
## Stack context

This is the base of a 3-PR stack promoting the chat API from
`/api/experimental` to `/api/v2`: server compatibility mounts (this PR),
codersdk promotion (#28497), and frontend path updates (#28498).

## Summary

Double-mount the stable chat and MCP handlers under `/api/v2` while
retaining the existing experimental routes for the one-release
compatibility window decided in CODAGT-921. CODAGT-922 tracks removing
the compatibility mounts.

The shared route builders preserve existing authentication and
middleware behavior. Experiment-gated, debug, tombstone, and legacy
default-organization model routes remain experimental-only. Signed file
URLs, external OAuth callback URLs, and mixed-version replica relays
also remain on the experimental prefix during the transition.

Update Swagger and the generated API reference for the promoted routes,
including the workspace lookup and a runnable raw-body chat file upload
example. Retain internal endpoints outside the published reference,
share chat-file rate limits across both prefixes, enable CORS for the v2
MCP routes, and cover dual mounts plus exclusions with compatibility
tests. Remote dogfood UAT passed for the promoted chat, model, MCP, and
file flows.

> [!NOTE]
> Xum acted on Mike's behalf in this pull request.
<!-- xum-attribution: model=claude-fable-5 thinking=high -->
@ibetitsmike
ibetitsmike force-pushed the mike/chat-api-v2-site branch from a451bd5 to 9293fc3 Compare August 26, 2026 14:17
Base automatically changed from mike/chat-api-v2-sdk to main August 26, 2026 16:46
ibetitsmike added a commit that referenced this pull request Aug 26, 2026
## Stack context

This is the second PR in the 3-PR chat API promotion stack: server
compatibility mounts (#28496), codersdk promotion (this PR), and
frontend path updates (#28498).

## Summary

Move the promoted chat and MCP SDK methods from `ExperimentalClient` to
`Client` and update them to use `/api/v2`. Methods for routes that
remain experimental stay on `ExperimentalClient`.

Update in-repository callers and generated types. The multi-replica chat
stream relay dials `/api/v2` directly: mixed-version replica sets are
not a supported upgrade path, so no experimental-path fallback is kept
(per review). Remote dogfood UAT passed for the composed stack.

> [!NOTE]
> Xum acted on Mike's behalf in this pull request.
<!-- xum-attribution: model=claude-fable-5 thinking=high -->
@ibetitsmike
ibetitsmike force-pushed the mike/chat-api-v2-site branch from 9293fc3 to 480ae9e Compare August 26, 2026 16:46
@ibetitsmike ibetitsmike reopened this Aug 26, 2026
@ibetitsmike

Copy link
Copy Markdown
Collaborator Author

CI note: the post-merge restack push (16:46 UTC) never reached GitHub Actions — no github-actions check suite exists for 480ae9e — because of the ongoing GitHub Actions outage (Incident with Actions | https://stspg.io/pg14nv9m3095 | started 2026-08-26T15:11:58.254Z). The close/reopen above was an attempt to retrigger CI; that event is queued behind the same outage. I'll kick CI again once Actions recovers if the backlog doesn't process it automatically.

🤖 Xum acted for Mike (@ibetitsmike).

@github-actions github-actions Bot locked and limited conversation to collaborators Aug 26, 2026
@coder coder unlocked this conversation Aug 26, 2026
@ibetitsmike
ibetitsmike merged commit 7394d2c into main Aug 26, 2026
50 of 55 checks passed
@ibetitsmike
ibetitsmike deleted the mike/chat-api-v2-site branch August 26, 2026 17:25
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants