v3.0.1 fixes/complets first-run configuration

Also now a robust prompting for missing configuration lines on each
startup.

Author: 3ls-it

openai-assistant

@@ -18,9 +18,11 @@ import os
 from pathlib import Path
 import runpy
 import subprocess as proc
+import sys
 import threading
 import time
 from time import sleep
+from typing import Any
 #import warnings
 
 # OpenAI imports
@@ -1082,44 +1084,102 @@ class ConfigManager:
 
 
     # Helper to set-up/check initial configuration
-    def config_app(self, settings: dict) -> None:
+    def config_app(self, settings: dict[str, Any]) -> bool:
         """
-        Sets up initial run configuration|checks exsisting configuration.
+        Sets up initial run configuration and checks existing configuration.
+        Prompts interactively for missing values.
         """
-        # Required and optional settings
-        REQUIRED  = ("MODEL", "TEMP", "YOUR_NAME", "ASSISTANT_NAME", "OPENAI_API_KEY", "PROMPT_ID", "VECTOR_STORE_ID")
-        DEFAULTS  = {"MODEL": "gpt-5.1", "TEMP": 0.3}
-
-        # Prompt for missing required settings
-        updated  = False
-        for key in REQUIRED:
-            if key not in settings or (key == "OPENAI_API_KEY" and not settings.get(key)):
-                if key in DEFAULTS:
-                    settings[key] = DEFAULTS[key]
-                    print(f"No {key} found. Using default: {settings[key]}")
-                else:
-                    settings[key] = input(f"Enter {key}: ").strip()
+
+        REQUIRED = (
+            "MODEL",
+            "YOUR_NAME",
+            "ASSISTANT_NAME",
+            "OPENAI_API_KEY",
+            "PROMPT_ID",
+            "VECTOR_STORE_ID",
+        )
+
+        DEFAULTS = {
+            "MODEL": "gpt-5.1",
+            "YOUR_NAME": "User",
+            "ASSISTANT_NAME": "Assistant",
+        }
+
+        # Nice labels for prompts
+        LABELS = {
+            "MODEL": "model",
+            "YOUR_NAME": "your name",
+            "ASSISTANT_NAME": "assistant name",
+            "OPENAI_API_KEY": "OpenAI API key",
+            "PROMPT_ID": "Prompt ID",
+            "VECTOR_STORE_ID": "Vector Store ID",
+        }
+
+        first_run = not self.config_path.exists()
+        updated = False
+        incomplete = False
+        missing_ids: list[str] = []
+
+        # 1. MODEL / YOUR_NAME / ASSISTANT_NAME:
+        #    prompt with default, allow Enter to accept default.
+        for key in ("MODEL", "YOUR_NAME", "ASSISTANT_NAME"):
+            current = (settings.get(key) or "").strip()
+            if not current:
+                default = DEFAULTS[key]
+                label = LABELS.get(key, key)
+                resp = input(f"Enter {label} [{default}]: ").strip()
+                settings[key] = resp or default
+                print(f"Using {key} = {settings[key]!r}")
+                updated = True
+
+        # 2. IDs & API key: require a value, but allow blank with warning.
+        for key in ("OPENAI_API_KEY", "PROMPT_ID", "VECTOR_STORE_ID"):
+            current = (settings.get(key) or "").strip()
+            if not current:
+                label = LABELS.get(key, key)
+                resp = input(f"Enter {label} (leave blank to skip for now): ").strip()
+                settings[key] = resp  # may be empty string
                 updated = True
+                if not resp:
+                    incomplete = True
+                    missing_ids.append(key)
 
-        # Write settings file if needed
-        if updated or not self.config_dir.exists():
+        # 3. Write settings file if needed
+        if updated or first_run:
             self.config_dir.mkdir(parents=True, exist_ok=True)
             with open(self.config_path, "w", encoding="utf-8") as f:
-                f.write("# ~/.openai-assistant/settings.py  (auto-generated by openai-assistant)\n")
+                f.write(
+                    "# ~/.openai-assistant/settings.py  "
+                    "(auto-generated by openai-assistant)\n"
+                )
                 for key in REQUIRED:
-                    val = settings[key]
+                    val = settings.get(key, "")
                     if isinstance(val, str):
                         f.write(f"{key} = '{val}'\n")
                     else:
                         f.write(f"{key} = {val}\n")
-            print(f"Wrote settings to {self.config_path}")
+            print(f"Wrote settings to [cyan]{self.config_path}[/]")
+
+        # 4. Warn about incomplete configuration if any IDs are missing
+        if incomplete:
+            missing_str = ", ".join(missing_ids)
+            print(
+                "\n[red b][WARNING][/] Configuration is incomplete.\n"
+                f"Missing/empty: [#f5d676]{missing_str}[/]\n\n"
+                f"You can re-run the setup by running the program again,\n"
+                f"or edit [cyan]{self.config_path}[/] manually.\n"
+            )
+            return False
 
+        # 5. Ensure data directories exist
         if not self.chats_dir.exists():
             self.chats_dir.mkdir(parents=True, exist_ok=True)
-
         if not self.images_dir.exists():
             self.images_dir.mkdir(parents=True, exist_ok=True)
 
+        return True
+    # End 
+
 
     def cleanup_chat_files(self, min_size_bytes=50):
         """
@@ -1174,7 +1234,9 @@ def main():
 
         # Get and load settings, prompt for missing settings
         settings = config_man.load_settings()
-        config_man.config_app(settings)
+        is_config = config_man.config_app(settings)
+        if not is_config:
+            sys.exit()
 
         # Apply resets if requested
         if args.reset_prompt:

settings_example.py

@@ -1,6 +1,5 @@
 # ~/.openai-assistant/settings.py  (auto-generated by openai-assistant)
 MODEL = 'gpt-5.1'
-TEMP = 0.3
 YOUR_NAME = ''
 ASSISTANT_NAME = ''
 OPENAI_API_KEY = ''