fix(ShortcutManager): return empty string for invalid keycode during …

…normalization [KHCP-11186] (#2137)

Datadog reported that in `normalizeKeyCode` method, `.replace` was being accessed on `undefined`. So added a check to early return with an empty string in case the parameter passed to the method is `undefined`, or an invalid string.

Extended explanation [here](https://konghq.atlassian.net/browse/KHCP-11186?focusedCommentId=129798)

src/utilities/ShortcutManager.ts

@@ -118,8 +118,8 @@ function triggerShortcuts<CommandKeyword extends string>(event: KeyboardEvent, k
 }
 
 function normalizeKeyCode(code: string): string {
-  // Returns relevant modifier keys as the empty string which is going to be filtered out.
-  if (MODIFIER_KEY_CODES.includes(code)) {
+  // If keycode is a relevant modifier key or an invalid string, return empty string.
+  if (!code || MODIFIER_KEY_CODES.includes(code)) {
     return ''
   }