chore: optimize error (#624)

Author: ByteZhang1024

packages/core/src/api/cosmos/CosmosSignTransaction.ts

@@ -1,10 +1,13 @@
-import { CosmosSignTx as HardwareCosmosSignTx } from '@onekeyfe/hd-transport';
+import { ERRORS, HardwareErrorCode } from '@onekeyfe/hd-shared';
+
 import { serializedPath, validatePath } from '../helpers/pathUtils';
 import { BaseMethod } from '../BaseMethod';
 import { validateParams } from '../helpers/paramsValidator';
-import { CosmosSignTransactionParams } from '../../types';
 import { formatAnyHex } from '../helpers/hexUtils';
 
+import type { CosmosSignTx as HardwareCosmosSignTx } from '@onekeyfe/hd-transport';
+import type { CosmosSignTransactionParams } from '../../types';
+
 export default class CosmosSignTransaction extends BaseMethod<HardwareCosmosSignTx> {
   hasBundle = false;
 
@@ -40,15 +43,26 @@ export default class CosmosSignTransaction extends BaseMethod<HardwareCosmosSign
   }
 
   async run() {
-    const res = await this.device.commands.typedCall('CosmosSignTx', 'CosmosSignedTx', {
-      ...this.params,
-    });
+    try {
+      const res = await this.device.commands.typedCall('CosmosSignTx', 'CosmosSignedTx', {
+        ...this.params,
+      });
 
-    const { signature } = res.message;
+      const { signature } = res.message;
 
-    return {
-      path: serializedPath(this.params.address_n),
-      signature,
-    };
+      return {
+        path: serializedPath(this.params.address_n),
+        signature,
+      };
+    } catch (error) {
+      const { message } = error;
+      if (
+        message.includes('Failure_DataError,Json parse failed') ||
+        message.includes('Failure_DataError,Invalid message')
+      ) {
+        throw ERRORS.TypedError(HardwareErrorCode.CosmosInvalidJsonMessage, message);
+      }
+      throw error;
+    }
   }
 }

packages/core/src/device/DeviceCommands.ts

@@ -441,7 +441,7 @@ export class DeviceCommands {
         ERRORS.TypedError(
           HardwareErrorCode.RuntimeError,
           // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
-          `${(code as any) || 'Failure_UnknownCode'},${message || 'Failure_UnknownMessage'}`
+          `${(code as any) || 'Failure_UnknownCode'},${message || 'no error message'}`
         )
       );
     }

packages/shared/src/HardwareError.ts

@@ -455,11 +455,14 @@ export const HardwareErrorCode = {
   BridgeNeedsPermission: 821,
 
   CallQueueActionCancelled: 822,
-
   /**
    * Firmware downgrade not allowed
    */
   FirmwareDowngradeNotAllowed: 823,
+  /**
+   * Cosmos invalid JSON message, Only support Amino msgs.
+   */
+  CosmosInvalidJsonMessage: 824,
 
   /**
    * Lowlevel transport connect error
@@ -606,6 +609,8 @@ export const HardwareErrorCodeMessage: HardwareErrorCodeMessageMapping = {
   [HardwareErrorCode.FirmwareVerificationFailed]: 'Firmware verification failed',
   [HardwareErrorCode.BridgeNeedsPermission]: 'Bridge needs permission',
   [HardwareErrorCode.FirmwareDowngradeNotAllowed]: 'Firmware downgrade not allowed',
+  [HardwareErrorCode.CosmosInvalidJsonMessage]:
+    'Cosmos invalid JSON message, Only support Amino msgs.',
 
   /**
    * Lowlevel transport