From d4a6b4a3b5dc14a3587b2670df39111fdfdd26f1 Mon Sep 17 00:00:00 2001
From: Gavriel Cohen <gabicohen22@yahoo.com>
Date: Sun, 5 Apr 2026 00:09:28 +0300
Subject: [PATCH] fix: portable stat and subshell variable mutation in cleanup
 script

- Replace macOS-only `stat -f%z` with portable `wc -c` for Linux compat
- Replace `find | while` pipes with process substitution so TOTAL_FREED
  counter survives the loop (pipe runs in subshell, losing mutations)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 scripts/cleanup-sessions.sh | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/scripts/cleanup-sessions.sh b/scripts/cleanup-sessions.sh
index cf03fe0..10e509c 100755
--- a/scripts/cleanup-sessions.sh
+++ b/scripts/cleanup-sessions.sh
@@ -34,7 +34,7 @@ remove() {
     if [ -d "$target" ]; then
       size=$(du -sk "$target" 2>/dev/null | cut -f1)
     else
-      size=$(stat -f%z "$target" 2>/dev/null || echo 0)
+      size=$(wc -c < "$target" 2>/dev/null || echo 0)
       size=$((size / 1024))
     fi
     TOTAL_FREED=$((TOTAL_FREED + size))
@@ -44,7 +44,7 @@ remove() {
       size=$(du -sk "$target" 2>/dev/null | cut -f1)
       rm -rf "$target"
     else
-      size=$(stat -f%z "$target" 2>/dev/null || echo 0)
+      size=$(wc -c < "$target" 2>/dev/null || echo 0)
       size=$((size / 1024))
       rm -f "$target"
     fi
@@ -95,11 +95,11 @@ done
 for group_dir in "$SESSIONS_DIR"/*/; do
   debug_dir="$group_dir/.claude/debug"
   [ -d "$debug_dir" ] || continue
-  find "$debug_dir" -type f -mtime +3 ! -name "latest" -print0 2>/dev/null | while IFS= read -r -d '' f; do
+  while IFS= read -r -d '' f; do
     fname=$(basename "$f" .txt)
     is_active "$fname" && continue
     remove "$f"
-  done
+  done < <(find "$debug_dir" -type f -mtime +3 ! -name "latest" -print0 2>/dev/null)
 done
 
 # --- Prune todo files (>3 days, skip files named after active sessions) ---
@@ -107,7 +107,7 @@ done
 for group_dir in "$SESSIONS_DIR"/*/; do
   todos_dir="$group_dir/.claude/todos"
   [ -d "$todos_dir" ] || continue
-  find "$todos_dir" -type f -mtime +3 -print0 2>/dev/null | while IFS= read -r -d '' f; do
+  while IFS= read -r -d '' f; do
     fname=$(basename "$f" .json)
     # Todo filenames are like {session_id}-agent-{session_id}.json
     for aid in $ACTIVE_IDS; do
@@ -116,7 +116,7 @@ for group_dir in "$SESSIONS_DIR"/*/; do
       fi
     done
     remove "$f"
-  done
+  done < <(find "$todos_dir" -type f -mtime +3 -print0 2>/dev/null)
 done
 
 # --- Prune telemetry (>7 days, skip files named after active sessions) ---
@@ -124,7 +124,7 @@ done
 for group_dir in "$SESSIONS_DIR"/*/; do
   telem_dir="$group_dir/.claude/telemetry"
   [ -d "$telem_dir" ] || continue
-  find "$telem_dir" -type f -mtime +7 -print0 2>/dev/null | while IFS= read -r -d '' f; do
+  while IFS= read -r -d '' f; do
     fname=$(basename "$f")
     for aid in $ACTIVE_IDS; do
       if [[ "$fname" == *"$aid"* ]]; then
@@ -132,14 +132,14 @@ for group_dir in "$SESSIONS_DIR"/*/; do
       fi
     done
     remove "$f"
-  done
+  done < <(find "$telem_dir" -type f -mtime +7 -print0 2>/dev/null)
 done
 
 # --- Prune group logs (>7 days) ---
 
-find "$GROUPS_DIR"/*/logs -type f -mtime +7 -print0 2>/dev/null | while IFS= read -r -d '' f; do
+while IFS= read -r -d '' f; do
   remove "$f"
-done
+done < <(find "$GROUPS_DIR"/*/logs -type f -mtime +7 -print0 2>/dev/null)
 
 # --- Summary ---