Replies: 3 comments
-
|
Hi @theref terribly sorry for the late reply on this! We have been heads down building. I think this is a valid concern, and I think it would make sense for the We have made steps towards improving the developer experience for aggregating signatures for multisigs (we call the |
Beta Was this translation helpful? Give feedback.
-
|
No problem, we snuck around the issue by wrapping the multisig in a custom Viem wrapper and using that, but it's not idea because signing isn't actually implemented properly. Nice to see that progress is being made! Happy to talk to you through our use case if that helps with dev decisions - we'd definitely like the multisig implementation to be a bit more flexible, but I think many of those changes can only come with an upgrade to Entry Point 0.8.0 |
Beta Was this translation helpful? Give feedback.
-
|
I've raised #163 proposing that we make the The existing |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Summary
We're integrating TACo (Threshold Access Control) network's distributed signing with MetaMask Smart Accounts. Currently,
toMetaMaskSmartAccountrequires a local wallet as a signatory even when using external multisig signers. This creates an unnecessary dependency for applications using distributed signing networks.Current Implementation
Problem
When using distributed signing networks like TACo:
signatoryis just a placeholder and doesn't perform the actual signingUse Case
TACo provides distributed threshold signatures where:
signUserOp()functionExample of how we currently sign:
Desired Behavior
We'd like to wrap or integrate external signing networks (like TACo) with
toMetaMaskSmartAccountwithout requiring a dummy local wallet. We understand Viem has capabilities for custom account implementations and wrapping external signers, but we're unsure of the best pattern to apply here.Questions
signatoryfield support custom implementations that delegate to external networks?Benefits of Supporting This
Additional Context
Would love guidance on the best way to achieve this with the current toolkit or if this requires an enhancement.
Beta Was this translation helpful? Give feedback.
All reactions