chore!: promote AIBridge from ExperimentalHandler
#21278
Conversation
…-bridge-experimentalhandler
|
Should this be marked as a breaking change? |
ExperimentalHandlerExperimentalHandler
ssncferreira
left a comment
There was a problem hiding this comment.
Probably not in the scope of this PR, but just to confirm: should /api/experimental/mcp/http be promoted to /api/v2 as well, or is MCP intentionally staying experimental?
Its to my understanding that we are promoting AI Bridge and not the MCP server just yet 🙂 |
|
@jakehwll should this be instead a Also, can you add two lines to the PR body describing what is breaking and how to migrate? We can then copy those to ChangeLog as it is. |
ExperimentalHandlerExperimentalHandler
Thats a very good point, this was a "fix" in my mind as its something we were supposed to do in v2.29.0. I've updated and given a migration path 🙂 |
I think there is a small typo in the PR description for the migration, should be |
Co-authored-by: Susana Ferreira <susana@coder.com>
dceecec
into
jakehwll/ai-bridge-request-logs-model
Closes #21052 and is related to #21218 This pull request implements a bunch of changes the team requested to improve the overall user experience when navigating to the Request Logs page. Since these are mostly UI changes, screenshots do a much better job of showing what's different than me trying to explain everything in words, so I've included them below. > [!WARNING] > More globally we've changed how we're rendering `<PaginationHeader />`, therefore we've renamed it to `<PaginationAmount />`. This will effect multiple layouts however its a global improvement we're intentionally looking to make. ### Preview  ### Changes 1. Date formatting set to a standardised way including the month as a `MMM` (i.e `Dec`) 2. Names and avatars are now rendered larger, furthermore they render the `.name` field falling back on `.username`. 3. Font-size increased from `text-xs` to `text-sm` to increase overall legibility globally. 4. Initiator is now rendered inside of a `<Badge />` and gives longer names a chance to render. 5. Models are now rendered in a `<Badge />` alongside their icons. 6. Input/Output tokens are in a shared `<Badge />`-group alongside truncation (when necessary). 7. Badge from `5` is rendered with the proper name being visible on hover (for use if truncated). 8. Move globally the `Showing X of X` counts to the bottom of the tables (effects other components outside of AI Bridge). 9. Icons are now included within the dropdown for the `Provider` filter. 10. Truncation of elements is now handled so larger prompts don't cause the table to require further scrolling. 11. New merged Input/Output `<dt>` element with the badge | Position | Pull-request | | -------- | ------------ | | ✅ | [fix: improve AI Bridge request logs UI/UX](#21252) | | | [feat: add AI Bridge request logs model filter](#21259) | | | [chore!: promote AIBridge from `ExperimentalHandler`](#21278) | | | [feat: implement request log collapsing prompt (`<RequestLogsPrompt />`)](#21313) | --------- Co-authored-by: ケイラ <mckayla@hey.com>
Addressing feedback found in #21252
Important
This pull-request removes endpoints from
ExperimentalHandlerfromcoderd.goand promotes the endpoints within the frontend. This means that we will no longer be serving AI Bridge under the/api/experimental/prefix now that things reached release inv2.29.0.Migration
The
/api/experimental/aibridgeprefix has been removed. Any clients, scripts, or integrations that previously called AI Bridge endpoints under/api/experimental/aibridgemust be updated to use the/api/v2/aibridgestable API routes introduced in v2.29.0.ExperimentalHandler<RequestLogsPrompt />)<SyntaxHighlighter />for AI BridgeToken Usages Metadata