fix: run all save state queries in insight processor, not only the active one

src/readers/insightProcessor.ts

@@ -84,70 +84,69 @@ export const insightProcessor = async () => {
     if (insight.userId == null) return;
     const ss = await ums.getUserSaveState(insight.userId, insight.saveStateId);
 
-    const activeQuery = ss.queries.activeQueryId;
-    if (activeQuery == null || activeQuery == -1) {
-      log.error('No active query found in SaveState:', ss);
-      return;
-    }
+    const queries = ss.queries.openQueryArray;
     const visualizations = ss.visualizations.openVisualizationArray;
-    const visualQuery = ss.queries.openQueryArray[activeQuery].graph;
-    const queryBuilderSettings = ss.queries.openQueryArray[activeQuery].settings;
-    const convertedQuery = Query2BackendQuery(ss.id, visualQuery, queryBuilderSettings, []);
-    const query = query2Cypher(convertedQuery);
-    if (query == null) return;
-    try {
-      const result = await queryService(ss.dbConnections[0], query);
-
-      insight.status = false;
-
-      if (insight.alarmMode === 'always') {
-        insight.status = true;
-      } else if (insight.alarmMode === 'diff') {
-        insight = await diffCheck(insight, ss, result);
-      } else if (insight.alarmMode === 'conditional' && insight.conditionsCheck && insight.conditionsCheck.length > 0) {
-        insight = statCheck(insight, result);
-      }
-
-      if (insight.userId == null) return; // fixes ts but never is the case
-      await ums.updateInsight(insight.userId, insight.id, insight);
-
-      if (insight.status || message.force) {
-        if (insight.status) log.debug('Insight passed the check');
-        if (message.force) log.debug('Forced insight processing');
 
-        editor.read(async () => {
-          const cleanUpDom = setUpDom();
-          let html = $generateHtmlFromNodes(editor);
-          cleanUpDom();
-
-          html = await populateTemplate(html, result, visualizations);
-
-          for (const recipient of insight.recipients) {
-            if (mail == null) {
-              log.warn('Mail is not configured. Insight processor will be disabled');
-              return;
+    for (const queryIndex in queries) {
+      const visualQuery = ss.queries.openQueryArray[queryIndex].graph;
+      const queryBuilderSettings = ss.queries.openQueryArray[queryIndex].settings;
+      const convertedQuery = Query2BackendQuery(ss.id, visualQuery, queryBuilderSettings, []);
+      const query = query2Cypher(convertedQuery);
+      if (query == null) return;
+      try {
+        const result = await queryService(ss.dbConnections[0], query);
+
+        insight.status = false;
+
+        if (insight.alarmMode === 'always') {
+          insight.status = true;
+        } else if (insight.alarmMode === 'diff') {
+          insight = await diffCheck(insight, ss, result);
+        } else if (insight.alarmMode === 'conditional' && insight.conditionsCheck && insight.conditionsCheck.length > 0) {
+          insight = statCheck(insight, result);
+        }
+
+        if (insight.userId == null) return; // fixes ts but never is the case
+        await ums.updateInsight(insight.userId, insight.id, insight);
+
+        if (insight.status || message.force) {
+          if (insight.status) log.debug('Insight passed the check');
+          if (message.force) log.debug('Forced insight processing');
+
+          editor.read(async () => {
+            const cleanUpDom = setUpDom();
+            let html = $generateHtmlFromNodes(editor);
+            cleanUpDom();
+
+            html = await populateTemplate(html, result, visualizations);
+
+            for (const recipient of insight.recipients) {
+              if (mail == null) {
+                log.warn('Mail is not configured. Insight processor will be disabled');
+                return;
+              }
+
+              if (DEBUG_EMAIL) {
+                log.warn('DEBUG: Would have sent mail to', recipient);
+                continue;
+              }
+
+              log.debug('Sending mail to', recipient);
+              await mail.sendMail({
+                to: recipient,
+                from: SMTP_USER,
+                subject: `GraphPolaris report: ${insight.name}`,
+                html: html,
+              });
+              log.info('Mail sent to ', recipient);
             }
-
-            if (DEBUG_EMAIL) {
-              log.warn('DEBUG: Would have sent mail to', recipient);
-              continue;
-            }
-
-            log.debug('Sending mail to', recipient);
-            await mail.sendMail({
-              to: recipient,
-              from: SMTP_USER,
-              subject: `GraphPolaris report: ${insight.name}`,
-              html: html,
-            });
-            log.info('Mail sent to ', recipient);
-          }
-        });
-      } else {
-        log.debug('WARN: Insight did not pass the check');
+          });
+        } else {
+          log.debug('WARN: Insight did not pass the check');
+        }
+      } catch (err) {
+        log.error('Error processing insight', err);
       }
-    } catch (err) {
-      log.error('Error processing insight', err);
     }
   });
 };