Merge pull request 'fix: Inner CI/review wait loops bypass exit_marker fast-path (#338)' (#442) from fix/issue-338 into main

This commit is contained in:
johba 2026-03-21 02:18:59 +01:00
commit e8dc145184

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