Change to require a Signer so that it can be used with external wallets such as metamask (#25)

Author: serinuntius

src/preset/builder/simpleAccount.ts

@@ -16,21 +16,21 @@ import {
 import { UserOperationMiddlewareFn } from "../../types";
 
 export class SimpleAccount extends UserOperationBuilder {
-  private signer: ethers.Wallet;
+  private signer: ethers.Signer;
   private provider: ethers.providers.JsonRpcProvider;
   private entryPoint: EntryPoint;
   private factory: SimpleAccountFactory;
   private initCode: string;
   proxy: SimpleAccountImpl;
 
   private constructor(
-    signingKey: string,
+    signer: ethers.Signer,
     ERC4337NodeRpc: string,
     entryPoint: string,
     factory: string
   ) {
     super();
-    this.signer = new ethers.Wallet(signingKey);
+    this.signer = signer;
     this.provider = new ethers.providers.JsonRpcProvider(ERC4337NodeRpc);
     this.entryPoint = EntryPoint__factory.connect(entryPoint, this.provider);
     this.factory = SimpleAccountFactory__factory.connect(
@@ -50,14 +50,14 @@ export class SimpleAccount extends UserOperationBuilder {
   };
 
   public static async init(
-    signingKey: string,
+    signer: ethers.Signer,
     ERC4337NodeRpc: string,
     entryPoint: string,
     factory: string,
     paymasterMiddleware?: UserOperationMiddlewareFn
   ): Promise<SimpleAccount> {
     const instance = new SimpleAccount(
-      signingKey,
+      signer,
       ERC4337NodeRpc,
       entryPoint,
       factory

src/preset/middleware/signature.ts

@@ -2,7 +2,7 @@ import { ethers } from "ethers";
 import { UserOperationMiddlewareFn } from "../../types";
 
 export const EOASignature =
-  (signer: ethers.Wallet): UserOperationMiddlewareFn =>
+  (signer: ethers.Signer): UserOperationMiddlewareFn =>
   async (ctx) => {
     ctx.op.signature = await signer.signMessage(
       ethers.utils.arrayify(ctx.getUserOpHash())