nautilus_network/websocket/

subscription.rs

// -------------------------------------------------------------------------------------------------
//  Copyright (C) 2015-2026 Nautech Systems Pty Ltd. All rights reserved.
//  https://nautechsystems.io
//
//  Licensed under the GNU Lesser General Public License Version 3.0 (the "License");
//  You may not use this file except in compliance with the License.
//  You may obtain a copy of the License at https://www.gnu.org/licenses/lgpl-3.0.en.html
//
//  Unless required by applicable law or agreed to in writing, software
//  distributed under the License is distributed on an "AS IS" BASIS,
//  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
//  See the License for the specific language governing permissions and
//  limitations under the License.
// -------------------------------------------------------------------------------------------------

//! Adapter-managed subscription intent and acknowledgment tracking.
//!
//! [`SubscriptionState`] keeps confirmed, `pending_subscribe`, and `pending_unsubscribe` topics
//! separate. Server acknowledgments move topics between those states; late subscribe
//! acknowledgments do not revive cancelled intent, and stale unsubscribe acknowledgments do not
//! remove a later resubscription. [`SubscriptionState::all_topics`] returns confirmed and pending
//! subscribe intent for recovery, excluding pending unsubscriptions.
//!
//! Reference counts are independent of acknowledgment state. The first reference tells the caller
//! to send a subscribe request, and removing the last tells it to send an unsubscribe request. The
//! tracker records state but never sends protocol messages.
//!
//! # Topic format
//!
//! Topics use `channel{delimiter}symbol`; the first delimiter occurrence separates the channel
//! from the optional symbol. A topic without a delimiter represents the whole channel.

use std::{
    num::NonZeroUsize,
    ops::Deref,
    sync::{Arc, LazyLock},
};

use ahash::{AHashMap, AHashSet};
use dashmap::DashMap;
use parking_lot::RwLock;
use ustr::Ustr;

/// Marker for channel-level subscriptions (no specific symbol).
///
/// An empty string in the symbol set indicates a channel-level subscription
/// that applies to all symbols for that channel.
pub(crate) static CHANNEL_LEVEL_MARKER: LazyLock<Ustr> = LazyLock::new(|| Ustr::from(""));

/// Read-only snapshot of subscription topics grouped by channel.
///
/// The snapshot supports immutable map operations through [`Deref`], but cannot mutate the
/// underlying [`SubscriptionState`]. Use the subscription lifecycle methods to change state.
///
/// ```compile_fail
/// use nautilus_network::websocket::SubscriptionState;
///
/// let state = SubscriptionState::new('.');
/// let mut confirmed = state.confirmed();
/// confirmed.clear();
/// ```
#[derive(Clone, Debug, Default, Eq, PartialEq)]
pub struct SubscriptionSnapshot(AHashMap<Ustr, AHashSet<Ustr>>);

impl Deref for SubscriptionSnapshot {
    type Target = AHashMap<Ustr, AHashSet<Ustr>>;

    fn deref(&self) -> &Self::Target {
        &self.0
    }
}

/// Tracks subscription intent and acknowledgment state for WebSocket connections.
///
/// # State management
///
/// The tracker maintains desired subscription intent and three acknowledgment states:
///
/// - **Desired**: Topics that should be active, independent of response ordering or connection.
/// - **Confirmed**: Subscriptions acknowledged by the server and expected to stream data.
/// - **Pending subscribe**: Subscribe requests awaiting server acknowledgment.
/// - **Pending unsubscribe**: Unsubscribe requests awaiting server acknowledgment.
///
/// Late subscribe acknowledgments do not revive cancelled intent, and stale unsubscribe
/// acknowledgments do not remove a later resubscription.
///
/// # Reference counting
///
/// Reference counts remain independent of acknowledgment state. The first consumer tells the
/// caller to send a subscribe request, while removing the last tells it to send an unsubscribe
/// request. The tracker records these transitions but does not send protocol messages.
///
/// # Topic format
///
/// Topics use `channel{delimiter}symbol`, with delimiters such as `.` or `:`. A topic without the
/// delimiter represents a channel-level subscription.
///
/// # Thread safety
///
/// Clones share all state. Operations are thread-safe and can run concurrently from multiple
/// tasks.
#[derive(Clone, Debug)]
pub struct SubscriptionState {
    confirmed: Arc<DashMap<Ustr, AHashSet<Ustr>>>,
    pending_subscribe: Arc<DashMap<Ustr, AHashSet<Ustr>>>,
    pending_unsubscribe: Arc<DashMap<Ustr, AHashSet<Ustr>>>,
    desired: Arc<DashMap<Ustr, AHashSet<Ustr>>>,
    reference_counts: Arc<DashMap<Ustr, NonZeroUsize>>,
    state_lock: Arc<RwLock<()>>,
    delimiter: char,
}

impl SubscriptionState {
    /// Creates a new subscription state tracker with the specified topic delimiter.
    #[must_use]
    pub fn new(delimiter: char) -> Self {
        Self {
            confirmed: Arc::new(DashMap::new()),
            pending_subscribe: Arc::new(DashMap::new()),
            pending_unsubscribe: Arc::new(DashMap::new()),
            desired: Arc::new(DashMap::new()),
            reference_counts: Arc::new(DashMap::new()),
            state_lock: Arc::new(RwLock::new(())),
            delimiter,
        }
    }

    /// Returns the delimiter character used for topic splitting.
    #[must_use]
    pub fn delimiter(&self) -> char {
        self.delimiter
    }

    /// Returns a read-only snapshot of confirmed subscriptions.
    #[must_use]
    pub fn confirmed(&self) -> SubscriptionSnapshot {
        let _guard = self.state_lock.read();
        snapshot(&self.confirmed)
    }

    /// Returns a read-only snapshot of pending subscriptions.
    #[must_use]
    pub fn pending_subscribe(&self) -> SubscriptionSnapshot {
        let _guard = self.state_lock.read();
        snapshot(&self.pending_subscribe)
    }

    /// Returns a read-only snapshot of pending unsubscriptions.
    #[must_use]
    pub fn pending_unsubscribe(&self) -> SubscriptionSnapshot {
        let _guard = self.state_lock.read();
        snapshot(&self.pending_unsubscribe)
    }

    /// Returns the number of confirmed subscriptions.
    ///
    /// Counts both channel-level and symbol-level subscriptions.
    #[must_use]
    pub fn len(&self) -> usize {
        let _guard = self.state_lock.read();
        self.confirmed.iter().map(|entry| entry.value().len()).sum()
    }

    /// Returns true if there are no subscriptions (confirmed or pending).
    #[must_use]
    pub fn is_empty(&self) -> bool {
        let _guard = self.state_lock.read();
        self.confirmed.is_empty()
            && self.pending_subscribe.is_empty()
            && self.pending_unsubscribe.is_empty()
            && self.desired.is_empty()
    }

    /// Returns true if a channel:symbol pair is subscribed (confirmed or pending subscribe).
    #[must_use]
    pub fn is_subscribed(&self, channel: &Ustr, symbol: &Ustr) -> bool {
        let _guard = self.state_lock.read();

        if let Some(symbols) = self.confirmed.get(channel)
            && symbols.contains(symbol)
        {
            return true;
        }

        if let Some(symbols) = self.pending_subscribe.get(channel)
            && symbols.contains(symbol)
        {
            return true;
        }
        false
    }

    /// Returns all pending subscribe topics as strings.
    #[must_use]
    pub fn pending_subscribe_topics(&self) -> Vec<String> {
        let _guard = self.state_lock.read();
        self.topics_from_map(&self.pending_subscribe)
    }

    /// Returns all pending unsubscribe topics as strings.
    #[must_use]
    pub fn pending_unsubscribe_topics(&self) -> Vec<String> {
        let _guard = self.state_lock.read();
        self.topics_from_map(&self.pending_unsubscribe)
    }

    /// Returns all topics that should be active after reconnect recovery.
    ///
    /// The result includes confirmed and pending subscribe topics, but excludes pending
    /// unsubscribe topics.
    #[must_use]
    pub fn all_topics(&self) -> Vec<String> {
        let _guard = self.state_lock.read();
        let mut topics = self.topics_from_map(&self.confirmed);
        topics.extend(self.topics_from_map(&self.pending_subscribe));
        topics
    }

    /// Marks a topic as pending subscription.
    ///
    /// Call this after sending a subscribe request. This operation is idempotent for a confirmed
    /// topic and cancels any pending unsubscription for the same topic.
    pub fn mark_subscribe(&self, topic: &str) {
        let _guard = self.state_lock.write();
        let (channel, symbol) = split_topic(topic, self.delimiter);
        track_topic(&self.desired, channel, symbol);

        // If already confirmed, don't re-add to pending (idempotent)
        if is_tracked(&self.confirmed, channel, symbol) {
            return;
        }

        // Remove from pending_unsubscribe if present
        untrack_topic(&self.pending_unsubscribe, channel, symbol);

        track_topic(&self.pending_subscribe, channel, symbol);
    }

    /// Atomically tries to mark a topic as pending subscription.
    ///
    /// Returns `true` if the topic was newly marked as pending (should send subscribe).
    /// Returns `false` if the topic was already confirmed or pending (skip sending).
    ///
    /// The check and state transition are atomic across concurrent subscribe calls.
    #[must_use]
    pub fn try_mark_subscribe(&self, topic: &str) -> bool {
        let _guard = self.state_lock.write();
        let (channel, symbol) = split_topic(topic, self.delimiter);

        // If already desired, no action needed
        if !track_topic(&self.desired, channel, symbol) {
            return false;
        }

        let inserted = track_topic(&self.pending_subscribe, channel, symbol);
        if inserted {
            untrack_topic(&self.pending_unsubscribe, channel, symbol);
        }

        inserted
    }

    /// Confirms a subscription by moving it from pending to confirmed.
    ///
    /// Call this when the server acknowledges a subscribe request. A late confirmation cannot
    /// restore a topic that is no longer desired.
    pub fn confirm_subscribe(&self, topic: &str) {
        let _guard = self.state_lock.write();
        let (channel, symbol) = split_topic(topic, self.delimiter);

        if !is_tracked(&self.desired, channel, symbol)
            || is_tracked(&self.pending_unsubscribe, channel, symbol)
        {
            return;
        }

        untrack_topic(&self.pending_subscribe, channel, symbol);
        track_topic(&self.confirmed, channel, symbol);
    }

    /// Marks a topic as pending unsubscription.
    ///
    /// Removes the topic from confirmed and `pending_subscribe` state before adding it to
    /// `pending_unsubscribe`. This also handles unsubscription before initial confirmation.
    pub fn mark_unsubscribe(&self, topic: &str) {
        let _guard = self.state_lock.write();
        let (channel, symbol) = split_topic(topic, self.delimiter);
        untrack_topic(&self.desired, channel, symbol);
        track_topic(&self.pending_unsubscribe, channel, symbol);
        untrack_topic(&self.confirmed, channel, symbol);
        untrack_topic(&self.pending_subscribe, channel, symbol);
    }

    /// Confirms an unsubscription by removing it from pending and confirmed state.
    ///
    /// Call this when the server acknowledges an unsubscribe request. A stale acknowledgment is
    /// ignored if the topic is no longer pending unsubscription. `pending_subscribe` remains intact
    /// so an immediate resubscription survives a late unsubscribe acknowledgment.
    pub fn confirm_unsubscribe(&self, topic: &str) {
        let _guard = self.state_lock.write();
        let (channel, symbol) = split_topic(topic, self.delimiter);

        // Only process if topic is actually pending unsubscription
        // This ignores stale unsubscribe ACKs after user has re-subscribed
        if !is_tracked(&self.pending_unsubscribe, channel, symbol) {
            return; // Stale ACK, ignore
        }

        untrack_topic(&self.pending_unsubscribe, channel, symbol);
        untrack_topic(&self.confirmed, channel, symbol);
        // Don't clear pending_subscribe - it's a valid re-subscribe request
    }

    /// Marks a subscription as failed, moving it from confirmed back to pending.
    ///
    /// This keeps failed subscriptions available for retry after reconnect. A topic pending
    /// unsubscription is unchanged because its subscription was cancelled.
    pub fn mark_failure(&self, topic: &str) {
        let _guard = self.state_lock.write();
        let (channel, symbol) = split_topic(topic, self.delimiter);

        if !is_tracked(&self.desired, channel, symbol)
            || is_tracked(&self.pending_unsubscribe, channel, symbol)
        {
            return;
        }

        untrack_topic(&self.confirmed, channel, symbol);
        track_topic(&self.pending_subscribe, channel, symbol);
    }

    /// Resets acknowledgment state for a replacement connection.
    ///
    /// Returns the desired topics to replay. Confirmed topics become pending subscriptions,
    /// pending unsubscriptions are completed by the closed connection, and reference counts are
    /// preserved.
    #[allow(
        clippy::must_use_candidate,
        reason = "some adapters replay from separate subscription registries"
    )]
    pub fn reset_after_reconnect(&self) -> Vec<String> {
        let _guard = self.state_lock.write();
        let mut topics = self.topics_from_map(&self.confirmed);
        topics.extend(self.topics_from_map(&self.pending_subscribe));

        self.confirmed.clear();
        self.pending_subscribe.clear();
        self.pending_unsubscribe.clear();

        for topic in &topics {
            let (channel, symbol) = split_topic(topic, self.delimiter);
            track_topic(&self.pending_subscribe, channel, symbol);
        }

        topics
    }

    /// Increments the reference count for a topic.
    ///
    /// Returns `true` if this is the first subscription (caller should send subscribe
    /// message to server).
    ///
    /// # Panics
    ///
    /// Panics if the reference count exceeds `usize::MAX` subscriptions for a single topic.
    #[allow(
        clippy::must_use_candidate,
        reason = "callers use this for side effects"
    )]
    pub fn add_reference(&self, topic: &str) -> bool {
        let mut should_subscribe = false;
        let topic_ustr = Ustr::from(topic);

        self.reference_counts
            .entry(topic_ustr)
            .and_modify(|count| {
                *count = NonZeroUsize::new(count.get() + 1).expect("reference count overflow");
            })
            .or_insert_with(|| {
                should_subscribe = true;
                NonZeroUsize::MIN
            });

        should_subscribe
    }

    /// Decrements the reference count for a topic.
    ///
    /// Returns `true` if this was the last subscription (caller should send unsubscribe
    /// message to server).
    ///
    /// # Panics
    ///
    /// Panics if the internal reference count state becomes inconsistent (should never happen
    /// if the API is used correctly).
    #[allow(
        clippy::must_use_candidate,
        reason = "callers use this for side effects"
    )]
    pub fn remove_reference(&self, topic: &str) -> bool {
        let topic_ustr = Ustr::from(topic);

        // Use entry API to atomically decrement and remove if zero
        // This prevents race where another thread adds a reference between the check and remove
        if let dashmap::mapref::entry::Entry::Occupied(mut entry) =
            self.reference_counts.entry(topic_ustr)
        {
            let current = entry.get().get();

            if current == 1 {
                entry.remove();
                return true;
            }

            *entry.get_mut() = NonZeroUsize::new(current - 1)
                .expect("reference count should never reach zero here");
        }

        false
    }

    /// Returns the current reference count for a topic.
    ///
    /// Returns 0 if the topic has no references.
    #[must_use]
    pub fn get_reference_count(&self, topic: &str) -> usize {
        let topic_ustr = Ustr::from(topic);
        self.reference_counts
            .get(&topic_ustr)
            .map_or(0, |count| count.get())
    }

    /// Clears all subscription state.
    ///
    /// This resets desired intent, acknowledgment state, and reference counts.
    pub fn clear(&self) {
        let _guard = self.state_lock.write();
        self.confirmed.clear();
        self.pending_subscribe.clear();
        self.pending_unsubscribe.clear();
        self.desired.clear();
        self.reference_counts.clear();
    }

    // Converts a subscription map to sorted topic strings
    fn topics_from_map(&self, map: &DashMap<Ustr, AHashSet<Ustr>>) -> Vec<String> {
        let mut topics = Vec::new();
        let marker = *CHANNEL_LEVEL_MARKER;

        for entry in map {
            let channel = entry.key();
            let symbols = entry.value();

            // Check for channel-level subscription marker
            if symbols.contains(&marker) {
                topics.push(channel.to_string());
            }

            // Add symbol-level subscriptions (skip marker)
            for symbol in symbols {
                if *symbol != marker {
                    topics.push(format!("{channel}{}{symbol}", self.delimiter));
                }
            }
        }

        // Sort so resubscription after a reconnect replays topics in the same sequence
        // across runs; both the outer DashMap and the inner symbol sets are unordered.
        topics.sort();
        topics
    }
}

/// Splits a topic into channel and optional symbol using the specified delimiter.
#[must_use]
pub fn split_topic(topic: &str, delimiter: char) -> (&str, Option<&str>) {
    topic
        .split_once(delimiter)
        .map_or((topic, None), |(channel, symbol)| (channel, Some(symbol)))
}

fn snapshot(map: &DashMap<Ustr, AHashSet<Ustr>>) -> SubscriptionSnapshot {
    SubscriptionSnapshot(
        map.iter()
            .map(|entry| (*entry.key(), entry.value().clone()))
            .collect(),
    )
}

/// Tracks a topic in the given map by adding it to the channel's symbol set.
///
/// Channel-level subscriptions are stored using an empty string marker,
/// allowing both channel-level and symbol-level subscriptions to coexist.
fn track_topic(map: &DashMap<Ustr, AHashSet<Ustr>>, channel: &str, symbol: Option<&str>) -> bool {
    map.entry(Ustr::from(channel))
        .or_default()
        .insert(topic_symbol(symbol))
}

/// Removes a topic from the given map by removing it from the channel's symbol set.
///
/// Removes the entire channel entry if no subscriptions remain after removal.
fn untrack_topic(map: &DashMap<Ustr, AHashSet<Ustr>>, channel: &str, symbol: Option<&str>) {
    let symbol = topic_symbol(symbol);

    // Use entry API to atomically remove symbol and check if empty
    // This prevents race conditions where another thread adds a symbol between operations
    if let dashmap::mapref::entry::Entry::Occupied(mut entry) = map.entry(Ustr::from(channel)) {
        entry.get_mut().remove(&symbol);
        if entry.get().is_empty() {
            entry.remove();
        }
    }
}

/// Checks if a topic exists in the given map.
fn is_tracked(map: &DashMap<Ustr, AHashSet<Ustr>>, channel: &str, symbol: Option<&str>) -> bool {
    let symbol = topic_symbol(symbol);
    map.get(&Ustr::from(channel))
        .is_some_and(|entry| entry.contains(&symbol))
}

fn topic_symbol(symbol: Option<&str>) -> Ustr {
    symbol.map_or(*CHANNEL_LEVEL_MARKER, Ustr::from)
}

#[cfg(test)]
mod tests {
    use std::sync::{
        Barrier,
        atomic::{AtomicUsize, Ordering},
    };

    use rstest::rstest;

    use super::*;

    #[rstest]
    fn test_split_topic_with_symbol() {
        let (channel, symbol) = split_topic("tickers.BTCUSDT", '.');
        assert_eq!(channel, "tickers");
        assert_eq!(symbol, Some("BTCUSDT"));

        let (channel, symbol) = split_topic("orderBookL2:XBTUSD", ':');
        assert_eq!(channel, "orderBookL2");
        assert_eq!(symbol, Some("XBTUSD"));
    }

    #[rstest]
    fn test_split_topic_without_symbol() {
        let (channel, symbol) = split_topic("orderbook", '.');
        assert_eq!(channel, "orderbook");
        assert_eq!(symbol, None);
    }

    #[rstest]
    fn test_topic_without_symbol() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("orderbook");
        state.confirm_subscribe("orderbook");

        assert_eq!(state.len(), 1);
        assert_eq!(state.all_topics(), vec!["orderbook"]);
    }

    #[rstest]
    fn test_different_delimiters() {
        let state_dot = SubscriptionState::new('.');
        state_dot.mark_subscribe("tickers.BTCUSDT");
        assert_eq!(
            state_dot.pending_subscribe_topics(),
            vec!["tickers.BTCUSDT"]
        );

        let state_colon = SubscriptionState::new(':');
        state_colon.mark_subscribe("orderBookL2:XBTUSD");
        assert_eq!(
            state_colon.pending_subscribe_topics(),
            vec!["orderBookL2:XBTUSD"]
        );
    }

    #[rstest]
    fn test_different_delimiter_does_not_affect_storage() {
        // Verify delimiter is only used for parsing, not storage
        let state_dot = SubscriptionState::new('.');
        let state_colon = SubscriptionState::new(':');

        // Add same logical subscription with different delimiters
        state_dot.mark_subscribe("channel.SYMBOL");
        state_colon.mark_subscribe("channel:SYMBOL");

        // Both should work correctly
        assert_eq!(state_dot.pending_subscribe_topics(), vec!["channel.SYMBOL"]);
        assert_eq!(
            state_colon.pending_subscribe_topics(),
            vec!["channel:SYMBOL"]
        );
    }

    #[rstest]
    fn test_multiple_symbols_same_channel() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("tickers.BTCUSDT");
        state.mark_subscribe("tickers.ETHUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.ETHUSDT");

        assert_eq!(state.len(), 2);
        assert_eq!(
            state.all_topics(),
            vec!["tickers.BTCUSDT", "tickers.ETHUSDT"]
        );
    }

    #[rstest]
    fn test_mixed_channel_and_symbol_subscriptions() {
        let state = SubscriptionState::new('.');

        // Subscribe to channel-level first
        state.mark_subscribe("tickers");
        state.confirm_subscribe("tickers");
        assert_eq!(state.len(), 1);
        assert_eq!(state.all_topics(), vec!["tickers"]);

        // Add symbol-level subscription to same channel
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 2);

        // Both should be present
        assert_eq!(state.all_topics(), vec!["tickers", "tickers.BTCUSDT"]);

        // Add another symbol
        state.mark_subscribe("tickers.ETHUSDT");
        state.confirm_subscribe("tickers.ETHUSDT");
        assert_eq!(state.len(), 3);

        assert_eq!(
            state.all_topics(),
            vec!["tickers", "tickers.BTCUSDT", "tickers.ETHUSDT"]
        );

        // Unsubscribe from channel-level only
        state.mark_unsubscribe("tickers");
        state.confirm_unsubscribe("tickers");
        assert_eq!(state.len(), 2);

        assert_eq!(
            state.all_topics(),
            vec!["tickers.BTCUSDT", "tickers.ETHUSDT"]
        );
    }

    #[rstest]
    fn test_symbol_subscription_before_channel() {
        let state = SubscriptionState::new('.');

        // Subscribe to symbol first
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 1);

        // Then add channel-level
        state.mark_subscribe("tickers");
        state.confirm_subscribe("tickers");
        assert_eq!(state.len(), 2);

        // Both should be present after reconnect
        assert_eq!(state.all_topics(), vec!["tickers", "tickers.BTCUSDT"]);
    }

    #[rstest]
    fn test_edge_case_empty_channel_name() {
        let state = SubscriptionState::new('.');

        // Edge case: empty string as topic
        state.mark_subscribe("");
        state.confirm_subscribe("");

        assert_eq!(state.len(), 1);
        assert_eq!(state.all_topics(), vec![""]);
    }

    #[rstest]
    fn test_special_characters_in_topics() {
        let state = SubscriptionState::new('.');

        // Topics with special characters
        let special_topics = vec![
            "channel.symbol-with-dash",
            "channel.SYMBOL_WITH_UNDERSCORE",
            "channel.symbol123",
            "channel.symbol@special",
        ];

        for topic in &special_topics {
            state.mark_subscribe(topic);
            state.confirm_subscribe(topic);
        }

        assert_eq!(state.len(), special_topics.len());

        let all_topics = state.all_topics();

        for topic in &special_topics {
            assert!(
                all_topics.contains(&(*topic).to_string()),
                "Missing topic: {topic}"
            );
        }
    }

    #[rstest]
    fn test_edge_case_malformed_topics() {
        let state = SubscriptionState::new('.');

        // Topics with multiple delimiters (splits on first delimiter)
        state.mark_subscribe("channel.symbol.extra");
        state.confirm_subscribe("channel.symbol.extra");
        let topics = state.all_topics();
        assert!(topics.contains(&"channel.symbol.extra".to_string()));

        // Topic with leading delimiter (empty channel, symbol is "channel")
        state.mark_subscribe(".channel");
        state.confirm_subscribe(".channel");
        assert_eq!(state.len(), 2);

        // Topic with trailing delimiter - treated as channel-level (empty symbol = marker)
        // "channel." splits to ("channel", Some("")), and empty string is the channel marker
        state.mark_subscribe("channel.");
        state.confirm_subscribe("channel.");
        assert_eq!(state.len(), 3);

        // Topic without delimiter - explicitly channel-level
        state.mark_subscribe("tickers");
        state.confirm_subscribe("tickers");
        assert_eq!(state.len(), 4);

        // Verify all are retrievable (note: "channel." becomes "channel")
        let all = state.all_topics();
        assert_eq!(all.len(), 4);
        assert!(all.contains(&"channel.symbol.extra".to_string()));
        assert!(all.contains(&".channel".to_string()));
        assert!(all.contains(&"channel".to_string())); // "channel." treated as channel-level
        assert!(all.contains(&"tickers".to_string()));
    }

    #[rstest]
    fn test_new_state_is_empty() {
        let state = SubscriptionState::new('.');
        assert!(state.is_empty());
        assert_eq!(state.len(), 0);
    }

    #[rstest]
    fn test_subscription_map_accessors_return_isolated_snapshots() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");

        let confirmed = state.confirmed();
        let pending_subscribe = state.pending_subscribe();
        let pending_unsubscribe = state.pending_unsubscribe();

        state.mark_unsubscribe("tickers.BTCUSDT");

        assert_eq!(
            confirmed.get(&Ustr::from("tickers")),
            Some(&AHashSet::from_iter([Ustr::from("BTCUSDT")]))
        );
        assert!(pending_subscribe.is_empty());
        assert!(pending_unsubscribe.is_empty());
        assert!(state.confirmed().is_empty());
        assert_eq!(
            state.pending_unsubscribe().get(&Ustr::from("tickers")),
            Some(&AHashSet::from_iter([Ustr::from("BTCUSDT")]))
        );
    }

    #[rstest]
    fn test_is_subscribed_empty_state() {
        let state = SubscriptionState::new('.');
        let channel = Ustr::from("tickers");
        let symbol = Ustr::from("BTCUSDT");

        assert!(!state.is_subscribed(&channel, &symbol));
    }

    #[rstest]
    fn test_is_subscribed_pending() {
        let state = SubscriptionState::new('.');
        let channel = Ustr::from("tickers");
        let symbol = Ustr::from("BTCUSDT");

        state.mark_subscribe("tickers.BTCUSDT");

        assert!(state.is_subscribed(&channel, &symbol));
    }

    #[rstest]
    fn test_is_subscribed_confirmed() {
        let state = SubscriptionState::new('.');
        let channel = Ustr::from("tickers");
        let symbol = Ustr::from("BTCUSDT");

        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");

        assert!(state.is_subscribed(&channel, &symbol));
    }

    #[rstest]
    fn test_is_subscribed_after_unsubscribe() {
        let state = SubscriptionState::new('.');
        let channel = Ustr::from("tickers");
        let symbol = Ustr::from("BTCUSDT");

        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        state.mark_unsubscribe("tickers.BTCUSDT");

        // Pending unsubscribe should not count as subscribed
        assert!(!state.is_subscribed(&channel, &symbol));
    }

    #[rstest]
    fn test_is_subscribed_after_confirm_unsubscribe() {
        let state = SubscriptionState::new('.');
        let channel = Ustr::from("tickers");
        let symbol = Ustr::from("BTCUSDT");

        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        state.mark_unsubscribe("tickers.BTCUSDT");
        state.confirm_unsubscribe("tickers.BTCUSDT");

        assert!(!state.is_subscribed(&channel, &symbol));
    }

    #[rstest]
    fn test_all_topics_includes_confirmed_and_pending_subscribe() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        state.mark_subscribe("tickers.ETHUSDT");

        assert_eq!(
            state.all_topics(),
            vec!["tickers.BTCUSDT", "tickers.ETHUSDT"]
        );
    }

    #[rstest]
    fn test_all_topics_excludes_pending_unsubscribe() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        state.mark_unsubscribe("tickers.BTCUSDT");

        let topics = state.all_topics();
        assert!(topics.is_empty());
    }

    #[rstest]
    fn test_all_topics_is_sorted_within_each_group() {
        let state = SubscriptionState::new('.');

        // Insert scrambled across channels and symbols so hash order cannot pass.
        for topic in [
            "trades.SOLUSDT",
            "tickers.ETHUSDT",
            "trades.BTCUSDT",
            "tickers.BTCUSDT",
        ] {
            state.mark_subscribe(topic);
            state.confirm_subscribe(topic);
        }

        state.mark_subscribe("orders.XRPUSDT");
        state.mark_subscribe("orders.ADAUSDT");

        // Confirmed topics sort among themselves, then pending ones do the same.
        assert_eq!(
            state.all_topics(),
            vec![
                "tickers.BTCUSDT",
                "tickers.ETHUSDT",
                "trades.BTCUSDT",
                "trades.SOLUSDT",
                "orders.ADAUSDT",
                "orders.XRPUSDT",
            ]
        );
    }

    #[rstest]
    fn test_pending_subscribe_excludes_pending_unsubscribe() {
        let state = SubscriptionState::new('.');

        // Subscribe and confirm
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");

        // Mark for unsubscribe
        state.mark_unsubscribe("tickers.BTCUSDT");

        // Should be in pending_unsubscribe but NOT in all_topics
        assert_eq!(state.pending_unsubscribe_topics(), vec!["tickers.BTCUSDT"]);
        assert!(state.all_topics().is_empty());
        assert_eq!(state.len(), 0);
    }

    #[rstest]
    fn test_mark_subscribe() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("tickers.BTCUSDT");

        assert_eq!(state.pending_subscribe_topics(), vec!["tickers.BTCUSDT"]);
        assert_eq!(state.len(), 0); // Not confirmed yet
    }

    #[rstest]
    fn test_try_mark_subscribe_returns_true_once_across_concurrent_calls() {
        const CALLERS: usize = 32;

        let state = Arc::new(SubscriptionState::new('.'));
        let start = Arc::new(Barrier::new(CALLERS));
        let send_count = Arc::new(AtomicUsize::new(0));

        std::thread::scope(|scope| {
            for _ in 0..CALLERS {
                let state = Arc::clone(&state);
                let start = Arc::clone(&start);
                let send_count = Arc::clone(&send_count);

                scope.spawn(move || {
                    start.wait();

                    if state.try_mark_subscribe("tickers.BTCUSDT") {
                        send_count.fetch_add(1, Ordering::SeqCst);
                    }
                });
            }
        });

        assert_eq!(send_count.load(Ordering::SeqCst), 1);
        assert_eq!(state.pending_subscribe_topics(), ["tickers.BTCUSDT"]);
        assert!(state.pending_unsubscribe_topics().is_empty());
    }

    #[rstest]
    fn test_try_mark_subscribe_respects_lifecycle_state() {
        let state = SubscriptionState::new('.');
        let topic = "tickers.BTCUSDT";

        assert!(state.try_mark_subscribe(topic));
        assert!(!state.try_mark_subscribe(topic));

        state.confirm_subscribe(topic);
        assert!(!state.try_mark_subscribe(topic));

        state.mark_unsubscribe(topic);
        assert!(state.try_mark_subscribe(topic));
        assert_eq!(state.pending_subscribe_topics(), [topic]);
        assert!(state.pending_unsubscribe_topics().is_empty());
    }

    #[rstest]
    fn test_confirm_subscribe() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");

        assert!(state.pending_subscribe_topics().is_empty());
        assert_eq!(state.len(), 1);
    }

    #[rstest]
    fn test_mark_unsubscribe() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        state.mark_unsubscribe("tickers.BTCUSDT");

        assert_eq!(state.len(), 0); // Removed from confirmed
        assert_eq!(state.pending_unsubscribe_topics(), vec!["tickers.BTCUSDT"]);
    }

    #[rstest]
    fn test_confirm_unsubscribe() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        state.mark_unsubscribe("tickers.BTCUSDT");
        state.confirm_unsubscribe("tickers.BTCUSDT");

        assert!(state.is_empty());
    }

    #[rstest]
    fn test_unsubscribe_before_subscribe_confirmed() {
        let state = SubscriptionState::new('.');

        // User subscribes
        state.mark_subscribe("tickers.BTCUSDT");
        assert_eq!(state.pending_subscribe_topics(), vec!["tickers.BTCUSDT"]);

        // User immediately changes mind before server confirms
        state.mark_unsubscribe("tickers.BTCUSDT");

        // Should be removed from pending_subscribe and added to pending_unsubscribe
        assert!(state.pending_subscribe_topics().is_empty());
        assert_eq!(state.pending_unsubscribe_topics(), vec!["tickers.BTCUSDT"]);

        // Confirm the unsubscribe
        state.confirm_unsubscribe("tickers.BTCUSDT");

        // Should be completely gone
        assert!(state.is_empty());
        assert!(state.all_topics().is_empty());
        assert_eq!(state.len(), 0);
    }

    #[rstest]
    fn test_late_subscribe_confirmation_after_unsubscribe() {
        let state = SubscriptionState::new('.');

        // User subscribes
        state.mark_subscribe("tickers.BTCUSDT");

        // User immediately unsubscribes
        state.mark_unsubscribe("tickers.BTCUSDT");

        // Late subscribe confirmation arrives from server
        state.confirm_subscribe("tickers.BTCUSDT");

        // Should NOT be added to confirmed (unsubscribe takes precedence)
        assert_eq!(state.len(), 0);
        assert!(state.pending_subscribe_topics().is_empty());

        // Confirm the unsubscribe
        state.confirm_unsubscribe("tickers.BTCUSDT");

        // Should still be empty
        assert!(state.is_empty());
        assert!(state.all_topics().is_empty());
    }

    #[rstest]
    fn test_late_subscribe_ack_after_unsubscribe_ack_does_not_restore_topic() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("tickers.BTCUSDT");
        state.mark_unsubscribe("tickers.BTCUSDT");

        state.confirm_unsubscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");

        assert!(state.is_empty());
        assert!(state.all_topics().is_empty());
        assert!(state.pending_subscribe_topics().is_empty());
        assert!(state.pending_unsubscribe_topics().is_empty());
    }

    #[rstest]
    fn test_resubscribe_before_unsubscribe_ack() {
        // Regression test for race condition:
        // User unsubscribes, then immediately resubscribes before the unsubscribe ACK arrives.
        // The unsubscribe ACK should NOT clear the pending_subscribe entry.
        let state = SubscriptionState::new('.');

        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 1);

        state.mark_unsubscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 0);
        assert_eq!(state.pending_unsubscribe_topics(), vec!["tickers.BTCUSDT"]);

        // User immediately resubscribes (before unsubscribe ACK)
        state.mark_subscribe("tickers.BTCUSDT");
        assert_eq!(state.pending_subscribe_topics(), vec!["tickers.BTCUSDT"]);

        // Stale unsubscribe ACK arrives - should be ignored (pending_unsubscribe already cleared)
        state.confirm_unsubscribe("tickers.BTCUSDT");
        assert!(state.pending_unsubscribe_topics().is_empty());
        assert_eq!(state.pending_subscribe_topics(), vec!["tickers.BTCUSDT"]); // Must still be pending

        // Subscribe ACK confirms successfully
        state.confirm_subscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 1);
        assert!(state.pending_subscribe_topics().is_empty());

        // Topic available for reconnect
        let all = state.all_topics();
        assert_eq!(all.len(), 1);
        assert!(all.contains(&"tickers.BTCUSDT".to_string()));
    }

    #[rstest]
    fn test_stale_unsubscribe_ack_after_resubscribe_confirmed() {
        // Regression test for P1 bug: Stale unsubscribe ACK removing confirmed topic.
        // Scenario: User unsubscribes, immediately resubscribes, subscribe ACK arrives
        // FIRST (out of order), then stale unsubscribe ACK arrives.
        // The stale ACK must NOT remove the topic from confirmed state.
        let state = SubscriptionState::new('.');

        // Initial subscription
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 1);

        // User unsubscribes
        state.mark_unsubscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 0);
        assert_eq!(state.pending_unsubscribe_topics(), vec!["tickers.BTCUSDT"]);

        // User immediately resubscribes (before unsubscribe ACK)
        state.mark_subscribe("tickers.BTCUSDT");
        assert!(state.pending_unsubscribe_topics().is_empty()); // Cleared by mark_subscribe
        assert_eq!(state.pending_subscribe_topics(), vec!["tickers.BTCUSDT"]);

        // Subscribe ACK arrives FIRST (out of order!)
        state.confirm_subscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 1); // Back in confirmed
        assert!(state.pending_subscribe_topics().is_empty());

        // NOW the stale unsubscribe ACK arrives
        // This must be ignored because topic is no longer in pending_unsubscribe
        state.confirm_unsubscribe("tickers.BTCUSDT");

        // Topic should STILL be confirmed (not removed by stale ACK)
        assert_eq!(state.len(), 1); // Must remain confirmed
        assert!(state.pending_unsubscribe_topics().is_empty());
        assert!(state.pending_subscribe_topics().is_empty());

        // Topic should be in all_topics (for reconnect)
        let all = state.all_topics();
        assert_eq!(all.len(), 1);
        assert!(all.contains(&"tickers.BTCUSDT".to_string()));
    }

    #[rstest]
    fn test_unsubscribe_clears_all_states() {
        let state = SubscriptionState::new('.');

        // Subscribe and confirm
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 1);

        // Unsubscribe
        state.mark_unsubscribe("tickers.BTCUSDT");

        // Should be removed from confirmed
        assert_eq!(state.len(), 0);
        assert_eq!(state.pending_unsubscribe_topics(), vec!["tickers.BTCUSDT"]);

        // Late subscribe confirmation somehow arrives (race condition)
        state.confirm_subscribe("tickers.BTCUSDT");

        // confirm_unsubscribe should clean everything
        state.confirm_unsubscribe("tickers.BTCUSDT");

        // Completely empty
        assert!(state.is_empty());
        assert_eq!(state.len(), 0);
        assert!(state.pending_subscribe_topics().is_empty());
        assert!(state.pending_unsubscribe_topics().is_empty());
        assert!(state.all_topics().is_empty());
    }

    #[rstest]
    fn test_concurrent_subscribe_confirmation_and_unsubscribe_preserve_intent() {
        const ITERATIONS: usize = 10_000;

        let state = Arc::new(SubscriptionState::new('.'));
        let start = Arc::new(Barrier::new(3));
        let finish = Arc::new(Barrier::new(3));
        let failed_iteration = Arc::new(AtomicUsize::new(usize::MAX));

        std::thread::scope(|scope| {
            let confirming_state = Arc::clone(&state);
            let confirming_start = Arc::clone(&start);
            let confirming_finish = Arc::clone(&finish);

            scope.spawn(move || {
                for _ in 0..ITERATIONS {
                    confirming_start.wait();
                    confirming_state.confirm_subscribe("tickers.BTCUSDT");
                    confirming_finish.wait();
                }
            });

            let unsubscribing_state = Arc::clone(&state);
            let unsubscribing_start = Arc::clone(&start);
            let unsubscribing_finish = Arc::clone(&finish);

            scope.spawn(move || {
                for _ in 0..ITERATIONS {
                    unsubscribing_start.wait();
                    unsubscribing_state.mark_unsubscribe("tickers.BTCUSDT");
                    unsubscribing_finish.wait();
                }
            });

            for iteration in 0..ITERATIONS {
                state.clear();
                state.mark_subscribe("tickers.BTCUSDT");
                start.wait();
                finish.wait();

                if !state.all_topics().is_empty()
                    || state.pending_unsubscribe_topics() != ["tickers.BTCUSDT"]
                {
                    _ = failed_iteration.compare_exchange(
                        usize::MAX,
                        iteration,
                        Ordering::SeqCst,
                        Ordering::SeqCst,
                    );
                }
            }
        });

        assert_eq!(
            failed_iteration.load(Ordering::SeqCst),
            usize::MAX,
            "late subscribe confirmation restored the unsubscribe intent"
        );
    }

    #[rstest]
    fn test_state_machine_invalid_transitions() {
        let state = SubscriptionState::new('.');

        // Confirm subscribe without matching intent - should be ignored
        state.confirm_subscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 0);

        // Confirm unsubscribe without marking first - should not crash
        state.confirm_unsubscribe("tickers.ETHUSDT");
        assert_eq!(state.len(), 0); // Nothing changes

        // Double confirm subscribe
        state.mark_subscribe("orderbook");
        state.confirm_subscribe("orderbook");
        state.confirm_subscribe("orderbook"); // Second confirm is idempotent
        assert_eq!(state.len(), 1);

        // Unsubscribe something that was never subscribed
        state.mark_unsubscribe("nonexistent");
        state.confirm_unsubscribe("nonexistent");
        assert_eq!(state.len(), 1); // Still 1
    }

    #[rstest]
    fn test_mark_failure() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        state.mark_failure("tickers.BTCUSDT");

        assert_eq!(state.len(), 0);
        assert_eq!(state.pending_subscribe_topics(), vec!["tickers.BTCUSDT"]);
    }

    #[rstest]
    fn test_mark_failure_moves_to_pending() {
        let state = SubscriptionState::new('.');

        // Subscribe and confirm
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 1);
        assert!(state.pending_subscribe_topics().is_empty());

        // Mark as failed
        state.mark_failure("tickers.BTCUSDT");

        // Should be removed from confirmed and back in pending
        assert_eq!(state.len(), 0);
        assert_eq!(state.pending_subscribe_topics(), vec!["tickers.BTCUSDT"]);

        // all_topics should still include it for reconnection
        assert_eq!(state.all_topics(), vec!["tickers.BTCUSDT"]);
    }

    #[rstest]
    fn test_mark_failure_respects_pending_unsubscribe() {
        let state = SubscriptionState::new('.');

        // Subscribe and confirm
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 1);

        // User unsubscribes
        state.mark_unsubscribe("tickers.BTCUSDT");
        assert_eq!(state.len(), 0);
        assert_eq!(state.pending_unsubscribe_topics(), vec!["tickers.BTCUSDT"]);

        // Meanwhile, a network error triggers mark_failure
        state.mark_failure("tickers.BTCUSDT");

        // Should NOT be added to pending_subscribe (user wanted to unsubscribe)
        assert!(state.pending_subscribe_topics().is_empty());
        assert_eq!(state.pending_unsubscribe_topics(), vec!["tickers.BTCUSDT"]);

        // all_topics should NOT include it
        assert!(state.all_topics().is_empty());

        // Confirm unsubscribe
        state.confirm_unsubscribe("tickers.BTCUSDT");
        assert!(state.is_empty());
    }

    #[rstest]
    fn test_reconnection_scenario() {
        let state = SubscriptionState::new('.');

        // Initial subscriptions
        state.add_reference("tickers.BTCUSDT");
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");

        state.add_reference("tickers.ETHUSDT");
        state.mark_subscribe("tickers.ETHUSDT");
        state.confirm_subscribe("tickers.ETHUSDT");

        state.add_reference("orderbook");
        state.mark_subscribe("orderbook");
        state.confirm_subscribe("orderbook");

        assert_eq!(state.len(), 3);

        // Simulate disconnect - topics should be available for resubscription
        let topics_to_resubscribe = state.all_topics();
        assert_eq!(topics_to_resubscribe.len(), 3);
        assert!(topics_to_resubscribe.contains(&"tickers.BTCUSDT".to_string()));
        assert!(topics_to_resubscribe.contains(&"tickers.ETHUSDT".to_string()));
        assert!(topics_to_resubscribe.contains(&"orderbook".to_string()));

        // On reconnect, mark all as pending again
        for topic in &topics_to_resubscribe {
            state.mark_subscribe(topic);
        }

        // Simulate server confirmations
        for topic in &topics_to_resubscribe {
            state.confirm_subscribe(topic);
        }

        // Should still have all 3 subscriptions
        assert_eq!(state.len(), 3);
        assert_eq!(state.all_topics().len(), 3);
    }

    #[rstest]
    fn test_reconnection_with_partial_state() {
        let state = SubscriptionState::new('.');

        // Setup: Some confirmed, some pending subscribe, some pending unsubscribe
        // Confirmed
        state.add_reference("confirmed.BTCUSDT");
        state.mark_subscribe("confirmed.BTCUSDT");
        state.confirm_subscribe("confirmed.BTCUSDT");

        // Pending subscribe (not yet confirmed)
        state.add_reference("pending.ETHUSDT");
        state.mark_subscribe("pending.ETHUSDT");

        // Pending unsubscribe (user cancelled)
        state.mark_subscribe("cancelled.XRPUSDT");
        state.confirm_subscribe("cancelled.XRPUSDT");
        state.mark_unsubscribe("cancelled.XRPUSDT");

        // Verify state before reconnect
        assert_eq!(state.len(), 1); // Only confirmed.BTCUSDT
        let all = state.all_topics();
        assert_eq!(all.len(), 2); // confirmed + pending_subscribe (not pending_unsubscribe)
        assert!(all.contains(&"confirmed.BTCUSDT".to_string()));
        assert!(all.contains(&"pending.ETHUSDT".to_string()));
        assert!(!all.contains(&"cancelled.XRPUSDT".to_string())); // Should NOT be included

        // Simulate disconnect and reconnect
        let topics_to_resubscribe = state.reset_after_reconnect();
        assert_eq!(
            topics_to_resubscribe,
            [
                "confirmed.BTCUSDT".to_string(),
                "pending.ETHUSDT".to_string()
            ]
        );
        assert_eq!(state.reset_after_reconnect(), topics_to_resubscribe);
        assert_eq!(state.len(), 0);
        assert_eq!(
            state.pending_subscribe_topics(),
            [
                "confirmed.BTCUSDT".to_string(),
                "pending.ETHUSDT".to_string()
            ]
        );
        assert!(state.pending_unsubscribe_topics().is_empty());
        assert_eq!(state.get_reference_count("confirmed.BTCUSDT"), 1);
        assert_eq!(state.get_reference_count("pending.ETHUSDT"), 1);

        // Server confirms both
        for topic in &topics_to_resubscribe {
            state.confirm_subscribe(topic);
        }

        // Verify final state
        assert_eq!(state.len(), 2); // Both confirmed
        let final_topics = state.all_topics();
        assert_eq!(final_topics.len(), 2);
        assert!(final_topics.contains(&"confirmed.BTCUSDT".to_string()));
        assert!(final_topics.contains(&"pending.ETHUSDT".to_string()));
        assert!(!final_topics.contains(&"cancelled.XRPUSDT".to_string()));
    }

    #[rstest]
    fn test_reference_counting_single_topic() {
        let state = SubscriptionState::new('.');

        assert!(state.add_reference("tickers.BTCUSDT"));
        assert_eq!(state.get_reference_count("tickers.BTCUSDT"), 1);

        assert!(!state.add_reference("tickers.BTCUSDT"));
        assert_eq!(state.get_reference_count("tickers.BTCUSDT"), 2);

        assert!(!state.remove_reference("tickers.BTCUSDT"));
        assert_eq!(state.get_reference_count("tickers.BTCUSDT"), 1);

        assert!(state.remove_reference("tickers.BTCUSDT"));
        assert_eq!(state.get_reference_count("tickers.BTCUSDT"), 0);
    }

    #[rstest]
    fn test_reference_counting_multiple_topics() {
        let state = SubscriptionState::new('.');

        assert!(state.add_reference("tickers.BTCUSDT"));
        assert!(state.add_reference("tickers.ETHUSDT"));

        assert!(!state.add_reference("tickers.BTCUSDT"));
        assert_eq!(state.get_reference_count("tickers.BTCUSDT"), 2);
        assert_eq!(state.get_reference_count("tickers.ETHUSDT"), 1);

        assert!(!state.remove_reference("tickers.BTCUSDT"));
        assert!(state.remove_reference("tickers.ETHUSDT"));
    }

    #[rstest]
    fn test_remove_reference_nonexistent_topic() {
        let state = SubscriptionState::new('.');

        // Removing reference to topic that was never added
        let should_unsubscribe = state.remove_reference("nonexistent");

        // Should return false and not crash
        assert!(!should_unsubscribe);
        assert_eq!(state.get_reference_count("nonexistent"), 0);
    }

    #[rstest]
    fn test_reference_count_underflow_safety() {
        let state = SubscriptionState::new('.');

        // Remove without ever adding
        assert!(!state.remove_reference("never.added"));
        assert_eq!(state.get_reference_count("never.added"), 0);

        // Add one, remove multiple times
        state.add_reference("once.added");
        assert_eq!(state.get_reference_count("once.added"), 1);

        assert!(state.remove_reference("once.added")); // Should return true (last ref)
        assert_eq!(state.get_reference_count("once.added"), 0);

        assert!(!state.remove_reference("once.added")); // Should not crash, returns false
        assert!(!state.remove_reference("once.added")); // Multiple times
        assert_eq!(state.get_reference_count("once.added"), 0);

        // Verify we can add again after underflow attempts
        assert!(state.add_reference("once.added"));
        assert_eq!(state.get_reference_count("once.added"), 1);
    }

    #[rstest]
    #[tokio::test(flavor = "multi_thread", worker_threads = 4)]
    async fn test_concurrent_reference_counting_same_topic() {
        let state = Arc::new(SubscriptionState::new('.'));
        let topic = "tickers.BTCUSDT";
        let mut handles = vec![];

        // Spawn 10 tasks all adding 10 references to the same topic
        for _ in 0..10 {
            let state_clone = Arc::clone(&state);

            let handle = tokio::spawn(async move {
                for _ in 0..10 {
                    state_clone.add_reference(topic);
                }
            });
            handles.push(handle);
        }

        for handle in handles {
            handle.await.unwrap();
        }

        // Should have exactly 100 references (10 tasks * 10 refs each)
        assert_eq!(state.get_reference_count(topic), 100);

        // Now remove 50 references sequentially
        for _ in 0..50 {
            state.remove_reference(topic);
        }

        // Should have exactly 50 references remaining
        assert_eq!(state.get_reference_count(topic), 50);
    }

    #[rstest]
    fn test_clear() {
        let state = SubscriptionState::new('.');
        state.mark_subscribe("tickers.BTCUSDT");
        state.confirm_subscribe("tickers.BTCUSDT");
        state.add_reference("tickers.BTCUSDT");

        state.clear();

        assert!(state.is_empty());
        assert_eq!(state.get_reference_count("tickers.BTCUSDT"), 0);
    }

    #[rstest]
    fn test_clear_resets_all_state() {
        let state = SubscriptionState::new('.');

        // Add multiple subscriptions and references
        for i in 0..10 {
            let topic = format!("channel{i}.SYMBOL");
            state.add_reference(&topic);
            state.add_reference(&topic); // Add twice
            state.mark_subscribe(&topic);
            state.confirm_subscribe(&topic);
        }

        assert_eq!(state.len(), 10);
        assert!(!state.is_empty());

        // Clear everything
        state.clear();

        // Verify complete reset
        assert_eq!(state.len(), 0);
        assert!(state.is_empty());
        assert!(state.all_topics().is_empty());
        assert!(state.pending_subscribe_topics().is_empty());
        assert!(state.pending_unsubscribe_topics().is_empty());

        // Verify reference counts are cleared
        for i in 0..10 {
            let topic = format!("channel{i}.SYMBOL");
            assert_eq!(state.get_reference_count(&topic), 0);
        }
    }

    #[rstest]
    #[tokio::test(flavor = "multi_thread", worker_threads = 4)]
    async fn test_concurrent_subscribe_same_topic() {
        let state = Arc::new(SubscriptionState::new('.'));
        let mut handles = vec![];

        // Spawn 10 tasks all subscribing to the same topic
        for _ in 0..10 {
            let state_clone = Arc::clone(&state);
            let handle = tokio::spawn(async move {
                state_clone.add_reference("tickers.BTCUSDT");
                state_clone.mark_subscribe("tickers.BTCUSDT");
                state_clone.confirm_subscribe("tickers.BTCUSDT");
            });
            handles.push(handle);
        }

        for handle in handles {
            handle.await.unwrap();
        }

        // Reference count should be exactly 10
        assert_eq!(state.get_reference_count("tickers.BTCUSDT"), 10);
        assert_eq!(state.len(), 1);
    }

    #[rstest]
    #[tokio::test(flavor = "multi_thread", worker_threads = 4)]
    async fn test_concurrent_subscribe_unsubscribe() {
        let state = Arc::new(SubscriptionState::new('.'));
        let mut handles = vec![];

        // Spawn 20 tasks, each adding 2 references to their own unique topic
        // This ensures deterministic behavior - we know exactly what the final state should be
        for i in 0..20 {
            let state_clone = Arc::clone(&state);

            let handle = tokio::spawn(async move {
                let topic = format!("tickers.SYMBOL{i}");
                // Add 2 references
                state_clone.add_reference(&topic);
                state_clone.add_reference(&topic);
                state_clone.mark_subscribe(&topic);
                state_clone.confirm_subscribe(&topic);

                // Remove 1 reference (should still have 1 remaining)
                state_clone.remove_reference(&topic);
            });
            handles.push(handle);
        }

        for handle in handles {
            handle.await.unwrap();
        }

        // Each of the 20 topics should still have 1 reference
        for i in 0..20 {
            let topic = format!("tickers.SYMBOL{i}");
            assert_eq!(state.get_reference_count(&topic), 1);
        }

        // Should have exactly 20 confirmed subscriptions
        assert_eq!(state.len(), 20);
        assert!(!state.is_empty());
    }

    #[rstest]
    #[tokio::test(flavor = "multi_thread", worker_threads = 8)]
    async fn test_concurrent_stress_mixed_operations() {
        let state = Arc::new(SubscriptionState::new('.'));
        let mut handles = vec![];

        // Spawn 50 tasks doing random interleaved operations
        for i in 0..50 {
            let state_clone = Arc::clone(&state);

            let handle = tokio::spawn(async move {
                let topic1 = format!("channel.SYMBOL{i}");
                let topic2 = format!("channel.SYMBOL{}", i + 100);

                // Add references
                state_clone.add_reference(&topic1);
                state_clone.add_reference(&topic2);

                // Mark and confirm subscriptions
                state_clone.mark_subscribe(&topic1);
                state_clone.confirm_subscribe(&topic1);
                state_clone.mark_subscribe(&topic2);

                // Interleave some unsubscribes
                if i % 3 == 0 {
                    state_clone.mark_unsubscribe(&topic1);
                    state_clone.confirm_unsubscribe(&topic1);
                }

                // More reference operations
                state_clone.add_reference(&topic2);
                state_clone.remove_reference(&topic2);

                // Confirm topic2
                state_clone.confirm_subscribe(&topic2);
            });
            handles.push(handle);
        }

        for handle in handles {
            handle.await.unwrap();
        }

        let actual = state.all_topics().into_iter().collect::<AHashSet<_>>();
        let expected = (0..50)
            .flat_map(|i| {
                let topic2 = format!("channel.SYMBOL{}", i + 100);
                (i % 3 != 0)
                    .then(|| format!("channel.SYMBOL{i}"))
                    .into_iter()
                    .chain(std::iter::once(topic2))
            })
            .collect::<AHashSet<_>>();

        assert_eq!(actual, expected);
        assert_eq!(state.len(), 83);
        assert!(state.pending_subscribe_topics().is_empty());
        assert!(state.pending_unsubscribe_topics().is_empty());
        assert_eq!(state.reference_counts.len(), 100);
    }

    #[rstest]
    #[tokio::test(flavor = "multi_thread", worker_threads = 8)]
    async fn test_stress_rapid_resubscribe_pattern() {
        // Stress test the race condition we fixed: rapid unsubscribe -> resubscribe
        let state = Arc::new(SubscriptionState::new('.'));
        let mut handles = vec![];

        for i in 0..100 {
            let state_clone = Arc::clone(&state);

            let handle = tokio::spawn(async move {
                let topic = format!("rapid.SYMBOL{}", i % 10); // 10 unique topics, lots of contention

                // Initial subscribe
                state_clone.mark_subscribe(&topic);
                state_clone.confirm_subscribe(&topic);

                // Rapid unsubscribe -> resubscribe (race condition scenario)
                state_clone.mark_unsubscribe(&topic);
                // Immediately resubscribe before unsubscribe ACK
                state_clone.mark_subscribe(&topic);
                // Now unsubscribe ACK arrives
                state_clone.confirm_unsubscribe(&topic);
                // Subscribe ACK arrives
                state_clone.confirm_subscribe(&topic);
            });
            handles.push(handle);
        }

        for handle in handles {
            handle.await.unwrap();
        }

        check_invariants(&state, "After rapid resubscribe stress test");
    }

    #[rstest]
    #[tokio::test(flavor = "multi_thread", worker_threads = 8)]
    async fn test_stress_failure_recovery_loop() {
        // Stress test failure -> recovery loops
        // Each task gets its own unique topic to avoid race conditions in the test itself
        let state = Arc::new(SubscriptionState::new('.'));
        let mut handles = vec![];

        for i in 0..30 {
            let state_clone = Arc::clone(&state);

            let handle = tokio::spawn(async move {
                let topic = format!("failure.SYMBOL{i}"); // Unique topic per task

                // Subscribe and confirm
                state_clone.mark_subscribe(&topic);
                state_clone.confirm_subscribe(&topic);

                // Simulate multiple failures and recoveries
                for _ in 0..5 {
                    state_clone.mark_failure(&topic);
                    state_clone.confirm_subscribe(&topic); // Re-confirm after retry
                }
            });
            handles.push(handle);
        }

        for handle in handles {
            handle.await.unwrap();
        }

        check_invariants(&state, "After failure recovery loops");

        // All should eventually be confirmed (30 unique topics)
        assert_eq!(state.len(), 30);
    }

    #[rstest]
    fn test_exhaustive_two_step_transitions() {
        let operations = [
            "mark_subscribe",
            "confirm_subscribe",
            "mark_unsubscribe",
            "confirm_unsubscribe",
            "mark_failure",
        ];

        for &op1 in &operations {
            for &op2 in &operations {
                let state = SubscriptionState::new('.');
                let topic = "test.TOPIC";

                // Apply two operations
                apply_op(&state, op1, topic);
                apply_op(&state, op2, topic);

                // Verify invariants hold
                check_invariants(&state, &format!("{op1} -> {op2}"));
                check_topic_exclusivity(&state, topic, &format!("{op1} -> {op2}"));
            }
        }
    }

    fn apply_op(state: &SubscriptionState, op: &str, topic: &str) {
        match op {
            "mark_subscribe" => state.mark_subscribe(topic),
            "confirm_subscribe" => state.confirm_subscribe(topic),
            "mark_unsubscribe" => state.mark_unsubscribe(topic),
            "confirm_unsubscribe" => state.confirm_unsubscribe(topic),
            "mark_failure" => state.mark_failure(topic),
            _ => panic!("Unknown operation: {op}"),
        }
    }

    /// Verifies all invariants of the subscription state.
    ///
    /// # Invariants
    ///
    /// 1. **Mutual exclusivity**: A topic cannot exist in multiple states simultaneously
    ///    (one of: confirmed, `pending_subscribe`, `pending_unsubscribe`, or none).
    /// 2. **`all_topics` consistency**: `all_topics()` must equal `confirmed ∪ pending_subscribe`
    /// 3. **len consistency**: `len()` must equal total count of symbols in confirmed map
    /// 4. **`is_empty` consistency**: `is_empty()` true iff all maps are empty
    /// 5. **Reference count non-negative**: All reference counts >= 0
    fn check_invariants(state: &SubscriptionState, label: &str) {
        // Collect all topics from each state
        let confirmed_topics: AHashSet<String> = state
            .topics_from_map(&state.confirmed)
            .into_iter()
            .collect();
        let pending_sub_topics: AHashSet<String> =
            state.pending_subscribe_topics().into_iter().collect();
        let pending_unsub_topics: AHashSet<String> =
            state.pending_unsubscribe_topics().into_iter().collect();

        // INVARIANT 1: Mutual exclusivity - no topic in multiple states
        let confirmed_and_pending_sub: Vec<_> =
            confirmed_topics.intersection(&pending_sub_topics).collect();
        assert!(
            confirmed_and_pending_sub.is_empty(),
            "{label}: Topic in both confirmed and pending_subscribe: {confirmed_and_pending_sub:?}"
        );

        let confirmed_and_pending_unsub: Vec<_> = confirmed_topics
            .intersection(&pending_unsub_topics)
            .collect();
        assert!(
            confirmed_and_pending_unsub.is_empty(),
            "{label}: Topic in both confirmed and pending_unsubscribe: {confirmed_and_pending_unsub:?}"
        );

        let pending_sub_and_unsub: Vec<_> = pending_sub_topics
            .intersection(&pending_unsub_topics)
            .collect();
        assert!(
            pending_sub_and_unsub.is_empty(),
            "{label}: Topic in both pending_subscribe and pending_unsubscribe: {pending_sub_and_unsub:?}"
        );

        // INVARIANT 2: all_topics() == confirmed ∪ pending_subscribe
        let all_topics: AHashSet<String> = state.all_topics().into_iter().collect();
        let expected_all: AHashSet<String> = confirmed_topics
            .union(&pending_sub_topics)
            .cloned()
            .collect();
        assert_eq!(
            all_topics, expected_all,
            "{label}: all_topics() doesn't match confirmed ∪ pending_subscribe"
        );

        // Ensure pending_unsubscribe is NOT in all_topics
        for topic in &pending_unsub_topics {
            assert!(
                !all_topics.contains(topic),
                "{label}: pending_unsubscribe topic {topic} incorrectly in all_topics()"
            );
        }

        // INVARIANT 3: len() == sum of confirmed symbol counts
        let expected_len: usize = state
            .confirmed
            .iter()
            .map(|entry| entry.value().len())
            .sum();
        assert_eq!(
            state.len(),
            expected_len,
            "{label}: len() mismatch. Expected {expected_len}, was {}",
            state.len()
        );

        // INVARIANT 4: is_empty() consistency
        let should_be_empty = state.confirmed.is_empty()
            && pending_sub_topics.is_empty()
            && pending_unsub_topics.is_empty();
        assert_eq!(
            state.is_empty(),
            should_be_empty,
            "{label}: is_empty() inconsistent. Maps empty: {should_be_empty}, is_empty(): {}",
            state.is_empty()
        );

        // INVARIANT 5: Reference counts non-negative (NonZeroUsize enforces > 0, absence = 0)
        for entry in state.reference_counts.iter() {
            let count = entry.value().get();
            assert!(
                count > 0,
                "{label}: Reference count should be NonZeroUsize (> 0), was {count} for {:?}",
                entry.key()
            );
        }
    }

    /// Checks that a topic exists in exactly one of the three states or none.
    fn check_topic_exclusivity(state: &SubscriptionState, topic: &str, label: &str) {
        let (channel, symbol) = split_topic(topic, state.delimiter);

        let in_confirmed = is_tracked(&state.confirmed, channel, symbol);
        let in_pending_sub = is_tracked(&state.pending_subscribe, channel, symbol);
        let in_pending_unsub = is_tracked(&state.pending_unsubscribe, channel, symbol);

        let count = [in_confirmed, in_pending_sub, in_pending_unsub]
            .iter()
            .filter(|&&x| x)
            .count();

        assert!(
            count <= 1,
            "{label}: Topic {topic} in {count} states (should be 0 or 1). \
             confirmed: {in_confirmed}, pending_sub: {in_pending_sub}, pending_unsub: {in_pending_unsub}"
        );
    }

    #[cfg(test)]
    mod property_tests {
        use ahash::AHashMap;
        use proptest::prelude::*;

        use super::*;

        #[derive(Debug, Clone)]
        enum Operation {
            MarkSubscribe(String),
            ConfirmSubscribe(String),
            MarkUnsubscribe(String),
            ConfirmUnsubscribe(String),
            MarkFailure(String),
            AddReference(String),
            RemoveReference(String),
            Clear,
        }

        #[derive(Debug, Clone, Copy, PartialEq, Eq)]
        enum ModelState {
            Confirmed,
            PendingSubscribe,
            PendingUnsubscribe,
        }

        // Strategy for generating valid topics
        fn topic_strategy() -> impl Strategy<Value = String> {
            prop_oneof![
                // Symbol-level topics
                (any::<u8>(), any::<u8>())
                    .prop_map(|(ch, sym)| { format!("channel{}.SYMBOL{}", ch % 5, sym % 10) }),
                // Channel-level topics (no symbol)
                any::<u8>().prop_map(|ch| format!("channel{}", ch % 5)),
            ]
        }

        // Strategy for generating random operations
        fn operation_strategy() -> impl Strategy<Value = Operation> {
            topic_strategy().prop_flat_map(|topic| {
                prop_oneof![
                    Just(Operation::MarkSubscribe(topic.clone())),
                    Just(Operation::ConfirmSubscribe(topic.clone())),
                    Just(Operation::MarkUnsubscribe(topic.clone())),
                    Just(Operation::ConfirmUnsubscribe(topic.clone())),
                    Just(Operation::MarkFailure(topic.clone())),
                    Just(Operation::AddReference(topic.clone())),
                    Just(Operation::RemoveReference(topic)),
                    Just(Operation::Clear),
                ]
            })
        }

        // Apply an operation to the state
        fn apply_operation(state: &SubscriptionState, op: &Operation) {
            match op {
                Operation::MarkSubscribe(topic) => state.mark_subscribe(topic),
                Operation::ConfirmSubscribe(topic) => state.confirm_subscribe(topic),
                Operation::MarkUnsubscribe(topic) => state.mark_unsubscribe(topic),
                Operation::ConfirmUnsubscribe(topic) => state.confirm_unsubscribe(topic),
                Operation::MarkFailure(topic) => state.mark_failure(topic),
                Operation::AddReference(topic) => {
                    state.add_reference(topic);
                }
                Operation::RemoveReference(topic) => {
                    state.remove_reference(topic);
                }
                Operation::Clear => state.clear(),
            }
        }

        fn apply_model_operation(model: &mut AHashMap<String, ModelState>, op: &Operation) {
            match op {
                Operation::MarkSubscribe(topic) => {
                    if model.get(topic) != Some(&ModelState::Confirmed) {
                        model.insert(topic.clone(), ModelState::PendingSubscribe);
                    }
                }
                Operation::ConfirmSubscribe(topic) => {
                    if matches!(
                        model.get(topic),
                        Some(ModelState::PendingSubscribe | ModelState::Confirmed)
                    ) {
                        model.insert(topic.clone(), ModelState::Confirmed);
                    }
                }
                Operation::MarkUnsubscribe(topic) => {
                    model.insert(topic.clone(), ModelState::PendingUnsubscribe);
                }
                Operation::ConfirmUnsubscribe(topic) => {
                    if model.get(topic) == Some(&ModelState::PendingUnsubscribe) {
                        model.remove(topic);
                    }
                }
                Operation::MarkFailure(topic) => {
                    if matches!(
                        model.get(topic),
                        Some(ModelState::PendingSubscribe | ModelState::Confirmed)
                    ) {
                        model.insert(topic.clone(), ModelState::PendingSubscribe);
                    }
                }
                Operation::AddReference(_) | Operation::RemoveReference(_) => {}
                Operation::Clear => model.clear(),
            }
        }

        fn assert_state_matches_model(
            state: &SubscriptionState,
            model: &AHashMap<String, ModelState>,
        ) {
            let topics_for = |expected_state| {
                model
                    .iter()
                    .filter(|&(_topic, state)| *state == expected_state)
                    .map(|(topic, _state)| topic.clone())
                    .collect::<AHashSet<_>>()
            };
            let confirmed = state
                .topics_from_map(&state.confirmed)
                .into_iter()
                .collect::<AHashSet<_>>();
            let pending_subscribe = state
                .pending_subscribe_topics()
                .into_iter()
                .collect::<AHashSet<_>>();
            let pending_unsubscribe = state
                .pending_unsubscribe_topics()
                .into_iter()
                .collect::<AHashSet<_>>();
            let expected_confirmed = topics_for(ModelState::Confirmed);
            let expected_pending_subscribe = topics_for(ModelState::PendingSubscribe);
            let expected_pending_unsubscribe = topics_for(ModelState::PendingUnsubscribe);
            let expected_all = expected_confirmed
                .union(&expected_pending_subscribe)
                .cloned()
                .collect::<AHashSet<_>>();
            let all = state.all_topics().into_iter().collect::<AHashSet<_>>();

            assert_eq!(confirmed, expected_confirmed);
            assert_eq!(pending_subscribe, expected_pending_subscribe);
            assert_eq!(pending_unsubscribe, expected_pending_unsubscribe);
            assert_eq!(all, expected_all);
            assert_eq!(state.len(), confirmed.len());
            assert_eq!(state.is_empty(), model.is_empty());
        }

        proptest! {
            #![proptest_config(ProptestConfig::with_cases(500))]

            /// Property: Invariants hold after any sequence of operations.
            #[rstest]
            fn prop_invariants_hold_after_operations(
                operations in prop::collection::vec(operation_strategy(), 1..50)
            ) {
                let state = SubscriptionState::new('.');
                let mut model = AHashMap::new();

                for (i, op) in operations.iter().enumerate() {
                    apply_operation(&state, op);
                    apply_model_operation(&mut model, op);

                    check_invariants(&state, &format!("After op {i}: {op:?}"));
                    assert_state_matches_model(&state, &model);
                }

                check_invariants(&state, "Final state");
                assert_state_matches_model(&state, &model);
            }

            /// Reference-count operations match an independent count model.
            #[rstest]
            fn prop_reference_counting_matches_reference(
                ops in prop::collection::vec(
                    topic_strategy().prop_flat_map(|t| {
                        prop_oneof![
                            Just(Operation::AddReference(t.clone())),
                            Just(Operation::RemoveReference(t)),
                        ]
                    }),
                    1..100
                )
            ) {
                let state = SubscriptionState::new('.');
                let mut expected = AHashMap::new();

                for op in &ops {
                    match op {
                        Operation::AddReference(topic) => {
                            let count = expected.entry(topic.clone()).or_insert(0usize);
                            let should_subscribe = *count == 0;
                            *count += 1;
                            prop_assert_eq!(state.add_reference(topic), should_subscribe);
                        }
                        Operation::RemoveReference(topic) => {
                            let count = expected.get(topic).copied().unwrap_or(0);
                            let should_unsubscribe = count == 1;
                            if should_unsubscribe {
                                expected.remove(topic);
                            } else if count > 1 {
                                *expected.get_mut(topic).unwrap() -= 1;
                            }
                            prop_assert_eq!(state.remove_reference(topic), should_unsubscribe);
                        }
                        _ => unreachable!("reference-count strategy only generates reference operations"),
                    }

                    prop_assert_eq!(state.reference_counts.len(), expected.len());
                    for (topic, count) in &expected {
                        prop_assert_eq!(state.get_reference_count(topic), *count);
                    }
                }
            }

            /// Property: all_topics() always equals confirmed ∪ pending_subscribe.
            #[rstest]
            fn prop_all_topics_is_union(
                operations in prop::collection::vec(operation_strategy(), 1..50)
            ) {
                let state = SubscriptionState::new('.');

                for op in &operations {
                    apply_operation(&state, op);

                    // Verify all_topics() == confirmed ∪ pending_subscribe
                    let all_topics: AHashSet<String> = state.all_topics().into_iter().collect();
                    let confirmed: AHashSet<String> = state.topics_from_map(&state.confirmed).into_iter().collect();
                    let pending_sub: AHashSet<String> = state.pending_subscribe_topics().into_iter().collect();
                    let expected: AHashSet<String> = confirmed.union(&pending_sub).cloned().collect();

                    assert_eq!(all_topics, expected);

                    // Ensure pending_unsubscribe topics are NOT in all_topics
                    let pending_unsub: AHashSet<String> = state.pending_unsubscribe_topics().into_iter().collect();
                    for topic in pending_unsub {
                        assert!(!all_topics.contains(&topic));
                    }
                }
            }

            /// Property: clear() resets to empty state.
            #[rstest]
            fn prop_clear_resets_completely(
                operations in prop::collection::vec(operation_strategy(), 1..30)
            ) {
                let state = SubscriptionState::new('.');

                // Apply random operations
                for op in &operations {
                    apply_operation(&state, op);
                }

                // Clear and verify complete reset
                state.clear();

                assert!(state.is_empty());
                assert_eq!(state.len(), 0);
                assert!(state.all_topics().is_empty());
                assert!(state.pending_subscribe_topics().is_empty());
                assert!(state.pending_unsubscribe_topics().is_empty());
                assert!(state.confirmed.is_empty());
                assert!(state.pending_subscribe.is_empty());
                assert!(state.pending_unsubscribe.is_empty());
                assert!(state.reference_counts.is_empty());
            }

            /// Property: Topics are mutually exclusive across states.
            #[rstest]
            fn prop_topic_mutual_exclusivity(
                operations in prop::collection::vec(operation_strategy(), 1..50),
                topic in topic_strategy()
            ) {
                let state = SubscriptionState::new('.');

                for (i, op) in operations.iter().enumerate() {
                    apply_operation(&state, op);
                    check_topic_exclusivity(&state, &topic, &format!("After op {i}: {op:?}"));
                }
            }
        }
    }
}