Validate and auto-repair identity on startup

Add validation and repair for the stored identity. Introduces is_valid_uuid() in config.py and ensure_identity() in store.py to detect missing or malformed identity IDs, regenerate a new UUID (and a name only if absent), persist the fix, and return (was_repaired, identity). Update cli._require_root to call ensure_identity() and notify the user if the identity was regenerated. This ensures the store always has a valid identity before running commands.

Author: CptPlastic

src/lore/cli.py

@@ -189,6 +189,13 @@ def _require_root() -> Path:
             "[red]No .lore directory found. Run `lore init` first.[/red]"
         )
         raise typer.Exit(code=1)
+    from .store import ensure_identity
+    repaired, identity = ensure_identity(root)
+    if repaired:
+        console.print(
+            f"\n  [bold {_A}]▲[/bold {_A}]  Identity was missing or corrupt — regenerated: "
+            f"[bold {_A}]{identity['name']}[/bold {_A}]  [dim]({identity['id']})[/dim]\n"
+        )
     return root
 
 

src/lore/config.py

@@ -49,6 +49,15 @@ def generate_identity() -> dict[str, str]:
     return {"name": name, "id": uid}
 
 
+def is_valid_uuid(value: str) -> bool:
+    """Return True if *value* is a well-formed UUID string."""
+    try:
+        uuid.UUID(value)
+        return True
+    except (ValueError, AttributeError):
+        return False
+
+
 DEFAULT_CONFIG: dict[str, Any] = {
     "version": 1,
     "identity": {"name": "", "id": ""},

src/lore/store.py

@@ -7,7 +7,7 @@
 from pathlib import Path
 from typing import Any
 
-from .config import memory_dir, load_config, save_config, DEFAULT_CONFIG, generate_identity
+from .config import memory_dir, load_config, save_config, DEFAULT_CONFIG, generate_identity, is_valid_uuid
 
 # Per-file YAML parse cache: path -> (mtime, parsed_data)
 # Re-parses only when the file's mtime changes — transparent for both CLI and TUI.
@@ -51,6 +51,33 @@ def init_store(root: Path) -> None:
             save_config(root, existing_cfg)
 
 
+def ensure_identity(root: Path) -> tuple[bool, dict]:
+    """Verify the store has a valid identity; repair silently if not.
+
+    Returns ``(was_repaired, identity_dict)``.
+    If the id is missing or not a valid UUID the id is regenerated.
+    If the name is also absent a fresh name is generated too.
+    The existing name is preserved when only the id is broken.
+    """
+    cfg = load_config(root)
+    identity = cfg.get("identity", {})
+    ident_id   = identity.get("id", "")
+    ident_name = identity.get("name", "")
+
+    if ident_id and is_valid_uuid(ident_id):
+        return False, identity
+
+    # Repair: keep name when still present, always issue a fresh id.
+    if ident_name:
+        new_identity: dict = {"name": ident_name, "id": str(uuid.uuid4())}
+    else:
+        new_identity = generate_identity()
+
+    cfg["identity"] = new_identity
+    save_config(root, cfg)
+    return True, new_identity
+
+
 def _short_id() -> str:
     return str(uuid.uuid4())[:8]