fix(wallet): revert to returning complete tx data instead of just has…

…h in rpc wallet signing

aepps using sdk v11.0.1 and below expects whole tx object.

src/AeSdkWallet.ts

@@ -20,6 +20,7 @@ import {
   WalletInfo
 } from './utils/aepp-wallet-communication/rpc/types'
 import { EncodedData } from './utils/encoder'
+import jsonBig from './utils/json-big'
 
 type RpcClientWallet = RpcClient<AeppApi, WalletApi>
 
@@ -264,7 +265,9 @@ export default class AeSdkWallet extends AeSdk {
               return { signedTransaction: await this.signTransaction(tx, { onAccount }) }
             }
             try {
-              return { transactionHash: (await this.send(tx, { onAccount, verify: false })).hash }
+              return jsonBig.parse(jsonBig.stringify({
+                transactionHash: await this.send(tx, { onAccount, verify: false })
+              }))
             } catch (error) {
               const validation = await verifyTransaction(tx, this.api)
               if (validation.length > 0) throw new RpcInvalidTransactionError(validation)

src/utils/aepp-wallet-communication/rpc/types.ts

@@ -1,3 +1,4 @@
+import { send } from '../../../ae/spend'
 import { EncodedData } from '../../encoder'
 import { METHODS, SUBSCRIPTION_TYPES, WALLET_TYPE } from '../schema'
 
@@ -43,7 +44,14 @@ export interface WalletApi {
 
   [METHODS.sign]: ((
     p: { tx: EncodedData<'tx'>, onAccount: EncodedData<'ak'>, returnSigned: boolean }
-  ) => Promise<{ transactionHash?: EncodedData<'th'>, signedTransaction?: EncodedData<'tx'> }>)
+  ) => Promise<{
+    /**
+     * @deprecated this is not a hash at all, will be removed later at the same time
+     * as dropping ability to broadcast transaction by wallet
+     */
+    transactionHash?: Awaited<ReturnType<typeof send>>
+    signedTransaction?: EncodedData<'tx'>
+  }>)
 
   [METHODS.signMessage]: (
     p: { message: string, onAccount: EncodedData<'ak'> }