Correctly import by slug

app/models/assistant/export.rb

@@ -42,8 +42,8 @@ def import_from_file(path: Rails.root.join(DEFAULT_ASSISTANT_FILE), users: User.
       assistants.each do |assistant|
         assistant = assistant.with_indifferent_access
         users.each do |user|
-          asst = user.assistants.find_or_create_by(name: assistant["name"])
-          asst.assign_attributes(assistant.except("name"))
+          asst = user.assistants.find_or_create_by(slug: assistant["slug"])
+          asst.assign_attributes(assistant.except("slug"))
           asst.save!
         end
       end

assistants.yml

@@ -3,6 +3,9 @@ assistants:
 - name: GPT-4o
   slug: gpt-4o
   language_model_api_name: gpt-4o
+- name: GPT-4o Mini
+  slug: gpt-4o-mini
+  language_model_api_name: gpt-4o-mini
 - name: Claude 3.5 Sonnet
   slug: claude-sonnet
   language_model_api_name: claude-3-5-sonnet-20241022

test/models/assistant/export_test.rb

@@ -21,7 +21,7 @@ class Assistant::ExportTest < ActiveSupport::TestCase
     assert File.exist?(path)
     storage = JSON.load_file(path)
     assistants = storage["assistants"]
-    assert_equal assistants.first.keys.sort, %w[name description instructions tools language_model_api_name].sort
+    assert_equal assistants.first.keys.sort, %w[name description instructions slug language_model_api_name].sort
   end
 
   test "export_to_file yaml" do
@@ -30,7 +30,7 @@ class Assistant::ExportTest < ActiveSupport::TestCase
     assert File.exist?(path)
     storage = YAML.load_file(path)
     assistants = storage["assistants"]
-    assert_equal assistants.first.keys.sort, %w[name description instructions tools language_model_api_name].sort
+    assert_equal assistants.first.keys.sort, %w[name description instructions slug language_model_api_name].sort
   end
 
   test "import_from_file with only new models" do
@@ -77,15 +77,14 @@ class Assistant::ExportTest < ActiveSupport::TestCase
     assert user.assistants.find_by(external_id: "new external_id")
   end
 
-  test "import_from_file with existing models by name" do
+  test "import_from_file with existing models by slug" do
     user = users(:keith)
     assistant = user.assistants.not_deleted.first
     assistants = [{
-      name: assistant.name,
+      name: "new name",
+      slug: assistant.slug,
       description: "new description",
       instructions: "new instructions",
-      tools: "new tools",
-      external_id: "new external_id",
       language_model_api_name: language_models(:gpt_4o).api_name
     }]
     storage = {
@@ -97,10 +96,9 @@ class Assistant::ExportTest < ActiveSupport::TestCase
       Assistant.import_from_file(path:, users: [user])
     end
     assistant.reload
+    assert_equal "new name", assistant.name
     assert_equal "new description", assistant.description
     assert_equal "new instructions", assistant.instructions
-    assert_equal "new tools", assistant.tools
-    assert_equal "new external_id", assistant.external_id
     assert_equal language_models(:gpt_4o).api_name, assistant.language_model_api_name
   end
 end