johba/disinto
1
0
Fork 0
Code Issues 4 Pull requests Projects Releases Packages Wiki Activity Actions

fix: Inner CI/review wait loops bypass exit_marker fast-path (#338)

Browse source 
Add exit_marker file check to the CI wait loop and review wait loop in
phase-handler.sh, matching the pattern already used in monitor_phase_loop
(agent-session.sh). This makes crash detection consistent across all
polling paths.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
openhands 2026-03-21 00:55:38 +00:00
parent 6a499db011
commit e7be534c7d
1 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

8
dev/phase-handler.sh
View file

@ -249,8 +249,8 @@ Write PHASE:awaiting_review to the phase file, then stop and wait for review fee
sleep 30 sleep 30
CI_POLL_ELAPSED=$(( CI_POLL_ELAPSED + 30 )) CI_POLL_ELAPSED=$(( CI_POLL_ELAPSED + 30 ))
# Check session still alive during CI wait # Check session still alive during CI wait (exit_marker + tmux fallback)
if ! tmux has-session -t "${SESSION_NAME}" 2>/dev/null; then if [ -f "/tmp/claude-exited-${SESSION_NAME}.ts" ] || ! tmux has-session -t "${SESSION_NAME}" 2>/dev/null; then
log "session died during CI wait" log "session died during CI wait"
break break
fi fi
@ -392,8 +392,8 @@ Instructions:
sleep 300 # 5 min between review checks sleep 300 # 5 min between review checks
REVIEW_POLL_ELAPSED=$(( REVIEW_POLL_ELAPSED + 300 )) REVIEW_POLL_ELAPSED=$(( REVIEW_POLL_ELAPSED + 300 ))
# Check session still alive # Check session still alive (exit_marker + tmux fallback)
if ! tmux has-session -t "${SESSION_NAME}" 2>/dev/null; then if [ -f "/tmp/claude-exited-${SESSION_NAME}.ts" ] || ! tmux has-session -t "${SESSION_NAME}" 2>/dev/null; then
log "session died during review wait" log "session died during review wait"
REVIEW_FOUND=false REVIEW_FOUND=false
break break