fix: connect on every nwc request to ensure reconnect happens

Author: rolznz

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@getalby/sdk",
-  "version": "3.2.1",
+  "version": "3.2.2",
   "description": "The SDK to integrate with Nostr Wallet Connect and the Alby API",
   "repository": "https://github.com/getAlby/js-sdk.git",
   "bugs": "https://github.com/getAlby/js-sdk/issues",

src/webln/NostrWeblnProvider.ts

@@ -242,10 +242,7 @@ export class NostrWebLNProvider implements WebLNProvider, Nip07Provider {
   }
 
   async enable() {
-    if (this.connected) {
-      return Promise.resolve();
-    }
-    await this.relay.connect();
+    return Promise.resolve();
   }
 
   close() {
@@ -271,7 +268,7 @@ export class NostrWebLNProvider implements WebLNProvider, Nip07Provider {
   // WebLN compatible response
   // TODO: use NIP-47 get_info call
   async getInfo(): Promise<GetInfoResponse> {
-    this.checkConnected();
+    await this.checkConnected();
 
     const supports = ["lightning", "nostr"];
     const version = "Alby JS SDK";
@@ -312,8 +309,8 @@ export class NostrWebLNProvider implements WebLNProvider, Nip07Provider {
     }
   }
 
-  getBalance() {
-    this.checkConnected();
+  async getBalance() {
+    await this.checkConnected();
 
     return this.executeNip47Request<GetBalanceResponse, { balance: number }>(
       "get_balance",
@@ -327,8 +324,8 @@ export class NostrWebLNProvider implements WebLNProvider, Nip07Provider {
     );
   }
 
-  sendPayment(invoice: string) {
-    this.checkConnected();
+  async sendPayment(invoice: string) {
+    await this.checkConnected();
 
     return this.executeNip47Request<SendPaymentResponse, Nip47PayResponse>(
       "pay_invoice",
@@ -340,8 +337,8 @@ export class NostrWebLNProvider implements WebLNProvider, Nip07Provider {
     );
   }
 
-  keysend(args: KeysendArgs) {
-    this.checkConnected();
+  async keysend(args: KeysendArgs) {
+    await this.checkConnected();
 
     return this.executeNip47Request<SendPaymentResponse, Nip47PayResponse>(
       "pay_keysend",
@@ -369,8 +366,8 @@ export class NostrWebLNProvider implements WebLNProvider, Nip07Provider {
     throw new Error("Method not implemented.");
   }
 
-  makeInvoice(args: string | number | RequestInvoiceArgs) {
-    this.checkConnected();
+  async makeInvoice(args: string | number | RequestInvoiceArgs) {
+    await this.checkConnected();
 
     const requestInvoiceArgs: RequestInvoiceArgs | undefined =
       typeof args === "object" ? (args as RequestInvoiceArgs) : undefined;
@@ -394,8 +391,8 @@ export class NostrWebLNProvider implements WebLNProvider, Nip07Provider {
     );
   }
 
-  lookupInvoice(args: LookupInvoiceArgs) {
-    this.checkConnected();
+  async lookupInvoice(args: LookupInvoiceArgs) {
+    await this.checkConnected();
 
     return this.executeNip47Request<LookupInvoiceResponse, Nip47Transaction>(
       "lookup_invoice",
@@ -412,8 +409,8 @@ export class NostrWebLNProvider implements WebLNProvider, Nip07Provider {
     );
   }
 
-  listTransactions(args: ListTransactionsArgs) {
-    this.checkConnected();
+  async listTransactions(args: ListTransactionsArgs) {
+    await this.checkConnected();
 
     // maybe we can tailor the response to our needs
     return this.executeNip47Request<
@@ -527,12 +524,11 @@ export class NostrWebLNProvider implements WebLNProvider, Nip07Provider {
     });
   }
 
-  private checkConnected() {
-    if (!this.connected) {
-      throw new Error(
-        "please call enable() and await the promise before calling this function",
-      );
+  private async checkConnected() {
+    if (!this.secret) {
+      throw new Error("Missing secret key");
     }
+    await this.relay.connect();
   }
 
   private executeNip47Request<T, R>(