fix config in the future but it kinda does the thing it is supposed to right now

Author: djcopley

shelloracle/config/config.py

@@ -1,9 +1,10 @@
 from collections.abc import MutableMapping
 from pathlib import Path
+from typing import Any
 
 import tomlkit
 
-data_home = Path.home() / "Library/Application Support" / "shelloracle"
+data_home = Path.home() / ".shelloracle"
 
 
 def _default_config() -> tomlkit.TOMLDocument:
@@ -20,27 +21,35 @@ class Configuration(MutableMapping):
     def __init__(self) -> None:
         self._ensure_config_exists()
 
-    def __getitem__(self, item):
+    def __getitem__(self, item: str) -> dict:
         with self.filepath.open("r") as file:
             config = tomlkit.load(file)
         return config[item]
 
-    def __setitem__(self, key, value):
+    def __setitem__(self, key: str, value: Any) -> None:
         with self.filepath.open("r") as file:
             config = tomlkit.load(file)
         config[key] = value
         config.multiline = True
         with self.filepath.open("w") as file:
             tomlkit.dump(config, file)
 
-    def __delitem__(self, key):
-        raise NotImplementedError()
+    def __delitem__(self, key: str) -> None:
+        with self.filepath.open("r") as file:
+            config = tomlkit.load(file)
+        del config[key]
+        with self.filepath.open("w") as file:
+            tomlkit.dump(config, file)
 
-    def __iter__(self):
-        raise NotImplementedError()
+    def __iter__(self) -> None:
+        with self.filepath.open("r") as file:
+            config = tomlkit.load(file)
+        return iter(config)
 
     def __len__(self) -> int:
-        raise NotImplementedError()
+        with self.filepath.open("r") as file:
+            config = tomlkit.load(file)
+        return len(config)
 
     def _ensure_config_exists(self) -> None:
         if self.filepath.exists():

shelloracle/config/setting.py

@@ -16,7 +16,7 @@ def __init__(self, *, default: T | None = None) -> None:
 
     def __set_name__(self, owner: type[Provider], name: str) -> None:
         self.name = name
-        # Set the default value in the config dictionary
+        # Set the default value in the config dictionary if it doesn't exist
         provider_table = config.global_config.get("provider", {})
         provider_table.setdefault(owner.name, {}).setdefault(name, self.default)
         config.global_config["provider"] = provider_table