@aeternity/aepp-sdk 10.0.0-beta.1 → 11.0.1

package/src/tx/tx.js

@@ -26,12 +26,21 @@ import ChainNode from '../chain/node'
 import Tx from './'
 import { buildTx, calculateFee, unpackTx } from './builder'
 import { ABI_VERSIONS, MIN_GAS_PRICE, PROTOCOL_VM_ABI, TX_TYPE, TX_TTL } from './builder/schema'
-import { buildContractId } from './builder/helpers'
+import { buildContractId, encode } from './builder/helpers'
 import { TxObject } from './tx-object'
+import {
+  ArgumentError,
+  UnsupportedABIversionError,
+  UnsupportedVMversionError,
+  UnsupportedProtocolError,
+  UnknownTxError
+} from '../utils/errors'
 
 async function spendTx ({ senderId, recipientId, amount, payload = '' }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.spend, { senderId, ...arguments[0], payload })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.spend, { senderId, ...arguments[0], payload }
+  )
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
     ? {
@@ -63,7 +72,9 @@ async function spendTx ({ senderId, recipientId, amount, payload = '' }) {
 
 async function namePreclaimTx ({ accountId, commitmentId }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.namePreClaim, { senderId: accountId, ...arguments[0] })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.namePreClaim, { senderId: accountId, ...arguments[0] }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -80,7 +91,9 @@ async function namePreclaimTx ({ accountId, commitmentId }) {
 
 async function nameClaimTx ({ accountId, name, nameSalt, vsn = 2 }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.nameClaim, { senderId: accountId, ...arguments[0], vsn })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.nameClaim, { senderId: accountId, ...arguments[0], vsn }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -90,14 +103,18 @@ async function nameClaimTx ({ accountId, name, nameSalt, vsn = 2 }) {
           type: TX_TYPE.nameClaim
         }).encodedTx
       }
-    : await this.api.postNameClaim({ ...arguments[0], nonce, ttl, fee: parseInt(fee) })
+    : await this.api.postNameClaim({
+      ...arguments[0], nonce, ttl, fee: parseInt(fee), name: encode(name, 'nm')
+    })
 
   return tx
 }
 
 async function nameTransferTx ({ accountId, nameId, recipientId }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.nameTransfer, { senderId: accountId, ...arguments[0] })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.nameTransfer, { senderId: accountId, ...arguments[0] }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -107,14 +124,18 @@ async function nameTransferTx ({ accountId, nameId, recipientId }) {
           type: TX_TYPE.nameTransfer
         }).encodedTx
       }
-    : await this.api.postNameTransfer({ ...arguments[0], recipientId, nonce, ttl, fee: parseInt(fee) })
+    : await this.api.postNameTransfer({
+      ...arguments[0], recipientId, nonce, ttl, fee: parseInt(fee)
+    })
 
   return tx
 }
 
 async function nameUpdateTx ({ accountId, nameId, nameTtl, pointers, clientTtl }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.nameUpdate, { senderId: accountId, ...arguments[0] })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.nameUpdate, { senderId: accountId, ...arguments[0] }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -131,7 +152,9 @@ async function nameUpdateTx ({ accountId, nameId, nameTtl, pointers, clientTtl }
 
 async function nameRevokeTx ({ accountId, nameId }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.nameRevoke, { senderId: accountId, ...arguments[0] })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.nameRevoke, { senderId: accountId, ...arguments[0] }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -146,11 +169,15 @@ async function nameRevokeTx ({ accountId, nameId }) {
   return tx
 }
 
-async function contractCreateTx ({ ownerId, code, vmVersion, abiVersion, deposit, amount, gas, gasPrice = MIN_GAS_PRICE, callData }) {
+async function contractCreateTx ({
+  ownerId, code, vmVersion, abiVersion, amount, gas, gasPrice = MIN_GAS_PRICE, callData
+}) {
   // Get VM_ABI version
   const ctVersion = this.getVmVersion(TX_TYPE.contractCreate, arguments[0])
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.contractCreate, { senderId: ownerId, ...arguments[0], ctVersion, gasPrice })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.contractCreate, { senderId: ownerId, ...arguments[0], ctVersion, gasPrice }
+  )
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   return this.nativeMode
     ? {
@@ -160,13 +187,27 @@ async function contractCreateTx ({ ownerId, code, vmVersion, abiVersion, deposit
         }).encodedTx,
         contractId: buildContractId(ownerId, nonce)
       }
-    : this.api.postContractCreate({ ...arguments[0], nonce, ttl, fee: parseInt(fee), gas: parseInt(gas), gasPrice, vmVersion: ctVersion.vmVersion, abiVersion: ctVersion.abiVersion })
+    : this.api.postContractCreate({
+      ...arguments[0],
+      nonce,
+      ttl,
+      fee: parseInt(fee),
+      gas: parseInt(gas),
+      gasPrice,
+      vmVersion: ctVersion.vmVersion,
+      abiVersion: ctVersion.abiVersion
+    })
 }
 
-async function contractCallTx ({ callerId, contractId, abiVersion, amount, gas, gasPrice = MIN_GAS_PRICE, callData }) {
+async function contractCallTx ({
+  callerId, contractId, abiVersion, amount, gas, gasPrice = MIN_GAS_PRICE, callData
+}) {
   const ctVersion = this.getVmVersion(TX_TYPE.contractCall, arguments[0])
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.contractCall, { senderId: callerId, ...arguments[0], gasPrice, abiVersion: ctVersion.abiVersion })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.contractCall,
+    { senderId: callerId, ...arguments[0], gasPrice, abiVersion: ctVersion.abiVersion }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -189,10 +230,14 @@ async function contractCallTx ({ callerId, contractId, abiVersion, amount, gas,
   return tx
 }
 
-async function oracleRegisterTx ({ accountId, queryFormat, responseFormat, queryFee, oracleTtl, abiVersion = ABI_VERSIONS.NO_ABI }) {
+async function oracleRegisterTx ({
+  accountId, queryFormat, responseFormat, queryFee, oracleTtl, abiVersion = ABI_VERSIONS.NO_ABI
+}) {
   // const { abiVersion: abi } = this.getVmVersion(TX_TYPE.oracleRegister, arguments[0])
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.oracleRegister, { senderId: accountId, ...arguments[0], abiVersion })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.oracleRegister, { senderId: accountId, ...arguments[0], abiVersion }
+  )
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
     ? {
@@ -228,7 +273,9 @@ async function oracleRegisterTx ({ accountId, queryFormat, responseFormat, query
 
 async function oracleExtendTx ({ oracleId, callerId, oracleTtl }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.oracleExtend, { senderId: callerId, ...arguments[0] })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.oracleExtend, { senderId: callerId, ...arguments[0] }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -245,7 +292,9 @@ async function oracleExtendTx ({ oracleId, callerId, oracleTtl }) {
 
 async function oraclePostQueryTx ({ oracleId, responseTtl, query, queryTtl, queryFee, senderId }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.oracleQuery, { senderId, ...arguments[0] })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.oracleQuery, { senderId, ...arguments[0] }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -272,7 +321,9 @@ async function oraclePostQueryTx ({ oracleId, responseTtl, query, queryTtl, quer
 
 async function oracleRespondTx ({ oracleId, callerId, responseTtl, queryId, response }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.oracleResponse, { senderId: callerId, ...arguments[0] })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.oracleResponse, { senderId: callerId, ...arguments[0] }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -282,13 +333,17 @@ async function oracleRespondTx ({ oracleId, callerId, responseTtl, queryId, resp
           type: TX_TYPE.oracleResponse
         }).encodedTx
       }
-    : await this.api.postOracleRespond({ oracleId, responseTtl, queryId, response, fee: parseInt(fee), ttl, nonce })
+    : await this.api.postOracleRespond({
+      oracleId, responseTtl, queryId, response, fee: parseInt(fee), ttl, nonce
+    })
   return tx
 }
 
 async function channelCloseSoloTx ({ channelId, fromId, payload, poi }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.channelCloseSolo, { senderId: fromId, ...arguments[0], payload })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.channelCloseSolo, { senderId: fromId, ...arguments[0], payload }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -318,7 +373,9 @@ async function channelCloseSoloTx ({ channelId, fromId, payload, poi }) {
 
 async function channelSlashTx ({ channelId, fromId, payload, poi }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.channelSlash, { senderId: fromId, ...arguments[0], payload })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.channelSlash, { senderId: fromId, ...arguments[0], payload }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -348,7 +405,9 @@ async function channelSlashTx ({ channelId, fromId, payload, poi }) {
 
 async function channelSettleTx ({ channelId, fromId, initiatorAmountFinal, responderAmountFinal }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.channelSettle, { senderId: fromId, ...arguments[0] })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.channelSettle, { senderId: fromId, ...arguments[0] }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -378,7 +437,9 @@ async function channelSettleTx ({ channelId, fromId, initiatorAmountFinal, respo
 
 async function channelSnapshotSoloTx ({ channelId, fromId, payload }) {
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.channelSnapshotSolo, { senderId: fromId, ...arguments[0], payload })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.channelSnapshotSolo, { senderId: fromId, ...arguments[0], payload }
+  )
 
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   const { tx } = this.nativeMode
@@ -404,11 +465,15 @@ async function channelSnapshotSoloTx ({ channelId, fromId, payload }) {
   return tx
 }
 
-async function gaAttachTx ({ ownerId, code, vmVersion, abiVersion, authFun, gas, gasPrice = MIN_GAS_PRICE, callData }) {
+async function gaAttachTx ({
+  ownerId, code, vmVersion, abiVersion, authFun, gas, gasPrice = MIN_GAS_PRICE, callData
+}) {
   // Get VM_ABI version
   const ctVersion = this.getVmVersion(TX_TYPE.contractCreate, arguments[0])
   // Calculate fee, get absolute ttl (ttl + height), get account nonce
-  const { fee, ttl, nonce } = await this.prepareTxParams(TX_TYPE.gaAttach, { senderId: ownerId, ...arguments[0], ctVersion, gasPrice })
+  const { fee, ttl, nonce } = await this.prepareTxParams(
+    TX_TYPE.gaAttach, { senderId: ownerId, ...arguments[0], ctVersion, gasPrice }
+  )
   // Build transaction using sdk (if nativeMode) or build on `AETERNITY NODE` side
   return {
     tx: TxObject({
@@ -437,17 +502,17 @@ async function payingForTx ({ tx, payerId, ...args }) {
 function getVmVersion (txType, { vmVersion, abiVersion } = {}) {
   const { consensusProtocolVersion } = this.getNodeInfo()
   const supportedProtocol = PROTOCOL_VM_ABI[consensusProtocolVersion]
-  if (!supportedProtocol) throw new Error('Not supported consensus protocol version')
+  if (!supportedProtocol) throw new UnsupportedProtocolError('Not supported consensus protocol version')
   const protocolForTX = supportedProtocol[txType]
-  if (!protocolForTX) throw new Error('Not supported tx type')
+  if (!protocolForTX) throw new UnknownTxError('Not supported tx type')
 
   abiVersion = abiVersion || protocolForTX.abiVersion[0]
   vmVersion = vmVersion || protocolForTX.vmVersion[0]
   if (!protocolForTX.vmVersion.includes(vmVersion)) {
-    throw new Error(`VM VERSION ${vmVersion} do not support by this node. Supported: [${protocolForTX.vmVersion}]`)
+    throw new UnsupportedVMversionError(`VM VERSION ${vmVersion} do not support by this node. Supported: [${protocolForTX.vmVersion}]`)
   }
   if (!protocolForTX.abiVersion.includes(abiVersion)) {
-    throw new Error(`ABI VERSION ${abiVersion} do not support by this node. Supported: [${protocolForTX.abiVersion}]`)
+    throw new UnsupportedABIversionError(`ABI VERSION ${abiVersion} do not support by this node. Supported: [${protocolForTX.abiVersion}]`)
   }
 
   return { vmVersion, abiVersion }
@@ -462,7 +527,7 @@ function getVmVersion (txType, { vmVersion, abiVersion } = {}) {
  */
 async function calculateTtl (ttl = TX_TTL, relative = true) {
   if (ttl === 0) return 0
-  if (ttl < 0) throw new Error('ttl must be greater than 0')
+  if (ttl < 0) throw new ArgumentError('ttl', 'greater or equal to 0', ttl)
 
   if (relative) {
     const { height } = await this.api.getCurrentKeyBlock()
@@ -480,7 +545,8 @@ async function calculateTtl (ttl = TX_TTL, relative = true) {
  */
 async function getAccountNonce (accountId, nonce) {
   if (nonce) return nonce
-  const { nonce: accountNonce } = await this.api.getAccountByPubkey(accountId).catch(() => ({ nonce: 0 }))
+  const { nonce: accountNonce } = await this.api.getAccountByPubkey(accountId)
+    .catch(() => ({ nonce: 0 }))
   return accountNonce + 1
 }
 
@@ -491,10 +557,19 @@ async function getAccountNonce (accountId, nonce) {
  * @param {Object} params Object which contains all tx data
  * @return {Object} { ttl, nonce, fee } Object with account nonce, absolute ttl and transaction fee
  */
-async function prepareTxParams (txType, { senderId, nonce: n, ttl: t, fee: f, gas, absoluteTtl, vsn, strategy }) {
-  n = n || (await this.api.getAccountNextNonce(senderId, { strategy }).catch(e => ({ nextNonce: 1 }))).nextNonce
+async function prepareTxParams (
+  txType,
+  { senderId, nonce: n, ttl: t, fee: f, gas, absoluteTtl, vsn, strategy }
+) {
+  n = n || (
+    await this.api.getAccountNextNonce(senderId, { strategy }).catch(e => ({ nextNonce: 1 }))
+  ).nextNonce
   const ttl = await calculateTtl.call(this, t, !absoluteTtl)
-  const fee = calculateFee(f, txType, { showWarning: this.showWarning, gas, params: { ...arguments[1], nonce: n, ttl }, vsn })
+  const fee = calculateFee(
+    f,
+    txType,
+    { showWarning: this.showWarning, gas, params: { ...arguments[1], nonce: n, ttl }, vsn }
+  )
   return { fee, ttl, nonce: n }
 }
 

package/src/tx/validator.js

@@ -4,6 +4,7 @@ import { encode, decode } from './builder/helpers'
 import BigNumber from 'bignumber.js'
 import { MIN_GAS_PRICE, PROTOCOL_VM_ABI, TX_TYPE } from './builder/schema'
 import { calculateFee, unpackTx } from './builder'
+import { UnsupportedProtocolError } from '../utils/errors'
 
 /**
  * Transaction validator
@@ -13,7 +14,7 @@ import { calculateFee, unpackTx } from './builder'
  */
 
 const validators = [
-  async ({ encodedTx, signatures }, { account, node, parentTxTypes }) => {
+  ({ encodedTx, signatures }, { account, node, parentTxTypes }) => {
     if ((encodedTx ?? signatures) === undefined) return []
     if (signatures.length !== 1) return [] // TODO: Support multisignature?
     const prefix = Buffer.from([
@@ -32,7 +33,7 @@ const validators = [
       checkedKeys: ['encodedTx', 'signatures']
     }]
   },
-  async ({ encodedTx, tx }, { node, parentTxTypes, txType }) => {
+  ({ encodedTx, tx }, { node, parentTxTypes, txType }) => {
     if ((encodedTx ?? tx) === undefined) return []
     return verifyTransaction(
       encode((encodedTx ?? tx).rlpEncoded, 'tx'),
@@ -104,7 +105,7 @@ const validators = [
   ({ ctVersion, abiVersion }, { txType, node }) => {
     const { consensusProtocolVersion } = node.getNodeInfo()
     const protocol = PROTOCOL_VM_ABI[consensusProtocolVersion]
-    if (!protocol) throw new Error(`Unsupported protocol: ${consensusProtocolVersion}`)
+    if (!protocol) throw new UnsupportedProtocolError(`Unsupported protocol: ${consensusProtocolVersion}`)
     // If not contract create tx
     if (!ctVersion) ctVersion = { abiVersion }
     const txProtocol = protocol[txType]
@@ -117,6 +118,25 @@ const validators = [
       }]
     }
     return []
+  },
+  async ({ contractId }, { txType, node }) => {
+    if (TX_TYPE.contractCall !== txType) return []
+    try {
+      const { active } = await node.api.getContract(contractId)
+      if (active) return []
+      return [{
+        message: `Contract ${contractId} is not active`,
+        key: 'ContractNotActive',
+        checkedKeys: ['contractId']
+      }]
+    } catch (error) {
+      if (!error.response?.body?.reason) throw error
+      return [{
+        message: error.response.body.reason,
+        key: 'ContractNotFound',
+        checkedKeys: ['contractId']
+      }]
+    }
   }
 ]
 
@@ -146,13 +166,13 @@ export default async function verifyTransaction (transaction, node, parentTxType
 
   const address = getSenderAddress(tx) ??
     (txType === TX_TYPE.signed ? getSenderAddress(tx.encodedTx.tx) : null)
-  const [account, height] = await Promise.all([
+  const [account, { height }] = await Promise.all([
     address && node.api.getAccountByPubkey(address).catch(() => ({
       id: address,
       balance: new BigNumber(0),
       nonce: 0
     })),
-    node.api.getCurrentKeyBlockHeight().then(({ height }) => height)
+    node.api.getCurrentKeyBlockHeight()
   ])
 
   return (await Promise.all(

package/src/utils/aepp-wallet-communication/connection/browser-runtime.js

@@ -18,15 +18,21 @@
 /**
  * Browser runtime connector module
  *
- * This is the complement to {@link module:@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection}.
+ * This is the complement to
+ * {@link module:@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection}.
  * @module @aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection/browser-runtime
  * @export BrowserRuntimeConnection
- * @example import BrowserRuntimeConnection from '@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection/browser-runtime'
+ * @example import { BrowserRuntimeConnection } from '@aeternity/aepp-sdk'
  */
 import stampit from '@stamp/it'
 
 import WalletConnection from '.'
 import { getBrowserAPI } from '../helpers'
+import {
+  RpcConnectionError,
+  AlreadyConnectedError,
+  NoWalletConnectedError
+} from '../../errors'
 
 /**
  * Disconnect
@@ -53,7 +59,7 @@ function disconnect () {
  * @return {void}
  */
 function connect (onMessage, onDisconnect) {
-  if (this.isConnected()) throw new Error('You already connected')
+  if (this.isConnected()) throw new AlreadyConnectedError('You already connected')
   this.handler = (msg, source) => {
     if (this.debug) console.log('Receive message: ', msg)
     onMessage(msg, source)
@@ -74,7 +80,7 @@ function connect (onMessage, onDisconnect) {
  * @return {void}
  */
 function sendMessage (msg) {
-  if (!this.port) throw new Error('You dont have connection. Please connect before')
+  if (!this.port) throw new NoWalletConnectedError('You dont have connection. Please connect before')
   if (this.debug) console.log('Send message: ', msg)
   this.port.postMessage(msg)
 }
@@ -104,7 +110,7 @@ function isConnected () {
  */
 export default stampit({
   init ({ connectionInfo = {}, port, debug = false }) {
-    if (!getBrowserAPI().runtime) throw new Error('Runtime is not accessible in your environment')
+    if (!getBrowserAPI().runtime) throw new RpcConnectionError('Runtime is not accessible in your environment')
     this.debug = debug
     this.connectionInfo = connectionInfo
     this.port = port || getBrowserAPI().runtime.connect(...[connectionInfo.id || undefined])

package/src/utils/aepp-wallet-communication/connection/browser-window-message.js

@@ -18,16 +18,24 @@
 /**
  * Browser window Post Message connector module
  *
- * This is the complement to {@link module:@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection}.
+ * This is the complement to
+ * {@link module:@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection}.
  * @module @aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection/browser-window-message
  * @export BrowserWindowMessageConnection
- * @example import BrowserWindowMessageConnection from '@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection/browser-window-message'
+ * @example
+ * import BrowserWindowMessageConnection
+ * from '@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection/browser-window-message'
  */
 import stampit from '@stamp/it'
 import WalletConnection from '.'
-import { v4 as uuid } from 'uuid'
+import { v4 as uuid } from '@aeternity/uuid'
 import { MESSAGE_DIRECTION } from '../schema'
 import { getBrowserAPI, isInIframe } from '../helpers'
+import {
+  AlreadyConnectedError,
+  NoWalletConnectedError,
+  MessageDirectionError
+} from '../../errors'
 
 /**
  * Check if connected
@@ -53,7 +61,7 @@ function connect (onMessage) {
   const receiveDirection = this.receiveDirection
   const debug = this.debug
   const forceOrigin = this.forceOrigin
-  if (this.listener) throw new Error('You already connected')
+  if (this.listener) throw new AlreadyConnectedError('You already connected')
 
   this.listener = (msg, source) => {
     if (!msg || typeof msg.data !== 'object') return
@@ -77,7 +85,7 @@ function connect (onMessage) {
  * @return {void}
  */
 function disconnect () {
-  if (!this.listener) throw new Error('You dont have connection. Please connect before')
+  if (!this.listener) throw new NoWalletConnectedError('You dont have connection. Please connect before')
   this.unsubscribeFn(this.listener)
   this.listener = null
 }
@@ -102,29 +110,44 @@ const getTarget = () => {
   const isContentScript = isExtensionContext && isWeb
   if (isContentScript) return window
   // When we is the main page we need to decide the target by our self
-  // Probably can be implemented some algo for checking DOM for Iframes and somehow decide which Iframe to talk
+  // Probably can be implemented some algo for checking DOM for Iframes and somehow decide which
+  // Iframe to talk
   return isInIframe() ? window.parent : undefined
 }
 
 /**
  * BrowserWindowMessageConnection
  * @function
- * @alias module:@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection/browser-window-message
+ * @alias module:@aeternity/aepp-sdk/es/utils/aepp-wallet-communication\
+ * /connection/browser-window-message
  * @rtype Stamp
  * @param {Object} [params={}] - Initializer object
  * @param {Object} [params.target=window.parent] - Target window for message
  * @param {Object} [params.self=window] - Host window for message
  * @param {Object} [params.origin] - Origin of receiver
- * @param {Object} [params.sendDirection] - Optional field for wrapping messages in additional structure({ type: 'to_aepp' || 'to_waellet', data }).Used for handling messages netween content script and page
- * @param {Object} [params.receiveDirection='to_aepp'] - Optional(default: 'to_aepp') field for unwrapping messages from additional structure({ type: 'to_aepp' || 'to_waellet', data }).Used for handling messages netween content script and page
+ * @param {String} [params.sendDirection] Wrapping messages into additional struct
+ * ({ type: 'to_aepp' || 'to_waellet', data })
+ * Used for handling messages between content script and page
+ * @param {String} [params.receiveDirection='to_aepp'] Unwrapping messages from additional struct
+ * ({ type: 'to_aepp' || 'to_waellet', data })
+ * Used for handling messages between content script and page
  * @param {Object} [params.connectionInfo={}] - Connection info object
  * @param {Boolean} [params.debug=false] - Debug flag
  * @return {Object}
  */
 export default stampit({
-  init ({ connectionInfo = {}, target = getTarget(), self = window, origin, sendDirection, receiveDirection = MESSAGE_DIRECTION.to_aepp, debug = false, forceOrigin = false } = {}) {
-    if (sendDirection && !Object.keys(MESSAGE_DIRECTION).includes(sendDirection)) throw new Error(`sendDirection must be one of [${Object.keys(MESSAGE_DIRECTION)}]`)
-    if (!Object.keys(MESSAGE_DIRECTION).includes(receiveDirection)) throw new Error(`receiveDirection must be one of [${Object.keys(MESSAGE_DIRECTION)}]`)
+  init ({
+    connectionInfo = {},
+    target = getTarget(),
+    self = window,
+    origin,
+    sendDirection,
+    receiveDirection = MESSAGE_DIRECTION.to_aepp,
+    debug = false,
+    forceOrigin = false
+  } = {}) {
+    if (sendDirection && !Object.keys(MESSAGE_DIRECTION).includes(sendDirection)) throw new MessageDirectionError(`sendDirection must be one of [${Object.keys(MESSAGE_DIRECTION)}]`)
+    if (!Object.keys(MESSAGE_DIRECTION).includes(receiveDirection)) throw new MessageDirectionError(`receiveDirection must be one of [${Object.keys(MESSAGE_DIRECTION)}]`)
     this.connectionInfo = { id: uuid(), ...connectionInfo }
 
     const selfP = self
@@ -137,7 +160,6 @@ export default stampit({
     this.subscribeFn = (listener) => selfP.addEventListener('message', listener, false)
     this.unsubscribeFn = (listener) => selfP.removeEventListener('message', listener, false)
     this.postFn = (msg) => targetP.postMessage(msg, this.origin || '*')
-    if (!this.connectionInfo.id) throw new Error('ID required.')
   },
   methods: { connect, sendMessage, disconnect, isConnected }
 }, WalletConnection)

package/src/utils/aepp-wallet-communication/connection/index.js

@@ -19,7 +19,8 @@
  * Wallet Connection base module
  * @module @aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection
  * @export WalletConnection
- * @example import WalletConnection from '@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection'
+ * @example
+ * import WalletConnection from '@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection'
  */
 
 import { required } from '@stamp/required'

package/src/utils/aepp-wallet-communication/content-script-bridge.js

@@ -20,9 +20,12 @@
  *
  * @module @aeternity/aepp-sdk/es/utils/aepp-wallet-communication/content-script-bridge
  * @export ContentScriptBridge
- * @example import ContentScriptBridge from '@aeternity/aepp-sdk/es/utils/wallet-communication/content-script-bridge
+ * @example
+ * import ContentScriptBridge
+ * from '@aeternity/aepp-sdk/es/utils/wallet-communication/content-script-bridge
  */
 import stampit from '@stamp/it'
+import { UnsupportedPlatformError, MissingParamError } from '../errors'
 
 /**
  * Start message proxy
@@ -56,21 +59,25 @@ function stop () {
 
 /**
  * ContentScriptBridge stamp
- * Provide functionality to easly redirect messages from page to extension and from extension to page through content script
+ * Provide functionality to easly redirect messages from page to extension and from extension to
+ * page through content script
  * Using Runtime(Extension) and WindowPostMessage(Web-Page) connections
  * @function
  * @alias module:@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/content-script-bridge
  * @rtype Stamp
  * @param {Object} params - Initializer object
- * @param {Object} params.pageConnection - Page connection object(@link module:@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection/browser-window-message)
- * @param {Object} params.extConnection - Extension connection object(module: @aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection/browser-runtime)
+ * @param {Object} params.pageConnection - Page connection object
+ * (@link module:@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection/
+ * browser-window-message)
+ * @param {Object} params.extConnection - Extension connection object
+ * (@link module:@aeternity/aepp-sdk/es/utils/aepp-wallet-communication/connection/browser-runtime)
  * @return {Object}
  */
 export default stampit({
   init ({ pageConnection, extConnection, allowCrossOrigin = false }) {
-    if (!window) throw new Error('Window object not found, you can run bridge only in browser')
-    if (!pageConnection) throw new Error('pageConnection required')
-    if (!extConnection) throw new Error('extConnection required')
+    if (!window) throw new UnsupportedPlatformError('Window object not found, you can run bridge only in browser')
+    if (!pageConnection) throw new MissingParamError('pageConnection required')
+    if (!extConnection) throw new MissingParamError('extConnection required')
     this.allowCrossOrigin = allowCrossOrigin
     this.pageConnection = pageConnection
     this.extConnection = extConnection

package/src/utils/aepp-wallet-communication/helpers.js

@@ -1,7 +1,7 @@
 /**
  * Browser helper functions
  */
-import { isAccountBase } from '../../account/base'
+import { NoBrowserFoundError } from '../errors'
 
 export const getBrowserAPI = (force = false) => {
   const { chrome, browser } = window
@@ -9,7 +9,7 @@ export const getBrowserAPI = (force = false) => {
   if (typeof chrome !== 'undefined' && chrome === Object(chrome)) return chrome
   // Firefox support
   if (typeof browser !== 'undefined' && browser === Object(browser)) return browser
-  if (!force) throw new Error('Browser is not detected')
+  if (!force) throw new NoBrowserFoundError()
   return {}
 }
 
@@ -51,18 +51,10 @@ export const getHandler = (schema, msg, { debug = false } = {}) => {
 
 export const message = (method, params = {}) => ({ method, params })
 
-export const responseMessage = (id, method, { error, result } = {}) => ({ id, method, ...(error ? { error } : { result }) })
+export const responseMessage = (id, method, { error, result } = {}) =>
+  ({ id, method, ...(error ? { error } : { result }) })
 
-export const sendResponseMessage = (client) => (id, method, data) => client.sendMessage(responseMessage(id, method, data), true)
-
-export const voidFn = () => undefined
+export const sendResponseMessage = (client) => (id, method, data) =>
+  client.sendMessage(responseMessage(id, method, data), true)
 
 export const isValidAccounts = (accounts) => (['', 'connected', 'current'].filter(k => typeof (k ? accounts[k] : accounts) === 'object')).length === 3
-
-export const resolveOnAccount = (addresses, onAccount, opt = {}) => {
-  if (!addresses.find(a => a === onAccount)) {
-    if (typeof opt.onAccount !== 'object' || !isAccountBase(opt.onAccount)) throw new Error('Provided onAccount should be an AccountBase')
-    onAccount = opt.onAccount
-  }
-  return onAccount
-}