DEV: Remove 'experimental' from translation features

nattsw · nattsw · commit 37b63f4a882a · 2025-06-17T17:03:57.000+08:00
diff --git a/app/jobs/regular/detect_translate_post.rb b/app/jobs/regular/detect_translate_post.rb
@@ -35,7 +35,7 @@ def execute(args)
         end
       end
 
-      locales = SiteSetting.experimental_content_localization_supported_locales.split("|")
+      locales = SiteSetting.content_localization_supported_locales.split("|")
       return if locales.blank?
 
       locales.each do |locale|
diff --git a/app/jobs/regular/detect_translate_topic.rb b/app/jobs/regular/detect_translate_topic.rb
@@ -33,7 +33,7 @@ def execute(args)
         end
       end
 
-      locales = SiteSetting.experimental_content_localization_supported_locales.split("|")
+      locales = SiteSetting.content_localization_supported_locales.split("|")
       return if locales.blank?
 
       locales.each do |locale|
diff --git a/app/jobs/regular/localize_categories.rb b/app/jobs/regular/localize_categories.rb
@@ -11,7 +11,7 @@ def execute(args)
       return if !SiteSetting.discourse_ai_enabled
       return if !SiteSetting.ai_translation_enabled
 
-      locales = SiteSetting.experimental_content_localization_supported_locales.split("|")
+      locales = SiteSetting.content_localization_supported_locales.split("|")
       return if locales.blank?
 
       cat_id = args[:from_category_id] || Category.order(:id).first&.id
diff --git a/app/jobs/regular/localize_posts.rb b/app/jobs/regular/localize_posts.rb
@@ -11,7 +11,7 @@ def execute(args)
       return if !SiteSetting.discourse_ai_enabled
       return if !SiteSetting.ai_translation_enabled
 
-      locales = SiteSetting.experimental_content_localization_supported_locales.split("|")
+      locales = SiteSetting.content_localization_supported_locales.split("|")
       return if locales.blank?
 
       limit = args[:limit] || BATCH_SIZE
diff --git a/app/jobs/regular/localize_topics.rb b/app/jobs/regular/localize_topics.rb
@@ -11,7 +11,7 @@ def execute(args)
       return if !SiteSetting.discourse_ai_enabled
       return if !SiteSetting.ai_translation_enabled
 
-      locales = SiteSetting.experimental_content_localization_supported_locales.split("|")
+      locales = SiteSetting.content_localization_supported_locales.split("|")
       return if locales.blank?
 
       limit = args[:limit] || BATCH_SIZE
diff --git a/app/jobs/scheduled/category_localization_backfill.rb b/app/jobs/scheduled/category_localization_backfill.rb
@@ -8,7 +8,7 @@ class CategoryLocalizationBackfill < ::Jobs::Scheduled
     def execute(args)
       return if !SiteSetting.discourse_ai_enabled
       return if !SiteSetting.ai_translation_enabled
-      return if SiteSetting.experimental_content_localization_supported_locales.blank?
+      return if SiteSetting.content_localization_supported_locales.blank?
 
       Jobs.enqueue(:localize_categories)
     end
diff --git a/app/jobs/scheduled/post_localization_backfill.rb b/app/jobs/scheduled/post_localization_backfill.rb
@@ -9,7 +9,7 @@ def execute(args)
       return if !SiteSetting.discourse_ai_enabled
       return if !SiteSetting.ai_translation_enabled
 
-      return if SiteSetting.experimental_content_localization_supported_locales.blank?
+      return if SiteSetting.content_localization_supported_locales.blank?
       return if SiteSetting.ai_translation_backfill_rate == 0
 
       Jobs.enqueue(:localize_posts, limit: SiteSetting.ai_translation_backfill_rate)
diff --git a/app/jobs/scheduled/topic_localization_backfill.rb b/app/jobs/scheduled/topic_localization_backfill.rb
@@ -9,7 +9,7 @@ def execute(args)
       return if !SiteSetting.discourse_ai_enabled
       return if !SiteSetting.ai_translation_enabled
 
-      return if SiteSetting.experimental_content_localization_supported_locales.blank?
+      return if SiteSetting.content_localization_supported_locales.blank?
       return if SiteSetting.ai_translation_backfill_rate == 0
 
       Jobs.enqueue(:localize_topics, limit: SiteSetting.ai_translation_backfill_rate)
diff --git a/spec/jobs/regular/detect_translate_post_spec.rb b/spec/jobs/regular/detect_translate_post_spec.rb
@@ -12,7 +12,7 @@
       SiteSetting.public_send("ai_translation_model=", "custom:#{fake_llm.id}")
     end
     SiteSetting.ai_translation_enabled = true
-    SiteSetting.experimental_content_localization_supported_locales = locales.join("|")
+    SiteSetting.content_localization_supported_locales = locales.join("|")
   end
 
   it "does nothing when translator is disabled" do
@@ -55,7 +55,7 @@
   end
 
   it "does not translate when no target languages are configured" do
-    SiteSetting.experimental_content_localization_supported_locales = ""
+    SiteSetting.content_localization_supported_locales = ""
     DiscourseAi::Translation::PostLocaleDetector.expects(:detect_locale).with(post).returns("en")
     DiscourseAi::Translation::PostLocalizer.expects(:localize).never
 
diff --git a/spec/jobs/regular/detect_translate_topic_spec.rb b/spec/jobs/regular/detect_translate_topic_spec.rb
@@ -12,7 +12,7 @@
       SiteSetting.public_send("ai_translation_model=", "custom:#{fake_llm.id}")
     end
     SiteSetting.ai_translation_enabled = true
-    SiteSetting.experimental_content_localization_supported_locales = locales.join("|")
+    SiteSetting.content_localization_supported_locales = locales.join("|")
   end
 
   it "does nothing when translator is disabled" do
@@ -55,7 +55,7 @@
   end
 
   it "does not translate when no target languages are configured" do
-    SiteSetting.experimental_content_localization_supported_locales = ""
+    SiteSetting.content_localization_supported_locales = ""
     DiscourseAi::Translation::TopicLocaleDetector.expects(:detect_locale).with(topic).returns("en")
     DiscourseAi::Translation::TopicLocalizer.expects(:localize).never
 
diff --git a/spec/jobs/regular/localize_categories_spec.rb b/spec/jobs/regular/localize_categories_spec.rb
@@ -15,7 +15,7 @@ def localize_all_categories(*locales)
       SiteSetting.public_send("ai_translation_model=", "custom:#{fake_llm.id}")
     end
     SiteSetting.ai_translation_enabled = true
-    SiteSetting.experimental_content_localization_supported_locales = "pt|zh_CN"
+    SiteSetting.content_localization_supported_locales = "pt|zh_CN"
 
     Jobs.run_immediately!
   end
@@ -37,7 +37,7 @@ def localize_all_categories(*locales)
   end
 
   it "does nothing when no target languages are configured" do
-    SiteSetting.experimental_content_localization_supported_locales = ""
+    SiteSetting.content_localization_supported_locales = ""
 
     DiscourseAi::Translation::CategoryLocalizer.expects(:localize).never
 
diff --git a/spec/jobs/regular/localize_posts_spec.rb b/spec/jobs/regular/localize_posts_spec.rb
@@ -12,7 +12,7 @@
       SiteSetting.public_send("ai_translation_model=", "custom:#{fake_llm.id}")
     end
     SiteSetting.ai_translation_enabled = true
-    SiteSetting.experimental_content_localization_supported_locales = locales.join("|")
+    SiteSetting.content_localization_supported_locales = locales.join("|")
   end
 
   it "does nothing when translator is disabled" do
@@ -30,7 +30,7 @@
   end
 
   it "does nothing when no target languages are configured" do
-    SiteSetting.experimental_content_localization_supported_locales = ""
+    SiteSetting.content_localization_supported_locales = ""
     DiscourseAi::Translation::PostLocalizer.expects(:localize).never
 
     job.execute({})
@@ -136,7 +136,7 @@
     fab!(:group_pm_topic) { Fabricate(:group_private_message_topic, recipient_group: group) }
     fab!(:group_pm_post) { Fabricate(:post, topic: group_pm_topic, locale: "es") }
 
-    before { SiteSetting.experimental_content_localization_supported_locales = "ja" }
+    before { SiteSetting.content_localization_supported_locales = "ja" }
 
     context "when ai_translation_backfill_limit_to_public_content is true" do
       before { SiteSetting.ai_translation_backfill_limit_to_public_content = true }
@@ -187,7 +187,7 @@
 
     before do
       SiteSetting.ai_translation_backfill_max_age_days = 5
-      SiteSetting.experimental_content_localization_supported_locales = "ja"
+      SiteSetting.content_localization_supported_locales = "ja"
     end
 
     it "only processes posts within the age limit" do
diff --git a/spec/jobs/regular/localize_topics_spec.rb b/spec/jobs/regular/localize_topics_spec.rb
@@ -12,7 +12,7 @@
       SiteSetting.public_send("ai_translation_model=", "custom:#{fake_llm.id}")
     end
     SiteSetting.ai_translation_enabled = true
-    SiteSetting.experimental_content_localization_supported_locales = locales.join("|")
+    SiteSetting.content_localization_supported_locales = locales.join("|")
   end
 
   it "does nothing when translator is disabled" do
@@ -30,7 +30,7 @@
   end
 
   it "does nothing when no target languages are configured" do
-    SiteSetting.experimental_content_localization_supported_locales = ""
+    SiteSetting.content_localization_supported_locales = ""
     DiscourseAi::Translation::TopicLocalizer.expects(:localize).never
 
     job.execute({})
@@ -139,7 +139,7 @@
       Fabricate(:group_private_message_topic, recipient_group: Fabricate(:group), locale: "es")
     end
 
-    before { SiteSetting.experimental_content_localization_supported_locales = "ja" }
+    before { SiteSetting.content_localization_supported_locales = "ja" }
 
     context "when ai_translation_backfill_limit_to_public_content is true" do
       before { SiteSetting.ai_translation_backfill_limit_to_public_content = true }
diff --git a/spec/jobs/scheduled/post_localization_backfill_spec.rb b/spec/jobs/scheduled/post_localization_backfill_spec.rb
@@ -3,7 +3,7 @@
 describe Jobs::PostLocalizationBackfill do
   before do
     SiteSetting.ai_translation_backfill_rate = 100
-    SiteSetting.experimental_content_localization_supported_locales = "en"
+    SiteSetting.content_localization_supported_locales = "en"
     Fabricate(:fake_model).tap do |fake_llm|
       SiteSetting.public_send("ai_translation_model=", "custom:#{fake_llm.id}")
     end
@@ -29,7 +29,7 @@
   it "does not enqueue psot translation if backfill languages are not set" do
     SiteSetting.discourse_ai_enabled = true
     SiteSetting.ai_translation_enabled = true
-    SiteSetting.experimental_content_localization_supported_locales = ""
+    SiteSetting.content_localization_supported_locales = ""
 
     described_class.new.execute({})
 
