fix(setup): default browser/TTS picker to free local backend, not paid Nous

[teknium1]

Summary

A fresh-install user who presses Enter through the Browser Automation (or Text-to-Speech) setup screen no longer gets walked into a paid Nous Portal login — the picker now defaults to the free local backend.

Both pickers listed the paid "Nous Subscription" gateway row first, so the menu cursor defaulted to index 0 (Nous). Selecting it runs the inline Nous Portal device-code login, launching a browser to a paid offering the user never chose.

Changes

• hermes_cli/tools_config.py: reorder the browser and tts provider lists so the free, no-key local backend is index 0 (Local Browser / Microsoft Edge TTS). Nous Subscription stays in the list as an explicit choice.
• Users already configured on Nous are unaffected — _detect_active_provider_index resolves their active row first, so the cursor lands on Nous (now index 1) for them.
• Two stale positional assertions (providers[0] == "Nous Subscription") relaxed to presence checks; that was visibility coverage, not an ordering contract.
• Added regression tests pinning the free local row as index 0 for both categories.

Validation

	Before	After
Fresh browser picker, Enter	Nous Subscription → Portal login	Local Browser
Fresh TTS picker, Enter	Nous Subscription → Portal login	Microsoft Edge TTS
User already on Nous	Nous (idx 0)	Nous (idx 1, cursor follows active)
BYOK env-configured	that provider	unchanged

scripts/run_tests.sh tests/hermes_cli/test_tools_config.py test_tts_picker.py test_image_gen_picker.py test_video_gen_picker.py → 127 passed, 0 failed.

Reported by Javier via Kujila.

Infographic

[github-actions]

🔎 Lint report: hermes/hermes-24a52332 vs origin/main

ruff

Total: 0 on HEAD, 0 on base (➖ 0)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 0 pre-existing issues carried over.

ty (type checker)

Total: 9707 on HEAD, 9701 on base (🆕 +6)

🆕 New issues (8):

Rule	Count
invalid-argument-type	5
unresolved-attribute	2
unsupported-operator	1

First entries

tests/hermes_cli/test_tools_config.py:707: [invalid-argument-type] invalid-argument-type: Argument to function `_configure_provider` is incorrect: Expected `dict[Unknown, Unknown]`, found `str | dict[str, str | list[Unknown]] | dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[dict[str, str]]]`
tests/hermes_cli/test_tools_config.py:1069: [unresolved-attribute] unresolved-attribute: Attribute `get` is not defined on `str` in union `str | dict[str, str | list[Unknown]] | dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[dict[str, str]]]`
hermes_cli/tools_config.py:2835: [unresolved-attribute] unresolved-attribute: Attribute `get` is not defined on `str` in union `str | dict[str, str | list[Unknown]] | dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[dict[str, str]]]`
tests/hermes_cli/test_tools_config.py:1368: [invalid-argument-type] invalid-argument-type: Argument to function `_reconfigure_provider` is incorrect: Expected `dict[Unknown, Unknown]`, found `str | dict[str, str | list[Unknown]] | dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[dict[str, str]]]`
tests/hermes_cli/test_tools_config.py:736: [invalid-argument-type] invalid-argument-type: Argument to function `_detect_active_provider_index` is incorrect: Expected `list[Unknown]`, found `str | list[dict[str, str | list[Unknown]] | dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[dict[str, str]]]] | list[dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[dict[str, str]]]] | ... omitted 4 union elements`
hermes_cli/tools_config.py:3168: [unsupported-operator] unsupported-operator: Operator `in` is not supported between objects of type `dict[Unknown, Unknown]` and `str | list[dict[str, str | list[Unknown]] | dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[dict[str, str]]]] | list[dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[dict[str, str]]]] | ... omitted 4 union elements`
tests/hermes_cli/test_tools_config.py:724: [invalid-argument-type] invalid-argument-type: Method `__getitem__` of type `bound method str.__getitem__(key: SupportsIndex | slice[SupportsIndex | None, SupportsIndex | None, SupportsIndex | None], /) -> str` cannot be called with key of type `Literal["browser_provider"]` on object of type `str`
tests/hermes_cli/test_tools_config.py:735: [invalid-argument-type] invalid-argument-type: Method `__getitem__` of type `bound method str.__getitem__(key: SupportsIndex | slice[SupportsIndex | None, SupportsIndex | None, SupportsIndex | None], /) -> str` cannot be called with key of type `Literal["tts_provider"]` on object of type `str`

✅ Fixed issues (5):

Rule	Count
invalid-argument-type	2
unresolved-attribute	2
unsupported-operator	1

First entries

tests/hermes_cli/test_tools_config.py:1334: [invalid-argument-type] invalid-argument-type: Argument to function `_reconfigure_provider` is incorrect: Expected `dict[Unknown, Unknown]`, found `str | dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[Unknown]] | dict[str, str | list[dict[str, str]]]`
hermes_cli/tools_config.py:3165: [unsupported-operator] unsupported-operator: Operator `in` is not supported between objects of type `dict[Unknown, Unknown]` and `str | list[dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[Unknown]] | dict[str, str | list[dict[str, str]]]] | list[dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[dict[str, str]]]] | ... omitted 4 union elements`
tests/hermes_cli/test_tools_config.py:1035: [unresolved-attribute] unresolved-attribute: Attribute `get` is not defined on `str` in union `str | dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[Unknown]] | dict[str, str | list[dict[str, str]]]`
hermes_cli/tools_config.py:2832: [unresolved-attribute] unresolved-attribute: Attribute `get` is not defined on `str` in union `str | dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[Unknown]] | dict[str, str | list[dict[str, str]]]`
tests/hermes_cli/test_tools_config.py:700: [invalid-argument-type] invalid-argument-type: Argument to function `_configure_provider` is incorrect: Expected `dict[Unknown, Unknown]`, found `str | dict[str, str | list[Unknown] | bool | list[str]] | dict[str, str | list[Unknown]] | dict[str, str | list[dict[str, str]]]`

Unchanged: 5021 pre-existing issues carried over.

Diagnostics are surfaced as warnings — this check never fails the build.