From d13a7b1a74c53b3e9f1fb3514de8c9736944f810 Mon Sep 17 00:00:00 2001
From: jordantgh <56029559+jordantgh@users.noreply.github.com>
Date: Wed, 20 Sep 2023 22:13:51 +0100
Subject: [PATCH] Fix setOptions() to properly handle modelOptions (#975)

For #974

- Adds an else to the check for this.modelOptions
- Allows the modelOptions to be updated when the model is already
  initialized
---
 api/app/clients/OpenAIClient.js | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/api/app/clients/OpenAIClient.js b/api/app/clients/OpenAIClient.js
index 9a6c61a3a4..866891ea38 100644
--- a/api/app/clients/OpenAIClient.js
+++ b/api/app/clients/OpenAIClient.js
@@ -59,6 +59,12 @@ class OpenAIClient extends BaseClient {
           typeof modelOptions.presence_penalty === 'undefined' ? 1 : modelOptions.presence_penalty,
         stop: modelOptions.stop,
       };
+    } else {
+      // Update the modelOptions if it already exists
+      this.modelOptions = {
+        ...this.modelOptions,
+        ...modelOptions,
+      };
     }
 
     if (process.env.OPENROUTER_API_KEY) {