Skip to content

Commit dceecec

Browse files
jakehwllssncferreira
jakehwll
and
ssncferreira
authored
chore!: promote AIBridge from ExperimentalHandler (#21278)
Addressing feedback found in #21252 > [!IMPORTANT] > This pull-request removes endpoints from `ExperimentalHandler` from `coderd.go` and 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 in [`v2.29.0`](https://github.com/coder/coder/releases/tag/v2.29.0). > > ### Migration > > The `/api/experimental/aibridge` prefix has been removed. Any clients, scripts, or integrations that previously called AI Bridge endpoints under `/api/experimental/aibridge` must be updated to use the `/api/v2/aibridge` stable API routes introduced in v2.29.0. | Position | Pull-request | | -------- | ------------ | | | [fix: improve AI Bridge request logs UI/UX](#21252) | | | [feat: add AI Bridge request logs model filter](#21259) | | ✅ | [fix: promote AIBridge from `ExperimentalHandler`](#21278) | --------- Co-authored-by: Susana Ferreira <susana@coder.com>
1 parent 6078455 commit dceecec

File tree

5 files changed

+273
-239
lines changed

5 files changed

+273
-239
lines changed

‎enterprise/coderd/aibridge_test.go‎

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -674,11 +674,6 @@ func TestAIBridgeRouting(t *testing.T) {
674674
path: "/api/v2/aibridge/openai/v1/chat/completions",
675675
expectedPath: "/openai/v1/chat/completions",
676676
},
677-
{
678-
name: "ExperimentalPrefix",
679-
path: "/api/experimental/aibridge/openai/v1/chat/completions",
680-
expectedPath: "/openai/v1/chat/completions",
681-
},
682677
}
683678

684679
for _, tc := range cases {

‎enterprise/coderd/coderd.go‎

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -226,10 +226,8 @@ func New(ctx context.Context, options *Options) (_ *API, err error) {
226226
return api.refreshEntitlements(ctx)
227227
}
228228

229-
api.AGPL.ExperimentalHandler.Group(func(r chi.Router) {
230-
// Deprecated.
231-
// TODO: remove with Beta release.
232-
r.Route("/aibridge", aibridgeHandler(api, apiKeyMiddleware))
229+
api.AGPL.ExperimentalHandler.Group(func(_ chi.Router) {
230+
// Add enterprise-only experimental routes here
233231
})
234232

235233
api.AGPL.APIHandler.Group(func(r chi.Router) {

0 commit comments

Comments
 (0)