fix: add defensive programming for malformed settings files

Signed-off-by: tylerslaton <[email protected]>
gptscript-ai · Sep 10, 2024 · 4a406cf · 4a406cf
1 parent cead33b
commit 4a406cf
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/actions/appSettings.tsx b/actions/appSettings.tsx
@@ -28,7 +28,11 @@ export async function getAppSettings() {
   const location = process.env.GPTSCRIPT_SETTINGS_FILE;
   if (fs.existsSync(location)) {
     const AppSettings = fs.readFileSync(location, 'utf-8');
-    return JSON.parse(AppSettings) as AppSettings;
+    try {
+      return JSON.parse(AppSettings) as AppSettings;
+    } catch {
+      console.error('Malformed settings file, using default settings...');
+    }
   }
   return defaultAppSettings;
 }

diff --git a/server/app.mjs b/server/app.mjs
@@ -145,7 +145,11 @@ const mount = async (
     },
   };
   if (existsSync(settingsLocation)) {
-    settings = JSON.parse(fs.readFileSync(settingsLocation, 'utf8'));
+    try {
+      settings = JSON.parse(fs.readFileSync(settingsLocation, 'utf8'));
+    } catch {
+      console.error('Malformed settings file, using default settings...');
+    }
   }
 
   let script;