From 0229215d89086c7f59874d32661964b5a768ceac Mon Sep 17 00:00:00 2001 From: Richard Watts Date: Fri, 17 Jan 2025 16:05:05 +0000 Subject: [PATCH] (fix) CJS distribution of objects with an initializer and a constructor appears to be broken, so initialize chainId in the constructor. --- .../packages/wallets/zilliqa/src/ZilliqaWalletClient.ts | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/typescript/packages/wallets/zilliqa/src/ZilliqaWalletClient.ts b/typescript/packages/wallets/zilliqa/src/ZilliqaWalletClient.ts index 3e22e33e3..122ae950f 100644 --- a/typescript/packages/wallets/zilliqa/src/ZilliqaWalletClient.ts +++ b/typescript/packages/wallets/zilliqa/src/ZilliqaWalletClient.ts @@ -27,10 +27,15 @@ export abstract class ZilliqaWalletClient extends WalletClientBase { export class ZilliqaJSViemWalletClient extends ZilliqaWalletClient { zilliqa: Zilliqa; viem: ViemEVMWalletClient; - chainId = 0; + // This was originally a static assignment; however, this causes + // typescript to emit invalid JS when compiling the CJS version of + // the distribution package, so it is now initialized in the + // constructor. + chainId; constructor(client: ViemWalletClient, node: string, account: Account, chainId: number, options?: ViemOptions) { super(); + this.chainId = 0; this.viem = new ViemEVMWalletClient(client, options); this.zilliqa = new Zilliqa(node); this.zilliqa.wallet.addByPrivateKey(account.privateKey);