mirror of
https://github.com/nesquena/hermes-webui.git
synced 2026-05-25 11:10:18 +00:00
fix: use documented aux model for update summaries
This commit is contained in:
@@ -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
@@ -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:
|
||||
|
||||
@@ -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
|
||||
|
||||
Reference in New Issue
Block a user