[kimi] feat: add agent_state message producer (#669) #698

Merged
kimi merged 1 commits from kimi/issue-669 into main 2026-03-21 13:46:11 +00:00
2 changed files with 142 additions and 1 deletions

View File

@@ -5,6 +5,7 @@ into the camelCase world-state payload consumed by the Workshop 3D renderer
and WebSocket gateway.
"""
import time
from datetime import UTC, datetime
@@ -40,3 +41,58 @@ def serialize_presence(presence: dict) -> dict:
"updatedAt": presence.get("liveness", datetime.now(UTC).strftime("%Y-%m-%dT%H:%M:%SZ")),
"version": presence.get("version", 1),
}
# ---------------------------------------------------------------------------
# Status mapping: ADR-023 current_focus → Matrix agent status
# ---------------------------------------------------------------------------
_STATUS_KEYWORDS: dict[str, str] = {
"thinking": "thinking",
"speaking": "speaking",
"talking": "speaking",
"idle": "idle",
}
def _derive_status(current_focus: str) -> str:
"""Map a free-text current_focus value to a Matrix status enum.
Returns one of: online, idle, thinking, speaking.
"""
focus_lower = current_focus.lower()
for keyword, status in _STATUS_KEYWORDS.items():
if keyword in focus_lower:
return status
if current_focus and current_focus != "idle":
return "online"
return "idle"
def produce_agent_state(agent_id: str, presence: dict) -> dict:
"""Build a Matrix-compatible ``agent_state`` message from presence data.
Parameters
----------
agent_id:
Unique identifier for the agent (e.g. ``"timmy"``).
presence:
Raw ADR-023 presence dict.
Returns
-------
dict
Message with keys ``type``, ``agent_id``, ``data``, and ``ts``.
"""
return {
"type": "agent_state",
"agent_id": agent_id,
"data": {
"display_name": presence.get("display_name", agent_id.title()),
"role": presence.get("role", "assistant"),
"status": _derive_status(presence.get("current_focus", "idle")),
"mood": presence.get("mood", "calm"),
"energy": presence.get("energy", 0.5),
"bark": presence.get("bark", ""),
},
"ts": int(time.time()),
}

View File

@@ -1,8 +1,10 @@
"""Tests for infrastructure.presence — presence state serializer."""
from unittest.mock import patch
import pytest
from infrastructure.presence import serialize_presence
from infrastructure.presence import produce_agent_state, serialize_presence
class TestSerializePresence:
@@ -78,3 +80,86 @@ class TestSerializePresence:
"""visitorPresent is always False — set by the WS layer, not here."""
assert serialize_presence(full_presence)["visitorPresent"] is False
assert serialize_presence({})["visitorPresent"] is False
class TestProduceAgentState:
"""Tests for produce_agent_state() — Matrix agent_state message producer."""
@pytest.fixture()
def full_presence(self):
"""A presence dict with all agent_state-relevant fields."""
return {
"display_name": "Timmy",
"role": "companion",
"current_focus": "thinking about tests",
"mood": "focused",
"energy": 0.9,
"bark": "Running test suite...",
}
@patch("infrastructure.presence.time")
def test_full_message_structure(self, mock_time, full_presence):
"""Returns dict with type, agent_id, data, and ts keys."""
mock_time.time.return_value = 1742529600
result = produce_agent_state("timmy", full_presence)
assert result["type"] == "agent_state"
assert result["agent_id"] == "timmy"
assert result["ts"] == 1742529600
assert isinstance(result["data"], dict)
def test_data_fields(self, full_presence):
"""data dict contains all required presence fields."""
data = produce_agent_state("timmy", full_presence)["data"]
assert data["display_name"] == "Timmy"
assert data["role"] == "companion"
assert data["status"] == "thinking"
assert data["mood"] == "focused"
assert data["energy"] == 0.9
assert data["bark"] == "Running test suite..."
def test_defaults_on_empty_presence(self):
"""Missing fields get sensible defaults."""
result = produce_agent_state("timmy", {})
data = result["data"]
assert data["display_name"] == "Timmy" # agent_id.title()
assert data["role"] == "assistant"
assert data["status"] == "idle"
assert data["mood"] == "calm"
assert data["energy"] == 0.5
assert data["bark"] == ""
def test_ts_is_unix_timestamp(self):
"""ts should be an integer Unix timestamp."""
result = produce_agent_state("timmy", {})
assert isinstance(result["ts"], int)
assert result["ts"] > 0
@pytest.mark.parametrize(
("focus", "expected_status"),
[
("thinking about code", "thinking"),
("speaking to user", "speaking"),
("talking with agent", "speaking"),
("idle", "idle"),
("", "idle"),
("writing tests", "online"),
("reviewing PR", "online"),
],
)
def test_status_derivation(self, focus, expected_status):
"""current_focus maps to the correct Matrix status."""
data = produce_agent_state("t", {"current_focus": focus})["data"]
assert data["status"] == expected_status
def test_agent_id_passed_through(self):
"""agent_id appears in the top-level message."""
result = produce_agent_state("spark", {})
assert result["agent_id"] == "spark"
def test_display_name_from_agent_id(self):
"""When display_name is missing, it's derived from agent_id.title()."""
data = produce_agent_state("spark", {})["data"]
assert data["display_name"] == "Spark"