TK-11856 rc-propagation (SHOPIFY): dw-seo-report refresh.sh exits nonzero on a failed weekly email

7a1b720aa875e5d7756e1fddefdc8347a2ffe862 · 2026-09-26 09:22:14 -0700 · Steve

Exit-code ONLY, no behavior change (Steve ruling 2026-09-26 Q1/Q2): every step still runs as
before; the wrapper now exits with the first nonzero rc of its side-effecting step(s) instead of
swallowing it, so cron-fire-canary's exit corroboration can see a failed run. No heartbeat file added.
Revert: git -C /Users/macstudio3/Projects/dw-seo-report revert <this sha>

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01YaiWjfgEuKQLTb5cdDreS3

Files touched

M refresh.sh

Diff

commit 7a1b720aa875e5d7756e1fddefdc8347a2ffe862
Author: Steve <steve@designerwallcoverings.com>
Date:   Sat Sep 26 09:22:14 2026 -0700

    TK-11856 rc-propagation (SHOPIFY): dw-seo-report refresh.sh exits nonzero on a failed weekly email
    
    Exit-code ONLY, no behavior change (Steve ruling 2026-09-26 Q1/Q2): every step still runs as
    before; the wrapper now exits with the first nonzero rc of its side-effecting step(s) instead of
    swallowing it, so cron-fire-canary's exit corroboration can see a failed run. No heartbeat file added.
    Revert: git -C /Users/macstudio3/Projects/dw-seo-report revert <this sha>
    
    Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
    Claude-Session: https://claude.ai/code/session_01YaiWjfgEuKQLTb5cdDreS3
---
 refresh.sh | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/refresh.sh b/refresh.sh
index fd00511..184048c 100755
--- a/refresh.sh
+++ b/refresh.sh
@@ -36,6 +36,10 @@ if [ "$1" = "--email" ]; then
   if python3 "$DIR/send_report.py" >> "$LOG" 2>&1; then
     echo "[$(ts)] emailed report to steve@dw via George" >> "$LOG"
   else
+    # TK-11856 rc-propagation (exit-code ONLY, no behavior change): the report is still rebuilt and
+    # this branch still logs exactly as before; we just exit NONZERO afterwards so a failed weekly
+    # send is visible as launchd's last-exit-code instead of a silent exit 0. Undo: git revert.
     echo "[$(ts)] EMAIL FAILED (George down?) — report still rebuilt" >> "$LOG"
+    exit 1
   fi
 fi

← e236ac6 Weekly George email of report (send_report.py + refresh.sh - · back to Dw Seo Report · (newest)