fix: Check directory existence before load_config() in test

The test_add_project_never_creates_directory test was checking directory
existence after load_config(), but BasicMemoryConfig's model validator
ensure_project_paths_exists creates missing directories on deserialization.
Move the assertion before load_config() so it verifies add_project()
behavior rather than testing the validator side effect.

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

Author: phernandez

tests/test_config.py

@@ -601,11 +601,14 @@ def test_add_project_never_creates_directory(self):
             nonexistent_path = str(temp_path / "nonexistent" / "project")
             config_manager.add_project("test-project", nonexistent_path)
 
-            # Verify project was added to config without creating the directory
+            # Check directory does NOT exist right after add_project(),
+            # before load_config() which triggers the model validator
+            assert not Path(nonexistent_path).exists()
+
+            # Verify project was persisted in config
             config = config_manager.load_config()
             assert "test-project" in config.projects
             assert config.projects["test-project"].path == nonexistent_path
-            assert not Path(nonexistent_path).exists()
 
     def test_model_post_init_uses_platform_native_separators(self, config_home, monkeypatch):
         """Test that model_post_init uses platform-native separators."""