Improve error handling when setting up and reading a new integration

server/adaptors/integrations/repository/integration_reader.ts

@@ -223,11 +223,24 @@ export class IntegrationReader {
 
       switch (asset.type) {
         case 'savedObjectBundle':
-          resultValue.push({
-            type: 'savedObjectBundle',
-            workflows: asset.workflows,
-            data: JSON.parse(serializedResult.value.data),
-          });
+          // Attempt to parse and process the integration data
+          try {
+            // Construct and push a savedObjectBundle with workflows and parsed data
+            resultValue.push({
+              type: 'savedObjectBundle',
+              workflows: asset.workflows,
+              data: JSON.parse(serializedResult.value.data),
+            });
+          } catch {
+            // Return error response if JSON parsing fails
+            return {
+              ok: false,
+              error: new Error(
+                `While parsing integration data for \`${serializedResult.value.name}\`:\n` +
+                  'The data field is not valid JSON.'
+              ),
+            };
+          }
           break;
         case 'query':
           resultValue.push({

server/adaptors/integrations/repository/utils.ts

@@ -30,7 +30,15 @@ export async function deepCheck(reader: IntegrationReader): Promise<Result<Integ
 
   // Deep checks not included in default config validation
   const assets = await reader.getAssets();
-  if (!assets.ok || assets.value.length === 0) {
+  // Check if there was an error retrieving assets
+  if (!assets.ok) {
+    return {
+      ok: false,
+      error: new Error(`Failed to process assets while loading: ${assets.error.message}`),
+    };
+  }
+  // Validate that at least one asset exists for the integration
+  if (assets.value.length === 0) {
     return { ok: false, error: new Error('An integration must have at least one asset') };
   }