@konemono/nostr-login 1.11.7 → 1.11.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@konemono/nostr-login",
-  "version": "1.11.7",
+  "version": "1.11.9",
   "description": "",
   "main": "./dist/index.esm.js",
   "types": "./dist/index.d.ts",
@@ -12,7 +12,6 @@
   "author": "a-fralou",
   "dependencies": {
     "@nostr-dev-kit/ndk": "^2.3.1",
-    "events": "^3.3.0",
     "nostr-tools": "^1.17.0",
     "tseep": "^1.2.1"
   },

package/src/modules/AuthNostrService.ts

@@ -610,8 +610,7 @@ class AuthNostrService extends EventEmitter implements Signer {
     const localSigner = new PrivateKeySigner(info.sk!);
     this.signer = new Nip46Signer(this.ndk, localSigner, info.signerPubkey!, info.iframeUrl ? new URL(info.iframeUrl!).origin : undefined);
 
-    
-    // ★ once を使う - 1回だけ実行される ★
+     // ★ once を使う - 1回だけ実行される ★
     this.signer.once('connectionLost', async () => {
       console.log('Connection lost, attempting to reconnect...');
       this.signer = null;
@@ -621,7 +620,6 @@ class AuthNostrService extends EventEmitter implements Signer {
       }
     });
 
-
     // we should notify the banner the same way as
     // the onAuthUrl does
     this.signer.on(`iframeRestart`, async () => {

package/src/modules/Nip46.ts

@@ -17,7 +17,6 @@ class NostrRpc extends NDKNostrRpc {
   protected _useNip44: boolean = false;
   protected eventEmitter: EventEmitter = new EventEmitter();
 
-
   public constructor(ndk: NDK, signer: PrivateKeySigner) {
     super(ndk, signer, ndk.debug.extend('nip46:signer:rpc'));
     this._ndk = ndk;
@@ -150,7 +149,7 @@ class NostrRpc extends NDKNostrRpc {
   }
 
   // タイムアウト対応のping
-  public async pingWithTimeout(remotePubkey: string, timeoutMs: number = 10000): Promise<void> {
+  public async pingWithTimeout(remotePubkey: string, timeoutMs: number = 3000): Promise<void> {
     return withTimeout(this.ping(remotePubkey), timeoutMs, `Ping timeout after ${timeoutMs}ms`);
   }
 
@@ -158,6 +157,14 @@ class NostrRpc extends NDKNostrRpc {
     return Math.random().toString(36).substring(7);
   }
 
+   public override once = <EventKey extends string | symbol = string>(
+    event: EventKey,
+    listener: (...args: any[]) => void
+  ): this => {
+    this.eventEmitter.once(event as string, listener);
+    return this;
+  }
+
   public async sendRequest(remotePubkey: string, method: string, params: string[] = [], kind = 24133, cb?: (res: NDKRpcResponse) => void): Promise<NDKRpcResponse> {
     const id = this.getId();
 
@@ -234,14 +241,6 @@ class NostrRpc extends NDKNostrRpc {
     return this;
   }
 
-  public override once = <EventKey extends string | symbol = string>(
-    event: EventKey,
-    listener: (...args: any[]) => void
-  ): this => {
-    this.eventEmitter.once(event as string, listener);
-    return this;
-  }
-
   public override emit = <EventKey extends string | symbol = string>(
     event: EventKey,
     ...args: any[]
@@ -382,7 +381,6 @@ export class ReadyListener {
   }
 }
 
-
 export class Nip46Signer extends NDKNip46Signer {
   private _userPubkey: string = '';
   private _rpc: IframeNostrRpc;
@@ -401,7 +399,6 @@ export class Nip46Signer extends NDKNip46Signer {
     });
 
     this.rpc = this._rpc;
-
   }
 
   get userPubkey() {
@@ -410,7 +407,10 @@ export class Nip46Signer extends NDKNip46Signer {
 
 
 
-  // 接続確認（必要時のみping）
+  // Nip46.tsのNip46Signerクラス内
+  // 接続確認（必要時のみping リトライ付き 最大10秒）
+  // リトライ回数: 2回（計3回試行）
+  //合計最大時間: 2秒(ping) × 3回 + 2秒(待機) × 2回 = 10秒
   private async ensureConnection(retries: number = 2): Promise<void> {
     if (!this.remotePubkey) return;
 
@@ -423,48 +423,18 @@ export class Nip46Signer extends NDKNip46Signer {
 
     for (let i = 0; i <= retries; i++) {
       try {
-        await this._rpc.pingWithTimeout(this.remotePubkey, 10000);
+        await this._rpc.pingWithTimeout(this.remotePubkey, 2000); // 2秒タイムアウト
         this.lastPingTime = now;
         console.log('Connection check OK');
         return;
       } catch (error) {
         if (i === retries) {
-          console.error('Ping failed, attempting reconnection');
-
-          // 再接続を試みる
-          try {
-            this.emit('connectionLost');
-
-            // 再接続完了を待つ（最大10秒）
-            await new Promise<void>((resolve, reject) => {
-              const timeout = setTimeout(() => reject(new Error('Reconnection timeout')), 10000);
-
-              const checkReconnection = async () => {
-                try {
-                  await this._rpc.pingWithTimeout(this.remotePubkey!, 10000);
-                  clearTimeout(timeout);
-                  this.lastPingTime = Date.now();
-                  console.log('Reconnection successful');
-                  resolve();
-                } catch (e) {
-                  // まだ再接続中、1秒後に再確認
-                  setTimeout(checkReconnection, 1000);
-                }
-              };
-
-              // 少し待ってから確認開始
-              setTimeout(checkReconnection, 2000);
-            });
-
-            return;
-          } catch (reconnectError) {
-            console.error('Reconnection failed:', reconnectError);
-            throw new Error('NIP-46 connection lost and reconnection failed');
-          }
+          console.error('Connection check failed after retries', error);
+          throw new Error('NIP-46 connection lost');
         }
 
-        const delay = Math.min(1000 * Math.pow(2, i), 5000);
-        console.log(`Ping failed, retrying in ${delay}ms...`);
+        const delay = 2000; // 2秒間隔で再送
+        console.log(`Ping failed (${i + 1}/${retries + 1}), retrying in ${delay}ms...`);
         await new Promise(resolve => setTimeout(resolve, delay));
       }
     }
@@ -473,7 +443,6 @@ export class Nip46Signer extends NDKNip46Signer {
   private async setSignerPubkey(signerPubkey: string, sameAsUser: boolean = false) {
     console.log('setSignerPubkey', signerPubkey);
 
-
     // ensure it's set
     this.remotePubkey = signerPubkey;
 
@@ -574,7 +543,7 @@ export class Nip46Signer extends NDKNip46Signer {
   }
 
   // EventEmitter互換メソッド
-  // ★ ここに once を追加 ★
+    // ★ ここに once を追加 ★
   public override on = <EventKey extends string | symbol = string>(
     event: EventKey,
     listener: (...args: any[]) => void