@konemono/nostr-login 1.7.58 → 1.7.60

package/package.json

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

package/src/modules/AmberDirectSigner.ts

@@ -52,14 +52,37 @@ 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);
+  public static resolvePending(id: string, type: string, result: string) {
+    console.log('AmberDirectSigner: resolvePending called', { id, type, pendingCount: this.pendingResolves.size });
+
+    // Try resolving by ID first
+    let resolve = this.pendingResolves.get(id);
+    
+    // Fallback: If no ID match, try resolving by type if only one is pending of that type
+    if (!resolve) {
+        console.log('AmberDirectSigner: No direct ID match for resolve, searching by type', { id, type });
+        for (const [pendingId, pendingResolve] of this.pendingResolves.entries()) {
+            const pending = sessionStorage.getItem(`amber_pending_${pendingId}`);
+            if (pending) {
+                const { type: pendingType } = JSON.parse(pending);
+                if (pendingType === type) {
+                    resolve = pendingResolve;
+                    id = pendingId;
+                    console.log('AmberDirectSigner: Found match by type', { id, type });
+                    break;
+                }
+            }
+        }
+    }
+
     if (resolve) {
-        console.log('Resolving pending Amber promise', id);
+        console.log('AmberDirectSigner: Resolving pending promise', { id, type });
         resolve(result);
         this.pendingResolves.delete(id);
         return true;
     }
+    
+    console.log('AmberDirectSigner: No pending promise found to resolve', { id, type });
     return false;
   }
 
@@ -142,32 +165,37 @@ export class AmberDirectSigner implements Signer {
 
   private generateUrl(content: string, type: string, id: string, recipient?: string): string {
     const callbackUrl = new URL(window.location.origin + window.location.pathname);
+    
+    // Amber parameters we want to carry back
     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)}`
-    ];
+    // NIP-55 standard and some common extensions
+    const params: string[] = [];
+    params.push(`type=${encodeURIComponent(type)}`);
+    params.push(`id=${encodeURIComponent(id)}`);
+    params.push(`callbackUrl=${encodeURIComponent(callbackUrl.toString())}`);
+    
+    // Multi-alias for app name to fix the "null" issue
+    params.push(`name=${encodeURIComponent(name)}`);
+    params.push(`app_name=${encodeURIComponent(name)}`);
+    params.push(`title=${encodeURIComponent(name)}`);
 
     if (recipient) {
-        params.push(`S.pubKey=${encodeURIComponent(recipient)}`);
+        params.push(`pubkey=${encodeURIComponent(recipient)}`);
+        params.push(`pubKey=${encodeURIComponent(recipient)}`);
+        params.push(`current_user=${encodeURIComponent(recipient)}`);
     } else if (this._pubkey) {
-        params.push(`S.pubKey=${encodeURIComponent(this._pubkey)}`);
-    }
-
-    if (type === 'sign_event') {
-        params.push(`S.event=${encodeURIComponent(content)}`);
-    } else {
-        params.push(`S.content=${encodeURIComponent(content)}`);
+        params.push(`pubkey=${encodeURIComponent(this._pubkey)}`);
+        params.push(`pubKey=${encodeURIComponent(this._pubkey)}`);
+        params.push(`current_user=${encodeURIComponent(this._pubkey)}`);
     }
 
-    return `intent:#Intent;action=${action};package=com.greenart7c3.nostrsigner;${params.join(';')};end`;
+    const dataPart = content ? encodeURIComponent(content) : '';
+    // Format: nostrsigner:<content>?<params>
+    return `nostrsigner:${dataPart}?${params.join('&')}`;
   }
 
   public static parseResponse(): { type: string; id: string; result: string } | null {
@@ -175,19 +203,34 @@ export class AmberDirectSigner implements Signer {
     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 (url.search || url.hash) {
+        console.log('AmberDirectSigner: Checking URL for response', { 
+            search: url.search, 
+            hash: url.hash 
+        });
+    }
+
+    const getParam = (name: string) => params.get(name) || hashParams.get(name);
+
+    // Support both our custom names and original intent names
+    const type = getParam('amberType') || getParam('type');
+    const id = getParam('amberId') || getParam('id');
+    const result = getParam('signature') || getParam('result') || getParam('pubKey') || getParam('pubkey');
+
+    if (type && result) {
+        console.log('AmberDirectSigner: Response params detected', { type, id, resultLength: result.length });
+        
+        const finalId = id || '';
 
-    if (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));
+        ['amberType', 'amberId', 'signature', 'result', 'pubKey', 'pubkey', 'type', 'id'].forEach(p => {
+            newUrl.searchParams.delete(p);
+        });
         newUrl.hash = ''; 
         window.history.replaceState({}, '', newUrl.toString());
 
-        return { type, id, result };
+        return { type, id: finalId, result };
     }
     return null;
   }

package/src/modules/AuthNostrService.ts

@@ -88,10 +88,19 @@ class AuthNostrService extends EventEmitter implements Signer {
 
     setTimeout(() => this.checkAmberResponse(), 100);
 
-    window.addEventListener('focus', () => {
-      console.log('Window focused, checking Amber response');
+    const check = () => {
       this.checkAmberResponse();
+    };
+
+    window.addEventListener('focus', check);
+    window.addEventListener('visibilitychange', () => {
+        if (document.visibilityState === 'visible') check();
     });
+    window.addEventListener('popstate', check);
+    window.addEventListener('hashchange', check);
+
+    // Periodic check as a safety net
+    setInterval(check, 1000);
   }
 
   private checkAmberResponse() {
@@ -100,7 +109,10 @@ class AuthNostrService extends EventEmitter implements Signer {
       console.log('Amber response detected', response);
 
       // Resolve pending promises if any (for non-reload cases)
-      AmberDirectSigner.resolvePending(response.id, response.result);
+      const resolved = AmberDirectSigner.resolvePending(response.id, response.type, response.result);
+      if (resolved) {
+          console.log('Resolved pending Amber promise via resolvePending');
+      }
 
       if (response.type === 'get_public_key') {
         const info: Info = {
@@ -181,6 +193,8 @@ class AuthNostrService extends EventEmitter implements Signer {
     if (link && !iframeUrl) {
       if (link === 'amber') {
         const signer = new AmberDirectSigner();
+        const url = (signer as any).generateUrl('', 'get_public_key', Math.random().toString(36).substring(7));
+        this.emit('onAuthUrl', { url });
         return (signer as any).getPublicKey(); // will redirect
       }
       window.open(link, '_blank', 'width=400,height=700');