diff --git a/crates/rutster-call-model/src/lib.rs b/crates/rutster-call-model/src/lib.rs index ca5291b..01fcdf5 100644 --- a/crates/rutster-call-model/src/lib.rs +++ b/crates/rutster-call-model/src/lib.rs @@ -47,7 +47,6 @@ mod tests { } } -use std::time::Instant; use uuid::Uuid; /// Newtype wrapping a `Uuid` for the channel id. @@ -127,17 +126,17 @@ pub enum Direction { /// - `media: Option` — second consumer. /// - `audiohooks: Vec` — escalation rung 2. /// - `tap: Option` — step 2. +/// +/// (F6 review patch: an earlier sketch carried `created_at: Instant` "for +/// the idle timeout," but the timer keys off `RtcSession.last_rtp_rx`, not +/// creation time. The field was dead; the work-pool would have populated +/// it on every `new_inbound` and never read it. Dropped along with its +/// `Instant` import.) #[derive(Debug)] pub struct Channel { pub id: ChannelId, pub state: ChannelState, pub direction: Direction, - /// For the 60 s idle timeout (spec §4.5). `Instant` is a monotonic - /// clock — choosing it over `SystemTime` means we're measuring - /// elapsed wall-time within this process, NOT a calendar time the - /// user could change mid-call. The monotonic clock is the right - /// tool for "has this peer been silent for 60 seconds?" - pub created_at: Instant, } impl Channel { @@ -147,7 +146,6 @@ impl Channel { id: ChannelId::new(), state: ChannelState::New, direction: Direction::Inbound, - created_at: Instant::now(), } } }