Skip to content

feat: Add model preview suffix for better clarity in model names#232

Merged
jdneo merged 2 commits into
mainfrom
ethan/fix-model-preview-suffix
May 19, 2026
Merged

feat: Add model preview suffix for better clarity in model names#232
jdneo merged 2 commits into
mainfrom
ethan/fix-model-preview-suffix

Conversation

@ethanyhou
Copy link
Copy Markdown
Contributor

fix #229

Copilot AI review requested due to automatic review settings May 19, 2026 06:25
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses issue #229 by ensuring preview models are clearly labeled in the chat model picker, aligning the UI with other IDEs.

Changes:

  • Added a new i18n string for the preview suffix ((Preview)).
  • Exposed the new key via Messages.java.
  • Updated the model picker dropdown item builder to append the preview suffix to preview model display names (while keeping the selection id based on the raw model name).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
com.microsoft.copilot.eclipse.ui/src/com/microsoft/copilot/eclipse/ui/i18n/messages.properties Adds model_preview_suffix string resource.
com.microsoft.copilot.eclipse.ui/src/com/microsoft/copilot/eclipse/ui/i18n/Messages.java Adds NLS field for model_preview_suffix.
com.microsoft.copilot.eclipse.ui/src/com/microsoft/copilot/eclipse/ui/chat/ModelPickerGroupsBuilder.java Appends preview suffix to preview model labels and adjusts selected-label formatting.

@jdneo jdneo merged commit ee6a2ed into main May 19, 2026
4 checks passed
@jdneo jdneo deleted the ethan/fix-model-preview-suffix branch May 19, 2026 07:12
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.

[Bug] Preview model suffix is not set correctly.

3 participants