From 400e914f1f6c0b3c76c20ef1ff6bd97dd43af806 Mon Sep 17 00:00:00 2001
From: didericis <eric@dideric.is>
Date: Fri, 8 May 2026 02:06:17 -0400
Subject: [PATCH] fix(cli): keep SLUG accessible to cleanup_all EXIT trap

cmd_start declared SLUG as local, but cleanup_all (registered as the
EXIT/INT/TERM trap) calls pipelock_stop "$SLUG" after cmd_start has
returned and the local is out of scope. With set -u this aborted
shell teardown with "SLUG: unbound variable". Drop the local to
match the convention already used for MANIFEST_FILE, CONTAINER, and
STAGE_DIR.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
---
 cli.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cli.sh b/cli.sh
index 78c7611..3a35264 100755
--- a/cli.sh
+++ b/cli.sh
@@ -285,7 +285,7 @@ cmd_start() {
     exit 2
   fi
 
-  local SLUG
+  # Not declared local: needed by cleanup_all after cmd_start returns (see MANIFEST_FILE note below).
   SLUG="$(slugify "$NAME")"
 
   local IMAGE="${CLAUDE_BOTTLE_IMAGE:-claude-bottle:latest}"