mirrors/hermes-agent
1
0
Fork 0
mirror of https://github.com/NousResearch/hermes-agent.git synced 2026-07-31 19:16:29 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

docs(compression): correct the stale in_place default in comments

Browse source 
2107b86024 flipped compression.in_place to True but left both explanatory
comments reading "Default False during rollout". The contradiction is
load-bearing: it is why two recent PRs (#71747, #48951) were built on the
premise that agent.session_id still rotates at every compaction.

Comment-only; no behavior change.
This commit is contained in:
teknium1 2026-07-25 22:32:43 -07:00 • committed by Teknium
parent 2c69316742
commit 72c013b662
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
agent/conversation_compression.py
View file

@ -1401,7 +1401,7 @@ def compress_context(
# parent_session_id child, no
# `name #N` renumber, no contextvar/env/logging re-sync, no memory/context-
# engine session-switch. The conversation keeps one durable id for life,
# eliminating the session-rotation bug cluster. Default False during rollout.
# eliminating the session-rotation bug cluster. Default True (2107b86024).
in_place = bool(getattr(agent, "compression_in_place", False))
# Set True once the in-place DB write actually completes (the DB block can
# raise and skip it). Surfaced to the gateway via agent._last_compaction_in_place.

4
hermes_cli/config.py
View file

@ -1532,8 +1532,8 @@ DEFAULT_CONFIG = {
# turns are soft-archived under the same id
# (active=0, compacted=1) — still searchable via
# session_search and recoverable, not deleted.
# Default False during rollout; will flip on
# after live validation.
# Default True since 2107b86024; set False to
# restore the legacy rotating-compaction path.
"model_thresholds": {}, # Per-model threshold overrides. Keys are
# substring-matched against the model name
# (longest match wins); values replace the