imp(jssdk): use BN_MAX_SUPPLY for deposit settings in gas estimation

frontend/packages/sdk/src/contracts/PinkBlueprint.ts

@@ -22,6 +22,7 @@ import assert from '../utils/assert'
 import { randomHex } from '../utils/hex'
 import signAndSend from '../utils/signAndSend'
 import { PinkContractPromise } from './PinkContract'
+import { BN_MAX_SUPPLY } from '../utils/constants'
 
 export interface PinkContractInstantiateCallOutcome extends ContractCallOutcome {
   salt: string
@@ -367,22 +368,16 @@ export class PinkBlueprintPromise {
       throw new Error(`Constructor not found: ${constructorOrId}`)
     }
 
-    const { gasPrice, depositPerByte } = this.phatRegistry.clusterInfo ?? {}
-    if (!gasPrice || !depositPerByte) {
+    const { gasPrice } = this.phatRegistry.clusterInfo ?? {}
+    if (!gasPrice) {
       throw new Error('No Gas Price or deposit Per Byte from cluster info.')
     }
 
-    // We assume the address have no tokens in the cluster account, so we will deposit into it
-    // at the same time we submit the transaction. The gas fee is far lower then 1 PHA in general
-    // so we plus 1 PHA here.
-    const msg = this.abi.findConstructor(constructorOrId).toU8a(args)
-    const deposit = depositPerByte.mul(new BN(msg.length * 1.05)).add(new BN(1e12))
-
     const [clusterBalance, onchainBalance, { gasRequired, storageDeposit }] = await Promise.all([
       this.phatRegistry.getClusterBalance(address),
       this.api.query.system.account<FrameSystemAccountInfo>(address),
       // We estimating the gas & storage deposit cost with deposit propose.
-      estimate(cert.address, { cert, deposit }, ...args),
+      estimate(cert.address, { cert, deposit: BN_MAX_SUPPLY }, ...args),
     ])
 
     // calculate the total costs

frontend/packages/sdk/src/contracts/PinkContract.ts

@@ -23,6 +23,7 @@ import type { AbiLike, FrameSystemAccountInfo } from '../types'
 import assert from '../utils/assert'
 import { randomHex } from '../utils/hex'
 import signAndSend from '../utils/signAndSend'
+import { BN_MAX_SUPPLY } from '../utils/constants'
 
 export type PinkContractCallOutcome<ResultType> = {
   output: ResultType
@@ -459,21 +460,15 @@ export class PinkContractPromise<
       throw new Error(`Message not found: ${messageOrId}`)
     }
 
-    const { gasPrice, depositPerByte } = this.phatRegistry.clusterInfo ?? {}
-    if (!gasPrice || !depositPerByte) {
+    const { gasPrice } = this.phatRegistry.clusterInfo ?? {}
+    if (!gasPrice) {
       throw new Error('No Gas Price or deposit Per Byte from cluster info.')
     }
 
-    // We assume the address have no tokens in the cluster account, so we will deposit into it
-    // at the same time we submit the transaction. The gas fee is far lower then 1 PHA in general
-    // so we plus 1 PHA here.
-    const msg = this.abi.findMessage(messageOrId).toU8a(args)
-    const deposit = depositPerByte.mul(new BN(msg.length)).add(new BN(1e12))
-
     const [clusterBalance, onchainBalance, { gasRequired, storageDeposit }] = await Promise.all([
       this.phatRegistry.getClusterBalance(address),
       this.api.query.system.account<FrameSystemAccountInfo>(address),
-      estimate(cert.address, { cert, deposit }, ...args),
+      estimate(cert.address, { cert, deposit: BN_MAX_SUPPLY }, ...args),
     ])
 
     // calculate the total costs

frontend/packages/sdk/src/utils/constants.ts

@@ -0,0 +1,3 @@
+import { BN } from '@polkadot/util'
+
+export const BN_MAX_SUPPLY = new BN(1e12).mul(new BN(1e9))