@konemono/nostr-login 1.11.0 → 1.11.1

package/dist/utils/index.d.ts

@@ -1,15 +1,12 @@
 import { Info, RecentType } from 'nostr-login-components/dist/types/types';
-import { RxNostr } from 'rx-nostr';
+import NDK, { NDKSigner } from '@nostr-dev-kit/ndk';
 import { NostrLoginOptions } from '../types';
-import { PrivateKeySigner } from '../modules/Signer';
-export declare function bytesToHex(bytes: Uint8Array): string;
-export declare function hexToBytes(hex: string): Uint8Array;
 export declare const localStorageSetItem: (key: string, value: string) => void;
 export declare const localStorageGetItem: (key: string) => any;
 export declare const localStorageRemoveItem: (key: string) => void;
-export declare const fetchProfile: (info: Info, rxNostr: RxNostr) => Promise<any>;
+export declare const fetchProfile: (info: Info, profileNdk: NDK) => Promise<import("@nostr-dev-kit/ndk").NDKUserProfile | null>;
 export declare const prepareSignupRelays: (signupRelays?: string) => string[];
-export declare const createProfile: (info: Info, rxNostr: RxNostr, signer: PrivateKeySigner, signupRelays?: string, outboxRelays?: string[]) => Promise<void>;
+export declare const createProfile: (info: Info, profileNdk: NDK, signer: NDKSigner, signupRelays?: string, outboxRelays?: string[]) => Promise<void>;
 export declare const bunkerUrlToInfo: (bunkerUrl: string, sk?: string) => Info;
 export declare const isBunkerUrl: (value: string) => boolean;
 export declare const getBunkerUrl: (value: string, optionsModal: NostrLoginOptions) => Promise<string>;

package/dist/utils/nip44.d.ts

@@ -2,8 +2,8 @@ export declare function encryptNip44(plaintext: string, conversationKey: Uint8Ar
 export declare function decryptNip44(payload: string, conversationKey: Uint8Array): string;
 export declare class Nip44 {
     private cache;
-    createKey(privkey: string | Uint8Array, pubkey: string): Uint8Array;
+    createKey(privkey: string, pubkey: string): Uint8Array;
     private getKey;
-    encrypt(privkey: string | Uint8Array, pubkey: string, text: string): string;
-    decrypt(privkey: string | Uint8Array, pubkey: string, data: string): string;
+    encrypt(privkey: string, pubkey: string, text: string): string;
+    decrypt(privkey: string, pubkey: string, data: string): string;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@konemono/nostr-login",
-  "version": "1.11.0",
+  "version": "1.11.1",
   "description": "",
   "main": "./dist/index.esm.js",
   "types": "./dist/index.d.ts",
@@ -11,17 +11,17 @@
   },
   "author": "a-fralou",
   "dependencies": {
-    "nostr-tools": "^2.19.4",
-    "rx-nostr": "^3.6.2",
-    "tseep": "^1.3.1"
+    "@nostr-dev-kit/ndk": "^2.3.1",
+    "nostr-tools": "^1.17.0",
+    "tseep": "^1.2.1"
   },
   "devDependencies": {
-    "@rollup/plugin-commonjs": "^29.0.0",
-    "@rollup/plugin-node-resolve": "^16.0.3",
+    "@rollup/plugin-commonjs": "^25.0.7",
+    "@rollup/plugin-node-resolve": "^15.2.3",
     "@rollup/plugin-terser": "^0.4.4",
     "nostr-login-components": "^1.0.3",
-    "prettier": "^3.7.4",
-    "rollup": "^4.55.1",
+    "prettier": "^3.2.2",
+    "rollup": "^4.9.6",
     "rollup-plugin-typescript2": "^0.36.0"
   },
   "license": "MIT"

package/src/modules/AuthNostrService.ts

@@ -1,21 +1,16 @@
-import { localStorageAddAccount, bunkerUrlToInfo, isBunkerUrl, fetchProfile, getBunkerUrl, localStorageRemoveCurrentAccount, createProfile, getIcon, bytesToHex, hexToBytes } from '../utils';
+import { localStorageAddAccount, bunkerUrlToInfo, isBunkerUrl, fetchProfile, getBunkerUrl, localStorageRemoveCurrentAccount, createProfile, getIcon } from '../utils';
 import { ConnectionString, Info } from 'nostr-login-components/dist/types/types';
-import { createRxNostr, RxNostr } from 'rx-nostr';
 import { generateSecretKey, getEventHash, getPublicKey, nip19 } from 'nostr-tools';
 import { NostrLoginAuthOptions, Response } from '../types';
-
-
-
-// Placeholder types for SimplePool transition
-// private pool: SimplePool;
-// private outboxPool: SimplePool;
+import NDK, { NDKEvent, NDKNip46Signer, NDKRpcResponse, NDKUser, NostrEvent } from '@nostr-dev-kit/ndk';
 import { NostrParams } from './';
 import { EventEmitter } from 'tseep';
 import { Signer } from './Nostr';
 import { Nip44 } from '../utils/nip44';
-import { IframeNostrRpc, Nip46Signer, ReadyListener } from './Nip46';
+import { IframeNostrRpc, Nip46Signer, ReadyListener, ensureNIP46Connection } from './Nip46';
 import { PrivateKeySigner } from './Signer';
 import { DEFAULT_NIP46_RELAYS } from '../const';
+import { bytesToHex, hexToBytes } from 'nostr-tools/lib/types/utils';
 
 const OUTBOX_RELAYS = ['wss://user.kindpag.es', 'wss://purplepag.es', 'wss://relay.nos.social'];
 const NOSTRCONNECT_APPS: ConnectionString[] = [
@@ -39,7 +34,8 @@ const NOSTRCONNECT_APPS: ConnectionString[] = [
 ];
 
 class AuthNostrService extends EventEmitter implements Signer {
-  private rxNostr: RxNostr;
+  private ndk: NDK;
+  private profileNdk: NDK;
   private signer: Nip46Signer | null = null;
   private localSigner: PrivateKeySigner | null = null;
   private params: NostrParams;
@@ -66,11 +62,17 @@ class AuthNostrService extends EventEmitter implements Signer {
   constructor(params: NostrParams) {
     super();
     this.params = params;
+    this.ndk = new NDK({
+      enableOutboxModel: false,
+      autoConnectUserRelays: false,
+      autoFetchUserMutelist: false,
+    });
 
-    this.rxNostr = createRxNostr({} as any);
-    // Default relays for initial connection if needed, though they are usually set per-action
-    // @ts-ignore
-    this.rxNostr.switchRelays(DEFAULT_NIP46_RELAYS);
+    this.profileNdk = new NDK({
+      enableOutboxModel: true,
+      explicitRelayUrls: OUTBOX_RELAYS,
+    });
+    this.profileNdk.connect();
 
     this.nip04 = {
       encrypt: this.encrypt04.bind(this),
@@ -82,11 +84,11 @@ class AuthNostrService extends EventEmitter implements Signer {
     };
   }
 
-  public isIframe(): boolean {
+  public isIframe() {
     return !!this.iframe;
   }
 
-  public async waitReady(): Promise<void> {
+  public async waitReady() {
     if (this.signerPromise) {
       try {
         await this.signerPromise;
@@ -100,12 +102,12 @@ class AuthNostrService extends EventEmitter implements Signer {
     }
   }
 
-  public cancelNostrConnect(): void {
+  public cancelNostrConnect() {
     this.releaseSigner();
     this.resetAuth();
   }
 
-  public cancelSignerInit(): void {
+  public cancelSignerInit() {
     if (this.signerAbortController) {
       this.signerAbortController.abort();
       this.signerAbortController = undefined;
@@ -134,7 +136,7 @@ class AuthNostrService extends EventEmitter implements Signer {
       iframeUrl?: string;
       customRelays?: string[];
     } = {},
-  ): Promise<Info> {
+  ) {
     // カスタムリレーが指定されていれば使用、そうでなければ単一リレーまたはデフォルト
     const relays = customRelays && customRelays.length > 0 ? customRelays : relay ? [relay] : DEFAULT_NIP46_RELAYS;
 
@@ -169,13 +171,11 @@ class AuthNostrService extends EventEmitter implements Signer {
     return info;
   }
 
-  public async createNostrConnect(): Promise<string> {
-    const skBytes = generateSecretKey();
-    // @ts-ignore
-    this.nostrConnectKey = bytesToHex(skBytes);
+  public async createNostrConnect() {
+    this.nostrConnectKey = bytesToHex(generateSecretKey());
     this.nostrConnectSecret = Math.random().toString(36).substring(7);
 
-    const pubkey = getPublicKey(skBytes);
+    const pubkey = getPublicKey(hexToBytes(this.nostrConnectKey));
     const meta = {
       name: encodeURIComponent(document.location.host),
       url: encodeURIComponent(document.location.origin),
@@ -232,14 +232,10 @@ class AuthNostrService extends EventEmitter implements Signer {
     return [nostrconnect, apps];
   }
 
-  public async localSignup(name: string, sk?: string): Promise<void> {
+  public async localSignup(name: string, sk?: string) {
     const signup = !sk;
-    if (!sk) {
-      const skBytes = generateSecretKey();
-      sk = bytesToHex(skBytes);
-    }
-    // @ts-ignore
-    const pubkey = getPublicKey(hexToBytes(sk!));
+    sk = sk || bytesToHex(generateSecretKey());
+    const pubkey = getPublicKey(hexToBytes(sk));
     const info: Info = {
       pubkey,
       sk,
@@ -250,118 +246,25 @@ class AuthNostrService extends EventEmitter implements Signer {
     await this.setLocal(info, signup);
   }
 
-  public async setLocal(info: Info, signup?: boolean): Promise<void> {
+  public async setLocal(info: Info, signup?: boolean) {
     this.releaseSigner();
-    this.localSigner = new PrivateKeySigner(hexToBytes(info.sk!));
+    this.localSigner = new PrivateKeySigner(info.sk!);
 
-    // TODO: nostr-toolsを使用するようにcreateProfileをリファクタリングする
-    if (signup) {
-      await createProfile(info, this.rxNostr, this.localSigner, this.params.optionsModal.signupRelays, this.params.optionsModal.outboxRelays);
-    }
+    if (signup) await createProfile(info, this.profileNdk, this.localSigner, this.params.optionsModal.signupRelays, this.params.optionsModal.outboxRelays);
 
     this.onAuth(signup ? 'signup' : 'login', info);
   }
 
-  public prepareImportUrl(url: string): string {
+  public prepareImportUrl(url: string) {
     // for OTP we choose interactive import
     if (this.params.userInfo?.authMethod === 'otp') return url + '&import=true';
 
     // for local we export our existing key
     if (!this.localSigner || this.params.userInfo?.authMethod !== 'local') throw new Error('Most be local keys');
-    // Ensure privateKey is Uint8Array for nsecEncode if needed, but nip19.nsecEncode usually takes Uint8Array
-    const skBytes = this.localSigner.privateKey!;
-    return url + '#import=' + nip19.nsecEncode(skBytes);
+    return url + '#import=' + nip19.nsecEncode(hexToBytes(this.localSigner.privateKey!));
   }
 
-  // リレー管理メソッド
-  public getRelayStatus(): Record<string, string> {
-    const status = this.rxNostr.getAllRelayStatus();
-    const result: Record<string, string> = {};
-    for (const [url, state] of Object.entries(status)) {
-      result[url] = String(state);
-    }
-    return result;
-  }
-
-  public getRelayStats(): {
-    total: number;
-    connected: number;
-    connecting: number;
-    disconnected: number;
-    error: number;
-  } {
-    const status = this.rxNostr.getAllRelayStatus();
-    const stats = {
-      total: 0,
-      connected: 0,
-      connecting: 0,
-      disconnected: 0,
-      error: 0,
-    };
-
-    for (const state of Object.values(status)) {
-      stats.total++;
-      const stateStr = String(state).toLowerCase();
-      if (stateStr === 'connected') stats.connected++;
-      else if (stateStr === 'connecting') stats.connecting++;
-      else if (stateStr === 'error') stats.error++;
-      else stats.disconnected++;
-    }
-
-    return stats;
-  }
-
-  public async connectToRelay(relayUrl: string): Promise<void> {
-    try {
-      this.rxNostr.reconnect(relayUrl);
-      console.log(`Connecting to relay: ${relayUrl}`);
-    } catch (error) {
-      console.error(`Failed to connect to relay ${relayUrl}:`, error);
-      throw error;
-    }
-  }
-
-  public async disconnectFromRelay(relayUrl: string): Promise<void> {
-    try {
-      // rx-nostrにはdisconnect APIがないため、リレーリストから除外する方法を使用
-      const currentRelays = Object.keys(this.rxNostr.getAllRelayStatus());
-      const newRelays = currentRelays.filter(url => url !== relayUrl);
-      if (newRelays.length > 0) {
-        // @ts-ignore
-        this.rxNostr.switchRelays(newRelays);
-      }
-      console.log(`Disconnected from relay: ${relayUrl}`);
-    } catch (error) {
-      console.error(`Failed to disconnect from relay ${relayUrl}:`, error);
-      throw error;
-    }
-  }
-
-  public async reconnectAllRelays(): Promise<void> {
-    try {
-      const relays = Object.keys(this.rxNostr.getAllRelayStatus());
-      console.log(`Reconnecting to ${relays.length} relays...`);
-
-      for (const relay of relays) {
-        try {
-          this.rxNostr.reconnect(relay);
-        } catch (error) {
-          console.warn(`Failed to reconnect to ${relay}:`, error);
-        }
-      }
-    } catch (error) {
-      console.error('Failed to reconnect all relays:', error);
-      throw error;
-    }
-  }
-
-  public async checkRelayHealth(): Promise<boolean> {
-    const stats = this.getRelayStats();
-    // 少なくとも1つのリレーが接続されていればOK
-    return stats.connected > 0;
-  }
-
-  public async importAndConnect(cs: ConnectionString): Promise<void> {
+  public async importAndConnect(cs: ConnectionString) {
     const { relay, domain, link, iframeUrl } = cs;
     if (!domain) throw new Error('Domain required');
 
@@ -378,22 +281,22 @@ class AuthNostrService extends EventEmitter implements Signer {
     this.onAuth('login', info);
   }
 
-  public setReadOnly(pubkey: string): void {
+  public setReadOnly(pubkey: string) {
     const info: Info = { pubkey, authMethod: 'readOnly' };
     this.onAuth('login', info);
   }
 
-  public setExtension(pubkey: string): void {
+  public setExtension(pubkey: string) {
     const info: Info = { pubkey, authMethod: 'extension' };
     this.onAuth('login', info);
   }
 
-  public setOTP(pubkey: string, data: string): void {
+  public setOTP(pubkey: string, data: string) {
     const info: Info = { pubkey, authMethod: 'otp', otpData: data };
     this.onAuth('login', info);
   }
 
-  public async setConnect(info: Info): Promise<void> {
+  public async setConnect(info: Info) {
     this.releaseSigner();
     await this.startAuth();
     await this.initSigner(info);
@@ -401,7 +304,7 @@ class AuthNostrService extends EventEmitter implements Signer {
     await this.endAuth();
   }
 
-  public async createAccount(nip05: string): Promise<{ bunkerUrl: string; sk: string | undefined }> {
+  public async createAccount(nip05: string) {
     const [name, domain] = nip05.split('@');
 
     // bunker's own url
@@ -425,19 +328,18 @@ class AuthNostrService extends EventEmitter implements Signer {
     };
   }
 
-  private releaseSigner(): void {
+  private releaseSigner() {
     this.signer = null;
     this.signerErrCallback?.('cancelled');
     this.localSigner = null;
 
     // disconnect from signer relays
-    // rx-nostr manages relays, we can switch to empty or default if needed
-    // or just leave them be as they might be reused?
-    // For now, let's just clear the signer instance.
-    // this.rxNostr.switchRelays([]); // Should we disconnect? Maybe better to keep connection for next signer?
+    for (const r of this.ndk.pool.relays.keys()) {
+      this.ndk.pool.removeRelay(r);
+    }
   }
 
-  public async logout(keepSigner = false): Promise<void> {
+  public async logout(keepSigner = false) {
     if (!keepSigner) this.releaseSigner();
 
     // move current to recent
@@ -449,7 +351,7 @@ class AuthNostrService extends EventEmitter implements Signer {
     this.emit('updateAccounts');
   }
 
-  private setUserInfo(userInfo: Info | null): void {
+  private setUserInfo(userInfo: Info | null) {
     this.params.userInfo = userInfo;
     this.emit('onUserInfo', userInfo);
 
@@ -459,14 +361,13 @@ class AuthNostrService extends EventEmitter implements Signer {
     }
   }
 
-  public exportKeys(): string {
+  public exportKeys() {
     if (!this.params.userInfo) return '';
     if (this.params.userInfo.authMethod !== 'local') return '';
-    const skBytes = hexToBytes(this.params.userInfo.sk!);
-    return nip19.nsecEncode(skBytes);
+    return nip19.nsecEncode(hexToBytes(this.params.userInfo.sk!));
   }
 
-  private onAuth(type: 'login' | 'signup' | 'logout', info: Info | null = null): void {
+  private onAuth(type: 'login' | 'signup' | 'logout', info: Info | null = null) {
     if (type !== 'logout' && !info) throw new Error('No user info in onAuth');
 
     // make sure we emulate logout first
@@ -480,8 +381,7 @@ class AuthNostrService extends EventEmitter implements Signer {
 
     if (info) {
       // async profile fetch
-      // TODO: nostr-toolsを使用するようにfetchProfileをリファクタリングする
-      fetchProfile(info, this.rxNostr as any).then(p => {
+      fetchProfile(info, this.profileNdk).then(p => {
         if (this.params.userInfo !== info) return;
 
         const userInfo = {
@@ -540,7 +440,7 @@ class AuthNostrService extends EventEmitter implements Signer {
     }
   }
 
-  private async createIframe(iframeUrl?: string): Promise<{ iframe: HTMLIFrameElement; port: MessagePort } | undefined> {
+  private async createIframe(iframeUrl?: string) {
     if (!iframeUrl) return undefined;
 
     // ensure iframe
@@ -599,18 +499,18 @@ class AuthNostrService extends EventEmitter implements Signer {
   //   }
   // }
 
-  public async sendNeedAuth(): Promise<void> {
+  public async sendNeedAuth() {
     const [nostrconnect] = await this.getNostrConnectServices();
     const event = new CustomEvent('nlNeedAuth', { detail: { nostrconnect } });
     console.log('nostr-login need auth', nostrconnect);
     document.dispatchEvent(event);
   }
 
-  public isAuthing(): boolean {
+  public isAuthing() {
     return !!this.readyCallback;
   }
 
-  public async startAuth(): Promise<void> {
+  public async startAuth() {
     console.log('startAuth');
     if (this.readyCallback) throw new Error('Already started');
 
@@ -618,7 +518,7 @@ class AuthNostrService extends EventEmitter implements Signer {
     this.readyPromise = new Promise<void>(ok => (this.readyCallback = ok));
   }
 
-  public async endAuth(): Promise<void> {
+  public async endAuth() {
     console.log('endAuth', this.params.userInfo);
     if (this.params.userInfo && this.params.userInfo.iframeUrl) {
       // create iframe
@@ -634,23 +534,23 @@ class AuthNostrService extends EventEmitter implements Signer {
     this.readyCallback = undefined;
   }
 
-  public resetAuth(): void {
+  public resetAuth() {
     if (this.readyCallback) this.readyCallback();
     this.readyCallback = undefined;
   }
 
-  private async listen(info: Info): Promise<void> {
+  private async listen(info: Info) {
     if (!info.iframeUrl) return this.signer!.listen(this.nostrConnectSecret);
     const r = await this.starterReady!.wait();
     if (r[0] === 'starterError') throw new Error(r[1]);
     return this.signer!.setListenReply(r[1], this.nostrConnectSecret);
   }
 
-  public async connect(info: Info, perms?: string): Promise<void> {
+  public async connect(info: Info, perms?: string) {
     return this.signer!.connect(info.token, perms);
   }
 
-  public async initSigner(info: Info, { listen = false, connect = false, eventToAddAccount = false } = {}): Promise<void> {
+  public async initSigner(info: Info, { listen = false, connect = false, eventToAddAccount = false } = {}) {
     // mutex
     if (this.signerPromise) {
       try {
@@ -696,28 +596,23 @@ class AuthNostrService extends EventEmitter implements Signer {
     return this.signerPromise;
   }
 
-  private async initSignerInternal(info: Info, listen: boolean, connect: boolean, eventToAddAccount: boolean, resolve: () => void): Promise<void> {
+  private async initSignerInternal(info: Info, listen: boolean, connect: boolean, eventToAddAccount: boolean, resolve: () => void) {
     // pre-connect if we're creating the connection (listen|connect) or
     // not iframe mode
     if (info.relays && !info.iframeUrl) {
-      // @ts-ignore
-      this.rxNostr.switchRelays(info.relays);
+      for (const r of info.relays) {
+        this.ndk.addExplicitRelay(r, undefined);
+      }
     }
 
-    // rx-nostr connects automatically
+    // wait until we connect, otherwise
+    // signer won't start properly
+    // NOTE: Deferred connection. ensureNIP46Connection will be called on demand.
+    // await this.ndk.connect();
 
     // create and prepare the signer
-    const localSigner = new PrivateKeySigner(hexToBytes(info.sk!));
-
-    // TODO: 元のNDKロジック:
-    // This initialized the Nip46Signer with NDK instance.
-    //
-    // New Implementation Plan:
-    // 1. Pass the SimplePool instance or relay list to Nip46Signer.
-    // 2. Nip46Signer should manage its own connection or use the shared pool to publish/subscribe.
-
-    // TODO: ndk引数を削除する
-    this.signer = new Nip46Signer(this.rxNostr, localSigner, info.signerPubkey!, info.iframeUrl ? new URL(info.iframeUrl!).origin : undefined);
+    const localSigner = new PrivateKeySigner(info.sk!);
+    this.signer = new Nip46Signer(this.ndk, localSigner, info.signerPubkey!, info.iframeUrl ? new URL(info.iframeUrl!).origin : undefined);
 
     // we should notify the banner the same way as
     // the onAuthUrl does
@@ -769,7 +664,7 @@ class AuthNostrService extends EventEmitter implements Signer {
       iframeUrl = '',
       customRelays,
     }: { name: string; bunkerUrl: string; sk?: string; domain?: string; iframeUrl?: string; customRelays?: string[] },
-  ): Promise<void> {
+  ) {
     try {
       const info = bunkerUrlToInfo(bunkerUrl, sk);
       if (isBunkerUrl(name)) info.bunkerUrl = name;
@@ -806,15 +701,17 @@ class AuthNostrService extends EventEmitter implements Signer {
     }
   }
 
-  public async signEvent(event: any): Promise<any> {
+  public async signEvent(event: any) {
     if (this.localSigner) {
-      // this.localSigner.privateKey is Uint8Array
-      event.pubkey = getPublicKey(this.localSigner.privateKey!);
+      event.pubkey = getPublicKey(hexToBytes(this.localSigner.privateKey!));
       event.id = getEventHash(event);
       event.sig = await this.localSigner.sign(event);
     } else {
       await this.ensureSigner();
 
+      // 署名前に接続を確認 (rpc.sendRequest内でも行われるが、安全のため)
+      await ensureNIP46Connection(this.ndk, 5000);
+
       event.pubkey = this.signer!.remotePubkey;
       event.id = getEventHash(event);
       event.sig = await this.signer!.sign(event);
@@ -823,89 +720,72 @@ class AuthNostrService extends EventEmitter implements Signer {
     return event;
   }
 
-  private async ensureSigner(): Promise<void> {
+  private async ensureSigner() {
     // signerがキャンセル等で破棄されている場合は再初期化
     if (!this.signer && this.params.userInfo) {
       console.log('Signer was destroyed, reinitializing...');
       await this.initSigner(this.params.userInfo);
-      return; // initSignerで接続も行われるので終了
+      return;
     }
 
     if (!this.signer) {
       throw new Error('No signer available');
     }
 
-    // リレー接続を確認・再接続
-    // TODO: 元のNDKロジック:
-    // Checks ndk.pool.stats() for connectivity.
-    //
-    // New Implementation Plan:
-    // 1. Iterate through SimplePool.relays or listConnectionStatus().
-    // 2. If no relays are connected, modify code to reconnect.
-
-    // TODO: rx-nostrの統計を確認する
-    // const stats = this.pool.listConnectionStatus();
-    // const stats = this.pool.listConnectionStatus();
-    // const stats = this.ndk.pool.stats();
-    // console.log('NDK pool stats:', stats);
-    const relayStates = this.rxNostr.getAllRelayStatus();
-    const connectedCount = Object.values(relayStates).filter(s => String(s) === 'connected').length;
-
-    if (connectedCount === 0) {
-      console.log('RxNostr relays disconnected, reinitializing signer...');
-
-      // リレーが完全に切断されている場合、signerも再初期化する必要がある
-      // （RPCサブスクリプションも切断されているため）
-      if (this.params.userInfo) {
-        // 古いsignerを破棄
-        this.signer = null;
-
-        // 既存のリレーを一度切断 (rx-nostr reconnects automatically, but we might want to force it)
-        this.rxNostr.reconnect(Object.keys(relayStates)[0]);
-
-        // signerを再初期化（リレー接続も含む）
-        await this.initSigner(this.params.userInfo);
-      } else {
-        throw new Error('Cannot reconnect: no user info');
-      }
-    }
+    // 接続確立は署名時またはRPCリクエスト時に行われるため、
+    // ここでの冗長な接続チェックは削除
   }
 
+  private async codec_call(method: string, pubkey: string, param: string) {
+    return new Promise<string>((resolve, reject) => {
+      this.signer!.rpc.sendRequest(this.signer!.remotePubkey!, method, [pubkey, param], 24133, (response: NDKRpcResponse) => {
+        if (!response.error) {
+          resolve(response.result);
+        } else {
+          reject(response.error);
+        }
+      });
+    });
+  }
 
-
-  public async encrypt04(pubkey: string, plaintext: string): Promise<string> {
+  public async encrypt04(pubkey: string, plaintext: string) {
     if (this.localSigner) {
-      return this.localSigner.encrypt(pubkey as any, plaintext);
+      return this.localSigner.encrypt(new NDKUser({ pubkey }), plaintext);
     } else {
       await this.ensureSigner();
-      return this.signer!.encrypt(pubkey, plaintext);
+      return this.signer!.encrypt(new NDKUser({ pubkey }), plaintext);
     }
   }
 
-  public async decrypt04(pubkey: string, ciphertext: string): Promise<string> {
+  public async decrypt04(pubkey: string, ciphertext: string) {
     if (this.localSigner) {
-      return this.localSigner.decrypt(pubkey as any, ciphertext);
+      return this.localSigner.decrypt(new NDKUser({ pubkey }), ciphertext);
     } else {
+      // decrypt is broken in ndk v2.3.1, and latest
+      // ndk v2.8.1 doesn't allow to override connect easily,
+      // so we reimplement and fix decrypt here as a temporary fix
       await this.ensureSigner();
-      return this.signer!.decrypt(pubkey, ciphertext);
+      return this.codec_call('nip04_decrypt', pubkey, ciphertext);
     }
   }
 
-  public async encrypt44(pubkey: string, plaintext: string): Promise<string> {
+  public async encrypt44(pubkey: string, plaintext: string) {
     if (this.localSigner) {
       return this.nip44Codec.encrypt(this.localSigner.privateKey!, pubkey, plaintext);
     } else {
+      // no support of nip44 in ndk yet
       await this.ensureSigner();
-      return this.signer!.encryptNip44(pubkey, plaintext);
+      return this.codec_call('nip44_encrypt', pubkey, plaintext);
     }
   }
 
-  public async decrypt44(pubkey: string, ciphertext: string): Promise<string> {
+  public async decrypt44(pubkey: string, ciphertext: string) {
     if (this.localSigner) {
       return this.nip44Codec.decrypt(this.localSigner.privateKey!, pubkey, ciphertext);
     } else {
+      // no support of nip44 in ndk yet
       await this.ensureSigner();
-      return this.signer!.decryptNip44(pubkey, ciphertext);
+      return this.codec_call('nip44_decrypt', pubkey, ciphertext);
     }
   }
 }