@konemono/nostr-login 1.13.2 → 1.13.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@konemono/nostr-login",
-  "version": "1.13.2",
+  "version": "1.13.3",
   "description": "Extended fork of nostr-login with multi-relay support, QR scanner, and improved stability",
   "main": "./dist/index.esm.js",
   "types": "./dist/index.d.ts",
@@ -30,7 +30,7 @@
     "@rollup/plugin-commonjs": "^25.0.7",
     "@rollup/plugin-node-resolve": "^15.2.3",
     "@rollup/plugin-terser": "^0.4.4",
-    "@konemono/nostr-login-components": "^1.2.2",
+    "@konemono/nostr-login-components": "^1.2.3",
     "prettier": "^3.2.2",
     "rollup": "^4.9.6",
     "rollup-plugin-typescript2": "^0.36.0"

package/src/modules/AuthNostrService.ts

@@ -653,7 +653,7 @@ class AuthNostrService extends EventEmitter implements Signer {
           }
         }
 
-        await this.ndk.connect();
+        await this.ndk.connect(10000); // 10秒のタイムアウトでリレー接続
 
         const localSigner = new PrivateKeySigner(info.sk!);
         this.signer = new Nip46Signer(this.ndk, localSigner, info.signerPubkey!, iframeOrigin);
@@ -797,7 +797,7 @@ class AuthNostrService extends EventEmitter implements Signer {
     this.ensureRelaysInPool();
 
     try {
-      await this.ndk.connect();
+      await this.ndk.connect(10000);
     } catch (e) {
       console.warn('forceReconnect: ndk.connect() threw, will poll for connection...', e);
     }
@@ -819,7 +819,7 @@ class AuthNostrService extends EventEmitter implements Signer {
       this.ensureRelaysInPool();
 
       try {
-        await this.ndk.connect();
+        await this.ndk.connect(10000);
       } catch (e) {
         console.warn('ensureRelayConnection: ndk.connect() threw, will poll for connection...', e);
       }

package/src/modules/Nip46.ts

@@ -42,28 +42,43 @@ class NostrRpc extends NDKNostrRpc {
     // リレーが不安定な状態ではEOSEが届かず無限にハングする。
     // NIP-46ではリアルタイムのレスポンスのみ必要なので、
     // EOSE待ちをスキップして直接subscribeする。
+    // cacheUsage: ONLY_RELAY で確実にリレーからのみ購読する。
+    // onEvent を使って race condition を回避（NDK推奨）。
     const sub = this._ndk.subscribe(filter, {
       closeOnEose: false,
       groupable: false,
-    });
-
-    sub.on('event', async (event: NDKEvent) => {
-      try {
-        const parsedEvent = await this.parseEvent(event);
-        if ((parsedEvent as NDKRpcRequest).method) {
-          this.emit('request', parsedEvent);
-        } else {
-          this.emit(`response-${parsedEvent.id}`, parsedEvent);
+      cacheUsage: NDKSubscriptionCacheUsage.ONLY_RELAY,
+      onEvent: async (event: NDKEvent) => {
+        try {
+          const parsedEvent = await this.parseEvent(event);
+          if ((parsedEvent as NDKRpcRequest).method) {
+            this.emit('request', parsedEvent);
+          } else {
+            this.emit(`response-${parsedEvent.id}`, parsedEvent);
+          }
+        } catch (e) {
+          console.error('error parsing event in subscription', e);
         }
-      } catch (e) {
-        console.error('error parsing event in subscription', e);
-      }
+      },
     });
 
     this.sub = sub;
     return sub;
   }
 
+  /**
+   * subscription が無ければ作る。localSigner の pubkey で kind:24133 を購読。
+   */
+  public async ensureSubscription(): Promise<void> {
+    if (this.sub) return;
+    const pubkey = this._signer.pubkey;
+    console.log('ensureSubscription: subscribing for', pubkey);
+    await this.subscribe({
+      'kinds': [24133],
+      '#p': [pubkey],
+    });
+  }
+
   public stop() {
     if (this.sub) {
       this.sub.stop();
@@ -147,23 +162,27 @@ class NostrRpc extends NDKNostrRpc {
     }
   }
 
-  // 修正: listen メソッドの改善
+  /**
+   * subscriptionを開始してから connect response を待つ。
+   * listen 後もsubscriptionは維持する（後続の get_public_key 等で必要）。
+   */
   public async listen(nostrConnectSecret: string): Promise<string> {
     const pubkey = this._signer.pubkey;
     console.log('nostr-login listening for conn to', pubkey, 'expecting secret:', nostrConnectSecret);
 
-    const sub = await this.subscribe({
-      'kinds': [24133],
-      '#p': [pubkey],
-    });
+    await this.ensureSubscription();
 
     return new Promise<string>((ok, err) => {
       const timeout = setTimeout(() => {
-        this.stop();
         err(new Error('Connection timeout: no response from signer'));
       }, 60000); // 60秒のタイムアウト
 
-      sub.on('event', async (event: NDKEvent) => {
+      // subscribe の onEvent ハンドラが response-${id} を emit するが、
+      // listen のレスポンスは request id が無い（unsolicited）ので
+      // 'request' イベントとして来る可能性がある。
+      // 代わりに subscribe 側でパースされたイベントを
+      // 直接 NDKSubscription の event で受け取る。
+      const handler = async (event: NDKEvent) => {
         try {
           const parsedEvent = await this.parseEvent(event);
           console.log('listen parsedEvent', parsedEvent);
@@ -180,36 +199,41 @@ class NostrRpc extends NDKNostrRpc {
             // secretの厳密な検証
             if (response.result === nostrConnectSecret) {
               clearTimeout(timeout);
-              this.stop();
+              // subscriptionは維持する（後続リクエストで使うため stop しない）
               console.log('Connection established with signer:', event.pubkey);
               ok(event.pubkey);
             } else if (response.result === 'ack') {
               // ackは古い実装用の互換性のため警告のみ
               console.warn('Received "ack" instead of secret. This may indicate an older signer implementation.');
               clearTimeout(timeout);
-              this.stop();
               ok(event.pubkey);
             } else {
               console.error('Invalid response:', response);
               clearTimeout(timeout);
-              this.stop();
               err(new Error(response.error || 'Invalid connection response'));
             }
           }
         } catch (e) {
           console.error('Error parsing event in listen', e, event.rawEvent());
         }
-      });
+      };
 
-      sub.on('eose', () => {
-        console.log('EOSE received in listen');
-      });
+      if (this.sub) {
+        this.sub.on('event', handler);
+      }
     });
   }
 
+  /**
+   * subscriptionが無ければ開始してから connect リクエストを送る。
+   * レスポンスは subscription の event ハンドラ経由で受け取る。
+   */
   public async connect(pubkey: string, token?: string, perms?: string) {
     console.log('Sending connect request to', pubkey, 'with perms:', perms);
 
+    // connect のレスポンスを受け取るために subscription が必要
+    await this.ensureSubscription();
+
     return new Promise<void>((ok, err) => {
       const timeout = setTimeout(() => {
         err(new Error('Connect timeout: no response from signer'));