Enhance WSL2 support for Ollama IP detection and update documentation

- Added automatic detection of the Windows host IP for Ollama when running inside WSL2, simplifying user configuration.
- Implemented regression tests for Ollama base URL resolution logic.
- Updated README to document the new WSL workflow and manual override steps.
- Updated CHANGELOG to reflect these enhancements.

Author: Wojciech Napierała

CHANGELOG.md

@@ -2,6 +2,15 @@
 
 All notable changes to this project will be documented in this file. The format follows [Keep a Changelog](https://keepachangelog.com/) and dates use ISO-8601 (`YYYY-MM-DD`).
 
+## [0.1.4] - 2025-11-07
+
+### Added
+- Automatic detection of the Windows host IP for Ollama when running inside WSL2, removing the need for manual configuration in common setups.
+- Regression tests covering Ollama base URL resolution logic.
+
+### Documentation
+- Documented the WSL Ollama workflow and manual override steps in `README.md`.
+
 ## [0.1.2] - 2025-11-07
 
 ### Added

README.md

@@ -36,7 +36,12 @@ The project targets **Python 3.12**. Create a virtual environment with that inte
 
    - Redis (port 6379) persists working memory.
    - ChromaDB persists via the Python package; no container is required.
-   - Ollama (port 11434) is available for local inference; pull models with `ollama pull <model>` and set `OLLAMA_BASE_URL` when using the `local` workflow.
+   - Ollama (port 11434) is available for local inference; pull models with `ollama pull <model>`. When running DRM inside WSL2, the executor now auto-detects the Windows host IP, but you can still override it with `OLLAMA_BASE_URL` if needed. For manual discovery:
+
+     ```bash
+     export WINHOST=$(ip route | grep -oE 'via [0-9.]+' | head -1 | cut -d' ' -f2)
+     export OLLAMA_BASE_URL="http://$WINHOST:11434"
+     ```
 
 3. **Launch the app**
 

core/task_executor.py

@@ -10,12 +10,16 @@
     v0.6 - 2025-11-07 - Prefixed Ollama models for LiteLLM provider resolution.
     v0.7 - 2025-11-07 - Added explicit Ollama provider hints for LiteLLM routing.
     v0.8 - 2025-11-07 - Emitted detailed error context when workflows fail.
+    v0.9 - 2025-11-07 - Auto-detected Ollama base URL when running under WSL.
 """
 
 from __future__ import annotations
 
+import ipaddress
 import logging
+import subprocess
 import time
+from functools import lru_cache
 from os import getenv
 from typing import Any, Dict, Mapping, Optional, Tuple, cast
 
@@ -34,9 +38,67 @@
 LOGGER = logging.getLogger("drm.executor")
 
 
+@lru_cache(maxsize=1)
+def _detect_windows_host_ip(timeout_seconds: float = 2.0) -> Optional[str]:
+    """Detect the Windows host IP when running inside WSL2.
+
+    Returns the first "default via" IP discovered by ``ip route`` if the
+    environment appears to be Windows Subsystem for Linux. The lookup is cached
+    to avoid repeated subprocess execution.
+    """
+
+    try:
+        with open("/proc/version", encoding="utf-8") as version_file:
+            if "microsoft" not in version_file.read().lower():
+                return None
+    except OSError:
+        return None
+
+    try:
+        result = subprocess.run(
+            ["ip", "route"],
+            check=False,
+            capture_output=True,
+            text=True,
+            timeout=timeout_seconds,
+        )
+    except (OSError, subprocess.SubprocessError):
+        return None
+
+    if result.returncode != 0 or not result.stdout:
+        return None
+
+    for line in result.stdout.splitlines():
+        tokens = line.strip().split()
+        if not tokens or tokens[0] != "default":
+            continue
+        try:
+            via_index = tokens.index("via")
+        except ValueError:
+            continue
+        if via_index + 1 >= len(tokens):
+            continue
+        candidate = tokens[via_index + 1]
+        if _is_ipv4(candidate):
+            return candidate
+
+    return None
+
+
+def _is_ipv4(value: str) -> bool:
+    """Return ``True`` when *value* is a valid IPv4 address."""
+
+    try:
+        return isinstance(ipaddress.ip_address(value), ipaddress.IPv4Address)
+    except ValueError:
+        return False
+
+
 class TaskExecutor:
     """Executes prompts through configured LLM workflows."""
 
+    DEFAULT_OLLAMA_BASE = "http://localhost:11434"
+
     def __init__(
         self,
         config: AppConfig,
@@ -213,7 +275,7 @@ def _build_provider_kwargs(self, provider: str) -> Dict[str, object]:
             }
 
         if provider.lower() == "ollama":
-            base_url = getenv("OLLAMA_BASE_URL", "http://localhost:11434").rstrip("/")
+            base_url = self._resolve_ollama_base_url()
             return {
                 "base_url": base_url,
                 "api_base": base_url,
@@ -269,3 +331,22 @@ def _resolve_model_name(self, workflow_cfg: WorkflowModelConfig) -> str:
             return f"ollama/{model_name}"
 
         return model_name
+
+    def _resolve_ollama_base_url(self) -> str:
+        """Resolve the Ollama base URL with WSL host detection fallback."""
+
+        provided = getenv("OLLAMA_BASE_URL")
+        if provided:
+            base_url = provided.rstrip("/")
+            self._logger.debug("Using OLLAMA_BASE_URL override: %s", base_url)
+            return base_url
+
+        detected_host = _detect_windows_host_ip()
+        if detected_host:
+            base_url = f"http://{detected_host}:11434"
+            self._logger.debug(
+                "Detected Windows host IP %s for Ollama base URL.", detected_host
+            )
+            return base_url
+
+        return self.DEFAULT_OLLAMA_BASE

tests/test_task_executor.py

@@ -0,0 +1,79 @@
+"""Tests for TaskExecutor provider configuration helpers.
+
+Updates: v0.1 - 2025-11-07 - Added coverage for Ollama base URL resolution.
+"""
+
+from __future__ import annotations
+
+import pytest
+
+from config import settings
+from core.task_executor import TaskExecutor
+
+
+@pytest.fixture()
+def task_executor() -> TaskExecutor:
+    """Provide a TaskExecutor configured with a minimal workflow set."""
+
+    config = settings.AppConfig.model_validate(
+        {
+            "version": "0.1",
+            "llm": {
+                "default_workflow": "local",
+                "workflows": {
+                    "local": {
+                        "provider": "ollama",
+                        "model": "gemma3:1b",
+                        "temperature": 0.2,
+                    }
+                },
+                "timeouts": {
+                    "request_seconds": 10,
+                    "retry_attempts": 1,
+                    "retry_backoff_seconds": 1,
+                },
+                "enable_debug": False,
+            },
+            "memory": {
+                "redis": {"host": "localhost", "port": 6379, "db": 0, "ttl_seconds": 120},
+                "chromadb": {
+                    "persist_directory": "data/chromadb",
+                    "collection": "test",
+                },
+            },
+            "review": {
+                "enabled": False,
+                "auto_reviewer_model": None,
+                "auto_reviewer_provider": None,
+            },
+            "embedding": None,
+            "telemetry": {"log_level": "INFO"},
+        }
+    )
+    return TaskExecutor(config)
+
+
+def test_resolve_ollama_base_url_prefers_env_override(
+    task_executor: TaskExecutor, monkeypatch: pytest.MonkeyPatch
+) -> None:
+    monkeypatch.setenv("OLLAMA_BASE_URL", "http://172.16.0.1:9000/")
+    base_url = task_executor._resolve_ollama_base_url()
+    assert base_url == "http://172.16.0.1:9000"
+
+
+def test_resolve_ollama_base_url_detects_wsl_host(
+    task_executor: TaskExecutor, monkeypatch: pytest.MonkeyPatch
+) -> None:
+    monkeypatch.delenv("OLLAMA_BASE_URL", raising=False)
+    monkeypatch.setattr("core.task_executor._detect_windows_host_ip", lambda: "172.31.52.230")
+    base_url = task_executor._resolve_ollama_base_url()
+    assert base_url == "http://172.31.52.230:11434"
+
+
+def test_resolve_ollama_base_url_falls_back_to_localhost(
+    task_executor: TaskExecutor, monkeypatch: pytest.MonkeyPatch
+) -> None:
+    monkeypatch.delenv("OLLAMA_BASE_URL", raising=False)
+    monkeypatch.setattr("core.task_executor._detect_windows_host_ip", lambda: None)
+    base_url = task_executor._resolve_ollama_base_url()
+    assert base_url == TaskExecutor.DEFAULT_OLLAMA_BASE