refactor: Simplify CMake dependencies and fix clang-tidy warnings

Streamlined CMakeLists.txt files to avoid redundant transitive dependencies,
and resolved remaining clang-tidy warnings for struct initialization and
unused return values to improve CI code quality checks.

Author: libraz

src/CMakeLists.txt

@@ -26,13 +26,7 @@ add_executable(mygramdb main.cpp)
 
 target_link_libraries(mygramdb PRIVATE
   mygramdb_config
-  mygramdb_index
-  mygramdb_storage
   mygramdb_server
-  mygramdb_utils
-  mygramdb_query
-  spdlog::spdlog
-  Threads::Threads
 )
 
 # Link MySQL library if available

src/cli/mygram-cli.cpp

@@ -331,7 +331,7 @@ class MygramClient {
         continue;
       }
 
-      struct sockaddr_in server_addr{};
+      struct sockaddr_in server_addr = {};
       server_addr.sin_family = AF_INET;
       server_addr.sin_port = htons(config_.port);
 

src/client/mygramclient.cpp

@@ -147,14 +147,14 @@ class MygramClient::Impl {
     }
 
     // Set socket timeout
-    struct timeval timeout_val{};
+    struct timeval timeout_val = {};
     timeout_val.tv_sec = static_cast<decltype(timeout_val.tv_sec)>(config_.timeout_ms / 1000);
     timeout_val.tv_usec =
         static_cast<decltype(timeout_val.tv_usec)>((config_.timeout_ms % 1000) * 1000);
     setsockopt(sock_, SOL_SOCKET, SO_RCVTIMEO, &timeout_val, sizeof(timeout_val));
     setsockopt(sock_, SOL_SOCKET, SO_SNDTIMEO, &timeout_val, sizeof(timeout_val));
 
-    struct sockaddr_in server_addr{};
+    struct sockaddr_in server_addr = {};
     server_addr.sin_family = AF_INET;
     server_addr.sin_port = htons(config_.port);
 

src/server/CMakeLists.txt

@@ -11,10 +11,6 @@ target_include_directories(mygramdb_server PUBLIC
 
 target_link_libraries(mygramdb_server PUBLIC
   mygramdb_query
-  mygramdb_index
-  mygramdb_storage
-  mygramdb_utils
-  spdlog::spdlog
   httplib::httplib
   nlohmann_json::nlohmann_json
   Threads::Threads

src/server/tcp_server.cpp

@@ -82,7 +82,7 @@ bool TcpServer::Start() {
   }
 
   // Bind
-  struct sockaddr_in address{};
+  struct sockaddr_in address = {};
   std::memset(&address, 0, sizeof(address));
   address.sin_family = AF_INET;
   address.sin_addr.s_addr = INADDR_ANY;
@@ -169,7 +169,7 @@ void TcpServer::AcceptThreadFunc() {
   spdlog::info("Accept thread started");
 
   while (!should_stop_) {
-    struct sockaddr_in client_addr{};
+    struct sockaddr_in client_addr = {};
     socklen_t client_len = sizeof(client_addr);
 
     int client_fd =
@@ -824,7 +824,7 @@ std::string TcpServer::ProcessRequest(const std::string& request, ConnectionCont
 
         // Load from directory
         // Check if directory exists
-        struct stat file_stat{};
+        struct stat file_stat = {};
         if (stat(filepath.c_str(), &file_stat) != 0 || !S_ISDIR(file_stat.st_mode)) {
           loading_ = false;
           return FormatError("Snapshot directory not found: " + filepath);

tests/client/CMakeLists.txt

@@ -8,8 +8,6 @@ target_link_libraries(mygramclient_test
   mygramclient_static
   mygramdb_server
   mygramdb_mysql
-  mygramdb_index
-  mygramdb_storage
   GTest::gtest_main
 )
 

tests/query/CMakeLists.txt

@@ -32,7 +32,6 @@ add_executable(result_sorter_test
 
 target_link_libraries(result_sorter_test
   mygramdb_query
-  mygramdb_storage
   GTest::gtest_main
 )
 

tests/storage/CMakeLists.txt

@@ -54,11 +54,8 @@ if(USE_MYSQL AND MYSQL_FOUND)
   )
 
   target_link_libraries(snapshot_save_load_test
-    mygramdb_storage
-    mygramdb_index
-    mygramdb_mysql
     mygramdb_query
-    mygramdb_utils
+    mygramdb_mysql
     mygramdb_config
   )
 endif()

tests/storage/document_store_concurrent_test.cpp

@@ -122,7 +122,7 @@ TEST(DocumentStoreConcurrentTest, ConcurrentLoadAndRead) {
   // Thread that loads from file
   threads.emplace_back([&store2, &test_file, &load_done]() {
     std::this_thread::sleep_for(std::chrono::milliseconds(10));
-    store2.LoadFromFile(test_file + ".docs");
+    [[maybe_unused]] auto result = store2.LoadFromFile(test_file + ".docs");
     load_done = true;
   });