fix: lint/spelling/shear CI failures

- ruff F541: remove f-prefix from string without placeholders in test_keyboard_lcm.py
- typos: serialised → serialized in ws.rs
- lint.py: add blank lines before doc comments and methods in viewer.rs
- lint.py: fix double space in ws.rs comment
- cargo-shear: remove unused clap dependency

Author: jeff-hykin

dimos/Cargo.toml

@@ -26,7 +26,6 @@ rerun = { path = "../crates/top/rerun", default-features = false, features = [
   "server",
 ] }
 
-clap = { workspace = true, features = ["derive"] }
 bincode.workspace = true
 futures-util.workspace = true
 mimalloc.workspace = true

dimos/src/interaction/ws.rs

@@ -41,7 +41,7 @@ pub enum WsEvent {
     Stop,
 }
 
-/// Sends `WsEvent`s (serialised to JSON) to a remote WebSocket server.
+/// Sends `WsEvent`s (serialized to JSON) to a remote WebSocket server.
 ///
 /// Maintains a persistent connection with automatic reconnection. The
 /// internal sender is `Clone`, so you can hand copies to multiple producers
@@ -143,7 +143,7 @@ async fn run_client(url: String, mut rx: mpsc::Receiver<String>) {
 
                 // Read task: consume incoming frames (ping → auto pong) so the
                 // server's keepalive pings get answered and the connection stays
-                // alive.  Exits when the server closes or an error occurs.
+                // alive. Exits when the server closes or an error occurs.
                 let mut read_handle = tokio::spawn(async move {
                     while let Some(frame) = reader.next().await {
                         match frame {

dimos/src/viewer.rs

@@ -17,8 +17,10 @@ static GLOBAL: re_memory::AccountingAllocator<mimalloc::MiMalloc> =
 
 /// Default WebSocket server URL
 const DEFAULT_WS_URL: &str = "ws://127.0.0.1:3030/ws";
+
 /// Minimum time between click events (debouncing)
 const CLICK_DEBOUNCE_MS: u64 = 100;
+
 /// Maximum rapid clicks before logging a warning
 const RAPID_CLICK_THRESHOLD: usize = 5;
 
@@ -36,9 +38,13 @@ impl eframe::App for DimosApp {
     }
 
     fn save(&mut self, storage: &mut dyn eframe::Storage) { self.inner.save(storage); }
+
     fn clear_color(&self, visuals: &egui::Visuals) -> [f32; 4] { self.inner.clear_color(visuals) }
+
     fn persist_egui_memory(&self) -> bool { self.inner.persist_egui_memory() }
+
     fn auto_save_interval(&self) -> Duration { self.inner.auto_save_interval() }
+
     fn raw_input_hook(&mut self, ctx: &egui::Context, raw_input: &mut egui::RawInput) {
         self.inner.raw_input_hook(ctx, raw_input);
     }

test_keyboard_lcm.py

@@ -7,6 +7,7 @@
 import struct
 import sys
 
+
 def test_twist_encoding():
     """Verify Twist encoding matches Python LCM reference."""
     # This is what the Rust viewer now produces for Twist (no Header)
@@ -33,7 +34,7 @@ def rot(h):
     assert buf.hex() == expected_hex, f"Encoding mismatch:\n  got:    {buf.hex()}\n  expect: {expected_hex}"
 
     print("PASS: Twist encoding matches Python LCM reference (56 bytes)")
-    print(f"  Channel: /cmd_vel#geometry_msgs.Twist")
+    print("  Channel: /cmd_vel#geometry_msgs.Twist")
     print(f"  Hash:    0x{twist_hash:016x}")
 
 def test_channel_name():