fix(nostr): await nip-07 extension injection to resolve race conditions

Browse Source

This commit fixes a critical race condition where `window.nostr` might not be injected
before the app attempts to use it, causing failures in decryption (subscriptions, hashtags, DMs)
and login flows.

Changes:
- Updated `ensureExtensionPermissions` in `js/nostr/client.js` to verify `window.nostr` existence
  even when permissions are cached. If missing, it now awaits `waitForNip07Extension` instead of
  returning a false success.
- Updated `ensureActiveSignerForPubkey` in `js/nostr/client.js` to wait for the extension
  if permissions are cached but `window.nostr` is missing, ensuring a valid signer is created.

Verification:
- Confirmed `waitForNip07Extension` is imported in `js/nostr/client.js`.
- Verified `HashtagPreferencesService` already implements multi-scheme decryption and optimistic updates.
- Verified `SubscriptionsManager` implements multi-scheme decryption.
- The race condition fix ensures the signer is correctly initialized with capabilities, enabling the decryption logic to work.

Co-authored-by: PR0M3TH3AN <[email protected]>

...

This commit is contained in:

google-labs-jules[bot]andPR0M3TH3AN committed 2026-01-26 20:58:02 +00:00

1 parent a57947da0c

commit 1bb4d1ec85

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download Patch File Download Diff File Expand all files Collapse all files

Diff Content Not Available