Skip to content

fix(aux): remove stale session_search model menu entry#27926

Closed
hehehe0803 wants to merge 1 commit into
NousResearch:mainfrom
hehehe0803:fix/aux-session-search-menu-drift
Closed

fix(aux): remove stale session_search model menu entry#27926
hehehe0803 wants to merge 1 commit into
NousResearch:mainfrom
hehehe0803:fix/aux-session-search-menu-drift

Conversation

@hehehe0803

Copy link
Copy Markdown
Contributor

Summary

Fixes the auxiliary-model menu/config drift for session_search.

session_search became a direct DB-backed/session-store tool and no longer uses an auxiliary LLM. DEFAULT_CONFIG["auxiliary"] already documents and reflects that by omitting session_search, but _AUX_TASKS still exposed session_search in the hermes model auxiliary task menu.

This caused the auxiliary-config tests on current main to fail because the menu referenced a task that no longer exists in the default auxiliary config schema.

Fixes #27925

Changes

Test Plan

python -m py_compile hermes_cli/main.py tests/hermes_cli/test_aux_config.py
scripts/run_tests.sh tests/hermes_cli/test_aux_config.py -q

Result:

21 passed

Privacy / Generality

This fix is fully repo-local and reproducible from current main. It does not include any private paths, hostnames, IP addresses, account names, credentials, or user-specific environment details.

@alt-glitch alt-glitch added type/bug Something isn't working P2 Medium — degraded but workaround exists comp/cli CLI entry point, hermes_cli/, setup wizard area/config Config system, migrations, profiles labels May 18, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

Duplicate of #27835 (same fix: remove session_search from _AUX_TASKS). See also companion issue #27925.

@teknium1

Copy link
Copy Markdown
Contributor

Merged via PR #28335 (cherry-picked onto current main with your authorship preserved via rebase-merge — commit 87ace43). Thanks for the contribution!

@teknium1 teknium1 closed this May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/config Config system, migrations, profiles comp/cli CLI entry point, hermes_cli/, setup wizard P2 Medium — degraded but workaround exists type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(aux): remove stale session_search auxiliary-model menu entry

3 participants