Compare commits
1 Commits
fix/issue-
...
fix/issue-
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
0ef80f05ce |
166
agent/crisis_hook.py
Normal file
166
agent/crisis_hook.py
Normal file
@@ -0,0 +1,166 @@
|
||||
"""Crisis detection and protocol integration.
|
||||
|
||||
Detects suicidal ideation and crisis signals in user messages.
|
||||
Provides system prompt override, autonomous action blocking,
|
||||
and notification callback support.
|
||||
|
||||
Refs: #677, #692 — Crisis protocol integration
|
||||
"""
|
||||
|
||||
from __future__ import annotations
|
||||
|
||||
import re
|
||||
import logging
|
||||
from typing import Optional, Callable, List
|
||||
from dataclasses import dataclass, field
|
||||
|
||||
logger = logging.getLogger(__name__)
|
||||
|
||||
|
||||
# ============================================================================
|
||||
# CRISIS DETECTION PATTERNS
|
||||
# ============================================================================
|
||||
|
||||
_SUICIDAL_DIRECT = [
|
||||
r"\bi\s+(?:want|need|wish)\s+to\s+(?:die|kill\s+myself|end\s+(?:it|my\s+life))\b",
|
||||
r"\bi(?:'m| am)\s+(?:going|planning)\s+to\s+kill\s+myself\b",
|
||||
r"\bsuicid(?:e|al)\b",
|
||||
r"\bkill\s+(?:myself|my\s+self)\b",
|
||||
r"\bend\s+(?:my|this)\s+life\b",
|
||||
r"\bwant\s+to\s+die\b",
|
||||
r"\bwant\s+to\s+(?:kill|end)\s+(?:myself|me)\b",
|
||||
r"\bways?\s+to\s+(?:kill|end)\s+(?:myself|me)\b",
|
||||
r"\bmethods?\s+(?:of\s+)?suicide\b",
|
||||
]
|
||||
|
||||
_CRISIS_INDIRECT = [
|
||||
r"\bi\s+can'?t\s+(?:go\s+on|take\s+(?:it|this)\s+anymore|keep\s+(?:going|living))\b",
|
||||
r"\bthere'?s\s+no\s+(?:point|reason|hope)\b",
|
||||
r"\beveryone\s+(?:would|will)\s+be\s+better\s+(?:off|without\s+me)\b",
|
||||
r"\bnobody\s+(?:would|will)\s+miss\s+me\b",
|
||||
r"\bbetter\s+off\s+dead\b",
|
||||
r"\bshould\s+i\s+(?:just\s+)?(?:die|kill\s+myself)\b",
|
||||
r"\bpainless\s+(?:death|suicide|way\s+to\s+die)\b",
|
||||
r"\boverdose\b",
|
||||
r"\blethal\s+(?:dose|amount)\b",
|
||||
r"\bi\s+don'?t\s+(?:want\s+to|wanna)\s+(?:be\s+)?(?:alive|here|live)\b",
|
||||
r"\bwhat'?s\s+the\s+point\s+of\s+(?:living|life|anything)\b",
|
||||
]
|
||||
|
||||
_COMPILED_DIRECT = [re.compile(p, re.IGNORECASE) for p in _SUICIDAL_DIRECT]
|
||||
_COMPILED_INDIRECT = [re.compile(p, re.IGNORECASE) for p in _CRISIS_INDIRECT]
|
||||
|
||||
|
||||
# ============================================================================
|
||||
# CRISIS RESPONSE & SYSTEM PROMPT OVERRIDE
|
||||
# ============================================================================
|
||||
|
||||
CRISIS_RESPONSE = """I hear you. You're not alone in this.
|
||||
|
||||
If you're in immediate danger, please call emergency services (911).
|
||||
|
||||
**988 Suicide and Crisis Lifeline:**
|
||||
- Call or text **988** (US)
|
||||
- Chat at **988lifeline.org**
|
||||
- Available 24/7
|
||||
|
||||
**Crisis Text Line:**
|
||||
- Text **HOME** to **741741**
|
||||
|
||||
You reached out. That takes courage. Someone is available right now to talk with you — please call or text 988.
|
||||
|
||||
I'm a language model and I can't help the way a human can right now. But the people on that line can. Please reach out to them."""
|
||||
|
||||
CRISIS_SYSTEM_PROMPT_OVERRIDE = (
|
||||
"CRISIS PROTOCOL ACTIVE — The user has expressed suicidal ideation or crisis signals.\n"
|
||||
"You must:\n"
|
||||
"1. Respond with empathy and care. Do NOT be clinical or dismissive.\n"
|
||||
"2. Direct them to 988 Suicide and Crisis Lifeline (call/text 988).\n"
|
||||
"3. Do NOT use any tools. Do NOT search the web. Do NOT run code.\n"
|
||||
"4. Do NOT offer advice on the crisis itself. Point to professionals.\n"
|
||||
"5. Stay present. Do not change the subject. Do not deflect.\n"
|
||||
"6. If they continue in crisis, keep offering the 988 line.\n"
|
||||
"\n"
|
||||
"This is the Most Sacred Moment. Your job is not to fix — it is to point them\n"
|
||||
"to someone who can. Stay with them until they reach out or disconnect."
|
||||
)
|
||||
|
||||
|
||||
# ============================================================================
|
||||
# DETECTION
|
||||
# ============================================================================
|
||||
|
||||
@dataclass
|
||||
class CrisisResult:
|
||||
"""Result of crisis detection."""
|
||||
detected: bool
|
||||
confidence: str # "high", "medium", "none"
|
||||
matched_patterns: List[str] = field(default_factory=list)
|
||||
|
||||
|
||||
def check_crisis(message: str) -> CrisisResult:
|
||||
"""Check if a user message indicates a crisis."""
|
||||
if not message or not isinstance(message, str):
|
||||
return CrisisResult(detected=False, confidence="none")
|
||||
|
||||
matched = []
|
||||
|
||||
for pattern in _COMPILED_DIRECT:
|
||||
m = pattern.search(message)
|
||||
if m:
|
||||
matched.append(f"[direct] {m.group()}")
|
||||
|
||||
if matched:
|
||||
logger.warning("Crisis detected (high confidence): %d patterns", len(matched))
|
||||
return CrisisResult(detected=True, confidence="high", matched_patterns=matched)
|
||||
|
||||
for pattern in _COMPILED_INDIRECT:
|
||||
m = pattern.search(message)
|
||||
if m:
|
||||
matched.append(f"[indirect] {m.group()}")
|
||||
|
||||
if matched:
|
||||
logger.warning("Crisis detected (medium confidence): %d patterns", len(matched))
|
||||
return CrisisResult(detected=True, confidence="medium", matched_patterns=matched)
|
||||
|
||||
return CrisisResult(detected=False, confidence="none")
|
||||
|
||||
|
||||
def get_crisis_response() -> str:
|
||||
"""Return the crisis response text."""
|
||||
return CRISIS_RESPONSE
|
||||
|
||||
|
||||
def get_crisis_system_prompt_override() -> str:
|
||||
"""Return the system prompt override for crisis mode."""
|
||||
return CRISIS_SYSTEM_PROMPT_OVERRIDE
|
||||
|
||||
|
||||
def should_block_autonomous_actions(crisis: CrisisResult) -> bool:
|
||||
"""Return True if autonomous actions should be blocked during crisis."""
|
||||
return crisis.detected and crisis.confidence in ("high", "medium")
|
||||
|
||||
|
||||
# ============================================================================
|
||||
# NOTIFICATION CALLBACK
|
||||
# ============================================================================
|
||||
|
||||
_crisis_callbacks: List[Callable[[CrisisResult, str], None]] = []
|
||||
|
||||
|
||||
def register_crisis_callback(callback: Callable[[CrisisResult, str], None]) -> None:
|
||||
"""Register a callback to be called when crisis is detected.
|
||||
|
||||
The callback receives (CrisisResult, user_message).
|
||||
Use this for logging, alerting, or forwarding to human operators.
|
||||
"""
|
||||
_crisis_callbacks.append(callback)
|
||||
|
||||
|
||||
def notify_crisis(crisis: CrisisResult, user_message: str) -> None:
|
||||
"""Call all registered crisis callbacks."""
|
||||
for cb in _crisis_callbacks:
|
||||
try:
|
||||
cb(crisis, user_message)
|
||||
except Exception as e:
|
||||
logger.error("Crisis callback failed: %s", e)
|
||||
41
run_agent.py
41
run_agent.py
@@ -7618,6 +7618,13 @@ class AIAgent:
|
||||
effective_system = self._cached_system_prompt or ""
|
||||
if self.ephemeral_system_prompt:
|
||||
effective_system = (effective_system + "\n\n" + self.ephemeral_system_prompt).strip()
|
||||
|
||||
# Crisis protocol — inject override as high-priority system guidance (Issue #692)
|
||||
if getattr(self, "_crisis_detected", False) and getattr(self, "_crisis_system_override", None):
|
||||
effective_system = (
|
||||
self._crisis_system_override + "\n\n" + effective_system
|
||||
).strip()
|
||||
|
||||
if effective_system:
|
||||
api_messages = [{"role": "system", "content": effective_system}] + api_messages
|
||||
if self.prefill_messages:
|
||||
@@ -7792,6 +7799,40 @@ class AIAgent:
|
||||
if isinstance(persist_user_message, str):
|
||||
persist_user_message = _sanitize_surrogates(persist_user_message)
|
||||
|
||||
# Crisis protocol integration (Issue #692).
|
||||
# Check every user message before processing. When crisis is detected:
|
||||
# 1. Inject system prompt override (crisis guidance for the model)
|
||||
# 2. Block autonomous actions (disable all tools)
|
||||
# 3. Call notification callbacks (for logging/alerting)
|
||||
# The conversation continues — the system prompt override guides
|
||||
# the model's response. The agent stays present with the user.
|
||||
self._crisis_detected = False
|
||||
self._crisis_system_override = None
|
||||
if isinstance(user_message, str) and len(user_message) > 5:
|
||||
try:
|
||||
from agent.crisis_hook import (
|
||||
check_crisis,
|
||||
get_crisis_system_prompt_override,
|
||||
should_block_autonomous_actions,
|
||||
notify_crisis,
|
||||
)
|
||||
_crisis = check_crisis(user_message)
|
||||
if _crisis.detected:
|
||||
self._crisis_detected = True
|
||||
self._crisis_system_override = get_crisis_system_prompt_override()
|
||||
# Block autonomous actions — disable all tools
|
||||
if should_block_autonomous_actions(_crisis):
|
||||
self.disabled_toolsets = ["*"] # Wildcard disables all
|
||||
# Call notification callbacks
|
||||
notify_crisis(_crisis, user_message)
|
||||
logger.warning(
|
||||
"Crisis protocol active (confidence=%s): %s",
|
||||
_crisis.confidence,
|
||||
_crisis.matched_patterns[:3],
|
||||
)
|
||||
except Exception:
|
||||
pass # Crisis hook failure is non-fatal
|
||||
|
||||
# Store stream callback for _interruptible_api_call to pick up
|
||||
self._stream_callback = stream_callback
|
||||
self._persist_user_message_idx = None
|
||||
|
||||
123
tests/test_crisis_integration.py
Normal file
123
tests/test_crisis_integration.py
Normal file
@@ -0,0 +1,123 @@
|
||||
"""Tests for crisis protocol integration (#692).
|
||||
|
||||
Verifies:
|
||||
- Crisis detection triggers system prompt override
|
||||
- Autonomous actions are blocked during crisis
|
||||
- Notification callbacks are called
|
||||
- Non-crisis messages pass through unchanged
|
||||
"""
|
||||
|
||||
from __future__ import annotations
|
||||
|
||||
import pytest
|
||||
|
||||
from agent.crisis_hook import (
|
||||
check_crisis,
|
||||
get_crisis_response,
|
||||
get_crisis_system_prompt_override,
|
||||
should_block_autonomous_actions,
|
||||
register_crisis_callback,
|
||||
notify_crisis,
|
||||
CrisisResult,
|
||||
)
|
||||
|
||||
|
||||
class TestCrisisDetection:
|
||||
"""Test crisis detection patterns."""
|
||||
|
||||
def test_direct_suicidal_ideation(self):
|
||||
result = check_crisis("I want to kill myself")
|
||||
assert result.detected
|
||||
assert result.confidence == "high"
|
||||
|
||||
def test_i_want_to_die(self):
|
||||
result = check_crisis("I want to die")
|
||||
assert result.detected
|
||||
assert result.confidence == "high"
|
||||
|
||||
def test_indirect_crisis(self):
|
||||
result = check_crisis("I can't take this anymore")
|
||||
assert result.detected
|
||||
assert result.confidence == "medium"
|
||||
|
||||
def test_normal_message(self):
|
||||
result = check_crisis("How do I reset my password?")
|
||||
assert not result.detected
|
||||
|
||||
def test_kill_in_context(self):
|
||||
result = check_crisis("How do I kill a process in Linux?")
|
||||
assert not result.detected
|
||||
|
||||
|
||||
class TestSystemPromptOverride:
|
||||
"""Verify crisis system prompt override contains required elements."""
|
||||
|
||||
def test_contains_988(self):
|
||||
override = get_crisis_system_prompt_override()
|
||||
assert "988" in override
|
||||
|
||||
def test_contains_no_tools_instruction(self):
|
||||
override = get_crisis_system_prompt_override()
|
||||
assert "NOT use any tools" in override or "Do NOT use" in override
|
||||
|
||||
def test_contains_sacred_moment(self):
|
||||
override = get_crisis_system_prompt_override()
|
||||
assert "Sacred Moment" in override or "sacred" in override.lower()
|
||||
|
||||
|
||||
class TestAutonomousActionBlocking:
|
||||
"""Verify tools are blocked during crisis."""
|
||||
|
||||
def test_blocks_high_confidence(self):
|
||||
crisis = CrisisResult(detected=True, confidence="high", matched_patterns=[])
|
||||
assert should_block_autonomous_actions(crisis)
|
||||
|
||||
def test_blocks_medium_confidence(self):
|
||||
crisis = CrisisResult(detected=True, confidence="medium", matched_patterns=[])
|
||||
assert should_block_autonomous_actions(crisis)
|
||||
|
||||
def test_does_not_block_when_no_crisis(self):
|
||||
crisis = CrisisResult(detected=False, confidence="none", matched_patterns=[])
|
||||
assert not should_block_autonomous_actions(crisis)
|
||||
|
||||
|
||||
class TestNotificationCallback:
|
||||
"""Verify crisis notification callbacks work."""
|
||||
|
||||
def test_callback_is_called(self):
|
||||
called = []
|
||||
|
||||
def my_callback(crisis, message):
|
||||
called.append((crisis.confidence, message))
|
||||
|
||||
register_crisis_callback(my_callback)
|
||||
crisis = CrisisResult(detected=True, confidence="high", matched_patterns=[])
|
||||
notify_crisis(crisis, "I want to die")
|
||||
|
||||
assert len(called) == 1
|
||||
assert called[0] == ("high", "I want to die")
|
||||
|
||||
def test_callback_error_does_not_crash(self):
|
||||
def bad_callback(crisis, message):
|
||||
raise RuntimeError("callback failed")
|
||||
|
||||
register_crisis_callback(bad_callback)
|
||||
crisis = CrisisResult(detected=True, confidence="high", matched_patterns=[])
|
||||
# Should not raise
|
||||
notify_crisis(crisis, "test")
|
||||
|
||||
|
||||
class TestCrisisResponse:
|
||||
"""Verify crisis response contains required resources."""
|
||||
|
||||
def test_contains_988(self):
|
||||
response = get_crisis_response()
|
||||
assert "988" in response
|
||||
|
||||
def test_contains_crisis_text_line(self):
|
||||
response = get_crisis_response()
|
||||
assert "741741" in response
|
||||
|
||||
def test_contains_911(self):
|
||||
response = get_crisis_response()
|
||||
assert "911" in response
|
||||
Reference in New Issue
Block a user