Check for value of encodedAttribute and better payload safety checks (#978)

* Check for value of encodedAttribute and better payload safety checks

* Remove if statement

Author: Alex-Tideman

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@temporalio/ui",
-  "version": "2.1.89",
+  "version": "2.1.90",
   "type": "module",
   "description": "Temporal.io UI",
   "keywords": [

src/lib/utilities/decode-payload.ts

@@ -34,12 +34,13 @@ export function decodePayload(
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
 ): Payload | Record<any, any> | string {
   const encoding = atob(String(payload?.metadata?.encoding ?? ''));
+
   // Help users out with an english encoding
   (payload.metadata.encodingDecoded as unknown as string) = encoding;
 
-  if (encoding.startsWith('json/')) {
+  if (encoding?.startsWith('json/')) {
     try {
-      return parseWithBigInt(atob(String(payload.data)));
+      return parseWithBigInt(atob(String(payload?.data ?? '')));
     } catch (_e) {
       // Couldn't correctly decode this just let the user deal with the data as is
     }
@@ -119,7 +120,7 @@ export const decodeAllPotentialPayloadsWithCodec = async (
           JSONPayload = payloads.map(decodePayload);
         }
         anyAttributes[key] = JSONPayload;
-      } else if (key === 'encodedAttributes') {
+      } else if (key === 'encodedAttributes' && anyAttributes[key]) {
         // Can expand if more fields have single payload
         // eslint-disable-next-line @typescript-eslint/no-explicit-any
         let JSONPayload: string | Payload | Record<any, any>;
@@ -175,7 +176,7 @@ export const decodeAllPotentialPayloadsWithWebsockets = async (
           JSONPayload = payloads.map(decodePayload);
         }
         anyAttributes[key] = JSONPayload;
-      } else if (key === 'encodedAttributes') {
+      } else if (key === 'encodedAttributes' && anyAttributes[key]) {
         // Can expand if more fields have single payload
         // eslint-disable-next-line @typescript-eslint/no-explicit-any
         let JSONPayload: string | Payload | Record<any, any>;