mirror of
https://github.com/NousResearch/hermes-agent.git
synced 2026-05-18 04:41:56 +00:00
feat(curator): show rename map in user-visible summary (#22910)
* feat(curator): show rename map (where skills went) in user-visible summary
The full data has always been on disk in REPORT.md, but the user-visible
curator summary (gateway 💾 line, CLI session-start panel,
`hermes curator status`) was counts-only — "consolidated 4 into 2
umbrellas" with no names. Users only discovered renames when something
they expected was gone.
New `_build_rename_summary()` formats the rename map and appends it to
`final_summary`:
auto: 1 marked stale; llm: consolidated 2 into 1, pruned 1
archived 3 skill(s):
• docx-extraction → document-tools
• pdf-extraction → document-tools
• old-stale-thing — pruned (stale)
full report: hermes curator status
Empty on no-op ticks (no archives), so most ticks add zero log noise.
Cap of 10 entries keeps agent.log readable when a 50-skill
consolidation lands; the full list is always in REPORT.md.
`hermes curator status` indents continuation lines so the multi-line
summary reads as one logical field.
5 new tests in tests/agent/test_curator_classification.py covering
empty / consolidation / pruning / cap / mixed cases.
* feat(curator): show recent run summary once on `hermes update`
The rename map is now visible from where users actually look — the
update flow they explicitly run, instead of just the live gateway log
or transient CLI session-start panel.
Behavior:
- After `hermes update`, if the most recent curator run produced a
rename map (multi-line summary) that the user hasn't seen yet, print
it once with a 'last run Xh ago' header and a one-time-message
footer.
- Stamp `last_run_summary_shown_at = last_run_at` after printing so
subsequent `hermes update` invocations are silent until a newer
curator run lands.
- Silent on no-op runs (single-line summary like 'auto: no changes;
llm: no change'). Still stamps shown so we don't reconsider on
every update.
- Silent when the curator has never run (the existing first-run
notice handles that case).
Output:
ℹ Skill curator — last run 4h ago
auto: 1 marked stale; llm: consolidated 2 into 1, pruned 1
archived 3 skill(s):
• docx-extraction → document-tools
• pdf-extraction → document-tools
• old-stale-thing — pruned (stale)
full report: hermes curator status
(This message shows once per curator run. View anytime: hermes curator status)
State migration:
- `_default_state()` gains `last_run_summary_shown_at: None`. Existing
state files lack the field; `.get()` returns None; the comparison
treats any prior run as 'not yet shown' and prints once on next
update. Self-healing.
Wiring:
- Both `hermes update` paths in main.py call the new
`_print_curator_recent_run_notice()` right after the existing
first-run notice. Best-effort try/except so a state-load bug
never breaks the update flow.
6 tests in tests/hermes_cli/test_curator_recent_run_notice.py:
no-run / single-line / multi-line / show-once / new-run-resets /
time-formatter buckets.
This commit is contained in:
parent
b67ea7ff47
commit
4375b82cd9
5 changed files with 499 additions and 1 deletions
|
|
@ -886,3 +886,137 @@ def test_reconcile_mixed_declarations_and_legacy_calls(curator_env):
|
|||
|
||||
assert "legacy-prune" in pruned_by_name
|
||||
assert "no-evidence fallback" in pruned_by_name["legacy-prune"]["source"]
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# _build_rename_summary — surfaces the "where did my skills go?" map to the
|
||||
# user-visible curator summary (gateway 💾 line, CLI Rich panel,
|
||||
# `hermes curator status`). The full data has always been in REPORT.md on
|
||||
# disk; this helper makes it visible without digging.
|
||||
# ---------------------------------------------------------------------------
|
||||
|
||||
|
||||
def test_rename_summary_empty_when_nothing_archived(curator_env):
|
||||
"""No removals = empty string (no log noise on no-op ticks)."""
|
||||
result = curator_env._build_rename_summary(
|
||||
before_names={"alpha", "beta"},
|
||||
after_report=[
|
||||
{"name": "alpha", "state": "active"},
|
||||
{"name": "beta", "state": "active"},
|
||||
],
|
||||
tool_calls=[],
|
||||
model_final="",
|
||||
)
|
||||
assert result == ""
|
||||
|
||||
|
||||
def test_rename_summary_consolidation_shows_target(curator_env):
|
||||
"""Consolidated skills render as `name → umbrella` with the actual target."""
|
||||
result = curator_env._build_rename_summary(
|
||||
before_names={"pdf-extraction", "docx-extraction", "document-tools"},
|
||||
after_report=[{"name": "document-tools", "state": "active"}],
|
||||
tool_calls=[
|
||||
{
|
||||
"name": "skill_manage",
|
||||
"arguments": json.dumps({
|
||||
"action": "delete",
|
||||
"name": "pdf-extraction",
|
||||
"absorbed_into": "document-tools",
|
||||
}),
|
||||
},
|
||||
{
|
||||
"name": "skill_manage",
|
||||
"arguments": json.dumps({
|
||||
"action": "delete",
|
||||
"name": "docx-extraction",
|
||||
"absorbed_into": "document-tools",
|
||||
}),
|
||||
},
|
||||
],
|
||||
model_final="",
|
||||
)
|
||||
assert "archived 2 skill(s):" in result
|
||||
assert "pdf-extraction → document-tools" in result
|
||||
assert "docx-extraction → document-tools" in result
|
||||
assert "full report: hermes curator status" in result
|
||||
|
||||
|
||||
def test_rename_summary_pruned_marked_explicitly(curator_env):
|
||||
"""Pruned skills (no umbrella) say `pruned (stale)` so users don't think they were merged."""
|
||||
result = curator_env._build_rename_summary(
|
||||
before_names={"old-flaky-thing", "keeper"},
|
||||
after_report=[{"name": "keeper", "state": "active"}],
|
||||
tool_calls=[
|
||||
{
|
||||
"name": "skill_manage",
|
||||
"arguments": json.dumps({
|
||||
"action": "delete",
|
||||
"name": "old-flaky-thing",
|
||||
"absorbed_into": "",
|
||||
}),
|
||||
},
|
||||
],
|
||||
model_final="",
|
||||
)
|
||||
assert "old-flaky-thing — pruned (stale)" in result
|
||||
assert "→" not in result.split("old-flaky-thing")[1].splitlines()[0]
|
||||
|
||||
|
||||
def test_rename_summary_caps_at_ten_with_more_indicator(curator_env):
|
||||
"""Large consolidations don't blow up the log line — cap + `… and N more`."""
|
||||
removed = [f"skill-{i}" for i in range(15)]
|
||||
tool_calls = [
|
||||
{
|
||||
"name": "skill_manage",
|
||||
"arguments": json.dumps({
|
||||
"action": "delete",
|
||||
"name": name,
|
||||
"absorbed_into": "umbrella",
|
||||
}),
|
||||
}
|
||||
for name in removed
|
||||
]
|
||||
result = curator_env._build_rename_summary(
|
||||
before_names=set(removed) | {"umbrella"},
|
||||
after_report=[{"name": "umbrella", "state": "active"}],
|
||||
tool_calls=tool_calls,
|
||||
model_final="",
|
||||
)
|
||||
assert "archived 15 skill(s):" in result
|
||||
assert "… and 5 more" in result
|
||||
# Exactly 10 bullets shown
|
||||
bullet_count = sum(1 for ln in result.splitlines() if ln.startswith(" • "))
|
||||
assert bullet_count == 10
|
||||
|
||||
|
||||
def test_rename_summary_mixed_consolidation_and_pruning(curator_env):
|
||||
"""Consolidated entries come first, pruned entries follow — matches REPORT.md ordering."""
|
||||
result = curator_env._build_rename_summary(
|
||||
before_names={"merge-me", "drop-me", "umbrella"},
|
||||
after_report=[{"name": "umbrella", "state": "active"}],
|
||||
tool_calls=[
|
||||
{
|
||||
"name": "skill_manage",
|
||||
"arguments": json.dumps({
|
||||
"action": "delete",
|
||||
"name": "merge-me",
|
||||
"absorbed_into": "umbrella",
|
||||
}),
|
||||
},
|
||||
{
|
||||
"name": "skill_manage",
|
||||
"arguments": json.dumps({
|
||||
"action": "delete",
|
||||
"name": "drop-me",
|
||||
"absorbed_into": "",
|
||||
}),
|
||||
},
|
||||
],
|
||||
model_final="",
|
||||
)
|
||||
lines = result.splitlines()
|
||||
merge_idx = next(i for i, ln in enumerate(lines) if "merge-me" in ln)
|
||||
drop_idx = next(i for i, ln in enumerate(lines) if "drop-me" in ln)
|
||||
assert merge_idx < drop_idx, "consolidated should render before pruned"
|
||||
assert "merge-me → umbrella" in lines[merge_idx]
|
||||
assert "drop-me — pruned (stale)" in lines[drop_idx]
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue