add ai_translation_max_post_length

Author: nattsw

config/locales/server.en.yml

@@ -118,6 +118,12 @@ en:
     ai_discord_allowed_guilds: "Discord guilds (servers) where the bot is allowed to search"
     ai_bot_enable_dedicated_ux: "Allow for full screen bot interface, instead of a PM"
 
+    ai_translation_enabled: "Enables the AI translation feature"
+    ai_translation_model: "The model to use for translation. This model must support translation. Personas can override this setting."
+    ai_translation_backfill_limit_to_public_content: "When enabled, only content in public categories will be translated. When disabled, content in group PMs and private categories will also be sent for translation."
+    ai_translation_max_post_length: "The maximum length of a post to be translated. Posts longer than this will not be translated."
+    ai_translation_backfill_max_age_days: "The maximum age of a post and topic to be translated. Posts and topics older than this will not be translated."
+
   reviewables:
     reasons:
       flagged_by_toxicity: The AI plugin flagged this after classifying it as toxic.

config/settings.yml

@@ -491,6 +491,10 @@ discourse_ai:
     default: true
     client: false
     area: "ai-features/translation"
+  ai_translation_max_post_length:
+    default: 1000
+    client: false
+    area: "ai-features/translation"
   ai_translation_backfill_max_age_days:
     default: 5
     client: false

lib/translation/post_localizer.rb

@@ -4,7 +4,11 @@ module DiscourseAi
   module Translation
     class PostLocalizer
       def self.localize(post, target_locale = I18n.locale)
-        return if post.blank? || target_locale.blank? || post.locale == target_locale.to_s
+        if post.blank? || target_locale.blank? || post.locale == target_locale.to_s ||
+             post.raw.blank?
+          return
+        end
+        return if post.raw.length > SiteSetting.ai_translation_max_post_length
         target_locale = target_locale.to_s.sub("-", "_")
 
         translated_raw =

spec/lib/translation/post_localizer_spec.rb

@@ -18,7 +18,7 @@ def post_raw_translator_stub(opts)
       allow(mock).to receive(:translate).and_return(opts[:translated])
     end
 
-    it "returns nil if post is blank" do
+    it "returns nil if post does not exist" do
       expect(described_class.localize(nil, "ja")).to eq(nil)
     end
 
@@ -33,6 +33,19 @@ def post_raw_translator_stub(opts)
       expect(described_class.localize(post, "en")).to eq(nil)
     end
 
+    it "returns nil if post raw is blank" do
+      post.raw = ""
+
+      expect(described_class.localize(post, "ja")).to eq(nil)
+    end
+
+    it "returns nil if post raw is too long" do
+      SiteSetting.ai_translation_max_post_length = 10
+      post.raw = "This is a very long post that exceeds the limit."
+
+      expect(described_class.localize(post, "ja")).to eq(nil)
+    end
+
     it "translates with post and locale" do
       post_raw_translator_stub({ text: post.raw, target_locale: "ja", translated: translated_raw })