hermes-admin/hermes-agent
1
0
Fork 0
Code Issues Pull Requests Actions 31 Packages Projects Releases Wiki Activity

docs(skills): explain why an undecodable ledger warns, without review jargon

Browse Source 
The docstring of test_list_entries_treats_an_undecodable_ledger_as_empty_and_warns
cited "(re-gate M8/S1)" (tests/tools/test_skill_ledger_delta.py:88; re-gate
G2 S1) — a review-process artifact meaningless to future readers. State the
behavioural reason instead: silence would make a corrupt ledger look like a
fresh install with no history.
This commit is contained in:
kshitijk4poor
2026-09-22 13:20:41 +05:30
committed by kshitij
parent d3684a7d79
commit 78dfd6e6e7
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
tests/tools/test_skill_ledger_delta.py
View File

@@ -86,7 +86,8 @@ def test_compact_leaves_an_undecodable_ledger_untouched(ledger_home, caplog):
def test_list_entries_treats_an_undecodable_ledger_as_empty_and_warns(ledger_home, caplog):
"""A ledger that exists but is not UTF-8 lists as empty (rollback then fails closed on
``get_entry`` -> None) and, unlike a merely missing ledger, is warned about (re-gate M8/S1)."""
``get_entry`` -> None) and, unlike a merely missing ledger, is warned about: corruption
would otherwise be indistinguishable from a fresh install with no history."""
from tools import skill_ledger
ledger = skill_ledger.ledger_path()
ledger.parent.mkdir(parents=True, exist_ok=True)