Wrap single handler invocations with try/catch

atomist-skills · Feb 26, 2021 · f8877fe · f8877fe
1 parent 119f91a
commit f8877fe
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/lib/map.ts b/lib/map.ts
@@ -17,16 +17,21 @@
 import camelCase = require("lodash.camelcase");
 import { EventHandler } from "./handler/handler";
 import { warn } from "./log/console";
+import { prepareStatus } from "./message";
 import { toArray } from "./util";
 
 export function wrapEventHandler(eh: EventHandler): EventHandler {
 	return async ctx => {
 		if (Array.isArray(ctx.data)) {
 			const results = [];
 			for (const event of ctx.data) {
-				const result = await eh({ ...ctx, data: event });
-				if (result) {
-					results.push(result);
+				try {
+					const result = await eh({ ...ctx, data: event });
+					if (result) {
+						results.push(result);
+					}
+				} catch (e) {
+					results.push(prepareStatus(e, ctx));
 				}
 			}
 			return {