Allow ZEROCLAW_PROVIDER_URL env variable to override api_url (zeroclaw-labs#3414)

* Ignore JetBrains .idea folder

* fix(ollama): support stringified JSON tool call arguments

* providers: allow ZEROCLAW_PROVIDER_URL env var to override Ollama base URL

Supports container deployments where Ollama runs on a Docker network host
(e.g. http://ollama:11434) without requiring config.toml changes.

Includes regression test ensuring the environment override works.

* fix(clippy): replace Default::default() with ProviderRuntimeOptions::default()

---------

Co-authored-by: Argenis <theonlyhennygod@gmail.com>

Author: faustoheikkinen

.gitignore

@@ -41,3 +41,6 @@ credentials.json
 
 # Coverage artifacts
 lcov.info
+
+# IDE's stuff
+.idea

src/providers/mod.rs

@@ -1074,11 +1074,20 @@ fn create_provider_with_url_and_options(
         "anthropic" => Ok(Box::new(anthropic::AnthropicProvider::new(key))),
         "openai" => Ok(Box::new(openai::OpenAiProvider::with_base_url(api_url, key))),
         // Ollama uses api_url for custom base URL (e.g. remote Ollama instance)
-        "ollama" => Ok(Box::new(ollama::OllamaProvider::new_with_reasoning(
-            api_url,
-            key,
-            options.reasoning_enabled,
-        ))),
+        "ollama" => {
+
+                let env_url = std::env::var("ZEROCLAW_PROVIDER_URL").ok();
+
+                let api_url = env_url
+                    .as_deref()
+                    .or(api_url);
+
+                Ok(Box::new(ollama::OllamaProvider::new_with_reasoning(
+                    api_url,
+                    key,
+                    options.reasoning_enabled,
+                )))
+        },
         "gemini" | "google" | "google-gemini" => {
             let state_dir = options
                 .zeroclaw_dir
@@ -3104,4 +3113,22 @@ mod tests {
         assert_eq!(options.extra_headers.len(), 1);
         assert_eq!(options.extra_headers.get("X-Title").unwrap(), "zeroclaw");
     }
+
+    #[test]
+    fn env_provider_url_overrides_api_url() {
+        std::env::set_var("ZEROCLAW_PROVIDER_URL", "http://env-ollama:11434");
+
+        let options = ProviderRuntimeOptions::default();
+
+        let provider = create_provider_with_url_and_options(
+            "ollama",
+            Some("http://config-ollama:11434"),
+            None,
+            &options,
+        );
+
+        assert!(provider.is_ok());
+
+        std::env::remove_var("ZEROCLAW_PROVIDER_URL");
+    }
 }

src/providers/ollama.rs

@@ -89,10 +89,26 @@ struct OllamaToolCall {
 #[derive(Debug, Deserialize)]
 struct OllamaFunction {
     name: String,
-    #[serde(default)]
+    #[serde(default, deserialize_with = "deserialize_args")]
     arguments: serde_json::Value,
 }
 
+// ─── serde Helpers ───────────────────────────────────────────────────────────
+fn deserialize_args<'de, D>(deserializer: D) -> Result<serde_json::Value, D::Error>
+where
+    D: serde::Deserializer<'de>,
+{
+    let value = serde_json::Value::deserialize(deserializer)?;
+
+    if let Some(s) = value.as_str() {
+        match serde_json::from_str::<serde_json::Value>(s) {
+            Ok(v) => Ok(v),
+            Err(_) => Ok(serde_json::json!({})),
+        }
+    } else {
+        Ok(value)
+    }
+}
 // ─── Implementation ───────────────────────────────────────────────────────────
 
 impl OllamaProvider {