refactor(sessions::paths): move paths to own crate - #276

crates/sessions/src/lib.rs (1)

23-37: ⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Missing constructor from Uuid.

SessionId only provides nil() and parse_str(), but there's no way to create one from an existing Uuid. Callers that generate fresh IDs (e.g., Uuid::new_v4()) would have to do an inefficient string round-trip.

Consider adding a constructor and/or From<Uuid> impl:

Suggested additions

 impl SessionId {
+    /// Creates a new session ID from an existing UUID.
+    #[must_use]
+    pub fn new(id: Uuid) -> Self {
+        Self(id)
+    }
+
     #[must_use]
     pub fn nil() -> Self {
         Self(Uuid::nil())
     }

And optionally:

impl From<Uuid> for SessionId {
    fn from(id: Uuid) -> Self {
        Self(id)
    }
}

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@crates/sessions/src/lib.rs` around lines 23 - 37, SessionId lacks a
constructor from an existing Uuid which forces callers to stringify/parse; add a
simple constructor (e.g., pub fn from_uuid(id: Uuid) -> Self or pub fn new(id:
Uuid) -> Self) to SessionId and implement impl From<Uuid> for SessionId { fn
from(id: Uuid) -> Self { Self(id) } } so callers can directly wrap
Uuid::new_v4() without a string round-trip; update references to use the new
constructor instead of parse_str where appropriate (symbols: SessionId,
parse_str, nil, From<Uuid>).