trying to make deps optional

fire_seq_search_server/Cargo.toml

@@ -60,10 +60,16 @@ pulldown-cmark = { version = "0.9.2", default-features = false }
 pdf-extract-temporary-mitigation-panic = "0.7.1"
 
 
+
+# TODO Currently turn them off will make cargo build fail
+#   I should make these deps optional, so those who doesn't want LLM could have a smaller binary
 sha256 = { version = "1.5.0", optional = true }
-reqwest = { version = "0.12", features = ["json"], optional = true }
-serde_derive = { version = "1.0.209", optional = true}
+reqwest = { version = "0.12", features = ["json"], optional = false }
+serde_derive = { version = "1.0.209", optional = false}
 
 [features]
 #default = ["llm"]
-llm = ["sha256", "reqwest", "serde_derive"]
+llm = ["sha256",
+    #"serde_derive",
+        #"request"
+]

fire_seq_search_server/src/local_llm/mod.rs

@@ -17,13 +17,15 @@ use std::borrow::Cow;
 use std::borrow::Cow::Borrowed;
 
 
-use reqwest;
-use reqwest::StatusCode;
-use shellexpand::tilde;
-
-use serde_derive::Deserialize;
-use serde_derive::Serialize;
-use serde;
+//#[cfg(feature = "llm")]
+use {
+    reqwest,
+    reqwest::StatusCode,
+    shellexpand::tilde,
+
+    serde_derive::Deserialize,
+    serde_derive::Serialize,
+};