Feat/1566 display factors in model list#1567
Merged
deathbeam merged 5 commits intoMay 25, 2026
Merged
Conversation
Expose the 'multiplier' field from model billing configuration into the
provider definitions. This change allows the plugin to track and display
usage cost multipliers for specific LLM models.
Changes include:
- Updating providers.lua to pull 'multiplier' from billing data.
- Updating init.lua to propagate the multiplier to the model selection UI.
- Adding a visual indicator in the model picker to show the multiplier.
Notable behavior change:
The model selection menu will now display billing multipliers, such as
'[x2]', next to supported models.
Testing:
Verify the model selector displays the multiplier correctly when a model
has billing information defined:
:lua print(vim.inspect(require('CopilotChat.config.providers').copilot))
Check that models without billing info remain unaffected.
Update the model selection interface to explicitly display 'x not specified' when a model multiplier is undefined. Previously, missing multipliers resulted in no indicator being shown. This change ensures visual consistency in the status line or UI components that list model configurations. Notable change: - The string 'x not specified' will now appear in the indicators table when a multiplier is nil. Testing: - Open the model selection menu and verify that models without a defined multiplier now show the fallback text instead of being empty.
Refactored the provider and UI logic for model selection. What changed: - Removed the log.info call for Copilot models in providers.lua as it added unnecessary noise to the logs. - Simplified the 'select_model' logic in init.lua by removing the conditional fallback for missing multipliers. Notable behavior changes: - Models without a multiplier will no longer show 'x not specified' in the picker list. Testing: - Check that the model picker still renders correctly. - Verify that logs are clean during model fetching.
for more information, see https://pre-commit.ci
Contributor
Author
|
This PR is to have the multipliers in the model list: |
deathbeam
approved these changes
May 25, 2026
Collaborator
|
thanks! |
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Closes #1566