Skip to content
This repository was archived by the owner on Jun 24, 2025. It is now read-only.

feat(llm): add warning to the top of LLM Chat Notes and LLM settings that the feature is experimental #2217

Merged
merged 1 commit into from
Jun 8, 2025
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions apps/client/src/translations/en/translation.json
Original file line number Diff line number Diff line change
Expand Up @@ -1260,6 +1260,7 @@
},
"create_new_ai_chat": "Create new AI Chat",
"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_description": "Choose the AI provider for chat and completion features",
"select_model": "Select model...",
Expand Down
4 changes: 4 additions & 0 deletions apps/client/src/widgets/llm_chat/validation.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
* Validation functions for LLM Chat
*/
import options from "../../services/options.js";
import { t } from "../../services/i18n.js";

/**
* Validate providers configuration
Expand Down Expand Up @@ -37,6 +38,9 @@ export async function validateProviders(validationWarning: HTMLElement): Promise
// Check for configuration issues with providers in the precedence list
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
for (const provider of precedenceList) {
if (provider === 'openai') {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -203,6 +203,11 @@ export default class AiSettingsWidget extends OptionsWidget {
// Get selected provider
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
const providerWarnings: string[] = [];
if (selectedProvider === 'openai') {
Expand All @@ -222,10 +227,8 @@ export default class AiSettingsWidget extends OptionsWidget {
}
}

// Combine all warnings
const allWarnings = [
...providerWarnings
];
// Add provider warnings to all warnings
allWarnings.push(...providerWarnings);

// Show or hide warnings
if (allWarnings.length > 0) {
Expand Down