From 7626c2e2b5d1f20a890ad053326f5ad9339f45bc Mon Sep 17 00:00:00 2001
From: Scott Severance <scott@scottseverance.net>
Date: Tue, 3 Nov 2026 07:40:05 +0000
Subject: [PATCH] fix: validate message role is not none or empty

---
 agent/core/conversation.py | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/agent/core/conversation.py b/agent/core/conversation.py
index 488cf74..b344869 100644
--- a/agent/core/conversation.py
+++ b/agent/core/conversation.py
@@ -17,6 +17,10 @@ class Message:
     name: Optional[str] = None  # tool name for tool messages
     timestamp: float = field(default_factory=time.time)
 
+    def __post_init__(self):
+        if not self.role or not self.role.strip():
+            raise ValueError("Message role cannot be None or empty")
+
     def to_dict(self) -> dict:
         d: dict[str, Any] = {"role": self.role, "content": self.content}
         if self.tool_calls: