Do not fail fast on run checks endpoint when a single check fails and return error message alongside any results that completed

Signed-off-by: sblausten <sam@roadie.io>

.changeset/proud-birds-worry.md

@@ -0,0 +1,5 @@
+---
+'@backstage/plugin-tech-insights-backend': patch
+---
+
+Complete check results run when a single check errors so that we don't block other checks from working due to an error in a single check

plugins/tech-insights-backend/src/service/router.ts

@@ -103,16 +103,30 @@ export async function createRouter<
         checks,
         entities,
       }: { checks: string[]; entities: CompoundEntityRef[] } = req.body;
+      let error = undefined;
       const tasks = entities.map(async entity => {
         const entityTriplet =
           typeof entity === 'string' ? entity : stringifyEntityRef(entity);
-        const results = await factChecker.runChecks(entityTriplet, checks);
-        return {
-          entity: entityTriplet,
-          results,
-        };
+        try {
+          const results = await factChecker.runChecks(entityTriplet, checks);
+          return {
+            entity: entityTriplet,
+            results,
+          };
+        } catch (e: any) {
+          const errorMessage = `Failed to run check for entity ${entityTriplet} due to error: ${e.message}`;
+          logger.error(errorMessage);
+          error = errorMessage;
+          return {
+            entity: entityTriplet,
+            results: [],
+          };
+        }
       });
       const results = await Promise.all(tasks);
+      if (error) {
+        return res.status(500).send({ error, results });
+      }
       return res.json(results);
     });
   } else {