its-consulting/ralph-claude-code
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix grep -c fallback logic causing syntax error in double bracket test

Browse source 
The original `|| echo 0` fallback was executing even when grep -c succeeded
but returned 0 matches (exit code 1), resulting in "0\n0" being assigned
to variables. This multi-line string caused syntax errors in [[ ]] tests.

Fixed by removing the fallback and explicitly handling empty variables
instead, which properly addresses the root cause of the line 280 error.
This commit is contained in:
frankbria 2025-09-05 22:49:52 -07:00
parent 2ed13c3150
commit 0635b652b4
1 changed files with 6 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

8
ralph_loop.sh
View file

@ -272,8 +272,12 @@ should_exit_gracefully() {
# 4. Check fix_plan.md for completion
if [[ -f "@fix_plan.md" ]]; then
local total_items=$(($(grep -c "^- \[" "@fix_plan.md" 2>/dev/null || echo 0)))
local completed_items=$(($(grep -c "^- \[x\]" "@fix_plan.md" 2>/dev/null || echo 0)))
local total_items=$(grep -c "^- \[" "@fix_plan.md" 2>/dev/null)
local completed_items=$(grep -c "^- \[x\]" "@fix_plan.md" 2>/dev/null)
# Handle case where grep returns no matches (exit code 1)
[[ -z "$total_items" ]] && total_items=0
[[ -z "$completed_items" ]] && completed_items=0
log_status "INFO" "DEBUG: @fix_plan.md check - total_items:$total_items, completed_items:$completed_items" >&2