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

test(persistence): pin metadata-only adoption and stored-row digests

Browse Source 
Extend the kept active-row test: a reaction between flushes must not replace
the live multimodal user content with its text projection (red on the previous
tip at the image assertion), and the reaction metadata is synced. The user row
carries an int message_id so the stored-row (TEXT affinity) digest path is
exercised.
This commit is contained in:
kshitijk4poor
2026-09-27 14:43:00 +05:30
committed by kshitij
parent ca7e4c1636
commit 43b1dd7c5f
1 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
tests/agent/test_tool_call_incremental_persistence.py
View File

@@ -762,7 +762,8 @@ def test_flush_sanitized_active_user_and_tool_rows_do_not_append_duplicates(tmp_
session_id = "sess-sanitized-active-rows"
db = _attach_real_session_db(agent, tmp_path / "state.db", session_id)
messages = [
{"role": "user", "content": [
# Int message_id: SQLite TEXT affinity stores it as "12345", so the digest must hash stored rows.
{"role": "user", "message_id": 12345, "content": [
{"type": "text", "text": "hi \ud800 there " + "x" * 4000},
{"type": "image_url", "image_url": {"url": "data:image/png;base64,AAAA"}},
]},
@@ -779,6 +780,8 @@ def test_flush_sanitized_active_user_and_tool_rows_do_not_append_duplicates(tmp_
with other:
other.execute("UPDATE messages SET content = ? WHERE id = ?", ("winner", durable_ids[2]))
other.close()
# A same-process metadata write (reaction) changes the user row's digest but not its content.
assert db.set_message_reaction(session_id, durable_ids[0], "\U0001F44D")
assert _sanitize_messages_surrogates(messages) is True
assert not any(message.get("_db_persisted") for message in (messages[0], messages[2]))
@@ -790,8 +793,10 @@ def test_flush_sanitized_active_user_and_tool_rows_do_not_append_duplicates(tmp_
assert [row["id"] for row in rows] == durable_ids
assert [message["_row_id"] for message in messages] == durable_ids
assert rows[0]["content"].startswith("hi \ufffd there")
# Our own rewrite never copies the lossy durable projection back: the live image part survives.
# Neither our own rewrite nor a metadata-only change copies the lossy durable projection back: the live
# image part survives while the reaction metadata is synced.
assert messages[0]["content"][1]["type"] == "image_url"
assert messages[0]["display_metadata"] == rows[0]["display_metadata"]
assert rows[2]["content"] == "winner"
assert messages[2]["content"] == "winner"