fix: use documented aux model for update summaries

This commit is contained in:
Jordan SkyLF
2026-05-14 08:34:27 -07:00
parent 18297f3aff
commit 339e62528f
3 changed files with 11 additions and 4 deletions
+2
View File
@@ -22,6 +22,8 @@
- **PR #2234** by @Jordan-SkyLF (post-v0.51.61 rebase) — Update summary category handling now preserves all explicit `Notice:` and `Worth knowing:` bullets the summarizer returns instead of forcing a three-item split. Distinct categories are deduplicated against each other so the same content can't appear twice across sections. Keeps the existing fallback grouping when the model doesn't return explicit prefixes. The summary panel becomes scrollable when longer summaries need more vertical room. Caps large update-summary commit input to the latest 24 commit subjects and discloses that scope in the generated summary while keeping the full comparison link available.
- **PR #2234** by @Jordan-SkyLF (follow-up to the v0.51.62 category refinement) — Update summary generation now prefers the documented `auxiliary.compression` text-model slot before falling back to the main configured model, avoiding a WebUI-only `auxiliary.update_summary` magic key that would not appear in Hermes Agent setup/config documentation.
### Added
- **PR #2238** by @franksong2702 (fixes #2231) — Phone-width layouts (≤640px) keep the hamburger drawer entry pattern, but the drawer now lays out `.sidebar-nav` as a vertical 52px strip with stable 44px touch targets and a left-edge selection indicator instead of a cramped horizontal icon row. PWA chrome alignment: `theme-color` meta tag now follows the app chrome `--sidebar` color instead of the chat background `--bg`, so iOS Safari / PWA status bars visibly match the titlebar/sidebar. Phone composer also reserves the bottom safe area so it is not clipped by rounded corners or the home indicator. Before/after screenshots shipped under `docs/pr-media/2231/`.
+5 -1
View File
@@ -5467,8 +5467,12 @@ def handle_post(handler, parsed) -> bool:
try:
from agent.auxiliary_client import get_text_auxiliary_client
# Update summaries are a short text-compression/summarization task.
# Reuse the documented auxiliary.compression slot instead of
# inventing a WebUI-only auxiliary task name that users cannot
# discover in the Hermes Agent setup/config UI.
aux_client, aux_model = get_text_auxiliary_client(
"update_summary",
"compression",
main_runtime=main_runtime,
)
if aux_client is not None and aux_model:
+4 -3
View File
@@ -419,13 +419,14 @@ class TestForceUpdateRoute:
class TestUpdateSummaryRouteModelSelection:
"""Update summaries should use the auxiliary update-summary model before main model fallback."""
"""Update summaries should use a known text auxiliary model before main model fallback."""
def test_summary_route_prefers_update_summary_auxiliary_model(self):
def test_summary_route_prefers_documented_compression_auxiliary_model(self):
src = read('api/routes.py')
assert 'get_text_auxiliary_client' in src
assert '"update_summary"' in src
assert '"compression"' in src
assert '"update_summary"' not in src
assert 'main_runtime=main_runtime' in src
assert 'update summary auxiliary model failed; falling back to main model' in src
assert 'from run_agent import AIAgent' in src