fix: use config_home fixture in BASIC_MEMORY_HOME tests to resolve CI permission issues

- Replace Path.home() with config_home fixture for temporary directories
- Use paths within test directory instead of hardcoded absolute paths
- Follow existing test patterns from conftest.py for proper CI compatibility
- Maintain all original test coverage while fixing permission issues

Co-authored-by: Drew Cain <groksrc@users.noreply.github.com>

Author: claude[bot]

tests/test_config.py

@@ -11,53 +11,55 @@
 class TestBasicMemoryConfig:
     """Test BasicMemoryConfig behavior with BASIC_MEMORY_HOME environment variable."""
 
-    def test_default_behavior_without_basic_memory_home(self, monkeypatch):
+    def test_default_behavior_without_basic_memory_home(self, config_home, monkeypatch):
         """Test that config uses default path when BASIC_MEMORY_HOME is not set."""
         # Ensure BASIC_MEMORY_HOME is not set
         monkeypatch.delenv("BASIC_MEMORY_HOME", raising=False)
 
         config = BasicMemoryConfig()
 
         # Should use the default path (home/basic-memory)
-        expected_path = str(Path.home() / "basic-memory")
+        expected_path = str(config_home / "basic-memory")
         assert config.projects["main"] == expected_path
 
-    def test_respects_basic_memory_home_environment_variable(self, monkeypatch):
+    def test_respects_basic_memory_home_environment_variable(self, config_home, monkeypatch):
         """Test that config respects BASIC_MEMORY_HOME environment variable."""
-        custom_path = "/app/data"
+        custom_path = str(config_home / "app" / "data")
         monkeypatch.setenv("BASIC_MEMORY_HOME", custom_path)
 
         config = BasicMemoryConfig()
 
         # Should use the custom path from environment variable
         assert config.projects["main"] == custom_path
 
-    def test_model_post_init_respects_basic_memory_home(self, monkeypatch):
+    def test_model_post_init_respects_basic_memory_home(self, config_home, monkeypatch):
         """Test that model_post_init creates main project with BASIC_MEMORY_HOME when missing."""
-        custom_path = "/custom/memory/path"
+        custom_path = str(config_home / "custom" / "memory" / "path")
         monkeypatch.setenv("BASIC_MEMORY_HOME", custom_path)
 
         # Create config without main project
-        config = BasicMemoryConfig(projects={"other": "/some/path"})
+        other_path = str(config_home / "some" / "path")
+        config = BasicMemoryConfig(projects={"other": other_path})
 
         # model_post_init should have added main project with BASIC_MEMORY_HOME
         assert "main" in config.projects
         assert config.projects["main"] == custom_path
 
-    def test_model_post_init_fallback_without_basic_memory_home(self, monkeypatch):
+    def test_model_post_init_fallback_without_basic_memory_home(self, config_home, monkeypatch):
         """Test that model_post_init falls back to default when BASIC_MEMORY_HOME is not set."""
         # Ensure BASIC_MEMORY_HOME is not set
         monkeypatch.delenv("BASIC_MEMORY_HOME", raising=False)
 
         # Create config without main project
-        config = BasicMemoryConfig(projects={"other": "/some/path"})
+        other_path = str(config_home / "some" / "path")
+        config = BasicMemoryConfig(projects={"other": other_path})
 
         # model_post_init should have added main project with default path
-        expected_path = str(Path.home() / "basic-memory")
+        expected_path = str(config_home / "basic-memory")
         assert "main" in config.projects
         assert config.projects["main"] == expected_path
 
-    def test_basic_memory_home_with_relative_path(self, monkeypatch):
+    def test_basic_memory_home_with_relative_path(self, config_home, monkeypatch):
         """Test that BASIC_MEMORY_HOME works with relative paths."""
         relative_path = "relative/memory/path" 
         monkeypatch.setenv("BASIC_MEMORY_HOME", relative_path)
@@ -67,13 +69,14 @@ def test_basic_memory_home_with_relative_path(self, monkeypatch):
         # Should use the exact value from environment variable
         assert config.projects["main"] == relative_path
 
-    def test_basic_memory_home_overrides_existing_main_project(self, monkeypatch):
+    def test_basic_memory_home_overrides_existing_main_project(self, config_home, monkeypatch):
         """Test that BASIC_MEMORY_HOME is used even when main project exists in constructor."""
-        custom_path = "/override/memory/path"
+        custom_path = str(config_home / "override" / "memory" / "path")
         monkeypatch.setenv("BASIC_MEMORY_HOME", custom_path)
 
         # Try to create config with a different main project path
-        config = BasicMemoryConfig(projects={"main": "/original/path"})
+        original_path = str(config_home / "original" / "path")
+        config = BasicMemoryConfig(projects={"main": original_path})
 
         # The default_factory should override with BASIC_MEMORY_HOME value
         # Note: This tests the current behavior where default_factory takes precedence