Walkthrough

This PR introduces a comprehensive UI module with tabbed navigation and keyboard event handling, restructures the application to use polling-based order fetching with background tasks, updates database schema to support trade key management, adds dependency upgrades, and implements direct message utilities for Nostr-based order communication.

Changes

Sequence Diagram

sequenceDiagram
    participant User as User (TUI)
    participant MainLoop as Main Event Loop
    participant BGTask as Background Tasks
    participant Client as Nostr Client
    participant DB as Database
    participant Mostro as Mostro Relay

    rect rgb(177, 204, 51, 0.1)
    note over MainLoop: Initialization
    MainLoop->>DB: Initialize pool & schema
    MainLoop->>Client: Configure with identity keys
    MainLoop->>Client: Add relays & connect()
    MainLoop->>BGTask: Spawn fetch_events task
    MainLoop->>BGTask: Spawn listen_for_order_messages task
    end

    rect rgb(177, 204, 51, 0.1)
    note over MainLoop: Main UI Loop
    User->>MainLoop: KeyEvent (e.g., Enter to create order)
    MainLoop->>MainLoop: Route via key_handler
    MainLoop->>MainLoop: Update AppState/UiMode
    MainLoop->>MainLoop: Call ui_draw → Render frame
    end

    rect rgb(177, 204, 51, 0.1)
    note over BGTask: Background Order Fetch
    BGTask->>Client: fetch_events_list(Orders, Pending)
    Client->>Mostro: Subscribe & query events
    Mostro-->>Client: SmallOrder events
    Client-->>BGTask: Events received
    BGTask->>MainLoop: Send via order_result_tx channel
    end

    rect rgb(177, 204, 51, 0.1)
    note over MainLoop: Create/Send New Order
    User->>MainLoop: Fill form & press Enter
    MainLoop->>MainLoop: Validate & construct Order
    MainLoop->>Client: send_dm(take\\_order payload)
    Client->>Mostro: Send PrivateDirectMessage/GiftWrap
    Mostro-->>Client: Response event
    MainLoop->>DB: Order::new() → persist to DB
    MainLoop->>MainLoop: Show OrderResult popup
    end

    rect rgb(177, 204, 51, 0.1)
    note over BGTask: Message Listener
    BGTask->>DB: Get active_order_trade_indices
    loop Every 5s
        BGTask->>Client: Fetch GiftWrap events for trade keys
        Client->>Mostro: Query recent events
        Mostro-->>Client: GiftWrap/DM events
        BGTask->>BGTask: Decrypt & parse DMs
        BGTask->>MainLoop: Send MessageNotification via channel
    end
    end

    rect rgb(177, 204, 51, 0.1)
    note over MainLoop: Handle Message Notification
    MainLoop->>User: Show NewMessageNotification popup
    User->>MainLoop: Enter invoice (AddInvoice action)
    MainLoop->>Client: execute_add\\_invoice → send\\_dm
    Client->>Mostro: Send AddInvoice payload
    Mostro-->>MainLoop: Ack/response
    MainLoop->>DB: Update order state
    MainLoop->>MainLoop: Close popup, return to Orders tab
    end

Estimated code review effort

🎯 5 (Critical) | ⏱️ ~120+ minutes

Areas requiring detailed attention:

• src/ui/key_handler.rs — Complex state machine with 20+ handler functions managing multiple UiMode transitions, async task spawning, and channel coordination; requires careful trace of control flow across modes (CreatingOrder → ConfirmingOrder → WaitingForMostro, etc.).
• src/models.rs — Significant database schema changes (removed buyer/seller\_trade\_pubkey, added trade\_keys/counterparty\_pubkey/buyer\_invoice) with new async persistence methods (insert\_db, update\_db); verify SQL migration logic and foreign key integrity.
• src/util/dm_utils.rs — Non-trivial cryptographic operations (GiftWrap creation with optional signing, conversation key derivation, payload encryption/decryption); background listener spawned as tokio task with periodic polling and deduplication logic.
• src/main.rs — Architectural refactoring from event-driven to polling-based with background tasks; replaced on-struct parsing with utility-based flows; verify channel coordination between main loop and background tasks, error propagation, and graceful shutdown.
• src/ui/mod.rs — Large UI state machine definition with 9 UiMode variants and multiple shared state structures (Arc<Mutex<...>>); verify state consistency and locking patterns.
• src/util/order_utils.rs — End-to-end order workflows (send\_new\_order, take\_order, execute\_add\_invoice) with form validation, payload construction, DM exchange, and DB persistence; complex error handling and Mostro response parsing.
• Database schema migrations — Verify that db.rs schema creation handles the new columns correctly and doesn't break backward compatibility.

Possibly related PRs

• Improve settings #20 — Updates Settings struct and settings.toml with configuration fields; shares settings-driven architecture and relay/currency configuration patterns with this PR.

Poem

🐰 Hops of joy! A grand refactor done,
Keyboards dance, orders fetch as one,
Trade keys blend with messages so keen,
Tabs and modals grace the bustling screen!
From relays to DB, the flow runs true—
A UI garden, polished fresh and new! 🌸

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}