Merge pull request #38 from lucaromagnoli/fix/windows-unique-ptr

fix: Windows compilation error with std::unique_ptr<void>

Author: lucaromagnoli

.github/workflows/ci.yml

@@ -7,13 +7,6 @@ on:
       - '**.md'
       - 'docs/**'
       - 'LICENSE'
-  pull_request:
-    branches: [ main, develop ]
-    paths-ignore:  # Skip CI for documentation changes
-      - '**.md'
-      - 'docs/**'
-      - '.github/**'
-      - 'LICENSE'
 
 jobs:
   build-and-test:

CMakeLists.txt

@@ -50,8 +50,10 @@ FetchContent_Declare(
     GIT_TAG v0.15.3
 )
 
-# Configure httplib to use OpenSSL
+# Configure httplib to use OpenSSL and disable unnecessary features
 set(HTTPLIB_USE_OPENSSL ON)
+set(HTTPLIB_USE_ZLIB OFF CACHE BOOL "" FORCE)
+set(HTTPLIB_USE_BROTLI OFF CACHE BOOL "" FORCE)
 
 # Make dependencies available
 FetchContent_MakeAvailable(nlohmann_json httplib)
@@ -94,7 +96,7 @@ target_link_libraries(llmcpp
 
 # Platform-specific libraries
 if(WIN32)
-    target_link_libraries(llmcpp PRIVATE ws2_32 wsock32)
+    target_link_libraries(llmcpp PRIVATE ws2_32 wsock32 crypt32)
 elseif(APPLE)
     target_link_libraries(llmcpp PRIVATE 
         "-framework CoreFoundation" 

src/openai/OpenAIHttpClient.cpp

@@ -100,7 +100,7 @@ class OpenAIHttpClient::HttpClientImpl {
 #ifdef CPPHTTPLIB_OPENSSL_SUPPORT
     std::unique_ptr<httplib::SSLClient> client_;
 #else
-    std::unique_ptr<void> client_;  // Placeholder when SSL not available
+    std::unique_ptr<httplib::Client> client_;  // Use regular HTTP client when SSL not available
 #endif
     std::string hostname_;
     std::string basePath_;