Skip to content

fix(agents): update edit screen to use aigw provider selector#2393

Merged
alenkacz merged 3 commits intoredpanda-data:masterfrom
alenkacz:av/fix-agent-edit-aigw-provider
Apr 20, 2026
Merged

fix(agents): update edit screen to use aigw provider selector#2393
alenkacz merged 3 commits intoredpanda-data:masterfrom
alenkacz:av/fix-agent-edit-aigw-provider

Conversation

@alenkacz
Copy link
Copy Markdown
Contributor

@alenkacz alenkacz commented Apr 20, 2026

Screenshot 2026-04-20 at 13 50 36 Screenshot 2026-04-20 at 14 34 45 ## Summary - Replace disabled "LLM Provider (Gateway)" field on agent edit screen with an editable dropdown that loads providers from the AI Gateway API - Models dynamically update when switching providers, matching create screen behavior - Add links from provider field to the LLM provider detail page (`/adp/llm-providers/{name}`) - Fix model rendering for Bedrock providers (skip `openaiCompatible` free-text fallback when using gateway) - Fix provider icon resolution using `LLMProviderType` mapping

Test plan

  • Open an agent that uses aigw, verify provider dropdown shows aigw providers
  • Switch provider in edit mode, verify models reload dynamically
  • Verify "View provider" link navigates to correct LLM provider detail page
  • Verify read-only view shows provider name as a link
  • Verify non-gateway agents still show the standard provider/model dropdowns

🤖 Generated with Claude Code

alenkacz and others added 2 commits April 20, 2026 14:36
Replace the disabled gateway field with an editable provider dropdown
that loads providers from the AI Gateway API, matching the create screen
behavior. Models dynamically update when switching providers. Adds links
to the LLM provider detail page.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
[LLMProviderType.LLM_PROVIDER_TYPE_OPENAI]: 'openai',
[LLMProviderType.LLM_PROVIDER_TYPE_ANTHROPIC]: 'anthropic',
[LLMProviderType.LLM_PROVIDER_TYPE_GOOGLE]: 'google',
[LLMProviderType.LLM_PROVIDER_TYPE_BEDROCK]: 'openaiCompatible',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we did not mean to use bedrock as openai compatible?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you also try to update the buf schema registry to make sure we use the latest in both ADP and here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah we don't support bedrock I am working on that in a different PR

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@alenkacz alenkacz enabled auto-merge April 20, 2026 14:40
@alenkacz alenkacz merged commit 9e0ba89 into redpanda-data:master Apr 20, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants