Skip to content

Phase 2 user-admin chat: unify dispute chat message model with NostrEvent and gift wrap storage #501

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Catrya wants to merge 2 commits into
base: main
Choose a base branch
from
+204 −197
Open

Phase 2 user-admin chat: unify dispute chat message model with NostrEvent and gift wrap storage #501

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
02a9e41
refactor: unify dispute chat message model with NostrEvent and gift w…
Catrya Feb 25, 2026
a9813da
fix: use direct construction in _updateMessageState to fix error clea…
Catrya Feb 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 9 additions & 8 deletions docs/DISPUTE_CHAT_MULTIMEDIA_PLAN.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,13 +8,14 @@ Currently the app has two completely different chat mechanisms:

| | P2P Chat (User-User) | Dispute Chat (User-Admin) |
|-|----------------------|--------------------------|
| **Wrap** | `p2pWrap()` — 1 layer | `mostroWrap()` — 3 layers (Rumor→Seal→Wrap) |
| **Key** | Shared key (ECDH) | Admin pubkey (direct) |
| **Routing (`p` tag)** | Shared key pubkey | Trade key / Admin pubkey |
| **Wrap** | `p2pWrap()` — 1 layer | `p2pWrap()` — 1 layer *(Phase 1)* |
| **Key** | Shared key (ECDH) | Admin shared key (ECDH) *(Phase 1)* |
| **Routing (`p` tag)** | Shared key pubkey | Admin shared key pubkey *(Phase 1)* |
| **Subscription** | `SubscriptionManager.chat` | Independent subscription |
| **Message model** | `NostrEvent` | `DisputeChat` (plain String) |
| **Content format** | Plain text / JSON | MostroMessage JSON |
| **Multimedia** | Images + files via Blossom | Not supported |
| **Message model** | `NostrEvent` | `DisputeChatMessage(NostrEvent)` *(Phase 2)* |
| **Storage** | Gift wrap (encrypted) | Gift wrap (encrypted) *(Phase 2)* |
| **Content format** | Plain text / JSON | Plain text *(Phase 1)* |
| **Multimedia** | Images + files via Blossom | Not supported yet |
| **Message bubble** | `MessageBubble` (routes text/image/file) | `DisputeMessageBubble` (text only) |
| **Input widget** | `MessageInput` (text + attachment) | `DisputeMessageInput` (text only) |

Expand Down Expand Up @@ -139,7 +140,7 @@ The admin/solver needs to implement the counterpart:
---

## Phase 2: Unify Message Model
**Status:** `TODO`
**Status:** `DONE`
**PR scope:** Internal refactor — model and notifier changes, no visible UI change
**Depends on:** Phase 1 completed

Expand Down Expand Up @@ -674,4 +675,4 @@ group('MediaCacheMixin')

---

*Last updated: 2026-02-16*
*Last updated: 2026-02-25*
Loading