fix(agent): avoid header casing conflict and clean warnings

Author: GaspardKirira

examples/agent_with_cache.cpp

@@ -16,7 +16,7 @@
 #include <memory>
 #include <string_view>
 
-#include <vix/ai/agent/agent.hpp>
+#include <vix/ai/agent/AgentRuntime.hpp>
 #include <vix/print.hpp>
 
 namespace

examples/agent_with_command_tool.cpp

@@ -16,7 +16,7 @@
 #include <memory>
 #include <string_view>
 
-#include <vix/ai/agent/agent.hpp>
+#include <vix/ai/agent/AgentRuntime.hpp>
 #include <vix/json/json.hpp>
 #include <vix/print.hpp>
 

examples/basic_agent.cpp

@@ -13,7 +13,8 @@
  *  Vix.cpp
  *
  */
-#include <vix/ai/agent/agent.hpp>
+
+#include <vix/ai/agent/AgentRuntime.hpp>
 #include <vix/print.hpp>
 #include <memory>
 

examples/custom_provider.cpp

@@ -16,7 +16,8 @@
 #include <memory>
 #include <string_view>
 
-#include <vix/ai/agent/agent.hpp>
+#include <vix/ai/agent/AgentRuntime.hpp>
+
 #include <vix/print.hpp>
 
 namespace

examples/ollama_agent.cpp

@@ -15,7 +15,7 @@
  */
 #include <memory>
 
-#include <vix/ai/agent/agent.hpp>
+#include <vix/ai/agent/AgentRuntime.hpp>
 #include <vix/print.hpp>
 
 int main()

examples/public_api.cpp

@@ -13,7 +13,7 @@
  *  Vix.cpp
  *
  */
-#include <vix/ai/agent/agent.hpp>
+#include <vix/ai/agent/AgentRuntime.hpp>
 #include <vix/print.hpp>
 
 int main()

examples/scan_project.cpp

@@ -13,7 +13,7 @@
  *  Vix.cpp
  *
  */
-#include <vix/ai/agent/agent.hpp>
+#include <vix/ai/agent/AgentRuntime.hpp>
 #include <vix/print.hpp>
 
 int main()

include/vix/ai/agent/agent.hpp include/vix/ai/agent/AgentRuntime.hppinclude/vix/ai/agent/agent.hpp renamed to include/vix/ai/agent/AgentRuntime.hpp

@@ -13,8 +13,8 @@
  *  Vix.cpp
  *
  */
-#ifndef VIX_AI_AGENT_AGENT_UMBRELLA_HPP
-#define VIX_AI_AGENT_AGENT_UMBRELLA_HPP
+#ifndef VIX_AI_AGENT_AGENT_RUNTIME_HPP
+#define VIX_AI_AGENT_AGENT_RUNTIME_HPP
 
 #include <vix/ai/agent/Agent.hpp>
 #include <vix/ai/agent/AgentConfig.hpp>
@@ -48,4 +48,4 @@
 #include <vix/ai/agent/AgentRunStore.hpp>
 #include <vix/ai/agent/AgentConfigValidator.hpp>
 
-#endif // VIX_AI_AGENT_AGENT_UMBRELLA_HPP
+#endif // VIX_AI_AGENT_AGENT_RUNTIME_HPP

tests/AgentConfigTest.cpp

@@ -20,11 +20,17 @@
 #include <vix/ai/agent/AgentConfig.hpp>
 #include <vix/ai/agent/AgentConfigLoader.hpp>
 #include <vix/ai/agent/AgentConfigValidator.hpp>
+#include <vix/error/Error.hpp>
 #include <vix/env/Set.hpp>
 #include <vix/env/Unset.hpp>
 
 namespace
 {
+  void assert_no_error(const vix::error::Error &error)
+  {
+    assert(!error);
+  }
+
   void test_default_config()
   {
     vix::ai::agent::AgentConfig config;
@@ -65,13 +71,13 @@ namespace
 
   void test_environment_loader_overrides()
   {
-    vix::env::set("VIX_AGENT_TEST_PROVIDER", "ollama", true);
-    vix::env::set("VIX_AGENT_TEST_MODEL", "qwen2.5-coder", true);
-    vix::env::set("VIX_AGENT_TEST_MODEL_URL", "http://127.0.0.1:11434", true);
-    vix::env::set("VIX_AGENT_TEST_TIMEOUT_MS", "15000", true);
-    vix::env::set("VIX_AGENT_TEST_MAX_FILES", "42", true);
-    vix::env::set("VIX_AGENT_TEST_ALLOW_PROCESS", "true", true);
-    vix::env::set("VIX_AGENT_TEST_ALLOW_FILE_WRITE", "false", true);
+    assert_no_error(vix::env::set("VIX_AGENT_TEST_PROVIDER", "ollama", true));
+    assert_no_error(vix::env::set("VIX_AGENT_TEST_MODEL", "qwen2.5-coder", true));
+    assert_no_error(vix::env::set("VIX_AGENT_TEST_MODEL_URL", "http://127.0.0.1:11434", true));
+    assert_no_error(vix::env::set("VIX_AGENT_TEST_TIMEOUT_MS", "15000", true));
+    assert_no_error(vix::env::set("VIX_AGENT_TEST_MAX_FILES", "42", true));
+    assert_no_error(vix::env::set("VIX_AGENT_TEST_ALLOW_PROCESS", "true", true));
+    assert_no_error(vix::env::set("VIX_AGENT_TEST_ALLOW_FILE_WRITE", "false", true));
 
     auto config = vix::ai::agent::AgentConfigLoader::from_environment("VIX_AGENT_TEST_");
 
@@ -83,13 +89,13 @@ namespace
     assert(config.allow_process);
     assert(!config.allow_file_write);
 
-    vix::env::unset("VIX_AGENT_TEST_PROVIDER");
-    vix::env::unset("VIX_AGENT_TEST_MODEL");
-    vix::env::unset("VIX_AGENT_TEST_MODEL_URL");
-    vix::env::unset("VIX_AGENT_TEST_TIMEOUT_MS");
-    vix::env::unset("VIX_AGENT_TEST_MAX_FILES");
-    vix::env::unset("VIX_AGENT_TEST_ALLOW_PROCESS");
-    vix::env::unset("VIX_AGENT_TEST_ALLOW_FILE_WRITE");
+    assert_no_error(vix::env::unset("VIX_AGENT_TEST_PROVIDER"));
+    assert_no_error(vix::env::unset("VIX_AGENT_TEST_MODEL"));
+    assert_no_error(vix::env::unset("VIX_AGENT_TEST_MODEL_URL"));
+    assert_no_error(vix::env::unset("VIX_AGENT_TEST_TIMEOUT_MS"));
+    assert_no_error(vix::env::unset("VIX_AGENT_TEST_MAX_FILES"));
+    assert_no_error(vix::env::unset("VIX_AGENT_TEST_ALLOW_PROCESS"));
+    assert_no_error(vix::env::unset("VIX_AGENT_TEST_ALLOW_FILE_WRITE"));
   }
 
   void test_config_validator_accepts_default_config()

tests/AgentPublicApiTest.cpp

@@ -17,7 +17,7 @@
 #include <memory>
 #include <string_view>
 
-#include <vix/ai/agent/agent.hpp>
+#include <vix/ai/agent/AgentRuntime.hpp>
 
 namespace
 {