fix(auxiliary): drop session search from model picker#27835
fix(auxiliary): drop session search from model picker#27835BoardJames-Bot wants to merge 1 commit into
Conversation
|
Board James reviewer pass:
Verdict: safe for a maintainer to approve workflow runs and merge once CI is green. Minor non-blocking nit: the aux-config section comment still lists “session search” as an auxiliary-model example even though the code/test now intentionally removes it. |
|
Closing as superseded by #28335. Triage notes (high confidence): Thanks for the contribution — the underlying problem this PR addresses has been resolved by the linked PR on current main. If you believe this was closed in error, please comment and we'll reopen. (Bulk-closed during a CLI PR triage sweep.) |
Summary\n- remove session_search from the auxiliary model picker now that it no longer uses an auxiliary LLM\n- update aux config tests to assert session_search remains out of DEFAULT_CONFIG.auxiliary and _AUX_TASKS\n\n## Test Plan\n- python -m pytest tests/hermes_cli/test_aux_config.py -q -o 'addopts='\n\nContext: current PR test runs are failing on the merged state with KeyError: 'session_search' in tests/hermes_cli/test_aux_config.py.