feat(enterprise): implement logs command for cluster event retrieval (#241)

* feat(enterprise): implement logs command for cluster event retrieval

- Add LogsCommands enum with List subcommand for retrieving cluster logs
- Support query parameters: stime, etime, order, limit, offset
- Fix LogEntry structure to match actual API (time, type, extra fields)
- Update LogsQuery to use correct API parameters
- Add comprehensive tests for all query parameter combinations
- Wire up command in main enterprise command structure

Note: Only implements GET /v1/logs endpoint as that's the only logs endpoint
that exists in the Redis Enterprise API. Issue #157 requests features that
don't exist in the API (node/database specific logs, log files, rotation, etc.)

* chore: clean up logs implementation

- Remove unnecessary #[allow(dead_code)] attribute
- Add module documentation

* chore: add #[allow(dead_code)] to match cloud implementation pattern

Suppress false positive warnings for functions that are used by the binary
but appear unused from the library's perspective.

Author: joshrotenberg

crates/redis-enterprise/src/logs.rs

@@ -10,37 +10,34 @@ use crate::error::Result;
 use serde::{Deserialize, Serialize};
 use serde_json::Value;
 
-/// Log entry
+/// Log entry (cluster event)
 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct LogEntry {
-    pub id: u64,
+    /// Timestamp when event happened
     pub time: String,
-    pub level: String,
-    pub component: Option<String>,
-    pub message: String,
-    pub node_uid: Option<u32>,
-    pub bdb_uid: Option<u32>,
-    pub user: Option<String>,
 
+    /// Event type - determines what additional fields are available
+    #[serde(rename = "type")]
+    pub event_type: String,
+
+    /// Additional fields based on event type
     #[serde(flatten)]
     pub extra: Value,
 }
 
 /// Logs query parameters
-#[derive(Debug, Serialize)]
+#[derive(Debug, Serialize, Default)]
 pub struct LogsQuery {
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub limit: Option<u32>,
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub offset: Option<u32>,
+    pub stime: Option<String>,
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub level: Option<String>,
+    pub etime: Option<String>,
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub component: Option<String>,
+    pub order: Option<String>,
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub node_uid: Option<u32>,
+    pub limit: Option<u32>,
     #[serde(skip_serializing_if = "Option::is_none")]
-    pub bdb_uid: Option<u32>,
+    pub offset: Option<u32>,
 }
 
 /// Logs handler for querying event logs
@@ -63,9 +60,4 @@ impl LogsHandler {
             self.client.get("/v1/logs").await
         }
     }
-
-    /// Get specific log entry
-    pub async fn get(&self, id: u64) -> Result<LogEntry> {
-        self.client.get(&format!("/v1/logs/{}", id)).await
-    }
 }