wire multiple strategies

Author: SamSaffron

app/models/ai_persona.rb

@@ -129,8 +129,6 @@ def class_instance
       tool_details
     ]
 
-    persona_class = DiscourseAi::AiBot::Personas::Persona.system_personas_by_id[self.id]
-
     instance_attributes = {}
     attributes.each do |attr|
       value = self.read_attribute(attr)
@@ -139,14 +137,6 @@ def class_instance
 
     instance_attributes[:username] = user&.username_lower
 
-    if persona_class
-      instance_attributes.each do |key, value|
-        # description/name are localized
-        persona_class.define_singleton_method(key) { value } if key != :description && key != :name
-      end
-      return persona_class
-    end
-
     options = {}
     force_tool_use = []
 
@@ -179,6 +169,16 @@ def class_instance
         klass
       end
 
+    persona_class = DiscourseAi::AiBot::Personas::Persona.system_personas_by_id[self.id]
+    if persona_class
+      instance_attributes.each do |key, value|
+        # description/name are localized
+        persona_class.define_singleton_method(key) { value } if key != :description && key != :name
+      end
+      persona_class.define_method(:options) { options }
+      return persona_class
+    end
+
     ai_persona_id = self.id
 
     Class.new(DiscourseAi::AiBot::Personas::Persona) do

config/locales/server.en.yml

@@ -306,7 +306,7 @@ en:
             description: "Language model to use for artifact edits"
           update_algorithm:
             name: "Update Algorithm"
-            description: "Algorithm used for artifact updates (full vs diff)"
+            description: "Ask LLM to fully replace, or use diff to update"
         google:
           base_query:
             name: "Base Search Query"

lib/ai_bot/tools/update_artifact.rb

@@ -65,9 +65,24 @@ def invoke
             return error_response("Attempting to update an artifact you are not allowed to")
           end
 
+          llm =
+            (
+              options[:editor_llm].present? &&
+                LlmModel.find_by(id: options[:editor_llm].to_i)&.to_llm
+            ) || self.llm
+
+          strategy =
+            (
+              if options[:update_algorithm] == "diff"
+                ArtifactUpdateStrategies::Diff
+              else
+                ArtifactUpdateStrategies::Full
+              end
+            )
+
           begin
             new_version =
-              ArtifactUpdateStrategies::Full.new(
+              strategy.new(
                 llm: llm,
                 post: post,
                 user: post.user,