Walkthrough

Replaced group-based notifications with an address-centric notifier: added GetAddressNotifications(ctx) which ensures a group exists, opens a WebSocket to listen for newtransaction events, queries UTXOs for reported tx hashes, and emits UTXO slices on a buffered channel with reconnection and context-cancellation handling. (≤50 words)

Changes

Sequence Diagram(s)

sequenceDiagram
    participant Caller
    participant Service as NBXplorer Service
    participant WS as WebSocket
    participant Query as UTXO Query
    participant Channel as Notification Channel

    Caller->>Service: GetAddressNotifications(ctx)
    activate Service
    Service->>Service: Ensure/create group
    Service->>WS: Open WebSocket connection
    activate WS
    WS-->>Service: Connected

    loop Listen for events
        WS-->>Service: newtransaction event
        Service->>Service: Parse tx hash
        Service->>Query: searchNewUTXOs(txHash)
        activate Query
        Query-->>Service: []ports.Utxo
        deactivate Query
        Service->>Channel: send []ports.Utxo (buffered)
    end

    alt Read error or disconnect
        Service->>WS: reconnect loop until context done
    end

    Service-->>Caller: returns <-chan []ports.Utxo
    deactivate Service

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

• Review WebSocket connection lifecycle, reconnection/backoff behavior, and error handling
• Verify correct parsing of newtransaction events and robust extraction of transaction hashes
• Inspect searchNewUTXOs usage and UTXO-to-address association correctness
• Confirm buffered channel sizing and non-blocking semantics
• Validate context cancellation paths and resource cleanup

Suggested reviewers

• altafan

Pre-merge checks and finishing touches

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.}

@boqishan thank you for the contrib! The comment was quite outdated, nice catch :).

I left a suggestion for a better description of what's going on under the hood with this function.

@altafan Modified. Please review it again.