Replace hand-rolled ~350-entry emoji list with unicode-emoji-json (~1,900 emojis) and emojilib
keywords for richer search matching. Move EmojiSearchService to a singleton with Dexie-backed
caching for instant availability on startup. Add recency+frequency emoji usage tracking shared
across the emoji picker and editor autocomplete.
- Use AGGREGATOR_RELAYS as fallback for follows without kind:10002,
not the user's personal relays. Personal inbox/write relays were
being assigned as outbox for hundreds of unknown follows, inflating
counts and sending unnecessary queries to niche relays.
- Per-relay REQ badges now show assigned count (from reasoning) as
the primary number, with unassigned users shown dimmed as +N.
Tooltips show the full breakdown.
- Switch to useStableRelayFilterMap for structural comparison.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Stabilizes relay filter map references using isFilterEqual per relay
instead of JSON.stringify. Avoids serialization overhead for large
filter maps with many relays and pubkeys.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Skip duplicate events in setEventsMap (return prev if event.id exists)
- Only create new relayStates Map on actual state transitions (waiting→receiving),
not on every event — counter increments applied in-place
- Don't add unknown relays to the state map (skip defensive init)
- Cap streaming eventsMap at 2000 with 25% batch eviction of oldest events
- Decouple relay filter map from subscription lifecycle: store in ref,
only tear down subscriptions when the relay SET changes (not filter content)
- Use useStableRelayFilterMap for structural comparison instead of JSON.stringify
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Extend relay filter chunking to route #p tags to inbox/read relays,
matching the existing outbox/write routing for authors. Remove debug
console.log statements across the codebase while preserving error-level
logging. Delete unused logger utility. Expand test coverage for all
chunking scenarios.
Uncomment kind 10051 in kinds registry with NIP-EE attribution and add
it to the relay list editor so users can manage MLS KeyPackage relays.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Spellbook URLs only queried hardcoded aggregator relays, missing events
published to other relays. Now fetches the author's kind:10002 relay list
and includes their outbox relays when loading kind:30777 spellbook events.
Extract useUserRelays hook from inline pattern and refactor
useRepositoryRelays to use it.
* feat: add Educational Resource (kind 30142) with AMB metadata support
- Add feed and detail renderers for AMB Educational Resource events
- Add amb-helpers library with cached helper functions and tests
- Handle broken thumbnail images with BookOpen placeholder fallback
- Surface primary resource URL prominently in both renderers (bookmark-style)
- Register kind 30142 with GraduationCap icon in kind registry
- Link NIP-AMB badge to community NIP event (kind 30817)
* fix: address PR #260 review comments for Educational Resource renderers
- Rename Kind30142*Renderer to EducationalResource*Renderer (human-friendly names)
- Localize language names with Intl.DisplayNames via shared locale-utils
- Use ExternalLink component for license and reference URLs
- Localize ISO dates with formatISODate, fixing UTC timezone shift bug
- Remove # prefix from keyword labels in both feed and detail renderers
- Remove image/thumbnail from feed renderer
- Extract getBrowserLanguage to shared locale-utils, reuse in amb-helpers
* fix: mock getBrowserLanguage in tests for Node < 21 compat
Tests were directly mutating navigator.language which doesn't exist
as a global in Node < 21, causing ReferenceError in CI.
* feat: centralize publish flow with RxJS-based PublishService
Create a unified PublishService that:
- Provides consistent relay selection (outbox + state + hints + fallbacks)
- Emits RxJS observables for per-relay status updates
- Handles EventStore integration automatically
- Supports both fire-and-forget and observable-based publishing
Refactor all publish locations to use the centralized service:
- hub.ts: Use PublishService for ActionRunner publish
- delete-event.ts: Use PublishService (fixes missing eventStore.add)
- publish-spell.ts: Use PublishService with relay hint support
- PostViewer.tsx: Use publishWithUpdates() for per-relay UI tracking
This lays the groundwork for the event log feature by providing
observable hooks into all publish operations.
* feat: add LOG command for relay event introspection
Add an ephemeral event log system that tracks relay operations:
- EventLogService (src/services/event-log.ts):
- Subscribes to PublishService for PUBLISH events with per-relay status
- Monitors relay pool for CONNECT/DISCONNECT events
- Tracks AUTH challenges and results
- Captures NOTICE messages from relays
- Uses RxJS BehaviorSubject for reactive updates
- Circular buffer with configurable max entries (default 500)
- useEventLog hook (src/hooks/useEventLog.ts):
- React hook for filtering and accessing log entries
- Filter by type, relay, or limit
- Retry failed relays directly from the hook
- EventLogViewer component (src/components/EventLogViewer.tsx):
- Tab-based filtering (All/Publish/Connect/Auth/Notice)
- Expandable PUBLISH entries showing per-relay status
- Click to retry failed relays
- Auto-scroll to new entries (pause on scroll)
- Clear log button
- LOG command accessible via Cmd+K palette
* fix: prevent duplicate log entries and check relay OK response
- EventLogService: Check for existing entry before creating new one
when handling publish events (prevents duplicates from start/complete)
- PublishService: Check response.ok from pool.publish() to detect
relay rejections instead of assuming success on resolve
- Update test mock to return proper publish response format
* feat: keep relay selection in call site, compact logs
* chore: cleanup
* fix: make Timestamp component locale-aware via formatTimestamp
Timestamp was hardcoded to "es" locale. Now uses formatTimestamp()
from useLocale.ts for consistent locale-aware time formatting.
Added Timestamp to CLAUDE.md shared components documentation.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* feat: improve event-log reliability, add ERROR type and per-relay timing
Service improvements:
- Fix notice$ duplicate logging with per-relay dedup tracking
- Remove dead Array.isArray code path (notice$ emits strings)
- Increase relay poll interval from 1s to 5s
- Clean publishIdToEntryId map on terminal state, not just overflow
- Immutable entry updates (spread instead of in-place mutation)
- Extract NewEntry<T>/AddEntryInput helper types for clean addEntry signature
- Clear lastNoticePerRelay on log clear
New capabilities:
- ERROR log type: subscribes to relay.error$ for connection failure reasons
- RelayStatusEntry with updatedAt timestamp for per-relay response timing
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* feat: improve EventLogViewer with virtualization, timing, and error display
- Virtualize log list with react-virtuoso for 500-entry buffer performance
- Add ErrorEntry renderer for new ERROR log type (AlertTriangle icon)
- Show per-relay response time (e.g. "142ms", "2.3s") in publish details
- Make all entry types expandable (connect/disconnect now have details)
- Show absolute timestamp in all expanded detail views
- Group ERROR events under Connect tab filter
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
* fix: prevent duplicate PUBLISH log entries from completion event
PublishService emits publish$ twice: once at start, once on completion.
The eager publishIdToEntryId cleanup in handleStatusUpdate fired before
the completion emission, causing handlePublishEvent to create a second
entry. Removed eager cleanup — overflow eviction is sufficient.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---------
Co-authored-by: Claude <noreply@anthropic.com>
Removes ~45 lines of identical relay resolution boilerplate duplicated
across 6 renderers (Issue, Patch, PR - feed and detail views).
The hook encapsulates the 3-tier fallback: repo relays → owner outbox →
aggregators, and also returns the repository event needed for
getValidStatusAuthors.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* refactor(editor): replace DOM manipulation with React node views and floating-ui
- Convert all inline node views (emoji, blob attachment, event preview) from
imperative document.createElement() to React components via ReactNodeViewRenderer
- Replace tippy.js with @floating-ui/react-dom for suggestion popup positioning
- Create useSuggestionRenderer hook bridging Tiptap suggestion callbacks to React state
- Extract shared EmojiMention, SubmitShortcut, and inline node extensions to separate files
- Extract types (EmojiTag, BlobAttachment, SerializedContent) to editor/types.ts
- Extract serialization logic to editor/utils/serialize.ts
- Remove redundant DOM keydown listener from RichEditor (handled by SubmitShortcut extension)
- Remove tippy.js dependency (-1045 lines net, RichEditor 632→297, MentionEditor 1038→354)
https://claude.ai/code/session_01CzeTFzSETs9wSPH2feDE6u
* fix(editor): fix suggestion popover positioning, scrolling, and profile click behavior
- Replace UserName component in ProfileSuggestionList with plain text display
so clicking a suggestion autocompletes instead of opening their profile
(UserName has an onClick that calls addWindow and stopPropagation)
- Add react-virtuoso to ProfileSuggestionList for efficient lazy rendering
of up to 20 search results with fixed item height scrolling
- Add profile avatars with lazy loading and initial-letter fallback
- Fix SuggestionPopover positioning with autoUpdate for scroll/resize tracking
- Add size middleware to constrain popover max-height to available viewport space
https://claude.ai/code/session_01CzeTFzSETs9wSPH2feDE6u
* refactor(editor): convert emoji suggestion from grid to scrollable list with Virtuoso
Replace the 8-column grid layout with a vertical list matching the profile
suggestion style — each row shows the emoji preview alongside its :shortcode:
name. Uses react-virtuoso with fixedItemHeight for lazy rendering and smooth
keyboard-driven scrolling through large emoji sets.
https://claude.ai/code/session_01CzeTFzSETs9wSPH2feDE6u
* fix(editor): set mentionSuggestionChar to ':' for emoji nodes
When backspacing over a mention-based node, Tiptap inserts the node's
mentionSuggestionChar attribute as undo text. The EmojiMention extension
inherits Mention's default of '@', so deleting an emoji left '@' instead
of ':'. Fix by explicitly setting mentionSuggestionChar: ':' in the emoji
command's attrs for both RichEditor and MentionEditor.
https://claude.ai/code/session_01CzeTFzSETs9wSPH2feDE6u
* test(editor): add comprehensive test suite for custom TipTap extensions
Tests all 8 custom extensions using headless TipTap Editor instances in
jsdom environment (TipTap has no official testing package):
- EmojiMention: schema, renderText (unicode vs custom), mentionSuggestionChar
attribute handling, backspace behavior regression test
- BlobAttachmentRichNode/InlineNode: schema (block vs inline), attributes,
renderText URL serialization, parseHTML selectors
- NostrEventPreviewRichNode/InlineNode: schema, renderText encoding for
note/nevent/naddr back to nostr: URIs
- SubmitShortcut: Mod-Enter always submits, Enter behavior with
enterSubmits flag
- FilePasteHandler: media type filtering (image/video/audio), non-media
rejection, mixed paste filtering, edge cases (no files, no callback)
- NostrPasteHandler: bech32 regex matching (npub/note/nevent/naddr/nprofile),
nostr: prefix handling, URL exclusion, node creation (mention vs preview),
surrounding text preservation, multiple entities
- Serialization: formatBlobSize, serializeRichContent (emoji tag extraction,
blob dedup, address refs), serializeInlineContent (mention→nostr: URI,
emoji→shortcode, blob→URL, event preview encoding)
90 new tests total.
https://claude.ai/code/session_01CzeTFzSETs9wSPH2feDE6u
* fix(editor): paste handler and serialization bugs found via adversarial testing
NostrPasteHandler fixes:
- Punctuation after bech32 now matches (npub1..., npub1...! etc.)
Changed trailing lookahead from (?=$|\s) to (?=$|\s|[.,!?;:)\]}>])
- Fixed double-space between entities — unconditional " " after every
entity caused doubled spaces. Now only adds trailing space when entity
is at the very end of pasted text (for cursor positioning).
- Tightened regex character class from [\w] to [a-z0-9] to match actual
bech32 charset (rejects uppercase, underscore)
- Wrapped dispatch in try/catch to handle block-node-at-inline-position
errors gracefully (falls back to default paste)
Serialization fix:
- serializeRichContent now guards blob collection with `url && sha256`
matching the defensive checks already in serializeInlineContent.
Previously null sha256 would corrupt the dedup Set and null url would
produce invalid BlobAttachment entries.
Added 22 new edge case tests:
- Paste handler: punctuation boundaries, double-space regression,
malformed bech32 fallback, uppercase rejection, error resilience
- Serialization: empty editor, null sha256/url blobs, invalid pubkey
fallback, missing mention attrs, inline dedup, multi-paragraph
https://claude.ai/code/session_01CzeTFzSETs9wSPH2feDE6u
* fix(editor): raise suggestion search limits for profiles and emojis
Both suggestion dropdowns use Virtuoso for virtualized rendering, so
they can handle large result sets without performance issues. The
previous limits (20 profiles, 24 emojis) were too restrictive — users
with many custom emojis sharing a substring or large contact lists
couldn't scroll to find the right match.
Raised both limits to 200 to allow thorough browsing while still
bounding the result set.
https://claude.ai/code/session_01CzeTFzSETs9wSPH2feDE6u
* refactor(chat): rework emoji picker to scrollable list with search
Replace the fixed 1-row grid (8 emojis) with a scrollable virtualized
list matching the editor's EmojiSuggestionList look & feel:
- Search box at top with magnifying glass icon
- Virtuoso-backed scrollable list (8 visible items, unlimited results)
- Each row shows emoji icon + :shortcode: label
- Keyboard navigation: arrow keys to select, Enter to confirm
- Mouse hover highlights, click selects
- Frequently used emojis still shown first when no search query
- Narrower dialog (max-w-xs) for a compact picker feel
https://claude.ai/code/session_01CzeTFzSETs9wSPH2feDE6u
* fix: add address field to EmojiTag in editor types, fix GroupMessageOptions
- Add optional `address` field to EmojiTag in editor/types.ts to match
NIP-30 changes from main (30030 emoji set address)
- Extend GroupMessageOptions with MetaTagOptions to fix type error in
GroupMessageBlueprint's setMetaTags call
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
* fix(editor): restore address attr, fix serialization, UserName, no-scroll
- Restore `address` attribute in shared EmojiMention extension (emoji.ts)
that was dropped during refactor — required for NIP-30 emoji set tracking
- Extract `address` from emoji nodes in both serializeRichContent and
serializeInlineContent so it makes it into published events
- Fix MentionEditorProps.onSubmit signature: use EmojiTag[] (not the narrower
inline type) so address field flows through to callers
- Restore UserName component in ProfileSuggestionList for proper display
with Grimoire member badges and supporter flame
- Remove scrollbar when all items fit: set overflow:hidden on Virtuoso when
items.length <= MAX_VISIBLE (profile list, emoji list, emoji picker dialog)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---------
Co-authored-by: Claude <noreply@anthropic.com>
NIP-30 allows an optional 4th tag parameter specifying the source emoji
set address (e.g. "30030:pubkey:identifier"). This threads that address
through the full emoji pipeline so it appears in posts, replies, reactions,
and zap requests.
- Add local blueprints.ts with patched NoteBlueprint, NoteReplyBlueprint,
GroupMessageBlueprint, and ReactionBlueprint that emit the 4th param;
marked TODO to revert once applesauce-common supports it upstream
- Add address? to EmojiWithAddress, EmojiTag, EmojiSearchResult, and
EmojiTag in create-zap-request
- Store address in EmojiSearchService.addEmojiSet (30030:pubkey:identifier)
- Thread address through both editor serializers (MentionEditor, RichEditor)
and the emoji node TipTap attributes
- Fix EmojiPickerDialog to pass address when calling onEmojiSelect and
when re-indexing context emojis
- Update SendMessageOptions.emojiTags and sendReaction customEmoji param
to use EmojiTag throughout the adapter chain
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
