Merge pull request #2 from contextforge-org/FixHomeDirCreation

fix: Create the home dir if it doesn't exist

Author: gabe-l-hart

cforge/config.py

@@ -71,6 +71,7 @@ def get_settings() -> CLISettings:
     # NOTE: This duplicates the source of truth for the env var slightly so that
     #   we can use home as the source for the .env file as a 2-phase init.
     home = Path(os.getenv("CONTEXTFORGE_HOME", DEFAULT_HOME))
+    home.mkdir(exist_ok=True)
     with _chdir(home):
 
         settings = CLISettings(client_mode=True)

tests/test_config.py

@@ -6,6 +6,11 @@
 
 Tests for configuration management.
 """
+# Standard
+from pathlib import Path
+from unittest import mock
+import os
+import tempfile
 
 # First-Party
 from cforge.config import get_settings
@@ -27,3 +32,17 @@ def test_get_settings_singleton(self) -> None:
         settings1 = get_settings()
         settings2 = get_settings()
         assert settings1 is settings2
+
+    def test_get_settings_make_home(self) -> None:
+        """Test that get_settings will create the home dir if needed"""
+        try:
+            get_settings.cache_clear()
+            with tempfile.TemporaryDirectory() as work_dir:
+                home_dir = Path(work_dir) / "some_new_home"
+                assert not home_dir.exists()
+                with mock.patch.dict(os.environ, {"CONTEXTFORGE_HOME": str(home_dir)}):
+                    settings = get_settings()
+                    assert settings.contextforge_home == home_dir
+                    assert home_dir.exists()
+        finally:
+            get_settings.cache_clear()