feat: lazy load hdwallet adapter modules part 1

[gomesalexandre]

Description

i.e technically lazy loads said packages, but doesn't effectively do so for now, because of this bad boy:

web/src/context/WalletProvider/WalletProvider.tsx

Lines 767 to 783 in a5839be

	const walletAdapters = await SUPPORTED_WALLETS[keyManager]?.adapters.reduce<
	Promise<GenericAdapter[]>
	>(async (acc, cur) => {
	const adapters = await acc
	const options = getKeyManagerOptions(keyManager)
	try {
	const { loadAdapter } = cur
	const Adapter = await loadAdapter()
	// eslint is drunk, this isn't a hook
	// eslint-disable-next-line react-hooks/rules-of-hooks
	const adapter = Adapter.useKeyring(state.keyring, options)
	adapters.push(adapter)
	} catch (e) {
	console.error(e)
	}
	return acc
	}, Promise.resolve([]))

This however, puts us in a better position to get wallet adapters lazy loaded as a follow-up, the intent being to only populate (and by extension, import the npm packages) adapters when clicking on a wallet option.

Pull Request Type

• 🐛 Bug fix (Non-breaking Change: Fixes an issue)
• 🛠️ Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
• 💅 New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

• tackles Identify and implement lazy imports #5445

Risk

• Moderate. Theoretically, the surface area brings huge regression risks for all wallets. In effect, the flow is effectively the same for the time being.

Testing

• All wallets are still displayed as option
• Wallet pairing still work

Engineering

• ☝🏽

Operations

• ☝🏽

Screenshots (if applicable)

[gomesalexandre]

Current dependencies on/for this PR:

• develop
  • PR fix: don't initialize wallet in state.keyring effect #5462
    • PR feat: lazy load hdwallet adapter modules part 1 #5463 👈
      • PR feat: actually lazy load hdwallet adapter modules #5466
        • PR feat: walletprovider adapters type narrowing #5468
          • PR perf: defi only render current step if active #5476

This comment was auto-generated by Graphite.