From 359b216f9ab4afc200890a64b34673fa86fc0f43 Mon Sep 17 00:00:00 2001
From: Zhenbo Li <lizhenbo@yahoo.com>
Date: Sat, 21 Sep 2024 10:59:06 -0400
Subject: [PATCH] trying to make deps optional

---
 fire_seq_search_server/Cargo.toml           | 12 +++++++++---
 fire_seq_search_server/src/local_llm/mod.rs | 16 +++++++++-------
 2 files changed, 18 insertions(+), 10 deletions(-)

diff --git a/fire_seq_search_server/Cargo.toml b/fire_seq_search_server/Cargo.toml
index 5245ca7..9dd0367 100644
--- a/fire_seq_search_server/Cargo.toml
+++ b/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"
+]
diff --git a/fire_seq_search_server/src/local_llm/mod.rs b/fire_seq_search_server/src/local_llm/mod.rs
index 48b94b4..f6faaca 100644
--- a/fire_seq_search_server/src/local_llm/mod.rs
+++ b/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,
+};