feat: multi-witness memory write helper
This commit is contained in:
+150
-1
@@ -22,7 +22,7 @@ from chat.db.migrate import apply_migrations
|
||||
from chat.eventlog.log import append_event
|
||||
from chat.eventlog.projector import project
|
||||
from chat.llm.mock import MockLLMClient
|
||||
from chat.services.memory_write import record_turn_memory
|
||||
from chat.services.memory_write import record_turn_memory, record_turn_memory_for_present
|
||||
import chat.state.entities # noqa: F401 - register handlers
|
||||
import chat.state.memory # noqa: F401
|
||||
import chat.state.world # noqa: F401
|
||||
@@ -295,3 +295,152 @@ def test_post_turn_writes_memory_for_host_bot(client, tmp_path):
|
||||
assert w_guest == 0
|
||||
assert source == "direct"
|
||||
assert sig == 1
|
||||
|
||||
|
||||
# ---------------------------------------------------------------------------
|
||||
# T41: record_turn_memory_for_present — multi-witness helper.
|
||||
# ---------------------------------------------------------------------------
|
||||
|
||||
|
||||
def _seed_two_bots(db_path: Path) -> None:
|
||||
"""Author host + guest bots and create a two-bot chat."""
|
||||
with open_db(db_path) as conn:
|
||||
for bot_id, name in (("bot_a", "BotA"), ("bot_b", "BotB")):
|
||||
append_event(
|
||||
conn,
|
||||
kind="bot_authored",
|
||||
payload={
|
||||
"id": bot_id,
|
||||
"name": name,
|
||||
"persona": "...",
|
||||
"voice_samples": [],
|
||||
"traits": [],
|
||||
"backstory": "",
|
||||
"initial_relationship_to_you": "",
|
||||
"kickoff_prose": "",
|
||||
},
|
||||
)
|
||||
append_event(
|
||||
conn,
|
||||
kind="chat_created",
|
||||
payload={
|
||||
"id": "chat_ab",
|
||||
"host_bot_id": "bot_a",
|
||||
"guest_bot_id": "bot_b",
|
||||
"initial_time": "2026-04-26T20:00:00+00:00",
|
||||
"narrative_anchor": "Day 1",
|
||||
"weather": "",
|
||||
},
|
||||
)
|
||||
project(conn)
|
||||
|
||||
|
||||
def test_record_for_present_no_guest_writes_single_memory_with_witness_1_1_0(tmp_path):
|
||||
db = tmp_path / "t.db"
|
||||
apply_migrations(db)
|
||||
_seed_minimal(db)
|
||||
with open_db(db) as conn:
|
||||
result = record_turn_memory_for_present(
|
||||
conn,
|
||||
chat_id="chat_bot_a",
|
||||
host_bot_id="bot_a",
|
||||
guest_bot_id=None,
|
||||
narrative_text="BotA glances out the window.",
|
||||
scene_id=None,
|
||||
chat_clock_at="2026-04-26T20:00:00+00:00",
|
||||
)
|
||||
|
||||
# Returned dict has only the host key.
|
||||
assert set(result.keys()) == {"bot_a"}
|
||||
eid_h, mid_h = result["bot_a"]
|
||||
assert eid_h > 0
|
||||
assert mid_h is not None and mid_h > 0
|
||||
|
||||
rows = conn.execute(
|
||||
"SELECT owner_id, witness_you, witness_host, witness_guest "
|
||||
"FROM memories"
|
||||
).fetchall()
|
||||
assert len(rows) == 1
|
||||
owner_id, w_you, w_host, w_guest = rows[0]
|
||||
assert owner_id == "bot_a"
|
||||
assert w_you == 1
|
||||
assert w_host == 1
|
||||
assert w_guest == 0
|
||||
|
||||
# Exactly one memory_written event was appended.
|
||||
cur = conn.execute(
|
||||
"SELECT COUNT(*) FROM event_log WHERE kind = 'memory_written'"
|
||||
)
|
||||
assert cur.fetchone()[0] == 1
|
||||
|
||||
|
||||
def test_record_for_present_with_guest_writes_two_memories_with_witness_1_1_1(tmp_path):
|
||||
db = tmp_path / "t.db"
|
||||
apply_migrations(db)
|
||||
_seed_two_bots(db)
|
||||
with open_db(db) as conn:
|
||||
result = record_turn_memory_for_present(
|
||||
conn,
|
||||
chat_id="chat_ab",
|
||||
host_bot_id="bot_a",
|
||||
guest_bot_id="bot_b",
|
||||
narrative_text="BotA and BotB share a glance.",
|
||||
scene_id=None,
|
||||
chat_clock_at="2026-04-26T20:00:00+00:00",
|
||||
)
|
||||
|
||||
# Returned dict has both keys.
|
||||
assert set(result.keys()) == {"bot_a", "bot_b"}
|
||||
eid_h, mid_h = result["bot_a"]
|
||||
eid_g, mid_g = result["bot_b"]
|
||||
assert eid_h > 0 and eid_g > 0
|
||||
assert mid_h is not None and mid_h > 0
|
||||
assert mid_g is not None and mid_g > 0
|
||||
# Distinct event ids and memory ids.
|
||||
assert eid_h != eid_g
|
||||
assert mid_h != mid_g
|
||||
|
||||
rows = conn.execute(
|
||||
"SELECT owner_id, witness_you, witness_host, witness_guest "
|
||||
"FROM memories ORDER BY owner_id"
|
||||
).fetchall()
|
||||
assert len(rows) == 2
|
||||
owners = {r[0] for r in rows}
|
||||
assert owners == {"bot_a", "bot_b"}
|
||||
# All rows should have witness mask [1, 1, 1].
|
||||
for _owner, w_you, w_host, w_guest in rows:
|
||||
assert w_you == 1
|
||||
assert w_host == 1
|
||||
assert w_guest == 1
|
||||
|
||||
# Two memory_written events were appended.
|
||||
cur = conn.execute(
|
||||
"SELECT COUNT(*) FROM event_log WHERE kind = 'memory_written'"
|
||||
)
|
||||
assert cur.fetchone()[0] == 2
|
||||
|
||||
|
||||
def test_record_for_present_dict_keys_match(tmp_path):
|
||||
db = tmp_path / "t.db"
|
||||
apply_migrations(db)
|
||||
_seed_two_bots(db)
|
||||
with open_db(db) as conn:
|
||||
# No guest: keys == {host_bot_id}.
|
||||
result_no_guest = record_turn_memory_for_present(
|
||||
conn,
|
||||
chat_id="chat_ab",
|
||||
host_bot_id="bot_a",
|
||||
guest_bot_id=None,
|
||||
narrative_text="Just BotA's POV.",
|
||||
)
|
||||
assert set(result_no_guest.keys()) == {"bot_a"}
|
||||
|
||||
# With guest: keys == {host_bot_id, guest_bot_id}.
|
||||
result_with_guest = record_turn_memory_for_present(
|
||||
conn,
|
||||
chat_id="chat_ab",
|
||||
host_bot_id="bot_a",
|
||||
guest_bot_id="bot_b",
|
||||
narrative_text="Both bots witness this.",
|
||||
)
|
||||
assert set(result_with_guest.keys()) == {"bot_a", "bot_b"}
|
||||
|
||||
Reference in New Issue
Block a user