From b769eaa1827d162112dc74d4b810b78de8714155 Mon Sep 17 00:00:00 2001 From: openhands Date: Mon, 23 Mar 2026 09:39:48 +0000 Subject: [PATCH] fix: monitor_phase_loop docstring lists 'break' as a possible _MONITOR_LOOP_EXIT value but it is never set (#435) Co-Authored-By: Claude Opus 4.6 (1M context) --- lib/agent-session.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/agent-session.sh b/lib/agent-session.sh index 32c4f76..5652838 100644 --- a/lib/agent-session.sh +++ b/lib/agent-session.sh @@ -296,7 +296,7 @@ inject_formula() { } # Monitor a phase file, calling a callback on changes and handling idle timeout. -# Sets _MONITOR_LOOP_EXIT to the exit reason (idle_timeout, idle_prompt, done, failed, break). +# Sets _MONITOR_LOOP_EXIT to the exit reason (idle_timeout, idle_prompt, done, crashed, PHASE:failed, PHASE:escalate). # Sets _MONITOR_SESSION to the resolved session name (arg 4 or $SESSION_NAME). # Callbacks should reference _MONITOR_SESSION instead of $SESSION_NAME directly. # Args: phase_file idle_timeout_secs callback_fn [session_name]