feat: widen ai_provider_type enum for chatd providers#25394
Merged
Conversation
Docs preview📖 View docs preview for |
dannykopping
force-pushed
the
dk/aibridge-providers-deprecate-env
branch
2 times, most recently
from
b7c4544 to
b2477d9
Compare
dannykopping
force-pushed
the
dk/aibridge-providers-chatd-types
branch
from
4868118 to
2147859
Compare
This was referenced May 15, 2026
Contributor
Author
dannykopping
changed the title
dannykopping
force-pushed
the
dk/aibridge-providers-chatd-types
branch
from
4f29b23 to
cd662ae
Compare
dannykopping
force-pushed
the
dk/aibridge-providers-deprecate-env
branch
from
b2477d9 to
0cbdf1a
Compare
dannykopping
changed the base branch from
dk/aibridge-providers-deprecate-env
to
graphite-base/25394
dannykopping
force-pushed
the
graphite-base/25394
branch
from
0cbdf1a to
787b845
Compare
dannykopping
force-pushed
the
dk/aibridge-providers-chatd-types
branch
from
cd662ae to
4075c84
Compare
dannykopping
changed the base branch from
graphite-base/25394
to
dk/aibridge-providers-pool-reload
dannykopping
force-pushed
the
dk/aibridge-providers-pool-reload
branch
from
787b845 to
45ef2af
Compare
dannykopping
force-pushed
the
dk/aibridge-providers-chatd-types
branch
from
4075c84 to
a979851
Compare
dannykopping
changed the base branch from
dk/aibridge-providers-pool-reload
to
graphite-base/25394
dannykopping
force-pushed
the
dk/aibridge-providers-chatd-types
branch
from
a979851 to
292e691
Compare
dannykopping
changed the title
dannykopping
changed the title
dannykopping
changed the title
ai_provider_type enum for chatd providers
Adds six new values to the ai_provider_type Postgres enum so the chatd-side migration can preserve type fidelity when it lands: azure, bedrock, google, openai-compat, openrouter, vercel. Mirrors the new values in codersdk.AIProviderType and regenerates the SQL dump, Go models, and typesGenerated.ts enum union. No runtime behaviour change: validation, env/seed mapping, and dbsource routing for the new types come later in the stack. Bedrock here is the enum value used to tag native Bedrock providers when aibridge grows direct Bedrock support; current Bedrock providers still ride on type='anthropic' plus the bedrock discriminator in ai_providers.settings.
dannykopping
force-pushed
the
dk/aibridge-providers-chatd-types
branch
from
292e691 to
2a7877f
Compare
ibetitsmike
approved these changes
May 18, 2026
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
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
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.
Disclaimer: implemented by a Coder Agent using Claude Opus 4.7
Splits the
ai_provider_typeenum widening out of the original chatd PR so it can land as a focused database change ahead of the runtime and CLI plumbing it unlocks.Adds six values to the Postgres enum:
azure,bedrock,google,openai-compat,openrouter,vercel. Mirrors the new values incodersdk.AIProviderTypeand regenerates the SQL dump, Go models, andtypesGenerated.ts. No runtime behaviour change: validation widening, env/seed mapping, and dbsource routing for the new types live in #25435.bedrockhere is the enum value used to tag native Bedrock providers when aibridge grows direct Bedrock support; current Bedrock providers still ride ontype='anthropic'plus the bedrock discriminator inai_providers.settings.The down migration is a no-op because Postgres does not allow removing enum values safely, matching the precedent in
000495_ai_providers.down.sql.Part of the implementation of RFC: Migrate chatd providers to ai_providers and route Coder-agent traffic via aibridge.