sascha
/
Notes
mirror of https://github.com/TriliumNext/Notes
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2217 from TriliumNext/fix/llm-add-warning2 

feat(llm): add warning to the top of LLM Chat Notes and LLM settings that the feature is experimental
pull/2210/head
Elian Doran 2025-06-08 10:40:22 +07:00 committed by GitHub
parent fb8f3d1d29 29d9c9834a
commit 8c25b59bed
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
apps/client/src/translations/en/translation.json
Unescape Escape View File

@ -1260,6 +1260,7 @@
}, },
"create_new_ai_chat": "Create new AI Chat", "create_new_ai_chat": "Create new AI Chat",
"configuration_warnings": "There are some issues with your AI configuration. Please check your settings.", "configuration_warnings": "There are some issues with your AI configuration. Please check your settings.",
"experimental_warning": "The LLM feature is currently experimental - you have been warned.",
"selected_provider": "Selected Provider", "selected_provider": "Selected Provider",
"selected_provider_description": "Choose the AI provider for chat and completion features", "selected_provider_description": "Choose the AI provider for chat and completion features",
"select_model": "Select model...", "select_model": "Select model...",

4
apps/client/src/widgets/llm_chat/validation.ts
Unescape Escape View File

@ -2,6 +2,7 @@
* Validation functions for LLM Chat * Validation functions for LLM Chat
*/ */
import options from "../../services/options.js"; import options from "../../services/options.js";
import { t } from "../../services/i18n.js";
/** /**
* Validate providers configuration * Validate providers configuration
@ -37,6 +38,9 @@ export async function validateProviders(validationWarning: HTMLElement): Promise
// Check for configuration issues with providers in the precedence list // Check for configuration issues with providers in the precedence list
const configIssues: string[] = []; const configIssues: string[] = [];
// Always add experimental warning as the first item
configIssues.push(t("ai_llm.experimental_warning"));
// Check each provider in the precedence list for proper configuration // Check each provider in the precedence list for proper configuration
for (const provider of precedenceList) { for (const provider of precedenceList) {
if (provider === 'openai') { if (provider === 'openai') {

11
apps/client/src/widgets/type_widgets/options/ai_settings/ai_settings_widget.ts
Unescape Escape View File

@ -203,6 +203,11 @@ export default class AiSettingsWidget extends OptionsWidget {
// Get selected provider // Get selected provider
const selectedProvider = this.$widget.find('.ai-selected-provider').val() as string; const selectedProvider = this.$widget.find('.ai-selected-provider').val() as string;
// Start with experimental warning
const allWarnings = [
t("ai_llm.experimental_warning")
];
// Check for selected provider configuration // Check for selected provider configuration
const providerWarnings: string[] = []; const providerWarnings: string[] = [];
if (selectedProvider === 'openai') { if (selectedProvider === 'openai') {
@ -222,10 +227,8 @@ export default class AiSettingsWidget extends OptionsWidget {
} }
} }
// Combine all warnings // Add provider warnings to all warnings
const allWarnings = [ allWarnings.push(...providerWarnings);
...providerWarnings
];
// Show or hide warnings // Show or hide warnings
if (allWarnings.length > 0) { if (allWarnings.length > 0) {