mirror of
https://github.com/rmccorkl/TubeSage.git
synced 2026-07-22 06:45:31 +00:00
fix(ui): reword model/provider notices — "output budget" not "max tokens"
The standalone "Max tokens" field was removed, but the model- and provider-change notices still announced "Max tokens updated to N", implying a setting the user could go look at. N is a derived value (max output x (1 - reserve)). Reword the notices to "Output budget: N tokens (max output minus reserve)" so the number is honestly labelled as a computed result, not a hidden field. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
parent
12737a376b
commit
58d3a7d99c
1 changed files with 3 additions and 3 deletions
6
main.ts
6
main.ts
|
|
@ -2489,7 +2489,7 @@ ${contentToTranslate}
|
|||
|
||||
// No override and not in registry — compute from the same
|
||||
// conservative defaults the panel uses (16k maxOutput * 0.90 reserve).
|
||||
// This makes the "Max tokens updated to N" notice consistent with
|
||||
// This keeps the model-change "Output budget" notice consistent with
|
||||
// the panel's displayed values, instead of returning whatever was
|
||||
// last saved in settings.maxTokens (which is the *previous*
|
||||
// model's effective value and confuses the user).
|
||||
|
|
@ -4196,7 +4196,7 @@ class YouTubeTranscriptSettingTab extends PluginSettingTab {
|
|||
this.plugin.settings.maxTokens = effectiveMaxTokens;
|
||||
|
||||
// Show notice with effective token limit
|
||||
this.plugin.showNotice(`Max tokens set to ${effectiveMaxTokens} for ${value} provider`, 3000);
|
||||
this.plugin.showNotice(`Provider set to ${value}. Output budget: ${effectiveMaxTokens} tokens (max output minus reserve).`, 3000);
|
||||
|
||||
// Update settings
|
||||
await this.plugin.saveSettings();
|
||||
|
|
@ -4332,7 +4332,7 @@ class YouTubeTranscriptSettingTab extends PluginSettingTab {
|
|||
void (async () => {
|
||||
if (value !== 'custom') {
|
||||
await this.plugin.saveSettings();
|
||||
this.plugin.showNotice(`Model changed to ${value}. Max tokens updated to ${this.plugin.settings.maxTokens}`, 3000);
|
||||
this.plugin.showNotice(`Model changed to ${value}. Output budget: ${this.plugin.settings.maxTokens} tokens (max output minus reserve).`, 3000);
|
||||
} else {
|
||||
// Custom model selected - initialize custom model limits if they don't exist
|
||||
const currentModel = this.plugin.settings.selectedModels[provider];
|
||||
|
|
|
|||
Loading…
Reference in a new issue