Skip to content

feat: fix model nits#1376

Merged
k11kirky merged 1 commit intomainfrom
03-30-feat_fix_model_nits
Mar 30, 2026
Merged

feat: fix model nits#1376
k11kirky merged 1 commit intomainfrom
03-30-feat_fix_model_nits

Conversation

@k11kirky
Copy link
Copy Markdown
Contributor

@k11kirky k11kirky commented Mar 30, 2026

Problem

Model IDs in the ACP connection configuration options display with provider prefixes (e.g., "anthropic/claude-3-sonnet") instead of user-friendly names, making them harder to read in the UI.

Changes

  • Added formatModelId function to strip provider prefixes from model IDs for display purposes
  • Updated ACP connection adapter to format model option names using the new function while preserving original values
  • Improved type definitions for model options and groups with proper TypeScript interfaces
  • Fixed model sanitization logic to only apply default gateway model when not using codex adapter
  • Removed unused close capability from Claude agent initialization

How did you test this?

Manual testing of model selection UI to verify that model names display without provider prefixes while maintaining correct functionality for model selection and filtering.

closes: #1320

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@k11kirky k11kirky marked this pull request as ready for review March 30, 2026 20:44
@k11kirky k11kirky merged commit f1f061e into main Mar 30, 2026
16 checks passed
@k11kirky k11kirky deleted the 03-30-feat_fix_model_nits branch March 30, 2026 20:45
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.

Model selector UI issues

2 participants