From 692685a9acffc48f00250ed685cfc356a21e7693 Mon Sep 17 00:00:00 2001
From: Paul <108695806+pxrl@users.noreply.github.com>
Date: Fri, 29 Sep 2023 14:48:14 +0200
Subject: [PATCH] improve: Simplify wallet key type (#951)

The type is correctly filtered within getSigner(), but it causes
unnecessary type annoyances when reading the key type from the
environment or on the command-line.
---
 src/utils/SignerUtils.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/utils/SignerUtils.ts b/src/utils/SignerUtils.ts
index 8ff77d1ed..530b84841 100644
--- a/src/utils/SignerUtils.ts
+++ b/src/utils/SignerUtils.ts
@@ -8,7 +8,7 @@ export type SignerOptions = {
    * The type of wallet to use.
    * @note If using a GCKMS wallet, the gckmsKeys parameter must be set.
    */
-  keyType: "mnemonic" | "privateKey" | "gckms";
+  keyType: string;
   /**
    * Whether or not to clear the mnemonic/private key from the env after retrieving the signer.
    * @note Not including this parameter or setting it to false will not clear the mnemonic/private key from the env.
@@ -42,7 +42,7 @@ export async function getSigner({ keyType, gckmsKeys, cleanEnv }: SignerOptions)
       wallet = await getGckmsSigner(gckmsKeys);
       break;
     default:
-      throw new Error("Must define mnemonic, privatekey or gckms for wallet");
+      throw new Error(`getSigner: Unsupported key type (${keyType})`);
   }
   if (!wallet) {
     throw new Error("Must define mnemonic, privatekey or gckms for wallet");