FIX: Seeded LLM setting considers string of ID instead of

keegangeorge · keegangeorge · commit 190a20c1c68d · 2025-01-08T12:58:25.000-08:00
string with `custom:` prefix.
diff --git a/app/controllers/discourse_ai/admin/ai_spam_controller.rb b/app/controllers/discourse_ai/admin/ai_spam_controller.rb
@@ -15,7 +15,7 @@ def update
           llm_model_id = updated_params[:llm_model_id] = allowed_params[:llm_model_id]
           if llm_model_id.to_i < 0 &&
                !SiteSetting.ai_spam_detection_model_allowed_seeded_models_map.include?(
-                 "custom:#{llm_model_id}",
+                 llm_model_id.to_s,
                )
             return(
               render_json_error(

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ def update`
`15`	`15`	`llm_model_id = updated_params[:llm_model_id] = allowed_params[:llm_model_id]`
`16`	`16`	`if llm_model_id.to_i < 0 &&`
`17`	`17`	`!SiteSetting.ai_spam_detection_model_allowed_seeded_models_map.include?(`
`18`		`- "custom:#{llm_model_id}",`
	`18`	`+ llm_model_id.to_s,`
`19`	`19`	`)`
`20`	`20`	`return(`
`21`	`21`	`render_json_error(`