fix: log errors properly

Author: didinele

src/app.ts

@@ -40,8 +40,14 @@ export function getApp(env: Env) {
 		},
 	});
 
-	app.webhooks.on('issue_comment.created', async (data) => {
-		logger.debug('in comment create');
+	// eslint-disable-next-line promise/prefer-await-to-callbacks
+	app.webhooks.onError((err) => {
+		logger.error({ err }, 'Webhook error');
+	});
+
+	// TODO: Swap to create for prod
+	app.webhooks.on('issue_comment.edited', async (data) => {
+		logger.debug('in comment edit');
 
 		if (!data.payload.comment.body.startsWith('pack this')) {
 			logger.debug(`Comment does not start with 'pack this': ${data.payload.comment.body}`);

src/index.ts

@@ -44,7 +44,7 @@ const server = {
 				headers: { 'content-type': 'application/json' },
 			});
 		} catch (error) {
-			logger.error({ error }, 'Error processing webhook');
+			logger.error({ err: error }, 'Error processing webhook');
 			return new Response('internal server error', {
 				status: 500,
 				headers: { 'content-type': 'application/json' },