fix: Windows test failures by preserving HOME environment variable

The tests were using patch.dict("os.environ", {}, clear=True) which cleared
ALL environment variables including HOME/USERPROFILE. This caused Path.home()
to fail on Windows when ConfigManager tried to initialize.

Fixed by using os.environ.pop() to remove only specific env vars being tested
while preserving HOME/USERPROFILE for cross-platform compatibility.

Fixes #322

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: phernandez <paul@basicmachines.co>

Author: phernandez

tests/api/test_async_client.py

@@ -1,5 +1,6 @@
 """Tests for async_client configuration."""
 
+import os
 from unittest.mock import patch
 from httpx import AsyncClient, ASGITransport, Timeout
 
@@ -9,7 +10,11 @@
 
 def test_create_client_uses_asgi_when_no_remote_env():
     """Test that create_client uses ASGI transport when BASIC_MEMORY_USE_REMOTE_API is not set."""
-    with patch.dict("os.environ", {}, clear=True):
+    # Ensure env vars are not set (pop if they exist)
+    with patch.dict("os.environ", clear=False):
+        os.environ.pop("BASIC_MEMORY_USE_REMOTE_API", None)
+        os.environ.pop("BASIC_MEMORY_CLOUD_MODE", None)
+
         client = create_client()
 
         assert isinstance(client, AsyncClient)
@@ -32,7 +37,11 @@ def test_create_client_uses_http_when_cloud_mode_env_set():
 
 def test_create_client_configures_extended_timeouts():
     """Test that create_client configures 30-second timeouts for long operations."""
-    with patch.dict("os.environ", {}, clear=True):
+    # Ensure env vars are not set (pop if they exist)
+    with patch.dict("os.environ", clear=False):
+        os.environ.pop("BASIC_MEMORY_USE_REMOTE_API", None)
+        os.environ.pop("BASIC_MEMORY_CLOUD_MODE", None)
+
         client = create_client()
 
         # Verify timeout configuration