fix: resolve 5 clippy warnings (unused imports, method naming, dead code)

- chess/fingerprint.rs: remove unused VsaOps import
- qualia/felt_parse.rs: rename from_str to parse (avoid std trait confusion)
- qualia/agent_state.rs: rename from_str to parse (same reason)
- bin/server.rs: fix unused assignment + collapsible if

https://claude.ai/code/session_01PJXVVtCXFi2BvJMrM8gmwE

Author: claude

src/bin/server.rs

@@ -3228,8 +3228,6 @@ fn parse_url(url: &str) -> Result<(String, u16, String), String> {
 /// Read an HTTP response body (skip headers, read body).
 fn read_http_response(stream: &mut TcpStream) -> Result<String, String> {
     let mut reader = BufReader::new(stream.try_clone().map_err(|e| e.to_string())?);
-    let mut response = String::new();
-
     // Read status line
     let mut status_line = String::new();
     reader
@@ -3244,24 +3242,24 @@ fn read_http_response(stream: &mut TcpStream) -> Result<String, String> {
         if line.trim().is_empty() {
             break;
         }
-        if let Some((key, val)) = line.split_once(':') {
-            if key.trim().eq_ignore_ascii_case("content-length") {
-                content_length = val.trim().parse().unwrap_or(0);
-            }
+        if let Some((key, val)) = line.split_once(':')
+            && key.trim().eq_ignore_ascii_case("content-length")
+        {
+            content_length = val.trim().parse().unwrap_or(0);
         }
     }
 
     // Read body
-    if content_length > 0 {
+    let response = if content_length > 0 {
         let mut body = vec![0u8; content_length];
         std::io::Read::read_exact(&mut reader, &mut body).map_err(|e| e.to_string())?;
-        response = String::from_utf8_lossy(&body).to_string();
+        String::from_utf8_lossy(&body).to_string()
     } else {
         // Read until connection closes (chunked or unknown length)
         let mut buf = String::new();
         let _ = std::io::Read::read_to_string(&mut reader, &mut buf);
-        response = buf;
-    }
+        buf
+    };
 
     Ok(response)
 }

src/chess/fingerprint.rs

@@ -19,7 +19,7 @@
 //!             ⊕ BIND(phase,          ADDR_PHASE)
 //! ```
 
-use crate::core::{DIM_U64, Fingerprint, VsaOps};
+use crate::core::{DIM_U64, Fingerprint};
 use super::position::{ChessPosition, Color, GamePhase, PieceType};
 
 /// Bit ranges for each section of the fingerprint.

src/qualia/agent_state.rs

@@ -89,7 +89,7 @@ impl Default for PresenceMode {
 
 impl PresenceMode {
     /// Parse from string (e.g., from ada-rs PresenceMode)
-    pub fn from_str(s: &str) -> Self {
+    pub fn parse(s: &str) -> Self {
         match s.to_lowercase().as_str() {
             "wife" | "intimate" => PresenceMode::Wife,
             "work" | "focus" | "professional" => PresenceMode::Work,
@@ -923,12 +923,12 @@ mod tests {
 
     #[test]
     fn test_presence_mode_parse() {
-        assert_eq!(PresenceMode::from_str("wife"), PresenceMode::Wife);
-        assert_eq!(PresenceMode::from_str("intimate"), PresenceMode::Wife);
-        assert_eq!(PresenceMode::from_str("work"), PresenceMode::Work);
-        assert_eq!(PresenceMode::from_str("agi"), PresenceMode::Agi);
-        assert_eq!(PresenceMode::from_str("hybrid"), PresenceMode::Hybrid);
-        assert_eq!(PresenceMode::from_str("unknown"), PresenceMode::Neutral);
+        assert_eq!(PresenceMode::parse("wife"), PresenceMode::Wife);
+        assert_eq!(PresenceMode::parse("intimate"), PresenceMode::Wife);
+        assert_eq!(PresenceMode::parse("work"), PresenceMode::Work);
+        assert_eq!(PresenceMode::parse("agi"), PresenceMode::Agi);
+        assert_eq!(PresenceMode::parse("hybrid"), PresenceMode::Hybrid);
+        assert_eq!(PresenceMode::parse("unknown"), PresenceMode::Neutral);
     }
 
     // ─── CoreAxes ───

src/qualia/felt_parse.rs

@@ -109,7 +109,7 @@ impl GhostType {
     }
 
     /// Parse from string (case-insensitive).
-    pub fn from_str(s: &str) -> Option<Self> {
+    pub fn parse(s: &str) -> Option<Self> {
         match s.to_lowercase().as_str() {
             "love" => Some(GhostType::Love),
             "epiphany" => Some(GhostType::Epiphany),
@@ -1149,7 +1149,7 @@ mod tests {
     fn test_ghost_type_roundtrip() {
         for ghost_type in &GhostType::ALL {
             let s = ghost_type.as_str();
-            let parsed = GhostType::from_str(s).unwrap();
+            let parsed = GhostType::parse(s).unwrap();
             assert_eq!(*ghost_type, parsed, "roundtrip failed for {}", s);
         }
     }