@konemono/nostr-login 1.7.60 → 1.7.61

package/package.json

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

package/src/modules/AmberDirectSigner.ts

@@ -1,5 +1,7 @@
 import { Signer } from './Nostr';
 
+const PENDING_KEY = 'amber_pending_request';
+
 export class AmberDirectSigner implements Signer {
   private _pubkey: string = '';
   private static pendingResolves: Map<string, (result: any) => void> = new Map();
@@ -49,49 +51,47 @@ export class AmberDirectSigner implements Signer {
 
   private setPending(id: string, content: string, type: string) {
     const hash = this.getHash(content);
-    sessionStorage.setItem(`amber_pending_${id}`, JSON.stringify({ hash, type }));
+    const data = JSON.stringify({ id, type, hash, timestamp: Date.now() });
+    sessionStorage.setItem(`amber_pending_${id}`, data);
+    localStorage.setItem(PENDING_KEY, data); // Backup for reloads
   }
 
   public static resolvePending(id: string, type: string, result: string) {
-    console.log('AmberDirectSigner: resolvePending called', { id, type, pendingCount: this.pendingResolves.size });
+    console.log('AmberDirectSigner: resolvePending', { 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
+    // Fallback by type if only one pending of that type
     if (!resolve) {
-        console.log('AmberDirectSigner: No direct ID match for resolve, searching by type', { id, type });
+        console.log('AmberDirectSigner: Falling back to resolve by type', type);
         for (const [pendingId, pendingResolve] of this.pendingResolves.entries()) {
-            const pending = sessionStorage.getItem(`amber_pending_${pendingId}`);
+            const pending = sessionStorage.getItem(`amber_pending_${pendingId}`) || localStorage.getItem(PENDING_KEY);
             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;
-                }
+                try {
+                    const { type: pendingType } = JSON.parse(pending);
+                    if (pendingType === type || type.startsWith(pendingType)) {
+                        resolve = pendingResolve;
+                        id = pendingId;
+                        break;
+                    }
+                } catch (e) {}
             }
         }
     }
 
     if (resolve) {
-        console.log('AmberDirectSigner: Resolving pending promise', { id, type });
+        console.log('AmberDirectSigner: Promise resolved', id);
         resolve(result);
         this.pendingResolves.delete(id);
         return true;
     }
-    
-    console.log('AmberDirectSigner: No pending promise found to resolve', { id, type });
     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);
-    }
+    if (cached) return JSON.parse(cached);
 
     const id = Math.random().toString(36).substring(7);
     this.setPending(id, content, 'sign_event');
@@ -104,6 +104,7 @@ export class AmberDirectSigner implements Signer {
 
   async getPublicKey(): Promise<string> {
     const id = Math.random().toString(36).substring(7);
+    this.setPending(id, '', 'get_public_key');
     const url = this.generateUrl('', 'get_public_key', id);
     window.location.href = url;
     return new Promise((resolve) => {
@@ -166,35 +167,27 @@ 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
+    // Minimal parameters in callbackUrl to avoid conflicts
     callbackUrl.searchParams.set('amberType', type);
     callbackUrl.searchParams.set('amberId', id);
 
     const name = document.title || window.location.hostname || 'Nostr Login';
 
-    // 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(`pubkey=${encodeURIComponent(recipient)}`);
-        params.push(`pubKey=${encodeURIComponent(recipient)}`);
-        params.push(`current_user=${encodeURIComponent(recipient)}`);
     } else if (this._pubkey) {
         params.push(`pubkey=${encodeURIComponent(this._pubkey)}`);
-        params.push(`pubKey=${encodeURIComponent(this._pubkey)}`);
-        params.push(`current_user=${encodeURIComponent(this._pubkey)}`);
     }
 
-    const dataPart = content ? encodeURIComponent(content) : '';
-    // Format: nostrsigner:<content>?<params>
+    const dataPart = content ? encodeURIComponent(content) : 'get_public_key';
+    // Use nostrsigner: scheme as primary which is standard NIP-55
     return `nostrsigner:${dataPart}?${params.join('&')}`;
   }
 
@@ -203,23 +196,33 @@ export class AmberDirectSigner implements Signer {
     const params = new URLSearchParams(url.search);
     const hashParams = new URLSearchParams(url.hash.substring(1));
 
-    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');
+    let type = getParam('amberType') || getParam('type');
+    let id = getParam('amberId') || getParam('id');
     const result = getParam('signature') || getParam('result') || getParam('pubKey') || getParam('pubkey');
 
+    // Fallback to localStorage pending if params are missing/merged
+    if (result && (!type || !id)) {
+        const pending = localStorage.getItem(PENDING_KEY);
+        if (pending) {
+            try {
+                const { id: pId, type: pType } = JSON.parse(pending);
+                if (!type) type = pType;
+                if (!id) id = pId;
+                console.log('AmberDirectSigner: Recovered type/id from localStorage');
+            } catch (e) {}
+        }
+    }
+
     if (type && result) {
-        console.log('AmberDirectSigner: Response params detected', { type, id, resultLength: result.length });
-        
+        // Handle "ID stuck to type" or typo cases
+        let finalType = type.toLowerCase();
+        if (finalType.includes('get_pub')) finalType = 'get_public_key';
+        else if (finalType.includes('sign_event')) finalType = 'sign_event';
+        else if (finalType.includes('encrypt')) finalType = finalType.includes('44') ? 'nip44_encrypt' : 'nip04_encrypt';
+        else if (finalType.includes('decrypt')) finalType = finalType.includes('44') ? 'nip44_decrypt' : 'nip04_decrypt';
+
         const finalId = id || '';
 
         // Clean up URL
@@ -229,8 +232,10 @@ export class AmberDirectSigner implements Signer {
         });
         newUrl.hash = ''; 
         window.history.replaceState({}, '', newUrl.toString());
+        
+        localStorage.removeItem(PENDING_KEY); // Done
 
-        return { type, id: finalId, result };
+        return { type: finalType, id: finalId, result };
     }
     return null;
   }

package/src/modules/AuthNostrService.ts

@@ -108,13 +108,16 @@ class AuthNostrService extends EventEmitter implements Signer {
     if (response) {
       console.log('Amber response detected', response);
 
+      // Stop the "Connecting..." spinner
+      this.emit('onAuthUrl', { url: '' });
+
       // Resolve pending promises if any (for non-reload cases)
       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') {
+      if (response.type === 'get_public_key' || response.type.includes('pub')) {
         const info: Info = {
           pubkey: response.result,
           authMethod: 'amber' as any,