FIX: legacy reasoning models not working, missing provider params

1. Legacy reasoning models (o1-preview / o1-mini) do not support developer or system messages, do not use them.
2. LLM editor form not showing all provider params due to missing remap

Author: SamSaffron

assets/javascripts/discourse/components/ai-llm-editor-form.gjs

@@ -61,7 +61,7 @@ export default class AiLlmEditorForm extends Component {
       provider: model.provider,
       enabled_chat_bot: model.enabled_chat_bot,
       vision_enabled: model.vision_enabled,
-      provider_params: model.provider_params,
+      provider_params: this.computeProviderParams(model.provider),
       llm_quotas: model.llm_quotas,
     };
   }

lib/completions/dialects/chat_gpt.rb

@@ -55,12 +55,17 @@ def tools_dialect
           end
         end
 
-        # developer messages are preferred on reasoning models
+        # developer messages are preferred on recent reasoning models
         def supports_developer_messages?
-          llm_model.provider == "open_ai" &&
+          !legacy_reasoning_model? && llm_model.provider == "open_ai" &&
             (llm_model.name.start_with?("o1") || llm_model.name.start_with?("o3"))
         end
 
+        def legacy_reasoning_model?
+          llm_model.provider == "open_ai" &&
+            (llm_model.name.start_with?("o1-preview") || llm_model.name.start_with?("o1-mini"))
+        end
+
         def system_msg(msg)
           content = msg[:content]
           if disable_native_tools? && tools_dialect.instructions.present?
@@ -69,6 +74,8 @@ def system_msg(msg)
 
           if supports_developer_messages?
             { role: "developer", content: content }
+          elsif legacy_reasoning_model?
+            { role: "user", content: content }
           else
             { role: "system", content: content }
           end