Sma 46 multichain validation module

[livingrockrises]

Description

Multichain validation module implementation for multi chain userops signing.
Actual test script would be part of test cases soon - and also in backend scripts modulat-SA-v2 playground.

can change the base later if other PRs are merged first.

EDIT: This is done

I am trying to send one tx on bnb + polygon testnets soon using this module.
later we can send the ops which actually makes uses of enabled session.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

I have tested from backend-node scripts in sdk-examples repo here

let provider = new ethers.providers.JsonRpcProvider(config.rpcUrl);
  let signer = new ethers.Wallet(config.privateKey, provider);
  const eoa = await signer.getAddress();
  console.log(chalk.blue(`EOA address: ${eoa}`));

  // create bundler and paymaster instances
  const bundler1 = new Bundler({
    bundlerUrl: config.bundlerUrl,
    chainId: config.chainId,
    entryPointAddress: DEFAULT_ENTRYPOINT_ADDRESS,
  });

  const paymaster = new BiconomyPaymaster({
    paymasterUrl: config.biconomyPaymasterUrl
  });

  const multiChainModule = new MultiChainValidationModule({
    signer: signer,
    chainId: config.chainId, // to be removed
    moduleAddress: '0x2E817fe3749B81dA801fc08B247E081ec20eB080'
  })
  
  const biconomySmartAccountConfig1 = {
    signer: signer,
    chainId: config.chainId,
    rpcUrl: config.rpcUrl,
    paymaster: paymaster, 
    bundler: bundler1, 
    entryPointAddress: DEFAULT_ENTRYPOINT_ADDRESS,
    defaultValidationModule: multiChainModule,
    activeValidationModule: multiChainModule
  };

  // create biconomy smart account instance
  const biconomyAccount1 = new BiconomySmartAccountV2(biconomySmartAccountConfig1);

  // passing accountIndex is optional, by default it will be 0. You may use different indexes for generating multiple counterfactual smart accounts for the same user
  const biconomySmartAccount1 = await biconomyAccount1.init();


// For another chain instance

const bundler2 = new Bundler({
    bundlerUrl: "https://bundler.biconomy.io/api/v2/97/A5CBjLqSc.0dcbc53e-anPe-44c7-b22d-21071345f76a",
    chainId: 97,
    entryPointAddress: DEFAULT_ENTRYPOINT_ADDRESS,
  });

  const biconomySmartAccountConfig2 = {
    signer: signer,
    chainId: 97,
    rpcUrl: 'https://data-seed-prebsc-1-s2.binance.org:8545',
    // paymaster: paymaster, 
    bundler: bundler2, 
    entryPointAddress: DEFAULT_ENTRYPOINT_ADDRESS,
    defaultValidationModule: multiChainModule,
    activeValidationModule: multiChainModule
  };


  // create biconomy smart account instance
  const biconomyAccount2 = new BiconomySmartAccountV2(biconomySmartAccountConfig2);

  // passing accountIndex is optional, by default it will be 0. You may use different indexes for generating multiple counterfactual smart accounts for the same user
  const biconomySmartAccount2 = await biconomyAccount2.init();
  
  // Building user ops
  
  // any transaction object (say mintNFT)
  // or maybe enableModule (session key manager address)
  // or maybe enableModule and createSession batch 
  // createSession() tx data
  
  let partialUserOp1 = await biconomySmartAccount1.buildUserOp([transaction1]);
  
  let partialUserOp2 = await biconomySmartAccount2.buildUserOp([transaction2]);
  
  const returnedOps = await multiChainModule.signUserOps([{userOp: partialUserOp1, chainId: 80001}, {userOp: partialUserOp2, chainId: 97}]);
  
  // Sending individual ops with common multichain sig
  
  // Mumbai
  
  const userOpResponse1 = await biconomySmartAccount1.sendSignedUserOp(returnedOps[0] as any);
  console.log(chalk.green(`userOp Hash: ${userOpResponse1.userOpHash}`));
  const transactionDetails1 = await userOpResponse1.wait();
  
  
  // BNB testnet
  
  const userOpResponse2 = await biconomySmartAccount2.sendSignedUserOp(returnedOps[1] as any);
  console.log(chalk.green(`userOp Hash: ${userOpResponse2.userOpHash}`));
  const transactionDetails2 = await userOpResponse2.wait();
  

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[linear]

SMA-46 SA V2 - Add implementation of Multi-chain validation module

[AmanRaj1608]

Why is this needed, what needs to be updated?

[livingrockrises]

smartAccount.buildUserOp() makes a call to the bundler where need to provide one dummySignature for one of the enabled modules. so each module is supposed to provide dummy mock sig.

but I think as MultiChain is extension of ECDSA and can verifying ECDSA sigs as well the default ECDSA module sig also works on bundler side, when I was just calling sa.buildUserOp()

and the TODO note is specific to making above sig dynamic so that module address becomes a param. I will push this change now.

[livingrockrises]

done

[livingrockrises]

@AmanRaj1608 @tomarsachin2271 since it extends the ECDSAModule chainId becomes mandatory but in real sense this module itself doesn't need to maintain chainId as it will receive it as part of dto (array of objects - userop + chainId) in singUserOps method.
what should I do?

[AmanRaj1608]

I think we can remove the extend part as the MultiChainValidationModule is different module and not really depends on ECDSAOwnershipValidationModule

[livingrockrises]

ok let me do that.

[livingrockrises]

done

[livingrockrises]

also updated sample client side code.

[AmanRaj1608]

these should be taken by the user as param

[livingrockrises]

done

[AmanRaj1608]

Can refactor the commented codes and merge