From 9dbfb0575036d23dfe23a174069d07792b87ed43 Mon Sep 17 00:00:00 2001
From: awesomerobot <kris.aubuchon@discourse.org>
Date: Thu, 23 Jan 2025 15:13:06 -0500
Subject: [PATCH] remove log, update button, fix tests

---
 .../discourse/components/ai-embedding-editor.gjs    | 11 ++++++-----
 config/locales/client.en.yml                        |  6 +++---
 .../embeddings/ai_embedding_definition_spec.rb      | 13 +++++--------
 3 files changed, 14 insertions(+), 16 deletions(-)

diff --git a/assets/javascripts/discourse/components/ai-embedding-editor.gjs b/assets/javascripts/discourse/components/ai-embedding-editor.gjs
index 2329d5a5d..f3bdc31b8 100644
--- a/assets/javascripts/discourse/components/ai-embedding-editor.gjs
+++ b/assets/javascripts/discourse/components/ai-embedding-editor.gjs
@@ -222,7 +222,6 @@ export default class AiEmbeddingEditor extends Component {
           <h2>{{i18n "discourse_ai.embeddings.presets"}}</h2>
           <AdminSectionLandingWrapper>
             {{#each this.presets as |preset|}}
-              {{log preset}}
               <AdminSectionLandingItem
                 @titleLabelTranslated={{preset.name}}
                 @taglineLabel={{concat
@@ -248,10 +247,12 @@ export default class AiEmbeddingEditor extends Component {
 
       {{else}}
         {{#if this.editingModel.isNew}}
-          <div class="btn btn-flat back-button" {{on "click" this.resetForm}}>
-            {{icon "chevron-left"}}
-            {{i18n "back_button"}}
-          </div>
+          <DButton
+            @action={{this.resetForm}}
+            @label="back_button"
+            @icon="chevron-left"
+            class="btn-flat back-button"
+          />
         {{else}}
           <BackButton
             @route="adminPlugins.show.discourse-ai-embeddings"
diff --git a/config/locales/client.en.yml b/config/locales/client.en.yml
index 7b16d1f61..bf76df2bf 100644
--- a/config/locales/client.en.yml
+++ b/config/locales/client.en.yml
@@ -523,9 +523,9 @@ en:
         hints:
           dimensions_warning: "Once saved, this value can't be changed."
           matryoshka_dimensions: "Defines the size of nested embeddings used for hierarchical or multi-layered representation of data, similar to how nested dolls fit within each other."
-          embed_prompt: "Explains how to process text to create its numerical summary (embedding) for analysis or comparison."
-          search_prompt: "Explains how to process a search query to compare it with existing embeddings and find the best matches."
-          sequence_length: "The maximum number of words or tokens the system can process at once when creating embeddings or handling a query."
+          embed_prompt: "Tells the LLM how to process text to create its numerical summary (embedding) for analysis or comparison."
+          search_prompt: "Tells the LLM how to compare a search query with existing embeddings and find the best matches."
+          sequence_length: "The maximum number of tokens that can be processed at once when creating embeddings or handling a query."
           distance_function: "Determines how similarity between embeddings is calculated, using either cosine distance (measuring the angle between vectors) or negative inner product (measuring overlap of vector values)."
         display_name: "Name"
         provider: "Provider"
diff --git a/spec/system/embeddings/ai_embedding_definition_spec.rb b/spec/system/embeddings/ai_embedding_definition_spec.rb
index d7ea406f4..0ac48d4af 100644
--- a/spec/system/embeddings/ai_embedding_definition_spec.rb
+++ b/spec/system/embeddings/ai_embedding_definition_spec.rb
@@ -13,10 +13,9 @@
     visit "/admin/plugins/discourse-ai/ai-embeddings"
 
     find(".ai-embeddings-list-editor__new-button").click()
-    select_kit = PageObjects::Components::SelectKit.new(".ai-embedding-editor__presets")
-    select_kit.expand
-    select_kit.select_row_by_value(preset)
-    find(".ai-embedding-editor__next").click
+   
+    find("[data-preset-id='text-embedding-3-small'] button").click()
+
     find("input.ai-embedding-editor__api-key").fill_in(with: api_key)
     find(".ai-embedding-editor__save").click()
 
@@ -43,10 +42,8 @@
     visit "/admin/plugins/discourse-ai/ai-embeddings"
 
     find(".ai-embeddings-list-editor__new-button").click()
-    select_kit = PageObjects::Components::SelectKit.new(".ai-embedding-editor__presets")
-    select_kit.expand
-    select_kit.select_row_by_value("manual")
-    find(".ai-embedding-editor__next").click
+    
+    find("[data-preset-id='manual'] button").click()
 
     find("input.ai-embedding-editor__display-name").fill_in(with: "OpenAI's text-embedding-3-small")