Skip to content

Fix: Use real model value in aidermacs-set-model#182

Merged
MatthewZMD merged 1 commit intoMatthewZMD:mainfrom
OverbearingPearl:fix-abnormal-model-name-with-price
Nov 8, 2025
Merged

Fix: Use real model value in aidermacs-set-model#182
MatthewZMD merged 1 commit intoMatthewZMD:mainfrom
OverbearingPearl:fix-abnormal-model-name-with-price

Conversation

@OverbearingPearl
Copy link
Copy Markdown
Contributor

  • Use the actual model ID instead of the formatted string from the completion candidates.
  • This fixes an issue where the wrong model was being set.

- Use the actual model ID instead of the formatted string
  from the completion candidates.
- This fixes an issue where the wrong model was being set.
@OverbearingPearl
Copy link
Copy Markdown
Contributor Author

Hi @MatthewZMD
A regression was likely introduced by a past commit. After aidermacs-change-model, the code passes a string that includes pricing for subsequent API calls, which causes them to fail.

@MatthewZMD MatthewZMD merged commit 9cd6796 into MatthewZMD:main Nov 8, 2025
1 check passed
@MatthewZMD
Copy link
Copy Markdown
Owner

Makes sense, good job

@OverbearingPearl
Copy link
Copy Markdown
Contributor Author

Hi @MatthewZMD
When could this fix be released on ELPA?
I want to know what's the frequency of your releases?

@MatthewZMD
Copy link
Copy Markdown
Owner

MatthewZMD commented Nov 8, 2025

If this patch to be very stable, I can make a release momentarily and will be available on MELPA tomorrow

@OverbearingPearl
Copy link
Copy Markdown
Contributor Author

Got it. Thank you.

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