From 7c6dd3d750d2121c16fb72ab10de8c54a31c718e Mon Sep 17 00:00:00 2001
From: Shivam Mishra <scm.mymail@gmail.com>
Date: Thu, 2 May 2024 13:21:16 +0530
Subject: [PATCH] feat: add keydown handler in a try-catch-block

---
 .../mixins/keyboardEventListenerMixins.js     | 32 ++++++++++++-------
 1 file changed, 21 insertions(+), 11 deletions(-)

diff --git a/app/javascript/shared/mixins/keyboardEventListenerMixins.js b/app/javascript/shared/mixins/keyboardEventListenerMixins.js
index 9f6d43333..b1c3f0222 100644
--- a/app/javascript/shared/mixins/keyboardEventListenerMixins.js
+++ b/app/javascript/shared/mixins/keyboardEventListenerMixins.js
@@ -1,6 +1,7 @@
 import { isActiveElementTypeable, isEscape } from '../helpers/KeyboardHelpers';
 
 import { createKeybindingsHandler } from 'tinykeys';
+import * as Sentry from '@sentry/browser';
 
 // this is a store that stores the handler globally, and only gets reset on reload
 const taggedHandlers = [];
@@ -45,22 +46,31 @@ export default {
     },
     keydownWrapper(handler) {
       return e => {
-        const actionToPerform =
-          typeof handler === 'function' ? handler : handler.action;
-        const allowOnFocusedInput =
-          typeof handler === 'function' ? false : handler.allowOnFocusedInput;
+        const isFunction = typeof handler === 'function';
+        const actionToPerform = isFunction ? handler : handler.action;
+        const allowOnFocusedInput = isFunction
+          ? false
+          : handler.allowOnFocusedInput;
 
-        const isTypeable = isActiveElementTypeable(e);
+        try {
+          const isTypeable = isActiveElementTypeable(e);
 
-        if (isTypeable) {
-          if (isEscape(e)) {
-            e.target.blur();
+          if (isTypeable) {
+            if (isEscape(e)) e.target.blur();
+            if (!allowOnFocusedInput) return;
           }
 
-          if (!allowOnFocusedInput) return;
+          actionToPerform(e);
+        } catch {
+          // ignore errors
+          Sentry.captureException(e, {
+            context: {
+              component: this.$options?.name,
+              isFunction: isFunction,
+              allowOnFocusedInput: allowOnFocusedInput,
+            },
+          });
         }
-
-        actionToPerform(e);
       };
     },
   },