@konemono/nostr-login 1.7.56 → 1.7.58

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@konemono/nostr-login",
-  "version": "1.7.56",
+  "version": "1.7.58",
   "description": "",
   "main": "./dist/index.esm.js",
   "types": "./dist/index.d.ts",

package/src/modules/AmberDirectSigner.ts

@@ -1,8 +1,8 @@
-import { NDKUser } from '@nostr-dev-kit/ndk';
 import { Signer } from './Nostr';
 
 export class AmberDirectSigner implements Signer {
   private _pubkey: string = '';
+  private static pendingResolves: Map<string, (result: any) => void> = new Map();
 
   constructor(pubkey?: string) {
     this._pubkey = pubkey || '';
@@ -29,9 +29,9 @@ export class AmberDirectSigner implements Signer {
   private getHash(content: string): string {
     let hash = 0;
     for (let i = 0; i < content.length; i++) {
-      const char = content.charCodeAt(i);
-      hash = (hash << 5) - hash + char;
-      hash |= 0;
+        const char = content.charCodeAt(i);
+        hash = (hash << 5) - hash + char;
+        hash |= 0;
     }
     return hash.toString(36);
   }
@@ -41,8 +41,8 @@ export class AmberDirectSigner implements Signer {
     const cacheKey = `amber_result_${type}_${hash}`;
     const cached = sessionStorage.getItem(cacheKey);
     if (cached) {
-      sessionStorage.removeItem(cacheKey);
-      return cached;
+        sessionStorage.removeItem(cacheKey);
+        return cached;
     }
     return null;
   }
@@ -52,19 +52,40 @@ export class AmberDirectSigner implements Signer {
     sessionStorage.setItem(`amber_pending_${id}`, JSON.stringify({ hash, type }));
   }
 
+  public static resolvePending(id: string, result: string) {
+    const resolve = this.pendingResolves.get(id);
+    if (resolve) {
+        console.log('Resolving pending Amber promise', id);
+        resolve(result);
+        this.pendingResolves.delete(id);
+        return true;
+    }
+    return false;
+  }
+
   async signEvent(event: any): Promise<any> {
     const content = JSON.stringify(event);
     const cached = this.checkCache(content, 'sign_event');
     if (cached) {
-      return JSON.parse(cached);
+        return JSON.parse(cached);
     }
 
     const id = Math.random().toString(36).substring(7);
     this.setPending(id, content, 'sign_event');
     const url = this.generateUrl(content, 'sign_event', id);
     window.location.href = url;
-    // This will never resolve because of page reload
-    return new Promise(() => {});
+    return new Promise((resolve) => {
+        AmberDirectSigner.pendingResolves.set(id, (res) => resolve(JSON.parse(res)));
+    });
+  }
+
+  async getPublicKey(): Promise<string> {
+    const id = Math.random().toString(36).substring(7);
+    const url = this.generateUrl('', 'get_public_key', id);
+    window.location.href = url;
+    return new Promise((resolve) => {
+        AmberDirectSigner.pendingResolves.set(id, resolve);
+    });
   }
 
   async encrypt04(pubkey: string, plaintext: string): Promise<string> {
@@ -75,7 +96,9 @@ export class AmberDirectSigner implements Signer {
     this.setPending(id, plaintext, 'nip04_encrypt');
     const url = this.generateUrl(plaintext, 'nip04_encrypt', id, pubkey);
     window.location.href = url;
-    return new Promise(() => {});
+    return new Promise((resolve) => {
+        AmberDirectSigner.pendingResolves.set(id, resolve);
+    });
   }
 
   async decrypt04(pubkey: string, ciphertext: string): Promise<string> {
@@ -86,7 +109,9 @@ export class AmberDirectSigner implements Signer {
     this.setPending(id, ciphertext, 'nip04_decrypt');
     const url = this.generateUrl(ciphertext, 'nip04_decrypt', id, pubkey);
     window.location.href = url;
-    return new Promise(() => {});
+    return new Promise((resolve) => {
+        AmberDirectSigner.pendingResolves.set(id, resolve);
+    });
   }
 
   async encrypt44(pubkey: string, plaintext: string): Promise<string> {
@@ -97,7 +122,9 @@ export class AmberDirectSigner implements Signer {
     this.setPending(id, plaintext, 'nip44_encrypt');
     const url = this.generateUrl(plaintext, 'nip44_encrypt', id, pubkey);
     window.location.href = url;
-    return new Promise(() => {});
+    return new Promise((resolve) => {
+        AmberDirectSigner.pendingResolves.set(id, resolve);
+    });
   }
 
   async decrypt44(pubkey: string, ciphertext: string): Promise<string> {
@@ -108,67 +135,59 @@ export class AmberDirectSigner implements Signer {
     this.setPending(id, ciphertext, 'nip44_decrypt');
     const url = this.generateUrl(ciphertext, 'nip44_decrypt', id, pubkey);
     window.location.href = url;
-    return new Promise(() => {});
-  }
-
-  public async getPublicKey(): Promise<string> {
-    const id = Math.random().toString(36).substring(7);
-    const url = this.generateUrl('', 'get_public_key', id);
-    window.location.href = url;
-    return new Promise(() => {});
+    return new Promise((resolve) => {
+        AmberDirectSigner.pendingResolves.set(id, resolve);
+    });
   }
 
   private generateUrl(content: string, type: string, id: string, recipient?: string): string {
-    const url = new URL(window.location.href);
-    const searchParams = url.searchParams;
-    searchParams.delete('amberType');
-    searchParams.delete('amberId');
-    searchParams.delete('signature');
-    searchParams.delete('result');
-    url.hash = ''; // Amber/Android may not handle hash in callbackUrl well
-
-    url.searchParams.set('amberType', type);
-    url.searchParams.set('amberId', id);
-
-    const params = new URLSearchParams();
-    params.set('type', type);
-    params.set('id', id);
-    params.set('callbackUrl', url.toString());
-    params.set('name', document.location.host);
+    const callbackUrl = new URL(window.location.origin + window.location.pathname);
+    callbackUrl.searchParams.set('amberType', type);
+    callbackUrl.searchParams.set('amberId', id);
+
+    const action = `com.greenart7c3.nostrsigner.${type.toUpperCase()}`;
+    const name = document.title || window.location.hostname || 'Nostr Login';
+
+    const params = [
+        `S.type=${encodeURIComponent(type)}`,
+        `S.name=${encodeURIComponent(name)}`,
+        `S.callbackUrl=${encodeURIComponent(callbackUrl.toString())}`,
+        `S.id=${encodeURIComponent(id)}`
+    ];
+
     if (recipient) {
-      params.set('pubkey', recipient);
+        params.push(`S.pubKey=${encodeURIComponent(recipient)}`);
     } else if (this._pubkey) {
-      params.set('pubkey', this._pubkey);
+        params.push(`S.pubKey=${encodeURIComponent(this._pubkey)}`);
     }
 
-    return `nostrsigner:${encodeURIComponent(content)}?${params.toString()}`;
+    if (type === 'sign_event') {
+        params.push(`S.event=${encodeURIComponent(content)}`);
+    } else {
+        params.push(`S.content=${encodeURIComponent(content)}`);
+    }
+
+    return `intent:#Intent;action=${action};package=com.greenart7c3.nostrsigner;${params.join(';')};end`;
   }
 
   public static parseResponse(): { type: string; id: string; result: string } | null {
-    const params = new URLSearchParams(window.location.search);
-    let type = params.get('amberType');
-    let id = params.get('amberId');
-    let result = params.get('signature') || params.get('result'); // Amber uses signature for events, result for others?
-
-    // Also check hash
-    if (!type || !id || !result) {
-      const hashParams = new URLSearchParams(window.location.hash.substring(1));
-      type = type || hashParams.get('amberType');
-      id = id || hashParams.get('amberId');
-      result = result || hashParams.get('signature') || hashParams.get('result');
-    }
+    const url = new URL(window.location.href);
+    const params = new URLSearchParams(url.search);
+    const hashParams = new URLSearchParams(url.hash.substring(1));
+
+    const type = params.get('amberType') || hashParams.get('amberType');
+    const id = params.get('amberId') || hashParams.get('amberId');
+    const result = params.get('signature') || params.get('result') || params.get('pubKey') || params.get('pubkey') ||
+        hashParams.get('signature') || hashParams.get('result') || hashParams.get('pubKey') || hashParams.get('pubkey');
 
     if (type && id && result) {
-      // Clean up URL
-      const newUrl = new URL(window.location.href);
-      newUrl.searchParams.delete('amberType');
-      newUrl.searchParams.delete('amberId');
-      newUrl.searchParams.delete('signature');
-      newUrl.searchParams.delete('result');
-      newUrl.hash = ''; // Clear hash too
-      window.history.replaceState({}, '', newUrl.toString());
-
-      return { type, id, result };
+        // Clean up URL
+        const newUrl = new URL(window.location.href);
+        ['amberType', 'amberId', 'signature', 'result', 'pubKey', 'pubkey'].forEach(p => newUrl.searchParams.delete(p));
+        newUrl.hash = ''; 
+        window.history.replaceState({}, '', newUrl.toString());
+
+        return { type, id, result };
     }
     return null;
   }

package/src/modules/AuthNostrService.ts

@@ -86,9 +86,10 @@ class AuthNostrService extends EventEmitter implements Signer {
       decrypt: this.decrypt44.bind(this),
     };
 
-    setTimeout(() => this.checkAmberResponse(), 500);
+    setTimeout(() => this.checkAmberResponse(), 100);
 
     window.addEventListener('focus', () => {
+      console.log('Window focused, checking Amber response');
       this.checkAmberResponse();
     });
   }
@@ -96,11 +97,17 @@ class AuthNostrService extends EventEmitter implements Signer {
   private checkAmberResponse() {
     const response = AmberDirectSigner.parseResponse();
     if (response) {
+      console.log('Amber response detected', response);
+
+      // Resolve pending promises if any (for non-reload cases)
+      AmberDirectSigner.resolvePending(response.id, response.result);
+
       if (response.type === 'get_public_key') {
         const info: Info = {
           pubkey: response.result,
           authMethod: 'amber' as any,
         };
+        console.log('Amber login success', info);
         this.onAuth('login', info);
       } else {
         const pendingKey = `amber_pending_${response.id}`;