Merge pull request #30 from nick-galluzzo/refactor/cli-option-values-separation

refactor: separate display text from internal values in CLI options

Author: AstraBert

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "notebookllama"
-version = "0.3.0"
+version = "0.3.1"
 description = "An OSS and LlamaCloud-backed alternative to NotebookLM"
 readme = "README.md"
 requires-python = ">=3.13"

tools/cli/screens/initial.py

@@ -14,8 +14,8 @@ def get_form_elements(self) -> list:
         return [
             Select(
                 options=[
-                    ("With Default Settings", "With Default Settings"),
-                    ("With Custom Settings", "With Custom Settings"),
+                    ("With Default Settings", "default_settings"),
+                    ("With Custom Settings", "custom_settings"),
                 ],
                 prompt="Please select one of the following",
                 id="setup_type",
@@ -38,7 +38,7 @@ def handle_next(self) -> None:
 
         app = self.app
         if isinstance(app, EmbeddingSetupApp):
-            if app.config.setup_type == "With Default Settings":
+            if app.config.setup_type == "default_settings":
                 app.handle_default_setup()
             else:
                 app.push_screen(ProviderSelectScreen())