Problem

The hermes memory setup interactive picker (cmd_setup() in hermes_cli/memory_setup.py) hardcodes default=builtin_idx when calling the curses radiolist. This means every time the user opens the picker — even if they already have a provider configured — the selection defaults to "Built-in only" (the last item).

This creates significant user confusion. Users repeatedly select their provider (e.g. option 9 for Mnemosyne), exit, reopen the picker to verify, see "Built-in only" highlighted again, and assume their selection was not persisted.

Expected Behavior

The picker should default to the currently configured provider if one is set in config.yaml. Only if no provider is configured should it default to "Built-in only".

Current Code

# hermes_cli/memory_setup.py, line ~239
builtin_idx = len(items) - 1
selected = _curses_select("Memory provider setup", items, default=builtin_idx)

Suggested Fix

Read config.get("memory", {}).get("provider") before building the picker. If a provider is configured and present in the discovered list, set default to that index. Otherwise fall back to builtin_idx.

config = load_config()
provider_name = (config.get("memory") or {}).get("provider", "")

default_idx = builtin_idx
if provider_name:
    for i, (name, _, _) in enumerate(providers):
        if name == provider_name:
            default_idx = i
            break

selected = _curses_select("Memory provider setup", items, default=default_idx)

Impact

This is a pure UX improvement — no functional breakage. Users who already configured a provider will see it pre-selected. New users still see "Built-in only" as the default.