DasPoschi/hermes-webui
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

review-3142: align load_metadata_only comment with sidecar-first logic

Browse Source
This commit is contained in:
nesquena-hermes
2026-05-29 23:46:51 +00:00
parent 017179b189
commit 23f5ee15f7
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
api/models.py
View File

@@ -818,9 +818,12 @@ class Session:
index_message_count = None
if sidecar_message_count is None:
index_message_count = _lookup_index_message_count(sid)
# The sidebar index is a cache and can lag behind external sidecar
# appends/backfills. Prefer the largest known count so metadata-only
# active-session polls do not miss real remote updates.
# Modern sidecars carry an accurate message_count, so it is the
# source of truth and we skip the per-row _index.json read in the
# common case. The sidebar index is only a cache (it can lag behind
# external sidecar appends/backfills), so consult it solely as a
# fallback when the sidecar has no count. When both are present we
# still take the largest known count as a defensive measure.
known_counts = [
count for count in (index_message_count, sidecar_message_count)
if count is not None