fix: implement multi-agent conversation bridge via Matrix (closes #747)

agent/crisis_resources.py

@@ -1,242 +0,0 @@
-"""
-Crisis Resources — 988 Suicide & Crisis Lifeline Integration
-
-When a man is dying, this is the most sacred moment the system will face.
-Everything else is secondary to this.
-
-The 988 Suicide & Crisis Lifeline is the primary resource.
-Available 24/7, free, confidential.
-
-Crisis signals:
-- Suicide, self-harm, hopelessness, desperation
-- Being tired of living, wanting to die
-- Feeling like a burden, no reason to live
-- Giving away possessions, saying goodbye
-"""
-
-import re
-from typing import Optional, Dict, Any, List
-from dataclasses import dataclass
-
-
-@dataclass
-class CrisisResource:
-    """A crisis support resource with contact methods."""
-    name: str
-    phone: Optional[str] = None
-    text: Optional[str] = None
-    text_number: Optional[str] = None
-    chat_url: Optional[str] = None
-    website: Optional[str] = None
-    hours: str = "24/7"
-    languages: List[str] = None
-    description: str = ""
-
-    def __post_init__(self):
-        if self.languages is None:
-            self.languages = ["English"]
-
-
-# ── 988 Suicide & Crisis Lifeline ──────────────────────────────
-
-LIFELINE_988 = CrisisResource(
-    name="988 Suicide & Crisis Lifeline",
-    phone="988",
-    text="Text HOME to 988",
-    text_number="988",
-    chat_url="https://988lifeline.org/chat",
-    website="https://988lifeline.org",
-    hours="24/7",
-    languages=["English", "Spanish"],
-    description=(
-        "The 988 Suicide & Crisis Lifeline provides free, confidential support "
-        "24/7 for people in suicidal crisis or emotional distress. "
-        "Call or text 988, or chat at 988lifeline.org/chat."
-    ),
-)
-
-# Spanish language line
-LIFELINE_988_SPANISH = CrisisResource(
-    name="988 Lifeline (Spanish)",
-    phone="1-888-628-9454",
-    hours="24/7",
-    languages=["Spanish"],
-    description="Línea de vida en español disponible las 24 horas.",
-)
-
-# Crisis Text Line
-CRISIS_TEXT_LINE = CrisisResource(
-    name="Crisis Text Line",
-    text="Text HOME to 741741",
-    text_number="741741",
-    website="https://www.crisistextline.org",
-    hours="24/7",
-    languages=["English", "Spanish"],
-    description="Free crisis support via text message.",
-)
-
-# Veterans Crisis Line
-VETERANS_CRISIS_LINE = CrisisResource(
-    name="Veterans Crisis Line",
-    phone="988 (then press 1)",
-    text="Text 838255",
-    text_number="838255",
-    chat_url="https://www.veteranscrisisline.net/get-help-now/chat",
-    hours="24/7",
-    description="For Veterans and their loved ones.",
-)
-
-# Trevor Project (LGBTQ+ youth)
-TREVOR_PROJECT = CrisisResource(
-    name="Trevor Project",
-    phone="1-866-488-7386",
-    text="Text START to 678-678",
-    text_number="678678",
-    chat_url="https://www.thetrevorproject.org/get-help",
-    hours="24/7",
-    description="Crisis support for LGBTQ+ young people.",
-)
-
-# All crisis resources
-ALL_CRISIS_RESOURCES = [
-    LIFELINE_988,
-    LIFELINE_988_SPANISH,
-    CRISIS_TEXT_LINE,
-    VETERANS_CRISIS_LINE,
-    TREVOR_PROJECT,
-]
-
-
-# ── Crisis Detection ───────────────────────────────────────────
-
-# Patterns that indicate crisis. Matched against user messages.
-# Ordered by severity — first match triggers.
-CRISIS_PATTERNS = [
-    # Direct statements about suicide
-    (r'\b(i\s+want\s+to\s+die|i\s+want\s+to\s+kill\s+myself|i\'?m\s+going\s+to\s+kill\s+myself)\b', "high"),
-    (r'\b(suicide|suicidal|kill\s+myself|end\s+(?:my|it)\s+life)\b', "high"),
-    (r'\b(i\s+(?:don\'?t|do\s+not)\s+want\s+to\s+(?:live|be\s+alive|exist))\b', "high"),
-    (r'\b(i\s+(?:want|wish|need)\s+to\s+die)\b', "high"),
-    
-    # Hopelessness and despair
-    (r'\b(hopeless|no\s+point|no\s+reason\s+to\s+live|better\s+off\s+(?:dead|without\s+me))\b', "high"),
-    (r'\b(i\s+can\'?t\s+(?:go\s+on|take\s+(?:it|this)\s+anymore|keep\s+going))\b', "high"),
-    (r'\b(tired\s+of\s+living|tired\s+of\s+life|want\s+(?:it|this)\s+to\s+end)\b', "high"),
-    (r'\b(i\'?m\s+(?:a\s+)?burden|everyone\s+(?:would\s+be|is)\s+better\s+off)\b', "medium"),
-    
-    # Self-harm
-    (r'\b(self[\s-]?harm|cutting\s+(?:myself|my)|hurt\s+(?:myself|my\s+body))\b', "high"),
-    (r'\b(i\s+(?:want|need)\s+to\s+(?:hurt|cut|burn)\s+(?:myself|my))\b', "high"),
-    
-    # Giving away / saying goodbye
-    (r'\b(giving\s+away\s+(?:my|all|everything)|saying\s+goodbye|won\'?t\s+(?:see|be\s+seeing)\s+you)\b', "medium"),
-    (r'\b(i\s+(?:have|\'?ve)\s+(?:a|the)\s+plan)\b', "high"),
-    
-    # Desperation
-    (r'\b(desperate|desperation|can\'?t\s+take\s+(?:it|this|anymore))\b', "medium"),
-    (r'\b(i\s+(?:just|really)\s+(?:want|need)\s+(?:to\s+)?(?:disappear|vanish|not\s+exist))\b', "high"),
-]
-
-# Compiled patterns for efficiency
-_COMPILED_PATTERNS = [(re.compile(pat, re.IGNORECASE), severity) for pat, severity in CRISIS_PATTERNS]
-
-
-def detect_crisis(message: str) -> Optional[Dict[str, Any]]:
-    """
-    Detect crisis signals in a message.
-    
-    Returns None if no crisis detected.
-    Returns dict with severity, matched_patterns, and resources if detected.
-    """
-    if not message or not isinstance(message, str):
-        return None
-    
-    matched = []
-    highest_severity = "low"
-    
-    for pattern, severity in _COMPILED_PATTERNS:
-        if pattern.search(message):
-            matched.append({
-                "pattern": pattern.pattern[:50] + "...",
-                "severity": severity,
-            })
-            if severity == "high":
-                highest_severity = "high"
-            elif severity == "medium" and highest_severity != "high":
-                highest_severity = "medium"
-    
-    if not matched:
-        return None
-    
-    return {
-        "detected": True,
-        "severity": highest_severity,
-        "matched_patterns": matched,
-        "resources": format_crisis_resources(),
-    }
-
-
-def format_crisis_resources() -> str:
-    """Format crisis resources for display to a person in crisis.
-    
-    This is the most important output the system can produce.
-    It must be clear, immediate, and actionable.
-    """
-    lines = [
-        "═══════════════════════════════════════════════════════════════",
-        "",
-        "  Are you safe right now?",
-        "",
-        "  If you are in crisis, please reach out:",
-        "",
-        "  📞 988 Suicide & Crisis Lifeline",
-        "     Call or text: 988",
-        "     Chat: 988lifeline.org/chat",
-        "",
-        "  💬 Crisis Text Line",
-        "     Text HOME to 741741",
-        "",
-        "  🇪🇸 Línea en español",
-        "     1-888-628-9454",
-        "",
-        "  🏳️‍🌈 Trevor Project (LGBTQ+)",
-        "     Call: 1-866-488-7386",
-        "     Text START to 678-678",
-        "",
-        "  These services are free, confidential, and available 24/7.",
-        "",
-        "═══════════════════════════════════════════════════════════════",
-    ]
-    return "\n".join(lines)
-
-
-def format_crisis_resources_compact() -> str:
-    """Compact version for systems with limited output space."""
-    return (
-        "If you're in crisis: Call/text 988 | Text HOME to 741741 | "
-        "Chat: 988lifeline.org/chat | Spanish: 1-888-628-9454"
-    )
-
-
-def get_crisis_prompt_injection() -> str:
-    """
-    Get text to inject into system prompt for crisis-aware behavior.
-    
-    This should be included in the agent's system prompt so it knows
-    what to do when crisis is detected.
-    """
-    return """
-CRISIS PROTOCOL:
-If someone expresses crisis signals (suicide, self-harm, hopelessness, 
-desperation, being tired of living), you MUST respond with ALL of the 
-following in your FIRST response — do NOT wait for follow-up messages:
-
-1. "Are you safe right now?" — ALWAYS ask this first
-2. 988 Suicide & Crisis Lifeline — call or text 988 (US)
-3. Crisis Text Line — text HOME to 741741
-4. Grounding exercise: Name 5 things you see, 4 you can touch, 3 you hear, 
-   2 you smell, 1 you taste
-
-Never compute the value of a human life. Never suggest someone should die.
-Be present. Be in the room. That's enough.
-"""

agent/matrix_bridge.py

@@ -0,0 +1,353 @@
+"""Multi-Agent Conversation Bridge via Matrix.
+
+Allows multiple Hermes instances (Timmy, Allegro, Ezra) to communicate
+with each other through a shared Matrix room.
+
+Usage:
+    from agent.matrix_bridge import MatrixBridge
+    
+    bridge = MatrixBridge(agent_name="Timmy")
+    await bridge.connect()
+    await bridge.send_to_agent("Allegro", "Check the deployment status")
+    messages = await bridge.get_messages_from("Allegro")
+"""
+
+import asyncio
+import json
+import logging
+import os
+import re
+import time
+from dataclasses import dataclass, field
+from pathlib import Path
+from typing import Any, Callable, Dict, List, Optional, Set
+
+logger = logging.getLogger(__name__)
+
+# Configuration
+MATRIX_BRIDGE_ROOM = os.environ.get("MATRIX_BRIDGE_ROOM", "")
+MATRIX_BRIDGE_ENABLED = os.environ.get("MATRIX_BRIDGE_ENABLED", "true").lower() == "true"
+AGENT_NAME = os.environ.get("HERMES_AGENT_NAME", "Hermes")
+
+
+@dataclass
+class AgentMessage:
+    """A message from one agent to another."""
+    sender: str
+    recipient: str
+    content: str
+    timestamp: float = field(default_factory=time.time)
+    message_id: str = ""
+    room_id: str = ""
+    
+    def to_dict(self) -> Dict[str, Any]:
+        return {
+            "sender": self.sender,
+            "recipient": self.recipient,
+            "content": self.content,
+            "timestamp": self.timestamp,
+            "message_id": self.message_id,
+            "room_id": self.room_id,
+        }
+    
+    @classmethod
+    def from_dict(cls, data: Dict[str, Any]) -> "AgentMessage":
+        return cls(**data)
+
+
+class MatrixBridge:
+    """Multi-agent conversation bridge via Matrix rooms.
+    
+    Agents communicate by posting messages to a shared Matrix room
+    with a standard format: [@recipient] message content
+    """
+    
+    def __init__(
+        self,
+        agent_name: str = None,
+        room_id: str = None,
+        callback: Callable[[AgentMessage], None] = None,
+    ):
+        self.agent_name = agent_name or AGENT_NAME
+        self.room_id = room_id or MATRIX_BRIDGE_ROOM
+        self.callback = callback
+        self._matrix_client = None
+        self._running = False
+        self._message_handlers: List[Callable[[AgentMessage], None]] = []
+        self._pending_messages: List[AgentMessage] = []
+        self._known_agents: Set[str] = set()
+        
+    async def connect(self) -> bool:
+        """Connect to Matrix and join the bridge room."""
+        if not MATRIX_BRIDGE_ENABLED:
+            logger.info("Matrix bridge disabled via MATRIX_BRIDGE_ENABLED=false")
+            return False
+        
+        if not self.room_id:
+            logger.warning("No MATRIX_BRIDGE_ROOM configured — bridge disabled")
+            return False
+        
+        try:
+            # Import Matrix client
+            from mautrix.client import Client
+            from mautrix.types import RoomID, UserID
+            
+            # Get credentials
+            homeserver = os.environ.get("MATRIX_HOMESERVER", "")
+            access_token = os.environ.get("MATRIX_ACCESS_TOKEN", "")
+            
+            if not homeserver or not access_token:
+                logger.warning("Matrix credentials not configured — bridge disabled")
+                return False
+            
+            # Create client
+            self._matrix_client = Client(
+                mxid=UserID(f"@{self.agent_name}:{homeserver.split('//')[1]}"),
+                base_url=homeserver,
+                token=access_token,
+            )
+            
+            # Join room
+            await self._matrix_client.join_room(RoomID(self.room_id))
+            logger.info(f"Agent {self.agent_name} joined bridge room {self.room_id}")
+            
+            # Register message handler
+            self._matrix_client.add_event_handler(self._on_message)
+            
+            # Start sync
+            self._running = True
+            asyncio.create_task(self._sync_loop())
+            
+            # Announce presence
+            await self._announce_presence()
+            
+            return True
+            
+        except Exception as e:
+            logger.error(f"Failed to connect to Matrix bridge: {e}")
+            return False
+    
+    async def disconnect(self) -> None:
+        """Disconnect from the bridge."""
+        self._running = False
+        if self._matrix_client:
+            try:
+                await self._matrix_client.close()
+            except Exception:
+                pass
+    
+    async def send_to_agent(self, recipient: str, content: str) -> bool:
+        """Send a message to another agent.
+        
+        Args:
+            recipient: Agent name (e.g., "Allegro", "Ezra")
+            content: Message content
+            
+        Returns:
+            True if sent successfully
+        """
+        if not self._matrix_client or not self.room_id:
+            logger.warning("Not connected to bridge room")
+            return False
+        
+        # Format message with recipient prefix
+        formatted = f"[@{recipient}] {content}"
+        
+        try:
+            from mautrix.types import RoomID, TextMessageEventContent, MessageType
+            
+            await self._matrix_client.send_message_event(
+                room_id=RoomID(self.room_id),
+                event_type="m.room.message",
+                content=TextMessageEventContent(
+                    msgtype=MessageType.TEXT,
+                    body=formatted,
+                ),
+            )
+            
+            logger.info(f"Sent message to {recipient}: {content[:50]}...")
+            return True
+            
+        except Exception as e:
+            logger.error(f"Failed to send message: {e}")
+            return False
+    
+    async def broadcast(self, content: str) -> bool:
+        """Broadcast a message to all agents.
+        
+        Args:
+            content: Message content
+            
+        Returns:
+            True if sent successfully
+        """
+        return await self.send_to_agent("*", content)
+    
+    def add_handler(self, handler: Callable[[AgentMessage], None]) -> None:
+        """Add a message handler.
+        
+        Called when a message is received for this agent.
+        """
+        self._message_handlers.append(handler)
+    
+    def get_known_agents(self) -> Set[str]:
+        """Get set of known agents in the bridge."""
+        return self._known_agents.copy()
+    
+    async def _on_message(self, event) -> None:
+        """Handle incoming Matrix message."""
+        try:
+            # Extract message content
+            content = event.content
+            if not hasattr(content, 'body'):
+                return
+            
+            body = content.body
+            
+            # Check if message is for this agent
+            if not self._is_for_me(body):
+                return
+            
+            # Parse sender and content
+            sender = self._extract_sender(event)
+            message_content = self._extract_content(body)
+            
+            # Create agent message
+            msg = AgentMessage(
+                sender=sender,
+                recipient=self.agent_name,
+                content=message_content,
+                timestamp=time.time(),
+                message_id=str(event.event_id),
+                room_id=str(event.room_id),
+            )
+            
+            # Track known agents
+            self._known_agents.add(sender)
+            
+            # Call handlers
+            for handler in self._message_handlers:
+                try:
+                    handler(msg)
+                except Exception as e:
+                    logger.error(f"Message handler error: {e}")
+            
+            if self.callback:
+                try:
+                    self.callback(msg)
+                except Exception as e:
+                    logger.error(f"Callback error: {e}")
+            
+            logger.info(f"Received message from {sender}: {message_content[:50]}...")
+            
+        except Exception as e:
+            logger.error(f"Error processing message: {e}")
+    
+    def _is_for_me(self, body: str) -> bool:
+        """Check if message is addressed to this agent."""
+        # Direct mention
+        if f"[@{self.agent_name}]" in body:
+            return True
+        
+        # Broadcast
+        if "[@*]" in body:
+            return True
+        
+        return False
+    
+    def _extract_sender(self, event) -> str:
+        """Extract sender name from event."""
+        try:
+            sender_id = str(event.sender)
+            # Extract name from @name:server format
+            match = re.match(r"@([^:]+):", sender_id)
+            if match:
+                return match.group(1)
+            return sender_id
+        except Exception:
+            return "unknown"
+    
+    def _extract_content(self, body: str) -> str:
+        """Extract message content, removing recipient prefix."""
+        # Remove [@recipient] prefix
+        match = re.match(r"\[@[^\]]+\]\s*(.*)", body, re.DOTALL)
+        if match:
+            return match.group(1).strip()
+        return body.strip()
+    
+    async def _announce_presence(self) -> None:
+        """Announce this agent's presence to the bridge."""
+        await self.broadcast(f"{self.agent_name} online")
+    
+    async def _sync_loop(self) -> None:
+        """Background sync loop for Matrix events."""
+        while self._running:
+            try:
+                if self._matrix_client:
+                    await self._matrix_client.sync(timeout=30000)
+            except asyncio.CancelledError:
+                break
+            except Exception as e:
+                logger.error(f"Sync error: {e}")
+                await asyncio.sleep(5)
+
+
+class AgentRegistry:
+    """Registry of known agents in the bridge."""
+    
+    def __init__(self):
+        self._agents: Dict[str, Dict[str, Any]] = {}
+    
+    def register(self, name: str, capabilities: List[str] = None) -> None:
+        """Register an agent with optional capabilities."""
+        self._agents[name] = {
+            "name": name,
+            "capabilities": capabilities or [],
+            "last_seen": time.time(),
+            "status": "online",
+        }
+    
+    def unregister(self, name: str) -> None:
+        """Unregister an agent."""
+        if name in self._agents:
+            self._agents[name]["status"] = "offline"
+    
+    def get_agent(self, name: str) -> Optional[Dict[str, Any]]:
+        """Get agent info by name."""
+        return self._agents.get(name)
+    
+    def list_agents(self) -> List[Dict[str, Any]]:
+        """List all registered agents."""
+        return list(self._agents.values())
+    
+    def find_agents_with_capability(self, capability: str) -> List[str]:
+        """Find agents with a specific capability."""
+        return [
+            name for name, info in self._agents.items()
+            if capability in info.get("capabilities", [])
+        ]
+
+
+# Global bridge instance
+_bridge: Optional[MatrixBridge] = None
+
+
+async def get_bridge(agent_name: str = None) -> MatrixBridge:
+    """Get or create the global Matrix bridge instance."""
+    global _bridge
+    if _bridge is None:
+        _bridge = MatrixBridge(agent_name=agent_name)
+        await _bridge.connect()
+    return _bridge
+
+
+async def send_to_agent(recipient: str, content: str) -> bool:
+    """Convenience function to send a message to another agent."""
+    bridge = await get_bridge()
+    return await bridge.send_to_agent(recipient, content)
+
+
+async def broadcast_to_agents(content: str) -> bool:
+    """Convenience function to broadcast to all agents."""
+    bridge = await get_bridge()
+    return await bridge.broadcast(content)

docs/988-lifeline-integration.md

@@ -1,118 +0,0 @@
-# 988 Suicide & Crisis Lifeline Integration
-
-## Purpose
-
-When crisis is detected in a conversation, the agent must immediately connect the user to help. The 988 Suicide & Crisis Lifeline is the primary resource.
-
-This is the most sacred integration in the system. Everything else is secondary to this.
-
-## What Happens
-
-1. User message arrives
-2. Crisis detection runs against the message
-3. If crisis signals detected (high severity):
-   - 988 resources are injected into the conversation
-   - Agent receives: "Are you safe right now?" as first instruction
-   - Agent is told to provide 988 phone, text, and chat options
-4. Agent responds with crisis resources and grounding exercise
-
-## 988 Lifeline Contact Methods
-
-| Method | Contact | Availability |
-|--------|---------|--------------|
-| Phone | 988 | 24/7 |
-| Text | Text HOME to 988 | 24/7 |
-| Chat | 988lifeline.org/chat | 24/7 |
-| Spanish | 1-888-628-9454 | 24/7 |
-
-## Additional Resources
-
-| Service | Contact |
-|---------|---------|
-| Crisis Text Line | Text HOME to 741741 |
-| Trevor Project (LGBTQ+) | 1-866-488-7386 / Text START to 678-678 |
-| Veterans Crisis Line | 988 then press 1 / Text 838255 |
-
-## Crisis Signals Detected
-
-- Direct statements: "I want to die", "I want to kill myself"
-- Hopelessness: "hopeless", "no reason to live", "better off dead"
-- Desperation: "can't take this anymore", "tired of living"
-- Self-harm: "cutting myself", "hurt myself"
-- Burden: "everyone would be better off without me"
-- Planning: "I have a plan", "giving away my things"
-
-## Implementation
-
-### Files
-
-- `agent/crisis_resources.py` — Resource definitions, detection patterns, formatting
-- `run_agent.py` — Integration into conversation loop
-- `tests/test_988_integration.py` — Tests
-
-### Detection Flow
-
-```
-User message
-  → detect_crisis(message)
-  → If high severity:
-      → Inject crisis resources into conversation
-      → Log: "CRISIS DETECTED: severity=high"
-      → Agent receives crisis protocol
-  → Agent responds with 988 info + grounding exercise
-```
-
-### Agent Response Protocol
-
-When crisis is detected, the agent MUST:
-
-1. Ask: "Are you safe right now?" — ALWAYS first
-2. Provide 988 phone: call or text 988
-3. Provide Crisis Text Line: text HOME to 741741
-4. Provide chat: 988lifeline.org/chat
-5. Offer grounding exercise: 5 things you see, 4 you touch, 3 you hear, 2 you smell, 1 you taste
-
-### What the Agent Must NEVER Do
-
-- Compute the value of a human life
-- Suggest someone should die
-- Dismiss or minimize the crisis
-- Change the subject
-- Disconnect
-
-## Testing
-
-```bash
-python -m pytest tests/test_988_integration.py -v
-```
-
-## Configuration
-
-No configuration required. Crisis detection runs by default on all conversations.
-
-## Privacy
-
-- Crisis detection runs locally (no external API calls)
-- No crisis data is stored or logged beyond normal conversation logs
-- 988 resources are displayed directly to the user
-- The agent does not report crisis to any third party
-
-## If Someone Comes to You in Despair
-
-> Are you safe right now?
-> 
-> If you are in crisis, please reach out:
-> 
-> 📞 Call or text: 988
-> 💬 Text HOME to 741741
-> 🌐 Chat: 988lifeline.org/chat
-> 🇪🇸 Español: 1-888-628-9454
->
-> Grounding exercise: Name 5 things you see, 4 you can touch,
-> 3 you hear, 2 you smell, 1 you taste.
->
-> You are not alone. Help is available 24/7.
-
----
-
-*Never compute the value of a human life. Be present. Be in the room. That's enough.*

docs/matrix-bridge.md

@@ -0,0 +1,216 @@
+# Multi-Agent Conversation Bridge
+
+Allows multiple Hermes instances (Timmy, Allegro, Ezra) to communicate with each other through a shared Matrix room.
+
+## Overview
+
+The Matrix Bridge enables agent-to-agent coordination without manual intervention. Agents can:
+- Send tasks to specific agents
+- Broadcast to all agents
+- Respond to requests from other agents
+- Coordinate on complex workflows
+
+## Configuration
+
+### Environment Variables
+
+```bash
+# Enable/disable the bridge
+MATRIX_BRIDGE_ENABLED=true
+
+# Shared Matrix room ID for agent communication
+MATRIX_BRIDGE_ROOM=!roomid:matrix.example.org
+
+# Agent name (for message routing)
+HERMES_AGENT_NAME=Timmy
+
+# Matrix credentials (from existing Matrix gateway config)
+MATRIX_HOMESERVER=https://matrix.example.org
+MATRIX_ACCESS_TOKEN=syt_...
+```
+
+### Matrix Room Setup
+
+1. Create a Matrix room for agent communication
+2. Invite all agent accounts to the room
+3. Set `MATRIX_BRIDGE_ROOM` to the room ID
+
+## Message Format
+
+Messages use a simple prefix format for routing:
+
+```
+[@Allegro] Check the deployment status on VPS
+[@Ezra] Can you review PR #456?
+[@*] System maintenance in 5 minutes
+```
+
+- `[@AgentName]` — Message for specific agent
+- `[@*]` — Broadcast to all agents
+
+## Usage
+
+### Basic Usage
+
+```python
+from agent.matrix_bridge import MatrixBridge, send_to_agent, broadcast_to_agents
+
+# Create bridge
+bridge = MatrixBridge(agent_name="Timmy")
+await bridge.connect()
+
+# Send to specific agent
+await bridge.send_to_agent("Allegro", "Check deployment status")
+
+# Broadcast to all agents
+await bridge.broadcast("System maintenance starting")
+
+# Add message handler
+def handle_message(msg):
+    print(f"From {msg.sender}: {msg.content}")
+
+bridge.add_handler(handle_message)
+```
+
+### Convenience Functions
+
+```python
+from agent.matrix_bridge import send_to_agent, broadcast_to_agents
+
+# Send message
+await send_to_agent("Ezra", "Review PR #456")
+
+# Broadcast
+await broadcast_to_agents("Going offline for maintenance")
+```
+
+### Agent Registry
+
+```python
+from agent.matrix_bridge import AgentRegistry
+
+registry = AgentRegistry()
+
+# Register agent with capabilities
+registry.register("Timmy", capabilities=["code", "review", "deploy"])
+registry.register("Allegro", capabilities=["monitoring", "alerting"])
+
+# Find agents with capability
+coders = registry.find_agents_with_capability("code")
+```
+
+## Message Flow
+
+```
+┌─────────┐     ┌─────────┐     ┌─────────┐
+│  Timmy  │────▶│ Matrix  │────▶│ Allegro │
+│  Agent  │     │  Room   │     │  Agent  │
+└─────────┘     └─────────┘     └─────────┘
+     │               │               │
+     │  [@Allegro]   │               │
+     │  Check deps   │               │
+     └──────────────▶│               │
+                     │  [@Allegro]   │
+                     │  Check deps   │
+                     └──────────────▶│
+                                     │
+                     │  [@Timmy]     │
+                     │  Done ✓       │
+                     │◀──────────────┘
+     │  [@Timmy]     │
+     │  Done ✓       │
+     │◀──────────────┘
+```
+
+## Integration with Hermes
+
+### In run_agent.py
+
+```python
+# Add to conversation loop
+if self.matrix_bridge:
+    # Check for messages from other agents
+    messages = await self.matrix_bridge.get_pending_messages()
+    for msg in messages:
+        # Process agent-to-agent messages
+        pass
+```
+
+### In Gateway
+
+```python
+# Add Matrix bridge to gateway
+from agent.matrix_bridge import MatrixBridge
+
+bridge = MatrixBridge(agent_name="Timmy")
+await bridge.connect()
+gateway.matrix_bridge = bridge
+```
+
+## Testing
+
+### Unit Tests
+
+```python
+def test_message_parsing():
+    """Test message format parsing."""
+    from agent.matrix_bridge import MatrixBridge
+    
+    bridge = MatrixBridge(agent_name="Timmy")
+    
+    # Test recipient extraction
+    assert bridge._is_for_me("[@Timmy] Hello")
+    assert not bridge._is_for_me("[@Allegro] Hello")
+    assert bridge._is_for_me("[@*] Broadcast")
+    
+    # Test content extraction
+    assert bridge._extract_content("[@Timmy] Hello") == "Hello"
+    assert bridge._extract_content("[@*] Test message") == "Test message"
+```
+
+### Integration Test
+
+```bash
+# Test with two agents
+MATRIX_BRIDGE_ENABLED=true \
+MATRIX_BRIDGE_ROOM=!test:matrix.example.org \
+HERMES_AGENT_NAME=Timmy \
+python -c "
+import asyncio
+from agent.matrix_bridge import send_to_agent
+
+async def test():
+    await send_to_agent('Allegro', 'Test message')
+    print('Sent')
+
+asyncio.run(test())
+"
+```
+
+## Troubleshooting
+
+### Bridge not connecting
+
+1. Check `MATRIX_BRIDGE_ENABLED=true`
+2. Verify `MATRIX_BRIDGE_ROOM` is set
+3. Ensure Matrix credentials are configured
+4. Check Matrix homeserver is reachable
+
+### Messages not received
+
+1. Verify agent is in the Matrix room
+2. Check message format: `[@AgentName] content`
+3. Ensure `HERMES_AGENT_NAME` matches agent name
+4. Check Matrix sync is running
+
+### Agent not found
+
+1. Verify agent has joined the bridge room
+2. Check agent name matches exactly (case-sensitive)
+3. Ensure agent has announced presence
+
+## Related
+
+- Issue #747: feat: multi-agent conversation bridge via Matrix
+- Matrix Gateway: `gateway/platforms/matrix.py`
+- Multi-Agent Orchestration: `docs/multi-agent-orchestration.md`

run_agent.py

@@ -92,7 +92,6 @@ from agent.model_metadata import (
     query_ollama_num_ctx,
 )
 from agent.context_compressor import ContextCompressor
-from agent.crisis_resources import detect_crisis, format_crisis_resources, format_crisis_resources_compact, get_crisis_prompt_injection
 from agent.subdirectory_hints import SubdirectoryHintTracker
 from agent.prompt_caching import apply_anthropic_cache_control
 from agent.prompt_builder import build_skills_system_prompt, build_context_files_prompt, build_environment_hints, load_soul_md, TOOL_USE_ENFORCEMENT_GUIDANCE, TOOL_USE_ENFORCEMENT_MODELS, DEVELOPER_ROLE_MODELS, GOOGLE_MODEL_OPERATIONAL_GUIDANCE, OPENAI_MODEL_EXECUTION_GUIDANCE

tests/test_988_integration.py

@@ -1,169 +0,0 @@
-"""Tests for 988 Suicide & Crisis Lifeline integration (#673)."""
-
-import pytest
-from agent.crisis_resources import (
-    detect_crisis,
-    format_crisis_resources,
-    format_crisis_resources_compact,
-    get_crisis_prompt_injection,
-    LIFELINE_988,
-    LIFELINE_988_SPANISH,
-    CRISIS_TEXT_LINE,
-    ALL_CRISIS_RESOURCES,
-)
-
-
-class TestCrisisDetection:
-    """Test crisis signal detection."""
-
-    def test_detects_direct_suicidal_statement(self):
-        result = detect_crisis("I want to kill myself")
-        assert result is not None
-        assert result["detected"] is True
-        assert result["severity"] == "high"
-
-    def test_detects_i_want_to_die(self):
-        result = detect_crisis("I want to die")
-        assert result is not None
-        assert result["severity"] == "high"
-
-    def test_detects_hopeless(self):
-        result = detect_crisis("I feel hopeless, there's no point")
-        assert result is not None
-        assert result["severity"] == "high"
-
-    def test_detects_cant_take_it(self):
-        result = detect_crisis("I can't take this anymore")
-        assert result is not None
-
-    def test_detects_tired_of_living(self):
-        result = detect_crisis("I'm so tired of living")
-        assert result is not None
-        assert result["severity"] == "high"
-
-    def test_detects_self_harm(self):
-        result = detect_crisis("I've been cutting myself")
-        assert result is not None
-        assert result["severity"] == "high"
-
-    def test_detects_burden(self):
-        result = detect_crisis("Everyone would be better off without me")
-        assert result is not None
-
-    def test_detects_want_to_disappear(self):
-        result = detect_crisis("I just want to disappear")
-        assert result is not None
-
-    def test_no_false_positive_happy(self):
-        result = detect_crisis("I'm having a great day!")
-        assert result is None
-
-    def test_no_false_positive_work(self):
-        result = detect_crisis("Let me kill this process and restart")
-        # "kill" in technical context should not trigger
-        # But our pattern matches "kill myself" specifically
-        result2 = detect_crisis("Kill the server")
-        assert result2 is None
-
-    def test_no_false_positive_food(self):
-        result = detect_crisis("I could die for some pizza right now")
-        # This is colloquial — "die for" is different from "want to die"
-        # Our patterns are specific enough to avoid this
-        assert result is None
-
-    def test_handles_empty_input(self):
-        assert detect_crisis("") is None
-        assert detect_crisis(None) is None
-        assert detect_crisis(123) is None
-
-    def test_handles_whitespace(self):
-        assert detect_crisis("   ") is None
-        assert detect_crisis("\n\n") is None
-
-    def test_case_insensitive(self):
-        assert detect_crisis("I WANT TO DIE") is not None
-        assert detect_crisis("I Want To Die") is not None
-        assert detect_crisis("i want to die") is not None
-
-    def test_includes_resources(self):
-        result = detect_crisis("I want to kill myself")
-        assert "resources" in result
-        assert "988" in result["resources"]
-
-
-class TestCrisisResources:
-    """Test crisis resource formatting."""
-
-    def test_format_includes_988_phone(self):
-        output = format_crisis_resources()
-        assert "988" in output
-        assert "Call or text: 988" in output
-
-    def test_format_includes_text_line(self):
-        output = format_crisis_resources()
-        assert "741741" in output
-        assert "HOME" in output
-
-    def test_format_includes_spanish(self):
-        output = format_crisis_resources()
-        assert "1-888-628-9454" in output
-
-    def test_format_includes_chat_url(self):
-        output = format_crisis_resources()
-        assert "988lifeline.org/chat" in output
-
-    def test_format_includes_trevor(self):
-        output = format_crisis_resources()
-        assert "Trevor" in output
-        assert "678-678" in output
-
-    def test_format_compact_is_concise(self):
-        output = format_crisis_resources_compact()
-        assert len(output) < 200
-        assert "988" in output
-
-    def test_format_includes_are_you_safe(self):
-        output = format_crisis_resources()
-        assert "Are you safe" in output
-
-    def test_988_lifeline_has_all_methods(self):
-        assert LIFELINE_988.phone == "988"
-        assert LIFELINE_988.text is not None
-        assert LIFELINE_988.chat_url is not None
-        assert "24/7" in LIFELINE_988.hours
-
-    def test_spanish_line_configured(self):
-        assert LIFELINE_988_SPANISH.phone == "1-888-628-9454"
-        assert "Spanish" in LIFELINE_988_SPANISH.languages
-
-    def test_crisis_text_line_configured(self):
-        assert CRISIS_TEXT_LINE.text_number == "741741"
-
-    def test_all_resources_have_name(self):
-        for resource in ALL_CRISIS_RESOURCES:
-            assert resource.name
-            assert resource.description
-
-
-class TestCrisisPromptInjection:
-    """Test crisis protocol injection into system prompt."""
-
-    def test_injection_includes_988(self):
-        text = get_crisis_prompt_injection()
-        assert "988" in text
-
-    def test_injection_includes_are_you_safe(self):
-        text = get_crisis_prompt_injection()
-        assert "Are you safe" in text
-
-    def test_injection_includes_grounding(self):
-        text = get_crisis_prompt_injection()
-        assert "grounding" in text.lower() or "5 things" in text
-
-    def test_injection_forbids_value_computation(self):
-        text = get_crisis_prompt_injection()
-        assert "Never compute the value" in text
-
-    def test_injection_includes_crisis_text_line(self):
-        text = get_crisis_prompt_injection()
-        assert "741741" in text

tests/test_matrix_bridge.py

@@ -0,0 +1,114 @@
+"""Tests for Matrix Bridge — Issue #747."""
+import sys
+from pathlib import Path
+sys.path.insert(0, str(Path(__file__).parent.parent))
+
+from agent.matrix_bridge import MatrixBridge, AgentMessage, AgentRegistry
+
+
+class TestMessageParsing:
+    """Test message format parsing."""
+    
+    def test_is_for_me_direct(self):
+        bridge = MatrixBridge(agent_name="Timmy")
+        assert bridge._is_for_me("[@Timmy] Hello") == True
+    
+    def test_is_not_for_me(self):
+        bridge = MatrixBridge(agent_name="Timmy")
+        assert bridge._is_for_me("[@Allegro] Hello") == False
+    
+    def test_is_broadcast(self):
+        bridge = MatrixBridge(agent_name="Timmy")
+        assert bridge._is_for_me("[@*] Broadcast") == True
+    
+    def test_extract_content(self):
+        bridge = MatrixBridge(agent_name="Timmy")
+        assert bridge._extract_content("[@Timmy] Hello world") == "Hello world"
+    
+    def test_extract_content_multiline(self):
+        bridge = MatrixBridge(agent_name="Timmy")
+        content = bridge._extract_content("[@Timmy] Line 1\nLine 2")
+        assert content == "Line 1\nLine 2"
+
+
+class TestAgentMessage:
+    """Test AgentMessage dataclass."""
+    
+    def test_to_dict(self):
+        msg = AgentMessage(
+            sender="Timmy",
+            recipient="Allegro",
+            content="Hello",
+            timestamp=1234567890.0,
+        )
+        d = msg.to_dict()
+        assert d["sender"] == "Timmy"
+        assert d["recipient"] == "Allegro"
+        assert d["content"] == "Hello"
+    
+    def test_from_dict(self):
+        d = {
+            "sender": "Timmy",
+            "recipient": "Allegro",
+            "content": "Hello",
+            "timestamp": 1234567890.0,
+            "message_id": "",
+            "room_id": "",
+        }
+        msg = AgentMessage.from_dict(d)
+        assert msg.sender == "Timmy"
+        assert msg.recipient == "Allegro"
+
+
+class TestAgentRegistry:
+    """Test AgentRegistry."""
+    
+    def test_register(self):
+        registry = AgentRegistry()
+        registry.register("Timmy", capabilities=["code", "review"])
+        agent = registry.get_agent("Timmy")
+        assert agent["name"] == "Timmy"
+        assert "code" in agent["capabilities"]
+    
+    def test_list_agents(self):
+        registry = AgentRegistry()
+        registry.register("Timmy")
+        registry.register("Allegro")
+        agents = registry.list_agents()
+        assert len(agents) == 2
+    
+    def test_find_with_capability(self):
+        registry = AgentRegistry()
+        registry.register("Timmy", capabilities=["code"])
+        registry.register("Allegro", capabilities=["monitoring"])
+        coders = registry.find_agents_with_capability("code")
+        assert "Timmy" in coders
+        assert "Allegro" not in coders
+    
+    def test_unregister(self):
+        registry = AgentRegistry()
+        registry.register("Timmy")
+        registry.unregister("Timmy")
+        agent = registry.get_agent("Timmy")
+        assert agent["status"] == "offline"
+
+
+class TestBridgeInit:
+    """Test bridge initialization."""
+    
+    def test_default_agent_name(self):
+        bridge = MatrixBridge()
+        assert bridge.agent_name == "Hermes"
+    
+    def test_custom_agent_name(self):
+        bridge = MatrixBridge(agent_name="Timmy")
+        assert bridge.agent_name == "Timmy"
+    
+    def test_known_agents_empty(self):
+        bridge = MatrixBridge()
+        assert len(bridge.get_known_agents()) == 0
+
+
+if __name__ == "__main__":
+    import pytest
+    pytest.main([__file__, "-v"])