Improve Nostr connect UI

app/components/nostr-connect.gjs

@@ -7,6 +7,7 @@ import qrCode from '../modifiers/qr-code';
 
 export default class NostrConnectComponent extends Component {
   @service nostrAuth;
+  @service toast;
 
   get hasExtension() {
     return typeof window !== 'undefined' && typeof window.nostr !== 'undefined';
@@ -15,7 +16,8 @@ export default class NostrConnectComponent extends Component {
   @action
   async connectExtension() {
     try {
-      await this.nostrAuth.login('extension');
+      await this.nostrAuth.connectWithExtension();
+      this.toast.show('Nostr connected successfully');
       if (this.args.onConnect) {
         this.args.onConnect();
       }
@@ -28,7 +30,8 @@ export default class NostrConnectComponent extends Component {
   @action
   async connectApp() {
     try {
-      await this.nostrAuth.login('connect');
+      await this.nostrAuth.connectWithApp();
+      this.toast.show('Nostr connected successfully');
       if (this.args.onConnect) {
         this.args.onConnect();
       }
@@ -53,7 +56,7 @@ export default class NostrConnectComponent extends Component {
           </button>
         {{else}}
           <button
-            class="btn btn-secondary"
+            class="btn btn-outline"
             type="button"
             disabled
             title="No Nostr extension found in your browser."
@@ -72,7 +75,7 @@ export default class NostrConnectComponent extends Component {
       </div>
 
       {{#if (eq this.nostrAuth.connectStatus "waiting")}}
-        <div class="alert alert-info nostr-connect-status">
+        <div class="nostr-connect-status">
           {{#if this.nostrAuth.isMobile}}
             <p>Waiting for you to approve the connection in your mobile signer
               app...</p>

app/components/user-menu.gjs

@@ -48,7 +48,7 @@ export default class UserMenuComponent extends Component {
 
   @action
   disconnectNostr() {
-    this.nostrAuth.logout();
+    this.nostrAuth.disconnect();
   }
 
   <template>

app/services/nostr-auth.js

@@ -12,6 +12,8 @@ const STORAGE_KEY_CONNECT_LOCAL_KEY = 'marco:nostr_connect_local_key';
 const STORAGE_KEY_CONNECT_REMOTE_PUBKEY = 'marco:nostr_connect_remote_pubkey';
 const STORAGE_KEY_CONNECT_RELAY = 'marco:nostr_connect_relay';
 
+const DEFAULT_CONNECT_RELAY = 'wss://relay.nsec.app';
+
 export default class NostrAuthService extends Service {
   @service nostrRelay;
 
@@ -26,6 +28,12 @@ export default class NostrAuthService extends Service {
 
   constructor() {
     super(...arguments);
+
+    // Enable debug logging for applesauce packages
+    if (typeof localStorage !== 'undefined') {
+      localStorage.debug = 'applesauce:*';
+    }
+
     const saved = localStorage.getItem(STORAGE_KEY);
     const type = localStorage.getItem(STORAGE_KEY_TYPE);
     if (saved) {
@@ -38,7 +46,7 @@ export default class NostrAuthService extends Service {
   async _verifyPubkey() {
     if (this.signerType === 'extension') {
       if (typeof window.nostr === 'undefined') {
-        this.logout();
+        this.disconnect();
         return;
       }
       try {
@@ -50,14 +58,14 @@ export default class NostrAuthService extends Service {
         }
       } catch (e) {
         console.warn('Failed to verify extension nostr pubkey, logging out', e);
-        this.logout();
+        this.disconnect();
       }
     } else if (this.signerType === 'connect') {
       try {
         await this._initConnectSigner();
       } catch (e) {
         console.warn('Failed to verify connect nostr pubkey, logging out', e);
-        this.logout();
+        this.disconnect();
       }
     }
   }
@@ -93,8 +101,7 @@ export default class NostrAuthService extends Service {
     return null;
   }
 
-  async login(type = 'extension') {
+  async connectWithExtension() {
-    if (type === 'extension') {
     if (typeof window.nostr === 'undefined') {
       throw new Error('No NIP-07 Nostr extension found (e.g., Alby, nos2x).');
     }
@@ -110,11 +117,9 @@ export default class NostrAuthService extends Service {
       console.error('Failed to get public key from extension:', error);
       throw error;
     }
-    } else if (type === 'connect') {
+  }
-      this.connectStatus = 'waiting';
 
-      try {
+  _getLocalSigner() {
-        // Generate or retrieve a local ephemeral keypair
     let localKeyHex = localStorage.getItem(STORAGE_KEY_CONNECT_LOCAL_KEY);
     let localSigner;
     if (localKeyHex) {
@@ -127,11 +132,22 @@ export default class NostrAuthService extends Service {
         .join('');
       localStorage.setItem(STORAGE_KEY_CONNECT_LOCAL_KEY, localKeyHex);
     }
+    return localSigner;
+  }
+
+  async connectWithApp() {
+    this.connectStatus = 'waiting';
+
+    try {
+      const localSigner = this._getLocalSigner();
 
       // We use a specific relay for the connection handshake.
-        const relay = 'wss://relay.nsec.app';
+      const relay = DEFAULT_CONNECT_RELAY;
       localStorage.setItem(STORAGE_KEY_CONNECT_RELAY, relay);
 
+      // Override aggressive 10s EOSE timeout to allow time for QR scanning
+      this.nostrRelay.pool.relay(relay).eoseTimeout = 180000; // 3 minutes
+
       this._signerInstance = new NostrConnectSigner({
         pool: this.nostrRelay.pool,
         relays: [relay],
@@ -153,20 +169,30 @@ export default class NostrAuthService extends Service {
       this.connectUri = this._signerInstance.getNostrConnectURI({
         name: 'Marco',
         url: window.location.origin,
-          description: 'A privacy-respecting maps application.',
+        description: 'An unhosted maps application.',
         icons: [],
       });
 
       // Trigger the deep link intent immediately for the user if on mobile
       if (this.isMobile) {
+        console.debug('Mobile detected, triggering deep link intent.');
         window.location.href = this.connectUri;
       }
 
       // Start listening to the relay
+      console.debug('Opening signer connection to relay...');
       await this._signerInstance.open();
+      console.debug('Signer connection opened successfully.');
 
       // Wait for the remote signer to reply with their pubkey
+      console.debug('Waiting for remote signer to ack via relay...');
+      try {
         await this._signerInstance.waitForSigner();
+        console.debug('Remote signer ack received!');
+      } catch (waitErr) {
+        console.error('Error while waiting for remote signer ack:', waitErr);
+        throw waitErr;
+      }
 
       // Once connected, get the actual user pubkey
       this.pubkey = await this._signerInstance.getPublicKey();
@@ -188,21 +214,22 @@ export default class NostrAuthService extends Service {
       throw error;
     }
   }
-  }
 
   async _initConnectSigner() {
-    const localKeyHex = localStorage.getItem(STORAGE_KEY_CONNECT_LOCAL_KEY);
     const remotePubkey = localStorage.getItem(
       STORAGE_KEY_CONNECT_REMOTE_PUBKEY
     );
     const relay =
-      localStorage.getItem(STORAGE_KEY_CONNECT_RELAY) || 'wss://relay.nsec.app';
+      localStorage.getItem(STORAGE_KEY_CONNECT_RELAY) || DEFAULT_CONNECT_RELAY;
 
-    if (!localKeyHex || !remotePubkey) {
+    if (!remotePubkey) {
-      throw new Error('Missing Nostr Connect local state.');
+      throw new Error('Missing Nostr Connect remote pubkey.');
     }
 
-    const localSigner = PrivateKeySigner.fromKey(localKeyHex);
+    const localSigner = this._getLocalSigner();
+
+    // Override aggressive 10s EOSE timeout to allow time for QR scanning
+    this.nostrRelay.pool.relay(relay).eoseTimeout = 180000; // 3 minutes
 
     this._signerInstance = new NostrConnectSigner({
       pool: this.nostrRelay.pool,
@@ -237,7 +264,7 @@ export default class NostrAuthService extends Service {
     return await this.signer.signEvent(event);
   }
 
-  async logout() {
+  async disconnect() {
     this.pubkey = null;
     this.signerType = null;
     this.connectStatus = null;