fix: resolve NameError in settings Appearance tab by defining select_widget (#51)

* fix: resolve NameError in settings Appearance tab by defining select_widget

The  method created a Select widget inline inside
 but then referenced an undefined  variable
to call  and add a duplicate field, causing a NameError
crash whenever the Appearance category was selected.

Fix: store the Select widget in a local variable before passing it to
, and remove the duplicate field addition.

Closes #42

* test: fix help docs test by adding _import_module to fake kernel

The test_run_help_prints_docstring_for_known_command test was failing
because the fake kernel lacked a _import_module method, causing the
module import to fail silently and fall through to the error message.

Add a working _fake_import_module that uses importlib to resolve
the module path, allowing run_help to read and print the docstring.

* fix: resolve NameError crash at startup by defining argv before use

The  function referenced a bare name  that was
never imported or defined, causing a NameError crash on every invocation.

Fix: move  before the conditional block so it is
always defined when reached by the comparison at the end of the function.

Closes #40

* fix: remove dead _fake_import_module from test

---------

Co-authored-by: Ziang Zhang <zhangziang@ncti-gba.cn>
Co-authored-by: Akshaj Singhal <85437940+AkshajSinghal@users.noreply.github.com>

Author: 3 people

trushell/cli.py

@@ -29,9 +29,6 @@ def app_with_lower() -> None:
     if len(sys.argv) > 1:
         argv_copy = sys.argv.copy()
         if argv_copy[1].lower() not in {"--help", "-h", "version"}:
-            # Normalize the command name to lowercase for case-insensitive
-            # invocation, but preserve the case of subsequent arguments
-            # (e.g., filenames) which may be case-sensitive.
             first = argv_copy[1].lower()
             rest = argv_copy[2:]
             raw = " ".join([first] + rest)

trushell/commands/settings.py

@@ -167,15 +167,11 @@ def add_field(label_text: str, widget: Static | Input | Select | Switch) -> None
                 Input(value=str(self.dirty_settings.get("prompt_symbol", self.settings.get("prompt_symbol", "➜"))), id="prompt_symbol"),
             )
         elif current_cat == "Appearance":
-            add_field(
-                "Theme:",
-                Select(
-                    options=self.THEME_OPTIONS,
-                    value=str(self.dirty_settings.get("theme", self.settings.get("theme", "dark"))),
-                    id="theme_selector",
-                ),
+            select_widget = Select(
+                options=self.THEME_OPTIONS,
+                value=str(self.dirty_settings.get("theme", self.settings.get("theme", "dark"))),
+                id="theme_selector",
             )
-            select_widget.watch_value(lambda value: self._track_change("theme", value))
             add_field("Theme:", select_widget)
         elif current_cat == "Navigation":
             add_field(